@bobfrankston/rmfmail 1.2.241 → 1.2.242

package/client/compose/compose.css

@@ -275,6 +275,17 @@ body {
    read a large message being replied to (Bob 2026-06-23). Forcing min-height:0
    bounds each wrapper so the edit-area iframe gets a fixed height and TinyMCE's
    own internal scrollbar engages. */
+/* The toolbar wraps onto extra rows (toolbar_mode:"wrap" in rmf-tiny) rather
+   than hiding buttons behind a "…" popover that covers the message body. For
+   that to be an ENLARGED bar rather than a clipped one, the header must not be
+   a shrink target: it keeps its natural height and the edit area below gives up
+   the space (Bob 2026-08-09: "better to just make the bar bigger if it cannot
+   be wider"). Without this the header inherits flex-shrink:1 and a narrow
+   compose window silently crops the second row. */
+.editor-tinymce .tox-editor-header {
+    flex: 0 0 auto !important;
+}
+
 .editor-tinymce .tox-editor-container,
 .editor-tinymce .tox-sidebar-wrap,
 .editor-tinymce .tox-edit-area {

package/client/lib/rmf-tiny.js

@@ -135,6 +135,20 @@ export async function createTinyMceEditor(container, opts = {}) {
                 // 2026-07-28: pasted/inserted text needs a one-click quote.
                 "blockquote bullist numlist outdent indent | link table image code rmfcode | emoticons charmap | help",
             ].join(" | "),
+            // Wrap the toolbar onto as many rows as it needs instead of hiding
+            // the overflow behind a "…" that opens a FLOATING popover over the
+            // message body (TinyMCE default is toolbar_mode:"floating").
+            //
+            // Two things were wrong with the popover: it covers the text you
+            // are writing, and it does not dismiss on its own (Bob 2026-08-08:
+            // "you temporarily extend the menu as a popover but that does not
+            // go away on its own. It makes more sense to just wrap the buttons
+            // onto a separate line. That works because the width is stable so
+            // there is not any fibrillation"). Wrapping grows the header, which
+            // is the honest use of the space — the compose window width does
+            // not change while you type, so the row count is stable and there
+            // is no reflow churn.
+            toolbar_mode: "wrap",
             // Include "tools" so wordcount and searchreplace are reachable.
             menubar: "file edit view insert format tools",
             // View menu — append zoom commands. fullscreen omitted: this editor

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/rmfmail",
-  "version": "1.2.241",
+  "version": "1.2.242",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -38,7 +38,7 @@
     "@bobfrankston/mailx-store-web": "^0.1.76",
     "@bobfrankston/mailx-sync": "^0.1.29",
     "@bobfrankston/miscinfo": "^1.0.16",
-    "@bobfrankston/msger": "^0.1.422",
+    "@bobfrankston/msger": "^0.1.424",
     "@bobfrankston/node-tcp-transport": "^0.1.10",
     "@bobfrankston/oauthsupport": "^1.0.34",
     "@bobfrankston/rmf-tiny": "^0.1.45",
@@ -118,7 +118,7 @@
       "@bobfrankston/mailx-store-web": "^0.1.76",
       "@bobfrankston/mailx-sync": "^0.1.29",
       "@bobfrankston/miscinfo": "^1.0.16",
-      "@bobfrankston/msger": "^0.1.422",
+      "@bobfrankston/msger": "^0.1.424",
       "@bobfrankston/node-tcp-transport": "^0.1.10",
       "@bobfrankston/oauthsupport": "^1.0.34",
       "@bobfrankston/rmf-tiny": "^0.1.45",

package/packages/mailx-service/index.ts

@@ -2038,6 +2038,11 @@ export class MailxService implements MailxApi {
             let foldersSearched = 0;
             let foldersFailed = 0;
             const failedFolders: string[] = [];
+            // Folders the server says are gone: counted apart from transient
+            // failures so the UI never tells the user to retry something that
+            // cannot succeed.
+            let foldersGone = 0;
+            const goneFolders: string[] = [];
             let droppedHits = 0;
 
             // Bounded concurrency. The per-account ops queue already
@@ -2071,9 +2076,28 @@ export class MailxService implements MailxApi {
                     for (let j = 0; j < results.length; j++) {
                         const r = results[j];
                         if (r.status !== "fulfilled") {
+                            const why = String(r.reason?.message || r.reason || "");
+                            // A mailbox the server says does not exist is not a
+                            // timeout and will never succeed — every search pays
+                            // for it again and the user is told to "retry to
+                            // search them again", which cannot work. Gmail
+                            // answers `[NONEXISTENT] Unknown Mailbox` for the
+                            // `[Imap]/…` rows mailx keeps locally; Dovecot says
+                            // "doesn't exist". Drop the local folder row (the
+                            // IMAP sync path already does this on its own
+                            // wording) and count it separately.
+                            if (/\[NONEXISTENT\]|Unknown Mailbox|doesn't exist|does not exist/i.test(why)) {
+                                foldersGone++;
+                                goneFolders.push(`${acct.id}/${batch[j].path}`);
+                                console.log(`  [server-search] ${acct.id}/${batch[j].path}: mailbox not on server — removing the stale local folder row`);
+                                try { this.db.deleteFolder(batch[j].id); } catch (e: any) {
+                                    console.error(`  [server-search] could not remove stale folder ${batch[j].path}: ${e.message}`);
+                                }
+                                continue;
+                            }
                             foldersFailed++;
                             failedFolders.push(`${acct.id}/${batch[j].path}`);
-                            console.error(`  [server-search] ${acct.id}/${batch[j].path}: ${r.reason?.message || r.reason}`);
+                            console.error(`  [server-search] ${acct.id}/${batch[j].path}: ${why}`);
                             continue;
                         }
                         foldersSearched++;
@@ -2101,14 +2125,20 @@ export class MailxService implements MailxApi {
             // Newest first, then paginate.
             items.sort((a: any, b: any) => (b.date?.getTime?.() || 0) - (a.date?.getTime?.() || 0));
             const sliced = items.slice((page - 1) * pageSize, page * pageSize);
-            if (foldersFailed > 0 || droppedHits > 0) {
-                console.log(`  [server-search] q="${q}" — ${total} hits across ${foldersSearched} folders; ${foldersFailed} folders failed, ${droppedHits} hits dropped`);
+            if (foldersFailed > 0 || droppedHits > 0 || foldersGone > 0) {
+                console.log(`  [server-search] q="${q}" — ${total} hits across ${foldersSearched} folders; ${foldersFailed} failed, ${foldersGone} stale folder(s) removed, ${droppedHits} hits dropped${aborted ? " (superseded)" : ""}`);
             }
             return {
                 items: sliced, total, page, pageSize,
                 aborted,
-                partial: foldersFailed > 0 || droppedHits > 0,
+                // A superseded search is NOT a partial result — the user typed
+                // another character and this generation was abandoned on
+                // purpose. Reporting its untouched folders as failures is what
+                // produced "99 could not be searched (timeout / connection)"
+                // for a search nothing had gone wrong with (Bob 2026-08-09).
+                partial: !aborted && (foldersFailed > 0 || droppedHits > 0),
                 foldersSearched, foldersFailed, failedFolders, droppedHits,
+                foldersGone, goneFolders,
             };
         } else if (scope === "current" && accountId && folderId) {
             // Per-folder search — folder scope is the user's explicit