Skip to content

chore(admin-ui): lockfile security bumps — clears all 3 Dependabot highs (+2 npm-audit families) - #444

Merged
genwave-radio merged 2 commits into
mainfrom
chore/npm-security-bumps
Aug 9, 2026
Merged

chore(admin-ui): lockfile security bumps — clears all 3 Dependabot highs (+2 npm-audit families)#444
genwave-radio merged 2 commits into
mainfrom
chore/npm-security-bumps

Conversation

@genwave-radio

Copy link
Copy Markdown
Collaborator

Dean's ask: what's up with the 3 high bot findings. 🔍 Verdict: real alerts, near-zero actual risk in our topology, and all free to fix — so fixed.

The three Dependabot highs

Alert Package Chain Actual exposure
#37 nanoid <3.3.17 postcss (build chain) Infinite loop needs a custom generator called with size 0 — nothing in the admin UI does attacker-controlled nanoid calls
#32 js-yaml 4.x shadcn→cosmiconfig (dev CLI) Quadratic CPU parsing hostile YAML — dev tooling only, never shipped
#33 js-yaml 3.x ts-jest→istanbul (test tooling) Same, dev-only

Bonus (npm-audit-only, not yet surfaced by Dependabot)

brace-expansion (high, DoS family GHSA-mh99-v99m-4gvg — two locations) and @hono/node-server (moderate) via npm audit fix — same class, same pass.

Scope & verification

  • package-lock.json only — manifest untouched, no hand-edits, all bumps semver-compatible. Diff touches exactly the security trees (+ @modelcontextprotocol/sdk, the parent pinning hono).
  • npm audit: 0 vulnerabilities. Jest: 90 suites / 872 passed (run twice). next build: green (one transient favicon page-data flake on the first post-install build; clean on rerun — pre-existing Next quirk, not the bumps).
  • Reverted Next's unrelated auto-edits to next-env.d.ts/tsconfig.json that the local build tried to sneak in.

Alerts #32/#33/#37 auto-close when this lands on main.

…ot highs + 2 npm-audit families

nanoid 3.3.16→3.3.18 (GHSA-2v37-7h3g-55p8), js-yaml 4.3.0→4.3.1 +
3.15.0→3.15.1 (CVE-2026-59870), brace-expansion (GHSA-mh99-v99m-4gvg
family) and @hono/node-server via npm audit fix. All transitive, all
semver-compatible, manifest untouched. npm audit: 0 vulnerabilities.
@genwave-radio
genwave-radio merged commit f08f5bb into main Aug 9, 2026
10 checks passed
@genwave-radio
genwave-radio deleted the chore/npm-security-bumps branch August 9, 2026 20:42
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 9, 2026
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