@bobfrankston/mailx 1.0.408 → 1.0.411

package/client/app.js

@@ -3,7 +3,7 @@
  * Wires together all UI components and WebSocket connection.
  */
 import { initFolderTree, refreshFolderTree, updateFolderCounts, setFolderSynced, getFolderSynced, setOutboxTotal } from "./components/folder-tree.js";
-import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, getSelectedMessages, markBodiesCached } from "./components/message-list.js";
+import { initMessageList, loadMessages, loadUnifiedInbox, loadSearchResults, reloadCurrentFolder, clearSearchMode, getSelectedMessages, markBodiesCached } from "./components/message-list.js";
 import { showMessage, getCurrentMessage, initViewer } 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";
@@ -1184,11 +1184,15 @@ let reloadDebounceTimer = null;
 searchInput?.addEventListener("input", () => {
     clearTimeout(searchTimeout);
     if (searchInput.value.trim() === "") {
-        // Cleared — reset immediately, no debounce
+        // Cleared — reset immediately, no debounce. Must exit search mode
+        // first; otherwise reloadCurrentFolder() sees searchMode=true and
+        // re-runs the stale query (user-reported regression 2026-04-24).
+        clearSearchMode();
         const body = document.getElementById("ml-body");
         if (body)
             body.querySelectorAll(".filter-hidden").forEach(r => r.classList.remove("filter-hidden"));
         reloadCurrentFolder();
+        setTitle("mailx");
     }
     else {
         searchTimeout = setTimeout(() => doSearch(false), 300);
@@ -1201,11 +1205,13 @@ searchInput?.addEventListener("keydown", (e) => {
     }
     if (e.key === "Escape") {
         searchInput.value = "";
+        clearSearchMode();
         // Clear any client-side filters
         const body = document.getElementById("ml-body");
         if (body)
             body.querySelectorAll(".filter-hidden").forEach(r => r.classList.remove("filter-hidden"));
         reloadCurrentFolder();
+        setTitle("mailx");
     }
 });
 // Message state handles move/delete — no manual event listener needed

package/client/components/message-list.js

@@ -330,6 +330,13 @@ export function reloadCurrentFolder() {
         loadMessages(currentAccountId, currentFolderId, 1, "", false);
     }
 }
+/** Exit search mode without triggering a reload — caller decides what to load
+ *  next. Used by the search-input "cleared to empty" handler so the next
+ *  reloadCurrentFolder() call doesn't re-run the stale search query. */
+export function clearSearchMode() {
+    searchMode = false;
+    currentSearchQuery = "";
+}
 /** Load unified inbox (all accounts) */
 export async function loadUnifiedInbox(autoSelect = true) {
     unifiedMode = true;

package/client/components/message-viewer.js

@@ -373,8 +373,8 @@ export async function showMessage(accountId, uid, folderId, specialUse, isRetry
             if (anyUrl) {
                 unsubBtn.hidden = false;
                 unsubBtn.textContent = oneClick && httpUrl ? "Unsubscribe (1-click)" : "Unsubscribe";
-                unsubBtn.title = anyUrl;
-                unsubBtn.href = "#";
+                unsubBtn.removeAttribute("title");
+                unsubBtn.href = httpUrl || mailUrl || "#";
                 unsubBtn.onclick = async (e) => {
                     e.preventDefault();
                     const status = document.getElementById("status-sync");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.408",
+  "version": "1.0.411",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",

package/packages/mailx-service/index.js

@@ -278,11 +278,12 @@ export class MailxService {
                 parseListUnsubscribe(parsed2.headers));
             listUnsubscribe = listUnsubscribeHttp || listUnsubscribeMail;
         }
-        // EML path: read the real on-disk path from `envelope.bodyPath` (DB
-        // is authoritative since v1.0.361 — files are opaque UUIDs, not the
-        // old {folderId}/{uid}.eml layout). Synthesizing the legacy path
-        // here showed users a path that doesn't exist.
-        const emlPath = envelope.bodyPath || "";
+        // EML path: re-read the row after the fetch — `fetchMessageBody`
+        // writes the body to disk and updates `body_path` on success, but the
+        // `envelope` snapshot above pre-dates that write, so trusting it
+        // hides the Source button on every just-opened message.
+        const refreshed = this.db.getMessageByUid(accountId, uid, folderId);
+        const emlPath = refreshed?.bodyPath || envelope.bodyPath || "";
         return {
             ...envelope, bodyHtml, bodyText, hasRemoteContent, remoteAllowed: allowRemote,
             attachments, emlPath, deliveredTo, returnPath,