pi-link 0.1.16 → 0.1.17

package/CHANGELOG.md

@@ -6,6 +6,25 @@ This changelog is based on the git history from `2026-03-21` (initial commit) th
 
 ---
 
+## 0.1.17 — 2026-07-01
+
+### Changed
+
+- **`link_send` now rejects self-targets.** Sending to your own link name returns an immediate `self_target` error instead of delivering the message back to yourself, matching the existing `link_prompt` and `link_compact` behavior.
+
+### Fixed
+
+- **Disconnected link tool calls are now marked as errors.** `notConnectedResult()` tags `error: "not_connected"`, so callers and renderers treat a disconnected call as a failure instead of a success-looking result.
+- **Hub ignores duplicate `register` messages.** A second `register` on an already-registered client socket is now dropped instead of re-running the join flow (which could reassign the name and emit a spurious re-join).
+- **`link_prompt` and `link_compact` honor an already-aborted signal.** Both tools now check `signal.aborted` at the top of `execute` and return an `error: "aborted"` result immediately, rather than starting work that was cancelled before it began.
+
+### Internal
+
+- Removed the unused `ExtensionContext` parameter from `shouldConnect()`.
+- Clarified the keepalive comment: it presumes `sendUserMessage()` starts a run, with the sender's 30-minute hard ceiling as the backstop if it ever doesn't.
+
+---
+
 ## 0.1.16 — 2026-06-09
 
 ### Added

package/README.md

@@ -255,7 +255,7 @@ Note: `triggerTurn` does **not** cause the response to come back to the caller -
 
 > **Broadcast note:** Sending to `"*"` delivers to **all other terminals** - the sender is excluded.
 
-Pre-validates the target name against the local terminal list before sending, catching typos early. See [Message Routing](#message-routing--error-handling) for delivery semantics.
+Pre-validates the target name against the local terminal list before sending, catching typos early. See [Message Routing](#message-routing--error-handling) for delivery semantics. **Self-target rejection** - sending to yourself (`to` equals your own name) returns an immediate error.
 
 ### `link_prompt`
 

package/index.ts

@@ -433,7 +433,7 @@ export default function (pi: ExtensionAPI) {
 
   // ── Connection intent ──────────────────────────────────────────────────
 
-  function shouldConnect(_ctx: ExtensionContext): boolean {
+  function shouldConnect(): boolean {
     const data = latestCustomData("link-active") as
       | { active?: boolean }
       | undefined;
@@ -775,7 +775,9 @@ export default function (pi: ExtensionAPI) {
           });
         } else {
           pendingRemotePrompt = { id: msg.id, from: msg.from };
-          // Keepalive: periodic status push so sender knows we're alive
+          // Keepalive: periodic status push so sender knows we're alive.
+          // Keepalive presumes sendUserMessage() starts a run (platform contract);
+          // if it ever doesn't, the sender's 30 min hard ceiling is the backstop.
           if (keepaliveTimer) clearInterval(keepaliveTimer);
           keepaliveTimer = setInterval(
             () => pushStatus(true),
@@ -848,6 +850,7 @@ export default function (pi: ExtensionAPI) {
 
       // First message must be register
       if (msg.type === "register") {
+        if (clientName) return; // already registered — ignore duplicate
         clientName = uniqueName(msg.name);
         hubClients.set(clientWs, clientName);
         if (msg.cwd) hubTerminalCwds.set(clientName, msg.cwd);
@@ -1237,7 +1240,7 @@ export default function (pi: ExtensionAPI) {
       }
     }
 
-    if (flagName || shouldConnect(_ctx)) scheduleStartupConnect();
+    if (flagName || shouldConnect()) scheduleStartupConnect();
   });
 
   pi.on("session_shutdown", async () => {
@@ -1317,7 +1320,7 @@ export default function (pi: ExtensionAPI) {
   }
 
   function notConnectedResult() {
-    return textResult("Not connected to link");
+    return textResult("Not connected to link", { error: "not_connected" });
   }
 
   function truncatePreview(text: string, max = 60) {
@@ -1377,6 +1380,12 @@ export default function (pi: ExtensionAPI) {
 
       // Pre-validate target exists locally (best-effort, catches typos and definitely-absent names)
       if (params.to !== "*") {
+        if (params.to === terminalName) {
+          return textResult("Cannot send to yourself", {
+            to: params.to,
+            error: "self_target",
+          });
+        }
         const miss = targetNotFound(params.to);
         if (miss) return miss;
       }
@@ -1439,6 +1448,13 @@ export default function (pi: ExtensionAPI) {
     }),
 
     async execute(_toolCallId, params, signal) {
+      if (signal?.aborted) {
+        return textResult("Compact request aborted", {
+          to: params.to,
+          error: "aborted",
+        });
+      }
+
       if (role === "disconnected") return notConnectedResult();
 
       if (params.to === terminalName) {
@@ -1537,6 +1553,13 @@ export default function (pi: ExtensionAPI) {
     }),
 
     async execute(_toolCallId, params, signal) {
+      if (signal?.aborted) {
+        return textResult("Prompt request aborted", {
+          to: params.to,
+          error: "aborted",
+        });
+      }
+
       if (role === "disconnected") return notConnectedResult();
 
       if (params.to === terminalName) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-link",
-  "version": "0.1.16",
+  "version": "0.1.17",
   "description": "WebSocket-based inter-terminal communication for Pi. Connect multiple Pi terminals over a local link network.",
   "author": "alvivar",
   "license": "MIT",

package/skills/pi-link-coordination/SKILL.md

@@ -48,7 +48,7 @@ Synchronous RPC. Send a prompt, wait for the response.
 
 ### `link_send`
 
-Fire-and-forget. Send to one terminal or `to: "*"` to broadcast to every other connected terminal; there is no exclusion filter.
+Fire-and-forget. Send to one terminal or `to: "*"` to broadcast to every other connected terminal; there is no exclusion filter. Sending to yourself is rejected. A send to a name that isn't connected fails with an error listing who _is_ connected — delivery failure is visible to the sender, not silent.
 
 Set `triggerTurn: true` to queue async work on the receiver. The sender does **not** get an automatic response back.
 
@@ -69,7 +69,7 @@ Use `triggerTurn: false` for fire-and-forget status notifications only — when
 
 ### `link_compact`
 
-Blocks until the target finishes compacting, then returns. Ask another terminal to compact its context window into a summary, freeing space. `link_list` exposes context usage; `link_compact` is the lever when you decide a terminal should free context — and because the call only returns once compaction is done, you can immediately hand it more work with `link_send`/`link_prompt` (no sleep, no busy-bounce). Busy targets (mid-turn or already compacting) decline; retry when `link_list` shows them idle. You decide the threshold; pi-link just runs the compaction you ask for. Its optional `instructions` add an extra focus to that summary.
+Blocks until the target finishes compacting, then returns (3 min ceiling — the call then errors for you, though the target may still finish compacting on its own; check it before assuming failure). Ask another terminal to compact its context window into a summary, freeing space. `link_list` exposes context usage; `link_compact` is the lever when you decide a terminal should free context — and because the call only returns once compaction is done, you can immediately hand it more work with `link_send`/`link_prompt` (no sleep, no busy-bounce). Busy targets (mid-turn or already compacting) decline; retry when `link_list` shows them idle. You decide the threshold; pi-link just runs the compaction you ask for. Its optional `instructions` add an extra focus to that summary.
 
 ---