Skip to content

fix(#775): enforce Node.js >= 20 on startup#859

Merged
nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
snowrugar-beep:fix/issue-775-node-version-check
Jun 30, 2026
Merged

fix(#775): enforce Node.js >= 20 on startup#859
nanaf6203-bit merged 1 commit into
MettaChain:mainfrom
snowrugar-beep:fix/issue-775-node-version-check

Conversation

@snowrugar-beep

@snowrugar-beep snowrugar-beep commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Closes #775
Closes #776
Closes #777
Closes #778

Bootstrap now enforces Node.js >= 20 before importing any module. Several transitive deps (e.g. @nestjs/* v11) require Node 20+, but package.json's engines field only declared >= 18. The previous check was permissive at 18; this PR tightens it to 20, adds a Number.isNaN guard, and links users to the upgrade page.

  • Bumped threshold from < 18 to < 20 (matches issue spec)
  • Added Number.isNaN guard for malformed process.versions.node
  • Error message points users to https://nodejs.org/ to upgrade
  • Kept existing logger.error + process.exit(1) pattern for consistency
  • tsc clean, lint clean on main.ts, full test suite unaffected

@drips-wave

drips-wave Bot commented Jun 30, 2026

Copy link
Copy Markdown

@snowrugar-beep 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! 🚀

Learn more about application limits

@nanaf6203-bit nanaf6203-bit left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nanaf6203-bit nanaf6203-bit merged commit ec075db into MettaChain:main Jun 30, 2026
5 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

2 participants