@bobfrankston/mailx-imap 0.1.109 → 0.1.110

package/index.js

@@ -1764,7 +1764,18 @@ export class ImapManager extends EventEmitter {
                     // Apply changed-state FETCH — flag updates since prior modSeq.
                     for (const m of qr.changedMessages) {
                         try {
+                            // LOCAL-FIRST: never let a server reconcile clobber a flag
+                            // the user just changed locally but whose push hasn't drained
+                            // yet — that's the "I unflag it and it quickly comes back"
+                            // bug (Bob 2026-06-27). Skip any message with a pending flag
+                            // action; the drain will push it and a later reconcile syncs.
+                            const pendingFlag = this.db.findPendingSyncAction(accountId, "flags", m.uid, folderId);
+                            if (pendingFlag) {
+                                console.error(`  [flag-reconcile] SKIP uid=${m.uid} folder=${folderId} — pending local flag action wins`);
+                                continue;
+                            }
                             const flagsArr = Array.from(m.flags || []).map(f => String(f));
+                            console.error(`  [flag-reconcile] OVERWRITE uid=${m.uid} folder=${folderId} server=[${flagsArr.join(",")}]`);
                             this.db.updateMessageFlags(accountId, folderId, m.uid, flagsArr);
                         }
                         catch { /* row may have just been VANISHED */ }
@@ -4168,13 +4179,17 @@ export class ImapManager extends EventEmitter {
                             const MANAGED = ["\\Seen", "\\Flagged"];
                             const impliedRemove = MANAGED.filter(f => !desired.includes(f));
                             const toRemove = [...new Set([...explicitRemove, ...impliedRemove])];
+                            // console.error so it lands in the captured daemon log
+                            // (worker stdout is NOT piped; stderr is). Temporary
+                            // diagnostic for the "flag won't persist" hunt.
+                            console.error(`  [flag-drain] ${accountId} uid=${action.uid} folder="${folder.path}" +[${desired.join(",")}] -[${toRemove.join(",")}]`);
                             if (desired.length > 0) {
                                 await client.addFlags(folder.path, action.uid, desired);
                             }
                             if (toRemove.length > 0) {
                                 await client.removeFlags(folder.path, action.uid, toRemove);
                             }
-                            console.log(`  [sync] Updated flags UID ${action.uid} (+[${desired.join(",")}] -[${toRemove.join(",")}])`);
+                            console.error(`  [flag-drain] ${accountId} uid=${action.uid} OK (server STORE done)`);
                             break;
                         }
                         case "append": {

package/package.json

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