@bobfrankston/rmfmail 1.1.87 → 1.1.89

package/client/app.js

@@ -1921,8 +1921,11 @@ function recordSearchHistory(query) {
     if (!trimmed)
         return;
     const cur = loadSearchHistory();
-    // Move-to-front dedup: if already there, pull it; then prepend.
-    const filtered = cur.filter(q => q !== trimmed);
+    // Move-to-front dedup, AND drop any entry that is a prefix of this
+    // query — that collapses the in-progress typing ("Hod", "Hodd") into
+    // the final "Hoddie", so a live search the user never pressed Enter on
+    // still lands in history exactly once.
+    const filtered = cur.filter(q => q !== trimmed && !trimmed.startsWith(q));
     filtered.unshift(trimmed);
     if (filtered.length > SEARCH_HISTORY_MAX)
         filtered.length = SEARCH_HISTORY_MAX;
@@ -2039,10 +2042,11 @@ function doSearch(immediate = false) {
     loadSearchResults(query, effectiveScope, currentAccountId, currentFolderId, includeTrash);
     setTitle(`${APP_NAME} - Search: ${query}`);
     setActiveTabView({ kind: "search", query, scope: effectiveScope, accountId: currentAccountId, folderId: currentFolderId, includeTrash }, `Search: ${query}`);
-    // Only record on `immediate=true` (Enter / scope change) — debounced
-    // typing would otherwise add every keystroke as its own entry.
-    if (immediate)
-        recordSearchHistory(query);
+    // Record every executed search — Enter AND a settled debounced search
+    // alike. The prefix-pruning in recordSearchHistory() collapses the
+    // keystroke progression, so a search the user typed without pressing
+    // Enter ("Hoddie") still shows up in history.
+    recordSearchHistory(query);
 }
 // Track current folder for scoped search
 let currentAccountId = "";