Skip to content

fix(root): bump js-yaml 4.3.0 -> 4.3.1 to unblock release (HIGH osv finding) - #9445

Merged
lokesh-bitgo merged 1 commit into
masterfrom
WCN-2047-bump-js-yaml-4.3.1
Aug 7, 2026
Merged

fix(root): bump js-yaml 4.3.0 -> 4.3.1 to unblock release (HIGH osv finding)#9445
lokesh-bitgo merged 1 commit into
masterfrom
WCN-2047-bump-js-yaml-4.3.1

Conversation

@zahin-mohammad

Copy link
Copy Markdown
Contributor

Summary

  • The BitGoJS release severity gate (.github/workflows/publish.ymlosv-severity-gate) fails: 1 of 37 advisory groups is ≥ CVSS 7.0. The gate fails only on HIGH/CRITICAL; the other 36 are sub-threshold.
  • The sole blocker is js-yaml@4.3.0GHSA-5p4m-2wfm-xmqj, CVSS 7.5 (HIGH): quadratic CPU consumption in !!omap resolution (objectKeys.indexOf linear scan inside the per-element loop → event-loop DoS). Affected >=4.0.0 <4.3.1; first patched 4.3.1.
  • Root resolutions/overrides already pinned js-yaml to 4.3.0 (itself vulnerable). This bumps both pins to 4.3.1 (same v4 API) and updates the single yarn.lock block. js-yaml is dev-tooling only here (lerna, eslint, depcheck, cosmiconfig, mocha, nyc) — no runtime path.

Test plan

  • yarn install --frozen-lockfile passes (matches CI's sfw yarn install --with-frozen-lockfile)
  • yarn check-deps passes
  • yarn why js-yaml4.3.1; no js-yaml < 4.3.1 remains in yarn.lock
  • lockfile diff is the js-yaml block only (no unrelated re-resolution)
  • CI osv-severity-gate reports 0 groups ≥ 7.0

Out of scope

Four advisory groups sit at CVSS 6.9 (http-proxy-middleware, react-router-dom, tar, valibot); a rescore of any one re-blocks the release. Tracked in WCN-2047 as follow-up.

Ticket: WCN-2047

🤖 Generated with Claude Code

js-yaml@4.3.0 is affected by GHSA-5p4m-2wfm-xmqj (CVSS 7.5, HIGH):
quadratic CPU consumption in !!omap resolution, a DoS vector. This is
the only advisory group at/above CVSS 7.0, so it fails the release
severity gate (.github/workflows/publish.yml -> osv-severity-gate).

The root resolutions/overrides already pinned js-yaml to 4.3.0, which is
itself vulnerable. Bump both pins to the patched v4 release 4.3.1 (same
v4 API) and update the single lockfile block. js-yaml is dev-tooling
only here (lerna, eslint, depcheck, cosmiconfig, mocha, nyc).

Validated: yarn install --frozen-lockfile passes; yarn check-deps
passes; yarn why js-yaml resolves to 4.3.1; no js-yaml < 4.3.1 remains
in yarn.lock.

TICKET: WCN-2047

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

WCN-2047

@zahin-mohammad
zahin-mohammad marked this pull request as ready for review August 7, 2026 02:22
@zahin-mohammad
zahin-mohammad requested a review from a team as a code owner August 7, 2026 02:22

@diksha190 diksha190 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.

Approved.
SafeChain cooldown-blocked (6.5 days, age-only),
Socket deep score 84, no high/critical alerts.
Package: js-yaml@4.3.1.
Condition: dev-tooling only, fixes CVE-2026-59870 (CVSS 7.5).
Age flagged but waived given clean scan + fix urgency

@lokesh-bitgo
lokesh-bitgo merged commit 3d1f975 into master Aug 7, 2026
25 checks passed
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.

5 participants