Skip to content

chore: enforce patched handlebars and basic-ftp in release tooling#10

Open
t15k wants to merge 1 commit into
mainfrom
chore/cve-handlebars-basic-ftp
Open

chore: enforce patched handlebars and basic-ftp in release tooling#10
t15k wants to merge 1 commit into
mainfrom
chore/cve-handlebars-basic-ftp

Conversation

@t15k

@t15k t15k commented Jul 27, 2026

Copy link
Copy Markdown

Summary

Clears two Vanta/Dependabot alerts in the npm release stub under .github/scripts/release/:

  • CVE-2026-33937 — handlebars 4.7.8 (transitive via auto-changelog), critical AST code injection → bumped to 4.7.9
  • CVE-2026-27699 — basic-ftp 5.0.5 (transitive via release-it → proxy-agent → pac-proxy-agent → get-uri), critical path traversal in downloadToDir() → bumped to 5.3.1

Both patched versions already satisfy the parents' declared ranges, so this is a lockfile refresh plus an overrides block in package.json to enforce the floor against future lockfile regenerations. Caret ranges (^4.7.9, ^5.2.0) keep both deps within the majors their parents were built against (a bare >= floor let basic-ftp jump to 6.x, which get-uri doesn't declare support for).

Test plan

  • npm ls handlebars basic-ftp shows 4.7.9 / 5.3.1
  • npx release-it --version and npx auto-changelog --version both run against the refreshed tree

🤖 Generated with Claude Code

https://claude.ai/code/session_017AP7a9cvvepgtn1yf52773

Fixes Vanta/Dependabot alerts CVE-2026-33937 (handlebars <= 4.7.8,
AST code injection) and CVE-2026-27699 (basic-ftp < 5.2.0, path
traversal in downloadToDir). Both are transitive dev deps of the
release stub (via auto-changelog and release-it -> get-uri), so the
fix is an overrides floor plus a lockfile refresh: handlebars 4.7.9,
basic-ftp 5.3.1. Caret ranges keep both within the majors their
parents declare.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017AP7a9cvvepgtn1yf52773
@t15k
t15k requested a review from HayesGordon July 27, 2026 09:32
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