portable-agent-layer 0.75.0 → 0.75.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "portable-agent-layer",
-  "version": "0.75.0",
+  "version": "0.75.1",
   "description": "PAL — Portable Agent Layer: persistent personal context for AI coding assistants",
   "type": "module",
   "bin": {

package/src/hooks/lib/agent.ts

@@ -9,9 +9,13 @@
  * Detection reads PAL_AGENT first (set in-process by
  * `src/targets/opencode/plugin.ts`), then the host's own environment, and only
  * then the `--agent=` flag the install templates in `assets/templates/*` put
- * on the hook command line. See declaredAgent for why that order.
+ * on the hook command line. See declaredAgent for why that order. When nothing
+ * declares an agent at all, getActiveAgent falls back to the agent CLIs found
+ * on PATH rather than assuming claude.
  */
 
+import { findBinaryOnPath } from "./which";
+
 export type AgentType = "claude" | "cursor" | "codex" | "copilot" | "opencode" | "vscode";
 
 const KNOWN_AGENTS: ReadonlySet<AgentType> = new Set([
@@ -109,9 +113,36 @@ export function declaredAgent(): AgentType | undefined {
   return agentFromRuntimeEnv() ?? flag;
 }
 
-/** Which agent's conventions to follow. Assumes "claude" when undeclared. */
+/**
+ * The CLI each agent spawns for inference, in the order inference.ts routes
+ * them. vscode is absent because it has no CLI of its own — it runs Claude's.
+ */
+const AGENT_BINARIES: ReadonlyArray<readonly [AgentType, string]> = [
+  ["claude", "claude"],
+  ["codex", "codex"],
+  ["opencode", "opencode"],
+  ["copilot", "copilot"],
+  ["cursor", "cursor-agent"],
+];
+
+/**
+ * Which agent this machine actually has, for the case where nothing declared
+ * one — a plain terminal running `pal cli`, a cron child, a detached spawn.
+ * Assuming claude there names an agent that may not be installed, which routes
+ * inference to a binary that isn't on PATH and reports a host that isn't there.
+ */
+function agentFromInstalledBinary(): AgentType | undefined {
+  return AGENT_BINARIES.find(([, binary]) => findBinaryOnPath(binary) !== null)?.[0];
+}
+
+/**
+ * Which agent's conventions to follow.
+ *
+ * A declaration always wins; claude remains the last resort so a machine with
+ * no agent CLI at all behaves as it always has.
+ */
 export function getActiveAgent(): AgentType {
-  return declaredAgent() ?? "claude";
+  return declaredAgent() ?? agentFromInstalledBinary() ?? "claude";
 }
 
 export const isClaude = () => getActiveAgent() === "claude";

package/src/hooks/lib/which.ts

@@ -9,9 +9,26 @@
  *    names — passing the full `.cmd`/`.exe` path bypasses that fragility.
  */
 
-import { accessSync, constants, existsSync } from "node:fs";
+import { accessSync, constants, statSync } from "node:fs";
 import { delimiter, resolve as resolvePath } from "node:path";
 
+/**
+ * A directory carries the execute bit as "traversable", so X_OK alone accepts
+ * a folder that happens to share a CLI's name. statSync follows symlinks, so a
+ * linked binary still resolves. Windows has no executable bit — being a file
+ * under a PATHEXT extension is all it can offer.
+ */
+function isExecutableFile(candidate: string): boolean {
+  try {
+    if (!statSync(candidate).isFile()) return false;
+    if (process.platform === "win32") return true;
+    accessSync(candidate, constants.X_OK);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
 /** Resolve a binary on PATH to its full absolute path, or null if absent. */
 export function findBinaryOnPath(name: string): string | null {
   const PATH = process.env.PATH;
@@ -24,17 +41,7 @@ export function findBinaryOnPath(name: string): string | null {
     if (!dir) continue;
     for (const ext of exts) {
       const candidate = resolvePath(dir, name + ext);
-      try {
-        if (process.platform === "win32") {
-          // Windows has no executable bit — existence in PATHEXT is enough.
-          if (existsSync(candidate)) return candidate;
-        } else {
-          accessSync(candidate, constants.X_OK);
-          return candidate;
-        }
-      } catch {
-        /* not here — try next */
-      }
+      if (isExecutableFile(candidate)) return candidate;
     }
   }
   return null;