Repository navigation
deps(apps/web): bump postcss ^8.5.15 -> ^8.5.23 (6/6, fills gap after #103) - #107
martyy-code wants to merge 2 commits into
Conversation
The direct postcss range in apps/web/package.json was ^8.5.15, which kept pnpm's resolver at 8.5.15 for the wider dep graph even after PR #103 transitively bumped postcss in some sub-trees. Bumping the direct range to ^8.5.23 lifts resolution to 8.5.29 and clears all 10 postcss Dependabot alerts (4 distinct GHSAs). - apps/web/package.json: postcss ^8.5.15 -> ^8.5.23 - pnpm-lock.yaml: regenerated, postcss resolves to 8.5.29 Validated locally: - pnpm install --no-frozen-lockfile - pnpm build (turbo: 2 successful) - pnpm --filter errors test (146/146) apps/web/package-lock.json is not regenerated here; it is a historical artifact and requires clearing apps/web/node_modules first to avoid pnpm/npm cross-tooling breakage. Documented as follow-up in PR #105.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This PR (#107) is the 5th in the bulk-fix series. It raises the direct Series recap (open PRs):
Closed: #104 (sprintf-js dismiss without code change; the v1 override was cosmetic, dismissal is the right call). To come: #106 cherry-pick staging -> main after the above five land, so Dependabot re-scans main and the alert count drops. |
|
CI note: the failing Vercel check is unrelated to this PR. Root cause is a |
|
Reviewing the rationale: my prior "binding constraint" wording in #103 / here is wrong on the mechanism. The semver caret The real reason postcss stayed at 8.5.15 in the transitive graph was that This matters because it means a fresh Two concrete corrections to the rest of the series:
Net effect on the alert count is the same: postcss alerts clear once |
The previous wording said the direct `^8.5.15` range was the binding constraint. That was wrong: caret semver already permits 8.5.29, and no peer dependency was blocking the resolution. The real reason the lockfile still pinned vulnerable postcss versions was that it was a stale snapshot from the original commit; pnpm re-uses locked versions until the lockfile is regenerated. The regen alone would have cleared the alerts; this PR additionally raises the direct range to make the manifest explicit. Also drops the outdated follow-up note that pointed at PR #105, since PR #110 is the real cleanup for the npm lockfile.
Follow-up to #103. Bumps the direct postcss range in apps/web/package.json from ^8.5.15 to ^8.5.23. pnpm-lock.yaml now resolves postcss to 8.5.29.
The previous PR transitively lifted postcss in some dep chains, but the direct range floor at 8.5.15 kept the wider graph pinned there. Raising the direct range clears the remaining 10 postcss Dependabot alerts (4 distinct GHSAs).
Validated locally:
Note: apps/web/package-lock.json is not regenerated here (pnpm install does not write it; it is a historical artifact requiring a different workflow). Follow-up noted in PR #105.