-
Notifications
You must be signed in to change notification settings - Fork 0
chore(deps): update dependency @types/node to v26 #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -71,7 +71,7 @@ | |
| "devDependencies": { | ||
| "@sentry/node": "^9.4.0", | ||
| "@types/inquirer": "^9.0.7", | ||
| "@types/node": "^22.13.0", | ||
| "@types/node": "^26.0.0", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π‘ Changelog not updated for the dependency change The repository's contributor rules require the Unreleased section of the changelog to be updated for user-facing changes, but this change ( Rule reference
Was this helpful? React with π or π to provide feedback. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π Only a typings devDependency changed; no runtime code affected The diff touches only Was this helpful? React with π or π to provide feedback. |
||
| "@types/ws": "^8.5.14", | ||
| "tsup": "^8.0.0", | ||
| "typescript": "^5.9.3", | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π‘ Type definitions no longer match the oldest supported Node version
The project's Node type definitions are jumped to a major version that targets a much newer runtime (
"@types/node": "^26.0.0"atpackage.json:74) while the project still declares support for Node 18 and up, so code can be type-checked as valid yet fail on the older runtimes users are told are supported.Impact: Users on Node 18/20 may hit runtime errors from APIs that appear available during development.
Why the type-package major and the engines range must agree
package.json:49-51declares"engines": { "node": ">=18.0.0" }, andtsconfig.jsonsets"types": ["node"]. DefinitelyTyped's@types/nodemajors track a specific Node major line; a v26 typings set describes APIs (and API-shape changes) present only in much newer Node, so builds silently permit usage unavailable on Node 18/20. Either keep the typings aligned with the minimum supported major, or raise theenginesfloor along with this bump.Prompt for agents
Was this helpful? React with π or π to provide feedback.