@bobfrankston/rmfmail 1.0.700 → 1.0.702

package/client/app.js

@@ -4,6 +4,7 @@
  */
 import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal } from "./components/folder-tree.js";
 import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached, getCurrentFocused, releaseFocus, removeMessagesAndReconcile, setRowFlagged, scrollFocusedIntoView, refreshPriorityIndex } from "./components/message-list.js";
+import { seenOf, flaggedOf, draftOf, setSeen, setFlagged } from "@bobfrankston/mailx-types";
 import { getCurrentMessage, initViewer, popOutCurrentMessage, toggleFullscreenPreview, showPreviewBodyMenu, wrapHtmlBody } from "./components/message-viewer.js";
 import { connectWebSocket, onWsEvent, triggerSync, syncAccount, reauthenticate, getAccounts, getFolders, deleteMessages, undeleteMessage, restartServer, getSyncPending, getVersion, getSettings, saveSettings, getAutocompleteSettings, saveAutocompleteSettings, repairAccounts, updateFlags, markAsSpamMessages, logClientEvent, sendMessage as apiSendMessage } from "./lib/api-client.js";
 import * as messageState from "./lib/message-state.js";
@@ -1324,19 +1325,18 @@ document.getElementById("btn-flag")?.addEventListener("click", async () => {
     const sel = getCurrentFocused();
     if (!sel)
         return;
-    const isFlagged = sel.flags.includes("\\Flagged");
-    const newFlags = isFlagged
-        ? sel.flags.filter((f) => f !== "\\Flagged")
-        : [...sel.flags, "\\Flagged"];
+    const wasFlagged = flaggedOf(sel);
+    setFlagged(sel, !wasFlagged);
     try {
-        await updateFlags(sel.accountId, sel.uid, newFlags);
-        sel.flags = newFlags;
-        messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
+        await updateFlags(sel.accountId, sel.uid, sel.flags);
+        messageState.updateMessageFlags(sel.accountId, sel.uid, sel.flags);
         // Row owns its own DOM \u2014 go through the row object so class + star
         // update atomically and the list/preview stay in sync.
-        setRowFlagged(sel.accountId, sel.uid, newFlags.includes("\\Flagged"));
+        setRowFlagged(sel.accountId, sel.uid, !wasFlagged);
     }
     catch (e) {
+        // Revert local state on failure so visual + data stay consistent.
+        setFlagged(sel, wasFlagged);
         console.error(`Flag toggle failed: ${e.message}`);
     }
 });
@@ -1447,17 +1447,14 @@ document.getElementById("btn-mark-unread")?.addEventListener("click", () => {
     const sel = getCurrentFocused();
     if (!sel)
         return;
-    const isSeen = sel.flags.includes("\\Seen");
-    const newFlags = isSeen
-        ? sel.flags.filter((f) => f !== "\\Seen")
-        : [...sel.flags, "\\Seen"];
-    updateFlags(sel.accountId, sel.uid, newFlags).then(() => {
-        sel.flags = newFlags;
-        messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
+    const wasSeen = seenOf(sel);
+    setSeen(sel, !wasSeen);
+    updateFlags(sel.accountId, sel.uid, sel.flags).then(() => {
+        messageState.updateMessageFlags(sel.accountId, sel.uid, sel.flags);
         const row = document.querySelector(`.ml-row[data-uid="${sel.uid}"][data-account-id="${sel.accountId}"]`);
         if (row)
-            row.classList.toggle("unread", !newFlags.includes("\\Seen"));
-    }).catch(() => { });
+            row.classList.toggle("unread", wasSeen);
+    }).catch(() => { setSeen(sel, wasSeen); });
 });
 document.getElementById("btn-reply")?.addEventListener("click", () => openCompose("reply"));
 document.getElementById("btn-reply-all")?.addEventListener("click", () => openCompose("replyAll"));
@@ -2845,17 +2842,14 @@ document.addEventListener("keydown", (e) => {
         if (!sel)
             return;
         e.preventDefault();
-        const isSeen = sel.flags.includes("\\Seen");
-        const newFlags = isSeen
-            ? sel.flags.filter((f) => f !== "\\Seen")
-            : [...sel.flags, "\\Seen"];
-        updateFlags(sel.accountId, sel.uid, newFlags).then(() => {
-            sel.flags = newFlags;
-            messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
+        const wasSeen = seenOf(sel);
+        setSeen(sel, !wasSeen);
+        updateFlags(sel.accountId, sel.uid, sel.flags).then(() => {
+            messageState.updateMessageFlags(sel.accountId, sel.uid, sel.flags);
             const row = document.querySelector(`.ml-row[data-uid="${sel.uid}"][data-account-id="${sel.accountId}"]`);
             if (row)
-                row.classList.toggle("unread", !newFlags.includes("\\Seen"));
-        }).catch(() => { });
+                row.classList.toggle("unread", wasSeen);
+        }).catch(() => { setSeen(sel, wasSeen); });
     }
     // Z = locate the focused row in the list (scroll-to-selected). After
     // scrolling the list out of sync with the preview, this snaps back.
@@ -4515,7 +4509,7 @@ document.addEventListener("mailx-popout-message", (async (e) => {
     // currently sitting in the user's Drafts folder. The read-only popout
     // surface is missing every action button (Reply, Forward, Edit Draft,
     // …) so dumping a draft into it is a worst-case dead-end.
-    const isDraft = Array.isArray(msg?.flags) && msg.flags.includes("\\Draft");
+    const isDraft = !!msg && draftOf(msg);
     if (isDraft) {
         const accts = await getAccounts();
         const init = {