Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/mcp/server/fastmcp/utilities/func_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ def convert_result(self, result: Any) -> Any:
the structured output.
"""
if isinstance(result, CallToolResult):
if self.output_schema is not None:
if self.output_schema is not None and not result.isError:
assert self.output_model is not None, "Output model must be set if output schema is defined"
self.output_model.model_validate(result.structuredContent)
return result
Expand Down
14 changes: 14 additions & 0 deletions tests/server/fastmcp/test_func_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -878,6 +878,20 @@ def func_returning_annotated_tool_call_result() -> Annotated[CallToolResult, Per
meta.convert_result(func_returning_annotated_tool_call_result())


def test_tool_call_result_annotated_error_skips_structured_validation():
class PersonClass(BaseModel):
name: str

def func_returning_annotated_tool_call_result_error() -> Annotated[CallToolResult, PersonClass]: # pragma: no cover
return CallToolResult(content=[], isError=True)

meta = func_metadata(func_returning_annotated_tool_call_result_error)
result = func_returning_annotated_tool_call_result_error()

assert meta.output_schema is not None
assert meta.convert_result(result) is result


def test_tool_call_result_in_optional_is_rejected():
"""Test that Optional[CallToolResult] raises InvalidSignature"""

Expand Down
Loading