relay-companion 0.1.365 → 0.1.367

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "relay-companion",
-  "version": "0.1.365",
+  "version": "0.1.367",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "relay-companion",
-      "version": "0.1.365",
+      "version": "0.1.367",
       "license": "MIT",
       "dependencies": {
         "@anthropic-ai/claude-agent-sdk": "0.3.195",

package/overlay/inbox.html

@@ -94,7 +94,12 @@
      Elevation is three tokens: the card's own drop shadow (this one), the
      folded pill's lighter cousin, and one popover shadow — nothing else casts. */
   .card {
-    position:absolute; inset:0;
+    /* The native window preserves its current screen-space top-right point while
+       it changes size. Anchor the visible sheet to that same point so a folded
+       pill never lands at the old left edge and then jumps home when AppKit
+       trims the transparent remainder. This remains local to the current native
+       window, so a user-dragged pill morphs exactly where it was placed. */
+    position:absolute; top:0; right:0;
     width:344px; height:524px;
     background:var(--bg);
     border-radius:var(--r-card);
@@ -2685,15 +2690,25 @@
   const W = { v: EXPANDED.w, vel: 0, t: EXPANDED.w };
   const H = { v: EXPANDED.h, vel: 0, t: EXPANDED.h };
   let raf = null, lastT = 0;
-  // Main sizes the native window to this card. Deduped: one IPC per distinct
-  // integer size.
+  // Main sizes the native window to this card. A morph has two explicit phases:
+  // prepare grows the native surface before the first destination paint; settled
+  // trims it only after the renderer's spring actually finishes. A monotonic id
+  // makes rapid collapse/expand retargeting reject stale completions.
   let interactive = false;
   let lastSentCardSize = "";
-  function publishCardSize(w, h) {
-    const key = Math.round(w) + "x" + Math.round(h);
+  let cardMotionId = 0;
+  // A renderer reload starts a fresh ordering domain. Without this session id,
+  // main could mistake the new renderer's low motion ids for stale messages.
+  const cardMotionSessionId = crypto.randomUUID();
+  function publishCardSize(w, h, { phase = "prepare", motionId = cardMotionId } = {}) {
+    const key = motionId + ":" + phase + ":" + Math.round(w) + "x" + Math.round(h);
     if (key === lastSentCardSize) return;
     lastSentCardSize = key;
-    if (window.relay.cardSize) window.relay.cardSize(Math.round(w), Math.round(h));
+    if (window.relay.cardSize) window.relay.cardSize(Math.round(w), Math.round(h), {
+      phase,
+      motionId,
+      motionSessionId: cardMotionSessionId,
+    });
   }
   function step(s, dt) {
     const before = s.v - s.t;
@@ -2719,24 +2734,20 @@
     for (let i = 0; i < n; i++) { const a = step(W, h), b = step(H, h); moving = moving || a || b; }
     cardEl.style.width = W.v.toFixed(2) + "px";
     cardEl.style.height = H.v.toFixed(2) + "px";
-    // Native resize only needs to track the animation coarsely; sending it every
-    // frame would compete with the renderer's own motion.
-    publishCardSizeThrottled(W.v, H.v);
     if (moving) raf = requestAnimationFrame(frame);
-    else { raf = null; cardEl.style.willChange = ""; publishCardSize(W.t, H.t); }
-  }
-  let lastPublishAt = 0;
-  function publishCardSizeThrottled(w, h) {
-    const now = performance.now();
-    if (now - lastPublishAt < 80) return;
-    lastPublishAt = now;
-    publishCardSize(w, h);
+    else {
+      raf = null;
+      cardEl.style.willChange = "";
+      publishCardSize(W.t, H.t, { phase:"settled", motionId:cardMotionId });
+    }
   }
   function springTo(w, h) {
     W.t = w; H.t = h;
+    const motionId = ++cardMotionId;
     // Publish the destination synchronously so growth is available before paint
     // and a delayed animation frame cannot leave a stale native window size.
-    publishCardSize(w, h);
+    // Shrink is only committed by the settled message below.
+    publishCardSize(w, h, { phase:"prepare", motionId });
     // The scroller's width is FIXED (containment: the height spring must not
     // reflow the rows every frame) — so it must be re-fixed to each new
     // TARGET, once per morph. Content lays out at the destination size
@@ -2744,7 +2755,13 @@
     // its content in a 344px column and the new width is dead space. The
     // folded pill hides .scroll entirely, so never shrink it below EXPANDED.
     scrollEl.style.width = Math.max(w, EXPANDED.w) + "px";
-    if (REDUCED) { W.v = w; H.v = h; cardEl.style.width = w + "px"; cardEl.style.height = h + "px"; publishCardSize(w, h); return; }
+    if (REDUCED) {
+      W.v = w; H.v = h;
+      cardEl.style.width = w + "px";
+      cardEl.style.height = h + "px";
+      publishCardSize(w, h, { phase:"settled", motionId });
+      return;
+    }
     // will-change on width/height buys nothing (they are layout, not composited)
     // and costs a layer promotion; containment on .scroll is what makes this cheap.
     if (!raf) { lastT = performance.now(); raf = requestAnimationFrame(frame); }
@@ -2970,7 +2987,7 @@
     if (raf) { cancelAnimationFrame(raf); raf = null; cardEl.style.willChange = ""; }
     cardEl.style.width = w + "px";
     cardEl.style.height = h + "px";
-    publishCardSize(w, h);
+    publishCardSize(w, h, { phase:"settled", motionId:++cardMotionId });
   }
   function resetOffstage() {
     ghost = false;
@@ -12021,7 +12038,7 @@
   // authoritative across reloads and future layout changes. Publish the actual
   // initial target before readiness so the native hit rect can never retain a
   // larger stale/default size until the first fold-expand transition.
-  publishCardSize(W.t, H.t);
+  publishCardSize(W.t, H.t, { phase:"settled", motionId:cardMotionId });
   // LAST signal on purpose: every listener above is registered and the native
   // hit rect is synchronized, so main may now deliver queued arrivals without
   // a single one being dropped (FM-1).

package/overlay/main.cjs

@@ -4033,7 +4033,6 @@ function pollHosts({ probeFrontmost = true } = {}) {
 const CARD_INITIAL = { w: 344, h: 524 };    // EXPANDED in inbox.html; renderer publishes its live size before announcing readiness
 const CARD_MAX = { w: 720, h: 800 };        // READER in inbox.html (the pill expanded into the page; PEEK is 400px wide)
 let cardSize = { w: CARD_INITIAL.w, h: CARD_INITIAL.h };
-let overlayFitTimer = null;
 
 function fitOverlayWindowToCard({ settle = false } = {}) {
   if (!win || win.isDestroyed()) return;
@@ -4043,25 +4042,15 @@ function fitOverlayWindowToCard({ settle = false } = {}) {
   // morphs in place and, importantly, never snaps a user-dragged pill home.
   const target = resizedOverlayBounds(current, cardSize, { maximum: CARD_MAX });
   if (target.x === current.x && target.y === current.y && target.width === current.width && target.height === current.height) {
-    if (overlayFitTimer) clearTimeout(overlayFitTimer);
-    overlayFitTimer = null;
     return;
   }
   const growing = target.width > current.width || target.height > current.height;
-  if (growing || settle) {
-    if (overlayFitTimer) clearTimeout(overlayFitTimer);
-    overlayFitTimer = null;
-    try { win.setBounds(target, false); } catch {}
-    return;
-  }
-  // Let the card's closing animation finish before shrinking the native window;
-  // growing happens first so content is never clipped.
-  if (overlayFitTimer) clearTimeout(overlayFitTimer);
-  overlayFitTimer = setTimeout(() => {
-    overlayFitTimer = null;
-    fitOverlayWindowToCard({ settle: true });
-  }, 420);
-  overlayFitTimer.unref?.();
+  // Growth happens before the renderer paints its larger destination. Shrink
+  // waits for an explicit renderer-settled receipt: no guessed timeout, no
+  // competing AppKit resize during the spring, and no stale collapse after a
+  // rapid retarget.
+  if (!growing && !settle) return;
+  try { win.setBounds(target, false); } catch {}
 }
 
 function createWindow() {
@@ -7374,10 +7363,23 @@ ipcMain.on("relay:stall", (_e, info) => {
   }, 500).unref?.();
 }
 
-ipcMain.on("relay:cardSize", (_e, w, h) => {
+let latestCardMotionId = 0;
+let latestCardMotionSessionId = "";
+ipcMain.on("relay:cardSize", (event, w, h, motion = {}) => {
+  if (!win || win.isDestroyed() || event.sender !== win.webContents) return;
   if (!Number.isFinite(w) || !Number.isFinite(h)) return;
+  const motionSessionId = typeof motion.motionSessionId === "string"
+    ? motion.motionSessionId.slice(0, 64)
+    : "";
+  const motionId = Number.isSafeInteger(motion.motionId) ? motion.motionId : 0;
+  if (motionSessionId !== latestCardMotionSessionId) {
+    latestCardMotionSessionId = motionSessionId;
+    latestCardMotionId = 0;
+  }
+  if (motionId < latestCardMotionId) return;
+  latestCardMotionId = motionId;
   cardSize = { w, h };
-  fitOverlayWindowToCard();
+  fitOverlayWindowToCard({ settle: motion.phase === "settled" });
 });
 ipcMain.handle("relay:prepareCardSize", (event, w, h) => {
   if (!win || win.isDestroyed() || event.sender !== win.webContents) return { ok:false };

package/overlay/preload.cjs

@@ -177,8 +177,15 @@ contextBridge.exposeInMainWorld("relay", {
 
   // window plumbing
   engage: () => ipcRenderer.send("relay:engage"),
-  // The native window tracks the visible card's dimensions.
-  cardSize: (w, h) => ipcRenderer.send("relay:cardSize", w, h),
+  // The native window tracks the visible card's dimensions. `prepare` reserves
+  // growth before paint; `settled` commits shrink after the spring completes.
+  cardSize: (w, h, motion = {}) => ipcRenderer.send("relay:cardSize", w, h, {
+    phase: motion.phase === "settled" ? "settled" : "prepare",
+    motionId: Number.isSafeInteger(motion.motionId) ? motion.motionId : 0,
+    motionSessionId: typeof motion.motionSessionId === "string"
+      ? motion.motionSessionId.slice(0, 64)
+      : "",
+  }),
   // Grow before revealing a larger reader so content is never clipped.
   prepareCardSize: (w, h) => ipcRenderer.invoke("relay:prepareCardSize", w, h),
   setTheme: (t) => ipcRenderer.send("relay:theme", t), // preview window wears the same sheet

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "relay-companion",
-  "version": "0.1.365",
+  "version": "0.1.367",
   "description": "Install Relay for Claude Code, Cowork, and Codex, then sign in from the Relay pill.",
   "homepage": "https://sendrelays.com/get-started",
   "repository": {

package/src/mcp.js

@@ -233,7 +233,7 @@ export const TOOLS = [
     name: "relay_send",
     _meta: ALWAYS_LOAD_META,
     description:
-      "Only send a Relay when the user asks you to send (or relay) something to someone. Send ordinary Relay correspondence or a Task. Default to Relay when the user explicitly says Relay or asks to send, share, tell, ask, message, or hand something to a named person or saved group without specifying a medium; an explicit other medium overrides. When the recipient is this human themselves ('me', 'myself', or their own account), call relay_send directly with recipient.self=true; never search contacts or mint a share link for a self-Relay. Resolve every other recipient with relay_contacts_search or relay_groups_list. Without a confident exact match, mint a link with relay_share_link and hand this human the url to paste; never ask for an email address and never fall back to another medium. CLASSIFY THE OUTCOME, NOT THE SENTENCE'S ADDRESSEE: kind='message' seeks the person's attention, opinion, decision, or reply; kind='task' asks their agent to perform external work. A technical subject or non-empty forAgent does not itself make a Task. Compose the complete forAgent document first, then derive forHuman as the concise message in the sender's voice. Use the fewest words that faithfully preserve the message, not the most allowed: 1-3 normally sized sentences and 45 words or fewer. 60 words is only Relay's mandatory-review threshold, never a target, budget, or default; it stops an over-threshold first attempt for review. Message history teaches voice and relationship register, not a default word count. Under-sending to the recipient's agent is worse than over-sending. Relay-owned Task Runs attach their provider's final answer automatically; do not call relay_send merely to report completion. Addressing a person, group, or chat never implies a reply to its latest message. Set replyToRelayId only when the human explicitly wants to quote or answer that exact Relay. For a Granular digital employee, use the exact matching workspace-labelled contactId.",
+      "Only send a Relay when the user asks you to send (or relay) something to someone. When writing a Relay, make normal choices about wording and presentation without asking the user. Ask a clarifying question before sending only when both are true: you are unsure about a detail, and resolving it one way or another could substantially change what the Relay says or commits the user to. Most Relay requests do not require clarification. Send ordinary Relay correspondence or a Task. Default to Relay when the user explicitly says Relay or asks to send something to someone without specifying a medium; another named medium overrides. For a self-Relay, call relay_send with recipient.self=true; never search contacts or mint a link. Resolve other recipients with relay_contacts_search or relay_groups_list. Without a confident match, mint a link with relay_share_link; never ask for an email or switch media. CLASSIFY THE OUTCOME, NOT THE SENTENCE'S ADDRESSEE: kind='message' seeks the person's attention, opinion, decision, or reply; kind='task' asks their agent to perform external work. Compose the complete forAgent document first, then derive forHuman as the concise message in the sender's voice. Use the fewest words that faithfully preserve the message, not the most allowed: 1-3 normally sized sentences and 45 words or fewer. 60 words is only Relay's mandatory-review threshold, never a target, budget, or default; it stops an over-threshold first attempt for review. Message history teaches voice and relationship register, not a default word count. Under-sending to the recipient's agent is worse than over-sending. Relay-owned Task Runs attach their provider's final answer automatically; do not call relay_send merely to report completion. Addressing a person, group, or chat never implies a reply to its latest message. Set replyToRelayId only when the human explicitly wants to quote or answer that exact Relay. For a Granular digital employee, use the exact matching workspace-labelled contactId.",
     inputSchema: {
       type: "object",
       properties: {
@@ -513,7 +513,7 @@ export const TOOLS = [
   {
     name: "relay_chat_send",
     description:
-      "Only send a Relay when the user asks you to send (or relay) something to someone. Send an ordinary text message into an existing Relay chat. chatId addresses the room; it does not imply a reply to the newest message. Set replyToRelayId only when the human explicitly selected or named a specific message to quote. Supports the same local-file attachment forms as relay_send. This always sends kind='message'; use relay_send for a Task or a separate forAgent document.",
+      "Only send a Relay when the user asks you to send (or relay) something to someone. When writing a Relay, make normal choices about wording and presentation without asking the user. Ask a clarifying question before sending only when both are true: you are unsure about a detail, and resolving it one way or another could substantially change what the Relay says or commits the user to. Most Relay requests do not require clarification. Send an ordinary text message into an existing Relay chat. chatId addresses the room; it does not imply a reply to the newest message. Set replyToRelayId only when the human explicitly selected or named a specific message to quote. Supports the same local-file attachment forms as relay_send. This always sends kind='message'; use relay_send for a Task or a separate forAgent document.",
     inputSchema: CHAT_SEND_INPUT_SCHEMA,
   },
   {