Skip to content

doc: Add example of testing against TS SDK#237

Merged
pcarleton merged 1 commit intomodelcontextprotocol:mainfrom
nbarbettini:more-sdk-instructions
Apr 24, 2026
Merged

doc: Add example of testing against TS SDK#237
pcarleton merged 1 commit intomodelcontextprotocol:mainfrom
nbarbettini:more-sdk-instructions

Conversation

@nbarbettini
Copy link
Copy Markdown
Contributor

The CONTRIBUTING.md doc says new scenarios must be tested against a real SDK, but doesn't show an example of doing this. Reformatted the examples section and added a quick example of testing against the latest TS SDK.

Motivation and Context

Clearer instructions for a new contributor.

How Has This Been Tested?

I put myself in the shoes of a new contributor. 🙂

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

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 20, 2026

Open in StackBlitz

npx https://pkg.pr.new/@modelcontextprotocol/conformance@237

commit: 3e7c9cc

Copy link
Copy Markdown
Member

@pcarleton pcarleton left a comment

Choose a reason for hiding this comment

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

thanks for this!

might be worth us adding some shell scripts to make this super easy. e.g. even make it easy to run at different git commits / branches, since often the passing scenario impl is still on a branch

@pcarleton pcarleton merged commit 22b4dce into modelcontextprotocol:main Apr 24, 2026
4 checks passed
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