@bobfrankston/mailx-imap 0.1.80 → 0.1.82

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

@@ -1864,8 +1864,19 @@ export class ImapManager extends EventEmitter {
             // a "never-touched" non-special folder is deferred until the
             // user opens that folder (on-demand syncFolder call) or until
             // ~3 minutes after startup when the event loop is quiet.
-            const STARTUP_LAZY_DELAY_MS = 3 * 60 * 1000;
-            const startupQuietPoint = (this._startupAt || 0) + STARTUP_LAZY_DELAY_MS;
+            // TRUE lazy folder sync (C119, completed 2026-06-01). A folder is
+            // "lazy" if it's neither a priority special-use folder NOR one
+            // mailx has ever pulled a message from (highestUid===0). Lazy
+            // folders are NOT auto-synced at all — they sync on-demand the
+            // moment the user opens them (the client fires `syncFolderNow` on
+            // folder-open) and then join the periodic set automatically once
+            // they have a message (highestUid>0). Previously they were only
+            // deferred for a 3-minute startup window, after which a full
+            // 96-folder sweep ran every cycle — re-saturating the IMAP
+            // connection and pegging the event loop (the "mailx DOSes itself,
+            // TB doesn't" storm, Bob 2026-06-01). Permanent deferral closes it:
+            // a fresh rebuild now syncs only INBOX + special-use, and the rest
+            // fill in as the user actually visits them.
             const isLazyEligible = (f) => {
                 if (f.specialUse && priorityOrder.includes(f.specialUse))
                     return false;
@@ -1873,15 +1884,15 @@ export class ImapManager extends EventEmitter {
                     return false;
                 return true;
             };
-            const remaining = folders.filter(f => f.specialUse !== "inbox" && !(isLazyEligible(f) && Date.now() < startupQuietPoint));
+            const remaining = folders.filter(f => f.specialUse !== "inbox" && !isLazyEligible(f));
             remaining.sort((a, b) => {
                 const pa = priorityOrder.indexOf(a.specialUse || "") >= 0 ? priorityOrder.indexOf(a.specialUse || "") : 5;
                 const pb = priorityOrder.indexOf(b.specialUse || "") >= 0 ? priorityOrder.indexOf(b.specialUse || "") : 5;
                 return pa - pb;
             });
-            const deferredCount = folders.filter(f => f.specialUse !== "inbox" && isLazyEligible(f) && Date.now() < startupQuietPoint).length;
+            const deferredCount = folders.filter(f => f.specialUse !== "inbox" && isLazyEligible(f)).length;
             if (deferredCount > 0) {
-                console.log(`  [sync] ${accountId}: deferring first-sync of ${deferredCount} non-priority folder(s) until ${new Date(startupQuietPoint).toLocaleTimeString()} or user opens them`);
+                console.log(`  [sync] ${accountId}: lazy — ${deferredCount} never-touched folder(s) will sync on-demand when opened (not auto-swept)`);
             }
             const CONCURRENCY = 2;
             // First-sync of a fresh account on a cold Dovecot is dominated by
@@ -2896,14 +2907,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 +2934,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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.80",
+  "version": "0.1.82",
   "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.44",
     "@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.44",
       "@bobfrankston/iflow-direct": "^0.1.51",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",