stt/kong-customers-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(mcp): translate OpenAPI 3.0 nullable into JSON Schema

Browse Source 
The response schemas added yesterday made every paginated list tool fail. The
plugin's fields are documented as OpenAPI format and validated with a JSON
Schema validator, and `nullable: true` is not a JSON Schema keyword — so it is
ignored, `type: string` is enforced, and `previous: null` on the first page is
rejected as *null is not of type "string"*.

The failure lands after the backend has already done the work, surfaces to the
caller as a bare tool retry exhaustion, and appears nowhere in the gateway's own
logs. Found only because an orchestrating agent asked a question that paged.

The generator now rewrites `nullable: true` into `type: [T, "null"]` wherever it
inlines a schema, request bodies included. A 3.1 spec needs none of this.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Stéphane Tailland
2026-08-29 12:19:31 +02:00
parent 63d2d90107
commit 865a7376e3
1 changed files with 51 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files
kong/mcp/ai-mcp-proxy.yaml
+51 -34
View File
@@ -58,13 +58,15 @@ config:
type: integer
example: 123
next:
type: string
nullable: true
type:
- string
- 'null'
format: uri
example: http://api.example.org/accounts/?page=4
previous:
type: string
nullable: true
type:
- string
- 'null'
format: uri
example: http://api.example.org/accounts/?page=2
results:
@@ -93,33 +95,38 @@ config:
type: string
maxLength: 50
addressline2:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
city:
type: string
maxLength: 50
state:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
postalcode:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 15
country:
type: string
maxLength: 50
creditlimit:
type: string
type:
- string
- 'null'
format: decimal
pattern: ^-?\d{0,8}(?:\.\d{0,2})?$
nullable: true
salesrepemployeenumber:
type: integer
type:
- integer
- 'null'
maximum: 2147483647
minimum: -2147483648
nullable: true
required:
- addressline1
- city
@@ -183,33 +190,38 @@ config:
type: string
maxLength: 50
addressline2:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
city:
type: string
maxLength: 50
state:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
postalcode:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 15
country:
type: string
maxLength: 50
creditlimit:
type: string
type:
- string
- 'null'
format: decimal
pattern: ^-?\d{0,8}(?:\.\d{0,2})?$
nullable: true
salesrepemployeenumber:
type: integer
type:
- integer
- 'null'
maximum: 2147483647
minimum: -2147483648
nullable: true
required:
- addressline1
- city
@@ -304,33 +316,38 @@ config:
type: string
maxLength: 50
addressline2:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
city:
type: string
maxLength: 50
state:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 50
postalcode:
type: string
nullable: true
type:
- string
- 'null'
maxLength: 15
country:
type: string
maxLength: 50
creditlimit:
type: string
type:
- string
- 'null'
format: decimal
pattern: ^-?\d{0,8}(?:\.\d{0,2})?$
nullable: true
salesrepemployeenumber:
type: integer
type:
- integer
- 'null'
maximum: 2147483647
minimum: -2147483648
nullable: true
required:
- addressline1
- city