@mushi-mushi/core 1.18.0 → 1.19.0

package/dist/index.cjs

@@ -1000,6 +1000,30 @@ function createOfflineQueue(config = {}) {
       lsWrite(rows);
     }
   }
+  async function removeRow(backend, rowId) {
+    try {
+      if (backend === "indexeddb") await idbDelete(rowId);
+      else lsDelete(rowId);
+    } catch (err) {
+      queueLog.debug("Offline queue: row removal failed (will age out)", {
+        id: rowId,
+        err: String(err)
+      });
+    }
+  }
+  async function persistRow(backend, row) {
+    try {
+      if (backend === "indexeddb") await idbPutRow(row);
+      else lsUpdateRow(row);
+      return true;
+    } catch (err) {
+      queueLog.debug("Offline queue: row persist failed", {
+        id: row.id,
+        err: String(err)
+      });
+      return false;
+    }
+  }
   async function enqueue(report) {
     if (!enabled) return;
     const currentSize = await size();
@@ -1045,12 +1069,7 @@ function createOfflineQueue(config = {}) {
     for (const row of rows) {
       if (isExpired(row, now)) {
         const rowId = row.id;
-        try {
-          if (backend === "indexeddb") await idbDelete(rowId);
-          else lsDelete(rowId);
-        } catch {
-          lsDelete(rowId);
-        }
+        await removeRow(backend, rowId);
         queueLog.debug("Offline queue: evicting stale report", { id: rowId });
       } else {
         fresh.push(row);
@@ -1065,23 +1084,13 @@ function createOfflineQueue(config = {}) {
       const rowId = row.id;
       const report = await unwrapForSend(row);
       if (!report) {
-        try {
-          if (backend === "indexeddb") await idbDelete(rowId);
-          else lsDelete(rowId);
-        } catch {
-          lsDelete(rowId);
-        }
+        await removeRow(backend, rowId);
         failed++;
         continue;
       }
       const result = await client.submitReport(report);
       if (result.ok) {
-        try {
-          if (backend === "indexeddb") await idbDelete(rowId);
-          else lsDelete(rowId);
-        } catch {
-          lsDelete(rowId);
-        }
+        await removeRow(backend, rowId);
         sent++;
       } else {
         const permanent = result.error?.code === "HTTP_400" || result.error?.code === "HTTP_413" || result.error?.code === "HTTP_422" || result.error?.code === "INGEST_ERROR" || result.error?.code === "VALIDATION_ERROR" || // A payload that exceeds the size guard will never shrink on its own;
@@ -1093,21 +1102,11 @@ function createOfflineQueue(config = {}) {
         );
         const transient = !permanent && (result.error?.code === "NETWORK_ERROR" || result.error?.code === "HTTP_403" || result.error?.code === "HTTP_429" || result.error?.code === "HTTP_502" || result.error?.code === "HTTP_503" || result.error?.code === "HTTP_504" || typeof result.error?.code === "string" && result.error.code.startsWith("HTTP_5"));
         if (permanent) {
-          try {
-            if (backend === "indexeddb") await idbDelete(rowId);
-            else lsDelete(rowId);
-          } catch {
-            lsDelete(rowId);
-          }
+          await removeRow(backend, rowId);
         } else if (transient) {
           const nextAttempts = rowAttempts(row) + 1;
           if (nextAttempts >= MAX_DELIVERY_ATTEMPTS) {
-            try {
-              if (backend === "indexeddb") await idbDelete(rowId);
-              else lsDelete(rowId);
-            } catch {
-              lsDelete(rowId);
-            }
+            await removeRow(backend, rowId);
             queueLog.warn("Offline queue: giving up on report after repeated failures", {
               id: rowId,
               attempts: nextAttempts,
@@ -1115,17 +1114,20 @@ function createOfflineQueue(config = {}) {
             });
           } else {
             row.attempts = nextAttempts;
-            try {
-              if (backend === "indexeddb") await idbPutRow(row);
-              else lsUpdateRow(row);
-            } catch {
-              lsUpdateRow(row);
+            const persisted = await persistRow(backend, row);
+            if (persisted) {
+              queueLog.debug("Offline queue: transient failure, will retry", {
+                id: rowId,
+                attempts: nextAttempts,
+                code: result.error?.code
+              });
+            } else {
+              await removeRow(backend, rowId);
+              queueLog.warn("Offline queue: dropping report (attempt counter unpersistable)", {
+                id: rowId,
+                code: result.error?.code
+              });
             }
-            queueLog.debug("Offline queue: transient failure, will retry", {
-              id: rowId,
-              attempts: nextAttempts,
-              code: result.error?.code
-            });
           }
         }
         failed++;