| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
…baseline Modernizes the conformance CI to match the typescript-sdk pattern: - Pin @modelcontextprotocol/conformance via a single workflow-level CONFORMANCE_VERSION env var (was 0.1.10/0.1.13, drifted between jobs). - Add .github/actions/conformance/expected-failures.yml so the runner exits 0 on known-failing scenarios and 1 on regressions or stale entries. Client baseline is 16 scenarios grouped by SEP; server active suite is fully green. - Drop continue-on-error from both jobs so conformance actually gates. - Harden run-server.sh with a port-conflict guard, dead-process check, and curl --max-time, mirroring typescript-sdk's wrapper. The server --suite draft step (2026-07-28 scenarios) is a follow-up; the baseline file already has a placeholder for it. Supersedes #1921.
| run: >- | ||
| npx --yes @modelcontextprotocol/conformance@"$CONFORMANCE_VERSION" client | ||
| --command 'uv run --frozen python .github/actions/conformance/client.py' | ||
| --suite all |
There was a problem hiding this comment.
I'm actually not sure all is the correct one, counterintuitively - I believe it has 2 scenarios that aren't actually load bearing (I think something related to old client auth cases from a pre-2025-11-25 spec version)
Sorry, something went wrong.
There was a problem hiding this comment.
Land it for now though, we can fix if we end up with remaining failures that aren't covered by any SEPs
Sorry, something went wrong.
| - auth/scope-step-up | ||
| # SEP-990 (enterprise-managed authorization extension): no fixture handler / | ||
| # client support for the token-exchange + JWT bearer flow. | ||
| - auth/enterprise-managed-authorization |
There was a problem hiding this comment.
Do we not have EMA in Python at all? Do we need to build this into Python potentially @pcarleton?
Sorry, something went wrong.
There was a problem hiding this comment.
yea it does not apparently
Sorry, something went wrong.
|
This pull request is included in pre-release v2.0.0a2 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Modernizes the conformance CI to match the typescript-sdk pattern so the job actually gates and the failure set burns down per SEP. Supersedes #1921 (the composite-action approach) — pinning the npm package directly is what typescript-sdk settled on.
Motivation and Context
The conformance jobs were pinned to 0.1.10/0.1.13 (drifted between server and client), ran with continue-on-error: true so they never gated, and had no baseline file — failures were silently ignored. The conformance harness has since gained --expected-failures (#113) and the 0.2.0-alpha line with version-aware scenario selection.
Changes
The server --suite draft step (2026-07-28 scenarios) is a follow-up PR; the baseline file already has a server: placeholder for it.
How Has This Been Tested?
Run locally against the conformance harness at 0.2.0-alpha.3:
Breaking Changes
None for SDK users. For contributors: the conformance jobs now fail on regressions instead of silently passing. run-server.sh requires CONFORMANCE_VERSION to be set when run locally (the error message points at the workflow pin).
Types of changes
Checklist
Additional context
AI Disclaimer