Skip to content

Ensure registry API implements the reference openapi schema - #568

Merged
rdimitrov merged 2 commits into
modelcontextprotocol:mainfrom
rdimitrov:add-schema-test
Sep 29, 2025
Merged

rdimitrov merged 2 commits into
modelcontextprotocol:mainfrom
rdimitrov:add-schema-test

Conversation

@rdimitrov

@rdimitrov rdimitrov commented Sep 29, 2025 •

Copy link
Copy Markdown
Member

Motivation and Context

The following PR adds a simple unit test to ensure the served API handlers are a superset of the reference OpenAPI schema. We can extend/update this later to have a more thorough check, i.e. inputs, outputs, return codes, etc.

How Has This Been Tested?

Breaking Changes

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

Additional context

Fixes: #567

@rdimitrov
rdimitrov enabled auto-merge (squash) September 29, 2025 15:47

@tadasant tadasant left a comment

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.

Thank you!

@rdimitrov
rdimitrov merged commit 3762542 into modelcontextprotocol:main Sep 29, 2025
2 checks passed
@rdimitrov
rdimitrov deleted the add-schema-test branch September 29, 2025 21:52
slimslenderslacks pushed a commit to slimslenderslacks/registry that referenced this pull request Dec 18, 2025
…textprotocol#568)

<!-- Provide a brief summary of your changes -->

## Motivation and Context
<!-- Why is this change needed? What problem does it solve? -->
The following PR adds a simple unit test to ensure the served API
handlers are a superset of the reference OpenAPI schema.

## How Has This Been Tested?
<!-- Have you tested this in a real application? Which scenarios were
tested? -->

## Breaking Changes
<!-- Will users need to update their code or configurations? -->

## Types of changes
<!-- What types of changes does your code introduce? Put an `x` in all
the boxes that apply: -->
- [ ] 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
<!-- Go over all the following points, and put an `x` in all the boxes
that apply. -->
- [ ] I have read the [MCP
Documentation](https://modelcontextprotocol.io)
- [ ] 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

## Additional context
<!-- Add any other context, implementation notes, or design decisions
-->
Fixes: modelcontextprotocol#567

---------

Signed-off-by: Radoslav Dimitrov <[email protected]>
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.

Add tests to keep API schemas in sync

2 participants