Skip to content

fix: preserve tools/list pagination through proxy - #243

Open
RichaShiny wants to merge 1 commit into
sparfenyuk:mainfrom
RichaShiny:fix/forward-tools-list-cursor
Open

RichaShiny wants to merge 1 commit into
sparfenyuk:mainfrom
RichaShiny:fix/forward-tools-list-cursor

Conversation

@RichaShiny

Copy link
Copy Markdown

Fixes #233

Summary

  • forward the complete tools/list request parameters to the backing MCP session
  • preserve pagination cursors so clients can reach later tool pages and terminate pagination normally
  • add an in-memory proxy regression test covering page one, page two, and the final empty cursor

Testing

  • python -m pytest -q (92 passed, 1 skipped)
  • ruff check src/mcp_proxy/proxy_server.py tests/test_proxy_server.py
  • ruff format --check src/mcp_proxy/proxy_server.py tests/test_proxy_server.py
  • mypy --python-version 3.13 src

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: tools/list cursor param not forwarded to stdio server → pagination broken (infinite loop + page-2 tools unreachable)

1 participant