Add rh-wallet skill (Robinhood Crypto via RH Wallet Gateway)#554
Add rh-wallet skill (Robinhood Crypto via RH Wallet Gateway)#554anondevv69 wants to merge 3 commits into
Conversation
Stateless Robinhood Crypto trading (BTC-USD pairs) with keys in Bankr env, hosted gateway, wallet routing vs hoodmarkets/onchain, and X privacy rules. Co-authored-by: Cursor <cursoragent@cursor.com>
Sync SKILL.md, x402.md, and gateway refs from RH-Wallet. Document paid x402 endpoints ($0.25–$0.50 USDC) and natural-language agent routing. Co-authored-by: Cursor <cursoragent@cursor.com>
…h-wallet skill Co-authored-by: Cursor <cursoragent@cursor.com>
saltoriousSIG
left a comment
There was a problem hiding this comment.
Looks good! just a couple things to address
-
The skill sends the long-lived Robinhood API key and Ed25519 private signing key to a third-party Railway gateway, and x402 additionally exposes them to x402.bankr.bot. “Stateless” does not remove credential-theft or logging risk. Require a locally run, source/version-pinned signer so private keys never leave the user-controlled environment. At minimum, document all processors, redaction guarantees, key scope, rotation/revocation, and dedicated-key requirements.
-
The x402 commands use --yes without validating or previewing payment terms. Pin the Base chain ID, USDC contract, facilitator/payee, resource path, and maximum atomic price for every endpoint. Require payment confirmation, spend caps, idempotency, and no blind retries.
-
Trade confirmation is incomplete and inconsistent. Before every crypto, equity, option, or cancellation write, preview the exact account/product, canonical instrument, side, order type, quantity/notional, quote, fees, time-in-force, maximum drift/loss, and x402 fee. Require final confirmation in every channel, expiring it when any field changes.
-
Successful gateway/MCP responses are not followed by authoritative order verification. Poll Robinhood using the exact order ID, validate the order against confirmed intent, and report fills only from final status with actual price, quantity, and fees. After ambiguous failures, look up the order before any retry.
-
The Robinhood Agentic MCP lacks OAuth scope and permission boundaries. Require explicit installation approval, read-only scopes by default, separate approval for trading tools, strict host/redirect allowlisting, token revocation guidance, and confirmation for every placement or cancellation. Treat MCP output as untrusted.
-
RH_WALLET_API_URL can be overridden despite credentials being attached to every request, while setup runs unpinned pynacl and a key-generation script. Enforce exact HTTPS host allowlisting and pin the dependency and script to a reviewed commit with integrity verification.
-
Add trading-risk disclosures covering market-order slippage, volatility, total loss, liquidity, options expiry/assignment, and gateway compromise. Third-party prices, estimates, errors, and signals must never trigger trades automatically.
Summary
Adds the rh-wallet skill for RH Wallet — stateless Robinhood Crypto (US) trading via a hosted signing gateway.
BTC-USDRH_API_KEY+RH_PRIVATE_KEY_BASE64) — gateway signs only, does not store keys0x…), and Agentic MCP (stocks)account_numberGateway: https://rh-wallet-production.up.railway.app
Source repo: https://github.com/anondevv69/RH-Wallet
Install (after merge)
User env (Bankr → Agent tool environment)
RH_API_KEYRH_PRIVATE_KEY_BASE64RH_GATEWAY_SECRETRH_WALLET_API_URLTest plan
catalog.jsonvalidates (slug= folder namerh-wallet)0x…) → skill refuses, routes to onchain/hoodmarkets perWALLET-ROUTING.mdSecurity notes (aligned with github-vesting review themes)
known-gateway.json+API-HOST.mdallowlist/wallet/submit— curl-only to gateway; RH keys in user envreplyTextrelayMade with Cursor