chore(deps): upgrade @types/node 20.19.31 → 26.1.1 - #25
Open
devin-ai-integration[bot] wants to merge 1 commit into
Open
chore(deps): upgrade @types/node 20.19.31 → 26.1.1#25devin-ai-integration[bot] wants to merge 1 commit into
devin-ai-integration[bot] wants to merge 1 commit into
Conversation
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps the
@types/nodedevDependency six majors (Node 20 typings → Node 26 typings). No source changes were needed — the Node typings are only pulled into the Vitest/spec compilation unit (tsconfig.spec.json→"types": ["node"]) and the Cypress unit (src/test/javascript/cypress/tsconfig.json→"types": ["cypress", "node"]); the Angular app build (tsconfig.app.json) declares"types": ["@angular/localize"], so app code never sees the Node globals. Nothing insrc/main/webapp,webpack/(all.js, not.ts),cypress*.config.ts, orvitest-base.config.tstouches an API whose typing changed in the 20→26 delta.Coordinate diff
26.1.1is the newest stable26.xon the live npm registry (published 2026-07-08, >7 days old);26.1.0/26.0.xare older, and there are no non-prerelease versions above it in the 26 line.Resolution proof (no split / no downgrade)
Every one of the ~20 transitive consumers (
ts-node,@types/express*,@types/ws,@inquirer/*,webpack-dev-server, …) dedupes onto the single hoisted26.1.1. Lockfile diff is confined to the@types/nodeandundici-typesentries — no lockfile-wide churn.Override / peer pin
None needed. No peer-dependency constraint in the tree caps
@types/node, andnode.versioninpom.xmlwas not touched — the typings did not require a runtime Node bump (the box's local Node 24.13.0, as pinned by the frontend-maven-plugin, builds and tests fine against Node 26 typings, since@types/nodemajors track typing surface rather than requiring a matching runtime).API migration
Explicitly none. Grep for the literal old version string
20.19.31across the repo (excludingnode_modules/target) matched onlypackage.jsonandpackage-lock.json, both updated. No docs, CI workflow, or config file referenced the old version.Gate output
Command set:
./npmw run lint+./npmw test(Vitest) +./mvnw verify.main)./npmw run lint(eslint)./npmw test(Vitest)./mvnw verifyTests run: 163, Failures: 0, Errors: 0, Skipped: 0Tests run: 163, Failures: 0, Errors: 0, Skipped: 0Vitest after the bump:
./mvnw verifyalso runsnpm run webapp:build(ng build --configuration development), so the Angular AOT/type-check pass is covered by the gate and is green.Pre-existing on base, not addressed here
Both verified by re-running the identical command on clean
main(git stash→npm install→ run →git stash pop); output was byte-identical before and after the bump:tsc -p src/test/javascript/cypress/tsconfig.json --noEmitreports 20 errors —TS7034/TS7005implicit-anyonuser/authority/bankAccount/label/operationin the*.cy.tsspecs,TS18048possibly-undefinedresponse, andTS2306/TS2339/TS7010fromcypress-audit's non-moduleindex.d.tsand thelighthousereport generator inplugins/index.ts. This standalonetscinvocation is not part of the gate; the error list is identical on base, so none of it is attributable to the typings bump.DependencyConvergencewarning onorg.apache.commons:commons-text(1.13.1vialiquibase-core → opencsvvs1.14.0direct fromliquibase-core). Warning-only, does not fail the build, and is unchanged on base — Java-side, explicitly out of scope.npm auditreports 115 vulnerabilities in the existing dependency tree. Unchanged by this PR (the two changed packages are dev-only type declarations); fixing it would require lockfile-wide bumps, which are out of scope for a one-dependency PR.Environment note (not a repo defect): the box's default JDK is 17, which trips the JHipster enforcer rule "JHipster supports JDK 21 to 25". All Maven runs above — baseline and post-change alike — used
JAVA_HOME=/usr/lib/jvm/java-21-openjdk-amd64.Link to Devin session: https://partner-workshops.devinenterprise.com/sessions/0ac64a2d5bee4b4b9bf5fd7da864cae9
Requested by: @mbatchelor81