@bobfrankston/rmfmail 1.0.699 → 1.0.701

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 { FLAG, flagOps } 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,17 +1325,17 @@ document.getElementById("btn-flag")?.addEventListener("click", async () => {
     const sel = getCurrentFocused();
     if (!sel)
         return;
-    const isFlagged = sel.flags.includes("\\Flagged");
+    const isFlagged = flagOps.isFlagged(sel.flags);
     const newFlags = isFlagged
-        ? sel.flags.filter((f) => f !== "\\Flagged")
-        : [...sel.flags, "\\Flagged"];
+        ? flagOps.remove(sel.flags, FLAG.FLAGGED)
+        : flagOps.add(sel.flags, FLAG.FLAGGED);
     try {
         await updateFlags(sel.accountId, sel.uid, newFlags);
         sel.flags = newFlags;
         messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
         // 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, flagOps.isFlagged(newFlags));
     }
     catch (e) {
         console.error(`Flag toggle failed: ${e.message}`);
@@ -1447,16 +1448,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"];
+    const isSeen = flagOps.isSeen(sel.flags);
+    const newFlags = flagOps.set(sel.flags, FLAG.SEEN, !isSeen);
     updateFlags(sel.accountId, sel.uid, newFlags).then(() => {
         sel.flags = newFlags;
         messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
         const row = document.querySelector(`.ml-row[data-uid="${sel.uid}"][data-account-id="${sel.accountId}"]`);
         if (row)
-            row.classList.toggle("unread", !newFlags.includes("\\Seen"));
+            row.classList.toggle("unread", !flagOps.isSeen(newFlags));
     }).catch(() => { });
 });
 document.getElementById("btn-reply")?.addEventListener("click", () => openCompose("reply"));
@@ -2845,16 +2844,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"];
+        const isSeen = flagOps.isSeen(sel.flags);
+        const newFlags = flagOps.set(sel.flags, FLAG.SEEN, !isSeen);
         updateFlags(sel.accountId, sel.uid, newFlags).then(() => {
             sel.flags = newFlags;
             messageState.updateMessageFlags(sel.accountId, sel.uid, newFlags);
             const row = document.querySelector(`.ml-row[data-uid="${sel.uid}"][data-account-id="${sel.accountId}"]`);
             if (row)
-                row.classList.toggle("unread", !newFlags.includes("\\Seen"));
+                row.classList.toggle("unread", !flagOps.isSeen(newFlags));
         }).catch(() => { });
     }
     // Z = locate the focused row in the list (scroll-to-selected). After
@@ -4515,7 +4512,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 = Array.isArray(msg?.flags) && flagOps.isDraft(msg.flags);
     if (isDraft) {
         const accts = await getAccounts();
         const init = {