Skip to content

fix: resolve baseline CI audit vulnerabilities#142

Merged
luandro merged 2 commits into
mainfrom
fix/audit-vulnerabilities-2026-07-22
Jul 22, 2026
Merged

fix: resolve baseline CI audit vulnerabilities#142
luandro merged 2 commits into
mainfrom
fix/audit-vulnerabilities-2026-07-22

Conversation

@luandro

@luandro luandro commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Resolves baseline CI audit failure — adds npm overrides for transitive dependencies with vulnerabilities, plus direct version bumps for vitest/*.

Changes

Package From To Fixes
vitest + @vitest/browser 4.1.8 4.1.10 GHSA-p63j-vcc4-9vmv (critical)
fast-uri (transitive) <3.1.3 ≥3.1.4 (override) GHSA-4c8g-83qw-93j6, GHSA-v2hh-gcrm-f6hx (high)
sharp (transitive) <0.35.0 ≥0.35.3 (override) GHSA-f88m-g3jw-g9cj (high)
axios 1.x ≥1.18.1 (override) Multiple high-severity advisories

Also applies Prettier formatting from v3.8.3 → v3.9.6 resolution change.

Verification

  • npm audit --audit-level=high0 high/critical vulnerabilities
  • npm test → full suite passes
  • tsc --noEmit → clean
  • ✅ ESLint → clean (one pre-existing warning)
  • ✅ Prettier → all files formatted
  • ✅ All existing scripts unchanged

Related

Unblocks PR #139 and any other PR blocked by the baseline audit failure.

Greptile Summary

Updates vulnerable development and transitive dependencies while preserving the previously reported Storybook CI runner setup.

  • Bumps Vitest packages to 4.1.10.
  • Adds dependency overrides and regenerates Bun and npm lockfiles.
  • Documents and temporarily ignores the stale Bun advisory for the fixed Vitest version.

Confidence Score: 5/5

The PR appears safe to merge because the previously reported Storybook CI runner configuration remains intact and no blocking failure remains.

No blocking failure remains.

T-Rex T-Rex Logs

What T-Rex did

  • Validated the pre-change CI audit behavior, showing that the audit ignored exit code 1 and reported 1 critical plus 6 high findings.
  • Validated the after-change CI audit, confirming the exact .github/workflows/ci.yml:511 command with all nine --ignore arguments exits with code 0.
  • Captured a reproduction script that records Bun versions, installed-graph versions, lockfile root versions, and the audit command to enable repeatable validation.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
package.json Adds security-focused overrides and bumps the aligned Vitest packages while retaining the Storybook runner script and dependency.
bun.lock Resolves the updated Vitest packages and overridden transitive dependencies to the intended versions.
package-lock.json Regenerates npm dependency resolutions to reflect the manifest updates.
.github/workflows/ci.yml Adds a documented audit exception for an advisory whose fixed Vitest version is already installed.

Reviews (3): Last reviewed commit: "fix: resolve baseline CI audit vulnerabi..." | Re-trigger Greptile

@socket-security

socket-security Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​vitest/​ui@​4.1.8 ⏵ 4.1.10991007599 +2100
Updatedvitest@​4.1.8 ⏵ 4.1.1098 +110079 +198100
Updated@​vitest/​coverage-v8@​4.1.8 ⏵ 4.1.10991007999 +2100
Updated@​vitest/​browser-playwright@​4.1.8 ⏵ 4.1.1010010086 +198100

View full report

@socket-security

socket-security Bot commented Jul 22, 2026

Copy link
Copy Markdown

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

Comment thread package.json
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Preview deployment ready: https://fix-audit-vulnerabilities-20.comapeo-cloud-app.pages.dev

Commit: d9d9c15

Add npm overrides to fix transitive dependency CVEs:
- vitest + @vitest/* 4.1.8 → 4.1.10 (critical GHSA-p63j-vcc4-9vmv)
- fast-uri >=3.1.4 (high GHSA-4c8g-83qw-93j6, GHSA-v2hh-gcrm-f6hx)
- sharp >=0.35.3 (high GHSA-f88m-g3jw-g9cj)
- axios >=1.18.1 (multiple high-severity advisories)

No source file changes.
@luandro
luandro force-pushed the fix/audit-vulnerabilities-2026-07-22 branch from 84d8bb6 to e8672cb Compare July 22, 2026 12:44
Add npm overrides to fix transitive dependency CVEs and whitelist
stale bun advisory for @vitest/browser.

- vitest + @vitest/* 4.1.8 → 4.1.10 (critical GHSA-p63j-vcc4-9vmv)
  Whitelisted in bun audit because 4.1.10 is the fix but bun's
  advisory DB has not been updated yet.
- fast-uri >=3.1.4 (high GHSA-4c8g-83qw-93j6, GHSA-v2hh-gcrm-f6hx)
- sharp >=0.35.3 (high GHSA-f88m-g3jw-g9cj)
- axios >=1.18.1 (multiple high-severity advisories)

No source file changes.
@luandro
luandro merged commit b88f915 into main Jul 22, 2026
21 checks passed
@luandro
luandro deleted the fix/audit-vulnerabilities-2026-07-22 branch July 22, 2026 13:57
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