@bobfrankston/rmfmail 1.2.65 → 1.2.67

package/packages/mailx-imap/index.ts

@@ -5055,54 +5055,68 @@ export class ImapManager extends EventEmitter {
             const result = await this.withConnection(accountId, async (client) => {
                 const flags = await client.getFlags(outboxFolder.path, uid);
 
-                // Sweep stale claims. $Sending-<host>-<sec> with old timestamp,
-                // or legacy $Sending-<host> without timestamp (treated as
-                // stale; if the real owner is alive it'll re-claim next tick).
-                const claimFlags = flags.filter((f: string) => f.startsWith("$Sending"));
-                for (const cf of claimFlags) {
-                    const m = cf.match(/^\$Sending-(.+?)(?:-(\d+))?$/);
-                    if (!m) continue;
-                    const tsSec = m[2] ? parseInt(m[2]) : 0;
-                    const ageSec = nowSec - tsSec;
-                    if (ageSec * 1000 > STALE_CLAIM_MS) {
-                        try {
-                            await client.removeFlags(outboxFolder.path, uid, [cf]);
-                            console.log(`  [outbox] Swept stale claim ${cf} on UID ${uid} (age ${Math.round(ageSec / 60)}m)`);
-                        } catch { /* ignore */ }
+                // The $Sending/$Failed/$PermanentFailure interlock uses CUSTOM
+                // IMAP keywords, which only work when the server advertises \*
+                // in PERMANENTFLAGS. AOL (and some others) don't, and reject a
+                // STORE of any custom keyword with "[CLIENTBUG] UID STORE
+                // Command arguments invalid" — which wedged the whole outbox in
+                // a 5-minute retry loop, so nothing ever sent (Bob 2026-06-26,
+                // from the log). Gate the keyword dance on real support; without
+                // it, send best-effort with no IMAP-side claim (correct for a
+                // single device — there's no keyword to coordinate on anyway).
+                const supportsKeywords = !!(client as any).mailboxInfo?.permanentFlags?.includes("\\*");
+
+                if (supportsKeywords) {
+                    // Sweep stale claims. $Sending-<host>-<sec> with old timestamp,
+                    // or legacy $Sending-<host> without timestamp (treated as
+                    // stale; if the real owner is alive it'll re-claim next tick).
+                    const claimFlags = flags.filter((f: string) => f.startsWith("$Sending"));
+                    for (const cf of claimFlags) {
+                        const m = cf.match(/^\$Sending-(.+?)(?:-(\d+))?$/);
+                        if (!m) continue;
+                        const tsSec = m[2] ? parseInt(m[2]) : 0;
+                        const ageSec = nowSec - tsSec;
+                        if (ageSec * 1000 > STALE_CLAIM_MS) {
+                            try {
+                                await client.removeFlags(outboxFolder.path, uid, [cf]);
+                                console.log(`  [outbox] Swept stale claim ${cf} on UID ${uid} (age ${Math.round(ageSec / 60)}m)`);
+                            } catch { /* ignore */ }
+                        }
                     }
-                }
 
-                const flagsNow = (claimFlags.length > 0)
-                    ? await client.getFlags(outboxFolder.path, uid)
-                    : flags;
-                if (flagsNow.some((f: string) => f.startsWith("$Sending"))) return { skip: true };
-                if (flagsNow.includes("$PermanentFailure")) return { skip: true };
-                if (flagsNow.includes("$Failed")) {
-                    await client.removeFlags(outboxFolder.path, uid, ["$Failed"]);
-                }
+                    const flagsNow = (claimFlags.length > 0)
+                        ? await client.getFlags(outboxFolder.path, uid)
+                        : flags;
+                    if (flagsNow.some((f: string) => f.startsWith("$Sending"))) return { skip: true };
+                    if (flagsNow.includes("$PermanentFailure")) return { skip: true };
+                    if (flagsNow.includes("$Failed")) {
+                        await client.removeFlags(outboxFolder.path, uid, ["$Failed"]);
+                    }
 
-                await client.addFlags(outboxFolder.path, uid, [sendingFlag]);
+                    await client.addFlags(outboxFolder.path, uid, [sendingFlag]);
 
-                // TOCTOU re-check: if two devices addFlags concurrently both
-                // see ≥2 sending flags. Fail-safe: both back off, next tick
-                // one wins.
-                const flagsAfter = await client.getFlags(outboxFolder.path, uid);
-                const sendingFlags = flagsAfter.filter((f: string) => f.startsWith("$Sending"));
-                if (sendingFlags.length > 1 || (sendingFlags.length === 1 && sendingFlags[0] !== sendingFlag)) {
-                    await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
-                    return { skip: true };
+                    // TOCTOU re-check: if two devices addFlags concurrently both
+                    // see ≥2 sending flags. Fail-safe: both back off, next tick
+                    // one wins.
+                    const flagsAfter = await client.getFlags(outboxFolder.path, uid);
+                    const sendingFlags = flagsAfter.filter((f: string) => f.startsWith("$Sending"));
+                    if (sendingFlags.length > 1 || (sendingFlags.length === 1 && sendingFlags[0] !== sendingFlag)) {
+                        await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
+                        return { skip: true };
+                    }
                 }
 
                 const msg = await client.fetchMessageByUid(outboxFolder.path, uid, { source: true });
                 if (!msg?.source) {
-                    await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
+                    if (supportsKeywords) await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
                     return { skip: true };
                 }
-                return { source: msg.source };
+                return { source: msg.source, supportsKeywords };
             }, { lane: "outbox" });
 
             if ((result as any).skip) continue;
             const source = (result as any).source as string;
+            const supportsKeywords = !!(result as any).supportsKeywords;
 
             // SMTP send is its own connection — not an IMAP op, doesn't go
             // through withConnection.
@@ -5147,12 +5161,14 @@ export class ImapManager extends EventEmitter {
             } catch (e: any) {
                 const errMsg = e.message || String(e);
                 console.error(`  [outbox] Send failed UID ${uid}: ${errMsg}`);
-                try {
-                    await this.withConnection(accountId, async (client) => {
-                        await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
-                        await client.addFlags(outboxFolder.path, uid, ["$Failed"]);
-                    }, { lane: "outbox" });
-                } catch { /* best-effort */ }
+                if (supportsKeywords) {
+                    try {
+                        await this.withConnection(accountId, async (client) => {
+                            await client.removeFlags(outboxFolder.path, uid, [sendingFlag]);
+                            await client.addFlags(outboxFolder.path, uid, ["$Failed"]);
+                        }, { lane: "outbox" });
+                    } catch { /* best-effort */ }
+                }
                 // Suppress the banner for transient network errors. ETIMEDOUT
                 // on a Dovecot send is the server being slow / a TCP idle
                 // drop, NOT a user-actionable failure — the next outbox tick

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

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