relay-companion 0.1.89 → 0.1.90

package/overlay/inbox.html

@@ -703,7 +703,8 @@
      the reused .open-actions light-built panel — never from borders. */
   .sv-head { display:flex; align-items:center; justify-content:space-between; padding:10px 16px 8px; }
   .sv-title { font-family:var(--serif); font-size:16px; font-weight:500; color:var(--ink); }
-  .sv-account { display:flex; align-items:center; gap:12px; padding:4px 16px 0; }
+  /* First element in the pane now (no header above), so it owns the top inset. */
+  .sv-account { display:flex; align-items:center; gap:12px; padding:16px 16px 0; }
   .sv-avatar {
     flex:0 0 auto; width:38px; height:38px; border-radius:50%;
     background:var(--accent-soft); color:var(--accent);
@@ -1561,6 +1562,10 @@
   // The relay row whose open-actions card is expanded. Like openingIds, this
   // survives the innerHTML rebuilds renderRelays performs on every push.
   let expandedRelayId = null;
+  // A conversation expands its actions IN PLACE, exactly like a single relay.
+  // Taking over the whole pane (the old thread-detail push) lost the user's
+  // place in the list for what is really a three-item menu.
+  let expandedThreadId = null;
   function openingTargets(id) {
     return [...document.querySelectorAll("[data-opening-id]")].filter((e) => e.getAttribute("data-opening-id") === id);
   }
@@ -1676,8 +1681,13 @@
       if (t.msgs.length === 1 && inbound.length === 1 && byId.has(inbound[0].id)) {
         return relayRowHtml(byId.get(inbound[0].id), seen);
       }
+      const tExpanded = t.threadId === expandedThreadId;
+      const newestInbound = [...t.msgs].reverse().find((m) => m.direction === "in") || t.msgs[t.msgs.length - 1];
+      const tMenu = tExpanded && newestInbound
+        ? openActionsHtml(newestInbound.id, { thread: true, threadId: t.threadId, messageCount: t.msgs.length })
+        : "";
       return `
-      <div class="th-item${t.unreadCount ? " unread" : ""} deletable" role="button" tabindex="0" data-thread="${esc(t.threadId)}" data-party="${esc(t.party)}">
+      <div class="th-item${t.unreadCount ? " unread" : ""}${tExpanded ? " expanded" : ""} deletable" role="button" tabindex="0" data-thread="${esc(t.threadId)}" data-party="${esc(t.party)}">
         ${threadDeleteButton(t)}
         <span class="th-top">
           ${t.unreadCount ? '<span class="th-unread"></span>' : ""}
@@ -1686,13 +1696,20 @@
           <span class="th-time">${esc(timeAgo(t.latest.at))}</span>
         </span>
         <span class="th-title">${esc(t.latest.title)}</span>
+        ${tMenu}
       </div>`;
     }).join("");
     prevRelayIds = new Set(allRows.map((r) => r.id));
     wireRelayRows();
     for (const el of relaysListEl.querySelectorAll(".th-item")) {
-      el.addEventListener("click", () => openThreadDetail(el.getAttribute("data-thread"), el.getAttribute("data-party") || ""));
+      el.addEventListener("click", () => {
+        const id = el.getAttribute("data-thread");
+        expandedThreadId = expandedThreadId === id ? null : id;
+        expandedRelayId = null; // only one menu open at a time
+        renderRelays();
+      });
     }
+    wireInlineThreadActions();
   }
 
   // Grow the window as the user scrolls toward the end. Cheap: each extension
@@ -1711,7 +1728,7 @@
   // Preview plus the two host-open actions, as one small text-first menu.
   // Preview is the primary, non-navigation path; the other actions hand the
   // relay to a current or fresh host session.
-  function openActionsHtml(id, { thread, previewable = true } = {}) {
+  function openActionsHtml(id, { thread, previewable = true, threadId = "", messageCount = 0 } = {}) {
     const preview = thread ? `data-thread-preview="${esc(id)}"` : `data-preview="${esc(id)}"`;
     const cur = thread ? `data-thread-open-current="${esc(id)}"` : `data-open-current="${esc(id)}"`;
     const fresh = thread ? `data-thread-open-fresh="${esc(id)}"` : `data-open-fresh="${esc(id)}"`;
@@ -1724,9 +1741,37 @@
         <button class="oa-item${previewable ? "" : " oa-primary"}" type="button" role="menuitem" ${cur}>${thread ? "Open Thread in Current Chat" : "Open in Current Chat"}</button>
         <div class="oa-sep" role="separator"></div>
         <button class="oa-item" type="button" role="menuitem" ${fresh}>${thread ? "Open Thread in New Chat" : "Open in New Chat"}</button>
+        ${threadId ? `<div class="oa-sep" role="separator"></div>
+        <button class="oa-item" type="button" role="menuitem" data-thread-history="${esc(threadId)}">Read the Conversation${messageCount ? ` (${messageCount})` : ""}</button>` : ""}
       </div>`;
   }
 
+
+  // The inline conversation menu reuses the thread actions; scoped to the list so
+  // the detail view's own copy keeps working unchanged.
+  function wireInlineThreadActions() {
+    for (const btn of relaysListEl.querySelectorAll("[data-thread-preview]")) {
+      const id = btn.getAttribute("data-thread-preview");
+      btn.addEventListener("click", () => {
+        expandedThreadId = null;
+        window.relay.preview(id);
+        renderRelays();
+      });
+    }
+    for (const btn of relaysListEl.querySelectorAll("[data-thread-open-current]")) {
+      btn.addEventListener("click", () => openRelayFromUI(btn.getAttribute("data-thread-open-current"), "relay", "current"));
+    }
+    for (const btn of relaysListEl.querySelectorAll("[data-thread-open-fresh]")) {
+      btn.addEventListener("click", () => openRelayFromUI(btn.getAttribute("data-thread-open-fresh"), "relay", "fresh"));
+    }
+    for (const btn of relaysListEl.querySelectorAll("[data-thread-history]")) {
+      btn.addEventListener("click", () => {
+        const el = btn.closest(".th-item");
+        openThreadDetail(btn.getAttribute("data-thread-history"), el ? el.getAttribute("data-party") || "" : "");
+      });
+    }
+  }
+
   // Conversations get the same trash affordance as single relays — the unified
   // list made that inconsistent (a 1-message conversation renders as a relay row
   // WITH delete; a 2-message one rendered as a conversation row without). One
@@ -3065,7 +3110,6 @@
     if (info.paired) {
       const name = info.name || info.email || "Relay account";
       html += `
-        <div class="sv-head"><span class="sv-title">Your account.</span></div>
         <div class="sv-account">
           <span class="sv-avatar">${esc(cvInitials(info.name, info.email))}</span>
           <span class="sv-id">

package/overlay/main.cjs

@@ -2330,12 +2330,19 @@ function anchorTopRight() {
   const display = screen.getDisplayNearestPoint(screen.getCursorScreenPoint()) || screen.getPrimaryDisplay();
   const wa = display.workArea;
   const anchor = { x: wa.x + wa.width - WIN.width - MARGIN, y: wa.y + MARGIN, width: WIN.width, height: WIN.height };
-  if (process.env.RELAY_OVERLAY_TEST === "1") {
-    // Sandboxed harness runs must never materialize under the user's parked
-    // cursor: a pointer resting on the card holds notifications open BY DESIGN
-    // (macOS banners do the same), which would wedge every dwell-based scenario
-    // when the mouse happens to sit in the top-right corner. Flip the sandbox
-    // window to the opposite edge instead. Production placement is unchanged.
+  if (process.env.RELAY_OVERLAY_TEST === "1" || process.env.RELAY_OVERLAY_PERF === "1") {
+    // A harness run must not take over the developer's screen. Park sandbox
+    // windows just off the bottom-right of the work area: still a REAL composited
+    // window on the same display — so paint timing, dwell timers and hit tests
+    // measure exactly what production does — but out of sight while the suite
+    // runs, and never under the user's parked cursor (a pointer resting on the
+    // card holds notifications open BY DESIGN, which wedged dwell scenarios).
+    // RELAY_OVERLAY_TEST_ONSCREEN=1 puts it back on screen for eyeball debugging.
+    if (process.env.RELAY_OVERLAY_TEST_ONSCREEN !== "1") {
+      anchor.x = wa.x + wa.width - 8;
+      anchor.y = wa.y + wa.height - 8;
+      return anchor;
+    }
     try {
       const p = screen.getCursorScreenPoint();
       const overlaps =

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relay-companion",
-  "version": "0.1.89",
+  "version": "0.1.90",
   "description": "Relay companion for ordinary messages, with dormant coordination features available only by explicit opt-in.",
   "type": "module",
   "bin": {