@bobfrankston/mailx-imap 0.1.167 → 0.1.169

package/index.js

@@ -7,7 +7,7 @@ import { createAutoImapConfig, CompatImapClient } from "@bobfrankston/iflow-dire
 import { authenticateOAuth, clearReauthBackoff } from "@bobfrankston/oauthsupport";
 import { parseSerial, fixCharsetDeclString, storeBus } from "@bobfrankston/mailx-store";
 import { loadSettings, getConfigDir, getHistoryDays, getPrefetch, tokenDirName } from "@bobfrankston/mailx-settings";
-import { isJunkPreviewText } from "@bobfrankston/mailx-types";
+import { isJunkPreviewText, logIfChanged } from "@bobfrankston/mailx-types";
 import { EventEmitter } from "node:events";
 import * as fs from "node:fs";
 import * as path from "node:path";
@@ -2405,10 +2405,13 @@ export class ImapManager extends EventEmitter {
                     && statusMessageCount !== null
                     && serverUidsArr.length === statusMessageCount;
                 if (serverUidsArr.length === 0 && localUidsAll.length > 0 && !authoritative) {
-                    console.log(`  [sync] ${accountId}/${folder.path}: reconcile skipped — server UID list empty but local has ${localUidsAll.length} (treating as transient)`);
+                    // Same line every cycle for a folder stuck in this state —
+                    // logIfChanged prints the first one, each change, and an
+                    // hourly restatement. (Claude Code 2026-08-29)
+                    logIfChanged(`${accountId}/${folder.path}:reconcile`, `  [sync] ${accountId}/${folder.path}: reconcile skipped — server UID list empty but local has ${localUidsAll.length} (treating as transient)`);
                 }
                 else if (!authoritative && localUids.length > 0 && toDelete.length / localUids.length > 0.5) {
-                    console.log(`  [sync] ${accountId}/${folder.path}: reconcile REFUSED — would delete ${toDelete.length}/${localUids.length} (${Math.round(toDelete.length / localUids.length * 100)}%) — probably a sync bug, skipping`);
+                    logIfChanged(`${accountId}/${folder.path}:reconcile`, `  [sync] ${accountId}/${folder.path}: reconcile REFUSED — would delete ${toDelete.length}/${localUids.length} (${Math.round(toDelete.length / localUids.length * 100)}%) — probably a sync bug, skipping`);
                 }
                 else if (toDelete.length > 0) {
                     // DEFERRED DELETE — DO NOT delete immediately. Server-side
@@ -2888,10 +2891,10 @@ export class ImapManager extends EventEmitter {
             const serverUids = new Set(serverUidsArr);
             const localUids = this.db.getUidsForFolder(accountId, folder.id);
             if (serverUidsArr._truncated) {
-                console.log(`  [api] ${accountId}/${folder.path}: reconcile skipped — server list truncated (${serverUidsArr.length} ids)`);
+                logIfChanged(`${accountId}/${folder.path}:api-reconcile`, `  [api] ${accountId}/${folder.path}: reconcile skipped — server list truncated (${serverUidsArr.length} ids)`);
             }
             else if (serverUidsArr.length === 0 && localUids.length > 0) {
-                console.log(`  [api] ${accountId}/${folder.path}: reconcile skipped — server list empty but local has ${localUids.length}`);
+                logIfChanged(`${accountId}/${folder.path}:api-reconcile`, `  [api] ${accountId}/${folder.path}: reconcile skipped — server list empty but local has ${localUids.length}`);
             }
             else {
                 const toDelete = localUids.filter(uid => !serverUids.has(uid));
@@ -2925,7 +2928,7 @@ export class ImapManager extends EventEmitter {
                     }
                 }
                 if (!confirmed) {
-                    console.log(`  [api] ${accountId}/${folder.path}: reconcile refused — would delete ${toDelete.length}/${localUids.length} (${Math.round(toDelete.length / localUids.length * 100)}%) — probably a sync bug, skipping`);
+                    logIfChanged(`${accountId}/${folder.path}:api-reconcile`, `  [api] ${accountId}/${folder.path}: reconcile refused — would delete ${toDelete.length}/${localUids.length} (${Math.round(toDelete.length / localUids.length * 100)}%) — probably a sync bug, skipping`);
                 }
                 else {
                     for (const uid of toDelete) {
@@ -3511,10 +3514,41 @@ export class ImapManager extends EventEmitter {
                         this.syncFolder(accountId, inbox.id).catch(e => console.error(`  [idle] expunge reconcile failed: ${e.message}`));
                     }, 2000);
                 };
+                // A flag changed on a message we already have — someone starred
+                // or read it on another client. The server pushes it during
+                // IDLE as an unsolicited FETCH; before v1.2.296 that was
+                // dropped, so a star set on the phone reached the desktop only
+                // on the next periodic sync of INBOX. Measured on Bob's own
+                // log that was a median of 5 minutes and up to 20 (Bob
+                // 2026-08-29: "how long should it take for a * on the phone to
+                // get to my desktop?").
+                //
+                // The push says WHICH flags but by sequence number, which this
+                // connection cannot map to a UID reliably — so it triggers the
+                // folder's normal QRESYNC pass, which asks the server for
+                // exactly what changed since our modseq and took 53 ms in the
+                // same log. Debounced: marking ten messages read on the phone
+                // arrives as ten pushes.
+                let flagTimer = null;
+                const onFlagChange = () => {
+                    if (flagTimer)
+                        clearTimeout(flagTimer);
+                    flagTimer = setTimeout(() => {
+                        flagTimer = null;
+                        // Our own STOREs bounce straight back as flag pushes;
+                        // the local store already has those.
+                        if (this.db.getPendingSyncActions(accountId).length > 0)
+                            return;
+                        const inbox = this.db.getFolders(accountId).find(f => f.specialUse === "inbox");
+                        if (!inbox)
+                            return;
+                        this.syncFolder(accountId, inbox.id).catch(e => console.error(`  [idle] flag-change sync failed: ${e.message}`));
+                    }, 1500);
+                };
                 const stop = await watchClient.watchMailbox("INBOX", (newCount) => {
                     console.log(`  [idle] ${accountId}: ${newCount} new message(s)`);
                     this.syncInboxNewOnly(accountId).catch(e => console.error(`  [idle] sync error: ${e.message}`));
-                }, { notifySpec, onMailboxStatus, onExpunge });
+                }, { notifySpec, onMailboxStatus, onExpunge, onFlagChange });
                 this.watchers.set(accountId, async () => {
                     await stop();
                     await watchClient.logout();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.167",
+  "version": "0.1.169",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -9,10 +9,10 @@
   },
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/mailx-types": "^0.1.73",
-    "@bobfrankston/mailx-settings": "^0.1.70",
-    "@bobfrankston/mailx-store": "^0.1.107",
-    "@bobfrankston/iflow-direct": "^0.1.65",
+    "@bobfrankston/mailx-types": "^0.1.75",
+    "@bobfrankston/mailx-settings": "^0.1.72",
+    "@bobfrankston/mailx-store": "^0.1.108",
+    "@bobfrankston/iflow-direct": "^0.1.68",
     "@bobfrankston/tcp-transport": "^0.1.8",
     "@bobfrankston/smtp-direct": "^0.1.9",
     "@bobfrankston/mailx-sync": "^0.1.29",
@@ -37,10 +37,10 @@
   },
   ".transformedSnapshot": {
     "dependencies": {
-      "@bobfrankston/mailx-types": "^0.1.73",
-      "@bobfrankston/mailx-settings": "^0.1.70",
-      "@bobfrankston/mailx-store": "^0.1.107",
-      "@bobfrankston/iflow-direct": "^0.1.65",
+      "@bobfrankston/mailx-types": "^0.1.75",
+      "@bobfrankston/mailx-settings": "^0.1.72",
+      "@bobfrankston/mailx-store": "^0.1.108",
+      "@bobfrankston/iflow-direct": "^0.1.68",
       "@bobfrankston/tcp-transport": "^0.1.8",
       "@bobfrankston/smtp-direct": "^0.1.9",
       "@bobfrankston/mailx-sync": "^0.1.29",