Skip to content

Bump js-yaml to 3.15.2 to clear GHSA-2883-xcg3-v3hh - #13

Merged
KidkArolis merged 1 commit into
masterfrom
update-all-clear-soon-vulnerabilities-sup-4013
Sep 14, 2026
Merged

KidkArolis merged 1 commit into
masterfrom
update-all-clear-soon-vulnerabilities-sup-4013

Conversation

@matt-humaan

Copy link
Copy Markdown
Contributor

Clears the one npm audit finding in this repo, for SUP-4013.

The finding

js-yaml 3.15.1, high severity — GHSA-2883-xcg3-v3hh, where maxTotalMergeKeys fails to limit CPU use for empty merge sources.

It is dev-only, and reaches us as a transitive of the test runner:

ava@8.0.1 → supertap@3.0.1 → js-yaml@^3.14.1

The fix

3.15.2 is the patched release inside supertap's ^3.14.1 range, so a plain npm update js-yaml resolves it. No overrides needed. The diff is 3 lines of package-lock.json.

npm audit now reports 0 vulnerabilities. The demo/ workspace was already clean, and those are the only two manifests in the repo.

Verification

Fresh npm ci on node 22.23.2, then npm test (build + oxlint + oxfmt + ava) — 102 tests passed, matching the baseline taken before the change.

Not run: npm run test:browser (Playwright) and the React 18/19 CI matrix. A dev-only YAML test reporter can't affect either, and CI covers both on this PR.

Note for anyone updating deps here

The first attempt at this produced a 108-line diff, because npm 10 strips the libc fields from optional platform-specific packages — a field npm 11 wrote and npm 10 doesn't understand. This lockfile was generated with npm 11, so running npm update under npm 10 silently degrades it. This commit was made with npm 11.11.0 to keep the diff clean.

npm ci is unaffected, so CI on node 22 is fine either way.

🤖 Generated with Claude Code

js-yaml 3.15.1 is affected by a high-severity advisory where
maxTotalMergeKeys fails to limit CPU use for empty merge sources.

It reaches us only as a dev-only transitive of ava, via
supertap's js-yaml@^3.14.1. 3.15.2 is the patched release inside
that range, so this is a plain in-range bump with no overrides.

npm audit now reports 0 vulnerabilities. The demo workspace was
already clean.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@linear-code

linear-code Bot commented Sep 14, 2026

Copy link
Copy Markdown

SUP-4013

@matt-humaan
matt-humaan marked this pull request as ready for review September 14, 2026 16:39

@cursor cursor Bot 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.

Risk: low. Approved this in-range js-yaml lockfile bump; Bugbot passed with no findings and no reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver - Risk Based

@KidkArolis
KidkArolis merged commit 8aa6af2 into master Sep 14, 2026
8 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.

2 participants