-
Notifications
You must be signed in to change notification settings - Fork 3
Update dependency zod to v4.0.17 #37
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
Conversation
|
Updated the Zod schema validation library from version 4.0.16 to 4.0.17 in both the PR Review and PR Summary packages to ensure compatibility with recent fixes and improvements. Walkthrough
Model: o4-mini | Prompt Tokens: 390 | Completion Tokens: 303 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Enjoy a friendly code review guided by AI. View these suggestions as helpful possibilities rather than fixed rules. Pick what resonates with your vision. You’re steering the project, and AI is here to offer supportive insights.
Model: o4-mini | Prompt Tokens: 832 | Completion Tokens: 2129
| "octokit": "5.0.3", | ||
| "parse-diff": "0.11.1", | ||
| "zod": "4.0.16" | ||
| "zod": "4.0.17" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider using a caret (^) or tilde (~) version range instead of pinning to an exact patch number. This allows non-breaking updates (e.g., minor and patch releases) to be pulled in automatically, which can include important security fixes and performance improvements.
Example:
{
"dependencies": {
"zod": "^4.0.17"
}
}| "mollitia": "0.2.0", | ||
| "octokit": "5.0.3", | ||
| "parse-diff": "0.11.1", | ||
| "zod": "4.0.16" | ||
| "zod": "4.0.17" | ||
| }, | ||
| "devDependencies": { | ||
| "@eslint/js": "9.33.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that multiple dependencies are repeated across both pr-review and pr-summary packages. To avoid duplication and ensure consistency, you could consolidate shared dependencies in a workspace root package.json (if you're using a mono repo) or extract them into a shared config. This reduces maintenance overhead and potential version drift.
Example root package.json using npm workspaces:
{
"private": true,
"workspaces": [
"pr-review",
"pr-summary"
],
"dependencies": {
"mollitia": "^0.2.0",
"octokit": "^5.0.3",
"parse-diff": "^0.11.1",
"zod": "^4.0.17"
}
}| "octokit": "5.0.3", | ||
| "parse-diff": "0.11.1", | ||
| "zod": "4.0.16" | ||
| "zod": "4.0.17" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly here, switch the exact patch version to a caret range to automatically receive non-breaking updates:
{
"dependencies": {
"zod": "^4.0.17"
}
}Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This PR contains the following updates:
4.0.16->4.0.17Release Notes
colinhacks/zod (zod)
v4.0.17Compare Source
Configuration
📅 Schedule: Branch creation - "every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.