FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: use conformance repo's composite GitHub Action by maxisbey · Pull Request #1921 · modelcontextprotocol/python-sdk · GitHub

ci: use conformance repo's composite GitHub Action - #1921

Closed
maxisbey wants to merge 5 commits into
mainfrom
use-conformance-action
Closed

ci: use conformance repo's composite GitHub Action#1921
maxisbey wants to merge 5 commits into
mainfrom
use-conformance-action

Conversation

Copy link
Copy Markdown
Contributor

Summary

Replaces the custom conformance test scripts with the composite GitHub Action from modelcontextprotocol/conformance. This is a proof-of-concept to validate the action works in CI before tagging a release.

Depends on: modelcontextprotocol/conformance#113

Changes

  • Server job: Inlines the server startup, replaces run-server.sh's npx call with the action
  • Client job: Replaces the inline npx call with the action
  • Removes actions/setup-node steps (handled by the action)
  • Removes continue-on-error: true (Python SDK passes all tests)

Before

- uses: actions/setup-node@...
- run: npx @modelcontextprotocol/conformance@0.1.10 server --url ...

After

- uses: modelcontextprotocol/conformance@ab789d6
  with:
    mode: server
    url: http://localhost:3001/mcp

Once the conformance repo tags a release, the ref will be updated to a proper version tag.

Replace custom npx scripts with the composite action from
modelcontextprotocol/conformance. This simplifies the workflow by:

- Removing the manual actions/setup-node step (handled by the action)
- Replacing run-server.sh's npx call with the action
- Replacing the inline npx call for client tests with the action
- Removing continue-on-error (Python SDK passes all tests)

References conformance repo at commit ab789d6 (expected-failures branch)
until a tagged release is available.
maxisbey marked this pull request as ready for review January 21, 2026 16:28
maxisbey marked this pull request as draft January 21, 2026 16:30
Comment thread conformance-baseline.yml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I don't think it should go on the root directory, but besides that, all good.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

where do you think it should go?

Copy link
Copy Markdown
Contributor Author

will wait to merge this until the conformance repo does a release

Kludex commented Jun 2, 2026

Copy link
Copy Markdown
Member

Closing as stale.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL