@bobfrankston/rmfmail 1.2.210 → 1.2.213

package/client/app.js

@@ -338,9 +338,16 @@ const alertText = document.getElementById("alert-text");
 const alertDismiss = document.getElementById("alert-dismiss");
 const dismissedAlerts = new Set();
 let alertAutoDismissTimer = null;
+/** Latched once the daemon reports a read-only DB handle. Every other error
+ *  that follows is a SYMPTOM of it (each account's "attempt to write a
+ *  readonly database", every failed flag/move), and letting those overwrite
+ *  the banner buries the one message that names the actual problem. */
+let dbReadOnlyLatched = false;
 function showAlert(message, key, opts) {
     if (key && dismissedAlerts.has(key))
         return;
+    if (dbReadOnlyLatched && key !== "db-readonly")
+        return;
     if (alertBanner && alertText) {
         alertText.textContent = message;
         alertBanner.hidden = false;
@@ -389,20 +396,14 @@ function hideAlert(opts) {
     }
 }
 alertDismiss?.addEventListener("click", () => hideAlert({ force: true }));
-/** Show the alert banner with a "Restart" button wired to the mailxapi
- *  restartDaemon action. Used when a watched config file whose changes
- *  don't apply live (accounts.jsonc) has been modified. */
-function showRestartForConfigBanner() {
+/** Append a "Restart now" button to the alert banner, wired to the mailxapi
+ *  restartDaemon action. Shared by every banner whose only cure is a fresh
+ *  daemon (accounts.jsonc changed, read-only DB handle). No-op if one is
+ *  already there. */
+function addRestartButton() {
     if (!alertBanner || !alertText)
         return;
-    // Timestamp in the banner so repeated / spurious fires are visually
-    // distinguishable (and the user can see when the change actually
-    // happened, useful for debugging false triggers).
-    const ts = new Date().toLocaleTimeString([], { hour12: false });
-    alertText.textContent = `[${ts}] accounts.jsonc changed — restart to apply.`;
-    alertBanner.hidden = false;
-    alertBanner.dataset.key = "config-restart";
-    // Avoid duplicate buttons across repeat changes.
+    // Avoid duplicate buttons across repeat fires.
     const existing = alertBanner.querySelector("#alert-restart-btn");
     if (existing)
         return;
@@ -435,6 +436,21 @@ function showRestartForConfigBanner() {
     });
     alertText.after(btn);
 }
+/** Show the alert banner with a "Restart" button wired to the mailxapi
+ *  restartDaemon action. Used when a watched config file whose changes
+ *  don't apply live (accounts.jsonc) has been modified. */
+function showRestartForConfigBanner() {
+    if (!alertBanner || !alertText)
+        return;
+    // Timestamp in the banner so repeated / spurious fires are visually
+    // distinguishable (and the user can see when the change actually
+    // happened, useful for debugging false triggers).
+    const ts = new Date().toLocaleTimeString([], { hour12: false });
+    alertText.textContent = `[${ts}] accounts.jsonc changed — restart to apply.`;
+    alertBanner.hidden = false;
+    alertBanner.dataset.key = "config-restart";
+    addRestartButton();
+}
 // ── Wire up components ──
 const folderTree = document.getElementById("folder-tree");
 let currentFolderSpecialUse = "";
@@ -3310,6 +3326,21 @@ onWsEvent((event) => {
                 const slot = document.querySelector(".folder-loading");
                 if (slot)
                     slot.textContent = msg;
+                // The message-list pane too — on a phone the folder tree is
+                // off-canvas and the viewer pane is behind the list, so the
+                // tree/viewer slots above are both invisible at boot. This is
+                // the pane the user is actually staring at (Bob 2026-08-01:
+                // "blank screen and 'syncing' at the bottom"). `.ml-empty`
+                // exists only while the list has no rows, so live message
+                // rows are never overwritten.
+                const mlEmpty = document.querySelector("#ml-body .ml-empty");
+                if (mlEmpty)
+                    mlEmpty.textContent = msg;
+                // Zero-account welcome pane (the visible one on a phone until
+                // accounts land) — folder-tree.ts renders this slot.
+                const narrationSlot = document.getElementById("bootstrap-narration");
+                if (narrationSlot)
+                    narrationSlot.textContent = msg;
             }
             break;
         }
@@ -6141,7 +6172,20 @@ versionPromise.then((d) => {
         if (tip)
             el.title = tip;
     }
-    if (d.settingsError) {
+    if (d.dbWriteDenied) {
+        // The daemon's SQLite handle came up read-only (OS refused read-write
+        // at open; SQLite downgrades silently). Reads all work, so the app
+        // LOOKS fine — but nothing is ever saved: ★ does nothing, moves snap
+        // back, no new mail lands. Say so once, stickily, with the one action
+        // that fixes it, instead of letting it dribble out as a per-account
+        // "attempt to write a readonly database" banner (Bob 2026-08-02).
+        if (alertBanner)
+            alertBanner.style.background = "oklch(0.45 0.18 25)";
+        dbReadOnlyLatched = true;
+        showAlert("Database is READ-ONLY — nothing you do is being saved (★, moves, new mail). Restart rmfmail.", "db-readonly", { sticky: true });
+        addRestartButton();
+    }
+    else if (d.settingsError) {
         showAlert(d.settingsError, "settings-error");
         // Add repair button to the banner
         const banner = document.getElementById("alert-banner");