@bobfrankston/rmfmail 1.0.695 → 1.0.697

package/client/app.js

@@ -1232,21 +1232,23 @@ async function deleteSelectedMessages() {
         if (statusSync?.textContent?.includes("undo"))
             statusSync.textContent = "";
     }, 30000);
-    try {
-        const byAccount = new Map();
-        for (const msg of snapshot) {
-            const uids = byAccount.get(msg.accountId) || [];
-            uids.push(msg.uid);
-            byAccount.set(msg.accountId, uids);
-        }
-        for (const [accountId, uids] of byAccount) {
-            await deleteMessages(accountId, uids);
-        }
+    // Fire-and-forget per local-first: optimistic remove above already
+    // updated the UI; the daemon-side trash is sync DB + queued IMAP.
+    // An IPC 120s timeout doesn't mean the trash failed — surfacing it
+    // as a status-bar error would only mislead. Real errors are still
+    // reported by next sync's diagnostics.
+    const byAccount = new Map();
+    for (const msg of snapshot) {
+        const uids = byAccount.get(msg.accountId) || [];
+        uids.push(msg.uid);
+        byAccount.set(msg.accountId, uids);
     }
-    catch (e) {
-        console.error(`Delete failed: ${e.message}`);
-        if (statusSync)
-            statusSync.textContent = `Delete failed: ${e?.message || e}`;
+    for (const [accountId, uids] of byAccount) {
+        deleteMessages(accountId, uids).catch((e) => {
+            console.error(`Delete failed for ${accountId}: ${e?.message || e}`);
+            if (statusSync)
+                statusSync.textContent = `Delete sync issue (${accountId}): ${e?.message || e}`;
+        });
     }
 }
 async function undoDelete() {