ci(node): add matrix testing for Node 18 and LTS versions with engine checks - #243
Merged
Emmy123222 merged 7 commits intoSep 3, 2026
Conversation
node cannot run .ts files directly on Node 18, causing the CI gate job to fail with ERR_UNKNOWN_FILE_EXTENSION. Rewrite check-node-version as plain .js with zero dependencies so it runs on any Node version without tsx. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
…ements Vite 8 requires ^20.19.0 || >=22.12.0 but package.json declared >=18.0.0, causing Node 18 CI jobs to fail with util.styleText errors. Update engines field, CI matrix, and version check script to align with the actual supported range. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
|
@Risktaker001 is attempting to deploy a commit to the Emmanuel's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Risktaker001 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
…-2n8dgwdq Resolve merge conflict in .github/workflows/ci.yml: - Kept branch's Node 22 coverage gate with caching and real threshold verification - Included upstream's supply-chain job (SBOM generation + OSV vulnerability scanning) - Removed echo placeholder from fork's main 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
Contributor
Author
|
it has been resolved |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #202
Summary
Updates the CI workflow matrix to test across all supported Node.js versions (Node 18+ and current LTS versions) and enforces strict Node runtime validation using package
enginesconstraints.Changes Included
CI Matrix Pipeline (
.github/workflows/ci.yml):Added a job strategy matrix to run automated tests across Node 18 (minimum supported) and current LTS versions (Node 20, Node 22).
Aligned matrix checks across Express backend, Vercel deployments, browser runtime, and MCP server boundaries.
Engine Enforcement (
package.json,.nvmrc):Configured
package.json(enginesfield and.npmrc/engine-strict=true) to ensure early failure with a clear error message when executed on unsupported Node versions.Updated
.nvmrcto align local development defaults with current LTS while preserving Node 18+ compatibility.Coverage & Documentation:
Updated build and CI setup documentation to explicitly list supported Node runtime ranges.
Preserved x402 payment settlement semantics across test suites for paid routes.
Verification