SEP-2200: Clarify tool result content visibility#2200
SEP-2200: Clarify tool result content visibility#2200krubenok wants to merge 2 commits intomodelcontextprotocol:mainfrom
Conversation
docs/community/seps/2200-clarify-tool-result-content-visibility.mdx
Outdated
Show resolved
Hide resolved
|
Comments addressed! Lmk if you need follow ups. |
918ced8 to
5c5b770
Compare
|
I think it is a shame that so many SDKs make it easy for server authors to choose structured output and simply stringify it in the unstructured output (content) field without making a conscious decision as to whether this is the right choice. A lot of models do better with unstructured data. We should make it easy to do the right thing. Making structured output extremely convenient leads developers to assuming it is always superior. Perhaps this cannot be solved at the spec level, especially given we already have the fields with these names. But I think it would be good with a blog post after this is presumably included in a future spec version, making it clear that whether the model-oriented output is structured or not is a choice, where both options can be valid. |
|
Agreed @PederHP - as I started writing a server and looking into the defaults in the ecosystem that's what motivated me to ask the questions and write this. The MCP Inspector checks that
Edit: added inspector #1089 to track that fix as well. |
|
good work here folks, thank you! |
|
@PederHP i'm happy to draft the related blog post you mentioned or is the typical practice for that to come from one of the maintainers? Would you want the blog post to go in at the same time as this or would that be in a subsequent PR when the spec gets bumped to include this? |
Maintainer Activity CheckHi @olaservo! You're assigned to this SEP but there hasn't been any activity from you in 14 days. Please provide an update on:
If you're no longer able to sponsor this SEP, please let us know so we can find another maintainer. This is an automated message from the SEP lifecycle bot. |
Hi @krubenok I think we don't need to have the blog post ready at the same time as this, it can be a followup. Typically its been maintainers authoring them, but I think it would also be welcome for other contributors to propose or draft them. |
Just want to note that this was fixed - thank you @krubenok for opening the Inspector issue! |
evalstate
left a comment
There was a problem hiding this comment.
This is a breaking change to the intent of the protocol - not a clarification - and should be treated as such. In practice, OpenAI Apps SDK has effectively superceded the original design (which has drawbacks and could have been handled in the SDK automatically anyway) - so whilst I'm in support this isn't a clarification.
|
Thanks @evalstate and @cliffhall for the additional feedback. After tracing the full history of Here's a recap of my current understanding based on more digging: PR #371's description explicitly stated that So the current spec text where Shaun and Cliff are right that the current spec says @krubenok, lets address the above more directly in the SEP:
|
|
@olaservo Thanks, this was helpful. I updated the SEP to address this more directly:
Let me know if this framing now matches your concerns. |
Co-authored-by: Copilot <[email protected]>
a56569f to
2490c04
Compare
Co-authored-by: Copilot <[email protected]>

Summary
Testing
AI Assistance