agents-can-communicate 0.8.0 → 0.8.1

package/node_modules/@agents-can-communicate/hook-runner/src/client-pid.mjs

@@ -4,6 +4,35 @@ import path from "node:path";
 // enough that a table which disagrees with itself cannot spin.
 const MAX_HOPS = 16;
 
+// A client shipped as a node script runs as `node`, so its comm names the
+// interpreter. Gemini CLI is one, measured on 0.60.0. For these, the script the
+// interpreter was given names the client.
+const SCRIPT_HOSTS = new Set(["node"]);
+const SCRIPT_EXTENSION = /\.[mc]?js$/;
+
+/** What the script a script host runs could be called, without extensions. */
+function scriptsOf(entry) {
+  if (!SCRIPT_HOSTS.has(path.basename(entry.comm)) || typeof entry.args !== "string") return [];
+  // `args` starts with the interpreter as it was invoked, which is the comm
+  // itself on macOS; past it, the first word that is no option is the script -
+  // unless an option written without `=` came right before it, since that word
+  // may be the option's value (`--require ./preload.cjs`). Which options take
+  // one depends on the client's own node, so such a word is kept as a candidate
+  // and the search goes on; it ends at the first word no bare option precedes.
+  const rest = entry.args.startsWith(`${entry.comm} `) ? entry.args.slice(entry.comm.length)
+    : entry.args.replace(/^\S+/, "");
+  const candidates = [];
+  let afterBareOption = false;
+  for (const word of rest.trim().split(/\s+/)) {
+    if (word === "") continue;
+    if (word.startsWith("-")) { afterBareOption = !word.includes("="); continue; }
+    candidates.push(path.basename(word).replace(SCRIPT_EXTENSION, ""));
+    if (!afterBareOption) break;
+    afterBareOption = false;
+  }
+  return candidates;
+}
+
 /**
  * The pid of the client this hook is running for, or null when nobody knows.
  *
@@ -24,7 +53,7 @@ export function resolveClientPid({ table, from, command, maxHops = MAX_HOPS }) {
     seen.add(current);
     // `ps` reports some entries bare (`claude`) and some with a path
     // (`/bin/zsh`), so the comparison has to be on the basename.
-    if (path.basename(entry.comm) === command) return current;
+    if (path.basename(entry.comm) === command || scriptsOf(entry).includes(command)) return current;
     if (entry.ppid === current || entry.ppid <= 1) return null;
     current = entry.ppid;
   }

package/node_modules/@agents-can-communicate/hook-runner/src/native-binding.mjs

@@ -16,9 +16,8 @@ const DEFAULT_CADENCE_MS = 60_000;
 
 // Reasons the static rule refused: the client will not change within the
 // session, so the state is "unsupported" rather than a retryable "degraded".
-const STATIC_REASONS = new Set(["native_delivery_unsupported", "platform_not_captured",
-  "version_unavailable", "prerelease_not_captured", "below_minimum_version",
-  "known_bad_version"]);
+const STATIC_REASONS = new Set(["native_delivery_unsupported", "version_unavailable",
+  "below_minimum_version", "known_bad_version"]);
 
 const isPid = value => Number.isInteger(value) && value > 0;
 
@@ -77,7 +76,7 @@ export async function establishNativeBinding({ adapter, event, hookBinding, clie
         }, budget);
       }),
     ]);
-    const verdict = validateNativeHandshake(adapter, { clientVersion, platform, handshake });
+    const verdict = validateNativeHandshake(adapter, { clientVersion, handshake });
     if (!verdict.ok) {
       await discardEndpoint(adapter, handshake, runtimeDir);
       return outcome(STATIC_REASONS.has(verdict.reasonCode) ? "unsupported" : "degraded",

package/node_modules/@agents-can-communicate/hook-runner/src/process-table.mjs

@@ -8,27 +8,43 @@ const run = promisify(execFile);
 const DEFAULT_TIMEOUT_MS = 1_000;
 
 const LINE = /^\s*(\d+)\s+(\d+)\s+(.+?)\s*$/;
+// A pid, then the whole rest of the line: a command line has spaces anywhere.
+const ARGS_LINE = /^\s*(\d+)\s+(.+?)\s*$/;
+// Command lines are long; measured at 231 KB for 1,137 processes on a desktop
+// Mac, a quarter of execFile's default buffer.
+const ARGS_BUFFER = 16 * 1024 * 1024;
 
 /**
- * Every process on this machine, as pid -> parent and executable.
+ * Every process on this machine, as pid -> parent, executable and, when it
+ * could be read, command line.
  *
  * Returns an empty map rather than throwing when the platform has no `ps`
  * (Windows) or the call fails. An empty table resolves no client, which is the
- * "nobody knows" answer the caller already handles.
+ * "nobody knows" answer the caller already handles. The command lines are read
+ * at the same time and within the same timeout; they only name a client that
+ * is a script, so when that read fails every entry keeps what it had before.
  */
 export async function readProcessTable({ run: exec = run,
   timeoutMs = DEFAULT_TIMEOUT_MS } = {}) {
-  try {
-    const { stdout } = await exec("ps", ["-o", "pid=,ppid=,comm=", "-A"],
-      { timeout: timeoutMs, killSignal: "SIGKILL" });
-    const table = new Map();
-    for (const line of stdout.split("\n")) {
-      const match = LINE.exec(line);
-      if (match === null) continue;
-      table.set(Number(match[1]), { ppid: Number(match[2]), comm: match[3] });
-    }
-    return table;
-  } catch {
-    return new Map();
+  const options = { timeout: timeoutMs, killSignal: "SIGKILL" };
+  const [tree, commandLines] = await Promise.all([
+    exec("ps", ["-o", "pid=,ppid=,comm=", "-A"], options).then(({ stdout }) => stdout, () => null),
+    exec("ps", ["-A", "-o", "pid=,args="], { ...options, maxBuffer: ARGS_BUFFER })
+      .then(({ stdout }) => stdout, () => ""),
+  ]);
+  if (typeof tree !== "string") return new Map();
+  const args = new Map();
+  for (const line of String(commandLines).split("\n")) {
+    const match = ARGS_LINE.exec(line);
+    if (match !== null) args.set(Number(match[1]), match[2]);
   }
+  const table = new Map();
+  for (const line of tree.split("\n")) {
+    const match = LINE.exec(line);
+    if (match === null) continue;
+    const pid = Number(match[1]);
+    table.set(pid, { ppid: Number(match[2]), comm: match[3],
+      ...(args.has(pid) ? { args: args.get(pid) } : {}) });
+  }
+  return table;
 }

package/node_modules/@agents-can-communicate/hook-runner/src/runner.mjs

@@ -400,8 +400,10 @@ const HANDLERS = {
       const facts = await runtimeFacts(import.meta.url);
       await storeSessionBinding({ runtimeDir: paths.root, deadlineAt: deadline, harnessSessionId: event.sessionId,
         accSessionId: binding.accSessionId, generation: binding.generation, ...facts });
+      // The pid this binding already knew keeps the refreshed pin reapable if
+      // the lookup below cannot find the client.
       await writePin({ root: pinRoot, harnessSessionId: event.sessionId, runtimeRoot: facts.runtimeRoot,
-        version: facts.version, storeVersion: facts.storeVersion });
+        version: facts.version, storeVersion: facts.storeVersion, clientPid: binding.clientPid });
     }
     const clientVersion = await probeClientVersion(adapter,
       { timeoutMs: Math.max(1, Math.min(1_000, deadline - Date.now())) });

package/node_modules/@agents-can-communicate/installer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/installer",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/installer/src/delivery-diagnostics.mjs

@@ -2,9 +2,7 @@
 // reasons; installation and doctor must not turn a version floor into readiness.
 const REASONS = Object.freeze({
   native_delivery_unsupported: "this adapter has no native delivery channel",
-  platform_not_captured: "native delivery is not verified on this platform",
   version_unavailable: "the client version could not be verified",
-  prerelease_not_captured: "this client version is not a verified stable release",
   below_minimum_version: "the client is below the native delivery minimum version",
   known_bad_version: "this client version has a known native delivery failure",
   native_endpoint_unavailable: "the client's local delivery service is unavailable",

package/node_modules/@agents-can-communicate/installer/src/detect.mjs

@@ -13,9 +13,8 @@ const run = promisify(execFile);
 
 // Reasons the client itself will not change within an install: unsupported,
 // as opposed to degraded, which a repaired probe or shell may lift.
-const STATIC_REASONS = new Set(["native_delivery_unsupported", "platform_not_captured",
-  "version_unavailable", "prerelease_not_captured", "below_minimum_version",
-  "known_bad_version"]);
+const STATIC_REASONS = new Set(["native_delivery_unsupported", "version_unavailable",
+  "below_minimum_version", "known_bad_version"]);
 
 const DEFAULT_PROBE_TIMEOUT_MS = 3_000;
 
@@ -76,7 +75,7 @@ async function detectNative(adapter, entry, { context, platform, probeTimeoutMs,
     }
     try {
       facts.eligibility = evaluateNativeEligibility(adapter,
-        { clientVersion: entry.version, platform, probe: facts.probe });
+        { clientVersion: entry.version, probe: facts.probe });
     } catch {
       facts.eligibility = { eligible: false, reasonCode: "feature_probe_failed" };
     }

package/node_modules/@agents-can-communicate/mcp-server/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/mcp-server",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/protocol/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/protocol",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/storage-filesystem/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/storage-filesystem",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "private": true,
   "type": "module",
   "exports": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-can-communicate",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "accManagedUpdateProtocol": 2,
   "accStoreVersion": 6,
   "type": "module",