Skip to content

🔧 Use tilde ranges for security dependency overrides#6888

Merged
dubzzz merged 1 commit into
mainfrom
claude/update-pnpm-version-overrides-VAG2K
Apr 18, 2026
Merged

🔧 Use tilde ranges for security dependency overrides#6888
dubzzz merged 1 commit into
mainfrom
claude/update-pnpm-version-overrides-VAG2K

Conversation

@dubzzz

@dubzzz dubzzz commented Apr 18, 2026

Copy link
Copy Markdown
Owner

Description

Updates pnpm dependency overrides to use tilde (~) version ranges instead of exact versions for security-related packages. This allows patch-level updates while maintaining the minimum security threshold for:

  • tmp@~0.2.4
  • serialize-javascript@~7.0.5
  • follow-redirects@~1.16.0
  • lodash-es@~4.18.0
  • dompurify@~3.4.0

This change enables automatic patch updates for these critical security dependencies while preventing unintended minor or major version bumps that could introduce breaking changes.

Checklist

  • I have a full understanding of every line in this PR
  • I flagged the impact of my change (minor / patch / major) either by running pnpm run bump or by following the instructions from the changeset bot
  • I kept this PR focused on a single concern and did not bundle unrelated changes
  • I followed the gitmoji specification for the name of the PR
  • I added relevant tests and they would have failed without my PR (when applicable)

https://claude.ai/code/session_01EoNkHTqq5eiJyDnwSGmmMQ

Switch pnpm overrides from pinned versions to tilde ranges so security
patch releases can be picked up without manually bumping each entry.
@changeset-bot

changeset-bot Bot commented Apr 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c5d1659

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@pkg-pr-new

pkg-pr-new Bot commented Apr 18, 2026

Copy link
Copy Markdown
@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6888

fast-check

npm i https://pkg.pr.new/fast-check@6888

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6888

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6888

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6888

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6888

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6888

commit: c5d1659

@dubzzz
dubzzz enabled auto-merge (squash) April 18, 2026 19:32
@dubzzz
dubzzz merged commit fa411a0 into main Apr 18, 2026
50 checks passed
@dubzzz
dubzzz deleted the claude/update-pnpm-version-overrides-VAG2K branch April 18, 2026 19:35
@codecov

codecov Bot commented Apr 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.72%. Comparing base (7b71cb0) to head (c5d1659).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6888      +/-   ##
==========================================
- Coverage   94.75%   94.72%   -0.04%     
==========================================
  Files         211      211              
  Lines        5778     5778              
  Branches     1528     1527       -1     
==========================================
- Hits         5475     5473       -2     
- Misses        295      297       +2     
  Partials        8        8              
Flag Coverage Δ
tests 94.72% <ø> (-0.04%) ⬇️

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.

dubzzz added a commit that referenced this pull request Apr 18, 2026
## Description

Configure Renovate so it stops opening PRs that change the
`pnpm.overrides` entries in the root `package.json`.

These overrides are intentionally pinned via tilde ranges (see #6888) so
that patch-level updates flow naturally through `pnpm-lock.yaml`
(already covered by `lockFileMaintenance`), without rewriting the
constraint string in `package.json` on every patch release. Renovate
currently still opens PRs (e.g. #6884) to bump those override
constraints — this change disables that behavior.

The new `packageRule` matches the `pnpm.overrides` `depType` on the root
`package.json` and disables updates for it. Lockfile maintenance keeps
picking up resolvable patch versions automatically.

## Checklist

— _Don't delete this checklist and make sure you do the following before
opening the PR_

- [x] I have a full understanding of every line in this PR — whether the
code was hand-written, AI-generated, copied from external sources or
produced by any other tool
- [ ] I flagged the impact of my change (minor / patch / major) either
by running `pnpm run bump` or by following the instructions from the
changeset bot
- [x] I kept this PR focused on a single concern and did not bundle
unrelated changes
- [x] I followed the [gitmoji](https://gitmoji.dev/) specification for
the name of the PR, including the package scope (e.g. `🐛(vitest)
Something...`) when the change targets a package other than `fast-check`
- [ ] I added relevant tests and they would have failed without my PR
(when applicable)

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Nicolas DUBIEN <[email protected]>
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.

2 participants