Skip to content

fix(security): bump vitest and @vitest/coverage-v8 together to ^3.2.6 - #14

Merged
mizcausevic-dev merged 1 commit into
mainfrom
security/vitest-lockstep
Aug 5, 2026
Merged

fix(security): bump vitest and @vitest/coverage-v8 together to ^3.2.6#14
mizcausevic-dev merged 1 commit into
mainfrom
security/vitest-lockstep

Conversation

@mizcausevic-dev

Copy link
Copy Markdown
Owner

Clears GHSA-5xrq-8626-4rwp (CVSS 9.8).

Why Dependabot could not do this: it opens one PR for vitest and a separate one for @vitest/coverage-v8. They share a peer constraint, so neither merges alone (ERESOLVE). They must move in lockstep. That is why the Dependabot PRs on this repo show red CI.

npm audit fix is applied on top because the vitest 3.x tree pulls in vulnerable js-yaml, which trips an npm audit --audit-level=high gate.

Scope, stated honestly: this is a development dependency. The advisory only bites when someone runs vitest --ui locally. It never ships. It is worth doing because it is trivial and because this one advisory is 93% of the estate's entire critical alert count.

Verified in a clean-room checkout of this repo specifically: npm ci, then every script this repo defines among lint/typecheck/test/coverage/build/demo, then the audit gate. All green before the commit was made. Remaining criticals after: 0.

🤖 Generated with Claude Code

Clears GHSA-5xrq-8626-4rwp (CVSS 9.8), 'When Vitest UI server is listening,
arbitrary file can be read and executed'. Development scope: it only bites when
someone runs vitest --ui locally, it never ships. Patched because it is trivial
and this single advisory accounts for 93 percent of the estate's critical count.

Dependabot could not land this. It opens one PR for vitest and a separate one
for @vitest/coverage-v8, and the two share a peer constraint, so neither merges
alone (ERESOLVE). They have to move in lockstep.

npm audit fix is applied on top because the vitest 3.x tree pulls in vulnerable
js-yaml, which fails an 'npm audit --audit-level=high' CI gate.

Verified in a clean-room checkout for THIS repo specifically: npm ci, then every
script this repo defines among lint/typecheck/test/coverage/build/demo, then the
audit gate. All green before this commit was created.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@mizcausevic-dev
mizcausevic-dev merged commit 70f3188 into main Aug 5, 2026
2 checks passed
@mizcausevic-dev
mizcausevic-dev deleted the security/vitest-lockstep branch August 5, 2026 00:25
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.

1 participant