Skip to content
This repository was archived by the owner on Jun 23, 2026. It is now read-only.

fix(web): resolve npm audit high-severity advisories#40

Open
kanywst wants to merge 1 commit into
mainfrom
fix/web-npm-audit
Open

fix(web): resolve npm audit high-severity advisories#40
kanywst wants to merge 1 commit into
mainfrom
fix/web-npm-audit

Conversation

@kanywst

@kanywst kanywst commented Jun 23, 2026

Copy link
Copy Markdown
Member

Why

All open PRs were failing the audit/npm-vuln CI gate (npm audit --prefix web --omit=dev --audit-level=high). The failure is not PR-specific — it stems from high-severity advisories in main's web dependency tree:

  • undici 7.0.0–7.27.2 (high) — TLS bypass, cache poisoning, header injection, etc.
  • vite 7.0.0–7.3.3 (high) — server.fs.deny bypass, NTLMv2 hash disclosure
  • plus moderate/low: js-yaml, dompurify, @babel/core

What

npm audit fix in web/ — transitive bumps only, lockfile-only change, package.json untouched.

Verification

  • npm audit --omit=dev --audit-level=highfound 0 vulnerabilities
  • npm run build → 102 pages built, Complete

Once merged to main, the dependabot PRs (#35#39) will pass npm-vuln on rebase.

Bump transitive deps (undici, vite, js-yaml, dompurify, @babel/core) via
npm audit fix to clear GHSA advisories failing the audit/npm-vuln CI gate.
Lockfile-only change; package.json unchanged.
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • web/package-lock.json is excluded by !**/package-lock.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 47618e76-1d28-474a-8739-c87aba9f711c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/web-npm-audit

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates several npm dependencies in web/package-lock.json. Specifically, it bumps multiple @babel packages (such as @babel/core, @babel/traverse, and @babel/parser) to version 7.29.7, and updates other packages including vite, undici, dompurify, js-yaml, and browser-related mapping libraries like browserslist and caniuse-lite. Since there are no review comments provided, I have no additional feedback to offer.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant