Skip to content

chore(deps): update dependencies to latest and rebuild dist#34

Merged
kurok merged 1 commit into
mainfrom
chore/deps-latest
Jun 9, 2026
Merged

chore(deps): update dependencies to latest and rebuild dist#34
kurok merged 1 commit into
mainfrom
chore/deps-latest

Conversation

@kurok

@kurok kurok commented Jun 9, 2026

Copy link
Copy Markdown

What

Updates dependencies to their latest bundle-compatible versions and rebuilds dist/.

Dependency From To Notes
@actions/core 1.11.1 2.0.3 latest CommonJS major (see below)
@aws-sdk/client-ec2 3.1033.0 3.1064.0 minor; exact pin preserved
@eslint/js 10.0.1 new explicit devDep (eslint 10 no longer bundles it)
dotenv 8.6.0 17.4.2 dev/local only (npm run index)
eslint 9.39.4 10.4.1 dev only
globals 15.15.0 17.6.0 dev only

@actions/github (8.0.1), @vercel/ncc (0.38.4, exact pin kept), and jest (30.4.2) are already at their latest compatible versions.

Why these and not "absolute latest"

@actions/core@3 and @actions/github@9 are ESM-only ("type": "module", exports with only an import condition). @vercel/ncc@0.38.4 (the current latest ncc) cannot bundle themnpm run package fails to resolve the module — and this action is CommonJS. Since there's no security driver (see below), I held @actions/core at the latest CommonJS major (2.x) and @actions/github at 8.x. Worth revisiting if/when ncc gains ESM support.

Security

npm audit reports 0 vulnerabilities before and after, in both the production tree (bundled into dist/) and dev tree. This bump is freshness/maintenance, not a security fix.

Verification

  • npm audit (prod + dev): 0 vulnerabilities
  • npm run lint (eslint 10): clean
  • npm test: 52/52 pass
  • npm run package is deterministic; a clean npm ci && npm run package reproduces the committed dist/ exactly, so verify-dist will pass
  • ✅ Bundle integrity: loading the rebuilt dist/index.js reaches real action logic (structured logs + retry loop), confirming aws-sdk v3 + @actions/core 2 + @actions/github 8 all load and run on Node 24

Note

The @vercel/ncc range was briefly loosened to ^0.38.4 by npm install @latest; restored to the exact 0.38.4 pin to keep dist/ reproducible (matching the @aws-sdk/client-ec2 exact-pin convention).

Bump @actions/core 1.11.1 -> 2.0.3, @aws-sdk/client-ec2 3.1033.0 ->
3.1064.0, dotenv 8 -> 17, eslint 9 -> 10, globals 15 -> 17. Add
@eslint/js as an explicit devDependency (eslint 10 no longer bundles it).

@actions/core is held at the latest CommonJS major: 3.x is ESM-only and
cannot be bundled by ncc 0.38.4, which breaks this CommonJS action's
build. @actions/github is likewise kept at 8.x (9.x is ESM-only).

npm audit reports 0 vulnerabilities (prod + dev); all 52 tests pass; a
clean npm ci + npm run package reproduces the committed dist exactly.

Signed-off-by: Yuriy R <22548029+kurok@users.noreply.github.com>
@kurok kurok merged commit a9a3f47 into main Jun 9, 2026
4 checks passed
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