@bobfrankston/mailx 1.0.128 → 1.0.129

package/client/app.js

@@ -116,6 +116,19 @@ alertDismiss?.addEventListener("click", hideAlert);
 // ── Wire up components ──
 const folderTree = document.getElementById("folder-tree");
 let currentFolderSpecialUse = "";
+function clearViewer() {
+    const body = document.getElementById("mv-body");
+    const header = document.getElementById("mv-header");
+    const att = document.getElementById("mv-attachments");
+    if (body)
+        body.innerHTML = "";
+    if (header)
+        header.innerHTML = "";
+    if (att) {
+        att.innerHTML = "";
+        att.hidden = true;
+    }
+}
 initFolderTree(folderTree, (accountId, folderId, folderName, specialUse) => {
     currentFolderSpecialUse = specialUse;
     currentAccountId = accountId;
@@ -123,11 +136,13 @@ initFolderTree(folderTree, (accountId, folderId, folderName, specialUse) => {
     if (searchInput)
         searchInput.value = "";
     markAsSeen();
+    clearViewer();
     loadMessages(accountId, folderId, 1, specialUse);
     setTitle(`mailx - ${folderName}`);
 }, () => {
     // Unified inbox handler
     currentFolderSpecialUse = "inbox";
+    clearViewer();
     loadUnifiedInbox();
     setTitle("mailx - All Inboxes");
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.128",
+  "version": "1.0.129",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",

package/packages/mailx-imap/index.js

@@ -315,29 +315,30 @@ export class ImapManager extends EventEmitter {
         let messages;
         const firstSync = highestUid === 0;
         const historyDays = getHistoryDays(accountId);
-        const startDate = historyDays > 0
-            ? new Date(Date.now() - historyDays * 86400000)
+        // For first sync with unlimited history, start with 30 days — backfill extends later
+        const effectiveDays = (historyDays === 0 && firstSync) ? 30 : historyDays;
+        const startDate = effectiveDays > 0
+            ? new Date(Date.now() - effectiveDays * 86400000)
             : new Date(0);
         if (highestUid > 0) {
             // Incremental: fetch new messages — metadata only for speed, bodies on demand
             const fetched = await client.fetchMessagesSinceUid(folder.path, highestUid, { source: false });
             // Filter out the last known message (IMAP * always returns at least one)
             messages = fetched.filter(m => m.uid > highestUid);
-            // Gap detection: compare DB count vs IMAP count to catch interrupted syncs
-            const dbCount = this.db.getMessageCount(accountId, folderId);
-            try {
-                const imapCount = await client.getMessagesCount(folder.path);
-                const gap = imapCount - dbCount - messages.length;
-                if (gap > 0) {
-                    console.log(`  ${folder.path}: gap detected — IMAP has ${imapCount}, DB has ${dbCount}, ${messages.length} new → ${gap} missing`);
-                    // Full UID reconciliation: get all IMAP UIDs, find missing ones
-                    const existingUids = new Set(this.db.getUidsForFolder(accountId, folderId));
-                    const newUids = new Set(messages.map(m => m.uid));
-                    const allImapUids = await client.getUids(folder.path);
-                    const missingUids = allImapUids.filter(uid => !existingUids.has(uid) && !newUids.has(uid));
-                    if (missingUids.length > 0) {
-                        console.log(`  ${folder.path}: reconciling ${missingUids.length} missing messages`);
-                        // Fetch in chunks to avoid huge commands
+            // Gap detection: check for missing UIDs within the range we've already synced
+            // Only reconcile between our lowest and highest UID — don't try to fetch the entire folder history
+            const existingUids = this.db.getUidsForFolder(accountId, folderId);
+            if (existingUids.length > 0) {
+                try {
+                    const lowestUid = Math.min(...existingUids);
+                    // Fetch UIDs in our known range from IMAP
+                    const rangeUids = await client.getUids(folder.path);
+                    const rangeInScope = rangeUids.filter((uid) => uid >= lowestUid && uid <= highestUid);
+                    const existingSet = new Set(existingUids);
+                    const newSet = new Set(messages.map(m => m.uid));
+                    const missingUids = rangeInScope.filter((uid) => !existingSet.has(uid) && !newSet.has(uid));
+                    if (missingUids.length > 0 && missingUids.length <= 5000) {
+                        console.log(`  ${folder.path}: gap detected — ${missingUids.length} missing UIDs in range ${lowestUid}..${highestUid}`);
                         const chunkSize = 500;
                         for (let i = 0; i < missingUids.length; i += chunkSize) {
                             const chunk = missingUids.slice(i, i + chunkSize);
@@ -346,10 +347,13 @@ export class ImapManager extends EventEmitter {
                             messages.push(...recovered);
                         }
                     }
+                    else if (missingUids.length > 5000) {
+                        console.log(`  ${folder.path}: ${missingUids.length} missing UIDs — too many, skipping reconciliation (delete DB to force full re-sync)`);
+                    }
+                }
+                catch (e) {
+                    console.error(`  ${folder.path}: gap detection failed: ${e.message}`);
                 }
-            }
-            catch (e) {
-                console.error(`  ${folder.path}: gap detection failed: ${e.message}`);
             }
             // Backfill: if historyDays extends further back than our oldest message, fetch the gap
             const oldestDate = this.db.getOldestDate(accountId, folderId);