github / github/copilot-cli

Copilot CLI exposes both MCP `content` and `structuredContent`

オープン
#4,515 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

area:mcp area:tools
主要言語
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.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. 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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。