feat: wagmi v3 compatibility#2591
Conversation
🦋 Changeset detectedLatest commit: 2dffd25 The changes in this PR will be included in the next version bump. This PR includes changesets to release 20 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.
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
61fe9b7 to
b949d7b
Compare
727a0a0 to
4b9452c
Compare
4b9452c to
960cc8b
Compare
de6c9e2 to
94ff4b8
Compare
|
Would be nice to move this forward, thanks. |
|
Yes, this needs to happen. |
Fixes for wagmi v3 SSG Build ErrorsI've fixed the CI build failures caused by Root Causewagmi v3 requires all hooks (like Changes Made1. All Next.js examples with Pages Router:
2. Next.js App Router examples:
3. Documentation site (site/):
4. create-rainbowkit templates:
Verification
Feel free to cherry-pick these changes or pull from my fork! |
|
Our team is trying to progress with the wagmi update but our use of rainbowkit is preventing this update. Please advise when you expect this PR to land. Or if you think we should look for other options. |
Co-authored-by: phelix001 <phelix001@users.noreply.github.com>
960cc8b to
11e648b
Compare
|
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.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default mode and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 11e648b. Configure here.
| })); | ||
|
|
||
| vi.mock('wagmi', () => ({ | ||
| useAccount: wagmiMocks.useAccount, |
There was a problem hiding this comment.
Dead useAccountEffect mock leftover from wagmi v2
Low Severity
The test mock still declares and resets useAccountEffect (the wagmi v2 hook) even though all production code now uses useConnectionEffect. The mock exports useAccountEffect from the fake wagmi module and resets it in beforeEach, but nothing under test imports or calls it. This dead mock adds confusion about which hooks are actually in use.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 11e648b. Configure here.
| const { address } = useConnection(); | ||
|
|
||
| const { chainId } = useAccount(); | ||
| const { chainId } = useConnection(); |
There was a problem hiding this comment.
Redundant duplicate useConnection() hook calls
Low Severity
useConnection() is called twice in succession to destructure address and chainId separately. These could be combined into a single call like const { address, chainId } = useConnection(), eliminating the redundant hook invocation. While both calls return the same data, this pattern is confusing and creates unnecessary overhead.
Reviewed by Cursor Bugbot for commit 11e648b. Configure here.
11e648b to
827b28f
Compare
827b28f to
bc86912
Compare
bc86912 to
12584d8
Compare
12584d8 to
2dffd25
Compare
|
Let's get this to the finish line, it's time. I can't wait to move off of my fork. |
|
Seems like this could be finally ready to ship. What's the status @DanielSinclair ? |
|
🥺 |
|
please bring this back to life |
|
Is there a way to help this move forward ? Thank you @DanielSinclair |
|
Would be nice to see it merged |



Summary
@rainbow-me/rainbowkit-siwe-next-authnow peers on RainbowKit 3.x directly.Breaking changes
@rainbow-me/rainbowkit-siwe-next-authshould pair it with RainbowKit 3.x.Review structure
chore: align wagmi v3 dependency metadatafix: migrate RainbowKit internals to wagmi v3fix: dedupe wagmi in Next buildsexample: update wagmi v3 examples and templatesdocs: document RainbowKit wagmi v3 migrationValidation
git diff --check d9bf82e6d..HEADpnpm buildpnpm lintpnpm testCI=true pnpm test:cliNotes: local build/lint output still includes existing third-party or warning-level noise from WalletConnect/ox, Next page-data size, and Biome warnings, but all commands above exited successfully.