Skip to content

fix: pass SEP-2243 client conformance#1012

Open
DaleSeo wants to merge 1 commit into
mainfrom
fix/sep-2243-client-conformance
Open

fix: pass SEP-2243 client conformance#1012
DaleSeo wants to merge 1 commit into
mainfrom
fix/sep-2243-client-conformance

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jul 18, 2026

Copy link
Copy Markdown
Member

Fixes #978

Motivation and Context

Updates the Rust conformance client to pass the SEP-2243 standard, custom, and invalid tool-header scenarios. Header scenarios now negotiate the draft protocol through discovery, consume the conformance runner's toolCalls context, and reject malformed x-mcp-header tools before they reach callers.

The client previously initialized before draft request metadata was available, ignored camelCase tool-call context, and left invalid header-annotated tools visible even after rejecting them from the header cache.

How Has This Been Tested?

The conformance runner passes all checks for http-standard-headers (3/3), http-custom-headers (18/18), and http-invalid-tool-headers (11/11).

Breaking Changes

None.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

@github-actions github-actions Bot added T-documentation Documentation improvements T-core Core library changes T-transport Transport layer changes labels Jul 18, 2026
@DaleSeo
DaleSeo force-pushed the fix/sep-2243-client-conformance branch from 1d70824 to 619a260 Compare July 18, 2026 14:42
@github-actions github-actions Bot removed the T-documentation Documentation improvements label Jul 18, 2026
@DaleSeo DaleSeo linked an issue Jul 18, 2026 that may be closed by this pull request
@DaleSeo DaleSeo self-assigned this Jul 18, 2026
@DaleSeo
DaleSeo marked this pull request as ready for review July 18, 2026 14:56
@DaleSeo
DaleSeo requested a review from a team as a code owner July 18, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-core Core library changes T-transport Transport layer changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement SEP-2243: standardized & custom HTTP headers (client)

1 participant