modelcontextprotocol / modelcontextprotocol/conformance

sep-2322 absent-resultType probe is an invalid stimulus on the 2026-07-28 wire it negotiates

Open
#475 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
127
Forks
101
Avg merge
6d 1h
Merged PRs (30d)
7

Description

The test_mrtr_no_result_type probe in mrtr-client.ts deliberately omits resultType from its final result, and draft-result-fields.test.ts pins that omission as the check's stimulus. Problem is the referee only ever negotiates 2026-07-28 (server/discover returns supportedVersions: [DRAFT_PROTOCOL_VERSION]), and the released 2026-07-28 text scopes the absent-means-complete bridge to earlier revisions: "For backward compatibility with servers implementing earlier protocol versions, which do not include resultType, clients MUST treat an absent resultType as complete." The same spec says the result MUST include resultType, and the repo's own draft.schema.json has it in Result.required.

So the probe is an invalid stimulus on the very wire it negotiates, and a strict 2026-07-28 client fails sep-2322-default-result-type-complete for correctly rejecting a non-conformant reply. Downstream confirmation: cloudflare/agents baselines this scenario with the comment "Alpha.10 omits resultType after negotiating 2026-07-28. The final draft requires servers implementing that revision to include it; the absent-means-complete bridge applies only to earlier revisions."

Related: sep-2322.yaml still quotes the pre-final sentence without the version scoping ("If resultType is not specified, clients MUST assume a default value of complete for backwards compatibility"), which is probably where the check's shape came from. And the reason wire validation never flagged the probe is the inline-mock bypass from #418.

Following the #447 principle of running each revision's requirements at its own wire, I can see two fixes: gate the absent-resultType probe to runs where the referee negotiates a pre-2026-07-28 wire, or keep the 2026-07-28 wire and have the probe send resultType, retiring the default-check there. Since the omission is test-pinned as deliberate I did not want to just send a PR flipping it. Which way do you want this to go?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with test_mrtr_no_result_type in mrtr-client.ts and its expectations in draft-result-fields.test.ts, then read server/discover and draft.schema.json to confirm the negotiated revision and required fields. Compare the probe with sep-2322.yaml and the version-scoping evidence in the issue. Done means the test stimulus matches the negotiated wire requirements and the related conformance check no longer rejects a conformant client.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
api, testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.