relay-companion 0.1.74 → 0.1.76

package/overlay/inbox.html

@@ -578,8 +578,10 @@
   .cv-hint:empty { display:none; }
 
   /* ---- Threads tab: thread rows + one thread's message history ---- */
+  /* A div, not a button: it hosts the delete button, and nesting buttons is
+     invalid HTML (the parser hoists the inner one and breaks the row). */
   .th-item {
-    display:block; width:100%; text-align:left; appearance:none; border:0; cursor:pointer;
+    position:relative; display:block; width:100%; text-align:left; appearance:none; border:0; cursor:pointer;
     padding:11px 16px; border-bottom:1px solid var(--hair); background:transparent;
     font-family:var(--sans); transition:background-color .14s var(--settle);
   }
@@ -770,9 +772,9 @@
         </div>
         <div id="thDetail" class="gone">
           <div class="th-detail-head">
-            <button class="th-back" type="button" id="thBack" aria-label="Back to threads">
+            <button class="th-back" type="button" id="thBack" aria-label="Back to relays">
               <svg width="7" height="12" viewBox="0 0 8 14" fill="none" stroke="currentColor" stroke-width="1.8" stroke-linecap="round"><path d="M7 1L1 7l6 6"/></svg>
-              Threads
+              Relays
             </button>
             <span class="th-detail-name" id="thDetailName"></span>
           </div>
@@ -1582,7 +1584,8 @@
       }
       const sentOnly = inbound.length === 0;
       return `
-      <button class="th-item${t.unreadCount ? " unread" : ""}${sentOnly ? " th-sent-only" : ""}" type="button" data-thread="${esc(t.threadId)}">
+      <div class="th-item${t.unreadCount ? " unread" : ""}${sentOnly ? " th-sent-only" : ""} 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>' : ""}
           <span class="th-party">${sentOnly ? "To " : ""}${esc(t.party)}</span>
@@ -1590,12 +1593,12 @@
           <span class="th-time">${esc(timeAgo(t.latest.at))}</span>
         </span>
         <span class="th-title">${esc(t.latest.title)}</span>
-      </button>`;
+      </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.addEventListener("click", () => openThreadDetail(el.getAttribute("data-thread"), el.getAttribute("data-party") || ""));
     }
   }
 
@@ -1613,6 +1616,17 @@
       </div>`;
   }
 
+  // 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
+  // click removes every inbound relay in the conversation; outbound copies are
+  // the recipient's to delete, so a sent-only conversation has nothing to trash.
+  function threadDeleteButton(t) {
+    const ids = (t.msgs || []).filter((m) => m.direction === "in").map((m) => m.id).filter(Boolean);
+    if (!ids.length) return "";
+    return `<button class="row-delete" type="button" data-delete-thread="${esc(ids.join(","))}" data-stop="1" aria-label="Move this conversation to Recently Deleted" title="Move this conversation to Recently Deleted"><svg width="13" height="13" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="1.8" stroke-linecap="round" stroke-linejoin="round"><path d="M4 7h16M9 7V4h6v3M7 7l1 13h8l1-13M10 11v5M14 11v5"/></svg></button>`;
+  }
+
   // The classic relay row (used flat while peeking, and for single-message
   // conversations in the unified list). Row click toggles the open-actions
   // card; the buttons carry the actual opens.
@@ -1744,6 +1758,19 @@
       ta.addEventListener("input", () => { ta.style.height = "auto"; ta.style.height = Math.min(84, ta.scrollHeight) + "px"; });
       ta.addEventListener("mousedown", () => setInteractive(true));
     }
+    for (const btn of relaysListEl.querySelectorAll("[data-delete-thread]")) {
+      const ids = (btn.getAttribute("data-delete-thread") || "").split(",").filter(Boolean);
+      btn.addEventListener("click", async (event) => {
+        event.stopPropagation();
+        if (btn.disabled || !ids.length) return;
+        btn.disabled = true;
+        try {
+          for (const id of ids) await window.relay.deleteRelay(id);
+        } catch {
+          btn.disabled = false;
+        }
+      });
+    }
     for (const btn of relaysListEl.querySelectorAll("[data-delete]")) {
       const id = btn.getAttribute("data-delete");
       btn.addEventListener("click", async (event) => {
@@ -2530,6 +2557,7 @@
   // relays plus the live Sent list. Local data only — the pill shows the
   // user's own exchanges; the API's /v1/threads serves web and agents.
   let threadDetailId = null; // threadId while a thread's history is open
+  let threadDetailPartyHint = ""; // who it's with, for the header while messages load
   function threadMessages() {
     const msgs = [];
     for (const r of payload.relays || []) {
@@ -2602,8 +2630,14 @@
   function renderThreadDetail() {
     const thread = buildThreads().find((t) => t.threadId === threadDetailId);
     if (!thread) {
-      threadDetailId = null;
-      return renderThreads();
+      // A conversation whose messages live on the SENT side renders empty until
+      // the sent cache loads — and since 0.1.73 that refresh is lazy (30s when
+      // idle). Ask for it now and show a loading line rather than a blank pane;
+      // the next payload re-renders this view with the messages.
+      thDetailNameEl.textContent = threadDetailPartyHint || "Conversation";
+      thHistoryEl.innerHTML = '<div class="cv-empty">Loading conversation…</div>';
+      if (window.relay.refreshSent) window.relay.refreshSent().catch(() => {});
+      return;
     }
     thDetailNameEl.textContent = thread.party;
     // Thread-level opens act on the newest inbound message (falling back to the
@@ -2638,8 +2672,9 @@
     applyView();
     renderAll();
   });
-  function openThreadDetail(threadId) {
+  function openThreadDetail(threadId, partyHint = "") {
     threadDetailId = threadId;
+    threadDetailPartyHint = partyHint;
     activeView = "threads";
     syncTabs();
     applyView();

package/package.json

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

package/src/auto-update.js

@@ -623,6 +623,13 @@ export function spawnDetachedUpdate({
     `    echo "[relay-update] $(date) electron runtime missing; running electron install.js" >> ${LOG} 2>&1`,
     `    "$NODE" "$ELECTRON_DIR/install.js" >> ${LOG} 2>&1 || true`,
     `  fi`,
+    // A half-extracted dist (binary present, Frameworks missing) SIGABRTs at
+    // launch and passed the old executable-only check. Force a clean re-extract.
+    `  ELECTRON_FRAMEWORKS="$ELECTRON_DIR/dist/Electron.app/Contents/Frameworks/Electron Framework.framework"`,
+    `  if [ -x "$ELECTRON" ] && [ ! -d "$ELECTRON_FRAMEWORKS" ] && [ -f "$ELECTRON_DIR/install.js" ]; then`,
+    `    echo "[relay-update] $(date) electron dist incomplete (no Frameworks); re-extracting" >> ${LOG} 2>&1`,
+    `    rm -rf "$ELECTRON_DIR/dist" 2>/dev/null; "$NODE" "$ELECTRON_DIR/install.js" >> ${LOG} 2>&1 || true`,
+    `  fi`,
     // Verify the package at the exact launchd-owned root reached the exact discovered
     // version. A mismatched prefix or stale npm response must never trigger restart.
     `  INSTALLED_VERSION="$("$NODE" -e 'process.stdout.write(require(process.argv[1]).version)' "$PKG/package.json" 2>/dev/null)"`,
@@ -631,7 +638,7 @@ export function spawnDetachedUpdate({
       ? `  VERSION_OK=0; if [ "$INSTALLED_VERSION" = ${q(target)} ]; then VERSION_OK=1; else echo "[relay-update] $(date) verify FAILED: installed $INSTALLED_VERSION != exact target ${target}" >> ${LOG} 2>&1; fi`
       : `  VERSION_OK=1`,
     `  DESKTOP_OK=0`,
-    `  if [ "$VERSION_OK" = 1 ] && [ -x "$ELECTRON" ] && [ -f "$PKG/bin/relay.js" ] && grep -q '"version"' "$PKG/package.json" 2>/dev/null; then`,
+    `  if [ "$VERSION_OK" = 1 ] && [ -x "$ELECTRON" ] && [ -d "$ELECTRON_FRAMEWORKS" ] && [ -f "$PKG/bin/relay.js" ] && grep -q '"version"' "$PKG/package.json" 2>/dev/null; then`,
     // Rebuild version-coupled app/plist paths from the NEW package before restart.
     // --no-restart keeps sequencing here: pill first, daemon (our parent) last.
     `    if "$NODE" "$PKG/bin/relay.js" repair-desktop --no-restart${repairModeArgs ? ` ${repairModeArgs}` : ""} >> ${LOG} 2>&1; then DESKTOP_OK=1; else echo "[relay-update] $(date) desktop repair FAILED" >> ${LOG} 2>&1; fi`,

package/src/client.js

@@ -1,4 +1,16 @@
 import { apiUrl, deviceToken } from "./config.js";
+import { createRequire } from "node:module";
+
+// Reported on every device-authenticated call so the server always knows which
+// companion version each device runs — support and rollout questions get
+// answered from data, never from screenshots.
+const COMPANION_VERSION = (() => {
+  try {
+    return String(createRequire(import.meta.url)("../package.json").version || "");
+  } catch {
+    return "";
+  }
+})();
 
 /** Thin authenticated client for relay-api, used by the CLI, MCP server, and daemon. */
 export class RelayClient {
@@ -9,6 +21,7 @@ export class RelayClient {
 
   async #req(method, path, body, { auth = true } = {}) {
     const headers = { "Content-Type": "application/json" };
+    if (COMPANION_VERSION) headers["x-relay-version"] = COMPANION_VERSION;
     if (auth && this.token) headers.Authorization = `Bearer ${this.token}`;
     const res = await fetch(`${this.url}${path}`, {
       method,