build(deps): bump js-yaml from 4.1.1 to 4.2.0 in /tests/functional#2444
build(deps): bump js-yaml from 4.1.1 to 4.2.0 in /tests/functional#2444dependabot[bot] wants to merge 1 commit into
Conversation
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.1 to 4.2.0. - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](https://github.com/nodeca/js-yaml/commits) --- updated-dependencies: - dependency-name: js-yaml dependency-version: 4.2.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Hello dependabot[bot],My role is to assist you with the merge of this Available options
Available commands
Status report is not available. The following options are set: bypass_author_approval, bypass_jira_check |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: bypass_author_approval, bypass_jira_check |
There was a problem hiding this comment.
Dependency Bump Evaluation
Version change: js-yaml 4.1.1 -> 4.2.0 (minor)
Scope: Transitive dependency in tests/functional (only yarn.lock changed, not a direct import)
Changes:
- Added
maxDepth(100) andmaxMergeSeqLength(20) loader safety options - Stop resolving numbers with underscores as numeric scalars (#627)
- Several YAML parsing fixes (block mapping keys, flow scalar folding, number round-trips)
- Added sourcemaps to dist builds
Breaking changes: Numbers with underscores (e.g. 1_000) are no longer parsed as numeric scalars. No YAML files in this repository use underscored numbers, and js-yaml is not directly imported in tests/functional — it is a transitive dependency.
Security concerns: None introduced. This release fixes a DoS vulnerability (quadratic complexity in merge key deduplication for malformed files >10K) and adds safety guards (maxDepth, maxMergeSeqLength).
Impact on codebase: Minimal. The only direct js-yaml import in the repo is in tests/workflows/parse-deps.spec.ts (different package scope, unaffected by this bump). No code in tests/functional directly calls yaml.load() or similar APIs.
Recommendation: SAFE TO MERGE
Notes: CI checks are still in progress at time of review. The change is limited to a lockfile update of a transitive dependency with a security fix and no breaking impact on this codebase.
— Claude Code
Bumps js-yaml from 4.1.1 to 4.2.0.
Changelog
Sourced from js-yaml's changelog.
Commits
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.