Skip to content

Commit be9b05b

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 7e0a172 commit be9b05b

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
@@ -1580,7 +1580,7 @@ def __post_init__(self) -> None:
15801580
"The client rejects a wire structuredContent null as if it were missing: "
15811581
"CallToolResult.structured_content parses JSON null to None -- the same value as "
15821582
"field-absent, with no sentinel -- and the presence check in "
1583-
"ClientSession._validate_tool_result (src/mcp/client/session.py) reads is-None as "
1583+
"ClientSession.validate_tool_result (src/mcp/client/session.py) reads is-None as "
15841584
"'did not return structured content' and raises RuntimeError, so a conforming "
15851585
"null never reaches the schema validator. A fix needs an absent-vs-null sentinel "
15861586
"on the model before the presence check can tell the cases apart."
@@ -1618,8 +1618,10 @@ def __post_init__(self) -> None:
16181618
),
16191619
deferred=(
16201620
"Not implemented in the SDK: nothing inspects the declared $schema dialect -- "
1621-
"validate_tool_result (src/mcp/client/session.py) hands the advertised schema straight to "
1622-
"jsonschema.validate, so there is no SDK-authored unsupported-dialect rejection to pin."
1621+
"validate_tool_result (src/mcp/client/session.py) compiles the advertised schema with "
1622+
"whichever validator class jsonschema selects for its $schema (an unrecognized dialect "
1623+
"falls back to the latest supported draft), so there is no SDK-authored "
1624+
"unsupported-dialect rejection to pin."
16231625
),
16241626
),
16251627
"client:output-schema:skip-on-error": Requirement(

0 commit comments

Comments
 (0)