@bobfrankston/rmfmail 1.0.700 → 1.0.702

package/client/app.bundle.js

@@ -634,6 +634,48 @@ var init_message_state = __esm({
   }
 });
 
+// packages/mailx-types/contact-rules.js
+var init_contact_rules = __esm({
+  "packages/mailx-types/contact-rules.js"() {
+    "use strict";
+  }
+});
+
+// packages/mailx-types/groups.js
+var init_groups = __esm({
+  "packages/mailx-types/groups.js"() {
+    "use strict";
+  }
+});
+
+// packages/mailx-types/index.js
+function _has(msg, flag) {
+  return !!msg.flags && msg.flags.includes(flag);
+}
+function _set(msg, flag, state) {
+  const present = (msg.flags || []).includes(flag);
+  if (state === present)
+    return;
+  const next = state ? [...msg.flags || [], flag] : (msg.flags || []).filter((f) => f !== flag);
+  msg.flags = next;
+}
+var _SEEN, _FLAGGED, _DRAFT, seenOf, flaggedOf, draftOf, setSeen, setFlagged;
+var init_mailx_types = __esm({
+  "packages/mailx-types/index.js"() {
+    "use strict";
+    init_contact_rules();
+    init_groups();
+    _SEEN = "\\Seen";
+    _FLAGGED = "\\Flagged";
+    _DRAFT = "\\Draft";
+    seenOf = (m) => _has(m, _SEEN);
+    flaggedOf = (m) => _has(m, _FLAGGED);
+    draftOf = (m) => _has(m, _DRAFT);
+    setSeen = (m, state) => _set(m, _SEEN, state);
+    setFlagged = (m, state) => _set(m, _FLAGGED, state);
+  }
+});
+
 // client/components/message-viewer.js
 function parsedCacheGet(accountId, uid) {
   const key = `${accountId}:${uid}`;
@@ -1047,7 +1089,7 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
       parsedCachePut(accountId, uid, msg);
     currentMessage = msg;
     currentAccountId = accountId;
-    if (!msg.flags.includes("\\Seen")) {
+    if (!seenOf(msg)) {
       let enabled = true;
       let delaySec = 2;
       try {
@@ -1059,21 +1101,24 @@ async function showMessage(accountId, uid, folderId, specialUse, isRetry = false
       }
       if (enabled) {
         const captureGen = gen;
-        const newFlags = [...msg.flags, "\\Seen"];
-        if (delaySec === 0) {
-          updateFlags(accountId, uid, newFlags);
-        } else {
-          setTimeout(() => {
-            if (captureGen !== showMessageGeneration)
-              return;
-            updateFlags(accountId, uid, newFlags);
-            msg.flags = newFlags;
-            try {
-              updateMessageFlags(accountId, uid, newFlags);
-            } catch {
-            }
-          }, delaySec * 1e3);
-        }
+        const apply = () => {
+          if (captureGen !== showMessageGeneration)
+            return;
+          setSeen(msg, true);
+          updateFlags(accountId, uid, msg.flags);
+          try {
+            updateMessageFlags(accountId, uid, msg.flags);
+          } catch {
+          }
+          try {
+            setRowSeen(accountId, uid, true);
+          } catch {
+          }
+        };
+        if (delaySec === 0)
+          apply();
+        else
+          setTimeout(apply, delaySec * 1e3);
       }
     }
     headerEl.hidden = false;
@@ -2021,6 +2066,8 @@ var init_message_viewer = __esm({
     init_api_client();
     init_context_menu();
     init_message_state();
+    init_message_list();
+    init_mailx_types();
     currentMessage = null;
     currentAccountId = "";
     lastAutoOpenedDraftUid = -1;
@@ -2192,6 +2239,7 @@ __export(message_list_exports, {
   removeMessagesAndReconcile: () => removeMessagesAndReconcile,
   scrollFocusedIntoView: () => scrollFocusedIntoView,
   setRowFlagged: () => setRowFlagged,
+  setRowSeen: () => setRowSeen,
   showThreadPopup: () => showThreadPopup
 });
 function cacheKey(mode, a, f, flagged, q) {
@@ -2313,6 +2361,11 @@ function setRowFlagged(accountId, uid, yes) {
   if (row)
     row.setFlaggedClass(yes);
 }
+function setRowSeen(accountId, uid, yes) {
+  const row = rowByKey.get(rowKey(accountId, uid));
+  if (row)
+    row.setUnreadClass(!yes);
+}
 function scrollFocusedIntoView() {
   if (focusedRow)
     focusedRow.el.scrollIntoView({ block: "center" });
@@ -2803,7 +2856,7 @@ async function showThreadPopup(pillEl, headMsg) {
   for (const msg of thread) {
     const item = document.createElement("div");
     item.className = "ml-thread-popup-item";
-    if (!msg.flags.includes("\\Seen"))
+    if (!seenOf(msg))
       item.classList.add("unread");
     const from = document.createElement("span");
     from.className = "ml-thread-popup-from";
@@ -2908,6 +2961,7 @@ var init_message_list = __esm({
     init_message_viewer();
     init_context_menu();
     init_folder_picker();
+    init_mailx_types();
     currentSpecialUse = "";
     lastClickedRow = null;
     loading = false;
@@ -2970,9 +3024,9 @@ var init_message_list = __esm({
         this.el = row;
         row.className = "ml-row";
         row.draggable = true;
-        if (!msg.flags.includes("\\Seen"))
+        if (!seenOf(msg))
           row.classList.add("unread");
-        if (msg.flags.includes("\\Flagged"))
+        if (flaggedOf(msg))
           row.classList.add("flagged");
         if (!msg.bodyPath)
           row.classList.add("not-downloaded");
@@ -3001,7 +3055,7 @@ var init_message_list = __esm({
         avatar.addEventListener("contextmenu", (e) => this.onAvatarContextMenu(e));
         const flag = document.createElement("span");
         flag.className = "ml-flag";
-        flag.textContent = msg.flags.includes("\\Flagged") ? "\u2605" : "\u2606";
+        flag.textContent = flaggedOf(msg) ? "\u2605" : "\u2606";
         flag.title = "Toggle flag";
         flag.addEventListener("click", (e) => this.onFlagClick(e));
         this.flagEl = flag;
@@ -3097,6 +3151,28 @@ var init_message_list = __esm({
       markBodyCached() {
         this.el.classList.remove("not-downloaded");
       }
+      // Visual-state accessors for flag toggles. Read the *visual* state
+      // (CSS class), not `this.msg.flags`, because the flag array can lag
+      // the rendered class — auto-mark-as-read removes the `unread` class
+      // immediately on click but `\Seen` doesn't land in `msg.flags` until
+      // the auto-mark timer fires (default 2 s). Right-click menus and
+      // keyboard shortcuts ask "what does the user see?" — that's what
+      // these getters answer. Setters keep both visual + flag-array in
+      // sync so the next read by either side agrees.
+      get isSeen() {
+        return !this.el.classList.contains("unread");
+      }
+      set isSeen(yes) {
+        this.setUnreadClass(!yes);
+        setSeen(this.msg, yes);
+      }
+      get isFlagged() {
+        return this.el.classList.contains("flagged");
+      }
+      set isFlagged(yes) {
+        this.setFlaggedClass(yes);
+        setFlagged(this.msg, yes);
+      }
       onAvatarClick(e) {
         e.stopPropagation();
         const body = document.getElementById("ml-body");
@@ -3155,13 +3231,13 @@ var init_message_list = __esm({
       }
       async onFlagClick(e) {
         e.stopPropagation();
-        const isFlagged = this.el.classList.contains("flagged");
-        const currentFlags = this.msg.flags || [];
-        const newFlags = isFlagged ? currentFlags.filter((f) => f !== "\\Flagged") : [...currentFlags, "\\Flagged"];
+        const newFlaggedState = !this.el.classList.contains("flagged");
+        const probe = { flags: [...this.msg.flags || []] };
+        setFlagged(probe, newFlaggedState);
         try {
-          await updateFlags(this.accountId, this.msg.uid, newFlags);
-          this.msg.flags = newFlags;
-          this.setFlaggedClass(!isFlagged);
+          await updateFlags(this.accountId, this.msg.uid, probe.flags);
+          this.msg.flags = probe.flags;
+          this.setFlaggedClass(newFlaggedState);
         } catch {
         }
       }
@@ -3175,10 +3251,8 @@ var init_message_list = __esm({
           const willBeSelected = !this.isSelected;
           this.setSelected(willBeSelected);
           lastClickedRow = this.el;
-          if (willBeSelected) {
+          if (willBeSelected)
             focusRow(this);
-            this.setUnreadClass(false);
-          }
           updateBulkBar();
           return;
         }
@@ -3188,13 +3262,11 @@ var init_message_list = __esm({
             clearSelection();
             selectRange(anchor, this.el);
             lastClickedRow = this.el;
-            this.setUnreadClass(false);
             focusRow(this);
           } else {
             clearSelection();
             focusRow(this);
             lastClickedRow = this.el;
-            this.setUnreadClass(false);
           }
         } else if (e.ctrlKey || e.metaKey) {
           this.setSelected(!this.isSelected);
@@ -3203,7 +3275,6 @@ var init_message_list = __esm({
           clearSelection();
           focusRow(this);
           lastClickedRow = this.el;
-          this.setUnreadClass(false);
         }
         updateBulkBar();
       }
@@ -3289,7 +3360,7 @@ var init_message_list = __esm({
           }
         }
         const isSeen = !this.el.classList.contains("unread");
-        const isFlagged = this.msg.flags.includes("\\Flagged");
+        const isFlagged = this.el.classList.contains("flagged");
         const accountId = this.accountId;
         const msg = this.msg;
         const self = this;
@@ -3297,17 +3368,13 @@ var init_message_list = __esm({
           {
             label: isSeen ? "Mark unread" : "Mark read",
             action: async () => {
-              const set = new Set(msg.flags);
-              if (isSeen)
-                set.delete("\\Seen");
-              else
-                set.add("\\Seen");
-              const newFlags = Array.from(set);
+              const probe = { flags: [...msg.flags || []] };
+              setSeen(probe, !isSeen);
               try {
-                await updateFlags(accountId, msg.uid, newFlags);
-                msg.flags = newFlags;
-                updateMessageFlags(accountId, msg.uid, newFlags);
-                self.setUnreadClass(!newFlags.includes("\\Seen"));
+                await updateFlags(accountId, msg.uid, probe.flags);
+                msg.flags = probe.flags;
+                updateMessageFlags(accountId, msg.uid, probe.flags);
+                self.setUnreadClass(isSeen);
               } catch {
               }
             }
@@ -3315,11 +3382,12 @@ var init_message_list = __esm({
           {
             label: isFlagged ? "Unflag" : "Flag",
             action: async () => {
-              const newFlags = isFlagged ? msg.flags.filter((f) => f !== "\\Flagged") : [...msg.flags, "\\Flagged"];
+              const probe = { flags: [...msg.flags || []] };
+              setFlagged(probe, !isFlagged);
               try {
-                await updateFlags(accountId, msg.uid, newFlags);
-                msg.flags = newFlags;
-                self.setFlaggedClass(newFlags.includes("\\Flagged"));
+                await updateFlags(accountId, msg.uid, probe.flags);
+                msg.flags = probe.flags;
+                self.setFlaggedClass(!isFlagged);
               } catch {
               }
             }
@@ -5805,6 +5873,7 @@ async function updateFolderCounts() {
 
 // client/app.ts
 init_message_list();
+init_mailx_types();
 init_message_viewer();
 init_api_client();
 init_message_state();
@@ -6711,14 +6780,14 @@ document.getElementById("btn-tb-spam")?.addEventListener("click", spamSelectedMe
 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;
-    updateMessageFlags(sel.accountId, sel.uid, newFlags);
-    setRowFlagged(sel.accountId, sel.uid, newFlags.includes("\\Flagged"));
+    await updateFlags(sel.accountId, sel.uid, sel.flags);
+    updateMessageFlags(sel.accountId, sel.uid, sel.flags);
+    setRowFlagged(sel.accountId, sel.uid, !wasFlagged);
   } catch (e) {
+    setFlagged(sel, wasFlagged);
     console.error(`Flag toggle failed: ${e.message}`);
   }
 });
@@ -6807,14 +6876,14 @@ document.getElementById("btn-compose")?.addEventListener("click", () => openComp
 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;
-    updateMessageFlags(sel.accountId, sel.uid, newFlags);
+  const wasSeen = seenOf(sel);
+  setSeen(sel, !wasSeen);
+  updateFlags(sel.accountId, sel.uid, sel.flags).then(() => {
+    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"));
+    if (row) row.classList.toggle("unread", wasSeen);
   }).catch(() => {
+    setSeen(sel, wasSeen);
   });
 });
 document.getElementById("btn-reply")?.addEventListener("click", () => openCompose("reply"));
@@ -7828,14 +7897,14 @@ document.addEventListener("keydown", (e) => {
     const sel = getCurrentFocused();
     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;
-      updateMessageFlags(sel.accountId, sel.uid, newFlags);
+    const wasSeen = seenOf(sel);
+    setSeen(sel, !wasSeen);
+    updateFlags(sel.accountId, sel.uid, sel.flags).then(() => {
+      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"));
+      if (row) row.classList.toggle("unread", wasSeen);
     }).catch(() => {
+      setSeen(sel, wasSeen);
     });
   }
   if (e.key.toLowerCase() === "z" && !e.ctrlKey && !e.metaKey && !e.altKey) {
@@ -9173,7 +9242,7 @@ document.addEventListener("mailx-popout-message", (async (e) => {
     alert(`Couldn't load message: ${err?.message || err}`);
     return;
   }
-  const isDraft = Array.isArray(msg?.flags) && msg.flags.includes("\\Draft");
+  const isDraft = !!msg && draftOf(msg);
   if (isDraft) {
     const accts = await getAccounts();
     const init = {