Skip to content

Add output_schema caveat to response limiting docs#3099

Merged
jlowin merged 1 commit intomainfrom
docs/response-limiting-schema-note
Feb 6, 2026
Merged

Add output_schema caveat to response limiting docs#3099
jlowin merged 1 commit intomainfrom
docs/response-limiting-schema-note

Conversation

@jlowin
Copy link
Copy Markdown
Member

@jlowin jlowin commented Feb 6, 2026

The new ResponseLimitingMiddleware truncates oversized responses into a single TextContent block — which means if a tool declares an output_schema, the truncated result won't conform to it. This adds a note to the docs so users aren't surprised.

Extends #3072

@marvin-context-protocol marvin-context-protocol Bot added documentation Updates to docs, examples, or guides. Primary change is documentation-related. server Related to FastMCP server implementation or server-side functionality. labels Feb 6, 2026
@jlowin jlowin merged commit 32c6826 into main Feb 6, 2026
14 of 15 checks passed
@jlowin jlowin deleted the docs/response-limiting-schema-note branch February 6, 2026 23:15
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 6, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request adds a documentation note to the middleware documentation page explaining that truncating responses when a tool defines an output_schema may result in the output being a plain TextContent instead of the expected structured output. The note is positioned adjacent to the Response Limiting behavior documentation and its usage examples, with no changes to code or runtime behavior.

Possibly related PRs

  • PR #3072: Introduces ResponseLimitingMiddleware that truncates tool responses, which is directly related to the documented behavior described in this documentation note.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/response-limiting-schema-note

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Updates to docs, examples, or guides. Primary change is documentation-related. server Related to FastMCP server implementation or server-side functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant