@bobfrankston/rmfmail 1.1.235 → 1.1.237

package/packages/mailx-imap/index.ts

@@ -4131,11 +4131,28 @@ export class ImapManager extends EventEmitter {
                 const [, original, host, pidStr] = m;
                 if (host !== this.hostname) continue;
                 const pid = parseInt(pidStr);
-                if (pid === myPid) continue; // it's us
-                let alive = false;
-                try { process.kill(pid, 0); alive = true; } catch { /* dead */ }
                 let ageMs = Infinity;
                 try { ageMs = Date.now() - fs.statSync(path.join(dir, f)).mtimeMs; } catch { /* */ }
+                if (pid === myPid) {
+                    // Our own claim. Normally we're actively sending it — leave
+                    // it. But the send is now bounded (60s APPEND timeout), so
+                    // an OWN claim older than STALE_CLAIM_MS means the send
+                    // hung past its timeout, or a prior tick orphaned it (the
+                    // release rename failed). Without reclaiming it the file
+                    // sits in `.sending-` forever — recovery used to skip every
+                    // own-PID claim unconditionally, so a transient connection
+                    // wedge pinned the message even after the link recovered
+                    // (Bob 2026-06-11: two messages stuck .sending-<ourpid>
+                    // while SELECT Outbox was already succeeding again).
+                    if (ageMs < STALE_CLAIM_MS) continue;
+                    try {
+                        fs.renameSync(path.join(dir, f), path.join(dir, original));
+                        console.log(`  [outbox] Recovered our own stale claim ${f} → ${original} (hung ${Math.round(ageMs / 60_000)}m)`);
+                    } catch { /* ignore */ }
+                    continue;
+                }
+                let alive = false;
+                try { process.kill(pid, 0); alive = true; } catch { /* dead */ }
                 // Live PID + recent mtime → genuine sibling owner, leave it.
                 // Live PID + ancient mtime → recycled PID, sweep. Dead PID → sweep.
                 if (alive && ageMs < STALE_CLAIM_MS) continue;
@@ -4261,7 +4278,17 @@ export class ImapManager extends EventEmitter {
                     }
                     try {
                         const raw = fs.readFileSync(claimedPath, "utf-8");
-                        await client.appendMessage(outboxPath, raw, ["\\Seen"]);
+                        // Bound the APPEND. On a wedged connection (Dovecot
+                        // ETIMEDOUT storm) the bare await can hang the full
+                        // 300s inactivity timeout, pinning the file in
+                        // `.sending-` state the whole time and reading to the
+                        // user as "stuck, not sending" (Bob 2026-06-11). A
+                        // 60s cap force-closes the socket and throws, so the
+                        // catch below releases the claim and the next tick
+                        // retries instead of hanging for 5 minutes.
+                        await withTimeout(
+                            client.appendMessage(outboxPath, raw, ["\\Seen"]),
+                            60_000, client, `outbox APPEND ${file}`);
                         fs.renameSync(claimedPath, path.join(sentDir, file));
                         console.log(`  [outbox] Moved ${file} to IMAP Outbox → sent/`);
                     } catch (e: any) {

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

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

package/packages/mailx-store/package.json

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

package/packages/mailx-types/package.json

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