Skip to content

fix: resolve MCP tool parsing errors in Claude integration#1078

Merged
copybara-service[bot] merged 1 commit intomainfrom
test_888509155
Mar 25, 2026
Merged

fix: resolve MCP tool parsing errors in Claude integration#1078
copybara-service[bot] merged 1 commit intomainfrom
test_888509155

Conversation

@copybara-service
Copy link

fix: resolve MCP tool parsing errors in Claude integration

The Claude model integration parsing logic failed when processing MCP tool responses because it only extracted output from the legacy result field. Extended extraction logic to:

  • Support native MCP content arrays.
  • Support legacy result structures natively.
  • Fallback to generic JSON serialization of the entire map.
    Additionally, updated AbstractMcpTool.wrapCallResult() format to match Python ADK.

@copybara-service copybara-service bot force-pushed the test_888509155 branch 5 times, most recently from ed30092 to e1dae37 Compare March 25, 2026 10:20
The Claude model integration parsing logic failed when processing MCP tool responses because it only extracted output from the legacy `result` field. Extended extraction logic to:
- Support native MCP `content` arrays.
- Support legacy `result` structures natively.
- Fallback to generic JSON serialization of the entire map.
Additionally, updated AbstractMcpTool.wrapCallResult() format to match Python ADK.

PiperOrigin-RevId: 889141233
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants