Skip to content

security: dependabot fix-or-dismiss sweep - #4

Merged
lua-stefan-kruger merged 1 commit into
mainfrom
security/dependabot-js-yaml-sweep
Jul 28, 2026
Merged

security: dependabot fix-or-dismiss sweep#4
lua-stefan-kruger merged 1 commit into
mainfrom
security/dependabot-js-yaml-sweep

Conversation

@lua-stefan-kruger

@lua-stefan-kruger lua-stefan-kruger commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Dependabot fix-or-dismiss sweep

Fixed

  • js-yaml 4.2.0 -> 4.3.0 (alert #11, high). The vulnerable copy was pinned to 4.2.0 by an existing js-yaml@^4.1.0 override; bumped that selector to 4.3.0. Transitive, dev-only (eslint/@eslint/eslintrc, requires ^4.1.0). The js-yaml@^3.13.1 -> 3.15.0 override is untouched, so the istanbul 3.x copy stays on the compatible major.

Dismissed (handled outside this PR)

  • Alerts #9 and #10 (js-yaml 3.x, patched 3.15.0): installed copies are already 3.15.0 (>= patched) via the existing override — rescan lag, dismissed as inaccurate.

Verification

  • npm run lint: green
  • npm test: 17 suites / 248 tests passed

Do not merge without review.


Note

Low Risk
Dev-only dependency pin via npm overrides; no runtime or application code changes.

Overview
Addresses a high Dependabot alert on transitive js-yaml by raising the existing js-yaml@^4.1.0 npm override from 4.2.0 to 4.3.0, with the matching package-lock.json resolution update. The 4.x copy is dev-only (pulled in through eslint / @eslint/eslintrc).

The separate js-yaml@^3.13.13.15.0 override is unchanged so the 3.x istanbul-related tree stays on the compatible major.

Reviewed by Cursor Bugbot for commit 3d7112e. Bugbot is set up for automated code reviews on this repo. Configure here.

Bump transitive js-yaml 4.x override 4.2.0 -> 4.3.0 (GHSA high, alert #11).
Dev-only (eslint/@eslint/eslintrc, ^4.1.0). 3.x override stays 3.15.0.
Lint + 248 tests green.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@lua-stefan-kruger
lua-stefan-kruger merged commit 67cdc36 into main Jul 28, 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