Skip to content

Upgrade Dynamic libraries to 4.96.3 in demo - #1117

Open
ted-palmer wants to merge 1 commit into
mainfrom
ted/int2-1294-relaykit-upgrade-dynamic-libraries
Open

Upgrade Dynamic libraries to 4.96.3 in demo#1117
ted-palmer wants to merge 1 commit into
mainfrom
ted/int2-1294-relaykit-upgrade-dynamic-libraries

Conversation

@ted-palmer

Copy link
Copy Markdown
Collaborator

No description provided.

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
relay-sdk-demo Ready Ready Preview Aug 31, 2026 7:59pm

Request Review

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Claude finished @ted-palmer's task in 22s —— View job


Code Review

  • Gather context (diff, commits, changed files)
  • Review dependency changes
  • Post review

Code Review — No issues found.

Reviewed 2 files (demo/package.json, pnpm-lock.yaml). This is a demo-only dependency bump. All ten @dynamic-labs/* runtime dependencies plus the @dynamic-labs/types devDependency are consistently upgraded from 4.87.24.96.3, with no version skew. The @dynamic-labs-connectors/abstract-global-wallet-evm package correctly stays on its own 4.6.7 line (connectors are versioned independently). The lockfile changes are the expected transitive fallout of the bump. No published packages (sdk, ui, hooks, adapters) are affected, so no changeset is required.

One thing worth a quick sanity check before merge: confirm the demo still builds/runs against 4.96.3, since a ~9 minor-version jump can carry breaking changes in the Dynamic SDK's React APIs even within the same major.

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Claude finished @ted-palmer's task in 24s —— View job


📝 Docs Drift Check — No docs impact

This PR does not touch surfaces that require a relayprotocol/relay-docs update.

@greptile-apps

greptile-apps Bot commented Aug 31, 2026

Copy link
Copy Markdown

Greptile Summary

This change upgrades the demo’s Dynamic Labs packages and lockfile resolution to version 4.96.3. The retained Abstract Global Wallet EVM connector was checked against the upgraded peer graph: it resolves to the new Dynamic packages, imports successfully, constructs successfully, and completes initialization. The potential connector incompatibility was disproved by this exercised integration path, and no issues were found.

Confidence Score: 5/5

Safe to merge based on the exercised dependency integration path and absence of remaining findings.

The changed dependency graph was validated in clean pre-change and updated installations. The retained connector successfully ran against the upgraded Dynamic peer packages, and no independently actionable defects remain.

Files Needing Attention: No files need further changes.

T-Rex T-Rex Logs

What T-Rex did

  • Ran an executable connector-relative peer-resolution and ESM integration harness in clean frozen-lockfile installations for both the previous revision and the new revision.
  • Observed that the connector resolved Dynamic peers to 4.87.2 before the upgrade and to 4.96.3 after the upgrade, with initialization completing successfully in both cases.
  • Validated the integration path with the dedicated script, which confirmed that the compatibility path succeeds; the before- and after-run logs show the expected peer resolutions and successful initialization.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "Upgrade Dynamic libraries to 4.96.3 in d..." | Re-trigger Greptile

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.

2 participants