@bobfrankston/mailx-imap 0.1.68 → 0.1.69

package/index.js

@@ -1250,7 +1250,7 @@ export class ImapManager extends EventEmitter {
                     for (const m of qr.changedMessages) {
                         try {
                             const flagsArr = Array.from(m.flags || []).map(f => String(f));
-                            this.db.updateMessageFlags(accountId, m.uid, flagsArr);
+                            this.db.updateMessageFlags(accountId, folderId, m.uid, flagsArr);
                         }
                         catch { /* row may have just been VANISHED */ }
                     }
@@ -3467,6 +3467,12 @@ export class ImapManager extends EventEmitter {
             // Delete previous draft — try UID first (fast path), and ALWAYS also try
             // searchByHeader(X-Mailx-Draft-ID) as a safety net. Running both catches
             // orphans from a crash-mid-save or a UID delete that failed silently.
+            //
+            // CRITICAL: also delete the LOCAL DB row for the superseded UID. Without
+            // that, every checkpoint save left a stale row behind — IMAP had only
+            // the latest copy but the local Drafts view rendered every past UID as
+            // its own row, producing the "Drafts is flooded with droppings" symptom
+            // (Bob 2026-05-27 — 8+ rows for a single in-progress reply).
             if (previousDraftUid) {
                 try {
                     await client.deleteMessageByUid(drafts.path, previousDraftUid);
@@ -3474,6 +3480,10 @@ export class ImapManager extends EventEmitter {
                 catch (e) {
                     console.error(`  [drafts] Delete prev UID ${previousDraftUid} failed: ${e.message}`);
                 }
+                try {
+                    this.db.deleteMessage(accountId, drafts.id, previousDraftUid, "previous draft superseded", "saveDraft");
+                }
+                catch { /* */ }
             }
             if (draftId) {
                 try {
@@ -3483,6 +3493,10 @@ export class ImapManager extends EventEmitter {
                             await client.deleteMessageByUid(drafts.path, uid);
                         }
                         catch { /* next */ }
+                        try {
+                            this.db.deleteMessage(accountId, drafts.id, uid, `draft superseded by newer save (id=${draftId})`, "saveDraft");
+                        }
+                        catch { /* */ }
                     }
                     if (uids.length > 0)
                         console.log(`  [drafts] Deleted ${uids.length} stale draft(s) by ID ${draftId}`);

package/package.json

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