Skip to content

Conversation

@v4ll3l1
Copy link
Contributor

@v4ll3l1 v4ll3l1 commented Nov 14, 2025

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades the project from Node 20 (LTS Iron) to Node 22, including significant dependency updates and code adjustments to align with the new Node version and updated libraries.

  • Upgraded all AWS SDK packages from v3.645.0 to v3.929.0
  • Upgraded Middy middleware from v4.7.0 to v6.4.5 (major version bump)
  • Updated development tooling including Nx (19.6.5 → 22.0.3), TypeScript (5.5.4 → 5.9.3), and ESLint dependencies
  • Removed callback parameter from middyfy handler invocation to align with Middy v6 changes

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.nvmrc Updated Node.js LTS version reference from "lts/iron" to "lts/jod"
.github/workflows/npm-publish.yml Updated Node.js version from 20.x to 22.x in CI workflow
.github/workflows/checks.yml Updated Node.js version from 20.x to 22.x in CI workflow
package.json Upgraded AWS SDK packages, Middy middleware (v4→v6), Nx tooling (v19→v22), TypeScript, and ESLint dependencies; added @middy/util package
packages/api/package.json Bumped package version from 0.1.0 to 0.2.4
packages/api/src/lib/apiHandler/apiHandler.ts Removed callback parameter from middyfy handler invocation (Middy v6 compatibility)
packages/s3/tsconfig.lib.json Added "node" to types array for Node.js type definitions
packages/dynamo/tsconfig.lib.json Added "node" to types array for Node.js type definitions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@sonarqubecloud
Copy link

@dgoerdes dgoerdes merged commit 755ca96 into main Nov 18, 2025
2 checks passed
@dgoerdes dgoerdes deleted the chore/update-to-node-22 branch November 18, 2025 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants