Skip to content

Conversation

@DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Aug 25, 2025

[0.7.3] - 2025-08-25

🐛 Fixes

fix: generate openAI-compatible json schemas for list types - @DaleSeo PR #272

The MCP server is generating JSON schemas that don't match OpenAI's function calling specification. It puts oneOf at the array level instead of using items to define the JSON schemas for the GraphQL list types. While some other LLMs are more flexible about this, it technically violates the JSON Schema specification that OpenAI strictly follows.

This PR updates the list type handling logic to move oneOf inside items for GraphQL list types.

@DaleSeo DaleSeo self-assigned this Aug 25, 2025
@DaleSeo DaleSeo requested review from a team as code owners August 25, 2025 19:38
@apollo-librarian
Copy link

apollo-librarian bot commented Aug 25, 2025

✅ Docs preview ready

The preview is ready to be viewed. View the preview

File Changes

0 new, 1 changed, 0 removed
* (developer-tools)/apollo-mcp-server/(latest)/install.mdx

Build ID: b52d1ae07e19a3e907d24d9f
Build Logs: View logs

URL: https://www.apollographql.com/docs/deploy-preview/b52d1ae07e19a3e907d24d9f

@github-actions
Copy link

Changeset file missing for PR

All changes should include an associated changeset file.
Please refer to README for more information on generating changesets.

Copy link
Contributor

@swcollard swcollard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies, I know we are in the middle of reconsidering branching strategies but I don't see the actual code change of the fix in this diff? Want to double check that is correct?

@DaleSeo
Copy link
Contributor Author

DaleSeo commented Aug 25, 2025

@swcollard The fix was already merged in PR #272.

@swcollard
Copy link
Contributor

@swcollard The fix was already merged in PR #272.

And that is already in main, perfect. 🚢

@DaleSeo DaleSeo merged commit 73dc0cd into main Aug 26, 2025
7 of 8 checks passed
@DaleSeo DaleSeo deleted the 0.7.3 branch August 26, 2025 12:08
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.

3 participants