@bobfrankston/mailx-imap 0.1.48 → 0.1.50

package/index.js

@@ -81,6 +81,14 @@ function toEmailAddress(addr) {
         address: addr?.address || ""
     };
 }
+/** Coerce a message date to a finite epoch-ms for the `messages.date NOT NULL`
+ *  column. An unparseable envelope date yields an *Invalid Date* — truthy and
+ *  `instanceof Date`, but `.getTime()` is NaN, which binds as NULL and fails
+ *  the constraint, aborting the whole sync batch. Default non-finite to now. */
+function toFiniteDateMs(d) {
+    const ms = d instanceof Date ? d.getTime() : typeof d === "number" ? d : NaN;
+    return Number.isFinite(ms) ? ms : Date.now();
+}
 /** Convert array of iflow addresses */
 function toEmailAddresses(addrs) {
     if (!addrs)
@@ -963,7 +971,7 @@ export class ImapManager extends EventEmitter {
                     messageId: msg.messageId || "",
                     inReplyTo: msg.inReplyTo || "",
                     references: [],
-                    date: msg.date instanceof Date ? msg.date.getTime() : (typeof msg.date === "number" ? msg.date : Date.now()),
+                    date: toFiniteDateMs(msg.date),
                     subject: msg.subject || "",
                     from: toEmailAddress(msg.from?.[0] || {}),
                     to: toEmailAddresses(msg.to || []),
@@ -1475,7 +1483,7 @@ export class ImapManager extends EventEmitter {
                         messageId: msg.messageId || "",
                         inReplyTo: msg.inReplyTo || "",
                         references: [],
-                        date: msg.date instanceof Date ? msg.date.getTime() : (typeof msg.date === "number" ? msg.date : Date.now()),
+                        date: toFiniteDateMs(msg.date),
                         subject: msg.subject || "",
                         from: toEmailAddress(msg.from?.[0] || {}),
                         to: toEmailAddresses(msg.to || []),
@@ -4301,16 +4309,20 @@ export class ImapManager extends EventEmitter {
                 const res = await fetch(url, {
                     headers: { Authorization: `Bearer ${token}` },
                 });
-                if (res.status === 410) {
-                    // Sync token expired (Google retains tokens for ~7 days).
-                    // Drop the stored token and recurse for a full sync — the
-                    // in-flight guard is on syncGoogleContacts (the public
-                    // wrapper), not Impl, so recursion is safe.
-                    this.db.setKv("contacts", accountId, null);
-                    return this.syncGoogleContactsImpl(accountId);
-                }
                 if (!res.ok) {
                     const err = await res.text();
+                    // Expired sync token. The People API returns HTTP 400
+                    // FAILED_PRECONDITION with reason EXPIRED_SYNC_TOKEN
+                    // (older docs / older behavior said 410 — handle both
+                    // by inspecting the body). The documented recovery is:
+                    // drop the token and re-call WITHOUT it (full sync).
+                    // The `syncToken` guard prevents infinite recursion if a
+                    // tokenless full sync itself somehow returns this.
+                    if (syncToken && /EXPIRED_SYNC_TOKEN|sync token is expired/i.test(err)) {
+                        console.log(`  [contacts] ${accountId}: sync token expired — clearing for full resync`);
+                        this.db.setKv("contacts", accountId, null);
+                        return this.syncGoogleContactsImpl(accountId);
+                    }
                     console.error(`  [contacts] API error for ${accountId}: ${res.status} ${err}`);
                     return changed;
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.48",
+  "version": "0.1.50",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -11,7 +11,7 @@
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.14",
     "@bobfrankston/mailx-settings": "^0.1.17",
-    "@bobfrankston/mailx-store": "^0.1.27",
+    "@bobfrankston/mailx-store": "^0.1.28",
     "@bobfrankston/iflow-direct": "^0.1.50",
     "@bobfrankston/tcp-transport": "^0.1.6",
     "@bobfrankston/smtp-direct": "^0.1.8",
@@ -39,7 +39,7 @@
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.14",
       "@bobfrankston/mailx-settings": "^0.1.17",
-      "@bobfrankston/mailx-store": "^0.1.27",
+      "@bobfrankston/mailx-store": "^0.1.28",
       "@bobfrankston/iflow-direct": "^0.1.50",
       "@bobfrankston/tcp-transport": "^0.1.6",
       "@bobfrankston/smtp-direct": "^0.1.8",