Skip to content

chore(deps): clear the brace-expansion advisory in the test toolchain - #22

Merged
robrigo merged 1 commit into
mainfrom
chore/deps-advisories
Aug 19, 2026
Merged

chore(deps): clear the brace-expansion advisory in the test toolchain#22
robrigo merged 1 commit into
mainfrom
chore/deps-advisories

Conversation

@robrigo

@robrigo robrigo commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Why

brace-expansion below 1.1.16 expands consecutive empty groups in exponential time, so a crafted pattern stalls the parser. It arrives here under jest's glob matching, development scope only. minimatch already declares a range that admits the fix, so this is a lockfile re-resolution with no manifest change, and the 2.x entries alongside it are outside the advisory and untouched.

Validation

yarn install --frozen-lockfile passes and the jest suite passes with counts identical to the pre-change run on the same worktree. The lockfile diff is three lines each way, brace-expansion only.

brace-expansion below 1.1.16 expands consecutive empty groups in
exponential time, so a crafted pattern stalls the parser. It arrives under
jest's glob matching, and minimatch's own range already admitted the fix,
so this is a lockfile re-resolution.
@robrigo
robrigo requested a lite review from Copilot August 19, 2026 02:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@robrigo robrigo closed this Aug 19, 2026
@robrigo robrigo reopened this Aug 19, 2026
@robrigo
robrigo merged commit 21c047b into main Aug 19, 2026
1 of 3 checks passed
@robrigo
robrigo deleted the chore/deps-advisories branch August 19, 2026 02:23
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.

2 participants