Summary
/plan confirm currently attempts immediate aggregation without first executing the plan tasks through the DagScheduler. This causes aggregation failed: no completed or skipped tasks to aggregate because no tasks have been run.
Current Behavior
/plan goal <text> → LlmPlanner decomposes into TaskGraph (works)
/plan confirm → tries LlmAggregator::aggregate() immediately → fails because graph has 0 completed tasks
- Error message: "Plan confirmed. Execution and aggregation will run when the scheduler is wired."
Expected Behavior
/plan goal <text> → creates TaskGraph (already works)
/plan confirm → runs DagScheduler to execute tasks → then aggregates results
- Results shown to user
Root Cause
handle_plan_confirm() in agent/mod.rs:587 has a comment: "In a future integration phase the scheduler will run first; for now we aggregate whatever results are already present in the graph."
The DagScheduler (orchestration/scheduler.rs) and the AgentRouter (orchestration/router.rs) exist and are unit-tested but are not wired into the confirm flow.
Additional Issue
/plan list returns "No recent plans" even when a plan is awaiting confirmation. The pending graph is stored in self.pending_graph but /plan list apparently looks elsewhere.
Environment
- v0.14.1, orchestration.enabled=true, provider=claude haiku-4.5
Summary
/plan confirmcurrently attempts immediate aggregation without first executing the plan tasks through the DagScheduler. This causesaggregation failed: no completed or skipped tasks to aggregatebecause no tasks have been run.Current Behavior
/plan goal <text>→ LlmPlanner decomposes into TaskGraph (works)/plan confirm→ triesLlmAggregator::aggregate()immediately → fails because graph has 0 completed tasksExpected Behavior
/plan goal <text>→ creates TaskGraph (already works)/plan confirm→ runs DagScheduler to execute tasks → then aggregates resultsRoot Cause
handle_plan_confirm()inagent/mod.rs:587has a comment: "In a future integration phase the scheduler will run first; for now we aggregate whatever results are already present in the graph."The DagScheduler (
orchestration/scheduler.rs) and the AgentRouter (orchestration/router.rs) exist and are unit-tested but are not wired into the confirm flow.Additional Issue
/plan listreturns "No recent plans" even when a plan is awaiting confirmation. The pending graph is stored inself.pending_graphbut/plan listapparently looks elsewhere.Environment