@bobfrankston/mailx-sync 0.1.20 → 0.1.21

package/gmail.d.ts

@@ -58,7 +58,7 @@ export declare class GmailApiProvider implements MailProvider {
      *  `\Flagged` is the presence of STARRED. We always send both add and
      *  remove so the end state matches regardless of what was there before,
      *  which makes the call idempotent and safe to retry. */
-    setFlags(folder: string, uid: number, flags: string[]): Promise<void>;
+    setFlags(folder: string, uid: number, flags: string[], messageId?: string): Promise<void>;
     /** Move a message to the trash label. Gmail treats trash as a label, not
      *  as a destination folder — `POST /messages/{id}/trash` is the native
      *  path (equivalent to setting TRASH and removing INBOX in one op).

package/gmail.js

@@ -530,10 +530,19 @@ export class GmailApiProvider {
      *  `\Flagged` is the presence of STARRED. We always send both add and
      *  remove so the end state matches regardless of what was there before,
      *  which makes the call idempotent and safe to retry. */
-    async setFlags(folder, uid, flags) {
-        const query = `in:${this.folderToLabel(folder)}`;
-        const ids = await this.listMessageIds(query, 1000);
-        const id = ids.find(id => idToUid(id) === uid);
+    async setFlags(folder, uid, flags, messageId) {
+        // Prefer the caller-supplied Gmail message id (stored provider_id).
+        // Without it we fall back to a capped list-and-hash search, which
+        // misses any message past the most-recent ~1000 — so starring an older
+        // message threw "UID not found in INBOX", the flag never reached Gmail,
+        // and the local star got reverted on the next sync (Bob 2026-06-12).
+        // Same fix already applied to trashMessage.
+        let id = messageId;
+        if (!id) {
+            const query = `in:${this.folderToLabel(folder)}`;
+            const ids = await this.listMessageIds(query, 1000);
+            id = ids.find(id => idToUid(id) === uid);
+        }
         if (!id)
             throw new Error(`Gmail setFlags: UID ${uid} not found in ${folder}`);
         const flagSet = new Set(flags);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-sync",
-  "version": "0.1.20",
+  "version": "0.1.21",
   "description": "Platform-agnostic mail provider implementations + sync orchestration. Single source of truth for Gmail/IMAP/Outlook protocol code, consumed by both desktop (Node) and Android (WebView) — eliminates the parallel mailx-imap/mailx-store-web Gmail providers that drifted in practice.",
   "main": "index.js",
   "types": "index.ts",