Skip to content

Improve JSON Schema #4622

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Improve JSON Schema #4622

wants to merge 6 commits into from

Conversation

colinhacks
Copy link
Owner

@colinhacks colinhacks commented Jun 5, 2025 •
edited
Loading

  • Do not include id in resulting schema. AJV throws an error when it encounters id in a Draft 7 or later schema
  • Improve override: now executes before defs are extracted so there should be no $refs

@colinhacks colinhacks changed the title misc 1 Improve JSON Schema Jun 5, 2025
@colinhacks colinhacks marked this pull request as draft June 5, 2025 00:16
@reteps reteps mentioned this pull request Aug 22, 2025
Open
@reteps
Copy link

reteps commented Sep 19, 2025

@colinhacks Any progress here? This is blocking our migration to Zod 4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@colinhacks @reteps