relay-companion 0.1.8 → 0.1.9

package/overlay/inbox.html

@@ -1550,6 +1550,9 @@
     else notifyArrival(row);
   });
   if (window.relay.onOpenFull) window.relay.onOpenFull(() => trayOpen());
+  // The OS window was just re-shown and main re-asserted click-through; reset the
+  // local flag so the next mousemove re-runs the interactivity handshake cleanly.
+  if (window.relay.onShown) window.relay.onShown(() => { interactive = false; });
   if (window.relay.onOpenDone) window.relay.onOpenDone((id) => stopOpening(id));
   window.relay.refresh().then((p) => {
     onPayload(p);

package/overlay/main.cjs

@@ -324,7 +324,7 @@ async function pushInbox(force) {
       if (dismissed) {
         if (hostRunning) {
           ghostActive = true;
-          if (!win.isVisible()) win.showInactive();
+          showOverlayWindow();
           win.webContents.send("newRelay", fresh[0], { ghost: true });
         }
       } else {
@@ -686,13 +686,25 @@ function anchorTopRight() {
   return { x: wa.x + wa.width - WIN.width - MARGIN, y: wa.y + MARGIN, width: WIN.width, height: WIN.height };
 }
 
+// Show the overlay window. Electron's forwarded-mousemove stream (the thing that
+// lets the renderer detect hover on a click-through window) can stall across a
+// hide()/show() cycle, leaving the window permanently click-through. Re-assert
+// setIgnoreMouseEvents on every hidden->shown transition and tell the renderer to
+// reset its interactivity handshake.
+function showOverlayWindow() {
+  if (!win || win.isDestroyed() || win.isVisible()) return;
+  win.showInactive();
+  win.setIgnoreMouseEvents(true, { forward: true });
+  win.webContents.send("shown");
+}
+
 function maybeShow() {
   if (!win || win.isDestroyed()) return;
   // Dismissed keeps the window hidden except while a ghost notification is up.
   // trayForcedVisible honors an explicit menu-bar click even when no host runs.
   const wanted = (hostRunning || trayForcedVisible) && (!dismissed || ghostActive);
   if (wanted) {
-    if (!win.isVisible()) win.showInactive();
+    showOverlayWindow();
   } else if (win.isVisible()) {
     win.hide();
   }

package/overlay/preload.cjs

@@ -6,6 +6,7 @@ contextBridge.exposeInMainWorld("relay", {
   onNewRelay: (cb) => ipcRenderer.on("newRelay", (_e, relay, opts) => cb(relay, opts || {})),
   onOpenDone: (cb) => ipcRenderer.on("openDone", (_e, id) => cb(id)),
   onOpenFull: (cb) => ipcRenderer.on("openFull", () => cb()), // menu-bar icon clicked
+  onShown: (cb) => ipcRenderer.on("shown", () => cb()), // window re-shown; reset hover handshake
 
   // pull the full payload on demand
   refresh: () => ipcRenderer.invoke("relay:get"),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relay-companion",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "description": "Relay companion: connects local coding agents to Relay tasks, approvals, and connector tools.",
   "type": "module",
   "bin": {

package/src/mcp.js

@@ -543,7 +543,21 @@ export async function runMcpServer() {
     try {
       return await handleCall(client, req.params.name, req.params.arguments || {});
     } catch (err) {
-      return { content: [{ type: "text", text: `Relay error: ${err.message}` }], isError: true };
+      // Surface the API's validation detail so the calling agent can self-correct
+      // instead of guessing what "invalid_request" meant.
+      let detail = "";
+      const issues = err && err.body && Array.isArray(err.body.issues) ? err.body.issues : null;
+      if (issues && issues.length) {
+        detail =
+          "\n" +
+          issues
+            .slice(0, 8)
+            .map((i) => `- ${Array.isArray(i.path) && i.path.length ? i.path.join(".") : "(request)"}: ${i.message}`)
+            .join("\n");
+      } else if (err && err.body && typeof err.body.instruction === "string") {
+        detail = `\n${err.body.instruction}`;
+      }
+      return { content: [{ type: "text", text: `Relay error: ${err.message}${detail}` }], isError: true };
     }
   });
 

package/src/notifications.js

@@ -287,6 +287,11 @@ function senderForNotification(notification) {
   // anyway via RELAY_HIDDEN_KINDS, but the staged senderName must still be coherent.)
   if (notification.kind === "task_request") return "Your agent";
   if (notification.kind === "share_approval") return "Your agent";
+  // Task lifecycle notices (completed/cancelled/failed) are the agent run reporting
+  // back, not a person — label them as the automation, never "Someone".
+  if (notification.kind === "task_completed") return "Your agent";
+  if (notification.kind === "task_cancelled") return "Your agent";
+  if (notification.kind === "task_failed") return "Your agent";
   if (notification.kind === "connector_reauth") return "Relay connectors";
   return "Someone";
 }