Skip to content
Closed
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
10 changes: 9 additions & 1 deletion src/mcp/server/mcpserver/utilities/func_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,15 @@ def convert_result(self, result: Any) -> CallToolResult | InputRequiredResult:
output_model = self.output_model if self.output_schema is not None else None
if isinstance(result, CallToolResult):
if output_model is not None and not result.is_error:
self._output_adapter(output_model).validate_python(result.structured_content)
adapter = self._output_adapter(output_model)
validated = adapter.validate_python(
result.structured_content, by_alias=True, by_name=True
)
if isinstance(validated, BaseModel):
structured_content = validated.model_dump(mode="json", by_alias=True)
else:
structured_content = adapter.dump_python(validated, mode="json", by_alias=True)
return result.model_copy(update={"structured_content": structured_content})
return result

unstructured_content = _convert_to_content(result)
Expand Down
21 changes: 20 additions & 1 deletion tests/server/mcpserver/test_func_metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import pytest
from dirty_equals import IsPartialDict
from mcp_types import CallToolResult, ContentBlock, EmbeddedResource, InputRequiredResult, TextContent
from pydantic import BaseModel, Field, ValidationError
from pydantic import BaseModel, ConfigDict, Field, ValidationError
from typing_extensions import NotRequired, ReadOnly, Required

from mcp import MCPDeprecationWarning
Expand Down Expand Up @@ -1322,6 +1322,25 @@ def func_with_aliases() -> ModelWithAliases: # pragma: no cover
assert structured_content_defaults["second"] is None


def test_call_tool_result_structured_content_uses_aliases():
"""CallToolResult.structured_content is dumped with aliases, matching outputSchema."""

class AliasOut(BaseModel):
model_config = ConfigDict(populate_by_name=True)
field_first: str = Field(alias="first")
field_second: str = Field(alias="second")

def via_call_tool_result() -> Annotated[CallToolResult, AliasOut]:
return CallToolResult(
content=[TextContent(type="text", text="ok")],
structured_content={"field_first": "a", "field_second": "b"},
)

meta = func_metadata(via_call_tool_result)
converted = meta.convert_result(via_call_tool_result())
assert converted.structured_content == {"first": "a", "second": "b"}


def test_basemodel_reserved_names():
"""Test that functions with parameters named after BaseModel methods work correctly"""

Expand Down
Loading