@nbardy/oompa 0.4.3 → 0.4.5

package/agentnet/src/agentnet/cli.clj

@@ -136,14 +136,19 @@
 (declare cmd-swarm parse-model-string)
 
 (defn- probe-model
-  "Send 'say ok' to a model via its harness CLI. Returns true if model responds."
+  "Send 'say ok' to a model via its harness CLI. Returns true if model responds.
+   Claude hangs without /dev/null stdin when spawned from bb."
   [harness model]
   (try
     (let [cmd (case harness
-                :claude ["claude" "--model" model "-p" "say ok" "--max-turns" "1"]
-                :codex  ["codex" "exec" "--dangerously-bypass-approvals-and-sandbox" "--skip-git-repo-check" "--model" model "--" "say ok"])
-          result (process/sh cmd {:out :string :err :string :timeout 30000})]
-      (zero? (:exit result)))
+                :claude ["claude" "--model" model "-p" "[oompa:probe] say ok" "--max-turns" "1"]
+                :codex  ["codex" "exec" "--dangerously-bypass-approvals-and-sandbox" "--skip-git-repo-check" "--model" model "--" "[oompa:probe] say ok"])
+          null-in (io/input-stream (io/file "/dev/null"))
+          proc (process/process cmd {:out :string :err :string :in null-in})
+          result (deref proc 30000 :timeout)]
+      (if (= result :timeout)
+        (do (.destroyForcibly (:proc proc)) false)
+        (zero? (:exit result))))
     (catch Exception _ false)))
 
 (defn- validate-models!

package/agentnet/src/agentnet/worker.clj

@@ -97,6 +97,26 @@
   "Root of the oompa package — set by bin/oompa.js, falls back to cwd."
   (or (System/getenv "OOMPA_PACKAGE_ROOT") "."))
 
+;; Resolve absolute paths for CLI binaries at first use.
+;; ProcessBuilder with :dir set can fail to find bare command names on some
+;; platforms (macOS + babashka), so we resolve once via `which` and cache.
+(def ^:private binary-paths* (atom {}))
+
+(defn- resolve-binary!
+  "Resolve the absolute path of a CLI binary. Caches result.
+   Throws if binary not found on PATH."
+  [name]
+  (or (get @binary-paths* name)
+      (let [result (try
+                     (process/sh ["which" name] {:out :string :err :string})
+                     (catch Exception _ {:exit -1 :out "" :err ""}))
+            path (when (zero? (:exit result))
+                   (str/trim (:out result)))]
+        (if path
+          (do (swap! binary-paths* assoc name path)
+              path)
+          (throw (ex-info (str "Binary not found on PATH: " name) {:binary name}))))))
+
 (defn- load-prompt
   "Load a prompt file. Tries path as-is first, then from package root."
   [path]
@@ -172,14 +192,14 @@
         ;; Build command — both harnesses run with cwd=worktree, no sandbox
         ;; so agents can `..` to reach project root for task management
         cmd (case harness
-              :codex (cond-> ["codex" "exec"
+              :codex (cond-> [(resolve-binary! "codex") "exec"
                               "--dangerously-bypass-approvals-and-sandbox"
                               "--skip-git-repo-check"
                               "-C" abs-worktree]
                        model (into ["--model" model])
                        reasoning (into ["-c" (str "model_reasoning_effort=\"" reasoning "\"")])
                        true (conj "--" full-prompt))
-              :claude (cond-> ["claude" "-p" "--dangerously-skip-permissions"
+              :claude (cond-> [(resolve-binary! "claude") "-p" "--dangerously-skip-permissions"
                                "--session-id" session-id]
                         model (into ["--model" model])))
 
@@ -227,13 +247,13 @@
 
         ;; Build command — cwd=worktree, no sandbox
         cmd (case review-harness
-              :codex (cond-> ["codex" "exec"
+              :codex (cond-> [(resolve-binary! "codex") "exec"
                               "--dangerously-bypass-approvals-and-sandbox"
                               "--skip-git-repo-check"
                               "-C" abs-wt]
                        review-model (into ["--model" review-model])
                        true (conj "--" review-prompt))
-              :claude (cond-> ["claude" "-p" "--dangerously-skip-permissions"]
+              :claude (cond-> [(resolve-binary! "claude") "-p" "--dangerously-skip-permissions"]
                         review-model (into ["--model" review-model])))
 
         ;; Run reviewer — cwd=worktree
@@ -265,13 +285,13 @@
         abs-wt (.getAbsolutePath (io/file worktree-path))
 
         cmd (case harness
-              :codex (cond-> ["codex" "exec"
+              :codex (cond-> [(resolve-binary! "codex") "exec"
                               "--dangerously-bypass-approvals-and-sandbox"
                               "--skip-git-repo-check"
                               "-C" abs-wt]
                        model (into ["--model" model])
                        true (conj "--" fix-prompt))
-              :claude (cond-> ["claude" "-p" "--dangerously-skip-permissions"]
+              :claude (cond-> [(resolve-binary! "claude") "-p" "--dangerously-skip-permissions"]
                         model (into ["--model" model])))
 
         result (try

package/bin/test-models

@@ -37,9 +37,9 @@ if [ -z "$MODELS" ]; then
   exit 1
 fi
 
-# Create results directory
+# Create results directory in /tmp so it doesn't litter the project
 RUN_ID=$(python3 -c "import uuid; print(str(uuid.uuid4())[:8])")
-RESULTS_DIR="tst_results_${RUN_ID}"
+RESULTS_DIR="/tmp/oompa_tst_${RUN_ID}"
 mkdir -p "$RESULTS_DIR"
 
 MODEL_COUNT=$(echo "$MODELS" | wc -l | tr -d ' ')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbardy/oompa",
-  "version": "0.4.3",
+  "version": "0.4.5",
   "description": "Git-worktree multi-agent swarm orchestrator for Codex and Claude",
   "license": "MIT",
   "type": "commonjs",