From 6a64d52b9cf932ead83b189e33bb9e19dceb6ef9 Mon Sep 17 00:00:00 2001
From: Ben Sima <ben@bsima.me>
Date: Thu, 20 Nov 2025 16:58:50 -0500
Subject: feat: implement task harvesting logic

- Added Omni/Agent/harvest-tasks.sh for Planner to pull updates from
workers - Updated WORKER_AGENT_GUIDE.md to instruct workers to commit
status updates to their base branch
---
 Omni/Agent/WORKER_AGENT_GUIDE.md | 59 ++++++++++++++++++++++------------------
 1 file changed, 33 insertions(+), 26 deletions(-)

(limited to 'Omni/Agent/WORKER_AGENT_GUIDE.md')

diff --git a/Omni/Agent/WORKER_AGENT_GUIDE.md b/Omni/Agent/WORKER_AGENT_GUIDE.md
index f1f6f60..64a55f8 100644
--- a/Omni/Agent/WORKER_AGENT_GUIDE.md
+++ b/Omni/Agent/WORKER_AGENT_GUIDE.md
@@ -64,38 +64,45 @@ task update t-123 in-progress
 
 ### Step 5: Submit for Review
 
-```bash
-# 1. Mark for review
-task update t-123 review
+1.  **Commit Implementation**:
+    ```bash
+    git add .
+    git commit -m "feat: implement t-123"
+    ```
 
-# 2. Commit implementation
-git add .
-git commit -m "feat: implement t-123"
+2.  **Signal Review Readiness**:
+    The Planner checks the `omni-worker-X` branch for status updates. You must switch back and update the status there.
 
-# 3. Switch back to worker base branch to prepare for next loop
-# This detaches the worker from the task branch so the Planner can check it out
-git checkout omni-worker-1
-./Omni/Agent/sync-tasks.sh
-```
+    ```bash
+    # Switch to base branch
+    git checkout omni-worker-1
+    
+    # Sync to get latest state (and any manual merges)
+    ./Omni/Agent/sync-tasks.sh
+    
+    # Mark task for review
+    task update t-123 review
+    
+    # Commit this status change to the worker branch
+    ./Omni/Agent/sync-tasks.sh --commit
+    ```
+
+    *Note: The Planner will now see 't-123' in 'Review' when it runs `harvest-tasks.sh`.*
 
 ## 3. Planner (Reviewer) Workflow
 
 The Planner Agent (running in the main repo) will:
-1.  See tasks in `Review` status (after checking out the worker's branch or waiting for them to be merged).
-    *   *Note: Since workers commit to local branches, you verify work by checking out their branch.*
-2.  Check out the worker's branch: `git checkout task/t-123`.
-3.  Review code and tests.
-4.  Merge to `live`:
-    ```bash
-    git checkout live
-    git merge task/t-123
-    # Conflicts in tasks.jsonl are handled automatically by .gitattributes driver
-    ```
-5.  Mark Done:
-    ```bash
-    task update t-123 done
-    git commit -am "task: t-123 done"
-    ```
+1.  **Harvest Updates**: Run `./Omni/Agent/harvest-tasks.sh` to pull "In Progress" and "Review" statuses from workers.
+2.  **Find Reviews**: Run `task list --status=review`.
+3.  **Review Code**:
+    *   Check out the feature branch: `git checkout task/t-123`.
+    *   Run tests and review code.
+4.  **Merge**:
+    *   `git checkout live`
+    *   `git merge task/t-123`
+5.  **Complete**:
+    *   `task update t-123 done`
+    *   `git commit -am "task: t-123 done"`
 
 ## Troubleshooting
 
-- 
cgit v1.2.3