@bobfrankston/mailx-imap 0.1.79 → 0.1.81

package/index.d.ts

@@ -395,7 +395,6 @@ export declare class ImapManager extends EventEmitter {
      *  no teal mark). With TTL, transient misses self-heal in 5 minutes;
      *  genuine zombies back off to once-per-12-hour retries — bounded load.
      *  Key shape `${accountId}:${folderId}:${uid}`. */
-    private prefetchFailures;
     private markPrefetchEmpty;
     private isPrefetchEmpty;
     /** Clear a UID's prefetch-failure record after a successful fetch, so a

package/index.js

@@ -2896,14 +2896,18 @@ export class ImapManager extends EventEmitter {
      *  no teal mark). With TTL, transient misses self-heal in 5 minutes;
      *  genuine zombies back off to once-per-12-hour retries — bounded load.
      *  Key shape `${accountId}:${folderId}:${uid}`. */
-    prefetchFailures = new Map();
+    // Backed by the persistent `prefetch_failures` table (was an in-memory
+    // Map that every daemon restart wiped — so ghost UIDs re-clogged prefetch
+    // on every launch; Bob 2026-06-01). markPrefetchEmpty/isPrefetchEmpty now
+    // read/write the DB so the 5min→12h backoff survives restarts.
     markPrefetchEmpty(accountId, folderId, uid) {
-        const key = `${accountId}:${folderId}:${uid}`;
-        const prev = this.prefetchFailures.get(key);
-        this.prefetchFailures.set(key, { count: (prev?.count ?? 0) + 1, lastTried: Date.now() });
+        try {
+            this.db.recordPrefetchFailure(accountId, folderId, uid);
+        }
+        catch { /* non-fatal */ }
     }
     isPrefetchEmpty(accountId, folderId, uid) {
-        const f = this.prefetchFailures.get(`${accountId}:${folderId}:${uid}`);
+        const f = this.db.getPrefetchFailure(accountId, folderId, uid);
         if (!f)
             return false;
         const backoffMs = f.count <= 1 ? 5 * 60_000
@@ -2919,7 +2923,10 @@ export class ImapManager extends EventEmitter {
      *  count and would back off longer than warranted if it ever re-enters
      *  the candidate set. */
     clearPrefetchEmpty(accountId, folderId, uid) {
-        this.prefetchFailures.delete(`${accountId}:${folderId}:${uid}`);
+        try {
+            this.db.clearPrefetchFailure(accountId, folderId, uid);
+        }
+        catch { /* non-fatal */ }
     }
     /** Background body-cache backfill. Public so the Reconciler can schedule
      *  the periodic tick under its priority/back-pressure rules; existing
@@ -2968,7 +2975,17 @@ export class ImapManager extends EventEmitter {
             // BATCH_SIZE rows that all get filtered to zero, and the loop
             // terminates without ever trying live messages further down the
             // size-asc list.
-            const raw = this.db.getMessagesWithoutBody(accountId, BATCH_SIZE * 4);
+            // Exclude error-cooled folders AT THE QUERY so a single bloated /
+            // perpetually-failing folder (a stuck server-side Outbox, a 300s
+            // archive) can't fill the entire size-asc batch and starve every
+            // healthy folder. Recomputed each iteration: a folder that trips
+            // its cooldown mid-loop drops out on the next pull (Bob 2026-06-01
+            // "so much unfetched — nothing draining"). Empty when no folder is
+            // cooling, so the common case is unchanged.
+            const coolingFolderIds = this.db.getFolders(accountId)
+                .filter(f => this.shouldSkipFolder(accountId, f.path))
+                .map(f => f.id);
+            const raw = this.db.getMessagesWithoutBody(accountId, BATCH_SIZE * 4, coolingFolderIds);
             const missing = raw.filter(m => !this.isPrefetchEmpty(accountId, m.folderId, m.uid))
                 .slice(0, BATCH_SIZE);
             if (missing.length === 0)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.79",
+  "version": "0.1.81",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -11,7 +11,7 @@
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.18",
     "@bobfrankston/mailx-settings": "^0.1.26",
-    "@bobfrankston/mailx-store": "^0.1.42",
+    "@bobfrankston/mailx-store": "^0.1.43",
     "@bobfrankston/iflow-direct": "^0.1.51",
     "@bobfrankston/tcp-transport": "^0.1.6",
     "@bobfrankston/smtp-direct": "^0.1.8",
@@ -39,7 +39,7 @@
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.18",
       "@bobfrankston/mailx-settings": "^0.1.26",
-      "@bobfrankston/mailx-store": "^0.1.42",
+      "@bobfrankston/mailx-store": "^0.1.43",
       "@bobfrankston/iflow-direct": "^0.1.51",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",