⚙️ setup: align Node.js runtime policy with 26 default (22/24/26 support) - #12
Conversation
Agent-Logs-Url: https://github.com/wgtechlabs/unthread-whatsapp-bot/sessions/0a178c3e-4f82-4d75-a5d9-7919f174d8cc Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
Agent-Logs-Url: https://github.com/wgtechlabs/unthread-whatsapp-bot/sessions/0a178c3e-4f82-4d75-a5d9-7919f174d8cc Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
Agent-Logs-Url: https://github.com/wgtechlabs/unthread-whatsapp-bot/sessions/0a178c3e-4f82-4d75-a5d9-7919f174d8cc Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Aligns this repository’s setup/runtime policy so local dev, Docker runtime, CI validation, and docs consistently reflect support for Node.js 22/24/26 with Node.js 26 as the default (while continuing to use Bun as the toolchain/runtime).
Changes:
- Document Node.js 22/24/26 support (default 26) in the README and add
.nvmrcdefaulting to 26. - Add
enginesconstraints inpackage.jsonand update Docker to use a Node.js 26 Alpine base while sourcing Bun from the official Bun image. - Add a CI workflow that runs lint/typecheck/tests with a Node.js version matrix (22/24/26) and a pinned Bun version.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates prerequisites and Docker notes to match the Node 22/24/26 policy with 26 as default. |
| package.json | Adds Node/Bun engine constraints to formalize supported runtime/toolchain ranges. |
| Dockerfile | Switches base image to Node 26 Alpine and copies in Bun for the app toolchain; updates healthcheck implementation. |
| .nvmrc | Sets Node 26 as the default local development runtime. |
| .github/workflows/ci.yml | Adds a CI workflow intended to validate under Node 22/24/26 while using Bun. |
Agent-Logs-Url: https://github.com/wgtechlabs/unthread-whatsapp-bot/sessions/92c82417-5f9c-4e2f-8c3b-3d6ac56cc425 Co-authored-by: warengonzaga <15052701+warengonzaga@users.noreply.github.com>
🔧 Container Build Complete - PR BuildBuild Status: ✅ Success 📦 Pull ImageDocker Hub: docker pull wgtechlabs/unthread-whatsapp-bot:pr-0ca8e1aGHCR: docker pull ghcr.io/wgtechlabs/unthread-whatsapp-bot:pr-0ca8e1a📋 Build Details
🏷️ Image Tags• 🔍 Testing Your Changes
🚀 Quick Start# Pull and run the container
Docker Hub: docker pull wgtechlabs/unthread-whatsapp-bot:pr-0ca8e1a
docker run <your-options> <image>🔒 Security Scan Results📋 Pre-Build Security Checks✅ Source Code Scan: 4 vulnerabilities found 🐳 Container Image Vulnerabilities
📋 View Vulnerability DetailsNode.js
📊 Detailed Security ReportsView detailed vulnerability reports in the GitHub Security tab. 🤖 Powered by Container Build Flow Action vv1.8.0 |
This aligns runtime/setup policy with the supported Node.js versions and removes drift between runtime, Docker, CI, and docs. Default runtime is now Node.js 26, while compatibility remains 22/24/26.
Runtime compatibility policy
package.jsonfor supported runtime/toolchain ranges..nvmrcdefaulting local development runtime to Node.js 26.Docker runtime alignment
CI support matrix
22,24, and26.Documentation consistency
22/24/26) with26as default.Original prompt
This pull request was created from Copilot chat.