@bobfrankston/rmfmail 1.1.188 → 1.1.190

package/packages/mailx-imap/index.ts

@@ -2902,6 +2902,15 @@ export class ImapManager extends EventEmitter {
                         :                12 * 3600_000;
         return Date.now() - f.lastTried < backoffMs;
     }
+    /** Clear a UID's prefetch-failure record after a successful fetch, so a
+     *  message that healed (e.g. a transient iflow protocol desync that
+     *  resolved on retry) doesn't carry a stale backoff count into the
+     *  future. Without this, a once-failed-then-succeeded UID keeps its
+     *  count and would back off longer than warranted if it ever re-enters
+     *  the candidate set. */
+    private clearPrefetchEmpty(accountId: string, folderId: number, uid: number): void {
+        this.prefetchFailures.delete(`${accountId}:${folderId}:${uid}`);
+    }
 
     /** Background body-cache backfill. Public so the Reconciler can schedule
      *  the periodic tick under its priority/back-pressure rules; existing
@@ -3103,10 +3112,22 @@ export class ImapManager extends EventEmitter {
                                             const parsed = await extractPreview(source);
                                             this.db.updateBodyMeta(accountId, uid, bodyPath, parsed.hasAttachments, parsed.preview);
                                             this.emit("bodyCached", accountId, uid);
+                                            this.clearPrefetchEmpty(accountId, folderId, uid); // healed — drop stale backoff
                                             counters.totalFetched++;
                                             madeProgress = true;
                                         } catch (e: any) {
                                             console.error(`  [prefetch] ${accountId}/${uid}: store write failed: ${e.message}`);
+                                            // Back off this UID. A corrupt body (the
+                                            // putMessage guard rejecting an IMAP command
+                                            // leaked into the FETCH response — iflow
+                                            // protocol desync) was NOT marked, so the
+                                            // same poisoned UID got re-fetched every
+                                            // prefetch cycle forever, spamming the log
+                                            // and wasting fetch turns (Bob 2026-05-28
+                                            // "why is prefetching still broken" — UID
+                                            // 59686 failing on a loop). markPrefetchEmpty
+                                            // applies the 5m→30m→2h→12h backoff.
+                                            this.markPrefetchEmpty(accountId, folderId, uid);
                                         }
                                     })());
                                 });

package/packages/mailx-imap/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.71",
+  "version": "0.1.73",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bobfrankston/mailx-imap",
-      "version": "0.1.71",
+      "version": "0.1.73",
       "license": "ISC",
       "dependencies": {
         "@bobfrankston/iflow-direct": "^0.1.27",

package/packages/mailx-imap/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.71",
+  "version": "0.1.73",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",