sdd-pipeline 1.2.3 → 1.2.6

package/lib/bundle/.claude/commands/sdd-cook.md

@@ -1,177 +1,197 @@
 ---
-# SDD Cook — Phase 4 Implementation
-# Executes tasks from MASTER-TASKS.md sequentially
-# Validates with subagents, updates task status
+# SDD Cook — Phase 4 Implementation Command
+# Parses MASTER-TASKS.md and executes implementation
 ---
 tool: Bash
 when: always
 args: optional
-description: Phase 4 — Execute tasks from MASTER-TASKS.md sequentially. Args: [--all | --task T-XXX | --no-test | --no-review]. Validates dependencies, runs testing + code-review, updates task status.
+description: Phase 4 — Execute tasks from MASTER-TASKS.md. Args: [--all | --task T-XXX] [--auto-mark] [--dry-run]. Implements tasks and marks them complete.
 ---
-$ErrorActionPreference = 'Stop'
+$ErrorActionPreference = 'Continue'
 
 # ============================================================
-# SDD Cook — Phase 4 Implementation
+# SDD Cook Command — Phase 4
 # ============================================================
-# Reads MASTER-TASKS.md, executes tasks sequentially,
-# validates with subagents, updates task status
+# This command:
+# 1. Loads tasks from MASTER-TASKS.md
+# 2. Shows execution plan
+# 3. Provides commands to implement each task
+# 4. Auto-marks tasks complete after implementation (with --auto-mark)
 # ============================================================
 
 # --- Parse arguments ---
 $rawArgs = $args -join ' '
 $targetTask = ""
-$skipTest = $false
-$skipReview = $false
 $executeAll = $false
+$autoMark = $false
+$dryRun = $false
 
+# Parse flags
 if ($rawArgs -match "--task\s+(\S+)") {
     $targetTask = $matches[1]
 }
 if ($rawArgs -match "--all") {
     $executeAll = $true
 }
-if ($rawArgs -match "--no-test") {
-    $skipTest = $true
+if ($rawArgs -match "--auto-mark") {
+    $autoMark = $true
 }
-if ($rawArgs -match "--no-review") {
-    $skipReview = $true
+if ($rawArgs -match "--dry-run") {
+    $dryRun = $true
 }
 
 # ============================================================
-# STEP 1: Prerequisites Check
+# Prerequisites Check
 # ============================================================
 Write-Host ""
-Write-Host "=== SDD Cook — Phase 4 Implementation ===" -ForegroundColor Cyan
+Write-Host "========================================" -ForegroundColor Cyan
+Write-Host "  SDD Cook — Phase 4 Implementation" -ForegroundColor Cyan
+Write-Host "========================================" -ForegroundColor Cyan
 Write-Host ""
 
-# Check for task directory
-$taskDirs = Get-ChildItem -Filter "task-*-*" -Directory -ErrorAction SilentlyContinue
+# SDD output paths
+$SDD_OUTPUT_BASE = "sdd"
+$SDD_TASKS_DIR = "$SDD_OUTPUT_BASE/tasks"
+
+# Find task directory
+$taskDirs = @()
+if (Test-Path $SDD_TASKS_DIR) {
+    $taskDirs = Get-ChildItem -Path $SDD_TASKS_DIR -Filter "task-*-*" -Directory -ErrorAction SilentlyContinue
+}
 if (-not $taskDirs) {
-    Write-Host "[ERROR] No task directory found (task-*-*/)." -ForegroundColor Red
-    Write-Host "Run: /sdd-tasks to generate tasks first." -ForegroundColor Yellow
+    $taskDirs = Get-ChildItem -Filter "task-*-*" -Directory -ErrorAction SilentlyContinue
+}
+
+if (-not $taskDirs) {
+    Write-Host "[ERROR] No task directory found." -ForegroundColor Red
+    Write-Host "Run: claude `"/sdd-tasks`" first." -ForegroundColor Yellow
     exit 1
 }
 
 $taskDir = $taskDirs[0].FullName
 $taskDirName = $taskDirs[0].Name
 
-# Check MASTER-TASKS.md exists
+# Check MASTER-TASKS.md
 $masterTasksPath = Join-Path $taskDir "MASTER-TASKS.md"
 if (-not (Test-Path $masterTasksPath)) {
-    Write-Host "[ERROR] MASTER-TASKS.md not found: $masterTasksPath" -ForegroundColor Red
-    Write-Host "Run: /sdd-tasks to generate tasks." -ForegroundColor Yellow
+    Write-Host "[ERROR] MASTER-TASKS.md not found." -ForegroundColor Red
     exit 1
 }
 
-# Check SPEC.md exists
-if (-not (Test-Path "SPEC.md")) {
+# Check SPEC.md
+$specPath = "$SDD_OUTPUT_BASE/SPEC.md"
+$specFile = if (Test-Path $specPath) { $specPath } else { "SPEC.md" }
+if (-not (Test-Path $specFile)) {
     Write-Host "[ERROR] SPEC.md not found." -ForegroundColor Red
-    Write-Host "Run: /sdd-spec to generate SPEC.md first." -ForegroundColor Yellow
+    Write-Host "Run: claude `"/sdd-spec`" first." -ForegroundColor Yellow
     exit 1
 }
 
-Write-Host "[OK] Prerequisites verified:" -ForegroundColor Green
-Write-Host "  - Task directory: $taskDirName" -ForegroundColor Gray
-Write-Host "  - MASTER-TASKS.md: Found" -ForegroundColor Gray
-Write-Host "  - SPEC.md: Found" -ForegroundColor Gray
+Write-Host "[OK] Prerequisites verified" -ForegroundColor Green
+Write-Host "  SPEC.md: $specFile" -ForegroundColor Gray
+Write-Host "  Task dir: $taskDirName" -ForegroundColor Gray
 Write-Host ""
 
 # ============================================================
-# STEP 2: Read MASTER-TASKS.md
+# Load Tasks
 # ============================================================
 Write-Host "--- Loading Tasks ---" -ForegroundColor Yellow
 
 $masterContent = Get-Content $masterTasksPath -Raw
 
-# Parse tasks from table
 $taskPattern = '\|\s*(T-\d+)\s*\|\s*([^\|]+)\s*\|\s*([^\|]+)\s*\|\s*\[([^\]]+)\]\s*\|'
 $tasks = @()
 if ($masterContent -match $taskPattern) {
     $matches = [regex]::Matches($masterContent, $taskPattern)
     foreach ($m in $matches) {
-        $task = @{
+        $tasks += @{
             Id = $m.Groups[1].Value
             Name = $m.Groups[2].Value.Trim()
             Estimate = $m.Groups[3].Value.Trim()
             Status = $m.Groups[4].Value.Trim()
         }
-        $tasks += $task
     }
 }
 
 if ($tasks.Count -eq 0) {
-    Write-Host "[ERROR] No tasks found in MASTER-TASKS.md" -ForegroundColor Red
+    Write-Host "[ERROR] No tasks found." -ForegroundColor Red
     exit 1
 }
 
+# Helper function to check if task is complete
+function Test-TaskComplete($status) {
+    return $status -in @("x", "X", "v", "V", "done", "completed")
+}
+
+Write-Host ""
 Write-Host "Found $($tasks.Count) tasks:" -ForegroundColor Green
+$completedCount = 0
 foreach ($t in $tasks) {
-    $statusIcon = if ($t.Status -eq "x") { "[V]" } else { "[ ]" }
-    $color = if ($t.Status -eq "x") { "Green" } else { "Gray" }
+    $isComplete = Test-TaskComplete $t.Status
+    if ($isComplete) { $completedCount++ }
+    $statusIcon = if ($isComplete) { "[V]" } else { "[ ]" }
+    $color = if ($isComplete) { "Green" } else { "Gray" }
     Write-Host "  $statusIcon $($t.Id): $($t.Name) ($($t.Estimate))" -ForegroundColor $color
 }
 Write-Host ""
+Write-Host "Progress: $completedCount/$($tasks.Count) complete" -ForegroundColor Cyan
+Write-Host ""
 
 # ============================================================
-# STEP 3: Find tasks to execute
+# Select Tasks to Execute
 # ============================================================
-$pendingTasks = $tasks | Where-Object { $_.Status -ne "x" }
+$pendingTasks = $tasks | Where-Object { -not (Test-TaskComplete $_.Status) }
 $tasksToExecute = @()
 
 if ($targetTask) {
-    # Execute specific task
     $specific = $pendingTasks | Where-Object { $_.Id -eq $targetTask }
     if ($specific) {
         $tasksToExecute += $specific
-        Write-Host "Executing specific task: $targetTask" -ForegroundColor Cyan
+        Write-Host "Mode: specific - Task $targetTask" -ForegroundColor Cyan
     } else {
         $found = $tasks | Where-Object { $_.Id -eq $targetTask }
         if ($found) {
-            Write-Host "[WARN] Task $targetTask is already complete." -ForegroundColor Yellow
+            Write-Host "[INFO] Task $targetTask already complete." -ForegroundColor Green
         } else {
             Write-Host "[ERROR] Task $targetTask not found." -ForegroundColor Red
         }
         exit 0
     }
 } elseif ($executeAll) {
-    # Execute all pending in order
     $tasksToExecute = $pendingTasks
-    Write-Host "Executing all pending tasks ($($tasksToExecute.Count) tasks)..." -ForegroundColor Cyan
+    Write-Host "Mode: all - $($tasksToExecute.Count) tasks" -ForegroundColor Cyan
 } else {
-    # Default: execute first pending task
     if ($pendingTasks.Count -gt 0) {
         $tasksToExecute += $pendingTasks[0]
-        Write-Host "Executing next pending task: $($pendingTasks[0].Id)" -ForegroundColor Cyan
+        Write-Host "Mode: next pending task" -ForegroundColor Cyan
     } else {
-        Write-Host "[INFO] All tasks are complete!" -ForegroundColor Green
+        Write-Host "[INFO] All tasks complete!" -ForegroundColor Green
         Write-Host ""
-        Write-Host "Next: /sdd-converge to validate implementation" -ForegroundColor Cyan
+        Write-Host "Next: claude `"/sdd-converge`"" -ForegroundColor Cyan
         exit 0
     }
 }
 
+Write-Host ""
+Write-Host "Tasks to execute: $($tasksToExecute.Count)" -ForegroundColor Green
+
 # ============================================================
-# STEP 4: Read task files and execute
+# Generate Execution Plan
 # ============================================================
 Write-Host ""
-Write-Host "--- Task Execution ---" -ForegroundColor Yellow
+Write-Host "========================================" -ForegroundColor Cyan
+Write-Host "  SDD Cook Execution Plan" -ForegroundColor Cyan
+Write-Host "========================================" -ForegroundColor Cyan
 Write-Host ""
 
-$completedCount = 0
-$totalTasks = $tasks.Count
-$completedBefore = ($tasks | Where-Object { $_.Status -eq "x" }).Count
+$taskNumber = 1
+$executionPlan = @()
 
 foreach ($task in $tasksToExecute) {
-    Write-Host ""
-    Write-Host "========================================" -ForegroundColor Magenta
-    Write-Host "Executing: $($task.Id) - $($task.Name)" -ForegroundColor Magenta
-    Write-Host "========================================" -ForegroundColor Magenta
-
     # Find task file
     $taskFiles = Get-ChildItem -Path $taskDir -Filter "$($task.Id)-*.md" -File
     if (-not $taskFiles) {
         Write-Host "[WARN] Task file not found: $($task.Id)-*.md" -ForegroundColor Yellow
-        Write-Host "Skipping $($task.Id)..." -ForegroundColor Yellow
         continue
     }
 
@@ -187,27 +207,9 @@ foreach ($task in $tasksToExecute) {
         }
     }
 
-    # Check dependencies
-    if ($dependsOn.Count -gt 0) {
-        Write-Host "Dependencies: $($dependsOn -join ', ')" -ForegroundColor Gray
-        $depsMet = $true
-        foreach ($dep in $dependsOn) {
-            $depTask = $tasks | Where-Object { $_.Id -eq $dep }
-            if ($depTask -and $depTask.Status -ne "x") {
-                Write-Host "[BLOCKED] $($task.Id) depends on $dep which is not complete." -ForegroundColor Red
-                $depsMet = $false
-                break
-            }
-        }
-        if (-not $depsMet) {
-            Write-Host "Skipping $($task.Id) - dependency not met." -ForegroundColor Yellow
-            continue
-        }
-    }
-
-    # Extract files to modify
+    # Extract files
     $filesToModify = @()
-    if ($taskContent -match '## Files to Modify/Create\s*\n((?:- .+\n)+)') {
+    if ($taskContent -match '(?s)## Files to (?:Modify|Create)\s*\n((?:- .+\n)+)') {
         $filesBlock = $matches[1]
         $files = $filesBlock -split '\n' | Where-Object { $_ -match '^\s*-\s+' }
         foreach ($f in $files) {
@@ -216,120 +218,117 @@ foreach ($task in $tasksToExecute) {
         }
     }
 
-    # Extract implementation steps
-    $implSteps = @()
-    if ($taskContent -match '## Implementation Steps\s*\n((?:\d+\. .+\n)+)') {
-        $stepsBlock = $matches[1]
-        $implSteps = $stepsBlock -split '\n' | Where-Object { $_ -match '^\d+\.' }
+    # Extract implementation summary
+    $implSummary = ""
+    if ($taskContent -match '(?s)## Implementation Steps\s*\n((?:\d+\. .+\n?)+)') {
+        $implSummary = "Steps: " + ($matches[1] -replace '\n', ' | ').Trim()
     }
 
-    # Extract verification checks
-    $verificationChecks = @()
-    if ($taskContent -match '## Verification\s*\n((?:- \[ \] .+\n)+)') {
-        $checksBlock = $matches[1]
-        $verificationChecks = $checksBlock -split '\n' | Where-Object { $_ -match '- \[ \]' }
+    $planItem = @{
+        Id = $task.Id
+        Name = $task.Name
+        File = $taskFile
+        Dependencies = $dependsOn
+        Files = $filesToModify
+        Summary = $implSummary
     }
+    $executionPlan += $planItem
 
-    Write-Host ""
-    Write-Host "Task Details:" -ForegroundColor Cyan
-    Write-Host "  Files: $($filesToModify -join ', ')" -ForegroundColor Gray
-    Write-Host "  Verification checks: $($verificationChecks.Count)" -ForegroundColor Gray
+    Write-Host "--------------------------------------------" -ForegroundColor Magenta
+    Write-Host "TASK $($task.Id): $($task.Name)" -ForegroundColor Magenta
+    Write-Host "--------------------------------------------" -ForegroundColor Magenta
+    Write-Host "  File: $taskFile" -ForegroundColor Gray
 
-    # ============================================================
-    # IMPLEMENTATION
-    # ============================================================
-    Write-Host ""
-    Write-Host "--- Implementation ---" -ForegroundColor Yellow
-    Write-Host "Executing implementation for $($task.Id)..." -ForegroundColor Green
-    Write-Host ""
-    Write-Host "[HINT] To implement this task:" -ForegroundColor Cyan
-    Write-Host "  1. Read SPEC.md for requirements" -ForegroundColor Gray
-    Write-Host "  2. Read $taskFile" -ForegroundColor Gray
-    Write-Host "  3. Implement changes per steps" -ForegroundColor Gray
-    Write-Host ""
-    Write-Host "[HINT] Files to modify:" -ForegroundColor Cyan
-    foreach ($f in $filesToModify) {
-        Write-Host "  - $f" -ForegroundColor Gray
-    }
-    Write-Host ""
-    Write-Host "[HINT] Implementation steps:" -ForegroundColor Cyan
-    foreach ($step in $implSteps) {
-        Write-Host "  $step" -ForegroundColor Gray
+    if ($dependsOn.Count -gt 0) {
+        Write-Host "  Dependencies: $($dependsOn -join ', ')" -ForegroundColor Yellow
     }
 
-    # ============================================================
-    # TESTING (if not skipped)
-    # ============================================================
-    if (-not $skipTest) {
-        Write-Host ""
-        Write-Host "--- Testing ---" -ForegroundColor Yellow
-        Write-Host "Verification checks:" -ForegroundColor Cyan
-        foreach ($check in $verificationChecks) {
-            $checkText = $check -replace '^\s*-\s+\[ \]\s*', ''
-            Write-Host "  [ ] $checkText" -ForegroundColor Gray
-        }
-        Write-Host ""
-        Write-Host "[HINT] Run tests and verify each check." -ForegroundColor Cyan
-        Write-Host "Mark checks complete after verification." -ForegroundColor Gray
+    if ($filesToModify.Count -gt 0) {
+        Write-Host "  Files: $($filesToModify -join ', ')" -ForegroundColor Gray
     }
 
-    # ============================================================
-    # CODE REVIEW (if not skipped)
-    # ============================================================
-    if (-not $skipReview) {
-        Write-Host ""
-        Write-Host "--- Code Review ---" -ForegroundColor Yellow
-        Write-Host "[HINT] After implementation, run: /ck:code-review" -ForegroundColor Cyan
-        Write-Host "[HINT] Verify:" -ForegroundColor Cyan
-        Write-Host "  - Spec compliance (SPEC.md requirements met)" -ForegroundColor Gray
-        Write-Host "  - No regression in touched files" -ForegroundColor Gray
-        Write-Host "  - Code quality and patterns" -ForegroundColor Gray
+    if ($implSummary) {
+        Write-Host "  $implSummary" -ForegroundColor Gray
     }
-
-    # ============================================================
-    # MANUAL CONFIRMATION
-    # ============================================================
-    Write-Host ""
-    Write-Host "========================================" -ForegroundColor Magenta
-    Write-Host "After completing $($task.Id), update status?" -ForegroundColor Yellow
-    Write-Host "========================================" -ForegroundColor Magenta
     Write-Host ""
-    Write-Host "To mark complete, run:" -ForegroundColor Cyan
-    Write-Host '  /sdd-task-status ' + $task.Id + ' done' -ForegroundColor Gray
-    Write-Host ""
-
-    # Update MASTER-TASKS.md
-    $masterContent = Get-Content $masterTasksPath -Raw
-    $newStatus = "[x]"
-    $updatedContent = $masterContent -replace "(`|$([regex]::Escape($task.Id))`|\s)\[([ `])\](?=\s*\|)", "`$1[x]"
-    Set-Content $masterTasksPath $updatedContent -Encoding UTF8
 
-    Write-Host "[OK] Status updated in MASTER-TASKS.md" -ForegroundColor Green
-    $completedCount++
+    $taskNumber++
 }
 
 # ============================================================
-# STEP 5: Summary
+# Implementation Commands
 # ============================================================
-Write-Host ""
 Write-Host "========================================" -ForegroundColor Cyan
-Write-Host "SDD Cook Execution Complete" -ForegroundColor Cyan
+Write-Host "  Implementation Commands" -ForegroundColor Cyan
 Write-Host "========================================" -ForegroundColor Cyan
 Write-Host ""
-Write-Host "Tasks completed this session: $completedCount" -ForegroundColor Green
-$completedAfter = $completedBefore + $completedCount
-Write-Host "Total progress: $completedAfter / $totalTasks" -ForegroundColor Gray
 
-$remaining = $totalTasks - $completedAfter
-if ($remaining -gt 0) {
-    Write-Host "Remaining: $remaining tasks" -ForegroundColor Yellow
+if ($dryRun) {
+    Write-Host "[DRY RUN] No changes will be made." -ForegroundColor Yellow
     Write-Host ""
-    Write-Host "Next: /sdd-cook --all to continue" -ForegroundColor Cyan
-} else {
+}
+
+Write-Host "To implement each task, run these commands in order:" -ForegroundColor White
+Write-Host ""
+
+$cmdNumber = 1
+foreach ($plan in $executionPlan) {
+    Write-Host "  $cmdNumber. Implement $($plan.Id): $($plan.Name)" -ForegroundColor Cyan
+    Write-Host "     claude `"/sdd-implement $($plan.Id)`"" -ForegroundColor Gray
     Write-Host ""
-    Write-Host "[V] All tasks complete!" -ForegroundColor Green
+    $cmdNumber++
+}
+
+Write-Host "  $cmdNumber. After implementation, mark tasks complete:" -ForegroundColor White
+Write-Host "     pwsh commands/sdd-task-status.ps1 -TaskId <T-XXX> -Status completed" -ForegroundColor Gray
+Write-Host ""
+
+# ============================================================
+# Auto-Mark Mode (if --auto-mark)
+# ============================================================
+if ($autoMark -and -not $dryRun) {
+    Write-Host "========================================" -ForegroundColor Green
+    Write-Host "  Auto-Mark Mode Enabled" -ForegroundColor Green
+    Write-Host "========================================" -ForegroundColor Green
     Write-Host ""
-    Write-Host "Next: /sdd-converge to validate implementation" -ForegroundColor Cyan
+    Write-Host "Tasks will be marked complete after implementation." -ForegroundColor White
+    Write-Host "Run each implementation command, then:" -ForegroundColor White
+    Write-Host ""
+
+    foreach ($plan in $executionPlan) {
+        Write-Host "  After implementing $($plan.Id):" -ForegroundColor Cyan
+        Write-Host "    pwsh commands/sdd-task-status.ps1 -TaskId '$($plan.Id)' -Status completed" -ForegroundColor Gray
+        Write-Host ""
+    }
+}
+
+# ============================================================
+# Quick Execute Mode (non-interactive)
+# ============================================================
+if (-not $dryRun) {
+    Write-Host "========================================" -ForegroundColor Cyan
+    Write-Host "  Quick Execute" -ForegroundColor Cyan
+    Write-Host "========================================" -ForegroundColor Cyan
+    Write-Host ""
+
+    # Show one-liner for executing all pending tasks
+    $taskIds = ($executionPlan | ForEach-Object { $_.Id }) -join ","
+    Write-Host "To execute all shown tasks:" -ForegroundColor White
+    Write-Host "  claude `"/sdd-implement --all`"" -ForegroundColor Gray
+    Write-Host ""
+
+    # Check if skills are available
+    $skillPath = "skills/sdd-cook/SKILL.md"
+    if (Test-Path $skillPath) {
+        Write-Host "[OK] Skill available: skills/sdd-cook/SKILL.md" -ForegroundColor Green
+        Write-Host "  Run: claude `"/sdd-implement $($executionPlan[0].Id)`"" -ForegroundColor Gray
+    } else {
+        Write-Host "[WARN] Skill not found: $skillPath" -ForegroundColor Yellow
+        Write-Host "  Run 'sdd init' to copy skills." -ForegroundColor Gray
+    }
 }
 
 Write-Host ""
+Write-Host "========================================" -ForegroundColor Cyan
+Write-Host "Ready to implement $($tasksToExecute.Count) task(s)" -ForegroundColor Green
+Write-Host "========================================" -ForegroundColor Cyan