@agenticmail/api 0.9.4 → 0.9.6

package/dist/index.js

@@ -2714,30 +2714,75 @@ function createMailRoutes(accountManager2, config, db, gatewayManager) {
       const folder = req.query.folder || "INBOX";
       const password = getAgentPassword(agent);
       const receiver = await getReceiver(agent.stalwartPrincipal, password, config);
-      const mailboxInfo = await receiver.getMailboxInfo(folder);
-      const envelopes = await receiver.listEnvelopes(folder, { limit, offset });
-      const uids = envelopes.map((e) => e.uid);
-      const rawMap = uids.length > 0 ? await receiver.batchFetch(uids, folder) : /* @__PURE__ */ new Map();
-      const messages = [];
-      for (const env of envelopes) {
-        let preview = "";
+      const PREVIEW_MAX_BYTES = 8192;
+      const client = receiver.getImapClient();
+      const lock = await client.getMailboxLock(folder);
+      const envelopes = [];
+      const rawMap = /* @__PURE__ */ new Map();
+      let total = 0;
+      try {
+        const searchResult = await client.search({ all: true }, { uid: true });
+        const allUids = Array.isArray(searchResult) ? searchResult : [];
+        total = allUids.length;
+        const sorted = allUids.slice().sort((a, b) => b - a);
+        const pageUids = sorted.slice(offset, offset + limit);
+        if (pageUids.length > 0) {
+          for await (const msg of client.fetch(pageUids.join(","), {
+            uid: true,
+            envelope: true,
+            flags: true,
+            size: true
+          })) {
+            const env = msg.envelope;
+            if (!env) continue;
+            envelopes.push({
+              uid: msg.uid,
+              subject: env.subject ?? "",
+              from: (env.from ?? []).map((a) => ({ name: a.name, address: a.address ?? "" })),
+              to: (env.to ?? []).map((a) => ({ name: a.name, address: a.address ?? "" })),
+              date: env.date ?? /* @__PURE__ */ new Date(),
+              flags: msg.flags ? [...msg.flags] : [],
+              size: msg.size ?? 0
+            });
+          }
+          envelopes.sort((a, b) => b.uid - a.uid);
+          for await (const msg of client.fetch(pageUids.join(","), {
+            uid: true,
+            source: { start: 0, maxLength: PREVIEW_MAX_BYTES }
+          })) {
+            if (msg.source) {
+              rawMap.set(
+                msg.uid,
+                Buffer.isBuffer(msg.source) ? msg.source : Buffer.from(msg.source)
+              );
+            }
+          }
+        }
+      } finally {
+        lock.release();
+      }
+      const messages = await Promise.all(envelopes.map(async (env) => {
         const raw = rawMap.get(env.uid);
+        let preview = "";
         if (raw) {
-          const parsed = await parseEmail2(raw);
-          preview = (parsed.text || "").slice(0, previewLen);
+          try {
+            const parsed = await parseEmail2(raw);
+            preview = (parsed.text || "").slice(0, previewLen);
+          } catch {
+          }
         }
-        messages.push({
+        return {
           uid: env.uid,
           subject: env.subject,
           from: env.from,
           to: env.to,
           date: env.date,
-          flags: [...env.flags],
+          flags: env.flags,
           size: env.size,
           preview
-        });
-      }
-      res.json({ messages, count: messages.length, total: mailboxInfo.exists });
+        };
+      }));
+      res.json({ messages, count: messages.length, total });
     } catch (err) {
       next(err);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agenticmail/api",
-  "version": "0.9.4",
+  "version": "0.9.6",
   "description": "REST API server for AgenticMail — email and SMS endpoints for AI agents",
   "type": "module",
   "main": "dist/index.js",
@@ -36,7 +36,7 @@
     "uuid": "^11.1.0"
   },
   "optionalDependencies": {
-    "@agenticmail/claudecode": "^0.2.2"
+    "@agenticmail/claudecode": "^0.2.4"
   },
   "devDependencies": {
     "@types/cors": "^2.8.17",

package/public/js/app.js

@@ -95,6 +95,19 @@ async function bootstrap() {
     const initial = (lastId && state.agents.find(a => a.id === lastId))
       ?? state.agents.find(isBridgeAgent)
       ?? state.agents[0];
+    // Seed the URL hash BEFORE selectAgent so selectAgent's loadList
+    // call lands on the right folder. We use history.replaceState
+    // (NOT `location.hash = ...`) so this does NOT fire a hashchange
+    // event — that would trigger a second route() → loadList() in
+    // parallel with selectAgent's, doubling the work on every
+    // bootstrap. Read the hash first so a deep-link refresh
+    // (e.g. /#/folder/sent) still wins.
+    const folderMatch = location.hash.match(/^#\/folder\/([a-z]+)$/);
+    if (folderMatch) {
+      state.selectedFolder = folderMatch[1];
+    } else if (!location.hash) {
+      history.replaceState(null, '', `${location.pathname}${location.search}#/folder/inbox`);
+    }
     if (initial) await selectAgent(initial);
     renderProfile();
     populateComposeFrom();
@@ -105,10 +118,14 @@ async function bootstrap() {
     // rendering. Idempotent — safe to call after bootstrap reruns.
     subscribeToActivity();
     maybeRequestNotificationPermission();
-    // Initial route: if the URL already has a hash (e.g. a refresh
-    // on /#/folder/sent), respect it; otherwise default to inbox.
-    if (!location.hash) location.hash = '#/folder/inbox';
-    else route();
+    // If the URL points at a message (not a folder), open it now —
+    // the folder list selectAgent already loaded stays in the
+    // background. Folder hashes need no extra work; selectAgent's
+    // loadList already handled them above.
+    const hash = location.hash;
+    const msgMatch = hash.match(/^#\/m\/(\d+)$/);
+    const draftMatch = hash.match(/^#\/d\/([a-zA-Z0-9-]+)$/);
+    if (msgMatch || draftMatch) route();
   } catch (err) {
     toast(`Failed to load agents: ${err.message}`, true);
   }
@@ -176,14 +193,18 @@ function route() {
   }
   const folderMatch = hash.match(/^#\/folder\/([a-z]+)$/);
   const folder = folderMatch ? folderMatch[1] : 'inbox';
-  if (state.selectedFolder !== folder) {
-    state.selectedFolder = folder;
-    // Reset pagination on every folder switch — a fresh folder
-    // starts at page 1. Preserved across silent SSE refreshes so
-    // a new arrival doesn't yank the user back from page 3.
-    state.pagination = { offset: 0, limit: 50, total: 0 };
-    renderSidebar(onFolderSelect);
-  }
+  // Only do work if the folder actually changed. Re-firing loadList
+  // for the SAME folder on every hashchange (e.g. closing a message
+  // detail back to the list) makes the UI feel sluggish — the list
+  // is already rendered, and a second digest fetch just churns
+  // through 50 messages on the IMAP server for no visible change.
+  if (state.selectedFolder === folder) return;
+  state.selectedFolder = folder;
+  // Reset pagination on every folder switch — a fresh folder
+  // starts at page 1. Preserved across silent SSE refreshes so
+  // a new arrival doesn't yank the user back from page 3.
+  state.pagination = { offset: 0, limit: 50, total: 0 };
+  renderSidebar(onFolderSelect);
   if (state.selectedAgent) loadList(state.selectedAgent, folder);
 }
 window.addEventListener('hashchange', route);

package/public/js/list-view.js

@@ -359,7 +359,15 @@ export function renderList() {
   const prevBtn = document.getElementById('pager-prev');
   const nextBtn = document.getElementById('pager-next');
   if (prevBtn) prevBtn.disabled = offset <= 0;
-  if (nextBtn) nextBtn.disabled = pageEnd >= total || state.messages.length < limit;
+  // Drive Next purely from the server-reported total. The previous
+  // `state.messages.length < limit` clause was meant as a "we hit the
+  // end" heuristic for the no-total fallback case, but it backfired
+  // on every folder that legitimately had fewer-than-`limit` items on
+  // a page (e.g. trailing partial page after deletions, or a folder
+  // whose IMAP STATUS returned a stale low count) — Next stayed
+  // permanently disabled. The new digest endpoint always returns an
+  // authoritative SEARCH-derived total, so a single check is enough.
+  if (nextBtn) nextBtn.disabled = pageEnd >= total;
 
   if (filtered.length === 0) {
     root.innerHTML = q