Skip to content

feat(dev-infra): Includes al, the dev-infra mcp in the dotagents config #109419

Open
kenzoengineer wants to merge 1 commit intomasterfrom
kjiang/add-devinfra-mcp-server
Open

feat(dev-infra): Includes al, the dev-infra mcp in the dotagents config #109419
kenzoengineer wants to merge 1 commit intomasterfrom
kjiang/add-devinfra-mcp-server

Conversation

@kenzoengineer
Copy link
Member

dev-infra has developed an experimental mcp server to help developers debug dev-infra related issues (devenv, devservices, ci, running the devserver, etc.)

This PR adds said MCP to the dotagents so that it's installed automatically when you run dotagents install


[[mcp]]
name = "al-devinfra-mcp"
url = "https://server-941971168213.us-west1.run.app/mcp"
Copy link
Member Author

@kenzoengineer kenzoengineer Feb 26, 2026

Choose a reason for hiding this comment

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

this server is currently unauthenticated and has no pretty domain,

i think for now its fine (security-wise, there's nothing sensitive in contained in the mcp, also we would want to allow open source contributors)

@kenzoengineer kenzoengineer marked this pull request as ready for review February 26, 2026 17:30
@kenzoengineer kenzoengineer requested a review from gricha February 26, 2026 17:30
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Wrong MCP transport type for Cursor configuration
    • Updated .cursor/mcp.json to use Cursor's valid streamableHttp transport type for al-devinfra-mcp instead of invalid http.

Create PR

Or push these changes by commenting:

@cursor push 2f3c98e54b
Preview (2f3c98e54b)
diff --git a/.cursor/mcp.json b/.cursor/mcp.json
--- a/.cursor/mcp.json
+++ b/.cursor/mcp.json
@@ -4,7 +4,7 @@
       "url": "https://mcp.sentry.dev/mcp/sentry?experimental=1"
     },
     "al-devinfra-mcp": {
-      "type": "http",
+      "type": "streamableHttp",
       "url": "https://server-941971168213.us-west1.run.app/mcp"
     }
   }
This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

},
"al-devinfra-mcp": {
"type": "http",
"url": "https://server-941971168213.us-west1.run.app/mcp"
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong MCP transport type for Cursor configuration

Medium Severity

The al-devinfra-mcp entry in .cursor/mcp.json uses "type": "http", but Cursor IDE does not recognize "http" as a valid transport type. Cursor's valid types are "command", "streamableHttp", and "sse". The "type": "http" value is the Claude Code (.mcp.json) format, likely copy-pasted without adjustment. This could cause the MCP server to fail to connect in Cursor. The existing sentry entry works by omitting type entirely, relying on auto-detection.

Fix in Cursor Fix in Web

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