@bobfrankston/mailx-sync 0.1.9 → 0.1.11

package/gmail.d.ts

@@ -35,8 +35,9 @@ export declare class GmailApiProvider implements MailProvider {
     fetchByUids(folder: string, uids: number[], options?: FetchOptions): Promise<ProviderMessage[]>;
     /** Bulk-fetch raw bodies for many UIDs in one "folder" (Gmail label).
      *  Lists the label once, builds UID→ID map, then streams bodies through
-     *  `onBody` with bounded concurrency (lets Gmail's HTTP/2 stream multiplex;
-     *  `fetch()`'s built-in 429/5xx retry handles backoff automatically).
+     *  `onBody`. Uses Gmail's HTTP batch endpoint (up to 100 sub-requests per
+     *  round-trip) when available, with a single-request fallback so a batch
+     *  protocol blip doesn't starve prefetch entirely.
      *
      *  NOTE: Gmail's model is labels, not folders — a single message can be in
      *  many labels. Treating each label as a folder causes duplicate fetches
@@ -44,6 +45,13 @@ export declare class GmailApiProvider implements MailProvider {
      *  model"). For now we mirror the IMAP folder grouping, accepting duplicate
      *  fetches of multi-labeled messages. */
     fetchBodiesBatch(folder: string, uids: number[], onBody: (uid: number, source: string) => void): Promise<void>;
+    /** POST /batch/gmail/v1 with up to 100 sub-requests. See
+     *  https://developers.google.com/gmail/api/guides/batch
+     *  for the multipart/mixed wire format. */
+    private batchFetchBodies;
+    /** Fallback path when batch fails — original bounded-concurrency loop.
+     *  Kept on the degraded path so a single bad batch doesn't halt prefetch. */
+    private fetchBodiesIndividually;
     fetchOne(folder: string, uid: number, options?: FetchOptions): Promise<ProviderMessage | null>;
     /** Apply the absolute flag state to a message.
      *  Gmail model: flags are labels. `\Seen` is the *absence* of UNREAD;

package/gmail.js

@@ -196,7 +196,17 @@ export class GmailApiProvider {
                 name,
                 delimiter: "/",
                 specialUse,
+<<<<<<< HEAD
                 flags: label.type === "system" ? ["\\Noselect"] : [],
+=======
+                // No \\Noselect on Gmail labels: the unwanted system labels
+                // (UNREAD/STARRED/IMPORTANT/CATEGORY_*/CHAT) are filtered
+                // above, leaving INBOX/SENT/DRAFT/TRASH/SPAM and user labels —
+                // ALL of which are selectable. The previous \\Noselect tag
+                // on system labels caused mailx-imap to skip them entirely,
+                // so INBOX never appeared in the folder tree.
+                flags: [],
+>>>>>>> 4b735cd (Initial commit)
             });
         }
         return folders;
@@ -337,8 +347,9 @@ export class GmailApiProvider {
     }
     /** Bulk-fetch raw bodies for many UIDs in one "folder" (Gmail label).
      *  Lists the label once, builds UID→ID map, then streams bodies through
-     *  `onBody` with bounded concurrency (lets Gmail's HTTP/2 stream multiplex;
-     *  `fetch()`'s built-in 429/5xx retry handles backoff automatically).
+     *  `onBody`. Uses Gmail's HTTP batch endpoint (up to 100 sub-requests per
+     *  round-trip) when available, with a single-request fallback so a batch
+     *  protocol blip doesn't starve prefetch entirely.
      *
      *  NOTE: Gmail's model is labels, not folders — a single message can be in
      *  many labels. Treating each label as a folder causes duplicate fetches
@@ -361,17 +372,124 @@ export class GmailApiProvider {
         }
         if (wanted.length === 0)
             return;
-        // Bounded concurrency — 2 in-flight, combined with the shared token
-        // bucket in `fetch()` (8 req/sec steady) keeps us comfortably below
-        // Gmail's per-user-per-second cap. Earlier 5-wide prefetch kept Google
-        // in sustained-abuse territory for hours, which triggered cooldowns
-        // that rejected even 1-unit calls like labels.list on fresh startup.
+        // Item 16 / C24 Gmail half: HTTP /batch prefetch. One multipart POST
+        // with up to 100 sub-requests replaces N round-trips. Typical ×10-20
+        // speedup on large folders. Each batch still costs the same tokens
+        // from the rate bucket as the individual GETs would have (no free
+        // ride), but wall-clock latency collapses to roughly one round-trip
+        // per 100 messages instead of per message.
+        const BATCH_SIZE = 100;
+        for (let i = 0; i < wanted.length; i += BATCH_SIZE) {
+            const slice = wanted.slice(i, i + BATCH_SIZE);
+            try {
+                await this.batchFetchBodies(slice, onBody);
+            }
+            catch (e) {
+                // Batch failed (malformed multipart, network reset, auth
+                // mid-flight). Fall back to a bounded-concurrency per-message
+                // loop for this slice so the user still gets bodies — we'd
+                // rather ship the feature degraded than lose prefetch.
+                console.error(`  [gmail batch] fell back to per-message: ${e.message}`);
+                await this.fetchBodiesIndividually(slice, onBody);
+            }
+        }
+    }
+    /** POST /batch/gmail/v1 with up to 100 sub-requests. See
+     *  https://developers.google.com/gmail/api/guides/batch
+     *  for the multipart/mixed wire format. */
+    async batchFetchBodies(items, onBody) {
+        if (items.length === 0)
+            return;
+        // One token covers the whole batch endpoint round-trip; Google's
+        // per-user quota charges each inner request individually, but the
+        // per-second request limit counts the batch as a single request —
+        // so rate-bucket budget matches wall-clock cost.
+        await this.acquireToken();
+        const token = await this.tokenProvider();
+        const boundary = `batch_mailx_${Date.now()}_${Math.floor(Math.random() * 1e9)}`;
+        const parts = [];
+        for (const { id } of items) {
+            parts.push(`--${boundary}\r\n` +
+                `Content-Type: application/http\r\n` +
+                `Content-ID: <${id}>\r\n` +
+                `\r\n` +
+                `GET /gmail/v1/users/me/messages/${id}?format=raw\r\n` +
+                `Accept: application/json\r\n` +
+                `\r\n`);
+        }
+        parts.push(`--${boundary}--\r\n`);
+        const body = parts.join("");
+        const res = await globalThis.fetch("https://gmail.googleapis.com/batch/gmail/v1", {
+            method: "POST",
+            headers: {
+                "Authorization": `Bearer ${token}`,
+                "Content-Type": `multipart/mixed; boundary=${boundary}`,
+            },
+            body,
+        });
+        if (!res.ok) {
+            const text = await res.text().catch(() => "");
+            throw new Error(`batch HTTP ${res.status}: ${text.slice(0, 200)}`);
+        }
+        const respBody = await res.text();
+        // Parse multipart — extract each inner application/http response,
+        // match back to the Content-ID to recover the Gmail message id,
+        // then decode the raw body JSON field.
+        const respCt = res.headers.get("content-type") || "";
+        const bMatch = respCt.match(/boundary=([^;]+)/i);
+        if (!bMatch)
+            throw new Error("batch response missing boundary");
+        const respBoundary = bMatch[1].replace(/^"|"$/g, "");
+        // Google sometimes replies with its own "batch_..." boundary even if
+        // we sent a different one; parsing by the response's boundary is the
+        // reliable path. Split on boundary delimiters.
+        const chunks = respBody.split(`--${respBoundary}`);
+        for (const chunk of chunks) {
+            const trimmed = chunk.trim();
+            if (!trimmed || trimmed === "--")
+                continue;
+            // Each chunk is: outer headers, blank line, inner response
+            // (HTTP/1.1 status line, headers, blank line, JSON body).
+            // Locate the inner JSON body by finding the double blank after
+            // the inner headers section.
+            const idMatch = chunk.match(/Content-ID:\s*<?response-<?([^>\s]+)>?/i);
+            const gmailId = idMatch ? idMatch[1].replace(/^<|>$/g, "") : "";
+            // Skip the outer headers: first blank line ends them.
+            const firstBlank = chunk.indexOf("\r\n\r\n");
+            if (firstBlank < 0)
+                continue;
+            const inner = chunk.slice(firstBlank + 4);
+            // Skip the inner HTTP status + headers: second blank line ends them.
+            const secondBlank = inner.indexOf("\r\n\r\n");
+            if (secondBlank < 0)
+                continue;
+            const jsonBody = inner.slice(secondBlank + 4).trim();
+            if (!jsonBody || jsonBody.startsWith("--"))
+                continue;
+            try {
+                const parsed = JSON.parse(jsonBody);
+                if (!parsed?.raw)
+                    continue;
+                const actualId = parsed.id || gmailId;
+                if (!actualId)
+                    continue;
+                const uid = idToUid(actualId);
+                const base64 = parsed.raw.replace(/-/g, "+").replace(/_/g, "/");
+                const source = new TextDecoder().decode(Uint8Array.from(atob(base64), c => c.charCodeAt(0)));
+                onBody(uid, source);
+            }
+            catch { /* malformed sub-response — skip, don't poison the batch */ }
+        }
+    }
+    /** Fallback path when batch fails — original bounded-concurrency loop.
+     *  Kept on the degraded path so a single bad batch doesn't halt prefetch. */
+    async fetchBodiesIndividually(items, onBody) {
         const CONCURRENCY = 2;
         let cursor = 0;
         const worker = async () => {
-            while (cursor < wanted.length) {
+            while (cursor < items.length) {
                 const idx = cursor++;
-                const { uid, id } = wanted[idx];
+                const { uid, id } = items[idx];
                 try {
                     const msg = await this.fetch(`/messages/${id}?format=raw`);
                     if (!msg?.raw)
@@ -381,12 +499,11 @@ export class GmailApiProvider {
                     onBody(uid, source);
                 }
                 catch (e) {
-                    // Per-message failure is non-fatal; keep worker alive for the rest.
                     console.error(`  [gmail batch] UID ${uid}: ${e.message}`);
                 }
             }
         };
-        await Promise.all(Array.from({ length: Math.min(CONCURRENCY, wanted.length) }, () => worker()));
+        await Promise.all(Array.from({ length: Math.min(CONCURRENCY, items.length) }, () => worker()));
     }
     async fetchOne(folder, uid, options = {}) {
         // Caller (mailx-imap) passes providerId straight from the DB row when

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/mailx-sync",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "description": "Platform-agnostic mail provider implementations + sync orchestration. Single source of truth for Gmail/IMAP/Outlook protocol code, consumed by both desktop (Node) and Android (WebView) — eliminates the parallel mailx-imap/mailx-store-web Gmail providers that drifted in practice.",
   "main": "index.js",
   "types": "index.ts",
@@ -19,8 +19,8 @@
   "author": "Bob Frankston",
   "license": "ISC",
   "dependencies": {
-    "@bobfrankston/iflow-direct": "^0.1.26",
-    "@bobfrankston/tcp-transport": "^0.1.4"
+    "@bobfrankston/iflow-direct": "file:../iflow-direct",
+    "@bobfrankston/tcp-transport": "file:../tcp-transport"
   },
   "exports": {
     ".": {
@@ -37,15 +37,5 @@
   },
   "devDependencies": {
     "@types/node": "^25.6.0"
-  },
-  ".dependencies": {
-    "@bobfrankston/iflow-direct": "file:../iflow-direct",
-    "@bobfrankston/tcp-transport": "file:../tcp-transport"
-  },
-  ".transformedSnapshot": {
-    "dependencies": {
-      "@bobfrankston/iflow-direct": "^0.1.26",
-      "@bobfrankston/tcp-transport": "^0.1.4"
-    }
   }
 }