Skip to content

fix(validation): silence unknown-format warnings in default AJV validator - #2856

Open
senorMk wants to merge 1 commit into
modelcontextprotocol:v1.xfrom
senorMk:quiet-unknown-formats
Open

senorMk wants to merge 1 commit into
modelcontextprotocol:v1.xfrom
senorMk:quiet-unknown-formats

Conversation

@senorMk

@senorMk senorMk commented Sep 24, 2026

Copy link
Copy Markdown

Fixes #2855

Servers using custom formats (e.g. google-duration) trigger one Ajv unknown-format warning per subschema when listTools compiles output validators. The default instance now drops only that noise via a custom logger. Known-format validation and all other warnings are unchanged.

Tests: new test/validation/ajv-unknown-format.test.ts (2 tests), existing validation suite passes (65 tests), eslint clean.

@senorMk
senorMk requested a review from a team as a code owner September 24, 2026 15:26
@pkg-pr-new

pkg-pr-new Bot commented Sep 24, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@modelcontextprotocol/sdk@2856

commit: 389b6d7

This branch has not been deployed

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

Labels

v1 Issues / PRs related to v1.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant