Register Robinhood Chain in wagmi to fix switch-chain errors - #2885
Conversation
Wallet confirmations on rain.strategies Robinhood orders failed with "Chain not configured" because 4663 was missing from the wagmi/AppKit network list. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: rainlanguage/raindex/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (5)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change adds Robinhood Chain with ID 4663 to webapp chain configuration, wagmi, and AppKit registration. It also adds a Blockscout explorer fallback in ChangesRobinhood Chain support
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The change registers Robinhood Chain and its explorer links with focused configuration coverage. No actionable production risk is identified, so the PR is mergeable pending normal checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
createConfig's chains tuple is readonly, so a direct cast to a mutable array failed the webapp typecheck. Co-authored-by: Cursor <cursoragent@cursor.com>
|
@coderabbitai assess this PR size classification for the totality of the PR with the following criterias and report it in your comment: S/M/L PR Classification Guidelines:This guide helps classify merged pull requests by effort and complexity rather than just line count. The goal is to assess the difficulty and scope of changes after they have been completed. Small (S)Characteristics:
Review Effort: Would have taken 5-10 minutes Examples:
Medium (M)Characteristics:
Review Effort: Would have taken 15-30 minutes Examples:
Large (L)Characteristics:
Review Effort: Would have taken 45+ minutes Examples:
Additional Factors to ConsiderWhen deciding between sizes, also consider:
Notes:
|
Summary
Chain not configuredbecause chain ID 4663 is in rain.strategies settings but was missing from the webapp's wagmi/AppKit network list.viem'sdefineChain(RPC, ETH native currency, Blockscout explorer) and register it insupportedChainsList.Test plan
supportedChainsListincludes Robinhood (4663) with RPC/explorer metadatadefaultConfigpasses 4663 into bothcreateConfigandcreateAppKitgetExplorerLinkreturnshttps://robinhoodchain.blockscout.com/tx/...for chain 4663Made with Cursor
Summary by CodeRabbit