docs(plan): align Task 6 with rebased HandleSuccess (preserve SetRoadblockCount)
This commit is contained in:
@@ -723,10 +723,9 @@ Expected: FAIL — parent goes `WaitingForReview` (routing not implemented), chi
|
|||||||
|
|
||||||
- [ ] **Step 3: Implement the routing**
|
- [ ] **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
|
```csharp
|
||||||
var finishedAt = DateTime.UtcNow;
|
|
||||||
var reviewResult = ComposeReviewResult(result.ResultMarkdown, result.Blocks);
|
var reviewResult = ComposeReviewResult(result.ResultMarkdown, result.Blocks);
|
||||||
bool isStandalone = task.ParentTaskId is null && task.PlanningPhase == PlanningPhase.None;
|
bool isStandalone = task.ParentTaskId is null && task.PlanningPhase == PlanningPhase.None;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user