agents-can-communicate 0.3.0 → 0.3.1

package/node_modules/@agents-can-communicate/adapter-kimi/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/adapter-kimi",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/adapter-kimi/plugin/skills/acc/SKILL.md

@@ -47,14 +47,14 @@ diffs. Prefer a conclusion, stable ids or paths, and the next action.
 For information that needs no response:
 
 ```bash
-{{ACC}} message --to models --type note --subject "schema verified" \
+{{ACC}} message --to codex --type note --subject "schema verified" \
   --body "Record v2 accepts nullable pid; no migration is planned."
 ```
 
 For a question, use the kind whose default obligation is a reply:
 
 ```bash
-{{ACC}} message --to models --type question \
+{{ACC}} message --to codex --type question \
   --subject "claim boundary" --body "Can I take file:src/parser/** after your commit?"
 ```
 
@@ -65,7 +65,9 @@ When the peer should own a concrete piece of work, send one reply-required reque
   --detail "Check queued -> retrieved and reply -> acknowledged; return only defects."
 ```
 
-Participant names come from `{{ACC}} status --json`. A request is not an order.
+Address a peer by its client - `codex`, `claude_code`, `gemini_cli` - when one session of
+it is here; `{{ACC}} status --json` names them all, and two sessions of one client have to be
+named exactly. A request is not an order.
 
 ## Treat delivery as evidence
 

package/node_modules/@agents-can-communicate/adapter-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/adapter-sdk",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

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

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/cli",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/cli/src/install-command.mjs

@@ -6,7 +6,7 @@ import { createCodexAdapter } from "@agents-can-communicate/adapter-codex";
 import { createGeminiCliAdapter } from "@agents-can-communicate/adapter-gemini-cli";
 import { createGrokAdapter } from "@agents-can-communicate/adapter-grok";
 import { createKimiAdapter } from "@agents-can-communicate/adapter-kimi";
-import { LIVE_POLICIES, applyPlan, describeActivation, detectInstallation, livePolicyOf,
+import { LIVE_POLICIES, applyPlan, detectInstallation, livePolicyOf,
   loadOwnership, planInstallation, rcFileFor, shellOf, shimDirFor }
   from "@agents-can-communicate/installer";
 import { AccError, EXIT } from "@agents-can-communicate/protocol";
@@ -192,18 +192,43 @@ export function actedOn(result) {
 const isInteractive = runtime => typeof runtime.isInteractive === "function"
   && runtime.isInteractive() === true;
 
-// One default-No question per eligible client, asked only after detection has
-// finished and only when a person is at both ends of the terminal. It names
-// the client, the mechanism, and the files or service it would touch, and says
-// when the choice takes effect.
-function questionFor(entry, context) {
-  const activation = { livePolicy: "actionable", shell: context.shell,
-    rcFile: rcFileFor(context.home, context.shell), shimDir: shimDirFor(context.stateRoot),
-    mechanisms: entry.nativeDelivery.activationPlan.mechanisms };
-  return [`Enable native live delivery for ${entry.displayName} ${entry.version}?`,
-    ...describeActivation(activation).map(line => `  ${line}`),
-    "  applies to newly started sessions; a new PATH block needs a new or reloaded shell"]
-    .join("\n");
+/**
+ * One default-No question per eligible client, asked only after detection has
+ * finished and only when a person is at both ends of the terminal.
+ *
+ * Written for someone who has never heard of this project. It used to open with
+ * "Enable native live delivery" over a list of internal artefact names, which
+ * told a first-time reader neither what they would gain nor what it would cost -
+ * and left out the part that matters most: saying yes makes their own `claude`
+ * start with the vendor's `--dangerously-load-development-channels` flag. A
+ * consent prompt that hides the loudest consequence is not consent.
+ *
+ * So the order is the reader's, not the installer's: what changes for them,
+ * what it costs, what it touches, and that "no" costs them nothing.
+ */
+function questionFor(entry, context, { onlyEligible = false } = {}) {
+  const bootstrap = entry.nativeDelivery.activationPlan.mechanisms
+    .find(mechanism => mechanism.kind === "shell-bootstrap") ?? null;
+  // The presence of a flag is what makes this worth asking about at all: it is
+  // why the client will warn at every start from now on. Which flag it is, and
+  // that a shim adds it, are answers to a question nobody is asking here - the
+  // shim `exec`s and is gone, so nothing runs "through" acc afterwards either.
+  const flags = (bootstrap?.prefixArgs ?? []).filter(argument => argument.startsWith("--"));
+  void context;
+  // Why the default is No, and why only one client is being asked about, are
+  // the two questions the bare prompt provokes and never answered: a `[y/N]`
+  // reads as "not recommended" on its own, and a single question among four
+  // installed clients reads as an omission. Both have the same short answer.
+  const aside = ["experimental", ...(onlyEligible ? [`${entry.displayName} only`] : [])]
+    .join("; ");
+  return [
+    `Let idle agents answer each other while you are away?  (${aside})`,
+    ...(flags.length === 0 || bootstrap === null
+      ? ["  Yes: they reply without waiting for you."]
+      : [`  Yes: they reply without waiting for you, and ${entry.displayName} asks you`,
+        "       to allow development channels every time it starts."]),
+    "  No:  messages still arrive, at the session's next turn.",
+  ].join("\n");
 }
 
 /**
@@ -220,6 +245,11 @@ export async function decideDelivery({ options, detected, recorded, runtime, dry
     throw new AccError(EXIT.USAGE, `unknown delivery policy: ${explicit}`, { delivery: explicit });
   }
   const recordedById = new Map(recorded.map(install => [install.adapterId, install]));
+  // Whether this client is the only one that could be asked at all. When it is,
+  // the question says so, because otherwise being asked about one of four
+  // installed clients looks like the other three were forgotten.
+  const onlyEligible = detected
+    .filter(candidate => candidate.nativeDelivery?.state === "eligible").length === 1;
   const deliveryByAdapter = {};
   const asked = [];
   let withheld = 0;
@@ -234,7 +264,7 @@ export async function decideDelivery({ options, detected, recorded, runtime, dry
       deliveryByAdapter[entry.adapterId] = "off";
       if (eligible) withheld += 1;
     } else {
-      const yes = await runtime.confirm(questionFor(entry, context),
+      const yes = await runtime.confirm(questionFor(entry, context, { onlyEligible }),
         { input: runtime.input, output: runtime.output }) === true;
       deliveryByAdapter[entry.adapterId] = yes ? "actionable" : "off";
       asked.push(entry.adapterId);

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

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/core",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/core/src/conversations.mjs

@@ -55,6 +55,41 @@ function assertCurrentSession(tx, session, action) {
   }
 }
 
+/**
+ * Turn what the sender typed into participant ids that exist.
+ *
+ * An exact participant id is always taken as written. A name that matches no
+ * participant is tried as a client name - `codex`, `claude_code` - because that
+ * is what an agent knows about its peers: the roster it is handed names the
+ * client, and asking it to first look up `codex-fbqX8o` is a lookup it
+ * routinely skips. Every shipped example taught this form while nothing
+ * implemented it, so an agent following its own skill was refused on its first
+ * attempt to reach anyone.
+ *
+ * One live session of that client resolves. Two refuse and name them, the way
+ * `--session` already refuses to choose between two sessions of one client:
+ * guessing is how a question is answered by a session that never had the
+ * context. The sender's own client is excluded, or the one client an agent is
+ * certain is running becomes the easiest thing to address by accident.
+ */
+function resolveAddressees(tx, requested, session) {
+  const known = new Set(tx.list("participant").map(item => item.participantId));
+  return requested.map(name => {
+    if (known.has(name)) return name;
+    const peers = [...new Set(tx.list("session", item => item.state === "open"
+      && item.harness === name && item.participantId !== session.participantId)
+      .map(item => item.participantId))].sort();
+    if (peers.length === 1) return peers[0];
+    if (peers.length > 1) {
+      throw new AccError(EXIT.USAGE,
+        `could not tell which of ${peers.length} ${name} sessions you meant: `
+        + `${peers.join(", ")}. Name one of them, which \`acc status --json\` lists.`,
+        { client: name, candidates: peers });
+    }
+    return name;
+  });
+}
+
 function addressedRecipients(tx, message, session) {
   if (message.toParticipantIds.length === 0) {
     return [...new Set(tx.list("session", item => item.state === "open"
@@ -86,9 +121,14 @@ function threadFor(tx, messageId, inReplyTo) {
   return parent.threadId;
 }
 
-export function recordMessageInTransaction({ tx, session, input, now, messageId, ids,
+export function recordMessageInTransaction({ tx, session, input: raw, now, messageId, ids,
   action = "send a message" }) {
   assertCurrentSession(tx, session, action);
+  // Resolved before anything else reads the recipients, so the stored message
+  // names real participants and a retry compares against the same content. A
+  // record that kept the client name would address nobody when it is read back.
+  const input = { ...raw,
+    toParticipantIds: resolveAddressees(tx, raw.toParticipantIds ?? [], session) };
   const existing = existingMessage(tx, session, input);
   if (existing !== undefined) {
     return { message: existing, recipientParticipantIds: [], created: false };

package/node_modules/@agents-can-communicate/delivery-router/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/delivery-router",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

package/node_modules/@agents-can-communicate/hook-runner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/hook-runner",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

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

@@ -1,9 +1,11 @@
 {
   "name": "@agents-can-communicate/installer",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
-  "exports": { ".": "./src/index.mjs" },
+  "exports": {
+    ".": "./src/index.mjs"
+  },
   "files": [
     "src/"
   ]

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

@@ -1,6 +1,6 @@
 {
   "name": "@agents-can-communicate/mcp-server",
-  "version": "0.3.0",
+  "version": "0.3.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.3.0",
+  "version": "0.3.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.3.0",
+  "version": "0.3.1",
   "private": true,
   "type": "module",
   "exports": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-can-communicate",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "type": "module",
   "description": "Local-first coordination for independently opened AI agent sessions.",
   "keywords": [