Skip to content

test: enable supported draft SEP coverage#971

Merged
DaleSeo merged 3 commits into
mainfrom
test/enable-draft-conformance
Jul 10, 2026
Merged

test: enable supported draft SEP coverage#971
DaleSeo merged 3 commits into
mainfrom
test/enable-draft-conformance

Conversation

@DaleSeo

@DaleSeo DaleSeo commented Jul 10, 2026

Copy link
Copy Markdown
Member

Motivation and Context

In preparation for the 2026-07-28 support, this PR adds CI coverage for the draft SEP behavior already implemented.

How Has This Been Tested?

cargo build -p mcp-conformance

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

Testing

@github-actions github-actions Bot added T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes T-dependencies Dependencies related changes labels Jul 10, 2026
@DaleSeo DaleSeo marked this pull request as ready for review July 10, 2026 20:48
@DaleSeo DaleSeo requested a review from a team as a code owner July 10, 2026 20:48
alexhancock
alexhancock previously approved these changes Jul 10, 2026
Comment thread conformance/src/bin/client.rs Outdated
/// the high-level `call_tool` helper drive the `input_required` retry rounds.
/// The mock server verifies requestState echo, fresh JSON-RPC ids on retry,
/// state omission, isolation between tools, and the `resultType` default.
async fn run_mrtr_client(server_url: &str) -> anyhow::Result<()> {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thoughts about replacing _mrtr in all these with _stateless?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexhancock Makes sense! Updated.

@DaleSeo DaleSeo force-pushed the test/enable-draft-conformance branch from 7eb02f6 to 4113884 Compare July 10, 2026 21:00
@DaleSeo DaleSeo merged commit 3662d20 into main Jul 10, 2026
33 of 34 checks passed
@DaleSeo DaleSeo deleted the test/enable-draft-conformance branch July 10, 2026 21:21
@DaleSeo DaleSeo self-assigned this Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2026-07-28 T-CI Changes to CI/CD workflows and configuration T-config Configuration file changes T-dependencies Dependencies related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants