Skip to content

fix(deps): resolve the fast-uri high-severity audit advisory - #331

Merged
BenSheridanEdwards merged 1 commit into
mainfrom
fix/fast-uri-audit
Jul 22, 2026
Merged

fix(deps): resolve the fast-uri high-severity audit advisory#331
BenSheridanEdwards merged 1 commit into
mainfrom
fix/fast-uri-audit

Conversation

@BenSheridanEdwards

Copy link
Copy Markdown
Owner

Why does this feature exist?

The CI workflow's dependency audit step (npm audit --audit-level=high, ci.yml) fails on every branch including main over one high severity advisory in transitive fast-uri 3.1.3 — GHSA-v2hh-gcrm-f6hx, host confusion via a literal backslash authority delimiter. fast-uri is a transitive dependency, so main went red independently of any diff; the #329 build was simply the first to run after the advisory was published.

What changed?

package-lock.json only: fast-uri 3.1.3 to 3.1.4 via npm audit fix. No direct dependency changed, no manifest change, no code changed.

Behavioural Proof (with video and screenshots)

Not applicable: a lockfile-only transitive bump with no rendered surface. Proof is npm audit --audit-level=high reporting 0 vulnerabilities after the bump, and the full test suite passing against the updated tree.

Verification Summary

  • npm audit --audit-level=high: 1 high severity vulnerability before, 0 after.
  • npm run build (tsc): clean.
  • Full node --test suite: 630 pass, 0 fail.
  • Diff is 5 lines in package-lock.json (version + integrity + resolved for fast-uri 3.1.4).

`npm audit --audit-level=high` (ci.yml) began failing on every commit once
GHSA-v2hh-gcrm-f6hx was published against fast-uri 3.0.0–3.1.3 (host
confusion via a literal backslash authority delimiter). It is a transitive
dependency, so the advisory turned main red independently of any code change
— the #329 build was simply the first to run after it dropped.

`npm audit fix` bumps fast-uri 3.1.3 → 3.1.4 (lockfile only, no manifest
change) and clears the advisory: `npm audit --audit-level=high` now reports
0 vulnerabilities. Build is clean and the full suite stays 630/630.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

🗺️ StyleProof report

📊 View the side-by-side visual report →


Coverage, determinism, or report/diff consistency evidence is incomplete — repair the capture or reflow source; visual approval cannot clear this failure.

@BenSheridanEdwards
BenSheridanEdwards merged commit 517b109 into main Jul 22, 2026
12 checks passed
BenSheridanEdwards added a commit that referenced this pull request Jul 22, 2026
Cut 4.7.0 from the six commits merged since 4.6.3. Two are feature work, so
this is a minor bump; the version bump is the Release workflow's signal to
tag, publish to npm, and cut the GitHub Release on merge.

Added
- Reports publish through the git-data API (#328).
- styleproof-init prunes reports by API with a retention + size-budget sweep (#329).
- CERTIFICATION_FAILED dogfooded end-to-end (#327).

Security
- Cleared two high-severity transitive advisories: brace-expansion 5.0.6→5.0.7
  (#330) and fast-uri 3.1.3→3.1.4, GHSA-v2hh-gcrm-f6hx (#331).

Verified locally: build, typecheck, lint, format:check, privacy:check all
clean; full node --test suite 630/630 with a complete install.

Co-authored-by: StyleProof Test <styleproof@example.test>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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