Skip to content

fix(etoro-trading): declare server placement for Grok Bot - #461

Merged
minupalaniappan merged 1 commit into
mainfrom
minu/etoro-trading-server-placement
Sep 29, 2026
Merged

minupalaniappan merged 1 commit into
mainfrom
minu/etoro-trading-server-placement

Conversation

@minupalaniappan

@minupalaniappan minupalaniappan commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

What this fixes

eToro Trading (third_party/etoro-trading, added in #410) installs in Grok Bot but never shows a connect / sign-in card, so users cannot reach eToro's hosted MCP at https://mcp.public-api.etoro.com from Settings → Connectors.

Cause: Grok Bot dials only SERVER-placed hosted MCP rows. etoro-trading/mcp.json has no placement field, so its row stays UNDECIDED. This is the same gap #438 fixed for TinyFish, beehiiv, Trello, Buffer and Plaud.

Fix: add "placement": "server" to the etoro-trading server entry, bump the plugin to 1.0.1 so clients re-fetch the manifest, and add a CHANGELOG line. eToro's own developer plugin (etoro, the API docs listing) is untouched.

Verified

Follow-ups

  • After merge: re-index cursor-public with force re-pull and move etoro-trading from public-draft to public-listed.
  • Grok Bot mobile sign-in still needs eToro to allow https://www.cursor.com/bot/mcp/oauth/callback and grokbot://mcp/oauth/callback on /api/sso/v1/register (currently 400 invalid_redirect_uri); requested from eToro separately.

Note

Low Risk
Manifest-only change (MCP placement + version bump); no runtime or auth logic modified.

Overview
Fixes Grok Bot so the eToro Trading connector can show a connect / sign-in card instead of staying in an undecided state.

Adds "placement": "server" to the hosted HTTP MCP entry in mcp.json (same pattern as other third-party hosted MCP plugins). Bumps the plugin to 1.0.1 and documents the change in CHANGELOG so clients pick up the updated manifest.

Reviewed by Cursor Bugbot for commit 0b38ff6. Bugbot is set up for automated code reviews on this repo. Configure here.

@minupalaniappan
minupalaniappan merged commit 69cf06f into main Sep 29, 2026
2 checks passed
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