@514labs/moose-lib 0.6.252-ci-1-g901efb04 → 0.6.252-ci-2-g41538689

package/dist/moose-runner.mjs

@@ -1967,6 +1967,7 @@ var handleMessage = async (logger2, streamingFunctionWithConfigList, message, pr
 var handleDLQForFailedMessages = async (logger2, producer, messages, error) => {
   let messagesHandledByDLQ = 0;
   let messagesWithoutDLQ = 0;
+  let dlqErrors = 0;
   for (const msg of messages) {
     if (msg.dlq && msg.originalValue) {
       const deadLetterRecord = {
@@ -1996,6 +1997,7 @@ var handleDLQForFailedMessages = async (logger2, producer, messages, error) => {
         messagesHandledByDLQ++;
       } catch (dlqError) {
         logger2.error(`Failed to send to DLQ: ${dlqError}`);
+        dlqErrors++;
       }
     } else if (!msg.dlq) {
       messagesWithoutDLQ++;
@@ -2005,11 +2007,25 @@ var handleDLQForFailedMessages = async (logger2, producer, messages, error) => {
       logger2.warn(`Cannot send to DLQ: original message value not available`);
     }
   }
-  if (messagesHandledByDLQ > 0 && messagesWithoutDLQ > 0) {
+  const allMessagesHandled = messagesHandledByDLQ === messages.length && messagesWithoutDLQ === 0 && dlqErrors === 0;
+  if (allMessagesHandled) {
+    logger2.log(
+      `All ${messagesHandledByDLQ} failed message(s) sent to DLQ, suppressing original error`
+    );
+  } else if (messagesHandledByDLQ > 0) {
     logger2.warn(
       `Partial DLQ success: ${messagesHandledByDLQ}/${messages.length} message(s) sent to DLQ`
     );
+    if (messagesWithoutDLQ > 0) {
+      logger2.error(
+        `Cannot handle batch failure: ${messagesWithoutDLQ} message(s) have no DLQ configured or missing original value`
+      );
+    }
+    if (dlqErrors > 0) {
+      logger2.error(`${dlqErrors} message(s) failed to send to DLQ`);
+    }
   }
+  return allMessagesHandled;
 };
 var sendMessages = async (logger2, metrics, targetTopic, producer, messages) => {
   if (messages.length === 0) return;
@@ -2028,8 +2044,15 @@ var sendMessages = async (logger2, metrics, targetTopic, producer, messages) =>
     if (e instanceof Error) {
       logError(logger2, e);
     }
-    await handleDLQForFailedMessages(logger2, producer, messages, e);
-    throw e;
+    const allHandledByDLQ = await handleDLQForFailedMessages(
+      logger2,
+      producer,
+      messages,
+      e
+    );
+    if (!allHandledByDLQ) {
+      throw e;
+    }
   }
 };
 var sendMessageMetrics = (logger2, metrics) => {