Skip to content

fix(deps): clear the fast-uri high advisory that fails the npm audit gate - #530

Merged
PeterGuy326 merged 1 commit into
mainfrom
fix/528-npm-audit-fast-uri
Sep 29, 2026
Merged

PeterGuy326 merged 1 commit into
mainfrom
fix/528-npm-audit-fast-uri

Conversation

@PeterGuy326

Copy link
Copy Markdown
Contributor

Preflight

Summary

verify fails on main because npm run security:check (npm audit --audit-level=high) exits 1 on the fast-uri high advisory (#528). Lockfile-only, non-breaking bumps:

Package Before After Reason
fast-uri 3.1.6 3.1.8 GHSA-qw65-cvwx-89v3, GHSA-58mr-gqgx-xq4g (high)
undici 6.28.0 6.29.0 GHSA-3wwx-pv8p-q78v (moderate)
undici 7.29.0 7.30.0 GHSA-3wwx-pv8p-q78v (moderate)

Produced with npm audit fix --package-lock-only; no manifest range changed, no breaking bump taken, node_modules untouched.

Evidence

  • Before: npm audit --audit-level=high --package-lock-only reports 4 vulnerabilities (1 high, 3 moderate) and exits 1 on main @ f2fca7b.
  • After: the same command reports 2 moderate vulnerabilities (the @vitest/mocker chain that requires vitest 5) and exits 0, so the --audit-level=high gate passes.

Assumptions

Acceptance

  • npm run security:check passes on CI (the verify workflow's audit gate).
  • No package.json change: lockfile-only, semver-compatible bumps; install/build behavior unchanged.

Closes #528

fast-uri 3.1.6 -> 3.1.8 clears GHSA-qw65-cvwx-89v3 and
GHSA-58mr-gqgx-xq4g (high); undici 6.28.0 -> 6.29.0 and 7.29.0 -> 7.30.0
clear GHSA-3wwx-pv8p-q78v (moderate). Lockfile-only via
npm audit fix --package-lock-only: no manifest range changes and no
breaking bump, so the verify security gate passes without forcing the
vitest 5 upgrade.

@Bindy-lbb Bindy-lbb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

复审通过:改动仅涉及 package-lock.json 中 fast-uri 3.1.8 与 undici 6.29.0/7.30.0 三个条目;package.json 未变,锁定版本符合对应依赖范围并高于 GHSA 公告中的修复版本,registry integrity 也匹配。当前 verify、Dependency review、CodeQL 等检查全绿。

@PeterGuy326
PeterGuy326 merged commit e21ea23 into main Sep 29, 2026
11 checks 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.

fix(ci): verify fails on the fast-uri high advisory in npm audit

2 participants