Skip to content

Integrate Robinhood Chain - #6150

Open
j0ntz wants to merge 1 commit into
developfrom
jon/robinhood-chain
Open

Integrate Robinhood Chain#6150
j0ntz wants to merge 1 commit into
developfrom
jon/robinhood-chain

Conversation

@j0ntz

@j0ntz j0ntz commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Technical Design Document

robinhood-chain.md

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

EdgeApp/edge-currency-accountbased#1087 (the robinhood plugin itself)

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

Description

Registers the robinhood currency plugin so users can create a Robinhood Chain wallet. Robinhood Chain is an Arbitrum L2 (EVM chain 4663) that uses ETH for gas.

Asana: https://app.asana.com/1/9976422036640/project/1213843652804305/task/1217382370104887

Five files, matching the opbnb and monad registrations: corePlugins.ts, envConfig.ts, WalletAndCurrencyConstants.ts, and the two locale files.

WalletAndCurrencyConstants.ts carries two list entries, not one. Beyond WALLET_TYPE_ORDER and the SPECIAL_CURRENCY_INFO block, robinhood also goes in CURRENCY_SETTINGS_KEYS, which WalletSettingsModal reads to decide whether to show the Asset Settings row and which AssetSettingsScene maps to build its list. Every peer EVM L2 is in it. Bugbot caught the omission; the rest of the GUI's hardcoded pluginId lists were checked and correctly exclude this chain, since they are ramp and payment-protocol asset maps for providers that do not support it.

showChainIcon: true is the one entry worth a look. The native asset is ETH, so without the chain badge a Robinhood Chain wallet is indistinguishable from an Ethereum mainnet wallet in the wallet list. arbitrum and base set it for the same reason.

The wallet name string is "My Robinhood Chain" rather than "My Robinhood", since the shorter form reads as a brokerage account.

Testing. Full jest suite, snapshots and eslint pass. Driven on the iOS simulator against EdgeApp/edge-currency-accountbased#1087 linked in: searching "Robinhood" in Choose Wallets to Add lists the chain, creating gives "My Robinhood Chain" on a wallet scene titled "Robinhood Chain Network", and importing a seed with real on-chain history renders its live balance. After the CURRENCY_SETTINGS_KEYS fix, re-driven to confirm the wallet's Asset Settings row renders and "Robinhood Chain" appears in the Asset Settings list between Ravencoin and Rootstock. Screenshots are attached in comments below.

One search behavior to know when testing: filterWalletCreateItemListBySearchText requires every whitespace-separated term to prefix-match a field, so typing the full "Robinhood Chain" returns nothing while "Robinhood" matches.

Follow-ups, neither blocking:

  • ROBINHOOD_INIT needs adding to the deployed env.json, or asCorePluginInit returns false and the plugin ships disabled.
  • Currency icons at content.edge.app/currencyIconsV3/robinhood/ (robinhood.png, chain_robinhood.png) do not exist yet, so the wallet renders a text placeholder in the app.

@j0ntz

j0ntz commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

📸 Test evidence

robinhood in create wallet

robinhood in create wallet

wallet in list

wallet in list

wallet detail

wallet detail

live balance from chain

live balance from chain

Captured by the agent's in-app test run (build-and-test).

@j0ntz
j0ntz marked this pull request as ready for review August 13, 2026 00:01
@j0ntz
j0ntz force-pushed the jon/robinhood-chain branch from d7f2c1a to f77e4a7 Compare August 13, 2026 00:01

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.

Tip: disable this comment in your organization's Code Review settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit f77e4a7. Configure here.

Comment thread src/constants/WalletAndCurrencyConstants.ts
Register the robinhood currency plugin so users can create a Robinhood
Chain wallet. The chain badge is shown on the wallet icon because the
native asset is ETH, which would otherwise be indistinguishable from
Ethereum mainnet.
@j0ntz
j0ntz force-pushed the jon/robinhood-chain branch from f77e4a7 to 030b1f7 Compare August 13, 2026 00:10
@j0ntz

j0ntz commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

📸 Test evidence (after fix)

asset settings

asset settings

hood shot 20 wallet settings modal

hood shot 20 wallet settings modal

Captured by the agent's in-app test run (build-and-test).

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.

1 participant