fix(openapi-server): widen fastify-type-provider-zod peer range to ^6 || ^7 - #406
Conversation
… || ^7 The optional peer was pinned to ^6, so consumers on fastify-type-provider-zod 7 got an unmet-peer warning on every install. v7 is compatible (downstream Fastify 5 + Zod conformance suite stays green), so accept both majors. Closes #405
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe peerDependencies entry for fastify-type-provider-zod in packages/openapi-server/package.json was changed from requiring version ^6 to accepting either ^6 or ^7. ChangesPeer dependency range update
Estimated code review effort: 1 (Trivial) | ~2 minutes Related issues
Suggested labels: dependencies, patch Suggested reviewers: none Poem A rabbit hops through version fields so tame, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Frozen-lockfile CI rejected the peer-range change because the lockfile specifier (^6) no longer matched the manifest (^6 || ^7). Resolved version is unchanged (6.1.0).
Fallow audit reportFound 3 findings. Details
Generated by fallow. |
Fallow audit report0 inline findings selected for GitHub review. |
Summary
Closes #405.
@codewithagents/openapi-serverdeclared its optional peerfastify-type-provider-zodas^6. Consumers who moved to v7 got an unmet-peer warning on every install:Change
Widen the optional peer range to
^6 || ^7.Why it's safe
peerDependenciesMeta), so it never forces an install.fastify-type-provider-zod6.1.0 → 7.0.0 keeps the full conformance/contract suite green; only the peer warning remained.^6.1.0; dependabot PR chore(deps): bump fastify-type-provider-zod from 6.1.0 to 7.0.0 #403 handles the actual v7 bump + CI proof.Metadata-only change; no generated output or runtime code affected.
Summary by CodeRabbit