@bobfrankston/rmfmail 1.2.171 → 1.2.173

package/TODO.md

@@ -22,6 +22,7 @@ These items don't need user input — source-only changes, compile-verified befo
 | **8** | **C157 — Android fresh install: paint folder list before message sync** [S/M] | S/M | First sync ran minutes with an empty tree. Prioritize per-account folder-LIST fetch + paint ahead of message sync so the skeleton appears in seconds; message counts stream in behind. |
 | **10** | **C159 — Harper local grammar check** [M] | M | **Desktop SHIPPED v1.2.168** (2026-07-22): daemon-side Harper engine + `grammarLint` IPC, generic squiggle overlay + suggestion popup across ALL THREE editors, Settings toggle default ON, quoted chains + spelling lints skipped. **Remaining: Android** (host LocalLinter in android-bootstrap, harper.js dist in APK assets like sql.js, ~+18 MB) + per-rule config + userdict merge. Details: `docs/harper-integration.md`. |
 | **9** | **C158 — Android double-init after fold-triggered reload** [M] | M | 2026-07-22 13:07:49 logit trail: ONE page reload after unfolding produced TWO module loads, duplicate "bridge installed", duplicate GDrive lookups, and THREE "Starting OAuth flow" launches. Activity did NOT recreate (ConfigChanges already declared; no OnCreate in trail) — find what reloads the WebView on fold and make android-bootstrap init idempotent (single-flight guard on the module-level init, dedupe OAuth intent launches). |
+| **11** | **C160 — Attachment delete: Gmail + Android halves** [M] | M | Desktop IMAP shipped v1.2.173 (right-click chip → Delete one/all: verified local strip + append-confirm-then-delete server replace, .pre-strip.bak backup, UID rebind). Remaining: Gmail REST path (messages.insert + trash of original — changes message id, provider_id rebind) and Android (bridge method + web-service strip using its own parser). Strip helper lives in mailx-service (enumerateMimeLeafParts) — move to mailx-types when Android needs it. |
 | **5** | **C124 — mailto handler on macOS** (Linux ✅ 2026-07-08 v1.2.115) | S | Linux half shipped: `--register-mailto` on linux writes `~/.local/share/applications/rmfmail.desktop` (`MimeType=x-scheme-handler/mailto;`, quoted node+mailx.js Exec, bare `%u`), runs `xdg-mime default` + `update-desktop-database`; `--unregister-mailto` removes it. **Remaining: macOS** — real `.app` bundle (LaunchServices won't register a bare script) with `Info.plist` `CFBundleURLTypes` declaring `mailto` + a tiny launcher binary; `LSSetDefaultHandlerForURLScheme` programmatically sets the default. Rust `rmfmailto-src/` crate already portable — drop the Win32 imports and `cargo build --target x86_64-apple-darwin` produces the macOS launcher. Needs a Mac to build/verify. |
 | ~~7~~ | ~~**C154 — Audit Android bridge vs MailxApi contract**~~ — **DONE** (2026-07-08, v1.2.115). Bridge now has a generic contract-backstop: every service method not explicitly adapted gets a positional pass-through, so a missing method can never again fail as `parent bridge has no method "X"`. Wire-shape mismatches adapted explicitly (`readJsoncFile`/`writeJsoncFile`/`formatJsonc`/`readConfigHelp` → `{content}`/`{ok}`, `drainStoreSync`/`syncFolderNow` → `{ok}`). Contract gained the missing entries (`getMessageSource`, `copyMessages`, `syncFolderNow`, `cancelServerSearch`, `getReminderSound`, `getCalendars`, optional `popoutWindow`) with web-service stubs per the policy (reads → graceful default, writes/OS → notImpl). Bonus fixes: bridge `getMessages` was dropping sort/sortDir/search and `searchMessages` was dropping scope/accountId/folderId — Android sort toggles and scoped search silently didn't work. | S | |
 ~~done — C120 read transport diagnostics~~ shipped v1.0.583 (per-folder timeout error now includes `[conn#X r=YB w=ZB writes=N sinceLastRead=Tms]` snapshot for the doomed socket).

package/client/android-bootstrap.bundle.js

@@ -2631,6 +2631,15 @@ async function idbDelete(key) {
     tx.onerror = () => reject(tx.error);
   });
 }
+async function idbListKeys(prefix) {
+  const db2 = await openSettingsDb();
+  return new Promise((resolve, reject) => {
+    const tx = db2.transaction(STORE_NAME2, "readonly");
+    const req = tx.objectStore(STORE_NAME2).getAllKeys(IDBKeyRange.bound(prefix, prefix + "\uFFFF"));
+    req.onsuccess = () => resolve(req.result.map(String));
+    req.onerror = () => reject(req.error);
+  });
+}
 function setGDriveTokenProvider(provider) {
   tokenProvider = provider;
 }
@@ -2830,6 +2839,20 @@ function parseJsonc(text) {
   stripped = stripped.replace(/,(\s*[}\]])/g, "$1");
   return JSON.parse(stripped);
 }
+async function stashPrevIdb(filename, aboutToBecome) {
+  try {
+    const old = await idbRead(filename);
+    if (!old || old === aboutToBecome)
+      return;
+    const stamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-").slice(0, 19);
+    await idbWrite(`prev:${filename}:${stamp}`, old);
+    const keys = await idbListKeys(`prev:${filename}:`);
+    for (const k of keys.sort().slice(0, Math.max(0, keys.length - PREV_KEEP))) {
+      await idbDelete(k);
+    }
+  } catch {
+  }
+}
 function refreshSharedFile(filename, cached) {
   const now = Date.now();
   if (now - (__lastSharedRefresh.get(filename) || 0) < SHARED_REFRESH_MS)
@@ -2922,6 +2945,7 @@ async function loadPreferences() {
 }
 async function savePreferences(prefs) {
   const content = JSON.stringify(prefs, null, 2);
+  await stashPrevIdb("preferences.jsonc", content);
   await idbWrite("preferences.jsonc", content);
   await gDriveWrite("preferences.jsonc", content);
 }
@@ -2986,6 +3010,7 @@ async function updateAllowlist(mutate) {
 }
 async function saveAllowlist(list) {
   const content = JSON.stringify(list, null, 2);
+  await stashPrevIdb("allowlist.jsonc", content);
   await idbWrite("allowlist.jsonc", content);
   try {
     const cloudRaw = await gDriveRead("allowlist.jsonc");
@@ -3068,7 +3093,7 @@ async function loadDeviceState() {
   }
   return {};
 }
-var IDB_NAME2, IDB_VERSION, STORE_NAME2, gDriveFolderId, gDriveFolderName, gDriveFolderPath, gDriveFolderOwner, tokenProvider, PROVIDERS, DEFAULT_PREFERENCES, DEFAULT_ALLOWLIST, SHARED_REFRESH_MS, __lastSharedRefresh, ALLOWLIST_KEYS, DEVICE_ID_KEY;
+var IDB_NAME2, IDB_VERSION, STORE_NAME2, gDriveFolderId, gDriveFolderName, gDriveFolderPath, gDriveFolderOwner, tokenProvider, PROVIDERS, DEFAULT_PREFERENCES, DEFAULT_ALLOWLIST, PREV_KEEP, SHARED_REFRESH_MS, __lastSharedRefresh, ALLOWLIST_KEYS, DEVICE_ID_KEY;
 var init_web_settings = __esm({
   "packages/mailx-store-web/web-settings.js"() {
     "use strict";
@@ -3143,6 +3168,7 @@ var init_web_settings = __esm({
       flaggedSenders: [],
       flaggedDomains: []
     };
+    PREV_KEEP = 5;
     SHARED_REFRESH_MS = 5 * 6e4;
     __lastSharedRefresh = /* @__PURE__ */ new Map();
     ALLOWLIST_KEYS = ["senders", "domains", "recipients", "flaggedSenders", "flaggedDomains"];