Skip to content

fix: bump vitest to ^3.2.6 (critical Dependabot advisory) - #13

Merged
chiliec merged 1 commit into
mainfrom
chore/security-dep-bumps
Jun 22, 2026
Merged

fix: bump vitest to ^3.2.6 (critical Dependabot advisory)#13
chiliec merged 1 commit into
mainfrom
chore/security-dep-bumps

Conversation

@chiliec

@chiliec chiliec commented Jun 22, 2026

Copy link
Copy Markdown
Member

Resolves the open Dependabot alerts on main. All are dev-scope — none ship in the published dist, so consumers of @viz-cx/core were never exposed.

Fixed

Critical (#3, #4) — vitest "UI server arbitrary file read/exec" (patched 3.2.6).

  • Bumps vitest + @vitest/coverage-v8 ^1.6.0^3.2.6.
  • Config is plain; all 106/106 unit tests pass under v3.
  • Coverage side-effect: vitest 3's v8 provider counts inner closures/arrows as uncovered even when their lines run (tx.ts: 100% lines / ~32% functions). Excluded ambient .d.ts and recalibrated only the functions floor — lines/statements/branches stay the meaningful gates at 98%/98%/92%.

Deferred (documented)

Low (#5) — esbuild dev-server file read on Windows, transitively via tsup.

  • Patched esbuild 0.28.1 is outside tsup 8.5.1's declared ^0.27.0 range; no tsup release uses 0.28 yet. Forcing it is unsupported.
  • Vector (dev server on Windows) doesn't apply to this repo's Linux CI / vitest run usage.
  • Revisit when tsup widens its esbuild range.

Gates (all green under vitest 3)

lint · lint:types · test (106/106) · build · test:types (tsd) · lint:exports (attw) · size (58.26 KB / 100 KB) · test:cov · pnpm install --frozen-lockfile (lockfile stays lockfileVersion: '9.0' for CI's pnpm 9).

Branch is off main, independent of #12 (viz-js-lib 0.12.7).

Resolves the critical advisory "Vitest UI server arbitrary file read/exec"
(fixed in 3.2.6). Bumps vitest and @vitest/coverage-v8 ^1.6.0 -> ^3.2.6.
Dev-dependency only; no change to the published surface.

vitest 3's v8 coverage provider counts inner closures/arrows as uncovered
functions even when their lines run, so coverage config now excludes ambient
.d.ts files and recalibrates the functions threshold below the v3 baseline.
Lines/statements/branches stay the meaningful gates (98%/98%/92%).

The low-severity esbuild advisory (dev-server file read on Windows, via tsup)
is left for later: patched esbuild 0.28.1 is outside tsup 8.5.1's ^0.27.0
range, and the vector does not apply to Linux CI / vitest-run usage.

Full gate green under vitest 3: lint, lint:types, test (106/106), build,
test:types, lint:exports, size, frozen-lockfile.
@chiliec
chiliec force-pushed the chore/security-dep-bumps branch from 8b516f5 to ec8b105 Compare June 22, 2026 12:19
@chiliec
chiliec merged commit 4dabc70 into main Jun 22, 2026
4 checks passed
@chiliec
chiliec deleted the chore/security-dep-bumps branch June 22, 2026 12:20
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