Skip to content

fix: patch high-severity vite CVEs and enforce npm audit in CI#71

Merged
dmchaledev merged 1 commit into
mainfrom
claude/nice-mendel-iexrnx
Jun 16, 2026
Merged

fix: patch high-severity vite CVEs and enforce npm audit in CI#71
dmchaledev merged 1 commit into
mainfrom
claude/nice-mendel-iexrnx

Conversation

@dmchaledev

Copy link
Copy Markdown
Contributor

Summary

  • vite 8.0.14 → 8.0.16 via npm audit fix — resolves two high-severity CVEs that were silently present in the dev dependency tree (pulled in by vitest)
  • Added npm audit --audit-level=high to CI — fails the build immediately if any high/critical vulnerability is introduced, so this can't recur undetected

Vulnerability details

vite 8.0.0–8.0.15 carried two high-severity advisories:

Advisory Description
GHSA-v6wh-96g9-6wx3 NTLMv2 hash disclosure via UNC path handling on Windows
GHSA-fx2h-pf6j-xcff server.fs.deny bypass on Windows alternate paths

These are dev-only (not shipped in the published package), but a security-headers tool carrying known CVEs in its own toolchain is worth fixing on principle.

Test plan

  • npm audit reports 0 vulnerabilities after fix
  • All 82 existing tests pass (vitest run)
  • npm run typecheck and npm run build succeed
  • CI workflow now gates on npm audit --audit-level=high before any other steps

https://claude.ai/code/session_01KiMW3nyqB4TYjncTEAF7AH


Generated by Claude Code

vite 8.0.0–8.0.15 (pulled in by vitest) had two high-severity CVEs:
- GHSA-v6wh-96g9-6wx3: NTLMv2 hash disclosure via UNC path on Windows
- GHSA-fx2h-pf6j-xcff: server.fs.deny bypass on Windows alternate paths

Updated package-lock.json to vite 8.0.16 via `npm audit fix`.
Added `npm audit --audit-level=high` as the first post-install CI step
so future vulnerabilities in dev dependencies are caught automatically.

https://claude.ai/code/session_01KiMW3nyqB4TYjncTEAF7AH
@dmchaledev dmchaledev merged commit 63e6d1c into main Jun 16, 2026
1 check passed
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