Skip to content

Conversation

@royra
Copy link
Collaborator

@royra royra commented Jun 30, 2025

User description

  • min supported version 18->20
  • runtime version 18->24

PR Type

Enhancement


Description

  • Upgrade Node.js runtime from v18 to v24

  • Update minimum supported version from 18 to 20

  • Update Docker base images to node:24-alpine

  • Fix TypeScript compatibility with new Node types


Changes diagram

flowchart LR
  A["Node v18"] --> B["Node v24"]
  C["Min support v18"] --> D["Min support v20"]
  E["Docker node:18"] --> F["Docker node:24"]
  G["@types/node 18"] --> H["@types/node 24"]
Loading

Changes walkthrough 📝

Relevant files
Bug fix
1 files
events.ts
Fix TypeScript compatibility with Node v24                             
+1/-1     
Configuration changes
15 files
compile-site.yaml
Update CI Node version to 24.x                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
Dockerfile
Update base image to node:24-alpine                                           
+2/-2     
Dockerfile.dev
Update development image to node:24-alpine                             
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
package.json
Update minimum Node engine to 20                                                 
+1/-1     
.nvmrc
Update Node version to v24.3.0                                                     
+1/-1     
Dockerfile
Update Docker images to node:24-alpine                                     
+3/-3     
Dependencies
10 files
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update Node engine requirement and types                                 
+2/-2     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update Node engine and type dependencies                                 
+2/-2     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update @types/node to version 24                                                 
+1/-1     
package.json
Update Node engine and type dependencies                                 
+2/-2     

Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • - min supported version 18->20
    - runtime version 18->24
    @qodo-merge-pro
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Type Casting

    The addition of type casting to NodeJS.EventEmitter may indicate a compatibility issue with Node v24 types. This should be verified to ensure the casting is necessary and doesn't mask underlying type issues.

    return await events.once(target as NodeJS.EventEmitter, event, { signal }).then(
      () => undefined,

    @qodo-merge-pro
    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants