-
Notifications
You must be signed in to change notification settings - Fork 798
Handle multi-modal and error responses from MCP tool calls #1618
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Docs Preview
|
80316ec
to
f542b2d
Compare
The issue suggested the following behavior:
I had a chat about this with @dmontagu and we decided to keep multiple text parts as a list, because this may be meaningful (for example, separate RAGged docs that shouldn't look like one doc) and LLMs should understand the list just fine. If there's only a single part (text or file), we do unwrap the list. |
Fixes #1614
Also uses a unique identifier for each file returned by a tool call.