@bobfrankston/rmfmail 1.2.243 → 1.2.245

package/packages/mailx-imap/index.ts

@@ -1117,7 +1117,22 @@ export class ImapManager extends EventEmitter {
                         // a rate-limit refusal stop ALL of them instead of each
                         // discovering it separately, once a minute, forever.
                         const em = e?.message || String(e);
-                        if (/[LIMIT]|rate limit/i.test(em)) this.noteRateLimited(accountId, em);
+                        // NOTE the escaping: [LIMIT] is a literal bracketed
+                        // response code. Written unescaped, [LIMIT] is a
+                        // CHARACTER CLASS matching any L/I/M/T — which is to
+                        // say almost every error message ever produced, so a
+                        // stray parse error would pause the whole account for
+                        // half an hour (shipped that way in v1.2.240, caught
+                        // 2026-08-10 when a connection-cap error tripped it).
+                        //
+                        // A connection cap belongs here too, deliberately:
+                        // Dovecot's "Maximum number of connections from
+                        // user+IP exceeded" is the same instruction as a rate
+                        // limit — stop opening sockets — and standing down for
+                        // a few minutes is what frees the slots.
+                        if (/\[LIMIT\]|rate limit|Maximum number of connections|too many connections/i.test(em)) {
+                            this.noteRateLimited(accountId, em);
+                        }
                         throw e;
                     }
                 };

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

@@ -1,12 +1,12 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.149",
+  "version": "0.1.150",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bobfrankston/mailx-imap",
-      "version": "0.1.149",
+      "version": "0.1.150",
       "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.149",
+  "version": "0.1.150",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",

package/packages/mailx-imap/test/rate-limit-backoff.test.mjs

@@ -59,3 +59,25 @@ for (const lane of ["sync", "quickCheck", "idle", "outbox", "fetch"]) {
 assert.strictEqual(attempted, 0, "no lane may reach the connect path during a cooldown");
 
 console.log("rate-limit backoff: 6 scenarios passed");
+
+// 7. THE CLASSIFIER ITSELF. Shipped in v1.2.240 as /[LIMIT]|rate limit/i —
+//    an unescaped character class matching any L, I, M or T, i.e. nearly
+//    every error string in existence. A stray parse error would have paused
+//    the account for up to 30 minutes. These assertions pin the intent:
+//    back off when the SERVER says stop, never on an ordinary failure.
+const shouldBackOff = (m) => /\[LIMIT\]|rate limit|Maximum number of connections|too many connections/i.test(m);
+for (const m of [
+    "Login failed: [LIMIT] LOGIN Rate limit hit.",
+    "Login failed: [UNAVAILABLE] Maximum number of connections from user+IP exceeded (mail_max_userip_connections=20)",
+    "429 rate limit exceeded",
+    "too many connections",
+]) assert.ok(shouldBackOff(m), `should back off: ${m}`);
+for (const m of [
+    "Login failed: AUTHENTICATE failed.",
+    "inactivity timeout",
+    "SELECT INBOX failed: [NONEXISTENT] Unknown Mailbox",
+    "socket hang up",
+    "ETIMEDOUT",
+    "simpleParser threw on MIME boundary",
+]) assert.ok(!shouldBackOff(m), `must NOT pause the account for: ${m}`);
+console.log("rate-limit classifier: 10 messages classified correctly");