@bobfrankston/mailx 1.0.234 → 1.0.235

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx",
-  "version": "1.0.234",
+  "version": "1.0.235",
   "description": "Local-first email client with IMAP sync and standalone native app",
   "type": "module",
   "main": "bin/mailx.js",
@@ -24,7 +24,7 @@
     "@bobfrankston/iflow-node": "^0.1.2",
     "@bobfrankston/miscinfo": "^1.0.8",
     "@bobfrankston/oauthsupport": "^1.0.22",
-    "@bobfrankston/msger": "^0.1.296",
+    "@bobfrankston/msger": "^0.1.297",
     "@capacitor/android": "^8.3.0",
     "@capacitor/cli": "^8.3.0",
     "@capacitor/core": "^8.3.0",
@@ -78,7 +78,7 @@
       "@bobfrankston/iflow-node": "^0.1.2",
       "@bobfrankston/miscinfo": "^1.0.8",
       "@bobfrankston/oauthsupport": "^1.0.22",
-      "@bobfrankston/msger": "^0.1.296",
+      "@bobfrankston/msger": "^0.1.297",
       "@capacitor/android": "^8.3.0",
       "@capacitor/cli": "^8.3.0",
       "@capacitor/core": "^8.3.0",

package/packages/mailx-imap/index.js

@@ -1009,9 +1009,15 @@ export class ImapManager extends EventEmitter {
         // the previous high. upsertMessage's primary-key dedup handles it.
         void highestUid;
         let stored = 0;
-        this.db.beginTransaction();
-        try {
-            for (const msg of msgs) {
+        let errors = 0;
+        // Don't wrap the whole batch in one transaction: a single bad row
+        // would roll back the entire batch. E.g. a message with a malformed
+        // Date header gave `new Date(rawStr).getTime() === NaN`, SQLite
+        // coerced that to NULL, the NOT NULL constraint failed, and the
+        // whole Gmail sync lost 200 messages per tick. Now each row runs
+        // standalone — bad rows are logged and skipped.
+        for (const msg of msgs) {
+            try {
                 const flags = [];
                 if (msg.seen)
                     flags.push("\\Seen");
@@ -1021,12 +1027,20 @@ export class ImapManager extends EventEmitter {
                     flags.push("\\Answered");
                 if (msg.draft)
                     flags.push("\\Draft");
+                // Sanitize date: reject NaN (from malformed RFC 822 Date headers)
+                // and fall back to "now" so the message still lands in the DB.
+                let dateMs = Date.now();
+                if (msg.date instanceof Date) {
+                    const t = msg.date.getTime();
+                    if (Number.isFinite(t))
+                        dateMs = t;
+                }
                 this.db.upsertMessage({
                     accountId, folderId, uid: msg.uid,
                     messageId: msg.messageId || "",
                     inReplyTo: msg.inReplyTo || "",
                     references: msg.references || [],
-                    date: msg.date instanceof Date ? msg.date.getTime() : Date.now(),
+                    date: dateMs,
                     subject: msg.subject || "",
                     from: toEmailAddress(msg.from?.[0] || {}),
                     to: toEmailAddresses(msg.to || []),
@@ -1035,12 +1049,15 @@ export class ImapManager extends EventEmitter {
                 });
                 stored++;
             }
-            this.db.commitTransaction();
-        }
-        catch (e) {
-            this.db.rollbackTransaction();
-            console.error(`  [api] storeApiMessages error: ${e.message}`);
+            catch (e) {
+                errors++;
+                if (errors <= 3) {
+                    console.error(`  [api] upsert ${accountId}/${folderId}/${msg.uid} (${msg.messageId}): ${e.message}`);
+                }
+            }
         }
+        if (errors > 0)
+            console.error(`  [api] storeApiMessages: ${errors} of ${msgs.length} rows failed (${stored} stored)`);
         return stored;
     }
     /** Kill and recreate the persistent ops connection */