-
Notifications
You must be signed in to change notification settings - Fork 2
Wire DagScheduler execution into /plan confirm flow #1434
Copy link
Copy link
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
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→ triesLlmAggregator::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
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request