Skip to content

Commit 68fbef9

Browse files
committed
call_tool overloads: collapse to Literal[False] + bool on all three layers
A runtime bool variable now matches the second overload (union return); literal False and the default still match the first (CallToolResult). Drops the redundant Literal[True] arm.
1 parent 78b1f9d commit 68fbef9

3 files changed

Lines changed: 2 additions & 16 deletions

File tree

src/mcp/client/client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -401,7 +401,7 @@ async def call_tool(
401401
input_responses: InputResponses | None = None,
402402
request_state: str | None = None,
403403
meta: RequestParamsMeta | None = None,
404-
allow_input_required: Literal[True],
404+
allow_input_required: bool,
405405
) -> CallToolResult | InputRequiredResult: ...
406406

407407
async def call_tool(

src/mcp/client/session.py

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -616,20 +616,6 @@ async def call_tool(
616616
allow_input_required: Literal[False] = False,
617617
) -> types.CallToolResult: ...
618618

619-
@overload
620-
async def call_tool(
621-
self,
622-
name: str,
623-
arguments: dict[str, Any] | None = None,
624-
read_timeout_seconds: float | None = None,
625-
progress_callback: ProgressFnT | None = None,
626-
*,
627-
input_responses: types.InputResponses | None = None,
628-
request_state: str | None = None,
629-
meta: RequestParamsMeta | None = None,
630-
allow_input_required: Literal[True],
631-
) -> types.CallToolResult | types.InputRequiredResult: ...
632-
633619
@overload
634620
async def call_tool(
635621
self,

src/mcp/client/session_group.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ async def call_tool(
215215
input_responses: types.InputResponses | None = None,
216216
request_state: str | None = None,
217217
meta: types.RequestParamsMeta | None = None,
218-
allow_input_required: Literal[True],
218+
allow_input_required: bool,
219219
) -> types.CallToolResult | types.InputRequiredResult: ...
220220

221221
async def call_tool(

0 commit comments

Comments
 (0)