Skip to content

deps: Bump js-yaml from 4.3.1 to 5.4.1 - #1558

Open
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/npm_and_yarn/js-yaml-5.4.1
Open

deps: Bump js-yaml from 4.3.1 to 5.4.1#1558
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/npm_and_yarn/js-yaml-5.4.1

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Aug 30, 2026

Copy link
Copy Markdown
Contributor

Bumps js-yaml from 4.3.1 to 5.4.1.

Changelog

Sourced from js-yaml's changelog.

[5.4.1] - 2026-08-26

Changed

  • Hard-limit merge sequence size to 100.

Security

  • Count empty mappings in merge sequences toward maxTotalMergeKeys to limit CPU usage, #797.

[5.4.0] - 2026-08-25

Added

  • Added the scalarStyleRules dumper option to customize string formatting. See Scalar styling for details.

Changed

  • [breaking] Flattened the low-level AST node style representation. Scalar and collection nodes now use SCALAR_STYLE and COLLECTION_STYLE values; explicit tags use the separate tagged property. Alias nodes now contain only kind and anchor. This only affects code that directly constructs or edits AST nodes.
  • [breaking] The sortKeys option was rewritten using AST mutation to avoid side effects.
  • Reworked scalar style selection. This can change formatting without changing loaded values; in particular, whitespace-only strings are now double-quoted.

Fixed

  • Accept a byte order mark at the start of each document in a stream, #791.
  • Produce valid flow mappings with quoteFlowKeys and flowSkipColonSpace, including alias and property-only keys, #786.
  • Preserve empty scalar items when converting block sequences to flow style.
  • Do not apply the 1024-character simple-key limit to flow mapping keys.
  • Count Unicode code points, rather than UTF-16 code units, for the 1024-character simple-key limit.
  • Add an explicit document-end marker after keep-chomped block scalars when needed to preserve trailing newlines.

[5.3.0] - 2026-08-14

This release focuses on reworking the documentation and making small architectural improvements before moving forward.

Added

  • Added completely new documentation.
  • Exported DUMP_SCHEMA, the default schema used by the dumper.
  • Added YAMLException.throwAt() for throwing an error at a source position.

Changed

... (truncated)

Commits

@dependabot dependabot Bot added dependencies Automated dependency update javascript Pull requests that update Javascript code labels Aug 30, 2026
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.3.1 to 5.4.1.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@4.3.1...5.4.1)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 5.4.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot force-pushed the dependabot/npm_and_yarn/js-yaml-5.4.1 branch from 306a1a6 to 4d07c49 Compare August 31, 2026 07:26
@d3xter666
d3xter666 force-pushed the dependabot/npm_and_yarn/js-yaml-5.4.1 branch from f114cad to 10c9542 Compare August 31, 2026 08:25
@d3xter666
d3xter666 requested a review from a team August 31, 2026 08:30
// See https://github.com/nodeca/js-yaml/issues/456 and https://github.com/nodeca/js-yaml/pull/381
configs = jsyaml.loadAll(configFile, undefined, {
// Using loadAll with CORE_SCHEMA (equivalent of v4's DEFAULT_SAFE_SCHEMA) to pass "filename".
configs = jsyamlLoadAll(configFile, undefined, {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment was already outdated, and with the upgrade to js-yaml v4, it was already removed: SAP/ui5-project#380

However, it seems that another PR (SAP/ui5-project#457), that was started before the upgrade did re-introduce the code/comment again, and referenced jsyaml.DEFAULT_SAFE_SCHEMA which already in js-yaml v4 is undefined, so the default was used.

Are you sure that the CORE_SCHEMA is equivalent to DEFAULT_SAFE_SCHEMA, or rather the default schema of v4, as DEFAULT_SAFE_SCHEMA did not exist anymore in js-yaml v4?

One important aspect of yaml parsing is merge keys (<<:). This is widely used by custom task/middleware configuration (e.g. https://github.com/ui5-community/ui5-ecosystem-showcase/tree/main/packages/ui5-tooling-modules).
We should keep that stable, and it looks like we do not have any tests that verify it.

@d3xter666
d3xter666 force-pushed the dependabot/npm_and_yarn/js-yaml-5.4.1 branch from 10c9542 to a1dec9b Compare August 31, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Automated dependency update javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants