Skip to content

Commit 4b9546d

Browse files
committed
Revert " update call_tool method to replace args with arguments parameter"
This reverts commit 47bc4ca.
1 parent 4ab9757 commit 4b9546d

File tree

2 files changed

+4
-28
lines changed

2 files changed

+4
-28
lines changed

src/mcp/client/session_group.py

Lines changed: 3 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@
1414
from dataclasses import dataclass
1515
from datetime import timedelta
1616
from types import TracebackType
17-
from typing import Any, TypeAlias, overload
17+
from typing import Any, TypeAlias
1818

1919
import anyio
2020
from pydantic import BaseModel
21-
from typing_extensions import Self, deprecated
21+
from typing_extensions import Self
2222

2323
import mcp
2424
from mcp import types
@@ -191,45 +191,21 @@ def tools(self) -> dict[str, types.Tool]:
191191
"""Returns the tools as a dictionary of names to tools."""
192192
return self._tools
193193

194-
@overload
195194
async def call_tool(
196195
self,
197196
name: str,
198-
arguments: dict[str, Any],
199-
read_timeout_seconds: timedelta | None = None,
200-
progress_callback: ProgressFnT | None = None,
201-
*,
202-
meta: dict[str, Any] | None = None,
203-
) -> types.CallToolResult: ...
204-
205-
@overload
206-
@deprecated("The 'args' parameter is deprecated. Use 'arguments' instead.")
207-
async def call_tool(
208-
self,
209-
name: str,
210-
*,
211197
args: dict[str, Any],
212198
read_timeout_seconds: timedelta | None = None,
213199
progress_callback: ProgressFnT | None = None,
214-
meta: dict[str, Any] | None = None,
215-
) -> types.CallToolResult: ...
216-
217-
async def call_tool(
218-
self,
219-
name: str,
220-
arguments: dict[str, Any] | None = None,
221-
read_timeout_seconds: timedelta | None = None,
222-
progress_callback: ProgressFnT | None = None,
223200
*,
224201
meta: dict[str, Any] | None = None,
225-
args: dict[str, Any] | None = None,
226202
) -> types.CallToolResult:
227203
"""Executes a tool given its name and arguments."""
228204
session = self._tool_to_session[name]
229205
session_tool_name = self.tools[name].name
230206
return await session.call_tool(
231207
session_tool_name,
232-
arguments if args is None else args,
208+
args,
233209
read_timeout_seconds=read_timeout_seconds,
234210
progress_callback=progress_callback,
235211
meta=meta,

tests/client/test_session_group.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def hook(name: str, server_info: types.Implementation) -> str: # pragma: no cov
6767
# --- Test Execution ---
6868
result = await mcp_session_group.call_tool(
6969
name="server1-my_tool",
70-
arguments={
70+
args={
7171
"name": "value1",
7272
"args": {},
7373
},

0 commit comments

Comments
 (0)