Repository navigation
MCPServer completion handler returning more than 100 values fails on 2026-07-28 sessions and violates the 100-item limit on 2025-11-25 sessions #3649
Description
Initial Checks
- I confirm that I'm using the newest release of my line (the latest 2.x, or the latest 1.x if I'm still on v1)
- I confirm that I searched for my issue in https://github.com/modelcontextprotocol/python-sdk/issues before opening this issue
Release line
2.x (current stable)
Description
An @mcp.completion() handler that returns more than 100 values behaves differently depending on the negotiated protocol version, and both behaviours are wrong:
- 2026-07-28 session: the whole
completion/completerequest fails with-32603 "Handler returned an invalid result". The user gets no suggestions at all. - 2025-11-25 session: all values go out on the wire. The spec says
values"Must not exceed 100 items", andtotal/hasMoreare left unset, so the client can't tell the list was too long.
It's easy to hit with the filter-by-prefix pattern from docs/servers/completions.md. When the user hasn't typed anything yet (argument.value == ""), every candidate matches, so any handler over a list of more than 100 options (countries, time zones, repos, file names, and so on) fails as soon as the field is focused.
The 100-item cap is only enforced by the 2026-07-28 wire model (Completion.values: Annotated[list[str], Field(max_length=100)] in mcp_types/_v2026_07_28). runner.py:381 checks results against that model and turns the ValidationError into a generic INTERNAL_ERROR. The 2025-11-25 wire model has no such constraint, so nothing stops the oversize list there. The MCPServer.completion wrapper (server.py:759-766) passes the handler's Completion through as-is.
For comparison, the TypeScript SDK's McpServer truncates to 100 and fills the pagination hints (createCompletionResult: values.slice(0, 100), total: suggestions.length, hasMore: suggestions.length > 100).
Expected: the same handler works on both protocol versions. The client receives the first 100 values with total / hasMore telling it more exist, as Completion's own docstring ("total … can exceed the number of values actually sent") and the docs' total= / has_more= note describe.
Server log on the 2026-07-28 session:
ERROR handler for 'completion/complete' returned an invalid result
pydantic_core._pydantic_core.ValidationError: 1 validation error for CompleteResult
completion.values
List should have at most 100 items after validation, not 150 [type=too_long, ...]
I reported this and would like to fix it. Proposed approach: in the MCPServer.completion wrapper, when the handler returns more than 100 values, send the first 100 and set total (to the full count) and has_more=True. Values the handler set explicitly win. Handlers returning ≤100 values are unchanged. The lowlevel Server stays as it is, since a lowlevel handler builds the CompleteResult itself. I'd add a regression test in tests/server/mcpserver/ using an in-memory Client, covering both mode="auto" (2026-07-28) and mode="legacy" (2025-11-25). It's roughly 10 lines of source. If you'd prefer a different behaviour (for example, raising a clear error at the wrapper instead of truncating), I'm happy to do that.
I used an AI assistant to help narrow this down; I ran the reproduction myself on 2.3.0 and on main.
Example Code
import anyio
from mcp_types import Completion, PromptReference
from mcp.client import Client
from mcp.server.mcpserver import MCPServer
COUNTRIES = [f"country-{i:03}" for i in range(150)]
mcp = MCPServer("demo")
@mcp.prompt()
def travel(country: str) -> str:
return f"Plan a trip to {country}"
@mcp.completion()
async def complete(ref, argument, context):
# Filter by the typed prefix, as in docs/servers/completions.md; an empty prefix matches everything.
return Completion(values=[c for c in COUNTRIES if c.startswith(argument.value)])
async def main() -> None:
for mode in ("auto", "legacy"):
async with Client(mcp, mode=mode) as client:
try:
result = await client.complete(
ref=PromptReference(type="ref/prompt", name="travel"),
argument={"name": "country", "value": ""},
)
c = result.completion
print(f"{client.protocol_version}: {len(c.values)} values, total={c.total}, has_more={c.has_more}")
except Exception as e:
print(f"{client.protocol_version}: {type(e).__name__}: {e}")
anyio.run(main)Output:
2026-07-28: MCPError: Handler returned an invalid result
2025-11-25: 150 values, total=None, has_more=None
Python & MCP Python SDK
Python 3.12.12, mcp 2.3.0, pydantic 2.13.5 (PyPI release)
Python 3.14.0, mcp main @ 91941ed, pydantic 2.12.5
Windows 11
Activity
musi22 commentedon Oct 8, 2026
Hi @maxisbey and team, I'd like to take this on.
The root cause is that
MCPServer.completionforwards the handler'sCompletiondirectly to the wire model without defensive pagination. On the2026-07-28protocol, Pydantic'sField(max_length=100)raises aValidationErrorwhich turns into an unexpectedINTERNAL_ERROR(-32603), while on2025-11-25it leaks all values over the wire un-paginated.Matching the TypeScript SDK's
createCompletionResultbehavior, the fix is to handle this at theMCPServer.completiondecorator level:len(completion.values) > 100, capvaluesto the first 100 (values[:100]).total = len(original_values)if not explicitly provided by the handler.has_more = True(or respect explicithas_moreif already provided).Serverhandlers untouched so raw protocol controllers maintain manual control.I'll include unit tests in
tests/server/mcpserver/testing bothmode="auto"andmode="legacy"clients.musi22 commentedon Oct 8, 2026
I have implemented and verified this fix (with regression tests covering both mode=" auto\ and mode=\legacy\ in ests/server/mcpserver/test_server.py) and opened PR #3660: #3660
KardeniaPoyu commentedon Oct 12, 2026
Still reproduces on current
main(4b5fe5a). I'm still planning to send the fix I proposed in the description, and I'll wait for a maintainer before opening a PR, per CONTRIBUTING.There's one behaviour question I'd like a maintainer to decide first. What should the wrapper do when the handler sets the pagination fields itself and they contradict a truncated list?
Completion(values=<150 items>, has_more=False): if we cap to 100 and keep the explicitFalse, the client is told there is nothing more while 50 values were dropped. I'd forcehas_more=Truewhenever the wrapper truncates.Completion(values=<150 items>, total=120):totalis smaller than what the handler returned. I'd usemax(total, len(values)).total=500,has_more=True) are kept as they are.I'd also log a warning when truncation happens. A handler returning more than 100 values is a server-side bug, and silently cutting the list hides it from the server author. The TS SDK truncates silently, so if you'd rather match it exactly, I'll leave the warning out.