Align with repo conventions: American spelling, top-level test function

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: olaservo

src/mcp/server/mcpserver/server.py

@@ -593,10 +593,10 @@ def add_tool(
                 - If None, auto-detects based on the function's return type annotation
                 - If True, creates a structured tool (return type annotation permitting)
                 - If False, unconditionally creates an unstructured tool
-            mirror_structured_content: Whether structured output is also serialised into a
+            mirror_structured_content: Whether structured output is also serialized into a
                 `content` text block alongside `structuredContent` (the spec's SHOULD,
                 default True). Set False to send `structuredContent` only, without the
-                duplicate serialised copy on the wire.
+                duplicate serialized copy on the wire.
         """
         self._tool_manager.add_tool(
             fn,
@@ -649,10 +649,10 @@ def tool(
                 - If None, auto-detects based on the function's return type annotation
                 - If True, creates a structured tool (return type annotation permitting)
                 - If False, unconditionally creates an unstructured tool
-            mirror_structured_content: Whether structured output is also serialised into a
+            mirror_structured_content: Whether structured output is also serialized into a
                 `content` text block alongside `structuredContent` (the spec's SHOULD,
                 default True). Set False to send `structuredContent` only, without the
-                duplicate serialised copy on the wire.
+                duplicate serialized copy on the wire.
 
         Example:
             ```python

src/mcp/server/mcpserver/tools/base.py

@@ -51,7 +51,7 @@ class Tool(BaseModel):
     meta: dict[str, Any] | None = Field(default=None, description="Optional metadata for this tool")
     mirror_structured_content: bool = Field(
         default=True,
-        description="Whether structured output is also mirrored into a serialised text content block",
+        description="Whether structured output is also mirrored into a serialized text content block",
     )
 
     @cached_property

src/mcp/server/mcpserver/utilities/func_metadata.py

@@ -117,12 +117,12 @@ def convert_result(
         being JSON-dumped into a text block.
 
         For a tool with an output schema, the structured output is returned as
-        `structuredContent`. By default it is *also* serialised into a `content`
+        `structuredContent`. By default it is *also* serialized into a `content`
         text block -- the spec's SHOULD, which keeps the value available to a
         client that consumes `content`. When ``mirror_structured_content`` is
-        False that serialised copy is omitted and `structuredContent` is the only
+        False that serialized copy is omitted and `structuredContent` is the only
         representation on the wire, so the same payload is not sent twice. Because
-        the serialised-text guidance is a SHOULD (not a MUST), opting out stays
+        the serialized-text guidance is a SHOULD (not a MUST), opting out stays
         conformant. The SDK makes no assumption about which field a client routes
         to a model versus elsewhere; that is the client's choice.
         """

tests/server/mcpserver/test_server.py

@@ -537,27 +537,6 @@ def get_user(user_id: int) -> UserOutput:
             assert isinstance(result.content[0], TextContent)
             assert '"name": "John Doe"' in result.content[0].text
 
-    async def test_tool_mirror_structured_content_false_omits_text_block(self):
-        """mirror_structured_content=False returns structuredContent only, with empty content."""
-
-        class UserOutput(BaseModel):
-            name: str
-            age: int
-
-        mcp = MCPServer()
-
-        @mcp.tool(mirror_structured_content=False)
-        def get_user(user_id: int) -> UserOutput:
-            """Get user by ID"""
-            return UserOutput(name="John Doe", age=30)
-
-        async with Client(mcp) as client:
-            result = await client.call_tool("get_user", {"user_id": 123})
-            assert result.is_error is False
-            assert result.structured_content == {"name": "John Doe", "age": 30}
-            # The serialised text mirror is suppressed; structuredContent is the sole representation.
-            assert result.content == []
-
     async def test_tool_structured_output_primitive(self):
         """Test tool with structured output returning primitive type"""
 
@@ -2366,3 +2345,25 @@ def greeting() -> str:  # pragma: no cover
     assert mcp._prompt_manager.list_prompts() == []
     with pytest.raises(ValueError, match="Unknown prompt: greeting"):
         mcp.remove_prompt("greeting")
+
+
+async def test_tool_mirror_structured_content_false_omits_text_block():
+    """mirror_structured_content=False returns structuredContent only, with empty content."""
+
+    class UserOutput(BaseModel):
+        name: str
+        age: int
+
+    mcp = MCPServer()
+
+    @mcp.tool(mirror_structured_content=False)
+    def get_user(user_id: int) -> UserOutput:
+        """Get user by ID"""
+        return UserOutput(name="John Doe", age=30)
+
+    async with Client(mcp) as client:
+        result = await client.call_tool("get_user", {"user_id": 123})
+        assert result.is_error is False
+        assert result.structured_content == {"name": "John Doe", "age": 30}
+        # The serialized text mirror is suppressed; structuredContent is the sole representation.
+        assert result.content == []