-
Notifications
You must be signed in to change notification settings - Fork 1.3k
MCP: backslashes need to be escaped when adding server info #1778
Copy link
Copy link
Open
Labels
area:mcpMCP server configuration, discovery, connectivity, OAuth, policy, and registryMCP server configuration, discovery, connectivity, OAuth, policy, and registry
Description
Describe the bug
When adding an MCP server from the CLI, you get a nice TUI to define the info for the server. However, if you put something into "Command" with backslashes (i.e. dotnet run --project D:\projects\terminal\src\tools\wt.mcp), the backslashes don't automatically get escaped! This results in the MCP server not working!
When you later check the MCP server configuration, it will remove the \ so it ends up looking like this: dotnet run --project D:projectsterminalsrctoolswt.mcp.
Affected version
No response
Steps to reproduce the behavior
/mcpato add a new server- fill out the details; in
Commanduse something with backslashes (see example above) - save
The MCP server fails to work. The error doesn't have anything to do with backslashes either! (which is expected tbh, but it made tracking this down a bit harder)
Expected behavior
Given that it's a TUI, there should not be a need to escape characters or backslashes.
Additional context
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area:mcpMCP server configuration, discovery, connectivity, OAuth, policy, and registryMCP server configuration, discovery, connectivity, OAuth, policy, and registry
Type
Fields
Give feedbackNo fields configured for Bug.