chore(deps): update dependency @types/node to v26 - #38
Conversation
|
PR author is in the excluded authors list. |
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
c31f034 to
b9a32d9
Compare
b9a32d9 to
0de18fc
Compare
89f618a to
08bf014
Compare
08bf014 to
4b094ae
Compare
| "devDependencies": { | ||
| "@eslint/js": "^9.39.4", | ||
| "@types/node": "^25.9.2", | ||
| "@types/node": "^26.0.0", |
There was a problem hiding this comment.
🔍 Major @types/node bump may not match supported Node runtimes
This is a devDependency-only major bump of @types/node from ^25 to ^26 (lockfile updated in the same commit). package.json declares no engines field, so there is no explicit statement of which Node runtimes the SDK supports; typing against v26 can silently allow use of APIs unavailable in older runtimes consumers might still run. Worth confirming CI typecheck/build passes and that the intended minimum Node version is documented.
Was this helpful? React with 👍 or 👎 to provide feedback.
fdae4da to
45ddeb4
Compare
| "devDependencies": { | ||
| "@eslint/js": "^9.39.4", | ||
| "@types/node": "^25.9.2", | ||
| "@types/node": "^26.0.0", |
There was a problem hiding this comment.
🔍 CONTRIBUTING documents pnpm but repo uses an npm lockfile
CONTRIBUTING.md instructs contributors to use pnpm install / pnpm test, yet the repo tracks package-lock.json and this bot bump edits it. This mismatch is pre-existing, not introduced here, but dependency-bump PRs will keep touching the npm lockfile while docs point elsewhere; worth aligning the docs or the tooling.
Was this helpful? React with 👍 or 👎 to provide feedback.
859df19 to
cbcca72
Compare
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This is a narrow Renovate update confined to ignored dependency metadata, affecting dev-time typechecking and lockfile resolution rather than production source behavior. Review by the core-team owners is warranted because both changed files are outside the author's ownership and the major type-package bump raises an unresolved Node-runtime compatibility question. Not approved because:
Review your spending limits in Billing settings. You can add or adjust custom eligibility rules. Learn more. |
cbcca72 to
df86971
Compare
🟢 WAVE BugBot — clearNo confident findings on the changed lines. Local review · $0 inference · wave-dispatch |
d120af9 to
22cbd92
Compare
22cbd92 to
0cc66ac
Compare
|
Your free trial PR review limit of 300 PRs has been reached. Please upgrade your plan to continue using CodeAnt AI. |


This PR contains the following updates:
^25.9.2→^26.0.0Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.