Drop unused overloads on Tool.run

ToolManager.call_tool forwards convert_result as a bool, never a literal,
so the Literal-keyed overloads never resolved. The overloads that matter
are on ToolManager.call_tool, which MCPServer.call_tool calls with
convert_result=True.

Author: Kludex

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

@@ -2,13 +2,13 @@
 
 from collections.abc import Callable
 from functools import cached_property
-from typing import TYPE_CHECKING, Any, Literal, overload
+from typing import TYPE_CHECKING, Any
 
 from pydantic import BaseModel, Field
 
 from mcp.server.mcpserver.exceptions import ToolError
 from mcp.server.mcpserver.utilities.context_injection import find_context_parameter
-from mcp.server.mcpserver.utilities.func_metadata import FuncMetadata, ToolResult, func_metadata
+from mcp.server.mcpserver.utilities.func_metadata import FuncMetadata, func_metadata
 from mcp.shared._callable_inspection import is_async_callable
 from mcp.shared.exceptions import UrlElicitationRequiredError
 from mcp.shared.tool_name_validation import validate_and_warn_tool_name
@@ -88,17 +88,6 @@ def from_function(
             meta=meta,
         )
 
-    @overload
-    async def run(
-        self, arguments: dict[str, Any], context: Context[LifespanContextT, RequestT], convert_result: Literal[True]
-    ) -> ToolResult: ...
-    @overload
-    async def run(
-        self,
-        arguments: dict[str, Any],
-        context: Context[LifespanContextT, RequestT],
-        convert_result: Literal[False] = False,
-    ) -> Any: ...
     async def run(
         self,
         arguments: dict[str, Any],