Skip to content

Commit 9d700ed

Browse files
authored
Merge pull request modelcontextprotocol#2710 from gsdv/fix/number-schema-integer-type
fix: NumberSchema min/max/default should be number, not integer
2 parents 4da3ae4 + cf0ba9d commit 9d700ed

3 files changed

Lines changed: 13 additions & 4 deletions

File tree

docs/specification/draft/changelog.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ Features listed here remain part of the specification but are scheduled for remo
5858

5959
## Other schema changes
6060

61-
N/A
61+
1. `schema.json` now correctly reflects that the Typescript definition of wminimum/maximum/default are `number`'s and not just `integers`. This was casued by using the generator using `--defaultNumberType integer` ([PR#2710](https://github.com/modelcontextprotocol/modelcontextprotocol/pull/2710)).
6262

6363
## Governance and process updates
6464

schema/draft/schema.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

schema/draft/schema.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2749,8 +2749,17 @@ export interface NumberSchema {
27492749
type: "number" | "integer";
27502750
title?: string;
27512751
description?: string;
2752+
/**
2753+
* @TJS-type number
2754+
*/
27522755
minimum?: number;
2756+
/**
2757+
* @TJS-type number
2758+
*/
27532759
maximum?: number;
2760+
/**
2761+
* @TJS-type number
2762+
*/
27542763
default?: number;
27552764
}
27562765

0 commit comments

Comments
 (0)