Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: github/copilot-sdk
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 70b7721
Choose a base ref
...
head repository: github/copilot-sdk
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 80bd426
Choose a head ref
  • 6 commits
  • 12 files changed
  • 2 contributors

Commits on Apr 9, 2026

  1. Fix CallToolResult handling across all SDKs

    When a tool handler returns an MCP CallToolResult object
    ({ content: [...], isError?: bool }), all four SDKs were
    JSON-serializing it instead of converting it to ToolResultObject.
    This caused the LLM to see raw JSON instead of actual tool output.
    
    Add detection and conversion of CallToolResult in Node.js, Python,
    Go, and .NET. The .NET SDK additionally handles Microsoft.Extensions.AI
    content types (TextContent, DataContent, and unknown subtypes via
    AIJsonUtilities serialization).
    
    Fixes #937
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    bbefa61 View commit details
    Browse the repository at this point in the history
  2. Fix formatting and lint issues

    Run prettier on Node.js files, ruff format on Python files,
    and remove unused ToolResultObject import from test file.
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    5a326f3 View commit details
    Browse the repository at this point in the history
  3. Remove unused _convert_call_tool_result import

    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    4f18d31 View commit details
    Browse the repository at this point in the history
  4. Address review feedback: add type guards in Python, fix Go comment typo

    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    2101205 View commit details
    Browse the repository at this point in the history
  5. Fix Python import sorting in test_tools_unit.py

    Sort imports in copilot.tools import block to satisfy ruff I001 rule.
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    e6729bc View commit details
    Browse the repository at this point in the history
  6. Remove isCallToolResult and convertCallToolResult from public exports

    These are internal implementation details used by session.ts and client.ts.
    Go and Python already keep them private (lowercase/underscore-prefixed).
    
    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    stephentoub and Copilot committed Apr 9, 2026
    Configuration menu
    Copy the full SHA
    80bd426 View commit details
    Browse the repository at this point in the history
Loading