@agentcomms/whatsapp 0.7.1 → 0.7.2

package/dist/cli.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { $ as withFileLock, A as CommsError, B as installExitStatus, C as checkStorePath, F as canPrompt, G as paint, H as lookupName, I as channelManifest, J as runCommand, K as renderInstall, L as colorEnabled, M as agentMarker, N as approvalKind, O as CHANNELS, P as approveChangeAtTerminal, Q as truncateDisplay, R as escapeForDisplay, S as storeIdentity, T as expandSource, U as nameAvailable, V as isCommsError, W as nameShapeProblem, X as serverPruneChange, Y as serverInstallChange, _ as chatRefOf, a as listChats, b as requireAccount, c as Presenter, d as draftMessage, et as writeResult, f as Visibility, g as requireSupportedNode, h as WhatsAppIndex, i as CHAT_KINDS, j as EXIT_CODES, k as CHANNEL_SERVERS, l as innerText, m as WhatsAppContext, n as whatsappStatus, o as readChat, p as listKey, q as renderPrune, r as probeStore, s as searchMessages, t as syncAccount, tt as VERSION, v as PLATFORM, w as defaultStorePath, x as requireNamedConfig, y as newWhatsAppAccount, z as gatedChangeAtTerminal } from "./sync-C2WSnOq1.mjs";
+import { $ as withFileLock, A as CommsError, B as installExitStatus, C as checkStorePath, F as canPrompt, G as paint, H as lookupName, I as channelManifest, J as runCommand, K as renderInstall, L as colorEnabled, M as agentMarker, N as approvalKind, O as CHANNELS, P as approveChangeAtTerminal, Q as truncateDisplay, R as escapeForDisplay, S as storeIdentity, T as expandSource, U as nameAvailable, V as isCommsError, W as nameShapeProblem, X as serverPruneChange, Y as serverInstallChange, _ as chatRefOf, a as listChats, b as requireAccount, c as Presenter, d as draftMessage, et as writeResult, f as Visibility, g as requireSupportedNode, h as WhatsAppIndex, i as CHAT_KINDS, j as EXIT_CODES, k as CHANNEL_SERVERS, l as innerText, m as WhatsAppContext, n as whatsappStatus, o as readChat, p as listKey, q as renderPrune, r as probeStore, s as searchMessages, t as syncAccount, tt as VERSION, v as PLATFORM, w as defaultStorePath, x as requireNamedConfig, y as newWhatsAppAccount, z as gatedChangeAtTerminal } from "./sync-DDPszuIn.mjs";
 import { rm } from "node:fs/promises";
 import path from "node:path";
 import { stripVTControlCharacters } from "node:util";
@@ -3495,7 +3495,7 @@ a macOS dialog may be waiting) · 77 permission needed (macOS privacy) · 78 con
 		ran = true;
 		exitCode = await runCommand(output(), async () => {
 			requireSupportedNode();
-			const { startWhatsAppStdioServer } = await import("./stdio-entry-BF9Sjjpl.mjs");
+			const { startWhatsAppStdioServer } = await import("./stdio-entry-IB-6jDdg.mjs");
 			await startWhatsAppStdioServer({
 				env,
 				...flags.account ? { account: String(flags.account) } : {}

package/dist/index.mjs

@@ -1,5 +1,5 @@
-import { tt as VERSION, u as composeDraft } from "./sync-C2WSnOq1.mjs";
-import { t as createWhatsAppMcpServer } from "./server-CPNq8D28.mjs";
+import { tt as VERSION, u as composeDraft } from "./sync-DDPszuIn.mjs";
+import { t as createWhatsAppMcpServer } from "./server-BS674Vx-.mjs";
 //#region src/index.ts
 /**
 * The library entry of `@agentcomms/whatsapp`.

package/dist/{server-CPNq8D28.mjs → server-BS674Vx-.mjs}

@@ -1,4 +1,4 @@
-import { D as strictToolArguments, E as UNTRUSTED_NOTICE, Z as toCommsError, a as listChats, d as draftMessage, i as CHAT_KINDS, m as WhatsAppContext, n as whatsappStatus, o as readChat, s as searchMessages, t as syncAccount, tt as VERSION } from "./sync-C2WSnOq1.mjs";
+import { D as strictToolArguments, E as UNTRUSTED_NOTICE, Z as toCommsError, a as listChats, d as draftMessage, i as CHAT_KINDS, m as WhatsAppContext, n as whatsappStatus, o as readChat, s as searchMessages, t as syncAccount, tt as VERSION } from "./sync-DDPszuIn.mjs";
 import { c as boolean, h as number, x as string } from "./schemas-DiGKNDNq.mjs";
 import { t as McpServer } from "./mcp-DXXb3Vv3-Co5BzjYe.mjs";
 //#region src/mcp/server.ts

package/dist/{stdio-entry-BF9Sjjpl.mjs → stdio-entry-IB-6jDdg.mjs}

@@ -1,4 +1,4 @@
-import { t as createWhatsAppMcpServer } from "./server-CPNq8D28.mjs";
+import { t as createWhatsAppMcpServer } from "./server-BS674Vx-.mjs";
 //#region src/mcp/stdio-entry.ts
 /**
 * Starting the stdio server. On stdio, stdout is the protocol, so the console is redirected to stderr first — the

package/dist/{sync-C2WSnOq1.mjs → sync-DDPszuIn.mjs}

@@ -11,9 +11,9 @@ import { constants as constants$1 } from "node:fs";
 //#endregion
 //#region src/version.ts
 /** The package version, read from package.json at build time. */
-const VERSION$1 = "0.7.1";
+const VERSION$1 = "0.7.2";
 //#endregion
-//#region ../core/dist/servers-XHOQHOxT.mjs
+//#region ../core/dist/servers-CI6C4CNi.mjs
 /**
 * Every failure a user or an agent can act on carries a specific, stable code from this registry, a message, an
 * optional one-line hint and structured `details`. The code decides the process exit status; agents and skills branch
@@ -3881,7 +3881,56 @@ const PIN_OPTIONS = Object.freeze([
 ]);
 /** Looks for an executable on PATH, the way a shell would. */
 async function whichExecutable(name, env) {
-	const paths = (env.PATH ?? "").split(delimiter).filter(Boolean);
+	return executableIn(name, (env.PATH ?? "").split(delimiter), env);
+}
+/**
+* Where a client's own command is usually installed, looked in after PATH: `~/.local/bin`, where Anthropic's native
+* installer puts `claude`, then `/opt/homebrew/bin` and `/usr/local/bin`, where Homebrew puts `claude` and `codex` on
+* Apple Silicon and on Intel.
+*
+* An install run from inside an MCP server has the PATH written into that server's entry at registration —
+* `node`'s own directory, `/usr/local/bin`, `/usr/bin`, `/bin` — and nothing of the person's shell. `claude` from
+* Homebrew on an Apple Silicon Mac, or from the native installer anywhere, is on none of those, so a registration
+* asked for from chat found no `claude`, registered nothing, and printed the entry instead. That PATH is left as it
+* is: it is in every entry already registered, and changing it would rewrite them all.
+*
+* The home is the environment's own `HOME`, never `os.homedir()`, so a test's temporary home is the only home looked
+* in. `AGENT_COMMS_CLIENT_CLI_DIRS` replaces the two system directories — a list, separated as PATH is, and empty for
+* none — which is how the tests keep every lookup inside their own directories: a `claude` or `codex` really
+* installed on the machine running them would otherwise be found, and run, by a test that meant to have none.
+*
+* Only absolute directories are looked in. A relative one would be read against whatever directory the server was
+* started from, so the command found — and run — would depend on where that was. And none on Windows: a client there
+* is a `.cmd` shim, which this cannot start without a shell, so a fallback could only find something it cannot run.
+*/
+function clientCliDirectories(env) {
+	if (process.platform === "win32") return [];
+	const home = env.HOME;
+	const system = env.AGENT_COMMS_CLIENT_CLI_DIRS !== void 0 ? env.AGENT_COMMS_CLIENT_CLI_DIRS.split(delimiter) : ["/opt/homebrew/bin", "/usr/local/bin"];
+	return [...home ? [join(home, ".local", "bin")] : [], ...system].filter((directory) => directory !== "" && isAbsolute(directory));
+}
+/**
+* A client's own command — `claude`, `codex` — on PATH, or else where it is usually installed: see
+* `clientCliDirectories`. Every place that looks for a client's command looks through here, so the one that plans a
+* registration and the one that makes it cannot disagree about whether it is there.
+*/
+async function findClientCli(name, env) {
+	return await whichExecutable(name, env) ?? executableIn(name, clientCliDirectories(env), env);
+}
+/** Where `findClientCli` looked, as a sentence ends: "on PATH or in ~/.local/bin, /opt/homebrew/bin or /usr/local/bin". */
+function clientCliSearch(env) {
+	const directories = clientCliDirectories(env);
+	if (directories.length === 0) return "on PATH";
+	const last = directories.at(-1);
+	const rest = directories.slice(0, -1);
+	return `on PATH or in ${rest.length > 0 ? `${rest.join(", ")} or ${last}` : last}`;
+}
+/**
+* The first of `directories` that holds an executable `name`. PATH is read by the caller, once: a test that counts
+* how often the installer looks at PATH counts exactly the looks it means.
+*/
+async function executableIn(name, directories, env) {
+	const paths = directories.filter(Boolean);
 	const extensions = process.platform === "win32" ? (env.PATHEXT ?? ".EXE;.CMD;.BAT").split(";") : [""];
 	for (const directory of paths) for (const extension of extensions) {
 		const candidate = join(directory, name + extension.toLowerCase());
@@ -4342,8 +4391,9 @@ async function backUp(context, client, name, previous, now) {
 }
 /**
 * Where an install with these options goes: the client's own CLI and the file it keeps its servers in, and whether
-* anything will be written there at all — `--print`, `--client json` and a client whose CLI is not on PATH all end
-* with an entry printed rather than registered.
+* anything will be written there at all — `--print`, `--client json` and a client whose CLI cannot be found all end
+* with an entry printed rather than registered. The CLI is looked for on PATH and then where it is usually installed:
+* see `findClientCli`.
 *
 * Exported because a registration is a change a person approves, and what they are shown has to be what then
 * happens. The core server plans an install with this before asking, and the install itself decides with it, so the
@@ -4352,7 +4402,7 @@ async function backUp(context, client, name, previous, now) {
 async function installTarget(context, options) {
 	const apply = options.apply ?? true;
 	const cliName = options.client === "claude-code" ? "claude" : options.client === "codex" ? "codex" : null;
-	const binary = cliName ? await whichExecutable(cliName, context.env) : null;
+	const binary = cliName ? await findClientCli(cliName, context.env) : null;
 	const own = knownClientConfigs(context.env).find((file) => file.client === options.client)?.path;
 	const configPath = options.client === "claude-code" ? void 0 : own;
 	return {
@@ -4607,7 +4657,7 @@ async function mcpInstall(context, product, options, planned) {
 		}
 		method = "cli";
 		applied = true;
-	} else if (cliName && apply) notApplied = `${cliName} was not found on PATH, so nothing was registered`;
+	} else if (cliName && apply) notApplied = `${cliName} was not found ${clientCliSearch(context.env)}, so nothing was registered`;
 	else if (configPath && writes) {
 		await mergeIntoJsonConfig(configPath, options.client, name, entry);
 		method = "file";
@@ -5474,7 +5524,7 @@ function wholeNumber(raw, rule) {
 	return value;
 }
 /** The package version, read from package.json at build time. */
-const VERSION = "0.7.1";
+const VERSION = "0.7.2";
 /**
 * Registering, pruning and listing the MCP servers of every channel, from the core.
 *
@@ -5729,7 +5779,7 @@ function serverPruneChange(core, env, request, own) {
 	};
 	const { packageName, version } = ownProduct(request.channel, own) ?? {
 		packageName: channelServer(request.channel).packageName,
-		version: "0.7.1"
+		version: "0.7.2"
 	};
 	const product = {
 		packageName,
@@ -5768,7 +5818,7 @@ function serverPruneChange(core, env, request, own) {
 	};
 }
 //#endregion
-//#region ../core/dist/tool-arguments-CXQsEUqg.mjs
+//#region ../core/dist/tool-arguments-DBqML6N8.mjs
 const APPLIED = Symbol.for("agentcomms.strictToolArguments");
 const TARGET = "draft-2020-12";
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentcomms/whatsapp",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "description": "WhatsApp for coding agents, read-only: list, read and search the chats WhatsApp for Mac keeps on this Mac, and draft replies as links the person sends. No network client; it never sends.",
   "type": "module",
   "license": "MIT",
@@ -38,7 +38,7 @@
     "@modelcontextprotocol/server": "^2.0.0",
     "commander": "^15.0.0",
     "zod": "^4.6.5",
-    "@agentcomms/core": "0.7.1"
+    "@agentcomms/core": "0.7.2"
   },
   "agentcomms": {
     "contract": 1,
@@ -87,5 +87,5 @@
     "typecheck": "tsc -p tsconfig.json",
     "test": "node --experimental-strip-types --disable-warning=ExperimentalWarning --test --test-timeout=600000 \"test/**/*.test.ts\""
   },
-  "gitHead": "4c6701c2ee05843704d3619c22a8262248e4e1aa"
+  "gitHead": "b0e3060e9ce327f2fd9b040c69bde81721c37e3e"
 }