-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Comparing changes
Open a pull request
base repository: github/copilot-sdk
base: 70b7721
head repository: github/copilot-sdk
compare: 80bd426
- 6 commits
- 12 files changed
- 2 contributors
Commits on Apr 9, 2026
-
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>Configuration menu - View commit details
-
Copy full SHA for bbefa61 - Browse repository at this point
Copy the full SHA bbefa61View commit details -
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>
Configuration menu - View commit details
-
Copy full SHA for 5a326f3 - Browse repository at this point
Copy the full SHA 5a326f3View commit details -
Remove unused _convert_call_tool_result import
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 4f18d31 - Browse repository at this point
Copy the full SHA 4f18d31View commit details -
Address review feedback: add type guards in Python, fix Go comment typo
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 2101205 - Browse repository at this point
Copy the full SHA 2101205View commit details -
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>
Configuration menu - View commit details
-
Copy full SHA for e6729bc - Browse repository at this point
Copy the full SHA e6729bcView commit details -
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>
Configuration menu - View commit details
-
Copy full SHA for 80bd426 - Browse repository at this point
Copy the full SHA 80bd426View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 70b7721...80bd426