feat: enable Robinhood chain in production - #12538
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 25 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. 📝 WalkthroughWalkthroughThe Robinhood feature flag is enabled in ChangesRobinhood feature flag
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.env (1)
105-105: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winKeep the feature flag declaration in dotenv-linter order.
dotenv-linterreports thatVITE_FEATURE_ROBINHOODshould precedeVITE_FEATURE_RUNEPOOL. Move this declaration beforeVITE_FEATURE_RUNEPOOLto keep the environment file lint-clean.Static analysis reports this key-order violation.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.env at line 105, Move the VITE_FEATURE_ROBINHOOD declaration before VITE_FEATURE_RUNEPOOL in the environment file, preserving both keys and their values.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.env:
- Line 105: Move the VITE_FEATURE_ROBINHOOD declaration before
VITE_FEATURE_RUNEPOOL in the environment file, preserving both keys and their
values.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 1f2ed1bc-92ec-4c28-bff8-7fd2bf5d12a2
📒 Files selected for processing (2)
.env.env.development
💤 Files with no reviewable changes (1)
- .env.development
Description
Turns on Robinhood Chain (EVM 4663) in production by flipping
VITE_FEATURE_ROBINHOODtotruein.envand dropping the now-redundant.env.developmentoverride.All the underlying support landed in #12497 (chain adapter, CAIP constants, base asset, hdwallet coinbase/ledger/keepkey/trezor/native/phantom/vultisig/gridplus/walletconnect, CSP headers, RPC/viem clients, asset generation, and swapper support in Relay/Across/deBridge/ButterSwap/Bebop/0x) — this PR only lifts the flag.
Also moves the flag out of the
# Dev toolsblock in.envand up with the other chain flags, where it belongs.Issue (if applicable)
closes #
Risk
Low. The change is a single feature-flag flip; no code paths change.
VITE_FEATURE_ROBINHOODstill defaults tofalseinsrc/config.ts, and every consumer (src/constants/chains.ts,PluginProvider) already gates on it, so the blast radius is limited to Robinhood being present in chain/asset lists and swapper routes.Robinhood Chain sends/receives (native ETH and ERC20s), and swaps routing through Robinhood on the swappers that already list it: Relay, Across, deBridge, ButterSwap, Bebop and 0x. Wallets supporting EVM 4663: Native, Ledger, KeepKey, Trezor, Coinbase, Phantom, Vultisig, GridPlus, WalletConnect v2, MetaMask multichain.
Testing
Engineering
developalready runs with the flag on locally (it wastruein.env.development), so this is effectively promoting the dev behavior to all environments.VITE_ROBINHOOD_NODE_URL(https://rpc.mainnet.chain.robinhood.com) resolves in the preview env.Operations
Functional testing in the preview env:
robinhoodchain.blockscout.com.Screenshots (if applicable)
Summary by CodeRabbit