docs: point to engines.node for the Node requirement - #80
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The change is documentation-only and correctly points to the existing canonical Node requirement in package.json.
Review effort: Lite
Findings: None
What changed in this PR
This PR reduces documentation drift by removing the hard-coded Node version from the README and instead pointing readers to the canonical engines.node requirement already defined in package.json.
Changes:
- Update the README install note to reference
package.json’sengines.nodefield as the source of truth for the required Node version.
| File | Description |
|---|---|
| README.md | Replaces the duplicated Node version requirement with a reference to package.json’s engines.node. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Problem
The README restated the Node range (
>=24.19.0) thatengines.nodeinpackage.jsonalready declares, so every Node bump needed a matching doc edit.Solution
Point the install note at
engines.nodeinpackage.json.Proof
pnpm run checkpasses; the pre-push verify gate passed.