@forwardimpact/libharness 1.2.0 → 1.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/libharness",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "Autonomous agent team harness — coordinate a lead and participant agents in one async session, with eval, benchmark, and trace tooling to prove the changes worked.",
   "keywords": [
     "orchestration",

package/src/agent-runner.js

@@ -7,6 +7,7 @@
  */
 
 import { AGENT_MODEL } from "@forwardimpact/libutil/models";
+import { resolveClaudeCodeExecutable } from "./claude-code-executable.js";
 
 const DEFAULT_ALLOWED_TOOLS = ["Bash", "Read", "Glob", "Grep", "Write", "Edit"];
 
@@ -56,6 +57,10 @@ export class AgentRunner {
    * @param {string|object} [deps.systemPrompt] - SDK system prompt (string replaces default; {type:'preset', preset:'claude_code', append} appends)
    * @param {string[]} [deps.disallowedTools] - Tools to explicitly remove from the model's context
    * @param {Record<string, object>} [deps.mcpServers] - MCP server configs to pass to the SDK query
+   * @param {string} [deps.pathToClaudeCodeExecutable] - Absolute path to the
+   *   native `claude` CLI the SDK should spawn. Set for compiled fit-* binaries,
+   *   which can't self-resolve the SDK's platform optional dependency; omitted
+   *   from source runs so the SDK resolves its own version-matched binary.
    * @param {object} deps.redactor
    * @param {import("@forwardimpact/libutil/runtime").Runtime} [deps.runtime] -
    *   Ambient collaborators. Only `proc.env` is read (to record Skill
@@ -79,6 +84,9 @@ export class AgentRunner {
     this.systemPrompt = deps.systemPrompt ?? null;
     this.disallowedTools = deps.disallowedTools ?? [];
     this.mcpServers = deps.mcpServers ?? null;
+    // Optional; read only through a truthy guard in #callOptions, so an absent
+    // value stays undefined rather than needing a `?? null` default.
+    this.pathToClaudeCodeExecutable = deps.pathToClaudeCodeExecutable;
     this.taskAmend = deps.taskAmend ?? null;
     this.sessionId = null;
     /** @type {AbortController|null} */
@@ -158,6 +166,9 @@ export class AgentRunner {
       }),
       ...(this.systemPrompt && { systemPrompt: this.systemPrompt }),
       ...(this.mcpServers && { mcpServers: this.mcpServers }),
+      ...(this.pathToClaudeCodeExecutable && {
+        pathToClaudeCodeExecutable: this.pathToClaudeCodeExecutable,
+      }),
     };
   }
 
@@ -250,7 +261,14 @@ export class AgentRunner {
   }
 }
 
-/** Factory function — wires real dependencies. */
+/**
+ * Factory function — wires real dependencies. Resolves the native `claude`
+ * executable for compiled fit-* binaries so the SDK doesn't fail to find its
+ * own platform optional dependency; an explicit `deps` value overrides it.
+ */
 export function createAgentRunner(deps) {
-  return new AgentRunner(deps);
+  return new AgentRunner({
+    pathToClaudeCodeExecutable: resolveClaudeCodeExecutable(),
+    ...deps,
+  });
 }

package/src/claude-code-executable.js

@@ -0,0 +1,44 @@
+/**
+ * Resolve the Claude Code CLI the Agent SDK should spawn.
+ *
+ * `query()` spawns a native `claude` binary that the SDK resolves from its own
+ * platform-specific optional dependency (`@anthropic-ai/claude-agent-sdk-<platform>`).
+ * `bun build --compile` bundles the SDK's JavaScript but not that separate
+ * native package — it is not part of the import graph — so a compiled fit-*
+ * binary can't self-resolve it and `query()` throws "Native CLI binary for
+ * <platform> not found".
+ *
+ * In a compiled binary we point the SDK at the standalone `claude` on PATH,
+ * installed beside fit-harness by the bootstrap action's `fit-install.sh`.
+ * Running from source keeps `node_modules`, where the SDK resolves its own
+ * version-matched binary, so there we return undefined and defer to the SDK.
+ */
+import { LIBCLI_IS_COMPILED } from "@forwardimpact/libcli";
+
+/**
+ * @param {object} [deps]
+ * @param {(cmd: string) => string | null | undefined} [deps.which] -
+ *   PATH resolver (injected for testing).
+ * @param {boolean} [deps.isCompiled] -
+ *   Whether this is a `bun --compile` binary (injected for testing).
+ * @returns {string | undefined} absolute path to `claude`, or undefined to
+ *   defer resolution to the SDK.
+ */
+export function resolveClaudeCodeExecutable({
+  which = defaultWhich,
+  isCompiled = LIBCLI_IS_COMPILED,
+} = {}) {
+  if (!isCompiled) return undefined;
+  return which("claude") ?? undefined;
+}
+
+/**
+ * A compiled fit-* binary runs under the Bun runtime, which exposes a
+ * synchronous PATH resolver. The `typeof` guard keeps this safe under Node too.
+ */
+function defaultWhich(cmd) {
+  if (typeof Bun !== "undefined" && typeof Bun.which === "function") {
+    return Bun.which(cmd);
+  }
+  return null;
+}

package/src/index.js

@@ -11,6 +11,7 @@ export {
   pickTraceArtifact,
 } from "./trace-github.js";
 export { AgentRunner, createAgentRunner } from "./agent-runner.js";
+export { resolveClaudeCodeExecutable } from "./claude-code-executable.js";
 export {
   composeProfilePrompt,
   composeLeadPrompt,