Skip to content

Clear the 17 open Dependabot alerts via pnpm overrides (API-942) - #7

Merged
kiernanmcgowan merged 2 commits into
mainfrom
kiernan/api-942-clear-the-17-open-dependabot-alerts-in-the-logo-api-registry
Aug 19, 2026
Merged

Clear the 17 open Dependabot alerts via pnpm overrides (API-942)#7
kiernanmcgowan merged 2 commits into
mainfrom
kiernan/api-942-clear-the-17-open-dependabot-alerts-in-the-logo-api-registry

Conversation

@kiernanmcgowan

@kiernanmcgowan kiernanmcgowan commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Fixes API-942.

Problem

This repo has 17 open Dependabot alerts, all transitive npm devDependencies of the shadcn/vitest toolchain in pnpm-lock.yaml. None are direct dependencies, and none ship in the published registry output, but the repo is public and five of the seven packages carry high-severity advisories.

Approach

Adds pnpm overrides for the seven vulnerable packages and regenerates the lockfile — no direct-dependency changes. Two adjustments from the groomed plan:

  • Overrides live in pnpm-workspace.yaml, not package.json — CI runs pnpm 11, which ignores the package.json "pnpm" field.
  • In-major ceilings on the four packages whose latest release crossed a major (undici, fast-uri, js-yaml, @hono/node-server), so no untested major bump rides along; and hono capped <4.13 because hono@4.13.3 was published within 24h and trips pnpm 11's minimumReleaseAge supply-chain policy — it resolves to 4.12.34, exactly the patched floor.

Resolved versions: undici 7.29.0, ip-address 10.5.0, fast-uri 3.1.5, postcss 8.5.26, js-yaml 4.3.1, hono 4.12.34, @hono/node-server 1.19.17 — each at or above its first-patched version, each within its prior major, each published >8 days ago (youngest is ip-address). The full 508-entry lockfile passes pnpm 11's supply-chain check.

Security alerts

Tests

Verified with pnpm 11 (CI's pinned version): pnpm install --frozen-lockfile from a clean state (the step that failed on the first push), pnpm test (16/16), pnpm typecheck, pnpm lint, pnpm build, and git diff --exit-code r/ (registry output byte-identical) all pass.

🤖 Generated with Claude Code

kiernanmcgowan and others added 2 commits August 18, 2026 17:12
All 17 alerts are transitive devDependencies of the shadcn/vitest
toolchain in pnpm-lock.yaml. Adds pnpm.overrides floors for the seven
vulnerable packages and regenerates the lockfile; the four packages
whose latest release crossed a major (undici, fast-uri, js-yaml,
@hono/node-server) are ceiling-capped to stay inside their current
major so no untested major bump rides along.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…icy (API-942)

CI runs pnpm 11, which ignores the package.json "pnpm" field (overrides
now live in pnpm-workspace.yaml) and enforces a 24h minimumReleaseAge
that rejected hono@4.13.3 (published <24h before the run). Moves the
seven overrides to pnpm-workspace.yaml, drops the dead package.json
block, and caps hono at <4.13 so it resolves to 4.12.34 — exactly the
patched floor. All seven resolved versions verified >8 days old, and
the full 508-entry lockfile passes pnpm 11's supply-chain check.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@kiernanmcgowan
kiernanmcgowan merged commit d7c0933 into main Aug 19, 2026
4 checks passed
@alexbaldwin
alexbaldwin deleted the kiernan/api-942-clear-the-17-open-dependabot-alerts-in-the-logo-api-registry branch September 1, 2026 20:33
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