Define node version to be 18 or later to fix readline/promises issue

[zackkatz]

This solves initial install errors:

npm warn cli npm v10.8.2 does not support Node.js v16.13.0. This version of npm supports the following node versions: `^18.17.0 || >=20.5.0`. You can find the latest version at https://nodejs.org/.
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_UNKNOWN_BUILTIN_MODULE]: No such built-in module: node:readline/promises
    at new NodeError (node:internal/errors:371:5)
    at ESMLoader.builtinStrategy (node:internal/modules/esm/translators:276:11)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:236:14)
    at async link (node:internal/modules/esm/module_job:67:21) {
  code: 'ERR_UNKNOWN_BUILTIN_MODULE'
}

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds an "engines" constraint to package.json requiring Node.js >= 18.0.0. No other files or behaviors changed.

Changes

Cohort / File(s)	Summary of changes
Package metadata package.json	Added "engines": { "node": ">=18.0.0" }. No changes to scripts, dependencies, or exports.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The title succinctly captures the main change of updating the Node.js engine requirement to version 18 or later and notes the specific readline/promises issue it resolves, making the purpose immediately clear to reviewers.
Description Check	✅ Passed	The description directly relates to the changeset by presenting the installation warnings and runtime error that motivated adding the Node.js engine requirement, thus providing relevant context for the update.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3d26724 and de29118.

📒 Files selected for processing (1)

• package.json (1 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

package.json

📄 CodeRabbit inference engine (CLAUDE.md)

package.json: Ensure package.json main/bin/types point to built files in dist/
Include the templates directory in the published package (e.g., via files field or .npmignore)

Files:

• package.json

🔇 Additional comments (1)

package.json (1)

7-9: Engine constraint solves the runtime breakage.

Requiring Node.js ≥18.0.0 matches the minimum version that ships node:readline/promises, so this guards installs from the reported failure. Looks good.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}