@bobfrankston/rmfmail 1.0.548 → 1.0.550

package/client/app.js

@@ -2044,13 +2044,16 @@ onWsEvent((event) => {
         case "syncActionFailed": {
             // Surface sync failures (move/delete/flag not applied on server)
             // so the user knows local-first actions haven't propagated yet.
-            // Status bar gets a short hint; the alert banner surfaces the
-            // detail with the action verb. Coalesces by `key=conflict` so
-            // multiple failures don't stack — last one wins, dismissable.
+            // Status-bar hint always; banner only when the service is telling
+            // us it has GIVEN UP (message starts with "Gave up"). A full sync
+            // over a flaky link fires syncActionFailed once per attempt, and
+            // a banner per attempt floods DOM work on the WebView main thread.
             const action = event.action === "move" ? "Move" : event.action === "delete" ? "Delete" : event.action;
             if (statusSync)
                 statusSync.textContent = `Sync failed: ${action}`;
-            showAlert(`Couldn't ${action.toLowerCase()} message on the server: ${event.error}. Local change kept; the reconciler will retry.`, "conflict");
+            if (typeof event.error === "string" && /gave up/i.test(event.error)) {
+                showAlert(`Couldn't ${action.toLowerCase()} message on the server after retries: ${event.error}. Local change kept.`, "conflict");
+            }
             break;
         }
         case "reload":