@bobfrankston/rmfmail 1.1.185 → 1.1.187

package/packages/mailx-imap/index.ts

@@ -1082,6 +1082,18 @@ export class ImapManager extends EventEmitter {
         source: string,
         flags: string[],
     ): Promise<void> {
+        // Guard against uid <= 0. Valid IMAP UIDs are always ≥ 1; a 0 here
+        // means the APPENDUID capture failed (server returned no UIDPLUS, or
+        // appendMessage resolved to 0 instead of null) and the caller didn't
+        // catch it. Inserting uid=0 created a phantom INBOX row with no
+        // message_id, no body, no subject — the "blank line in the summary"
+        // (Bob 2026-05-28). Refuse it; the caller's syncFolder fallback will
+        // pick the message up with its real UID on the next sync.
+        if (!uid || uid <= 0) {
+            console.error(`  [insert] refusing uid=${uid} for ${accountId}/${folder.path} — invalid IMAP UID (APPENDUID likely failed); deferring to sync`);
+            this.syncFolder(accountId, folder.id).catch(() => {});
+            return;
+        }
         // insertLocalRowFromSource runs right after sendMessage — that's a
         // user-initiated path but the parse cost is on the post-send
         // background work, not the click-through. Tag as background so a

package/packages/mailx-imap/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.69",
+  "version": "0.1.70",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bobfrankston/mailx-imap",
-      "version": "0.1.69",
+      "version": "0.1.70",
       "license": "ISC",
       "dependencies": {
         "@bobfrankston/iflow-direct": "^0.1.27",

package/packages/mailx-imap/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-imap",
-  "version": "0.1.69",
+  "version": "0.1.70",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",