Skip to content

feat: add conformance scenario for SEP-2164 resource-not-found error code#200

Open
pja-ant wants to merge 1 commit intomainfrom
sep-2164-conformance-test
Open

feat: add conformance scenario for SEP-2164 resource-not-found error code#200
pja-ant wants to merge 1 commit intomainfrom
sep-2164-conformance-test

Conversation

@pja-ant
Copy link
Copy Markdown

@pja-ant pja-ant commented Mar 27, 2026

Adds a conformance scenario for SEP-2164, which standardizes the resource-not-found error code from -32002 to -32602 (Invalid Params).

What's tested

The scenario sep-2164-resource-not-found attempts to read a URI the server does not recognize and verifies the error response:

Check ID Requirement Level Status on violation
sep-2164-error-code Server returns -32602 (not -32002, not empty contents) MUST / MUST NOT FAILURE
sep-2164-data-uri Error data includes the requested uri SHOULD WARNING

The scenario is scoped to specVersions: ['draft'] only — it does not run against dated spec versions until SEP-2164 is incorporated into a release.

Traceability

Includes src/scenarios/server/sep-2164.yaml mapping each normative requirement to its check, following the format proposed in SEP-2484.

Verification

  • Passes against the TypeScript SDK everything-server (returns -32602; data.uri yields WARNING since the SDK doesn't populate it yet)
  • Error message for legacy -32002 explicitly references SEP-2164 to guide SDK maintainers during migration
  • Adversarially verified to correctly FAIL on -32002, -32603, empty contents, and non-JSON-RPC errors

…code

Adds a draft-scoped scenario verifying servers return -32602 (Invalid
Params) for non-existent resources instead of the legacy -32002 or an
empty contents array. Includes a sep-2164.yaml traceability file mapping
each MUST/MUST NOT/SHOULD to its check per the proposed SEP-2484 process.
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 27, 2026

Open in StackBlitz

npx https://pkg.pr.new/@modelcontextprotocol/conformance@200

commit: 0701caa

@pja-ant pja-ant marked this pull request as ready for review March 27, 2026 23:43
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.

1 participant