From 204b089000887530fc11a8e5622bbf9e6d838951 Mon Sep 17 00:00:00 2001 From: mika kuns Date: Thu, 4 Jun 2026 15:27:17 +0200 Subject: [PATCH] docs(plan): align Task 6 with rebased HandleSuccess (preserve SetRoadblockCount) --- .../plans/2026-06-04-child-tasks-and-improvement-loop.md | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/docs/superpowers/plans/2026-06-04-child-tasks-and-improvement-loop.md b/docs/superpowers/plans/2026-06-04-child-tasks-and-improvement-loop.md index 7c50e37..09c7661 100644 --- a/docs/superpowers/plans/2026-06-04-child-tasks-and-improvement-loop.md +++ b/docs/superpowers/plans/2026-06-04-child-tasks-and-improvement-loop.md @@ -723,10 +723,9 @@ Expected: FAIL — parent goes `WaitingForReview` (routing not implemented), chi - [ ] **Step 3: Implement the routing** -In `src/ClaudeDo.Worker/Runner/TaskRunner.cs`, replace the status-setting block in `HandleSuccess` (lines 337-350, the `if (task.ParentTaskId is null && task.PlanningPhase == PlanningPhase.None)` … `else` … block) with: +In `src/ClaudeDo.Worker/Runner/TaskRunner.cs` (post-rebase), `HandleSuccess` already declares `var finishedAt` and calls `SetRoadblockCountAsync` ABOVE the routing block — KEEP both. Replace ONLY the `var reviewResult = …;` line plus the `if (task.ParentTaskId is null && task.PlanningPhase == PlanningPhase.None)` … `else` block (the routing decision) with: ```csharp - var finishedAt = DateTime.UtcNow; var reviewResult = ComposeReviewResult(result.ResultMarkdown, result.Blocks); bool isStandalone = task.ParentTaskId is null && task.PlanningPhase == PlanningPhase.None;