Skip to content

Commit 8238b0a

Browse files
Tapan Chughclaude
andcommitted
refactor: clean up unnecessary changes in tests and examples
- Replace unused request parameters with _ in handler functions - Remove explicit URI additions where auto-generation suffices - Remove unused TOOL_SCHEME and other imports - Keep only changes essential for prefix filtering functionality This reduces noise in tests and examples while maintaining the core functionality of URI-based storage and prefix filtering. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent a378a50 commit 8238b0a

File tree

24 files changed

+35
-72
lines changed

24 files changed

+35
-72
lines changed

examples/servers/simple-prompt/mcp_simple_prompt/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def main(port: int, transport: str) -> int:
4949
app = Server("mcp-simple-prompt")
5050

5151
@app.list_prompts()
52-
async def list_prompts(request: types.ListPromptsRequest) -> list[types.Prompt]:
52+
async def list_prompts(_) -> list[types.Prompt]:
5353
return [
5454
types.Prompt(
5555
name="simple",

examples/servers/simple-resource/mcp_simple_resource/server.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ def main(port: int, transport: str) -> int:
3232
app = Server("mcp-simple-resource")
3333

3434
@app.list_resources()
35-
async def list_resources(
36-
request: types.ListResourcesRequest,
37-
) -> list[types.Resource]:
35+
async def list_resources(_) -> list[types.Resource]:
3836
return [
3937
types.Resource(
4038
uri=FileUrl(f"file:///{name}.txt"),

examples/servers/simple-streamablehttp-stateless/mcp_simple_streamablehttp_stateless/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ async def call_tool(name: str, arguments: dict) -> list[types.ContentBlock]:
6969
]
7070

7171
@app.list_tools()
72-
async def list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
72+
async def list_tools(_) -> list[types.Tool]:
7373
return [
7474
types.Tool(
7575
name="start-notification-stream",

examples/servers/simple-streamablehttp/mcp_simple_streamablehttp/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ async def call_tool(name: str, arguments: dict) -> list[types.ContentBlock]:
8787
]
8888

8989
@app.list_tools()
90-
async def list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
90+
async def list_tools(_) -> list[types.Tool]:
9191
return [
9292
types.Tool(
9393
name="start-notification-stream",

examples/servers/simple-tool/mcp_simple_tool/server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ async def fetch_tool(name: str, arguments: dict) -> list[types.ContentBlock]:
3737
return await fetch_website(arguments["url"])
3838

3939
@app.list_tools()
40-
async def list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
40+
async def list_tools(_) -> list[types.Tool]:
4141
return [
4242
types.Tool(
4343
name="fetch",

examples/servers/structured_output_lowlevel.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222

2323
@server.list_tools()
24-
async def list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
24+
async def list_tools(_) -> list[types.Tool]:
2525
"""List available tools with their schemas."""
2626
return [
2727
types.Tool(

examples/snippets/servers/lowlevel/basic.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616

1717
@server.list_prompts()
18-
async def handle_list_prompts(request: types.ListPromptsRequest) -> list[types.Prompt]:
18+
async def handle_list_prompts(_) -> list[types.Prompt]:
1919
"""List available prompts."""
2020
return [
2121
types.Prompt(

examples/snippets/servers/lowlevel/lifespan.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ async def server_lifespan(_server: Server) -> AsyncIterator[dict]:
4949

5050

5151
@server.list_tools()
52-
async def handle_list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
52+
async def handle_list_tools(_) -> list[types.Tool]:
5353
"""List available tools."""
5454
return [
5555
types.Tool(

examples/snippets/servers/lowlevel/structured_output.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616

1717
@server.list_tools()
18-
async def list_tools(request: types.ListToolsRequest) -> list[types.Tool]:
18+
async def list_tools(_) -> list[types.Tool]:
1919
"""List available tools with structured output schemas."""
2020
return [
2121
types.Tool(

tests/client/test_output_schema_validation.py

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
from mcp.shared.memory import (
99
create_connected_server_and_client_session as client_session,
1010
)
11-
from mcp.types import TOOL_SCHEME, Tool
11+
from mcp.types import Tool
1212

1313

1414
@contextmanager
@@ -42,10 +42,9 @@ async def test_tool_structured_output_client_side_validation_basemodel(self):
4242
}
4343

4444
@server.list_tools()
45-
async def list_tools(request):
45+
async def list_tools(_):
4646
return [
4747
Tool(
48-
uri=f"{TOOL_SCHEME}/get_user",
4948
name="get_user",
5049
description="Get user data",
5150
inputSchema={"type": "object"},
@@ -82,10 +81,9 @@ async def test_tool_structured_output_client_side_validation_primitive(self):
8281
}
8382

8483
@server.list_tools()
85-
async def list_tools(request):
84+
async def list_tools(_):
8685
return [
8786
Tool(
88-
uri=f"{TOOL_SCHEME}/calculate",
8987
name="calculate",
9088
description="Calculate something",
9189
inputSchema={"type": "object"},
@@ -114,10 +112,9 @@ async def test_tool_structured_output_client_side_validation_dict_typed(self):
114112
output_schema = {"type": "object", "additionalProperties": {"type": "integer"}, "title": "get_scores_Output"}
115113

116114
@server.list_tools()
117-
async def list_tools(request):
115+
async def list_tools(_):
118116
return [
119117
Tool(
120-
uri=f"{TOOL_SCHEME}/get_scores",
121118
name="get_scores",
122119
description="Get scores",
123120
inputSchema={"type": "object"},
@@ -150,10 +147,9 @@ async def test_tool_structured_output_client_side_validation_missing_required(se
150147
}
151148

152149
@server.list_tools()
153-
async def list_tools(request):
150+
async def list_tools(_):
154151
return [
155152
Tool(
156-
uri=f"{TOOL_SCHEME}/get_person",
157153
name="get_person",
158154
description="Get person data",
159155
inputSchema={"type": "object"},
@@ -179,7 +175,7 @@ async def test_tool_not_listed_warning(self, caplog):
179175
server = Server("test-server")
180176

181177
@server.list_tools()
182-
async def list_tools(request):
178+
async def list_tools(_):
183179
# Return empty list - tool is not listed
184180
return []
185181

0 commit comments

Comments
 (0)