Skip to content

Conversation

@dsp-ant
Copy link
Member

@dsp-ant dsp-ant commented Dec 2, 2024

Fixes #151

jspahrsummers
jspahrsummers previously approved these changes Dec 3, 2024
Copy link
Member

@jspahrsummers jspahrsummers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! We should also remove this from the SDK—added to my to-do list, but anyone can get to it first 😅

@dsp-ant dsp-ant force-pushed the davidsp/tool-results branch from 9dd079a to e1fa303 Compare December 3, 2024 13:00
@dsp-ant dsp-ant enabled auto-merge December 3, 2024 13:00
@dsp-ant dsp-ant merged commit 855fdff into main Dec 3, 2024
25 checks passed
@dsp-ant dsp-ant deleted the davidsp/tool-results branch December 3, 2024 13:08
laith-abood pushed a commit to laith-abood/mcp-claude that referenced this pull request Jan 13, 2025
…/davidsp/tool-results

servers: make tool call result spec compatible
PazerOP referenced this pull request in PazerOP/mcp-template Jul 15, 2025
servers: make tool call result spec compatible
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.

Bug in example servers (toolResult confusion)

3 participants