Skip to content

fix(deps): override js-yaml to 4.3.2 for antora collector and swagger-ui extensions - #169

Merged
LukasHirt merged 1 commit into
mainfrom
fix/js-yaml-4-3-2-override
Oct 30, 2026
Merged

LukasHirt merged 1 commit into
mainfrom
fix/js-yaml-4-3-2-override

Conversation

@oc-tmueller

Copy link
Copy Markdown
Contributor

Resolves the four open Dependabot alerts for js-yaml (#8, #9, #10, #17 — one medium, three high): CPU-exhaustion issues in merge-key and !!omap handling, fixed in js-yaml 4.3.0–4.3.2.

Why an override

The only vulnerable copies are two nested js-yaml@4.1.1 installs, pulled in by build-time extensions that pin js-yaml ~4.1:

  • @antora/collector-extension@1.0.3
  • @opendevise/antora-swagger-ui-extension@1.0.0

Both are the latest releases on npm, so no upstream bump exists and Dependabot cannot resolve the alerts. The root js-yaml ^5.5.0 and Antora core (~5.4, 5.4.3) are not affected.

The override is scoped to those two packages and keeps them on the 4.x (v4-legacy) line instead of forcing the 5.x API on them. It can be dropped once upstream widens its range.

Verification

  • npm ls js-yaml: both extensions on 4.3.2, no 4.1.x left, 5.x copies unchanged.
  • Parsed all 167 tracked *.yml/*.yaml files plus the Libre Graph OpenAPI spec the collector fetches with both 4.1.1 and 4.3.2: identical results.
  • npm test: 39 passed, 0 failed (34 skipped).
  • npm run build succeeds (760 pages indexed). Collector output (ocis/9.0/developer/_attachments/v1.0.yaml) and both swagger-ui.html pages are still generated.

🤖 Generated with Claude Code

…-ui extensions

@antora/collector-extension 1.0.3 and @opendevise/antora-swagger-ui-extension
1.0.0, both the latest releases, pin js-yaml ~4.1 and so install 4.1.1, which
is affected by the merge-key and !!omap CPU-exhaustion advisories fixed in
4.3.0-4.3.2. No upstream release widens the range, so Dependabot cannot
resolve the alerts on its own.

Scope the override to those two packages and keep them on the 4.x (v4-legacy)
line rather than forcing the 5.x API on them; the root js-yaml ^5.5.0 and
Antora core's ~5.4 are untouched.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <323649642+oc-tmueller@users.noreply.github.com>
@oc-tmueller
oc-tmueller requested a review from a team as a code owner October 30, 2026 15:20
@LukasHirt
LukasHirt merged commit c667f20 into main Oct 30, 2026
6 checks passed
@LukasHirt
LukasHirt deleted the fix/js-yaml-4-3-2-override branch October 30, 2026 15:31
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