@bobfrankston/mailx-imap 0.1.87 → 0.1.89

package/index.js

@@ -2664,6 +2664,22 @@ export class ImapManager extends EventEmitter {
                         clearTimeout(expungeTimer);
                     expungeTimer = setTimeout(() => {
                         expungeTimer = null;
+                        // Skip while WE have sync actions draining. Every
+                        // delete/move we push EXPUNGEs on the server and bounces
+                        // straight back here as an onExpunge — and a reconcile
+                        // of a large INBOX is a full server-vs-local set-diff.
+                        // During a bulk delete that meant one ~135k-row reconcile
+                        // per message (the 2s debounce can't coalesce them — our
+                        // moves arrive ~2.5s apart), hammering the DB so the list
+                        // sat on "Loading…" with a stale preview (Bob 2026-06-12).
+                        // The local store already reflects our own moves, so these
+                        // are redundant. The last move empties the queue, so its
+                        // trailing expunge runs exactly ONE reconcile to catch any
+                        // genuinely-external change.
+                        if (this.db.getPendingSyncActions(accountId).length > 0) {
+                            console.log(`  [idle] ${accountId}: INBOX expunge during own drain — deferring reconcile`);
+                            return;
+                        }
                         const inbox = this.db.getFolders(accountId).find(f => f.specialUse === "inbox");
                         if (!inbox)
                             return;
@@ -3416,7 +3432,14 @@ export class ImapManager extends EventEmitter {
                     }
                     try {
                         if (action.action === "flags" && api.setFlags) {
-                            await api.setFlags(folder.path, action.uid, action.flags || []);
+                            // Pass the stored Gmail message id (provider_id) for
+                            // the same reason as trashMessage below: without it
+                            // setFlags falls back to a capped list-and-hash
+                            // search that misses older messages, so starring one
+                            // failed ("UID not found in INBOX") and the local
+                            // star reverted on the next sync (Bob 2026-06-12).
+                            const env = this.db.getMessageByUid(accountId, action.uid);
+                            await api.setFlags(folder.path, action.uid, action.flags || [], env?.providerId);
                             console.log(`  [api] ${accountId}: flags synced UID ${action.uid}`);
                         }
                         else if ((action.action === "delete" || action.action === "trash") && api.trashMessage) {

package/package.json

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