Skip to content

Conversation

lukad
Copy link
Collaborator

@lukad lukad commented Jun 30, 2025

As it turns out, {"oneOf": []} should match any value. Previously we would just return a validation error in this case, but I think that's not correct. I think the spec can be interpreted either way but matching any value seems more practical to me and this is also how v0 handles it.

Copy link
Collaborator

@antoniocapelo antoniocapelo left a comment

Choose a reason for hiding this comment

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

Simple change with tests, looks good to me 👍

@lukad lukad merged commit 03bfb78 into main Jun 30, 2025
2 checks passed
@lukad lukad deleted the fix-empty-oneof-v1 branch June 30, 2025 13:15
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.

2 participants