Skip to content

Conversation

@topocount
Copy link

@topocount topocount commented Oct 10, 2025

update example app to utilize updated shuttle libs

Why is this change needed?

This fell out of date and is a helpful reference for updating libdeps downstream.

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review


PR-Codex overview

This PR updates the example application for the @farcaster/shuttle library to use the latest versions of dependencies and improves its functionality by modifying imports and code structure.

Detailed summary

  • Updated @farcaster/shuttle dependency version to ^0.9.3.
  • Added start script in package.json.
  • Removed unused imports and added necessary imports in app.ts.
  • Replaced HubInfoRequest with GetInfoRequest in a specific function.
  • Adjusted database insertion logic to use unknown type casting.
  • Improved formatting and code organization for better readability.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

update example app to utilize updated shuttle libs
Copilot AI review requested due to automatic review settings October 10, 2025 02:54
@changeset-bot
Copy link

changeset-bot bot commented Oct 10, 2025

🦋 Changeset detected

Latest commit: 70ef8b4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@farcaster/shuttle Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the example app to use current shuttle library versions and API changes. The PR modernizes the example application to align with updated shuttle dependencies and fixes compatibility issues.

  • Updated shuttle library version from 0.5.0 to 0.9.3
  • Refactored imports and constructor parameters to match updated API signatures
  • Fixed type casting and property name changes for compatibility

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/shuttle/src/example-app/package.json Updated shuttle dependency version and reorganized package.json structure
packages/shuttle/src/example-app/app.ts Refactored imports, updated constructor calls, and fixed API compatibility issues
.changeset/twelve-mangos-provide.md Added changeset documentation for the update

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@socket-security
Copy link

socket-security bot commented Oct 10, 2025

@socket-security
Copy link

socket-security bot commented Oct 10, 2025

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
npm/[email protected] has Obfuscated code.

Confidence: 0.94

Location: Package overview

From: packages/hub-nodejs/examples/contract-signatures/yarn.locknpm/[email protected]

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at [email protected].

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/[email protected]. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant