Review fixes for SEP-2243 conformance scenarios - #1

Open

pcarleton wants to merge 17 commits into

mikekistler:mdk/sep-2243-conformance-testsfrom

modelcontextprotocol:paulc/sep-2243-review-fixes

Commits on May 14, 2026

client/http-standard-headers: SKIPPED (not FAILURE) for unexercised methods; make getChecks() idempotent
pcarleton
committed
c6ec829
client/http-standard-headers: de-dup guard in checkMcpNameHeader
pcarleton
committed
7c71145
client/http-standard-headers: replace -> /\//g for slug generation
pcarleton
committed
15bf72e
client/http-custom-headers: base64 regex (.+) -> (.*) so empty string round-trips
pcarleton
committed
8bf9b2e
client/http-custom-headers: emit SUCCESS for no-mirror-unannotated
pcarleton
committed
94ea27f
client/http-custom-headers: drop redundant optional-present check
pcarleton
committed
d700b7c
server/http-standard-headers: defaultArgs must use real number/boolean, not strings
pcarleton
committed
216ad98
server/http-standard-headers: createAcceptanceCheck must also assert no JSON-RPC error in body
pcarleton
committed
100a824
use DRAFT_PROTOCOL_VERSION constant instead of 'DRAFT-2026-v1' literal
pcarleton
committed
3356974
sep-2243.yaml: spec-first rewrite + move to src/seps/
pcarleton
committed
16d24c8
add negative test for HttpStandardHeadersScenario
pcarleton
committed
cb482e8
server/http-standard-headers: severity fixes for whitespace + malformed-base64 tests
pcarleton
committed
607190e
server/http-standard-headers: split rejection check into status (MUST) + error-code (SHOULD/MUST)
pcarleton
committed
006330f
rename check ids to sep-2243-* prefix; kebab-case throughout
pcarleton
committed
937b25d
client: extract BaseHttpScenario to client/http-base.ts; both client scenarios extend it
pcarleton
committed
1d2d267
use req.setEncoding('utf8') instead of per-chunk Buffer.toString()
pcarleton
committed
65afa3d

Commits on May 15, 2026

server/http-standard-headers: malformed-base64 padding/chars back to FAILURE
pcarleton
committed
cfa8a80

Navigation Menu

Review fixes for SEP-2243 conformance scenarios - #1

Review fixes for SEP-2243 conformance scenarios#1
pcarleton wants to merge 17 commits into
mikekistler:mdk/sep-2243-conformance-testsfrom
modelcontextprotocol:paulc/sep-2243-review-fixes

Commits on May 14, 2026

client/http-standard-headers: SKIPPED (not FAILURE) for unexercised methods; make getChecks() idempotent

client/http-standard-headers: de-dup guard in checkMcpNameHeader

client/http-standard-headers: replace -> /\//g for slug generation

client/http-custom-headers: base64 regex (.+) -> (.*) so empty string round-trips

client/http-custom-headers: emit SUCCESS for no-mirror-unannotated

client/http-custom-headers: drop redundant optional-present check

server/http-standard-headers: defaultArgs must use real number/boolean, not strings

server/http-standard-headers: createAcceptanceCheck must also assert no JSON-RPC error in body

use DRAFT_PROTOCOL_VERSION constant instead of 'DRAFT-2026-v1' literal

sep-2243.yaml: spec-first rewrite + move to src/seps/

add negative test for HttpStandardHeadersScenario

server/http-standard-headers: severity fixes for whitespace + malformed-base64 tests

server/http-standard-headers: split rejection check into status (MUST) + error-code (SHOULD/MUST)

rename check ids to sep-2243-* prefix; kebab-case throughout

client: extract BaseHttpScenario to client/http-base.ts; both client scenarios extend it

use req.setEncoding('utf8') instead of per-chunk Buffer.toString()

Commits on May 15, 2026

server/http-standard-headers: malformed-base64 padding/chars back to FAILURE