Skip to content

Align Node.js support with the nodejs22.x/nodejs24.x layer (T-17775) - #2

Closed
radike wants to merge 1 commit into
logtail:mainfrom
radike:t-17775-node22
Closed

Align Node.js support with the nodejs22.x/nodejs24.x layer (T-17775)#2
radike wants to merge 1 commit into
logtail:mainfrom
radike:t-17775-node22

Conversation

@radike

@radike radike commented Aug 5, 2026

Copy link
Copy Markdown

AWS Lambda's nodejs20.x runtime is reaching end of life. This aligns the extension's supported Node.js versions with the nodejs22.x/nodejs24.x runtimes (T-17775).

Changes

  • package.jsonengines.node: >=16.0.0>=22.0.0
  • CI — test matrix on Node 22 and 24 with fail-fast: false; the job name carries the version
  • CI — cache key now uses the matrix version; the previous key referenced env.NODE_VERSION, which was undefined
  • CI — new Build step running pnpm run build

Verification

Lint, all 22 tests, and the build pass on clean node:22 and node:24 containers with pnpm@8 and --frozen-lockfile; both versions produce an identical 136,272-byte dist/extension.zip.

Notes

  • The first CI run on this branch gets no cache hit on either job — the key changed, which is the intended fix, so both install from scratch once.
  • The engines bump is advisory: the repo has no engine-strict=true, so it documents intent without blocking older Node versions at install time.
  • @types/node deliberately stays at ^18: bumping it to ^22 resolves Request to node-fetch v3's type instead of the v2 type jest-fetch-mock expects, breaking three suites. That node-fetch v2/v3 cleanup belongs in a separate ticket.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@radike

radike commented Aug 5, 2026

Copy link
Copy Markdown
Author

Superseded by #3 — same commit (91720e8), now pushed directly to an in-repo branch after push access was granted, so CI runs without maintainer approval.

@radike radike closed this Aug 5, 2026
@radike
radike deleted the t-17775-node22 branch August 5, 2026 15:04
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