chore(deps): override postcss to >=8.5.10 to close XSS advisory in next bundle#79
Conversation
…xt bundle Adds a scoped override to website/package.json so every transitive postcss resolves to ^8.5.10. Previously next 15.5.18 declared postcss 8.4.31 exactly, leaving a duplicate vulnerable copy at node_modules/next/node_modules/postcss even after the workspace-level postcss was bumped. The override deduplicates the tree to a single postcss copy at 8.5.14 and closes Dependabot alert #29 (GHSA-qx2v-qp2m-jg93). Lockfile was regenerated from scratch (rm package-lock.json && npm install) because npm could not retroactively swap the bundled exact-pinned version in the existing lockfile. All other transitive bumps that came along are inside their declared semver ranges. `npx next build` succeeds against the regenerated tree. Verified: - npm audit -> 0 vulnerabilities - npm ls postcss -> single deduped copy at 8.5.14 - npx next build -> success, all routes prerender
|
Note
|
| Layer / File(s) | Summary |
|---|---|
Dependency Override website/package.json |
Added overrides block constraining next's postcss transitive dependency to ^8.5.10. |
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title accurately describes the main change: a dependency override to upgrade postcss and address a security vulnerability (XSS advisory) in the next bundle. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
fix/postcss-overrides
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
Deploying attestix with
|
| Latest commit: |
fb8c88a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://0940628d.attestix.pages.dev |
| Branch Preview URL: | https://fix-postcss-overrides.attestix.pages.dev |
Summary
Adds a scoped override to
website/package.jsonso every transitivepostcssresolves to^8.5.10:Previously
next@15.5.18pinspostcssto exact8.4.31, leaving a duplicate vulnerable copy atnode_modules/next/node_modules/postcsseven after the workspace-levelpostcsswas bumped. The override deduplicates the tree to a singlepostcss@8.5.14and closes Dependabot alert #29 (GHSA-qx2v-qp2m-jg93).package-lock.jsonwas regenerated from scratch (rm package-lock.json && npm install --legacy-peer-deps) because npm cannot retroactively swap the exact-pinned bundled version in the existing lockfile. All other transitive bumps that came along are inside their declared semver ranges inpackage.json.Test plan
npm audit-> 0 vulnerabilitiesnpm ls postcss-> single deduped copy at 8.5.14npx next build-> succeeds, all routes prerenderSummary by CodeRabbit