Skip to content

Conversation

chi2liu
Copy link
Contributor

@chi2liu chi2liu commented Aug 9, 2025

  • Merge all properties from allOf sub-schemas when allOf appears at requestBody top level
  • Properly combine required fields from all sub-schemas
  • Add comprehensive tests for allOf merging scenarios
  • Ensures all parameters from all allOf schemas are included in tool definitions

Fixes #1378

- Merge all properties from allOf sub-schemas when allOf appears at requestBody top level
- Properly combine required fields from all sub-schemas
- Add comprehensive tests for allOf merging scenarios
- Ensures all parameters from all allOf schemas are included in tool definitions

Fixes jlowin#1378
@github-actions github-actions bot added tests openapi Related to OpenAPI integration, parsing, or code generation features. labels Aug 9, 2025
@jlowin jlowin added the bug Something isn't working. Reports of errors, unexpected behavior, or broken functionality. label Aug 9, 2025
@jlowin
Copy link
Owner

jlowin commented Aug 9, 2025

Thank you!

@jlowin jlowin merged commit 5fe3287 into jlowin:main Aug 9, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working. Reports of errors, unexpected behavior, or broken functionality. openapi Related to OpenAPI integration, parsing, or code generation features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect tool inputSchema is generated from Openapi spec with top level allOf

2 participants