Add support for remote MCP servers via npx snippet - #38122
Conversation
…emplate switching
| "command": "npx", | ||
| /// The arguments to pass to the MCP server | ||
| "args": [ | ||
| "mcp-remote", |
There was a problem hiding this comment.
Using this doesn't seem right IMO. We should either add our own http transport or simply use the rust-sdk official lib.
There was a problem hiding this comment.
I think the benefit here is it allows people to use any transport if needed as sometimes you'll need special behaviour baked into a custom transport layer
There was a problem hiding this comment.
yeah also the main purpose of this is to guide users into a quick workaround for remote servers until built-in support is added.
|
relevant bug: |
…cept the installation
|
Same comment: going to close this PR in favor of #39021 Thanks for taking a stab at this! we'd rather take the approach of communicating with the remote servers natively, rather than call out to another binary though. |
Closes #37770
Release Notes:
Introduce server type selection and template switching for context server configuration. This update allows users to choose between local and remote server templates, improving flexibility in server setup. The editor content updates dynamically based on the selected template type.
btw i can confirm that the server command is run twice when adding a server causing 2 browser auth tabs to open (notice in the video below how 2 tabs are opened after adding the server entry) which is different from the behavior in the latest Zed 0.203.5 release which correctly only opens one auth tab
Screen.Recording.2025-09-14.at.1.07.42.AM.mp4