Skip to content

Pin nanoid to a patched 3.x - #168

Merged
YurMil merged 1 commit into
mainfrom
chore/pin-nanoid
Aug 17, 2026
Merged

Pin nanoid to a patched 3.x#168
YurMil merged 1 commit into
mainfrom
chore/pin-nanoid

Conversation

@YurMil

@YurMil YurMil commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Closes the new high advisory (#172): nanoid's custom generators loop indefinitely when size is zero.

nanoid@3.3.16 reaches the tree only through postcss, which is not somewhere this project hands a user-controlled size to. But postcss runs on every build, the fix is a version floor, and the existing pnpm.overrides block is already the established way this repo handles transitive advisories — so there is no reason to carry it.

One thing worth noting

Some neighbouring entries use >=. That does not work here:

"nanoid@<3.3.18": ">=3.3.18"   →  resolves nanoid 6.0.1

Three majors up and ESM-only, while postcss asks for ^3.3.16 and requires it as CommonJS. "^3.3.18" keeps the bump inside 3.x.

Verification

  • pnpm why nanoid reports a single nanoid@3.3.18, consumed only by postcss;
  • the lockfile contains no other nanoid version;
  • pnpm typecheck clean;
  • pnpm build green across all six locales — which exercises postcss end to end, the only consumer.

🤖 Generated with Claude Code

New high advisory: nanoid's custom generators loop indefinitely when size is
zero. The installed 3.3.16 comes in only through postcss, which is not a place
this project passes a user-controlled size to — but postcss runs on every build
and the fix is a version floor, so there is no reason to carry it.

Pinned with a caret rather than the >= used by some of the neighbouring
entries. ">=3.3.18" resolves to nanoid 6.0.1, three majors up and ESM-only,
which postcss cannot require — it asks for ^3.3.16. "^3.3.18" keeps the bump
inside 3.x.

pnpm why now reports a single nanoid@3.3.18 under postcss. typecheck and a
six-locale build pass, which exercises postcss throughout.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@supabase

supabase Bot commented Aug 17, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project bkcimygtsnckzexbfqxh because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cadautoscript-com Ready Ready Preview Aug 17, 2026 5:24am

@YurMil
YurMil merged commit e297bc0 into main Aug 17, 2026
8 checks passed
YurMil added a commit that referenced this pull request Aug 17, 2026
No dependabot alerts remain open. What is left in the runbook is now exactly
the work that needs Supabase project access, which is worth saying explicitly
so the next reader does not re-audit the closed items.

Co-Authored-By: Claude Opus 5 <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