@bobfrankston/rmfmail 1.2.77 → 1.2.78

package/client/android-bootstrap.bundle.js

@@ -9796,6 +9796,11 @@ var AndroidSyncManager = class {
   // advanced to fetching new mail, so the list stayed stale despite "Synced"
   // (Bob 2026-06-27). Coalesce concurrent calls to one in-flight run.
   syncAllInflight = null;
+  // Separate guard for the Phase-2/3 background pass (other folders + body
+  // prefetch). It runs DETACHED from the inbox guard so a long folder sweep
+  // can't make the 60s poll skip the inbox refresh (new mail must keep
+  // flowing). One background pass at a time — overlapping triggers no-op.
+  bgSyncInflight = null;
   on(_event, _handler) {
   }
   emit(event, ...args) {
@@ -9850,41 +9855,53 @@ var AndroidSyncManager = class {
   }
   async syncAll() {
     if (this.syncAllInflight) {
-      console.log("[sync] syncAll already in flight \u2014 coalescing this trigger");
+      console.log("[sync] inbox sync already in flight \u2014 coalescing this trigger");
       return this.syncAllInflight;
     }
     const t0 = Date.now();
-    console.log("[sync] syncAll START");
-    this.syncAllInflight = this._syncAllImpl().catch((e) => {
-      console.error(`[sync] syncAll error: ${e?.message || e}`);
+    console.log("[sync] syncAll START (inbox phase)");
+    this.syncAllInflight = this._syncInboxes().catch((e) => {
+      console.error(`[sync] inbox sync error: ${e?.message || e}`);
     }).finally(() => {
-      console.log(`[sync] syncAll DONE in ${Date.now() - t0}ms`);
+      console.log(`[sync] inbox phase DONE in ${Date.now() - t0}ms`);
       this.syncAllInflight = null;
     });
+    this._kickBackgroundSync();
     return this.syncAllInflight;
   }
-  async _syncAllImpl() {
-    const accounts = this.db.getAccounts();
-    vlog(`syncAll: ${accounts.length} accounts in DB: ${accounts.map((a) => a.id).join(",")}`);
-    for (const account of accounts) {
-      if (!this.providers.has(account.id)) continue;
-      try {
-        await withTimeout(`${account.id} inbox`, 6e4, async () => {
-          const folders = await this.syncFolders(account.id);
-          const inbox = folders.find((f) => f.specialUse === "inbox");
-          if (inbox) {
-            console.log(`[sync] ${account.id}: fetching INBOX (folderId=${inbox.id})`);
-            await this.syncFolder(account.id, inbox.id);
-            console.log(`[sync] ${account.id}: INBOX fetch done`);
-            emitEvent({ type: "syncComplete", accountId: account.id });
-          }
-        });
-      } catch (e) {
-        console.error(`[sync] ${account.id} inbox: ${e.message}`);
-      }
-    }
+  /** Phase 1: sync every account's INBOX IN PARALLEL so a slow account (e.g.
+   *  Gmail's metadata fetch, or an auth-failing IMAP account) can't delay new
+   *  mail for the others. Each account is time-boxed; network fetches overlap
+   *  even though JS is single-threaded (the awaits yield). */
+  async _syncInboxes() {
+    const accounts = this.db.getAccounts().filter((a) => this.providers.has(a.id));
+    vlog(`syncInboxes: ${accounts.length} accounts: ${accounts.map((a) => a.id).join(",")}`);
+    await Promise.all(accounts.map(
+      (account) => withTimeout(`${account.id} inbox`, 6e4, async () => {
+        const folders = await this.syncFolders(account.id);
+        const inbox = folders.find((f) => f.specialUse === "inbox");
+        if (inbox) {
+          console.log(`[sync] ${account.id}: fetching INBOX (folderId=${inbox.id})`);
+          await this.syncFolder(account.id, inbox.id);
+          console.log(`[sync] ${account.id}: INBOX fetch done`);
+          emitEvent({ type: "syncComplete", accountId: account.id });
+        }
+      }).catch((e) => console.error(`[sync] ${account.id} inbox: ${e?.message || e}`))
+    ));
+  }
+  /** Phase 2 + 3: remaining folders, then body prefetch. Runs in the
+   *  background under its own single-flight guard. */
+  _kickBackgroundSync() {
+    if (this.bgSyncInflight) return;
+    const t0 = Date.now();
+    this.bgSyncInflight = this._syncRemainingAndPrefetch().catch((e) => console.error(`[sync] background sweep error: ${e?.message || e}`)).finally(() => {
+      console.log(`[sync] background sweep DONE in ${Date.now() - t0}ms`);
+      this.bgSyncInflight = null;
+    });
+  }
+  async _syncRemainingAndPrefetch() {
+    const accounts = this.db.getAccounts().filter((a) => this.providers.has(a.id));
     for (const account of accounts) {
-      if (!this.providers.has(account.id)) continue;
       try {
         const folders = this.db.getFolders(account.id);
         const remaining = folders.filter((f) => f.specialUse !== "inbox");
@@ -9899,12 +9916,10 @@ var AndroidSyncManager = class {
         emitEvent({ type: "syncComplete", accountId: account.id });
       } catch (e) {
         console.error(`[sync] ${account.id}: ${e.message}`);
-        vlog(`syncAll: ${account.id} ERROR: ${e.message}`);
         emitEvent({ type: "syncError", accountId: account.id, error: e.message });
       }
     }
     for (const account of accounts) {
-      if (!this.providers.has(account.id)) continue;
       this.prefetchBodies(account.id).catch((e) => console.error(`[prefetch] ${account.id}: ${e.message}`));
     }
   }