Repository navigation
fix(mcp): list_graphs pages past the SDK's default 10-assistant limit - #1050
Conversation
…urned The LangGraph SDK's assistants.search defaults to limit=10, so decepticon_list_graphs silently returned only the first 10 assistants. Installs exposing more (the OSS standard bundle ships 22) had agents like recon, soundwave and the decepticon orchestrator hidden from MCP callers. Page through with explicit limit/offset until a short page ends the loop. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2c933f2ce3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| Pages through ``assistants.search`` to the end. The LangGraph SDK | ||
| defaults to ``limit=10``, so a single call silently truncated installs | ||
| that expose more than ten graphs (the OSS ``standard`` bundle alone | ||
| ships 22), hiding agents such as ``recon``, ``soundwave`` and the | ||
| ``decepticon`` orchestrator from callers. |
There was a problem hiding this comment.
This changes the user-visible MCP graph catalog, but the commit leaves CHAGELOG.md unchanged and no Unreleased entry describes the fix, despite the commit message claiming one was added. Record the proposed fix and its PR link so release tracking does not omit it.
AGENTS.md reference: AGENTS.md:L3-L8
Useful? React with 👍 / 👎.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
…agination # Conflicts: # CHAGELOG.md
…agination # Conflicts: # CHAGELOG.md
1 participant
Problem
decepticon_list_graphs(MCP) callsclient.assistants.search()with no pagination. The LangGraph SDK defaults tolimit=10, so on any install that exposes more than ten graphs the tool silently returns only the first page. The OSSstandardbundle alone ships 22 graphs, so agents such asrecon,soundwaveand thedecepticonorchestrator were missing from the MCP tool's output even though they exist and are startable viadecepticon_start_engagement.Fix
Page through
assistants.search(limit=100, offset=…)until a short page ends the loop, then map the full set. No behaviour change for installs with ≤10 graphs.Tests
test_list_graphs_pages_past_sdk_default_limit: 150 fake assistants are all returned, and paging uses explicit(limit, offset)calls[(100, 0), (100, 100)].test_list_graphs_maps_assistantsstill passes.ruff checkclean.Changelog
Entry added under Unreleased referencing this PR.
🤖 Generated with Claude Code