@bobfrankston/mailx-imap 0.1.81 → 0.1.83

package/index.js

@@ -1272,6 +1272,15 @@ export class ImapManager extends EventEmitter {
                 const __qrT0 = Date.now();
                 const qr = await client.resyncFolder(folder.path, prevUidValidity, prevModSeq);
                 console.log(`  [qresync] ${accountId}/${folder.path}: vanished=${qr.vanishedUids.length} changed=${qr.changedMessages.length} newModSeq=${qr.newHighestModSeq} in ${Date.now() - __qrT0}ms`);
+                // No real folder vanishes this many UIDs in one QRESYNC cycle.
+                // iflow-direct can mis-expand a VANISHED *range* (e.g. the
+                // server reports "VANISHED 1:4828883" when our modseq watermark
+                // is stale) into millions of individual UIDs. Applying that
+                // literally = 4.8M synchronous DB calls that wedge the whole
+                // event loop (frozen window) AND delete every local row caught
+                // in the range (INBOX wiped → the viewer's open message
+                // vanishes / shows a stale unrelated eml). Bob 2026-06-04.
+                const VANISHED_ABSURD_CAP = 50000;
                 if (qr.uidValidityChanged) {
                     // UIDVALIDITY rolled — our local UIDs are stale. Fall through
                     // to the full set-diff path; that'll discover the new state
@@ -1279,6 +1288,15 @@ export class ImapManager extends EventEmitter {
                     // from wiping anything we shouldn't.
                     console.log(`  [qresync] ${accountId}/${folder.path}: UIDVALIDITY changed (was ${prevUidValidity}, now ${qr.exists}); falling back to full sync`);
                 }
+                else if (qr.vanishedUids.length > VANISHED_ABSURD_CAP) {
+                    // Refuse the absurd VANISHED set; do NOT iterate/apply it and
+                    // do NOT return — fall through to the bounded set-diff
+                    // reconcile (same as the UIDVALIDITY-changed path). The
+                    // modseq watermark is left intact, so a healthy later QRESYNC
+                    // can still succeed.
+                    const localCount = this.db.getMessageCount(accountId, folderId);
+                    console.error(`  [qresync] ${accountId}/${folder.path}: REFUSING VANISHED of ${qr.vanishedUids.length} UIDs (folder has ${localCount} locally) — almost certainly a mis-parsed range; falling back to set-diff reconcile instead of mass-deleting/wedging.`);
+                }
                 else {
                     // Apply VANISHED — server says these UIDs are gone. No
                     // tombstone, no diff, just delete the local rows.
@@ -1864,8 +1882,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 +1902,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

package/package.json

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