@bobfrankston/mailx-store 0.1.42 → 0.1.43

package/db.d.ts

@@ -371,7 +371,7 @@ export declare class MailxDB {
      *  reply arrives. Idempotent — UPDATE on an already-1 row is a no-op. */
     markRepliedByMessageId(accountId: string, messageId: string): boolean;
     /** Get messages without cached bodies (for background prefetch) */
-    getMessagesWithoutBody(accountId: string, limit?: number): {
+    getMessagesWithoutBody(accountId: string, limit?: number, excludeFolderIds?: number[]): {
         uid: number;
         folderId: number;
     }[];

package/db.js

@@ -1998,14 +1998,24 @@ export class MailxDB {
         return r.changes > 0;
     }
     /** Get messages without cached bodies (for background prefetch) */
-    getMessagesWithoutBody(accountId, limit = 50) {
+    getMessagesWithoutBody(accountId, limit = 50, excludeFolderIds = []) {
         // Prefetch order: smallest first, NULLs last, recent within tiebreak.
         // Reasoning: on slow / metered Android networks, the user feels the
         // cache "fill up" much faster when the queue is dominated by short
         // notification mails (a handful of KB each) instead of a single
         // multi-megabyte attachment that monopolizes bandwidth for minutes.
         // Size 0 / NULL fall to the end so they don't masquerade as small.
-        return this.db.prepare("SELECT uid, folder_id as folderId FROM messages WHERE account_id = ? AND (body_path IS NULL OR body_path = '') ORDER BY (size IS NULL OR size = 0), size ASC, date DESC LIMIT ?").all(accountId, limit);
+        //
+        // excludeFolderIds: folders the caller has put in error-cooldown. We
+        // exclude them HERE (at the query) rather than after slicing, because a
+        // single bloated/failing folder (e.g. a server-side Outbox stuffed with
+        // hundreds of stuck messages) would otherwise fill the entire size-asc
+        // result and starve every healthy folder of its prefetch turn — the
+        // "so much unfetched, nothing draining" symptom (Bob 2026-06-01).
+        const exclusion = excludeFolderIds.length
+            ? ` AND folder_id NOT IN (${excludeFolderIds.map(() => "?").join(",")})`
+            : "";
+        return this.db.prepare(`SELECT uid, folder_id as folderId FROM messages WHERE account_id = ? AND (body_path IS NULL OR body_path = '')${exclusion} ORDER BY (size IS NULL OR size = 0), size ASC, date DESC LIMIT ?`).all(accountId, ...excludeFolderIds, limit);
     }
     /** Highest server UID we've seen in this folder. After the
      *  message_folders refactor, this reads from the membership table

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-store",
-  "version": "0.1.42",
+  "version": "0.1.43",
   "type": "module",
   "main": "index.js",
   "types": "index.d.ts",
@@ -10,7 +10,7 @@
   "license": "ISC",
   "dependencies": {
     "@bobfrankston/mailx-types": "^0.1.18",
-    "@bobfrankston/mailx-settings": "^0.1.25",
+    "@bobfrankston/mailx-settings": "^0.1.26",
     "@bobfrankston/mailx-bus": "^0.1.2",
     "mailparser": "^3.7.2"
   },
@@ -30,7 +30,7 @@
   ".transformedSnapshot": {
     "dependencies": {
       "@bobfrankston/mailx-types": "^0.1.18",
-      "@bobfrankston/mailx-settings": "^0.1.25",
+      "@bobfrankston/mailx-settings": "^0.1.26",
       "@bobfrankston/mailx-bus": "^0.1.2",
       "mailparser": "^3.7.2"
     }