Skip to content

Bump transitive js-yaml to 4.3.1+ to fix CVE-2026-59870 - #230

Closed
JoyceZhu with Copilot wants to merge 1 commit into
mainfrom
copilot/update-js-yaml-version
Closed

Bump transitive js-yaml to 4.3.1+ to fix CVE-2026-59870#230
JoyceZhu with Copilot wants to merge 1 commit into
mainfrom
copilot/update-js-yaml-version

Conversation

Copilot AI commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

The @eslint/eslintrc devDependency resolved js-yaml@4.3.0, which is vulnerable to CVE-2026-59870 (quadratic CPU consumption in !!omap resolution; DoS). Fixed in 4.3.1. The direct js-yaml devDependency was already ^5.2.2 and unaffected.

Changes

  • package.json — added a scoped override so only the transitive dependency is bumped, leaving the direct ^5.2.2 devDependency untouched:

    "overrides": {
      "postcss": "8.5.23",
      "@eslint/eslintrc": {
        "js-yaml": "^4.3.1"
      }
    }
  • package-lock.json — regenerated; @eslint/eslintrc's js-yaml now resolves to 4.3.1.

Notes

  • A top-level js-yaml override was intentionally avoided as it would downgrade the direct 5.2.2 dependency.
  • The pre-existing brace-expansion audit finding is unrelated and left out of scope.

Co-authored-by: JoyceZhu <6251669+JoyceZhu@users.noreply.github.com>
Copilot AI changed the title Bump js-yaml to 4.3.1+ to fix CVE-2026-59870 Bump transitive js-yaml to 4.3.1+ to fix CVE-2026-59870 Aug 10, 2026
Copilot AI requested a review from JoyceZhu August 10, 2026 18:10
@JoyceZhu
JoyceZhu marked this pull request as ready for review August 10, 2026 19:19
@JoyceZhu
JoyceZhu requested a review from a team as a code owner August 10, 2026 19:19
Copilot AI balanced review requested due to automatic review settings August 10, 2026 19:19
@JoyceZhu JoyceZhu closed this Aug 10, 2026

Copilot AI 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.

Pull request overview

Updates the vulnerable transitive js-yaml dependency while preserving the direct 5.x dependency.

Changes:

  • Adds a scoped @eslint/eslintrc override requiring js-yaml 4.3.1+.
  • Regenerates the lockfile with js-yaml 4.3.1.
Show a summary per file
File Description
package.json Adds the scoped security override.
package-lock.json Locks the transitive dependency to 4.3.1.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@JoyceZhu
JoyceZhu deleted the copilot/update-js-yaml-version branch August 10, 2026 19:21
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.

3 participants