chore: upgrade Lambda runtime from nodejs22.x to nodejs24.x #74

Closed

seahaven-openswe[bot] wants to merge 1 commit from chore/upgrade-nodejs24-lambda-runtime into main

seahaven-openswe[bot] commented

2026-07-04 05:27:53 +00:00

(Migrated from github.com)

Summary

Upgrade all nine Lambda NodejsFunction definitions from NODEJS_22_X to NODEJS_24_X across the five construct files. Also add dependabot ignores for @types/node >=26 to prevent premature major bumps while we remain on the nodejs24.x runtime.

Both package.json files already carry @types/node at ^24 (root: ^24.13.2, socket-mode: ^24.0.0) — no pin change was needed.

Closes #72

Validation

npm ci — clean
tsc --noEmit — zero errors
cdk synth blocked by missing Docker daemon in sandbox (OpenSearch CR provider asset); the tsc pass confirms all Runtime.NODEJS_24_X references resolve correctly

Tests

No automated tests affected. Smoke-testing the real Slack DM bot flows, App Home, WO/PO lookup, and notion-sync is called out as a post-deploy step.

Notes

Deploy to account 328440206208 / us-east-1 and smoke-test before merge
Node 22→24 changes built-in APIs, OpenSSL, and undici — exercise the real Slack round-trip, don't just typecheck

## Summary Upgrade all nine Lambda `NodejsFunction` definitions from `NODEJS_22_X` to `NODEJS_24_X` across the five construct files. Also add dependabot ignores for `@types/node >=26` to prevent premature major bumps while we remain on the `nodejs24.x` runtime. Both `package.json` files already carry `@types/node` at `^24` (root: `^24.13.2`, socket-mode: `^24.0.0`) — no pin change was needed. Closes #72 ## Validation - `npm ci` — clean - `tsc --noEmit` — zero errors - `cdk synth` blocked by missing Docker daemon in sandbox (OpenSearch CR provider asset); the `tsc` pass confirms all `Runtime.NODEJS_24_X` references resolve correctly ## Tests No automated tests affected. Smoke-testing the real Slack DM bot flows, App Home, WO/PO lookup, and notion-sync is called out as a post-deploy step. ## Notes - Deploy to account `328440206208` / `us-east-1` and smoke-test before merge - Node 22→24 changes built-in APIs, OpenSSL, and `undici` — exercise the real Slack round-trip, don't just typecheck

amoussa1229 (Migrated from github.com) reviewed 2026-07-04 05:27:53 +00:00

seahaven-openswe[bot] (Migrated from github.com) reviewed 2026-07-04 05:29:42 +00:00

seahaven-openswe[bot] (Migrated from github.com) left a comment

✅ Open SWE Review: No issues found

Open SWE reviewed this PR and found no potential bugs to report.

Open in Web

## ✅ Open SWE Review: No issues found Open SWE reviewed this PR and found no potential bugs to report. [Open in Web](https://openswe.seahaven.com/agents/reviews/Sea-Haven-Industries/seahaven-slack-bot/74)

This repo is archived. You cannot comment on pull requests.

No reviewers

amoussa1229