Skip to content

test(conformance): bump referee to 0.2.0-alpha.7; drop server-sse-polling from baseline (conformance#366)#2366

Merged
felixweinberger merged 1 commit into
v2-2026-07-28from
fweinberger/conformance-alpha7-bump
Jun 25, 2026
Merged

test(conformance): bump referee to 0.2.0-alpha.7; drop server-sse-polling from baseline (conformance#366)#2366
felixweinberger merged 1 commit into
v2-2026-07-28from
fweinberger/conformance-alpha7-bump

Conversation

@felixweinberger

Copy link
Copy Markdown
Contributor

Bump the conformance referee pin to 0.2.0-alpha.7 and reconcile baselines.

Motivation and Context

alpha.7 carries conformance#366 (server-sse-polling now negotiates 2025-11-25 for the priming/retry SHOULD checks). With it, server-sse-polling passes 3/3 against the SDK.

How Has This Been Tested?

All 6 conformance legs exit 0: server 42/0, server:draft 81/0, server:extensions 136 pass / 27 expected (tasks-*), server:2026 110/0, client:all 438/0, client:2026 374/0.

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 or updated documentation as needed

Additional context

  • server-sse-polling removed from expected-failures.yaml (now passes).
  • tasks-status-notifications still referee-side SKIP at alpha.7; comment updated.
  • All 9 tasks-* baseline entries unchanged.

…ling from baseline

conformance#366 landed in alpha.7 — the referee now negotiates 2025-11-25
when probing the SSE priming/retry SHOULD checks, so server-sse-polling
passes against the SDK's version-gated priming and is removed from the
expected-failures baseline.

tasks-status-notifications is still SKIPped unconditionally by the
referee (0/0), so it stays intentionally absent. No other tasks-*
changes at alpha.7.

All six conformance legs (server, server:draft, server:extensions,
server:2026, client:all, client:2026) exit 0 against the new pin.
@felixweinberger felixweinberger requested a review from a team as a code owner June 25, 2026 10:45
@changeset-bot

changeset-bot Bot commented Jun 25, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 70d3f90

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Jun 25, 2026

Copy link
Copy Markdown

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/client@2366

@modelcontextprotocol/codemod

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/codemod@2366

@modelcontextprotocol/server

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/server@2366

@modelcontextprotocol/server-legacy

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/server-legacy@2366

@modelcontextprotocol/express

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/express@2366

@modelcontextprotocol/fastify

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/fastify@2366

@modelcontextprotocol/hono

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/hono@2366

@modelcontextprotocol/node

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/node@2366

commit: 70d3f90

@felixweinberger felixweinberger merged commit 65848f2 into v2-2026-07-28 Jun 25, 2026
16 checks passed
@felixweinberger felixweinberger deleted the fweinberger/conformance-alpha7-bump branch June 25, 2026 10:48

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM — straightforward conformance referee pin bump (alpha.6 → alpha.7) with the baseline reconciled per the documented procedure; test-only, no production code touched.

Extended reasoning...

Overview

This PR touches only the conformance test harness: test/conformance/package.json bumps the @modelcontextprotocol/conformance devDependency pin from 0.2.0-alpha.6 to 0.2.0-alpha.7, pnpm-lock.yaml is the matching lockfile update, and test/conformance/expected-failures.yaml removes the now-passing server-sse-polling entry and updates the alpha-version references in comments. No SDK package source code is modified.

Security risks

None. The change is a version pin of a test-only devDependency in a private workspace package plus a YAML baseline edit; nothing ships to consumers and no auth/crypto/transport code is touched.

Level of scrutiny

Low. This follows the exact reconciliation procedure documented at the top of expected-failures.yaml ("Newer conformance releases are adopted by deliberately bumping the package.json pin and reconciling this file in the same change"), mirroring the prior alpha.6 bump pattern. The baseline runner itself enforces correctness: stale entries fail CI, so an incorrect removal of server-sse-polling would surface immediately in the conformance legs.

Other factors

The PR description reports all six conformance legs exiting 0 against alpha.7, and the comment edits (alpha.6 → alpha.7, removal of the conformance#256/#366 hold note) are consistent with the diff. The missing changeset is fine since @modelcontextprotocol/test-conformance is private and unpublished. The bug-hunting system found no issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant