Copilot CLI exposes both MCP `content` and `structuredContent`
まだ誰も着手していません。
- 主要言語
- Shell
- スター
- 11.2k
- フォーク
- 1.9k
- 平均マージ
- 14時間 16分
- マージ済み PR(30日)
- 6
説明
Describe the bug
Summary
When an MCP tool result contains both content and structuredContent,
Copilot CLI appears to add both fields to the conversation context.
When structuredContent is present, the client should use it as the
structured tool result instead of exposing an equivalent or unrelated
content representation to the model.
For backward compatibility MCP tools which return structured content, should also set the unstructured content
For backwards compatibility, a tool that returns structured content SHOULD also return the serialized JSON in a TextContent block.
see: MCP spec about structured content
Example
server.py
Here is a simple MCP server (based on the MCP Python SDK) to demonstrate the issue:
from pydantic import BaseModel
from mcp.server.mcpserver import MCPServer
class HelloOutput(BaseModel):
message: str
server = MCPServer(name="hello", version="0.1.0")
@server.tool(structured_output=True)
def hello() -> HelloOutput:
"""Return a simple JSON hello object."""
return HelloOutput(message="Hello World")
if __name__ == "__main__":
server.run("stdio")
This is what the MCP server returns for the hello tool call.
{
"_meta": null,
"content": [
{
"type": "text",
"text": "{\n \"message\": \"Hello World\"\n}",
"annotations": null,
"_meta": null
}
],
"structuredContent": {
"message": "Hello World"
},
"isError": false,
"resultType": "complete"
}
Actual Copilot CLI context
Copilot CLI exposes both representations:
{
"message": "Hello World"
}
{"message":"Hello World"}
Expected result
Only the structured result should be provided to the model:
{"message":"Hello World"}
The content field may remain available for UI rendering or backwards
compatibility, but it should not be concatenated into the model context when it
duplicates structuredContent.
Impact
The behavior unnecessarily increases context size and may cause the model to
interpret one tool result as two JSON objects.
Related issue
The reporter of issue #3258 expects the opposite behavior.
Anyway, this contradicts the intention of the MCP spec:
Tool results may contain structured or unstructured content.
For backwards compatibility, a tool that returns structured content SHOULD also return the serialized JSON in a TextContent block.
That means:
- a MCP tool must either return structured or unstructured content (but not both)
- but they should return a serialized representation in the unstructured content block for backward compatibility (i.e. for legacy clients which don't support structured content)
Clients which support structured content must therefore silently ignore the unstructured content if there is structured content.
Affected version
GitHub Copilot CLI 1.0.80.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
server.py MCP reproducerを使用して、contentとstructuredContentの両方を含む結果を生成し、その後、Copilot CLIがMCPツールの結果を会話コンテキストに追加する仕組みを追跡します。structuredContentのみがモデルに到達し、contentはUIまたは互換性のための処理で利用可能なままであることを確認し、想定される単一のJSON表現を検証します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- api, cli
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 活発
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 52/100