-
Notifications
You must be signed in to change notification settings - Fork 1.5k
chord: upgrade dependencies #5100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🦋 Changeset detectedLatest commit: a8e7617 The changes in this PR will be included in the next version bump. This PR includes changesets to release 25 packages
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 |
The latest updates on your projects. Learn more about Vercel for GitHub.
10 Skipped Deployments
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR upgrades various key dependencies across the project, including Viem, Wagmi, WalletConnect, Solana packages, and other third-party dependencies. The changes maintain backward compatibility while bringing the project up to date with the latest versions.
- Upgrades Viem from 2.37.2 to 2.37.9 and Wagmi from 2.16.9 to 2.17.5
- Updates WalletConnect packages from 2.21.7 to 2.21.9
- Upgrades Solana packages and various React/TypeScript dependencies
Reviewed Changes
Copilot reviewed 67 out of 68 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
packages/*/package.json | Version bumps for core dependencies across all packages |
examples/*/package.json | Updated dependencies in example applications |
apps/*/package.json | Dependency updates in demo and testing applications |
packages/adapters/wagmi/src/connectors/* | Type safety improvements for Wagmi connector methods |
packages/codemod/src/utils.js | Added newline to JSON output for consistency |
Comments suppressed due to low confidence (1)
packages/adapters/wagmi/src/client.ts:1
- [nitpick] The variable name 'isSocialsEnabled' should be 'areSocialsEnabled' for better grammatical correctness, or 'socialsEnabled' to match the pattern of 'isEmailEnabled'.
import {
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
81515c3
to
a8e7617
Compare
Description
Type of change
Associated Issues
For Linear issues: Closes APKT-xxx
For GH issues: closes #...
Showcase (Optional)
If there is a UI change include the screenshots with before and after state.
If new feature is being introduced, include the link to demo recording.
Checklist
Note
Upgrades key web3 dependencies across packages/apps and updates wagmi adapter code and examples to match new APIs.
apps/*
,examples/*
, andpackages/*
forwagmi
,viem
,@walletconnect/*
, Solana libs, and other third-party deps.packages/**/package.json
(e.g.,adapters/*
,appkit
,ui
,controllers
,siwe
,siwx
,pay
,universal-connector
,wallet-button
).packages/adapters/wagmi/src/client.ts
,connectors/AuthConnector.ts
, andconnectors/WalletConnectConnector.ts
to match upgraded deps.examples/**/package.json
and adjust code where needed (e.g.,examples/html-wagmi-cdn/src/main.js
) for new versions.apps/laboratory/src/components/Theming/RadioColor.tsx
.packages/codemod/src/utils.js
to support changes.Written by Cursor Bugbot for commit a8e7617. This will update automatically on new commits. Configure here.