Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • Use Cursor's native HTTP URL support instead of mcp-remote wrapper
  • Simplifies config snippet and deeplink for Cursor MCP client

Type of Change

  • Improvement

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel
Copy link

vercel bot commented Jan 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Jan 27, 2026 3:49am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 27, 2026

Greptile Overview

Greptile Summary

Simplified Cursor MCP server configuration by removing the mcp-remote wrapper and using Cursor's native HTTP URL support instead.

  • Modified getConfigSnippet() to generate direct URL configs for Cursor ({ url, headers }) instead of stdio-based npx commands
  • Updated getCursorInstallUrl() to create deeplinks with the same direct URL format
  • Added inline comments explaining that Cursor supports native HTTP while Claude Desktop and VS Code still require mcp-remote
  • Maintains backward compatibility for other clients (Claude Code, Claude Desktop, VS Code) which continue using their existing transport mechanisms

The change streamlines the user experience by providing a simpler configuration format that matches Cursor's capabilities.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are well-isolated, logically sound, and maintain backward compatibility for all other MCP clients. The modification correctly uses Cursor's native HTTP URL support while preserving existing behavior for Claude Desktop and VS Code. No breaking changes or risky operations introduced.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/settings-modal/components/workflow-mcp-servers/workflow-mcp-servers.tsx Simplified Cursor MCP config by removing mcp-remote wrapper and using native HTTP URL support

Sequence Diagram

sequenceDiagram
    participant User
    participant Component as WorkflowMcpServers
    participant Config as getConfigSnippet()
    participant Deeplink as getCursorInstallUrl()
    participant Cursor as Cursor Client

    User->>Component: Select "Cursor" tab
    Component->>Config: getConfigSnippet('cursor', isPublic, serverName)
    
    alt Public Server
        Config->>Config: Create { url: mcpServerUrl }
    else Private Server
        Config->>Config: Create { url: mcpServerUrl, headers: { 'X-API-Key': '$SIM_API_KEY' } }
    end
    
    Config-->>Component: Return JSON config string
    Component->>User: Display config snippet
    
    User->>Component: Click "Add to Cursor" button
    Component->>Deeplink: getCursorInstallUrl(isPublic, serverName)
    
    alt Public Server
        Deeplink->>Deeplink: Create { url: mcpServerUrl }
    else Private Server
        Deeplink->>Deeplink: Create { url: mcpServerUrl, headers: { 'X-API-Key': '$SIM_API_KEY' } }
    end
    
    Deeplink->>Deeplink: Base64 encode config
    Deeplink-->>Component: Return cursor://... deeplink URL
    Component->>Cursor: Open deeplink
    Cursor->>Cursor: Install MCP server with native HTTP transport
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit d8df08d into staging Jan 27, 2026
12 checks passed
@waleedlatif1 waleedlatif1 deleted the improvement/mcp branch January 27, 2026 03:54
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.

2 participants