fix(cli): require Node.js >=20 and surface a clear error on older runtimes#438
fix(cli): require Node.js >=20 and surface a clear error on older runtimes#438tannerlinsley merged 1 commit intomainfrom
Conversation
…times Older Node (e.g. 16) lacks events.addAbortListener, used transitively by the CLI, which produced a cryptic SyntaxError at module instantiation. Declare engines.node: ">=20" on both packages so installers warn, and add an early runtime check in the bin so users get an actionable message before any modules load. Closes #433 https://claude.ai/code/session_01N5k5sqBgxYX9XMatM6TGcE
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis PR enforces Node.js 20+ as a minimum runtime requirement for Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Older Node (e.g. 16) lacks events.addAbortListener, used transitively by
the CLI, which produced a cryptic SyntaxError at module instantiation.
Declare engines.node: ">=20" on both packages so installers warn, and add
an early runtime check in the bin so users get an actionable message
before any modules load.
Closes #433
https://claude.ai/code/session_01N5k5sqBgxYX9XMatM6TGcE
Summary by CodeRabbit