chore!: require Node 26#41
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ef6d2cb04d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "engines": { | ||
| "node": ">=20" | ||
| "node": ">=26" | ||
| }, |
There was a problem hiding this comment.
Add Node engine to published packages
This only updates the private workspace root, so npm consumers of the packages being major-bumped here still receive package manifests without a Node 26 engine requirement: packages/cli/package.json, packages/core/package.json, packages/mcp/package.json, and packages/react/package.json do not declare engines.node. Because npm publishes each workspace's own package.json, installing tokenometer or @tokenometer/* on Node 20/22 will not warn or reject even though the release notes say these packages now require Node 26.
Useful? React with 👍 / 👎.
Summary
Verification