chore(deps): fix pnpm audit findings (undici, postcss, nanoid) - #194
Draft
mcalthrop wants to merge 1 commit into
Draft
chore(deps): fix pnpm audit findings (undici, postcss, nanoid)#194mcalthrop wants to merge 1 commit into
mcalthrop wants to merge 1 commit into
Conversation
All seven findings were transitive-only, so each is resolved with a comparator-scoped override in pnpm-workspace.yaml: - undici: the existing `undici@>=6.0.0 <6.27.0` override pinned 8.8.0, which is itself vulnerable. Replaced with `undici@<8.9.0: ^8.9.0`, which covers both the original 6.x range and the vulnerable 8.0-8.8 line. Reached via @actions/core and @actions/github. - postcss: `postcss@<8.5.23: ^8.5.23`, reached via vite. - nanoid: `nanoid@<3.3.17: ^3.3.17`, reached via postcss. All bumps stay within the installed major, so no breaking changes. `pnpm audit --audit-level=low` is clean and the test suite passes. Refs GHSA-4cwx-7wf7-3272, GHSA-8xcm-r25x-g524, GHSA-m8rv-5g2x-5cg5, GHSA-jr45-8vmc-qm54, GHSA-v3r7-h72x-cjcm, GHSA-fxqj-rqcc-2cmp, GHSA-2v37-7h3g-55p8 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clears all seven
pnpm auditfindings (scope: all severities — 2 high, 5 moderate). Every finding was transitive-only, so each is fixed with a comparator-scoped override inpnpm-workspace.yaml. No direct dependency needed bumping, and nothing was suppressed viaignoreGhsas.Transitive overrides
undiciundici@<8.9.0→^8.9.0@actions/core,@actions/githubpostcsspostcss@<8.5.23→^8.5.23vitenanoidnanoid@<3.3.17→^3.3.17vite→postcssAll three stay within the installed major, so there is no API-break risk for the parents.
Note on
undiciThe pre-existing override was
"undici@>=6.0.0 <6.27.0": "8.8.0"— an exact pin that had itself become vulnerable. It is replaced by"undici@<8.9.0": "^8.9.0", which still covers the original 6.x range and the newly-vulnerable 8.0–8.8 line, and uses a caret range so future patch releases in the 8.x line flow through without another manual override bump.Advisories fixed
undici— GHSA-4cwx-7wf7-3272 (high) — cross-user information disclosure and parse-time crash via degenerate private cache directivesundici— GHSA-8xcm-r25x-g524 (moderate) — downstream response desynchronisation via retry interceptorundici— GHSA-m8rv-5g2x-5cg5 (moderate) — CRLF injection via blob-like bodytypepropertyundici— GHSA-jr45-8vmc-qm54 (moderate) — cross-user information disclosure via whitespace around equals inCache-Controldirectivesundici— GHSA-v3r7-h72x-cjcm (moderate) — cookie attribute injection via unsanitised domain and unparsedsetCookiefieldsnanoid— GHSA-2v37-7h3g-55p8 (high) — custom generators can loop indefinitely when size is zeropostcss— GHSA-fxqj-rqcc-2cmp (moderate) — attacker-controlledsourceMappingURLreads arbitrary.mapfiles whenfromis unsetVerification
pnpm install --frozen-lockfile— clean, no specifier driftpnpm test— 26 tests across 10 files pass, 100% coveragepnpm audit --audit-level=low— no known vulnerabilities found🤖 Generated with Claude Code