sqs-consumer 15.0.1 → 15.0.2

package/CHANGELOG.md

@@ -1,3 +1,41 @@
+## [15.0.2](https://github.com/bbc/sqs-consumer/compare/v15.0.1...v15.0.2) (2026-05-26)
+
+### Bug Fixes
+
+* handle batch failures differently ([#807](https://github.com/bbc/sqs-consumer/issues/807)) ([0988bca](https://github.com/bbc/sqs-consumer/commit/0988bca2b2bc0dee9d97c25268aeee99585e9d5c))
+* make coverage reports genhtml-compatible ([#803](https://github.com/bbc/sqs-consumer/issues/803)) ([8da9097](https://github.com/bbc/sqs-consumer/commit/8da9097f870d5cf2bf02dd970f2e90d76cd82d42))
+
+### Chores
+
+* change to only run on dispatch ([d35f4b3](https://github.com/bbc/sqs-consumer/commit/d35f4b366de9e52d405ff830c8d5c168a45c481e))
+* **deps-dev:** bump @cucumber/cucumber from 12.7.0 to 12.8.2 ([#796](https://github.com/bbc/sqs-consumer/issues/796)) ([de53034](https://github.com/bbc/sqs-consumer/commit/de53034e34e7c84d88dd716d4f94a100bac51fb5))
+* **deps-dev:** bump @cucumber/cucumber from 12.8.2 to 12.8.3 ([#808](https://github.com/bbc/sqs-consumer/issues/808)) ([7144968](https://github.com/bbc/sqs-consumer/commit/7144968c51d0e646b9d852a6fa72a6c94d70baad))
+* **deps-dev:** bump @semantic-release/github from 12.0.6 to 12.0.8 ([#800](https://github.com/bbc/sqs-consumer/issues/800)) ([c5fe1f8](https://github.com/bbc/sqs-consumer/commit/c5fe1f888fec145d61b313e38688fb6799c88b23))
+* **deps-dev:** bump @semantic-release/release-notes-generator ([#799](https://github.com/bbc/sqs-consumer/issues/799)) ([6c24b1b](https://github.com/bbc/sqs-consumer/commit/6c24b1bbfa9e98eedc9ee23712187b1aa5be6344))
+* **deps-dev:** bump @types/node from 25.5.0 to 25.6.2 ([#798](https://github.com/bbc/sqs-consumer/issues/798)) ([cecfd8d](https://github.com/bbc/sqs-consumer/commit/cecfd8da9bb8bf360ae16ebcf81796fa0f1c955e))
+* **deps-dev:** bump @vitest/coverage-v8 from 4.1.5 to 4.1.6 ([#809](https://github.com/bbc/sqs-consumer/issues/809)) ([efe4953](https://github.com/bbc/sqs-consumer/commit/efe49535be6732ae8a7ed47add521f1ce5f128d3))
+* **deps-dev:** bump oxfmt from 0.44.0 to 0.46.0 ([#786](https://github.com/bbc/sqs-consumer/issues/786)) ([a61d0b7](https://github.com/bbc/sqs-consumer/commit/a61d0b7308f40b0836198a8dfbd17181400fd8c7))
+* **deps-dev:** bump sinon and @types/sinon ([#788](https://github.com/bbc/sqs-consumer/issues/788)) ([da2a4d9](https://github.com/bbc/sqs-consumer/commit/da2a4d9ebc84fb0929c7131b3c57d293064e623a))
+* **deps-dev:** bump sqs-producer from 8.0.2 to 8.0.9 ([#810](https://github.com/bbc/sqs-consumer/issues/810)) ([da4afb0](https://github.com/bbc/sqs-consumer/commit/da4afb0ff29e296e6d3044b6db39880bdec62c19))
+* **deps-dev:** bump typedoc from 0.28.17 to 0.28.19 ([#787](https://github.com/bbc/sqs-consumer/issues/787)) ([9467cec](https://github.com/bbc/sqs-consumer/commit/9467ceca1e844f5934c0f3d9ba654f62b8739c60))
+* **deps-dev:** bump vitest from 4.1.2 to 4.1.5 ([#790](https://github.com/bbc/sqs-consumer/issues/790)) ([7720d4f](https://github.com/bbc/sqs-consumer/commit/7720d4fb0dab1dd1db766597cf1798014d388435))
+* **deps-dev:** bump vitest from 4.1.5 to 4.1.6 ([#811](https://github.com/bbc/sqs-consumer/issues/811)) ([f14a0ef](https://github.com/bbc/sqs-consumer/commit/f14a0ef5fd53443abdea215af4301d669066ef68))
+* **deps:** bump @aws-sdk/client-sqs from 3.1034.0 to 3.1036.0 ([#789](https://github.com/bbc/sqs-consumer/issues/789)) ([e54e664](https://github.com/bbc/sqs-consumer/commit/e54e664b474dc97f40cc82664f664b40b180b713))
+* **deps:** bump actions/add-to-project from 1.0.2 to 2.0.0 ([#792](https://github.com/bbc/sqs-consumer/issues/792)) ([b1eeec1](https://github.com/bbc/sqs-consumer/commit/b1eeec1efa8dc9e2d419e41e3393b02f3b6e7d67))
+* **deps:** bump actions/dependency-review-action from 4.9.0 to 5.0.0 ([#795](https://github.com/bbc/sqs-consumer/issues/795)) ([ea16075](https://github.com/bbc/sqs-consumer/commit/ea1607596771ed5b55e65c77c92c774cae1070c3))
+* **deps:** bump actions/stale from 10.2.0 to 10.3.0 ([#805](https://github.com/bbc/sqs-consumer/issues/805)) ([f689b90](https://github.com/bbc/sqs-consumer/commit/f689b90faa61e87f142e8dbb6d4f22fab3d81e96))
+* **deps:** bump dessant/lock-threads from 6.0.0 to 6.0.2 ([#812](https://github.com/bbc/sqs-consumer/issues/812)) ([96b83ad](https://github.com/bbc/sqs-consumer/commit/96b83adf35accacc504d0a7679350b4d9b1b5fa2))
+* **deps:** bump fast-xml-builder ([#794](https://github.com/bbc/sqs-consumer/issues/794)) ([dc325d0](https://github.com/bbc/sqs-consumer/commit/dc325d0ed59e006e58e51fb71de3c58f968856c7))
+* **deps:** bump github/codeql-action from 4.35.2 to 4.35.4 ([#793](https://github.com/bbc/sqs-consumer/issues/793)) ([fb6e76e](https://github.com/bbc/sqs-consumer/commit/fb6e76eb86e8307df849445f467f0085ae81e463))
+* **deps:** bump github/codeql-action from 4.35.4 to 4.35.5 ([#802](https://github.com/bbc/sqs-consumer/issues/802)) ([a232427](https://github.com/bbc/sqs-consumer/commit/a232427a7455a536e8adf81db0824ecadb9126f5))
+* **deps:** bump github/codeql-action from 4.35.5 to 4.36.0 ([#814](https://github.com/bbc/sqs-consumer/issues/814)) ([2725d58](https://github.com/bbc/sqs-consumer/commit/2725d5894510fd7945e16469be3222dfc3e76b01))
+* **deps:** bump zgosalvez/github-actions-report-lcov ([#801](https://github.com/bbc/sqs-consumer/issues/801)) ([b1de26f](https://github.com/bbc/sqs-consumer/commit/b1de26ff931a74c38c1bf1263441a04aaabbc357))
+* **deps:** bump zgosalvez/github-actions-report-lcov ([#813](https://github.com/bbc/sqs-consumer/issues/813)) ([2bc51e0](https://github.com/bbc/sqs-consumer/commit/2bc51e0671cb92e2c25b1fd5388b669e90d292d4))
+* remove git step ([a2b699c](https://github.com/bbc/sqs-consumer/commit/a2b699cf26ae1de480b086ac8f23cbb12eb16e3a))
+* remove unused action ([7b027a0](https://github.com/bbc/sqs-consumer/commit/7b027a03964910123534e8f63d6e55712f5b059b))
+* update security policy ([464c7f0](https://github.com/bbc/sqs-consumer/commit/464c7f03fe82a3a4ff73bcbcb611805c8bc7c38d))
+* updating brace expansion ([704890e](https://github.com/bbc/sqs-consumer/commit/704890e36c264ec2044fa07cf30130c11e1cd7e3))
+
 ## [15.0.1](https://github.com/bbc/sqs-consumer/compare/v15.0.0...v15.0.1) (2026-05-02)
 
 ### Chores

package/dist/cjs/consumer.d.ts

@@ -118,6 +118,7 @@ export declare class Consumer extends TypedEventEmitter {
      * @param timeout The new timeout that should be set
      */
     private changeVisibilityTimeoutBatch;
+    private emitBatchFailures;
     /**
      * Trigger the applications handleMessage function
      * @param message The message that was received from SQS
@@ -138,4 +139,5 @@ export declare class Consumer extends TypedEventEmitter {
      * @param messages The messages that should be deleted from SQS
      */
     private deleteMessageBatch;
+    private getSuccessfulBatchMessages;
 }

package/dist/cjs/consumer.js

@@ -39,12 +39,14 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
         this.alwaysAcknowledge = options.alwaysAcknowledge ?? false;
         this.extendedAWSErrors = options.extendedAWSErrors ?? false;
         this.strictReturn = options.strictReturn ?? false;
-        this.sqs =
-            options.sqs ||
-                new client_sqs_1.SQSClient({
-                    useQueueUrlAsEndpoint: options.useQueueUrlAsEndpoint ?? true,
-                    region: options.region || process.env.AWS_REGION || "eu-west-1",
-                });
+        if (options.sqs) {
+            this.sqs = options.sqs;
+        }
+        else {
+            const useQueueUrlAsEndpoint = options.useQueueUrlAsEndpoint ?? true;
+            const region = options.region || process.env.AWS_REGION || "eu-west-1";
+            this.sqs = new client_sqs_1.SQSClient({ useQueueUrlAsEndpoint, region });
+        }
     }
     /**
      * Creates a new SQS consumer.
@@ -72,10 +74,11 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
      * A reusable options object for sqs.send that's used to avoid duplication.
      */
     get sqsSendOptions() {
+        const abortSignal = this.abortController?.signal || new AbortController().signal;
         return {
             // return the current abortController signal or a fresh signal that has not been aborted.
             // This effectively defaults the signal sent to the AWS SDK to not aborted
-            abortSignal: this.abortController?.signal || new AbortController().signal,
+            abortSignal,
         };
     }
     /**
@@ -185,11 +188,13 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
                 this.emitError(err);
             }
             catch (listenerErr) {
-                logger_js_1.logger.warn(`An error event listener threw an error: ${listenerErr instanceof Error ? listenerErr.message : String(listenerErr)}`);
+                const listenerErrorMessage = listenerErr instanceof Error ? listenerErr.message : String(listenerErr);
+                logger_js_1.logger.warn(`An error event listener threw an error: ${listenerErrorMessage}`);
             }
             if ((0, errors_js_1.isConnectionError)(err)) {
+                const errorCode = err.code || "Unknown";
                 logger_js_1.logger.debug("authentication_error", {
-                    code: err.code || "Unknown",
+                    code: errorCode,
                     detail: "There was an authentication error. Pausing before retrying.",
                 });
                 currentPollingTimeout = this.authenticationErrorTimeout;
@@ -295,8 +300,8 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
             }
             const ackedMessages = await this.executeBatchHandler(messages);
             if (ackedMessages?.length > 0) {
-                await this.deleteMessageBatch(ackedMessages);
-                ackedMessages.forEach((message) => {
+                const deletedMessages = await this.deleteMessageBatch(ackedMessages);
+                deletedMessages.forEach((message) => {
                     this.emit("message_processed", message);
                 });
             }
@@ -363,12 +368,23 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
             })),
         };
         try {
-            return await this.sqs.send(new client_sqs_1.ChangeMessageVisibilityBatchCommand(params), this.sqsSendOptions);
+            const response = await this.sqs.send(new client_sqs_1.ChangeMessageVisibilityBatchCommand(params), this.sqsSendOptions);
+            if (response) {
+                this.emitBatchFailures(response.Failed, messages);
+            }
+            return response;
         }
         catch (err) {
             this.emit("error", (0, errors_js_1.toSQSError)(err, `Error changing visibility timeout: ${err.message}`, this.extendedAWSErrors, this.queueUrl, messages), messages);
         }
     }
+    emitBatchFailures(failed, messages) {
+        if (!failed?.length) {
+            return;
+        }
+        const failedIds = failed.map((entry) => entry.Id).filter((id) => Boolean(id));
+        this.emit("error", new Error(`Batch operation failed for entries with Ids: ${failedIds.join(", ")}`), messages);
+    }
     /**
      * Trigger the applications handleMessage function
      * @param message The message that was received from SQS
@@ -485,7 +501,7 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
             logger_js_1.logger.debug("skipping_delete", {
                 detail: "Skipping message delete since shouldDeleteMessages is set to false",
             });
-            return;
+            return messages;
         }
         logger_js_1.logger.debug("deleting_messages", {
             messageIds: messages.map((msg) => msg.MessageId),
@@ -498,11 +514,23 @@ class Consumer extends emitter_js_1.TypedEventEmitter {
             })),
         };
         try {
-            await this.sqs.send(new client_sqs_1.DeleteMessageBatchCommand(deleteParams), this.sqsSendOptions);
+            const response = await this.sqs.send(new client_sqs_1.DeleteMessageBatchCommand(deleteParams), this.sqsSendOptions);
+            if (!response) {
+                return messages;
+            }
+            this.emitBatchFailures(response.Failed, messages);
+            return this.getSuccessfulBatchMessages(response, messages);
         }
         catch (err) {
             throw (0, errors_js_1.toSQSError)(err, `SQS delete message failed: ${err.message}`, this.extendedAWSErrors, this.queueUrl, messages);
         }
     }
+    getSuccessfulBatchMessages(response, messages) {
+        if (!response.Successful) {
+            return [];
+        }
+        const successfulIds = new Set(response.Successful.map(({ Id }) => Id));
+        return messages.filter((message) => successfulIds.has(message.MessageId));
+    }
 }
 exports.Consumer = Consumer;

package/dist/cjs/errors.js

@@ -5,6 +5,8 @@ exports.isConnectionError = isConnectionError;
 exports.toSQSError = toSQSError;
 exports.toStandardError = toStandardError;
 exports.toTimeoutError = toTimeoutError;
+const DEFAULT_TIMEOUT_ERROR_MESSAGE = "Operation timed out.";
+const DEFAULT_STANDARD_ERROR_MESSAGE = "An unexpected error occurred:";
 class SQSError extends Error {
     constructor(message) {
         super(message);
@@ -13,18 +15,20 @@ class SQSError extends Error {
 }
 exports.SQSError = SQSError;
 class TimeoutError extends Error {
-    constructor(message = "Operation timed out.") {
-        super(message);
-        this.message = message;
+    constructor(message) {
+        const errorMessage = message === undefined ? DEFAULT_TIMEOUT_ERROR_MESSAGE : message;
+        super(errorMessage);
+        this.message = errorMessage;
         this.name = "TimeoutError";
         this.messageIds = [];
     }
 }
 exports.TimeoutError = TimeoutError;
 class StandardError extends Error {
-    constructor(message = "An unexpected error occurred:") {
-        super(message);
-        this.message = message;
+    constructor(message) {
+        const errorMessage = message === undefined ? DEFAULT_STANDARD_ERROR_MESSAGE : message;
+        super(errorMessage);
+        this.message = errorMessage;
         this.name = "StandardError";
         this.messageIds = [];
     }

package/dist/esm/consumer.d.ts

@@ -118,6 +118,7 @@ export declare class Consumer extends TypedEventEmitter {
      * @param timeout The new timeout that should be set
      */
     private changeVisibilityTimeoutBatch;
+    private emitBatchFailures;
     /**
      * Trigger the applications handleMessage function
      * @param message The message that was received from SQS
@@ -138,4 +139,5 @@ export declare class Consumer extends TypedEventEmitter {
      * @param messages The messages that should be deleted from SQS
      */
     private deleteMessageBatch;
+    private getSuccessfulBatchMessages;
 }

package/dist/esm/consumer.js

@@ -36,12 +36,14 @@ export class Consumer extends TypedEventEmitter {
         this.alwaysAcknowledge = options.alwaysAcknowledge ?? false;
         this.extendedAWSErrors = options.extendedAWSErrors ?? false;
         this.strictReturn = options.strictReturn ?? false;
-        this.sqs =
-            options.sqs ||
-                new SQSClient({
-                    useQueueUrlAsEndpoint: options.useQueueUrlAsEndpoint ?? true,
-                    region: options.region || process.env.AWS_REGION || "eu-west-1",
-                });
+        if (options.sqs) {
+            this.sqs = options.sqs;
+        }
+        else {
+            const useQueueUrlAsEndpoint = options.useQueueUrlAsEndpoint ?? true;
+            const region = options.region || process.env.AWS_REGION || "eu-west-1";
+            this.sqs = new SQSClient({ useQueueUrlAsEndpoint, region });
+        }
     }
     /**
      * Creates a new SQS consumer.
@@ -69,10 +71,11 @@ export class Consumer extends TypedEventEmitter {
      * A reusable options object for sqs.send that's used to avoid duplication.
      */
     get sqsSendOptions() {
+        const abortSignal = this.abortController?.signal || new AbortController().signal;
         return {
             // return the current abortController signal or a fresh signal that has not been aborted.
             // This effectively defaults the signal sent to the AWS SDK to not aborted
-            abortSignal: this.abortController?.signal || new AbortController().signal,
+            abortSignal,
         };
     }
     /**
@@ -182,11 +185,13 @@ export class Consumer extends TypedEventEmitter {
                 this.emitError(err);
             }
             catch (listenerErr) {
-                logger.warn(`An error event listener threw an error: ${listenerErr instanceof Error ? listenerErr.message : String(listenerErr)}`);
+                const listenerErrorMessage = listenerErr instanceof Error ? listenerErr.message : String(listenerErr);
+                logger.warn(`An error event listener threw an error: ${listenerErrorMessage}`);
             }
             if (isConnectionError(err)) {
+                const errorCode = err.code || "Unknown";
                 logger.debug("authentication_error", {
-                    code: err.code || "Unknown",
+                    code: errorCode,
                     detail: "There was an authentication error. Pausing before retrying.",
                 });
                 currentPollingTimeout = this.authenticationErrorTimeout;
@@ -292,8 +297,8 @@ export class Consumer extends TypedEventEmitter {
             }
             const ackedMessages = await this.executeBatchHandler(messages);
             if (ackedMessages?.length > 0) {
-                await this.deleteMessageBatch(ackedMessages);
-                ackedMessages.forEach((message) => {
+                const deletedMessages = await this.deleteMessageBatch(ackedMessages);
+                deletedMessages.forEach((message) => {
                     this.emit("message_processed", message);
                 });
             }
@@ -360,12 +365,23 @@ export class Consumer extends TypedEventEmitter {
             })),
         };
         try {
-            return await this.sqs.send(new ChangeMessageVisibilityBatchCommand(params), this.sqsSendOptions);
+            const response = await this.sqs.send(new ChangeMessageVisibilityBatchCommand(params), this.sqsSendOptions);
+            if (response) {
+                this.emitBatchFailures(response.Failed, messages);
+            }
+            return response;
         }
         catch (err) {
             this.emit("error", toSQSError(err, `Error changing visibility timeout: ${err.message}`, this.extendedAWSErrors, this.queueUrl, messages), messages);
         }
     }
+    emitBatchFailures(failed, messages) {
+        if (!failed?.length) {
+            return;
+        }
+        const failedIds = failed.map((entry) => entry.Id).filter((id) => Boolean(id));
+        this.emit("error", new Error(`Batch operation failed for entries with Ids: ${failedIds.join(", ")}`), messages);
+    }
     /**
      * Trigger the applications handleMessage function
      * @param message The message that was received from SQS
@@ -482,7 +498,7 @@ export class Consumer extends TypedEventEmitter {
             logger.debug("skipping_delete", {
                 detail: "Skipping message delete since shouldDeleteMessages is set to false",
             });
-            return;
+            return messages;
         }
         logger.debug("deleting_messages", {
             messageIds: messages.map((msg) => msg.MessageId),
@@ -495,10 +511,22 @@ export class Consumer extends TypedEventEmitter {
             })),
         };
         try {
-            await this.sqs.send(new DeleteMessageBatchCommand(deleteParams), this.sqsSendOptions);
+            const response = await this.sqs.send(new DeleteMessageBatchCommand(deleteParams), this.sqsSendOptions);
+            if (!response) {
+                return messages;
+            }
+            this.emitBatchFailures(response.Failed, messages);
+            return this.getSuccessfulBatchMessages(response, messages);
         }
         catch (err) {
             throw toSQSError(err, `SQS delete message failed: ${err.message}`, this.extendedAWSErrors, this.queueUrl, messages);
         }
     }
+    getSuccessfulBatchMessages(response, messages) {
+        if (!response.Successful) {
+            return [];
+        }
+        const successfulIds = new Set(response.Successful.map(({ Id }) => Id));
+        return messages.filter((message) => successfulIds.has(message.MessageId));
+    }
 }

package/dist/esm/errors.js

@@ -1,3 +1,5 @@
+const DEFAULT_TIMEOUT_ERROR_MESSAGE = "Operation timed out.";
+const DEFAULT_STANDARD_ERROR_MESSAGE = "An unexpected error occurred:";
 class SQSError extends Error {
     constructor(message) {
         super(message);
@@ -5,17 +7,19 @@ class SQSError extends Error {
     }
 }
 class TimeoutError extends Error {
-    constructor(message = "Operation timed out.") {
-        super(message);
-        this.message = message;
+    constructor(message) {
+        const errorMessage = message === undefined ? DEFAULT_TIMEOUT_ERROR_MESSAGE : message;
+        super(errorMessage);
+        this.message = errorMessage;
         this.name = "TimeoutError";
         this.messageIds = [];
     }
 }
 class StandardError extends Error {
-    constructor(message = "An unexpected error occurred:") {
-        super(message);
-        this.message = message;
+    constructor(message) {
+        const errorMessage = message === undefined ? DEFAULT_STANDARD_ERROR_MESSAGE : message;
+        super(errorMessage);
+        this.message = errorMessage;
         this.name = "StandardError";
         this.messageIds = [];
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sqs-consumer",
-  "version": "15.0.1",
+  "version": "15.0.2",
   "description": "Build SQS-based Node applications without the boilerplate",
   "keywords": [
     "aws-sqs",
@@ -63,38 +63,38 @@
     "generate-docs": "typedoc"
   },
   "dependencies": {
-    "@aws-sdk/client-sqs": "^3.1034.0",
+    "@aws-sdk/client-sqs": "^3.1036.0",
     "debug": "^4.4.3"
   },
   "devDependencies": {
-    "@cucumber/cucumber": "^12.7.0",
+    "@cucumber/cucumber": "^12.8.3",
     "@sebbo2002/semantic-release-jsr": "^3.2.1",
     "@semantic-release/changelog": "^6.0.3",
     "@semantic-release/commit-analyzer": "^13.0.1",
     "@semantic-release/git": "^10.0.1",
-    "@semantic-release/github": "^12.0.6",
+    "@semantic-release/github": "^12.0.8",
     "@semantic-release/npm": "^13.1.5",
-    "@semantic-release/release-notes-generator": "^14.1.0",
+    "@semantic-release/release-notes-generator": "^14.1.1",
     "@types/chai": "^5.2.2",
-    "@types/node": "^25.5.0",
-    "@types/sinon": "^17.0.4",
-    "@vitest/coverage-v8": "^4.1.2",
+    "@types/node": "^25.6.2",
+    "@types/sinon": "^21.0.1",
+    "@vitest/coverage-v8": "^4.1.6",
     "chai": "^6.2.2",
     "conventional-changelog-conventionalcommits": "^9.3.0",
     "jsr": "^0.14.3",
-    "oxfmt": "^0.44.0",
+    "oxfmt": "^0.46.0",
     "oxlint": "^1.61.0",
     "p-event": "^7.1.0",
     "semantic-release": "^25.0.3",
-    "sinon": "^21.0.0",
-    "sqs-producer": "^8.0.2",
+    "sinon": "^21.1.2",
+    "sqs-producer": "^8.0.9",
     "ts-node": "^10.9.2",
-    "typedoc": "^0.28.17",
+    "typedoc": "^0.28.19",
     "typescript": "^5.9.3",
-    "vitest": "^4.1.2"
+    "vitest": "^4.1.6"
   },
   "peerDependencies": {
-    "@aws-sdk/client-sqs": "^3.1034.0"
+    "@aws-sdk/client-sqs": "^3.1036.0"
   },
   "overrides": {
     "cross-spawn": "^7.0.3"

package/release.config.cjs

@@ -35,13 +35,6 @@ module.exports = {
     ],
     "@semantic-release/changelog",
     "@semantic-release/npm",
-    [
-      "@semantic-release/git",
-      {
-        assets: ["package.json", "package-lock.json", "CHANGELOG.md"],
-        message: "chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}",
-      },
-    ],
     "@semantic-release/github",
     "@sebbo2002/semantic-release-jsr",
   ],

package/src/consumer.ts

@@ -7,6 +7,7 @@ import {
   ReceiveMessageCommand,
 } from "@aws-sdk/client-sqs";
 import type {
+  BatchResultErrorEntry,
   Message,
   ChangeMessageVisibilityCommandInput,
   ChangeMessageVisibilityCommandOutput,
@@ -14,6 +15,7 @@ import type {
   ChangeMessageVisibilityBatchCommandOutput,
   DeleteMessageCommandInput,
   DeleteMessageBatchCommandInput,
+  DeleteMessageBatchCommandOutput,
   ReceiveMessageCommandInput,
   ReceiveMessageCommandOutput,
   QueueAttributeName,
@@ -93,12 +95,13 @@ export class Consumer extends TypedEventEmitter {
     this.alwaysAcknowledge = options.alwaysAcknowledge ?? false;
     this.extendedAWSErrors = options.extendedAWSErrors ?? false;
     this.strictReturn = options.strictReturn ?? false;
-    this.sqs =
-      options.sqs ||
-      new SQSClient({
-        useQueueUrlAsEndpoint: options.useQueueUrlAsEndpoint ?? true,
-        region: options.region || process.env.AWS_REGION || "eu-west-1",
-      });
+    if (options.sqs) {
+      this.sqs = options.sqs;
+    } else {
+      const useQueueUrlAsEndpoint = options.useQueueUrlAsEndpoint ?? true;
+      const region = options.region || process.env.AWS_REGION || "eu-west-1";
+      this.sqs = new SQSClient({ useQueueUrlAsEndpoint, region });
+    }
   }
 
   /**
@@ -131,10 +134,12 @@ export class Consumer extends TypedEventEmitter {
    * A reusable options object for sqs.send that's used to avoid duplication.
    */
   private get sqsSendOptions(): { abortSignal: AbortSignal } {
+    const abortSignal = this.abortController?.signal || new AbortController().signal;
+
     return {
       // return the current abortController signal or a fresh signal that has not been aborted.
       // This effectively defaults the signal sent to the AWS SDK to not aborted
-      abortSignal: this.abortController?.signal || new AbortController().signal,
+      abortSignal,
     };
   }
 
@@ -261,13 +266,15 @@ export class Consumer extends TypedEventEmitter {
         try {
           this.emitError(err);
         } catch (listenerErr) {
-          logger.warn(
-            `An error event listener threw an error: ${listenerErr instanceof Error ? listenerErr.message : String(listenerErr)}`,
-          );
+          const listenerErrorMessage =
+            listenerErr instanceof Error ? listenerErr.message : String(listenerErr);
+          logger.warn(`An error event listener threw an error: ${listenerErrorMessage}`);
         }
         if (isConnectionError(err)) {
+          const errorCode = err.code || "Unknown";
+
           logger.debug("authentication_error", {
-            code: err.code || "Unknown",
+            code: errorCode,
             detail: "There was an authentication error. Pausing before retrying.",
           });
           currentPollingTimeout = this.authenticationErrorTimeout;
@@ -396,9 +403,9 @@ export class Consumer extends TypedEventEmitter {
       const ackedMessages: Message[] = await this.executeBatchHandler(messages);
 
       if (ackedMessages?.length > 0) {
-        await this.deleteMessageBatch(ackedMessages);
+        const deletedMessages = await this.deleteMessageBatch(ackedMessages);
 
-        ackedMessages.forEach((message: Message): void => {
+        deletedMessages.forEach((message: Message): void => {
           this.emit("message_processed", message);
         });
       }
@@ -483,10 +490,16 @@ export class Consumer extends TypedEventEmitter {
       })),
     };
     try {
-      return await this.sqs.send(
+      const response = await this.sqs.send(
         new ChangeMessageVisibilityBatchCommand(params),
         this.sqsSendOptions,
       );
+
+      if (response) {
+        this.emitBatchFailures(response.Failed, messages);
+      }
+
+      return response;
     } catch (err) {
       this.emit(
         "error",
@@ -502,6 +515,23 @@ export class Consumer extends TypedEventEmitter {
     }
   }
 
+  private emitBatchFailures(
+    failed: BatchResultErrorEntry[] | undefined,
+    messages: Message[],
+  ): void {
+    if (!failed?.length) {
+      return;
+    }
+
+    const failedIds = failed.map((entry) => entry.Id).filter((id): id is string => Boolean(id));
+
+    this.emit(
+      "error",
+      new Error(`Batch operation failed for entries with Ids: ${failedIds.join(", ")}`),
+      messages,
+    );
+  }
+
   /**
    * Trigger the applications handleMessage function
    * @param message The message that was received from SQS
@@ -648,12 +678,12 @@ export class Consumer extends TypedEventEmitter {
    * Delete a batch of messages from the SQS queue.
    * @param messages The messages that should be deleted from SQS
    */
-  private async deleteMessageBatch(messages: Message[]): Promise<void> {
+  private async deleteMessageBatch(messages: Message[]): Promise<Message[]> {
     if (!this.shouldDeleteMessages) {
       logger.debug("skipping_delete", {
         detail: "Skipping message delete since shouldDeleteMessages is set to false",
       });
-      return;
+      return messages;
     }
     logger.debug("deleting_messages", {
       messageIds: messages.map((msg: Message) => msg.MessageId),
@@ -668,7 +698,18 @@ export class Consumer extends TypedEventEmitter {
     };
 
     try {
-      await this.sqs.send(new DeleteMessageBatchCommand(deleteParams), this.sqsSendOptions);
+      const response = await this.sqs.send(
+        new DeleteMessageBatchCommand(deleteParams),
+        this.sqsSendOptions,
+      );
+
+      if (!response) {
+        return messages;
+      }
+
+      this.emitBatchFailures(response.Failed, messages);
+
+      return this.getSuccessfulBatchMessages(response, messages);
     } catch (err) {
       throw toSQSError(
         err,
@@ -679,4 +720,16 @@ export class Consumer extends TypedEventEmitter {
       );
     }
   }
+
+  private getSuccessfulBatchMessages(
+    response: DeleteMessageBatchCommandOutput,
+    messages: Message[],
+  ): Message[] {
+    if (!response.Successful) {
+      return [];
+    }
+
+    const successfulIds = new Set(response.Successful.map(({ Id }) => Id));
+    return messages.filter((message) => successfulIds.has(message.MessageId));
+  }
 }

package/src/errors.ts

@@ -2,6 +2,9 @@ import type { Message } from "@aws-sdk/client-sqs";
 
 import type { AWSError } from "./types.js";
 
+const DEFAULT_TIMEOUT_ERROR_MESSAGE = "Operation timed out.";
+const DEFAULT_STANDARD_ERROR_MESSAGE = "An unexpected error occurred:";
+
 class SQSError extends Error {
   code: string;
   cause: AWSError;
@@ -26,9 +29,11 @@ class TimeoutError extends Error {
   cause: Error;
   time: Date;
 
-  constructor(message = "Operation timed out.") {
-    super(message);
-    this.message = message;
+  constructor(message?: string) {
+    const errorMessage = message === undefined ? DEFAULT_TIMEOUT_ERROR_MESSAGE : message;
+
+    super(errorMessage);
+    this.message = errorMessage;
     this.name = "TimeoutError";
     this.messageIds = [];
   }
@@ -39,9 +44,11 @@ class StandardError extends Error {
   cause: Error;
   time: Date;
 
-  constructor(message = "An unexpected error occurred:") {
-    super(message);
-    this.message = message;
+  constructor(message?: string) {
+    const errorMessage = message === undefined ? DEFAULT_STANDARD_ERROR_MESSAGE : message;
+
+    super(errorMessage);
+    this.message = errorMessage;
     this.name = "StandardError";
     this.messageIds = [];
   }