Skip to content

Conversation

Abdillah
Copy link
Contributor

@Abdillah Abdillah commented Aug 12, 2025

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Allowing Zod v4 transition.

Additional context & links

Nope

@Abdillah Abdillah requested a review from a team as a code owner August 12, 2025 23:15
@ArthurLobopro
Copy link

LGTM

@oEmanuelFirmino
Copy link

Approved

Copy link

@Luiz-Melo-FSAI Luiz-Melo-FSAI left a comment

Choose a reason for hiding this comment

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

This seems to be right.

@ArthurLobopro
Copy link

@Abdillah unfortunately there are some branch conflicts because of the newest version. Could you solve these conflicts before the official reviewer take a look?

@alexgleason
Copy link

Please add zod4 support. It has native JSON Schema conversions which will make it easier to call tools with the openai package. It's currently not possible to install zod4 alongside openai, due to the zod version in openai's package.json.

@Abdillah
Copy link
Contributor Author

Okay @ArthurLobopro, will try to resolve it

@Abdillah
Copy link
Contributor Author

@alexgleason That's what this PR do. Once this merged, you can use openai js along with zod v4. But the openai still uses v3.

@mduchev
Copy link

mduchev commented Sep 25, 2025

Any plans to release this soon?

@NegarYousefian
Copy link

Hi @Abdillah Do you have any plans to release the fix? Thank you 🙏

@RobertCraigie
Copy link
Collaborator

Merged through #1658, sorry for the delay!

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.

8 participants