Skip to content

feat!: use zod v4 #10922

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.

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat!: use zod v4 #10922

wants to merge 2 commits into from

Conversation

almeidx
Copy link
Member

@almeidx almeidx commented Jun 6, 2025

Updated builders to use zod v4 which is actually v3.25
Replaced zod-validation-error with z.prettifyError()

https://zod.dev/v4

Copy link

vercel bot commented Jun 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
discord-js ⬜️ Ignored (Inspect) Visit Preview Jun 10, 2025 9:45am
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Jun 10, 2025 9:45am

@almeidx almeidx changed the title Feat/zod 4 feat: use zod v4 Jun 6, 2025
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

Attention: Patch coverage is 94.48819% with 7 lines in your changes missing coverage. Please review.

Project coverage is 42.53%. Comparing base (d40ceed) to head (0ce95dc).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
packages/builders/src/components/Assertions.ts 80.55% 5 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #10922      +/-   ##
==========================================
- Coverage   42.69%   42.53%   -0.17%     
==========================================
  Files         262      263       +1     
  Lines       16086    16061      -25     
  Branches     1545     1542       -3     
==========================================
- Hits         6868     6831      -37     
- Misses       9206     9218      +12     
  Partials       12       12              
Flag Coverage Δ
builders 79.97% <94.48%> (-0.56%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-project-automation github-project-automation bot moved this from Todo to Review in Progress in discord.js Jun 6, 2025
@colinhacks
Copy link
Contributor

Those errors are fixed, upgrade to 3.25.56 👍

Qjuh
Qjuh previously requested changes Jun 6, 2025
@almeidx almeidx dismissed stale reviews from Qjuh and vladfrangu June 6, 2025 21:04

Resolved

@almeidx almeidx requested review from Qjuh and vladfrangu June 6, 2025 21:04
feat: validation error class

Co-authored-by: Qjuh <76154676+Qjuh@users.noreply.github.com>
@almeidx almeidx requested review from vladfrangu and Jiralite June 10, 2025 09:46
@Jiralite Jiralite changed the title feat: use zod v4 feat!: use zod v4 Jun 10, 2025
@Jiralite Jiralite added this to the builders 2.0.0 milestone Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Review in Progress
Development

Successfully merging this pull request may close these issues.

6 participants