feat!: Move to @fgiova/sqs-consumer#3
Conversation
BREAKING-CHANGE: some changes according to @fgiova/sqs-consumer synth
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the plugin from the legacy sqs-consumer library to @fgiova/sqs-consumer, updating the underlying SQS consumer implementation and modernizing the codebase. The changes include updating test infrastructure from ElasticMQ to LocalStack, migrating from ESLint to Biome for linting, and updating the minimum Fastify version requirement from 4.x to 5.x.
Key changes:
- Replace
sqs-consumerdependency with@fgiova/sqs-consumerand update the plugin API accordingly - Migrate test infrastructure from ElasticMQ to LocalStack for better AWS compatibility
- Switch from ESLint to Biome for code formatting and linting
Reviewed Changes
Copilot reviewed 17 out of 19 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/index.ts | Core plugin implementation updated to use new SQS consumer library with modified API structure |
| package.json | Dependencies updated to use new SQS consumer library, build tools, and linting setup |
| test/unit/index.ts | Test suite updated for new API structure and LocalStack integration |
| test/scripts/runners/localstack.js | New LocalStack container setup replacing ElasticMQ |
| test/helpers/sqsMessage.ts | Helper functions updated for LocalStack endpoint configuration |
| biome.json | New Biome configuration file |
| README.md | Documentation updated to reflect new API and requirements |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| // biome-ignore lint/correctness/noUnusedVariables: leave for clarity | ||
| container: localStackContainer, |
There was a problem hiding this comment.
The localStackContainer variable is marked as unused but kept for clarity. Consider either using it for proper cleanup or removing it entirely to avoid confusion.
| // biome-ignore lint/correctness/noUnusedVariables: leave for clarity | |
| container: localStackContainer, |
BREAKING-CHANGE: some changes according to @fgiova/sqs-consumer synth