[Bug][MCP] Repeated nextCursor can make tools/list synchronization loop indefinitely

[wxggzz]

问题概述

在当前 master（141eb6fef83422698aef7a981029e843e8161534）中，MCP Client 的 syncTools() 会持续拉取 tools/list，直到 nextCursor 为空，但没有检测服务端是否重复返回同一个 continuation cursor。

如果异常或不受信任的 MCP Server 返回空工具页并重复同一个非空 nextCursor，该循环不会触发现有的重复工具名校验，也不会结束，会持续发送 tools/list 请求。

源码位置：

• packages/mcp/mcp-client/src/tools.ts#L143-L174

影响

• 初次连接时，apply() 等待的 connection.ready 无法完成，插件启动会一直挂起；
• notifications/tools/list_changed 触发重同步时，串行的 syncChain 会被永久占用，后续同步无法执行；
• 客户端持续产生 JSON-RPC 请求，带来 CPU、日志和网络资源消耗；
• failOnStartupError 无法处理，因为当前路径既不 resolve 也不 reject。

最小复现

可以在 packages/mcp/mcp-client/tests/mcp-client.spec.ts 中构造一个 mock client：

let calls = 0
const client = createMockClient([])
client.listTools.mockImplementation(async () => {
  calls += 1
  if (calls > 3) throw new Error('sentinel: pagination did not stop')
  return { tools: [], nextCursor: 'same-cursor' }
})

await expect(syncTools(client as never, ctx, defaultOpts, new Map()))
  .rejects.toThrow(/repeated.*cursor/i)
expect(client.listTools).toHaveBeenCalledTimes(2)

当前实现不会在第二次响应后拒绝，而会继续请求，最终只会被上面的 sentinel 人为中断；移除 sentinel 后会持续循环。

期望行为

当 tools/list 返回已经出现过的非空 nextCursor 时，syncTools() 应立即以明确错误终止 fetch phase，并保持上一代已注册工具不变。

建议修复

在 fetch phase 内维护 Set<string>：

1. 收到非空 nextCursor 后，先检查是否已经出现；
2. 若重复，抛出类似 MCP server repeated a tools/list continuation cursor 的错误；
3. 否则记录 cursor 并继续下一页；
4. 增加“空工具页 + 重复 cursor”的回归测试，并保留现有正常两页分页测试。

仓库中已有一致的防环模式，可直接复用设计：

• API Proxy 的 seenCursors 防护
• Session Query 的 repeated cursor 防护

去重情况

提交前已搜索 Discussions 和 PR 中的 nextCursor、continuation cursor、repeated cursor、tools/list pagination 等关键词，暂未发现相同报告。