build(deps): patch js-yaml CPU exhaustion - #45
Merged
Merged
Conversation
Dependabot couldn't resolve past 3.15.1 on its own because the range comes from eslint, @expo/xcpretty and jest's coverage config, all capped below the fix. Overrides both js-yaml majors to the patched releases and updates the installed-version assertions in the dependency security check to match. Closes alerts #57 and #58.
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Overrides js-yaml to the patched releases, closing Dependabot alerts #57 and #58.
Details
js-yaml@<3.15.2->^3.15.2js-yaml@>=4.0.0 <4.3.2->^4.3.2tools/dependency-security-check.mjs's installed-version assertions to match, so the check actually verifies the patched code is what's running instead of drifting silently.build/output, so this doesn't ship a package release on its own.Testing steps
corepack pnpm install pnpm run test pnpm run security:checkBoth should exit clean: 72 tests passing and the audit reporting only the two pre-existing, already-ignored image-size findings.