Skip to content

Use submitted name if possible and server as fallback for MCP installation#417

Open
StefELianos wants to merge 1 commit into
cursor:mainfrom
StefELianos:fix/use_submitted_name_for_mcp_install
Open

Use submitted name if possible and server as fallback for MCP installation#417
StefELianos wants to merge 1 commit into
cursor:mainfrom
StefELianos:fix/use_submitted_name_for_mcp_install

Conversation

@StefELianos

@StefELianos StefELianos commented Jul 9, 2026

Copy link
Copy Markdown

Linked issue:

#416

Summary

  • Fixes MCP install deeplink name selection so it uses the submitted MCP component name instead of defaulting to the config key (often server).
  • Preserves previous fallback behaviour by using server when the component name is empty/missing.
  • Keeps MCP config payload handling unchanged; only the deeplink name parameter selection is adjusted.

Problem

When adding an MCP via Add to Cursor, the deeplink name could show as server even when a custom name was provided, because the name was derived from the config structure rather than the component display name.

Solution

In apps/cursor/src/components/plugins/detail/mcp-section.tsx:

  • Stop deriving deeplink name from mcpServers keys.
  • Use mcp.name.trim() for install deeplinks.
  • Fall back to "server" if that name is blank.

Note

Low Risk
Narrow change to deeplink name query parameter only; no auth, data, or config parsing changes beyond dropping unused name from resolveMcpConfig.

Overview
MCP “Add to Cursor” install deeplinks now pass the plugin component’s display name (mcp.name) instead of the first mcpServers config key (often server).

resolveMcpConfig only returns the parsed config; name selection moved to install-link generation with mcp.name.trim() || "server" when no explicit mcp_link is set. Config payload and preview behavior are unchanged.

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

@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

@StefELianos is attempting to deploy a commit to the Anysphere Team on Vercel.

A member of the Team first needs to authorize it.

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