github / github/copilot-cli

Support stdio transport server on the session/new request in the acp mode.

Đang mở
#3,889 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
area:mcp area:non-interactive
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

Agent Client Protocol says [All agents MUST support this transport](https://agentclientprotocol.com/protocol/v1/schema#mcpserver) but Copilot CLI has Rejected these stdio servers.

### Affected version

GitHub Copilot CLI 1.0.63

### Steps to reproduce the behavior

1. Create ACP Client and it connect to Copilot CLI
2. request a `session/new` request with `mcpServers` config include the stdio transport servers.
3. Copilot CLI will rejected these stdio servers. (I discovered Copilot CLI says reject in the process log)

### Expected behavior

These will launch correctly stdio transport MCP Servers via `session/new` request

### Additional context

Currently workaround:
mcpServers config pass to copilot cli via `--additional-mcp-config=JSON` args, these works fine.

Is that the behavior correct?

thanks.

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

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

Hướng nghiên cứu

Start at the ACP mode handler for the session/new request and trace how its mcpServers configuration is processed, comparing it with the --additional-mcp-config path that already works. Reproduce the request with a stdio transport server and verify that the server launches successfully through session/new without being rejected.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
shell
Lĩnh vực
api
Loại issue
Lỗi
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Sôi nổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
55/100

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.