@bobfrankston/mailx-imap 0.1.88 → 0.1.90

package/index.js

@@ -2829,6 +2829,15 @@ export class ImapManager extends EventEmitter {
         }
         if (!this.configs.has(accountId))
             return null;
+        // Back off a body that recently hung / failed. One un-fetchable message
+        // (the server stalls on its BODY[] — Bob 2026-06-12, UID 4966060) was
+        // being re-fetched on every view + prefetch tick, each attempt hanging
+        // ~90s and tying up the ops queue, so user actions (delete, open) were
+        // delayed until the client timed out. The 5min→12h backoff (shared with
+        // prefetch via isPrefetchEmpty) stops the tight re-fetch loop; the body
+        // simply shows as unavailable until the backoff lapses and one retry runs.
+        if (this.isPrefetchEmpty(accountId, folderId, uid))
+            return null;
         const folder = this.db.getFolders(accountId).find(f => f.id === folderId);
         if (!folder)
             return null;
@@ -2859,6 +2868,12 @@ export class ImapManager extends EventEmitter {
             if (e?.isNotFound)
                 throw e;
             console.error(`  Body fetch error (${accountId}/${uid}): ${e?.message || e}`);
+            // Record so the next view / prefetch backs off instead of
+            // re-attempting a hanging fetch and re-congesting the queue.
+            try {
+                this.markPrefetchEmpty(accountId, folderId, uid);
+            }
+            catch { /* */ }
             return null;
         }
     }
@@ -3432,7 +3447,14 @@ export class ImapManager extends EventEmitter {
                     }
                     try {
                         if (action.action === "flags" && api.setFlags) {
-                            await api.setFlags(folder.path, action.uid, action.flags || []);
+                            // Pass the stored Gmail message id (provider_id) for
+                            // the same reason as trashMessage below: without it
+                            // setFlags falls back to a capped list-and-hash
+                            // search that misses older messages, so starring one
+                            // failed ("UID not found in INBOX") and the local
+                            // star reverted on the next sync (Bob 2026-06-12).
+                            const env = this.db.getMessageByUid(accountId, action.uid);
+                            await api.setFlags(folder.path, action.uid, action.flags || [], env?.providerId);
                             console.log(`  [api] ${accountId}: flags synced UID ${action.uid}`);
                         }
                         else if ((action.action === "delete" || action.action === "trash") && api.trashMessage) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.88",
+  "version": "0.1.90",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -11,11 +11,11 @@
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.19",
     "@bobfrankston/mailx-settings": "^0.1.26",
-    "@bobfrankston/mailx-store": "^0.1.46",
+    "@bobfrankston/mailx-store": "^0.1.47",
     "@bobfrankston/iflow-direct": "^0.1.53",
     "@bobfrankston/tcp-transport": "^0.1.7",
     "@bobfrankston/smtp-direct": "^0.1.9",
-    "@bobfrankston/mailx-sync": "^0.1.20",
+    "@bobfrankston/mailx-sync": "^0.1.22",
     "@bobfrankston/oauthsupport": "^1.0.32"
   },
   "repository": {
@@ -39,11 +39,11 @@
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.19",
       "@bobfrankston/mailx-settings": "^0.1.26",
-      "@bobfrankston/mailx-store": "^0.1.46",
+      "@bobfrankston/mailx-store": "^0.1.47",
       "@bobfrankston/iflow-direct": "^0.1.53",
       "@bobfrankston/tcp-transport": "^0.1.7",
       "@bobfrankston/smtp-direct": "^0.1.9",
-      "@bobfrankston/mailx-sync": "^0.1.20",
+      "@bobfrankston/mailx-sync": "^0.1.22",
       "@bobfrankston/oauthsupport": "^1.0.32"
     }
   }