Skip to content

Conversation

@alexander-schefe
Copy link
Contributor

@alexander-schefe alexander-schefe commented Aug 7, 2025

migrated from monolithic aws-sdk v2 to the new modular aws-sdk v3. includes many code changes because the whole structure of aws-sdk changed completely.

All local tests like creating new users, writing, refusing, accepting reports, etc. I ran on the localhost site worked without issue, but you never know.

Can't really test the email-pdf thingy with localhost i think.

May need to be reviewed carefully.

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 migrates the application from the monolithic AWS SDK v2 to the modular AWS SDK v3. This is a significant architectural change that affects how AWS services are imported and used throughout the codebase.

Key changes include:

  • Migration from single aws-sdk package to individual service clients (@aws-sdk/client-s3, @aws-sdk/client-ses, etc.)
  • Updated API patterns from promise-based .promise() calls to command-based .send() operations
  • Improved error handling with try-catch blocks replacing error response checking

Reviewed Changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/print/src/s3.ts Migrated S3 operations to v3 SDK with proper stream handling
packages/print/src/handler.ts Updated Lambda client import and removed .promise() calls
packages/email/src/s3.ts Migrated S3 operations and fixed function naming
packages/email/src/handler.ts Updated SES client and added stream processing utilities
packages/backend/src/db/*.ts Migrated DynamoDB operations to v3 SDK with command pattern
packages/backend/src/aws/*.ts Updated S3 and Lambda clients to v3 SDK
Multiple package.json files Updated dependencies from aws-sdk v2 to individual v3 packages

@alexander-schefe alexander-schefe force-pushed the chore/major-migration-to-modular-aws-sdk-3 branch from 817acd5 to 5273adf Compare August 14, 2025 09:03
@alexander-schefe alexander-schefe merged commit d7ccdb1 into main Aug 14, 2025
10 checks passed
@alexander-schefe alexander-schefe deleted the chore/major-migration-to-modular-aws-sdk-3 branch August 14, 2025 10:31
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.

2 participants