Skip to content

build(deps): patch brace-expansion DoS - #38

Merged
GSTJ merged 1 commit into
masterfrom
security/brace-expansion-5.0.9
Aug 4, 2026
Merged

GSTJ merged 1 commit into
masterfrom
security/brace-expansion-5.0.9

Conversation

@GSTJ

@GSTJ GSTJ commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

We pinned both vulnerable brace-expansion branches to their patched releases, clearing GHSA-rgw5-rvv9-x895 from the development dependency graph.

Details

  • 1.x now resolves to 1.1.18.
  • 5.x now resolves to 5.0.9.
  • The dependency is only used by repository tooling and stays out of the published npm package.

Testing steps

pnpm install --frozen-lockfile
pnpm audit
pnpm test --runInBand
npm pack --dry-run

Each command should exit successfully. The audit should report no known vulnerabilities.

@GSTJ

GSTJ commented Aug 4, 2026

Copy link
Copy Markdown
Owner Author

Local proof from 9dbf97d: the audit is clear, the full suite passes, and an Expo 57 prebuild writes each native CodePush hook once.

Audit, checks and Expo prebuild output

@GSTJ
GSTJ merged commit 8ea4877 into master Aug 4, 2026
5 checks passed
@GSTJ
GSTJ deleted the security/brace-expansion-5.0.9 branch August 4, 2026 23:31
GSTJ added a commit that referenced this pull request Sep 21, 2026
* fix(release): bump a patch for build(deps) commits

Three security patches (#37, #38, #43, #44, #45) landed as
build(deps): commits and never reached npm: the changelog preset marks
build as effect: "changelog", so release-it correctly found nothing to
release and exited 0 every time. Nobody was watching for that, so the
patches sat on master.

findTypeEntry matches on type and scope before falling back to the
scope-less entry, so a build(deps) entry ahead of the generic build one
in TYPES lets dependency and security patches bump a patch while
scope-less build: (tooling, tsconfig, oxfmt) still doesn't move the
version. changelog-check.mjs's POLICY and assertions are updated to
match, including a build(deps) vs build bump test so this can't regress
silently again.

* docs: the Swift AppDelegate redirect already landed

bb07d7b fixed the SDK 53+ AppDelegate.swift bundle URL redirect back in
v1.0.13, but the SDK-support note above it was written 26 minutes
earlier the same day and never got updated. Confirmed by prebuild
against a fresh SDK 57 app: CodePush.bundleURL() lands in
ReactNativeDelegate.bundleURL() same as the Objective-C template does.

* style: satisfy oxfmt on changelog-check.mjs
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