@nbardy/oompa 0.4.8 → 0.5.0

package/agentnet/src/agentnet/worker.clj

@@ -225,14 +225,16 @@
 (defn- run-reviewer!
   "Run reviewer on worktree changes.
    Returns {:verdict :approved|:needs-changes|:rejected, :comments [...]}"
-  [{:keys [review-harness review-model]} worktree-path]
+  [{:keys [id swarm-id review-harness review-model]} worktree-path]
   (let [;; Get diff for context
         diff-result (process/sh ["git" "diff" "main" "--stat"]
                                 {:dir worktree-path :out :string :err :string})
         diff-summary (:out diff-result)
 
-        ;; Build review prompt
-        review-prompt (str "Review the changes in this worktree.\n\n"
+        ;; Build review prompt (tagged for claude-web-view worker detection)
+        swarm-id* (or swarm-id "unknown")
+        review-prompt (str "[oompa:" swarm-id* ":" id "] "
+                           "Review the changes in this worktree.\n\n"
                            "Diff summary:\n" diff-summary "\n\n"
                            "Check for:\n"
                            "- Code correctness\n"
@@ -277,8 +279,10 @@
 (defn- run-fix!
   "Ask worker to fix issues based on reviewer feedback.
    Returns {:output string, :exit int}"
-  [{:keys [harness model]} worktree-path feedback]
-  (let [fix-prompt (str "The reviewer found issues with your changes:\n\n"
+  [{:keys [id swarm-id harness model]} worktree-path feedback]
+  (let [swarm-id* (or swarm-id "unknown")
+        fix-prompt (str "[oompa:" swarm-id* ":" id "] "
+                        "The reviewer found issues with your changes:\n\n"
                         feedback "\n\n"
                         "Please fix these issues in the worktree.")
 
@@ -394,12 +398,19 @@
             {:keys [output exit done?]} (run-agent! worker wt-path context)]
 
         (cond
-          ;; Agent signaled done
-          done?
+          ;; Agent signaled done — only honor for can_plan workers.
+          ;; Executors (can_plan: false) can't decide work is done.
+          (and done? (:can-plan worker))
           (do
             (println (format "[%s] Received __DONE__ signal" worker-id))
             {:status :done})
 
+          ;; can_plan: false worker said __DONE__ — ignore it, treat as success
+          (and done? (not (:can-plan worker)))
+          (do
+            (println (format "[%s] Ignoring __DONE__ (executor cannot signal done)" worker-id))
+            {:status :continue})
+
           ;; Agent errored
           (not (zero? exit))
           (do

package/config/prompts/_task_header.md

@@ -35,12 +35,22 @@ cat > ../tasks/pending/task-NNN.edn << 'EOF'
 EOF
 ```
 
+### Planning vs Executing
+
+**WHEN PLANNING** (task queue is empty or nearly empty):
+- Your FIRST priority is creating tasks for other workers. They are waiting.
+- Read the project spec, identify gaps, and create 5-10 focused, well-detailed tasks.
+- Do NOT execute tasks in the same iteration you create them.
+- Commit the task files and finish your iteration so others can claim them immediately.
+
+**WHEN EXECUTING** (tasks exist in pending):
+- Claim one task, execute it end-to-end, complete it.
+- If work emerges during execution, create new tasks in `../tasks/pending/`.
+
 ### Rules
 
 - Before starting work: read the project spec and all tasks to understand scope.
-- First action: claim your task by moving it to `../tasks/current/`.
+- Claim your task by moving it to `../tasks/current/`.
 - If the `mv` fails (another worker claimed it first), pick a different task.
 - One task per commit (or a small, tightly-related set with overlapping files).
-- If tasks are missing or underspecified: stop and write tasks before coding.
-- If work emerges during execution: create new tasks in `../tasks/pending/`.
-- When all tasks are complete and the spec is satisfied, output: __DONE__
+- Only output __DONE__ if you have completed work AND no more tasks can be derived from the spec. Never __DONE__ on your first action.

package/package.json

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