saving message: Stop throwing error if retries fail

Author: jahooma

backend/src/llm-apis/message-cost-tracker.ts

@@ -353,15 +353,9 @@ export async function insertMessageRecordWithRetries(
           { messageId: params.messageId, error, attempt },
           `Failed to save message after ${maxRetries} attempts`,
         )
-        // TODO: Consider removing this exception for test mode.
-        if (process.env.NEXT_PUBLIC_CB_ENVIRONMENT === 'test') {
-          logger.error(
-            { messageId: params.messageId, error, attempt },
-            `(test mode) Couldn't fetch user cycle usage. Returning 0 consumed and 0 from purchased credits`,
-          )
-          return null
-        }
-        throw error
+        return null
+        // TODO: Consider rethrowing the error, if we are losing too much money.
+        // throw error
       } else {
         logger.warn(
           { messageId: params.messageId, error: error },
@@ -497,15 +491,10 @@ async function updateUserCycleUsageWithRetries(
           `Failed to update user cycle usage after ${maxRetries} attempts`,
         )
 
-        // TODO: Consider removing this exception for test mode.
-        if (process.env.NEXT_PUBLIC_CB_ENVIRONMENT === 'test') {
-          logger.error(
-            { userId, orgId, creditsUsed, error, attempt },
-            `(test mode) Couldn't fetch user cycle usage. Returning 0 consumed and 0 from purchased credits`,
-          )
-          return { consumed: 0, fromPurchased: 0 }
-        }
-        throw error
+        return { consumed: 0, fromPurchased: 0 }
+
+        // TODO: Consider rethrowing the error, if we are losing too much money.
+        // throw error
       } else {
         logger.warn(
           { userId, orgId, creditsUsed, error: error },