Fix pyright errors in Client and tests

Author: felixweinberger

src/mcp/client/client.py

@@ -193,7 +193,7 @@ async def list_resources(
         if params is not None:
             return await self.session.list_resources(params=params)
         elif cursor is not None:
-            return await self.session.list_resources(cursor)
+            return await self.session.list_resources(cursor)  # pyright: ignore[reportDeprecated]
         else:
             return await self.session.list_resources()
 
@@ -216,7 +216,7 @@ async def list_resource_templates(
         if params is not None:
             return await self.session.list_resource_templates(params=params)
         elif cursor is not None:
-            return await self.session.list_resource_templates(cursor)
+            return await self.session.list_resource_templates(cursor)  # pyright: ignore[reportDeprecated]
         else:
             return await self.session.list_resource_templates()
 
@@ -289,7 +289,7 @@ async def list_prompts(
         if params is not None:
             return await self.session.list_prompts(params=params)
         elif cursor is not None:
-            return await self.session.list_prompts(cursor)
+            return await self.session.list_prompts(cursor)  # pyright: ignore[reportDeprecated]
         else:
             return await self.session.list_prompts()
 
@@ -352,7 +352,7 @@ async def list_tools(
         if params is not None:
             return await self.session.list_tools(params=params)
         elif cursor is not None:
-            return await self.session.list_tools(cursor)
+            return await self.session.list_tools(cursor)  # pyright: ignore[reportDeprecated]
         else:
             return await self.session.list_tools()
 

tests/client/test_client.py

@@ -3,6 +3,7 @@
 import pytest
 from pydantic import AnyUrl
 
+import mcp.types as types
 from mcp.client.client import Client
 from mcp.server import Server
 from mcp.server.fastmcp import FastMCP
@@ -201,9 +202,9 @@ class TestClientCallbacks:
     @pytest.mark.anyio
     async def test_logging_callback(self, fastmcp_server: FastMCP):
         """Test that logging callback is called."""
-        log_messages = []
+        log_messages: list[types.LoggingMessageNotificationParams] = []
 
-        async def logging_callback(params):
+        async def logging_callback(params: types.LoggingMessageNotificationParams) -> None:
             log_messages.append(params)
 
         async with Client(

tests/client/transports/test_memory.py

@@ -109,5 +109,5 @@ async def test_raise_exceptions(self, fastmcp_server: FastMCP):
         """Test that raise_exceptions parameter is passed through."""
         transport = InMemoryTransport(fastmcp_server, raise_exceptions=True)
         # Just verify the transport can be created with the parameter
-        async with transport.connect() as (read_stream, write_stream):
+        async with transport.connect() as (read_stream, _write_stream):
             assert read_stream is not None