Skip to content

Commit 210f9c5

Browse files
committed
Reconcile the interaction suite manifest with main
Reword the unsupported-dialect deferral to describe the validator-class selection the client now performs, and name the public ClientSession.validate_tool_result in the null-structured-content note. No-Verification-Needed: prose-only edit to the test manifest
1 parent 17c97e0 commit 210f9c5

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

tests/interaction/_requirements.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1578,7 +1578,7 @@ def __post_init__(self) -> None:
15781578
"The client rejects a wire structuredContent null as if it were missing: "
15791579
"CallToolResult.structured_content parses JSON null to None -- the same value as "
15801580
"field-absent, with no sentinel -- and the presence check in "
1581-
"ClientSession._validate_tool_result (src/mcp/client/session.py) reads is-None as "
1581+
"ClientSession.validate_tool_result (src/mcp/client/session.py) reads is-None as "
15821582
"'did not return structured content' and raises RuntimeError, so a conforming "
15831583
"null never reaches the schema validator. A fix needs an absent-vs-null sentinel "
15841584
"on the model before the presence check can tell the cases apart."
@@ -1616,8 +1616,10 @@ def __post_init__(self) -> None:
16161616
),
16171617
deferred=(
16181618
"Not implemented in the SDK: nothing inspects the declared $schema dialect -- "
1619-
"validate_tool_result (src/mcp/client/session.py) hands the advertised schema straight to "
1620-
"jsonschema.validate, so there is no SDK-authored unsupported-dialect rejection to pin."
1619+
"validate_tool_result (src/mcp/client/session.py) compiles the advertised schema with "
1620+
"whichever validator class jsonschema selects for its $schema (an unrecognized dialect "
1621+
"falls back to the latest supported draft), so there is no SDK-authored "
1622+
"unsupported-dialect rejection to pin."
16211623
),
16221624
),
16231625
"client:output-schema:skip-on-error": Requirement(

0 commit comments

Comments
 (0)