@bobfrankston/rmfmail 1.2.2 → 1.2.3

package/packages/mailx-service/index.ts

@@ -291,19 +291,45 @@ export class MailxService implements MailxApi {
         }
     }
 
+    /** In-flight read coalescing. A slow daemon makes the UI re-fire the same
+     *  read (scroll loadMore retries, poll loops, re-render storms): the log
+     *  showed the IDENTICAL getUnifiedInbox queued dozens of times and all
+     *  draining at once after a 78s stall. Keyed by method+args, a duplicate
+     *  request that arrives while one is already running shares that single
+     *  promise instead of enqueuing another worker job. Same args ⇒ same
+     *  result, so this is always safe; different args (other pages) key
+     *  separately. This caps the worker queue at one job per distinct read. */
+    private inflightReads = new Map<string, Promise<any>>();
+
     /** Route a read through the worker bus if it's up, else run it in-process.
      *  A worker-side error (or a dead worker) transparently falls back so a
-     *  read never fails just because the isolation layer hiccuped. */
+     *  read never fails just because the isolation layer hiccuped. Identical
+     *  concurrent reads are coalesced (see inflightReads). */
     private async read<R>(method: string, args: any, local: () => R): Promise<R> {
-        const w = this.dbWorker;
-        if (w) {
-            try {
-                return await w.bus.request<any, R>(method, args);
-            } catch (e: any) {
-                console.error(`  [read-worker] ${method} failed (${e?.message || e}); in-process fallback`);
+        let key: string;
+        try { key = method + ":" + JSON.stringify(args); }
+        catch { key = ""; }   // unserializable args → don't coalesce
+        if (key) {
+            const existing = this.inflightReads.get(key);
+            if (existing) return existing as Promise<R>;
+        }
+        const run = (async (): Promise<R> => {
+            const w = this.dbWorker;
+            if (w) {
+                try {
+                    return await w.bus.request<any, R>(method, args);
+                } catch (e: any) {
+                    console.error(`  [read-worker] ${method} failed (${e?.message || e}); in-process fallback`);
+                }
             }
-        }
-        return local();
+            return local();
+        })();
+        if (key) {
+            this.inflightReads.set(key, run);
+            try { return await run; }
+            finally { this.inflightReads.delete(key); }
+        }
+        return run;
     }
     /** Transitional getter — direct DB access from MailxService for the
      *  ~50 callsites that haven't yet migrated to Store methods. Future:

package/packages/mailx-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.47",
+  "version": "0.1.48",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",