claude-cook 1.10.7 → 1.10.8

package/cook/workflows/pm-check.md

@@ -97,6 +97,7 @@ Aggregate events:
 | --------- | --------------- |
 | All wave N tickets are `done` | WAVE_COMPLETE |
 | All tickets across all waves are `done` | PHASE_COMPLETE |
+| Tickets in `inreview` awaiting review | NEEDS_REVIEW |
 | No changes detected | NO_CHANGE |
 
 </step>
@@ -129,11 +130,25 @@ Run /cook:pm-start {X+1} to begin next phase.
 4. If false: log and present suggestion to user
 5. Update STATE.md wave summary
 
-### WORKER_COMPLETED / WORKER_AUTO_COMPLETED
+### WORKER_COMPLETED (inprogress → inreview)
 1. Call `get_task(task_id)` to read worker output/notes
-2. Update TICKET-MAP.md: status=done, completed=timestamp
+2. Update TICKET-MAP.md: status=inreview, review_started=timestamp
+3. Trigger automated code review:
+   - Spawn `feature-dev:code-reviewer` agent via Task tool
+   - Provide: ticket title, acceptance criteria from ticket description, recent commits (use `git log --oneline -10` to find relevant commits)
+   - The reviewer examines the diff and returns a verdict: **PASS** or **FAIL** with issues
+4. Based on review result:
+   - **PASS:** Update ticket status to `done` via `mcp__vibe_kanban__update_task(task_id, status="done")`. Update TICKET-MAP.md: status=done, completed=timestamp, review=passed. Check if this completes the wave.
+   - **FAIL:** Update ticket status back to `inprogress` via `mcp__vibe_kanban__update_task(task_id, status="inprogress")`. Update TICKET-MAP.md: status=inprogress, review=failed. Append review feedback as a task note via `mcp__vibe_kanban__update_task(task_id, description=original_description + "\n\n## Review Feedback\n\n" + reviewer_issues)`. Re-dispatch the ticket so the worker can address review feedback.
+5. Log review result to PM-LOG.md
+
+### WORKER_AUTO_COMPLETED (inprogress → done, bypass review)
+1. Call `get_task(task_id)` to read worker output/notes
+2. Update TICKET-MAP.md: status=done, completed=timestamp, review=skipped
 3. Check if this completes the wave
 
+> **Note:** `inprogress → done` skips review (worker or human marked it done directly). To enforce review, workers should transition to `inreview` instead of `done`.
+
 ### TICKET_CANCELLED (potential failure)
 1. Call `get_task(task_id)` to read cancellation reason
 2. Check `pm.auto_replan_on_failure` in config

package/cook/workflows/pm-cycle.md

@@ -13,6 +13,7 @@ NEEDS_PLANNING    → No PLAN.md files
 NEEDS_SYNC        → Plans exist, no TICKET-MAP.md
 NEEDS_DISPATCH    → TICKET-MAP has todo tickets in ready wave
 MONITORING        → Tickets inprogress
+NEEDS_REVIEW      → Tickets inreview (awaiting code review)
 PHASE_COMPLETE    → All tickets done, more phases in roadmap
 MILESTONE_COMPLETE → All tickets done, last phase in roadmap
 ```
@@ -24,9 +25,12 @@ NEEDS_PLANNING ──spawn planner──→ NEEDS_SYNC
 NEEDS_SYNC ──create tickets──→ NEEDS_DISPATCH
 NEEDS_DISPATCH ──launch workers──→ MONITORING
 MONITORING ──poll + react──→ MONITORING (loop)
+                          ──→ NEEDS_REVIEW (worker completed, tickets inreview)
                           ──→ NEEDS_DISPATCH (wave complete, next wave ready)
                           ──→ PHASE_COMPLETE (all done)
                           ──→ MONITORING + replan (ticket failed)
+NEEDS_REVIEW ──code review──→ MONITORING (review passed → done, check wave)
+                            ──→ MONITORING (review failed → re-dispatch)
 PHASE_COMPLETE ──advance phase──→ NEEDS_PLANNING (next phase)
                               ──→ MILESTONE_COMPLETE (last phase)
 MILESTONE_COMPLETE ──stop signal──→ EXIT
@@ -92,17 +96,42 @@ Follow `pm-check.md` workflow:
 | Event | Reaction |
 |-------|----------|
 | Ticket: todo → inprogress | Update TICKET-MAP (worker started) |
-| Ticket: inprogress → done | Update TICKET-MAP, check wave completion |
-| Ticket: inprogress → inreview | Update TICKET-MAP (awaiting review) |
+| Ticket: inprogress → inreview | Update TICKET-MAP, transition to NEEDS_REVIEW |
+| Ticket: inprogress → done | Update TICKET-MAP (review skipped), check wave completion |
 | Ticket: any → cancelled | If auto_replan: run pm-replan (TARGETED). Else: log warning |
 | Wave complete | If auto_dispatch_next_wave: dispatch next wave. Else: log |
 | All tickets done | Mark as PHASE_COMPLETE (caught next cycle) |
-| No changes | Log: "No changes. {N} inprogress, {M} todo." |
+| No changes | Log: "No changes. {N} inprogress, {M} inreview, {O} todo." |
 
 4. Update TICKET-MAP.md + STATE.md + PM-LOG.md
 
 **Stuck detection:** If a ticket has been `inprogress` for longer than 30 minutes (configurable), log a warning. After 60 minutes, consider re-dispatching with a different executor.
 
+### NEEDS_REVIEW
+
+**Goal:** Run automated code review on tickets that workers have completed.
+
+1. Find all tickets with status `inreview` in TICKET-MAP.md
+2. For each `inreview` ticket:
+   a. Read ticket details via `mcp__vibe_kanban__get_task(task_id)`
+   b. Spawn `feature-dev:code-reviewer` agent via Task tool:
+      - Provide: ticket title, acceptance criteria, recent git commits
+      - Reviewer examines the diff for bugs, security issues, and code quality
+   c. Based on review verdict:
+      - **PASS:** Update ticket → `done` via `mcp__vibe_kanban__update_task(task_id, status="done")`. Update TICKET-MAP: status=done, review=passed.
+      - **FAIL:** Update ticket → `inprogress` via `mcp__vibe_kanban__update_task(task_id, status="inprogress")`. Append review feedback to ticket description. Re-dispatch worker to address feedback.
+3. After all reviews processed:
+   - Check wave completion (all done → dispatch next wave or PHASE_COMPLETE)
+4. Log review results to PM-LOG.md:
+   ```markdown
+   ## [{timestamp}] CODE_REVIEW
+
+   - Reviewed {N} tickets
+   - Passed: {list}
+   - Failed: {list with issue summaries}
+   - Actions: {re-dispatched / wave advanced / etc.}
+   ```
+
 ### PHASE_COMPLETE
 
 **Goal:** Advance to next phase in roadmap.

package/cook/workflows/pm-sync.md

@@ -176,10 +176,10 @@ Update the Wave Summary table:
 ```markdown
 ## Wave Summary
 
-| Wave | Total | Done | In Progress | Todo | Failed |
-| ---- | ----- | ---- | ----------- | ---- | ------ |
-| 1    | 3     | 0    | 0           | 3    | 0      |
-| 2    | 2     | 0    | 0           | 2    | 0      |
+| Wave | Total | Done | In Review | In Progress | Todo | Failed |
+| ---- | ----- | ---- | --------- | ----------- | ---- | ------ |
+| 1    | 3     | 0    | 0         | 0           | 3    | 0      |
+| 2    | 2     | 0    | 0         | 0           | 2    | 0      |
 ```
 
 Update `last_polled` timestamp.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-cook",
-  "version": "1.10.7",
+  "version": "1.10.8",
   "description": "A meta-prompting, context engineering and spec-driven development system for Claude Code, OpenCode and Gemini by TÂCHES.",
   "bin": {
     "claude-cook": "bin/install.js"

package/scripts/pm-loop.sh

@@ -174,12 +174,13 @@ print_progress() {
 
   if [ -n "$ticket_map" ] && [ -f "$ticket_map" ]; then
     local done_count=$(grep -c "| done " "$ticket_map" 2>/dev/null || echo "0")
+    local inreview_count=$(grep -c "| inreview " "$ticket_map" 2>/dev/null || echo "0")
     local inprogress_count=$(grep -c "| inprogress " "$ticket_map" 2>/dev/null || echo "0")
     local todo_count=$(grep -c "| todo " "$ticket_map" 2>/dev/null || echo "0")
-    local total=$((done_count + inprogress_count + todo_count))
+    local total=$((done_count + inreview_count + inprogress_count + todo_count))
 
     if [ "$total" -gt 0 ]; then
-      # Build progress bar
+      # Build progress bar (done + inreview count toward completion)
       local pct=0
       if [ "$total" -gt 0 ]; then
         pct=$((done_count * 100 / total))
@@ -191,10 +192,14 @@ print_progress() {
       for ((i=0; i<filled; i++)); do bar+="█"; done
       for ((i=0; i<empty; i++)); do bar+="░"; done
 
-      echo "  ┌─────────────────────────────────────────┐"
+      echo "  ┌──────────────────────────────────────────────────────┐"
       echo "  │  Progress: [$bar] ${pct}%"
-      echo "  │  Done: $done_count  Running: $inprogress_count  Todo: $todo_count  Total: $total"
-      echo "  └─────────────────────────────────────────┘"
+      if [ "$inreview_count" -gt 0 ]; then
+        echo "  │  Done: $done_count  Review: $inreview_count  Running: $inprogress_count  Todo: $todo_count"
+      else
+        echo "  │  Done: $done_count  Running: $inprogress_count  Todo: $todo_count  Total: $total"
+      fi
+      echo "  └──────────────────────────────────────────────────────┘"
     fi
   fi
 }
@@ -206,11 +211,12 @@ all_tickets_done() {
 
   if [ -n "$ticket_map" ] && [ -f "$ticket_map" ]; then
     local done_count=$(grep -c "| done " "$ticket_map" 2>/dev/null || echo "0")
+    local inreview_count=$(grep -c "| inreview " "$ticket_map" 2>/dev/null || echo "0")
     local inprogress_count=$(grep -c "| inprogress " "$ticket_map" 2>/dev/null || echo "0")
     local todo_count=$(grep -c "| todo " "$ticket_map" 2>/dev/null || echo "0")
-    local total=$((done_count + inprogress_count + todo_count))
+    local total=$((done_count + inreview_count + inprogress_count + todo_count))
 
-    if [ "$total" -gt 0 ] && [ "$inprogress_count" -eq 0 ] && [ "$todo_count" -eq 0 ]; then
+    if [ "$total" -gt 0 ] && [ "$inreview_count" -eq 0 ] && [ "$inprogress_count" -eq 0 ] && [ "$todo_count" -eq 0 ]; then
       return 0
     fi
   fi