@bobfrankston/mailx-imap 0.1.36 → 0.1.38

package/index.js

@@ -3140,6 +3140,26 @@ export class ImapManager extends EventEmitter {
                 }
             }
             const uid = typeof result === "number" ? result : result?.uid || null;
+            // Insert a local DB row + body file immediately so the user's
+            // Drafts folder shows the draft without waiting for the next
+            // full Drafts sync. Bob 2026-05-12: "I started a reply which
+            // I saved as a draft somewhere but I can't find it by looking
+            // in drafts nor by searching." The IMAP APPEND just returned,
+            // so we have both the source bytes and the real server UID —
+            // same pattern copyToSent uses. recalcFolderCounts +
+            // folderCountsChanged inside insertLocalRowFromSource bump
+            // the Drafts badge as well.
+            if (uid !== null && uid > 0) {
+                try {
+                    const sourceStr = typeof rawMessage === "string"
+                        ? rawMessage
+                        : rawMessage.toString("utf-8");
+                    await this.insertLocalRowFromSource(accountId, drafts, uid, sourceStr, ["\\Draft", "\\Seen"]);
+                }
+                catch (e) {
+                    console.error(`  [drafts] local-insert after APPEND failed: ${e?.message || e}`);
+                }
+            }
             return uid;
         });
     }
@@ -3152,6 +3172,34 @@ export class ImapManager extends EventEmitter {
             return;
         if (!draftUid && !draftId)
             return;
+        // Local-first: drop the draft from the local DB FIRST so the user's
+        // Drafts view reflects the Send click instantly. The IMAP delete is
+        // queued for the background reconciler. Without this, the draft
+        // stayed visible in Drafts until the IMAP round-trip completed (or
+        // forever if it failed) — Bob 2026-05-12: "when I send a letter you
+        // seem to still leave it in draft so I don't know if it has been
+        // sent or not. Once you hand it off it must leave draft."
+        let localDeletedUid = 0;
+        if (draftUid) {
+            const existing = this.db.getMessageByUid(accountId, draftUid, drafts.id);
+            if (existing) {
+                this.unlinkBodyFile(accountId, draftUid, drafts.id).catch(() => { });
+                this.db.deleteMessage(accountId, draftUid, "user sent the message (draft cleanup)", "mailx-imap deleteDraft (local)");
+                localDeletedUid = draftUid;
+            }
+        }
+        if (draftId && !localDeletedUid) {
+            // Look up any local draft rows by X-Mailx-Draft-ID. The DB
+            // doesn't index that header directly, but locally-saved drafts
+            // share a stable header set; rely on the UID path 99% of the
+            // time. This branch covers the "draft was saved on another
+            // device, came in via sync, no UID known yet" edge case — for
+            // now the IMAP path below handles it.
+        }
+        if (localDeletedUid > 0) {
+            this.db.recalcFolderCounts(drafts.id);
+            this.emit("folderCountsChanged", accountId, {});
+        }
         let succeeded = false;
         try {
             await this.withConnection(accountId, async (client) => {
@@ -3190,12 +3238,9 @@ export class ImapManager extends EventEmitter {
             console.error(`  [drafts] withConnection failed for ${accountId}: ${e?.message || e}`);
         }
         // Reliable cleanup: if the inline attempt didn't actually delete the
-        // draft, queue a sync_action so the regular processSyncActions loop
-        // retries it later. Without this, transient IMAP failures left stale
-        // drafts in the folder ("draft droppings" that pile up after sends).
-        // Only the UID path can be queued — searchByHeader retries on every
-        // future call anyway, and queuing a header-lookup isn't a real
-        // sync_action shape.
+        // draft on the server, queue a sync_action so the regular
+        // processSyncActions loop retries it later. The local row is already
+        // gone (above), so the user doesn't see stale UI either way.
         if (!succeeded && draftUid) {
             console.log(`  [drafts] Queueing sync_action for retry: delete UID ${draftUid} in ${drafts.path}`);
             this.db.queueSyncAction(accountId, "delete", draftUid, drafts.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.36",
+  "version": "0.1.38",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -9,10 +9,10 @@
   },
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/mailx-types": "^0.1.10",
-    "@bobfrankston/mailx-settings": "^0.1.14",
-    "@bobfrankston/mailx-store": "^0.1.18",
-    "@bobfrankston/iflow-direct": "^0.1.40",
+    "@bobfrankston/mailx-types": "^0.1.11",
+    "@bobfrankston/mailx-settings": "^0.1.16",
+    "@bobfrankston/mailx-store": "^0.1.19",
+    "@bobfrankston/iflow-direct": "^0.1.41",
     "@bobfrankston/tcp-transport": "^0.1.6",
     "@bobfrankston/smtp-direct": "^0.1.8",
     "@bobfrankston/mailx-sync": "^0.1.16",
@@ -37,10 +37,10 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/mailx-types": "^0.1.10",
-      "@bobfrankston/mailx-settings": "^0.1.14",
-      "@bobfrankston/mailx-store": "^0.1.18",
-      "@bobfrankston/iflow-direct": "^0.1.40",
+      "@bobfrankston/mailx-types": "^0.1.11",
+      "@bobfrankston/mailx-settings": "^0.1.16",
+      "@bobfrankston/mailx-store": "^0.1.19",
+      "@bobfrankston/iflow-direct": "^0.1.41",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",
       "@bobfrankston/mailx-sync": "^0.1.16",