github / github/copilot-cli

MCP: backslashes need to be escaped when adding server info

Đang mở
#1,778 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
area:mcp
Ngôn ngữ chính
Shell
Star
11.2k
Fork
1.9k
Merge trung bình
14 giờ 16 phút
Pull request đã merge (30 ngày)
6

Mô tả

### 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

1. `/mcp`
2. `a` to add a new server
3. fill out the details; in `Command` use something with backslashes (see example above)
4. 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_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.