Add Robinhood Trading MCP configuration - #149
Merged
Merged
Conversation
… inherits upstream MCP policy Co-authored-by: sanylax01 <sanylax01@gmail.com>
sanylax2
approved these changes
Sep 14, 2026
sanylax2
marked this pull request as ready for review
September 14, 2026 15:15
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Wires up the Robinhood Trading MCP (
https://agent.robinhood.com/mcp/trading, Streamable HTTP):.cursor/mcp.json— defines therobinhood-tradingHTTP MCP server. This is the config location Cursor reads for the IDE, Agent window, and CLI (matches Robinhood's own Cursor instructions)..cursor/environment.json— establishes a repository-managed Cloud Agent environment. NomcpServerAllowlist, so it inherits the upstream team/user MCP policy — no per-server gate to maintain as more MCPs are added (this environment is treated as trusted/internal).Why
Requested: make the Robinhood Trading MCP available, with the allowlist dropped so many MCP servers can be connected without per-server approval friction.
Verified
HTTP 401withWWW-Authenticate: Bearer— the healthy "up, needs auth" response.token_endpoint_auth_methods_supported: ["none"]— no API key/secret/password exists for this server.environment.jsonconforms to the environment schema (onlyname; nomcpServerAllowlist).Reviewer notes
.cursor/environment.json. Committing it makes the environment repository-managed (highest precedence) for future Cloud Agents on this repo. With no allowlist, upstream team/user MCP policy is inherited unchanged..cursor/mcp.jsoncovers Cursor IDE/Agent-window/CLI clients. For Cloud Agents, the Robinhood server must also be added in the Cloud Agents dashboard (HTTP MCP), and each user completes Robinhood's interactive OAuth (browser + Robinhood mobile app) once — a headless cloud agent cannot perform that handshake.Supersedes #148.