fix(mcp): pass the tools/list cursor through PaginatedRequestParams

Address the review on #825: the `list_tools(cursor=...)` overload is
deprecated in the pinned mcp 1.30.0, and the ty gate exits non-zero on
warnings, so `make lint-types` fails. Pass the cursor via
`params=types.PaginatedRequestParams(cursor=cursor)` instead, and teach
the three session fakes to take `params`.

Author: yudongyouqing

raven/mcp/client.py

@@ -249,7 +249,7 @@ async def _mcp_server_connection(
     cancels the turn that was connecting.
     """
     async with AsyncExitStack() as stack:
-        from mcp import ClientSession
+        from mcp import ClientSession, types
 
         read, write = await stack.enter_async_context(
             open_mcp_transport(cfg, transport_type, executor, http_auth=http_auth)
@@ -262,7 +262,7 @@ async def _mcp_server_connection(
         cursor = None
         seen_cursors = set()
         while True:
-            page = await session.list_tools(cursor=cursor)
+            page = await session.list_tools(params=types.PaginatedRequestParams(cursor=cursor))
             tools.extend(page.tools)
             cursor = page.nextCursor
             # A server that keeps returning the same cursor would page forever.

tests/test_mcp_client.py

@@ -105,7 +105,8 @@ async def __aexit__(self, exc_type, exc, traceback):
         async def initialize(self):
             return SimpleNamespace(capabilities=SimpleNamespace(tools=True))
 
-        async def list_tools(self, cursor=None):
+        async def list_tools(self, *, params=None):
+            cursor = params.cursor if params else None
             asked.append(cursor)
             return pages[cursor]
 

tests/test_mcp_manager.py

@@ -278,7 +278,7 @@ async def initialize(self):
                 await asyncio.Event().wait()
             return SimpleNamespace(capabilities=SimpleNamespace(tools=True))
 
-        async def list_tools(self, cursor=None):
+        async def list_tools(self, *, params=None):
             return SimpleNamespace(
                 tools=[SimpleNamespace(name="ping", description="", inputSchema={})], nextCursor=None
             )

tests/test_sandbox_unit.py

@@ -1801,7 +1801,7 @@ async def __aexit__(self, exc_type, exc, traceback):
             async def initialize(self):
                 return SimpleNamespace(capabilities=SimpleNamespace(tools=object()))
 
-            async def list_tools(self, cursor=None):
+            async def list_tools(self, *, params=None):
                 return SimpleNamespace(tools=[], nextCursor=None)
 
         monkeypatch.setattr(httpx, "AsyncClient", fake_http_client)