@bobfrankston/mailx-imap 0.1.149 → 0.1.151

package/index.js

@@ -1120,8 +1120,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))
+                        // 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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.149",
+  "version": "0.1.151",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -9,9 +9,9 @@
   },
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/mailx-types": "^0.1.42",
+    "@bobfrankston/mailx-types": "^0.1.44",
     "@bobfrankston/mailx-settings": "^0.1.51",
-    "@bobfrankston/mailx-store": "^0.1.83",
+    "@bobfrankston/mailx-store": "^0.1.84",
     "@bobfrankston/iflow-direct": "^0.1.65",
     "@bobfrankston/tcp-transport": "^0.1.8",
     "@bobfrankston/smtp-direct": "^0.1.9",
@@ -37,9 +37,9 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/mailx-types": "^0.1.42",
+      "@bobfrankston/mailx-types": "^0.1.44",
       "@bobfrankston/mailx-settings": "^0.1.51",
-      "@bobfrankston/mailx-store": "^0.1.83",
+      "@bobfrankston/mailx-store": "^0.1.84",
       "@bobfrankston/iflow-direct": "^0.1.65",
       "@bobfrankston/tcp-transport": "^0.1.8",
       "@bobfrankston/smtp-direct": "^0.1.9",

package/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");