Skip to content

Add constraints to zod peer#3085

Merged
RobinTail merged 1 commit intomasterfrom
contraint-zod-peer
Nov 24, 2025
Merged

Add constraints to zod peer#3085
RobinTail merged 1 commit intomasterfrom
contraint-zod-peer

Conversation

@RobinTail
Copy link
Copy Markdown
Owner

@RobinTail RobinTail commented Nov 24, 2025 •
edited by coderabbitai Bot
Loading

This should simplify elevating the requirements on zod version in the next major #2939 incl. scope packages (Zod plugin)

Summary by CodeRabbit

  • Chores
    • Refactored zod dependency management across packages to use a centralized catalog system for improved version consistency and maintainability.

✏️ Tip: You can customize this high-level summary in your review settings.

@RobinTail RobinTail added the dependencies Pull requests that update a dependency file label Nov 24, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Nov 24, 2025 •
edited
Loading

Walkthrough

These changes centralize zod peer dependency versioning in a pnpm monorepo by converting hardcoded semantic version constraints to catalog references, with version specifications managed through pnpm-workspace.yaml.

Changes

Cohort / File(s) Summary
Peer dependency catalog references
express-zod-api/package.json, zod-plugin/package.json		 Updated zod peer dependencies from concrete version ^4.0.0 to catalog placeholder catalog:peer
Workspace catalog definition
pnpm-workspace.yaml		 Added new zod entry under catalogs.prod with version ^4.0.0 to define the peer catalog resolution

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Homogeneous changes across package files following a consistent pattern
  • No logic alterations, purely dependency configuration
  • Coordination between workspace catalog and package.json references should be verified to ensure catalog entry matches all peer dependency references

Possibly related PRs

  • #2747: Directly implements the same catalog:peer pattern for zod in express-zod-api and workspace configuration
  • #2782: Modifies the same zod peer dependency declarations, potentially reverting or adjusting catalog:peer usage

Poem

🐰 A rabbit's tale of versions tight,
Catalogs gathered, dependencies right,
No more scattered specs to roam,
Workspace brings them all back home! 🌟

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add constraints to zod peer' accurately describes the main change: updating zod peer dependency specifications to use a catalog-based constraint system across multiple package.json files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch contraint-zod-peer

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

RobinTail
RobinTail commented Nov 24, 2025
View reviewed changes
Copy link
Copy Markdown
Owner Author

@RobinTail RobinTail left a comment

Choose a reason for hiding this comment

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

ok

@coveralls-official
Copy link
Copy Markdown

coveralls-official Bot commented Nov 24, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling bcad297 on contraint-zod-peer
into c2483f7 on master.

@RobinTail RobinTail merged commit 7a553e4 into master Nov 24, 2025
12 of 13 checks passed
@RobinTail RobinTail deleted the contraint-zod-peer branch November 24, 2025 08:09
@coderabbitai coderabbitai Bot mentioned this pull request Jan 2, 2026
Merged
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

dependencies Pull requests that update a dependency file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@RobinTail