@agenticmail/cli 0.9.42 → 0.9.44

package/dist/public/js/message-view.js

@@ -171,7 +171,7 @@ function renderMessage(msg) {
         <div class="message-date">${escapeHtml(formatDateFull(msg.date))}</div>
       </div>
     </div>
-    <div class="message-body">${renderBodyWithThreading(bodyText, buildAudienceLookup())}</div>
+    <div class="message-body">${renderBodyWithThreading(bodyText, buildAudienceLookup(msg))}</div>
     ${attachmentsHtml}
   `;
 
@@ -239,12 +239,43 @@ function formatBytes(bytes) {
  * header serialises the date with second precision and stripped
  * timezone info.
  */
-function buildAudienceLookup() {
+function buildAudienceLookup(currentMessage = null) {
+  // Two layered sources, server-authoritative wins:
+  //
+  //   1. currentMessage.quotedAudiences — populated by the API when
+  //      it parses the body, scans for `On <date>, <addr> wrote:`
+  //      headers, and matches them against the IMAP envelopes
+  //      (which carry Cc/Bcc on the wire — we just weren't surfacing
+  //      them before 0.9.30). This works even on direct deep links
+  //      where state.messages is empty.
+  //
+  //   2. state.messages — the inbox list view the operator is
+  //      currently looking at. Fallback for older API responses
+  //      that don't yet include the server-side resolution.
+  //
+  // Layer 1 takes precedence because the server actually has access
+  // to the full mailbox envelopes; the client only sees what was
+  // paginated into state.messages.
+  const serverIndex = new Map();
+  if (currentMessage && Array.isArray(currentMessage.quotedAudiences)) {
+    for (const a of currentMessage.quotedAudiences) {
+      if (!a || !a.sender) continue;
+      // Index by sender alone AND by sender::dateIso for fuzzy date match
+      const senderKey = String(a.sender).toLowerCase();
+      const t = a.dateIso ? new Date(a.dateIso).getTime() : NaN;
+      serverIndex.set(senderKey + '::' + (Number.isFinite(t) ? t : ''), { to: a.to, cc: a.cc, bcc: a.bcc, timeMs: t });
+      // Also keep a "most recent for sender" entry as last-resort fallback
+      const allKey = senderKey + '::*';
+      const prev = serverIndex.get(allKey);
+      if (!prev || (Number.isFinite(t) && t > prev.timeMs)) {
+        serverIndex.set(allKey, { to: a.to, cc: a.cc, bcc: a.bcc, timeMs: t });
+      }
+    }
+  }
   // Defensive: state.messages may be empty if the operator navigated
-  // straight to /#/m/N. Lookup degrades to "no match" — the renderer
-  // falls back to sender-only.
+  // straight to /#/m/N. Lookup degrades to layer 1 only.
   const list = Array.isArray(state.messages) ? state.messages : [];
-  if (list.length === 0) return () => null;
+  if (list.length === 0 && serverIndex.size === 0) return () => null;
   // Flatten to a normalised array we can scan. Each entry carries
   // the same shape the renderer expects.
   const entries = list.map(m => {
@@ -265,23 +296,29 @@ function buildAudienceLookup() {
     if (!senderAddr || !dateStr) return null;
     const senderL = senderAddr.toLowerCase();
     const t = new Date(dateStr).getTime();
+
+    // Layer 1: server-resolved index. Exact date match first, then
+    // sender-only fallback. The server already matched against IMAP
+    // envelopes so this is the authoritative source.
+    if (serverIndex.size > 0) {
+      const exactKey = senderL + '::' + (Number.isFinite(t) ? t : '');
+      const exact = serverIndex.get(exactKey);
+      if (exact) return { to: exact.to, cc: exact.cc, bcc: exact.bcc };
+      const senderOnly = serverIndex.get(senderL + '::*');
+      if (senderOnly) return { to: senderOnly.to, cc: senderOnly.cc, bcc: senderOnly.bcc };
+    }
+
+    // Layer 2: state.messages — surrounding inbox list. Bounded by
+    // what the operator's folder view happens to have loaded.
     if (!Number.isFinite(t)) return null;
-    // Match by sender + nearest date within 2 s. Quote headers
-    // typically lose timezone info during render so wall-clock can
-    // shift by hours; the inbox-list date is the canonical UTC.
-    // We prefer the closest match by absolute delta.
     let best = null;
-    let bestDelta = 2000;  // 2-second tolerance for same-second matches
+    let bestDelta = 2000;
     for (const e of entries) {
       if (e.from !== senderL) continue;
       const delta = Math.abs(e.timeMs - t);
       if (delta < bestDelta) { best = e; bestDelta = delta; }
     }
     if (!best) {
-      // Fallback: if the date didn't parse close to anything, just
-      // take the most recent message from that sender. Better than
-      // nothing for the common case of a reply to the most recent
-      // prior reply.
       for (const e of entries) {
         if (e.from !== senderL) continue;
         if (!best || e.timeMs > best.timeMs) best = e;

package/dist/public/styles.css

@@ -1019,19 +1019,20 @@ mark.search-hl {
    styled card with proper avatar + name + friendly date instead
    of leaving the raw ISO timestamp visible. Nested replies recurse
    so a 3-deep thread gets 3 nested cards. */
-/* Thread-quote: the boxed-up older email a reply quotes. Two issues
- * users flagged on the rendered view before:
+/* Thread-quote: the boxed-up older email a reply quotes. Issues we
+ * sequentially squashed:
  *
- *   1. Huge vertical gap before the quote — was 16px margin-top here
- *      PLUS empty <p> blocks from trailing newlines in the parent
- *      message body (now also fixed in message-view.js by trimming
- *      trailing whitespace before the quote). Dropped to a tight 4px
- *      since the horizontal divider below carries the separation.
- *   2. The pink left rule looked like a stub. We add a full-width
- *      hairline divider on ::before so the visual break between the
- *      parent message and the quote is unambiguous, and pin the rule
- *      so it reads as a thread spine continuing down through the
- *      whole quoted block.
+ *   1. Huge vertical gap before the quote — initially 16px margin-top
+ *      on .thread-quote PLUS empty <p>/<ul> blocks from the markdown
+ *      renderer treating reply sig-lines like `- marlow` as bullet
+ *      lists. message-view.js now trims trailing blank lines from
+ *      the prose buffer; we tighten the rendered block margins below
+ *      so the last <p>/<ul>/<ol> before a quote collapses its
+ *      bottom margin and the gap reads as a single hairline.
+ *   2. The pink left rule looked like a stub. ::before lays a 1px
+ *      full-width divider tucked behind the border so the visual
+ *      break is unambiguous and the rule reads as a continuous
+ *      thread spine through the quoted block.
  *
  * Nested quotes (replies-of-replies) get progressively warmer rule
  * colours (pink → purple → amber) so the depth is visible at a
@@ -1051,6 +1052,29 @@ mark.search-hl {
   height: 1px;
   background: var(--line, rgba(255,255,255,0.08));
 }
+/* Collapse the margin-bottom on whichever block lands immediately
+ * before a thread-quote. Markdown's default 1em paragraph/list
+ * margins were stacking with .thread-quote's margin-top, producing
+ * a ~50px dead zone above quoted replies. With :has() we target
+ * only the preceding sibling (other blocks elsewhere keep their
+ * default margins). Modern browsers all support :has() now. */
+.message-body p:has(+ .thread-quote),
+.message-body ul:has(+ .thread-quote),
+.message-body ol:has(+ .thread-quote),
+.message-body h1:has(+ .thread-quote),
+.message-body h2:has(+ .thread-quote),
+.message-body h3:has(+ .thread-quote),
+.message-body pre:has(+ .thread-quote),
+.message-body blockquote:has(+ .thread-quote) {
+  margin-bottom: 0;
+}
+/* Same tightening on the LAST child of a thread-quote-body — keeps
+ * nested quote ladders from accumulating spacing as they descend. */
+.thread-quote-body > p:last-child,
+.thread-quote-body > ul:last-child,
+.thread-quote-body > ol:last-child {
+  margin-bottom: 0;
+}
 .thread-quote .thread-quote {
   border-left-color: #c084fc;
   margin: 4px 0 8px;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agenticmail/cli",
-  "version": "0.9.42",
+  "version": "0.9.44",
   "description": "Email and SMS infrastructure for AI agents — the first platform to give agents real email addresses and phone numbers",
   "type": "module",
   "main": "dist/index.js",
@@ -31,13 +31,13 @@
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
-    "@agenticmail/api": "^0.9.29",
-    "@agenticmail/core": "^0.9.7",
+    "@agenticmail/api": "^0.9.30",
+    "@agenticmail/core": "^0.9.8",
     "json5": "^2.2.3"
   },
   "optionalDependencies": {
     "@agenticmail/claudecode": "^0.2.22",
-    "@agenticmail/codex": "^0.1.17"
+    "@agenticmail/codex": "^0.1.18"
   },
   "devDependencies": {
     "tsup": "^8.4.0",