@bobfrankston/rmfmail 1.2.73 → 1.2.75

package/packages/mailx-store/db.ts

@@ -2393,9 +2393,28 @@ export class MailxDB {
      *  for code paths that mean it; everyone else must pass folderId. */
     updateMessageFlags(accountId: string, folderId: number | null, uid: number, flags: string[]): void {
         if (folderId != null) {
-            this.db.prepare(
-                "UPDATE messages SET flags_json = ? WHERE account_id = ? AND folder_id = ? AND uid = ?"
-            ).run(JSON.stringify(flags), accountId, folderId, uid);
+            // Resolve the row via its folder MEMBERSHIP, not messages.folder_id.
+            // The message-id collapse (upsertMessage move-detect) keeps ONE
+            // `messages` row per message_id whose `folder_id` is an arbitrary
+            // "primary" folder, while the message appears in OTHER folders via
+            // `message_folders`. A `WHERE folder_id = ?` update therefore MISSED
+            // any message viewed in a folder other than its stored primary —
+            // 0 rows changed, so star/mark-read silently did nothing locally and
+            // the change "came right back" on the next render (Bob 2026-06-27).
+            // Match the membership (the same join getMessageByUid uses).
+            const r = this.db.prepare(`
+                UPDATE messages SET flags_json = ? WHERE id = (
+                    SELECT m.id FROM messages m
+                    JOIN message_folders mf ON mf.message_row_id = m.id
+                    WHERE m.account_id = ? AND mf.folder_id = ? AND mf.uid = ? LIMIT 1
+                )
+            `).run(JSON.stringify(flags), accountId, folderId, uid);
+            // Fallback for any legacy row lacking a membership entry.
+            if (r.changes === 0) {
+                this.db.prepare(
+                    "UPDATE messages SET flags_json = ? WHERE account_id = ? AND folder_id = ? AND uid = ?"
+                ).run(JSON.stringify(flags), accountId, folderId, uid);
+            }
         } else {
             this.db.prepare(
                 "UPDATE messages SET flags_json = ? WHERE account_id = ? AND uid = ?"

package/packages/mailx-types/package.json

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