Conversation
Automated dependency upgrade by OrbisAI Security
Contributor
|
…efresh Removes the overrides block from the prior fix and regenerates package-lock.json instead. Every consumer (@bedrock/config-yaml, postcss-loader -> cosmiconfig, swagger-jsdoc -> swagger-parser -> json-schema-ref-parser, @bedrock/test -> mocha) already declares a js-yaml range that permits the patched 4.3.2; the prior lockfile just had it pinned stale at 4.3.1. `npm ls js-yaml` now shows 4.3.2 everywhere with no overrides needed. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Author
|
Review comments addressed. Pls review. |
This branch has not been deployed
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
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
Resolve
js-yamlto the patched4.3.2release to fix CVE-2026-84375.Reworked per review feedback: the previous version of this PR pinned
js-yamlvia a
package.jsonoverridesblock. That's been removed. Instead, thelockfile was regenerated from scratch (
rm -rf node_modules package-lock.json && npm install), which resolvesjs-yamlto4.3.2on its own — everyconsumer in the tree (
@bedrock/config-yaml,postcss-loader→cosmiconfig,swagger-jsdoc→swagger-parser→@apidevtools/json-schema-ref-parser,@bedrock/test→mocha) alreadydeclares a
^4.xrange that permits4.3.2; the prior lockfile just had itpinned stale at
4.3.1. Confirmed withnpm ls js-yamlthat every resolvedcopy is now
4.3.2, with no overrides needed.Vulnerability
CVE-2026-84375package-lock.json(dependency:js-yaml)Description: js-yaml: Denial of Service vulnerability in YAML parsing
(CPU exhaustion via crafted YAML merge sources; affects
js-yaml>=4.0.0 <4.3.2and>=3.0.0 <3.15.2, patched in4.3.2/3.15.2).Changes
package.json: no override, no range changes — existing ranges alreadypermit the patched version.
package-lock.json: regenerated sojs-yamlresolves to4.3.2throughoutthe dependency tree.
Behavior Preservation
This change touches only dependency manifests (
package.json,package-lock.json); no source file in the repository is modified.js-yaml versions in the dependency tree are affected by CVE-2026-84375. This
PR updates dependency resolution to the upstream patched releases.
Reachability of attacker-controlled YAML input was not established.
Originally an automated security fix by OrbisAI Security, reworked manually per maintainer review.