releases/ClaudeDo
2
0
Fork 0
Code Issues 1 Pull Requests 2 Actions Packages Projects Releases 12 Wiki Activity

docs(plan): align Task 6 with rebased HandleSuccess (preserve SetRoadblockCount)

Browse Source
This commit is contained in:
mika kuns
2026-06-04 15:27:17 +02:00
parent da4ab0ca5e
commit 204b089000
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
docs/superpowers/plans/2026-06-04-child-tasks-and-improvement-loop.md
View File

@@ -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;