Skip to content

fix(deps): resolve open Dependabot alerts - #388

Merged
martinsafsten-codescene merged 2 commits into
mainfrom
fix/dependabot-lockfile-bumps
Nov 1, 2026
Merged

martinsafsten-codescene merged 2 commits into
mainfrom
fix/dependabot-lockfile-bumps

Conversation

@martinsafsten-codescene

@martinsafsten-codescene martinsafsten-codescene commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Resolves the 10 open Dependabot alerts.

Lockfile bumps (patch-level, within existing semver ranges; no new dependencies or install scripts):

Package Change Scope Alerts
follow-redirects 1.16.0 -> 1.16.2 runtime (axios) alerts 131, 132
fast-uri 3.1.8 -> 3.1.9 dev (ajv) alerts 133-136
serialize-javascript 7.1.2 -> 7.1.4 dev (mocha) alert 137
picomatch 2.3.1 -> 2.3.2 dev alert 48

extract-zip removed (Dependabot alerts 120, 123): it has unpatched symlink path-traversal advisories and no upstream fix. It was only used by the bundle-cli and update-cwf build scripts. Both now share scripts/extract-zip.js, which uses PowerShell Expand-Archive on Windows (as bundle-cli already did) and unzip elsewhere, then rejects any symlink resolving outside the destination. This also drops yauzl and its transitive packages from the lockfile.

Test plan

  • make lint
  • make build (includes bundling the CLI zip through the new extractor on Windows)
  • Full make test after the lockfile bump: 794 passing, 0 failing
  • CWF, Native CLI and new extract-zip tests after removing extract-zip
  • extract-zip symlink tests run on Linux (node:22 container); they skip on Windows
  • CI on macOS and Linux exercises the unzip path via bundle-cli-test

Dependabot alerts close automatically once the vulnerable versions leave the lockfile on main.

Updates transitive dependencies to patched releases:

- follow-redirects 1.16.0 -> 1.16.2 (runtime, via axios; #131, #132)
- fast-uri 3.1.8 -> 3.1.9 (dev, via ajv; #133-#136)
- serialize-javascript 7.1.2 -> 7.1.4 (dev, via mocha; #137)
- picomatch 2.3.1 -> 2.3.2 (dev; #48)

Lockfile-only change; package.json ranges already allowed these versions.
extract-zip (#120, #123) has no upstream fix and is left for a separate
change.
extract-zip has unpatched symlink path-traversal advisories (#120, #123)
and no upstream fix. It was only used by build scripts to unpack the CLI
and CWF archives.

Share one extractZipArchive helper between bundle-cli and update-cwf. It
uses PowerShell Expand-Archive on Windows (as bundle-cli already did) and
unzip elsewhere, then rejects any symlink that resolves outside the
destination directory. Removing the dependency also drops yauzl and its
transitive packages from the lockfile.

@codescene-access codescene-access Bot 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.

No application code in the PR — skipped Code Health checks.

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@codescene-access codescene-access Bot 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.

Code Coverage Gates Passed
New & Changed Code Coverage: 100% (required = 95%)

Review details by gate
  • New & Changed Code Coverage
    • Pass/Fail Reason:
      No executable lines of code changed. (The change touched 0 files).

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.

1 participant