@bobfrankston/rmfmail 1.2.217 → 1.2.219

package/packages/mailx-imap/index.ts

@@ -265,6 +265,14 @@ export class ImapManager extends EventEmitter {
     // (folder syncs holding all permits, timing out at 360s), leaving new mail
     // on 5-min polling for minutes at a time (Bob 2026-05-28).
     private watcherClients: Map<string, any> = new Map();
+    /** Accounts whose IDLE watcher is mid-establish. `watchers` only gains an
+     *  entry AFTER connect+login+SELECT+IDLE completes (seconds), so two
+     *  overlapping startWatching() sweeps both see "no watcher" and both open
+     *  a socket — a duplicate IDLE connection per account. Bob 2026-08-05:
+     *  boot's startWatching was still in flight when the 60s deadman fired,
+     *  giving two watchers each for bobma/gmail and pushing aol into
+     *  "[LIMIT] LOGIN Rate limit hit". This set closes that window. */
+    private watchStarting: Set<string> = new Set();
     private fetchClients: Map<string, any> = new Map();
     /** The Store is the architectural nexus — owner of MailxDB +
      *  FileMessageStore + the event bus. This package (mailx-imap) is a
@@ -3098,14 +3106,25 @@ export class ImapManager extends EventEmitter {
         // accounts with healthy IDLE are skipped). One log line per
         // detected gap so a chronically failing account is obvious in
         // the log.
+        //
+        // startWatching() sweeps EVERY account, so it must be called at most
+        // once per tick: the old per-account call meant N missing accounts →
+        // N concurrent sweeps, and since `watchers` is only populated after
+        // the connect completes, every sweep opened its own socket for every
+        // account (Bob 2026-08-05 boot: two IDLE watchers each for bobma and
+        // gmail, aol refused with "[LIMIT] LOGIN Rate limit hit").
         const deadmanInterval = setInterval(() => {
+            const missing: string[] = [];
             for (const [accountId] of this.configs) {
                 if (this.isApiAccount(accountId)) continue; // Gmail/Outlook = API, no IDLE
                 if (this.watchers.has(accountId)) continue;
-                console.log(`  [idle-deadman] ${accountId}: no IDLE watcher — attempting restart`);
-                this.startWatching().catch(e =>
-                    console.error(`  [idle-deadman] ${accountId} restart failed: ${e?.message || e}`));
+                if (this.watchStarting.has(accountId)) continue; // first attempt still connecting
+                missing.push(accountId);
             }
+            if (missing.length === 0) return;
+            console.log(`  [idle-deadman] no IDLE watcher for ${missing.join(", ")} — attempting restart`);
+            this.startWatching().catch(e =>
+                console.error(`  [idle-deadman] restart failed: ${e?.message || e}`));
         }, 60_000);
         this.syncIntervals.set("idle-deadman", deadmanInterval);
 
@@ -3204,6 +3223,16 @@ export class ImapManager extends EventEmitter {
                 this.watchers.delete(accountId);
                 this.watcherClients.delete(accountId);
             }
+            // Another sweep is already connecting this account. `watchers`
+            // stays empty for the whole connect+login+SELECT+IDLE round trip,
+            // so without this the second sweep opens a duplicate IDLE socket
+            // (and on connection-limited servers the duplicate is what gets
+            // refused). See the watchStarting field comment.
+            if (this.watchStarting.has(accountId)) {
+                console.log(`  [idle] ${accountId}: watcher already being established — skipping duplicate`);
+                continue;
+            }
+            this.watchStarting.add(accountId);
             try {
                 // IDLE keeps its own dedicated socket — once the connection
                 // is parked in IDLE, it's unusable for any other command, so
@@ -3301,6 +3330,8 @@ export class ImapManager extends EventEmitter {
                     ? parts.join(" ")
                     : `<no message> ${typeof e} ${e?.constructor?.name || ""}`.trim();
                 console.error(`  [idle] Failed to watch ${accountId}: ${desc}`);
+            } finally {
+                this.watchStarting.delete(accountId);
             }
         }
     }

package/packages/mailx-imap/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.140",
+  "version": "0.1.142",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bobfrankston/mailx-imap",
-      "version": "0.1.140",
+      "version": "0.1.142",
       "license": "ISC",
       "dependencies": {
         "@bobfrankston/iflow-direct": "^0.1.27",

package/packages/mailx-imap/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.140",
+  "version": "0.1.142",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",