github / github/copilot-cli

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

未关闭
#3,889 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
area:mcp area:non-interactive
主要语言
Shell
星标
11.2k
派生
1.9k
平均合并
14 小时 16 分钟
30 天内合并 PR
6

描述

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

贡献指南

打开贡献指南

调研方向

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.

由索引模型根据 Issue 内容生成。

评估

技术栈
shell
领域
api
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
活跃
描述清晰度
基本清楚
新手友好度
55/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。