pi-link 0.5.1 → 0.5.2

package/CHANGELOG.md

@@ -6,6 +6,22 @@ This changelog is based on the git history from `2026-03-21` (initial commit) th
 
 ---
 
+## 0.5.2 — 2026-09-29
+
+### Fixed
+
+- **Session-directory errors no longer look like an empty session history.** If the CLI cannot enumerate the session root for a reason other than a missing directory, it reports the error and exits with code `1`. The launcher no longer starts a new session on that false absence, `--list` no longer reports no sessions, and `--resolve` no longer reports the name as not found. A missing root still means no sessions yet; handling of individual session files and subdirectories is unchanged.
+
+- **A terminal named `__proto__` keeps its status, cwd and context snapshots.** The name-keyed records in the initial welcome and `link_list` details now use dictionaries without a prototype, so that accepted name is copied into those records and serialized as an ordinary own property rather than lost through JavaScript's inherited setter. No names are newly reserved.
+
+- **`pi-link --version` fails explicitly when package metadata cannot supply a version.** An unreadable or malformed `package.json`, or a missing version, now produces an error and a nonzero exit instead of printing `unknown` and exiting successfully. A healthy installation still prints only its version.
+
+### Changed
+
+- **Internal cleanup reuses existing status and delivery helpers.** Status formatting uses the existing identity formatter, disconnect relies on the inbox scheduler's existing timer guard, and client status updates use the shared hub broadcast path instead of duplicating it. Those updates keep their authoritative sender name, recipients and context rules; the hub now stores them through self-delivery after sending to peers, as the shared path already does. Two unreachable CLI `break` statements after `process.exit(0)` were also removed.
+
+---
+
 ## 0.5.1 — 2026-09-22
 
 ### Fixed

package/bin/pi-link.mjs

@@ -183,14 +183,17 @@ async function loadSessionRecord(filePath, scopeCwd) {
 // Returns meta + mtime + path for every readable session in `dir`, or only
 // those from `scopeCwd` when it is given. Custom layout is flat
 // (<dir>/*.jsonl); default layout has one subdir level per encoded cwd
-// (<dir>/<sub>/*.jsonl). Errors on individual files/dirs are silently skipped
-// — active or partially-written sessions are tolerated.
+// (<dir>/<sub>/*.jsonl). A missing `dir` means no sessions yet; any other
+// failure to read it is fatal, since reporting no sessions would be a lie.
+// Errors on individual files/dirs are silently skipped — active or
+// partially-written sessions are tolerated.
 async function scanSessions(dir, isCustom, scopeCwd) {
   let entries;
   try {
     entries = await readdir(dir, { withFileTypes: true });
-  } catch {
-    return [];
+  } catch (err) {
+    if (err.code === "ENOENT") return [];
+    fail(`cannot read session directory: ${err.message}`);
   }
 
   const tasks = [];
@@ -315,14 +318,14 @@ function printHelp() {
 }
 
 function printVersion() {
+  let pkg;
   try {
-    const pkg = JSON.parse(
-      readFileSync(new URL("../package.json", import.meta.url), "utf-8"),
-    );
-    console.log(pkg.version ?? "unknown");
-  } catch {
-    console.log("unknown");
+    pkg = JSON.parse(readFileSync(new URL("../package.json", import.meta.url), "utf-8"));
+  } catch (err) {
+    fail(`cannot read pi-link's package.json: ${err.message}`);
   }
+  if (!pkg.version) fail("pi-link's package.json has no version");
+  console.log(pkg.version);
 }
 
 function describeMode(mode) {
@@ -515,11 +518,9 @@ switch (state.mode) {
   case "help":
     printHelp();
     process.exit(0);
-    break; // unreachable; present to satisfy no-fallthrough lints
   case "version":
     printVersion();
     process.exit(0);
-    break; // unreachable; present to satisfy no-fallthrough lints
   case "list":
     await runList(state);
     break;

package/index.ts

@@ -421,9 +421,7 @@ export default function (pi: ExtensionAPI) {
   }
 
   function formatStatus(s: LinkStatus): string {
-    const dur = formatDuration(s.since);
-    if (s.kind === "tool") return `tool:${s.toolName} (${dur})`;
-    return `${s.kind} (${dur})`;
+    return `${statusIdentity(s)} (${formatDuration(s.since)})`;
   }
 
   function formatTokens(n: number): string {
@@ -963,14 +961,15 @@ export default function (pi: ExtensionAPI) {
         hubClients.set(clientWs, clientName);
         const list = terminalList();
 
-        // Confirm to the new client (with status, cwd and context snapshots)
-        const statuses: Record<string, LinkStatus> = {};
+        // Confirm to the new client (with status, cwd and context snapshots).
+        // Keyed by terminal name, so no prototype: a plain {} would swallow "__proto__".
+        const statuses: Record<string, LinkStatus> = Object.create(null);
         statuses[terminalName] = deriveStatus(); // hub's own status
         for (const [name, status] of terminalStatuses) statuses[name] = status;
-        const cwds: Record<string, string> = {};
+        const cwds: Record<string, string> = Object.create(null);
         if (currentCwd) cwds[terminalName] = currentCwd; // hub's own cwd
         for (const [name, cwd] of terminalCwds) cwds[name] = cwd;
-        const contexts: Record<string, ContextSnapshot> = {};
+        const contexts: Record<string, ContextSnapshot> = Object.create(null);
         const hubContext = captureContext();
         if (hubContext) contexts[terminalName] = hubContext; // hub's own context
         for (const [name, c] of terminalContexts) contexts[name] = c;
@@ -1001,21 +1000,16 @@ export default function (pi: ExtensionAPI) {
       // Ignore messages from unregistered clients
       if (!clientName) return;
 
-      // Status update — store and fan out to other clients only (not back to hub)
+      // Status update: rebuilt under the authoritative name, fanned out to the other
+      // clients, and stored by the hub through its own self-delivery.
       if (msg.type === "status_update") {
-        terminalStatuses.set(clientName, msg.status);
-        if (msg.context) terminalContexts.set(clientName, msg.context);
-        else if (msg.context === null) terminalContexts.delete(clientName);
         const normalized: StatusUpdateMsg = {
           type: "status_update",
           name: clientName,
           status: msg.status,
           context: msg.context, // undefined omitted by JSON; null forwarded to clear
         };
-        const json = JSON.stringify(normalized);
-        for (const [otherWs, name] of hubClients) {
-          if (name !== clientName) otherWs.send(json);
-        }
+        hubBroadcast(normalized, clientName);
         return;
       }
 
@@ -1340,7 +1334,7 @@ export default function (pi: ExtensionAPI) {
     updateStatus();
 
     // Inbox survives disconnect; flush unless a local /compact still gates it.
-    if (!flushTimer) releaseInbox();
+    releaseInbox();
   }
 
   function cleanup() {
@@ -1765,9 +1759,10 @@ export default function (pi: ExtensionAPI) {
     async execute() {
       if (role === "disconnected") return notConnectedResult();
 
-      const statuses: Record<string, string> = {};
-      const cwds: Record<string, string> = {};
-      const contexts: Record<string, ContextSnapshot> = {};
+      // Keyed by terminal name, so no prototype (see the welcome snapshots).
+      const statuses: Record<string, string> = Object.create(null);
+      const cwds: Record<string, string> = Object.create(null);
+      const contexts: Record<string, ContextSnapshot> = Object.create(null);
       const visible = visibleTerminals();
       const list = visible
         .map((name) => {

package/package.json

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