UAT: `agents plan execute` rich output uses generic single panel instead of spec-required multi-panel layout #6126

New issue

Open

opened 2026-04-09 15:16:20 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-09 15:16:20 +00:00

Owner

Summary

agents plan execute <PLAN_ID> rich output uses the generic _print_lifecycle_plan() function which renders a single flat panel. The specification requires a structured multi-panel layout with 4 distinct panels plus a ✓ OK Execution started message.

Expected Behavior (from spec §agents plan execute)

The spec defines the following rich output panels:

╭─ Execution ──────────────────────╮
│ Plan: 01HXM8C2ZK4Q7C2B3F2R4VYV6J │
│ Phase: execute                   │
│ Sandbox: git_worktree            │
│ Worker: local/executor           │
│ Started: 12:58:10                │
│ Attempt: 1                       │
╰──────────────────────────────────╯

╭─ Sandbox ──────────────────────────────────╮
│ Strategy: git_worktree                     │
│ Path: /repos/api/.worktrees/plan-01HXM8    │
│ Branch: cleveragents/plan-01HXM8C2         │
│ Status: active                             │
╰────────────────────────────────────────────╯

╭─ Strategy Summary ─────────────────────╮
│ Decisions: 8                           │
│ Invariants: 2                          │
│ Planned Child Plans: 2+                │
│ Estimated Files: ~12                   │
│ Risk: low                              │
╰────────────────────────────────────────╯

╭─ Progress ─────────╮
│ ⏳ Collect context │
│ • Run tools        │
│ • Build changeset  │
│ • Validate         │
╰────────────────────╯

✓ OK Execution started

For a resumed execution, the spec shows additional panels:

Execution Resumed panel with Checkpoint and Resumed From fields
Previous Progress panel with step-by-step history
Guidance Applied panel (if guidance was provided)
✓ OK Execution resumed from checkpoint cp_...

Actual Behavior

The implementation calls _print_lifecycle_plan(plan, title="Plan Executed") which renders a single generic panel with all fields flattened. No separate Execution, Sandbox, Strategy Summary, or Progress panels. No ✓ OK Execution started message — only a dim hint like Plan execution completed (execute/complete). Run 'agents plan apply <id>' when ready.

Code Location

src/cleveragents/cli/commands/plan.py, execute_plan() function (~line 2465):

_print_lifecycle_plan(plan, title="Plan Executed")

Note: The JSON/YAML output for plan execute already uses the correct _execute_output_dict() envelope — only the rich output is affected.

Steps to Reproduce

agents plan execute <PLAN_ID>

Impact

Users cannot see sandbox details, strategy summary (decisions, invariants, risk), or execution progress at a glance. The UX is significantly degraded compared to the spec.

Subtasks

Implement _print_plan_executed_panels() helper with execute-specific panel rendering
Show Execution panel (Plan, Phase, Sandbox, Worker, Started, Attempt)
Show Sandbox panel (Strategy, Path, Branch, Status)
Show Strategy Summary panel (Decisions, Invariants, Planned Child Plans, Estimated Files, Risk)
Show Progress panel with ⏳/• indicators
Show Previous Progress panel for resumed executions
Print ✓ OK Execution started or ✓ OK Execution resumed from checkpoint ... message

Definition of Done

agents plan execute rich output shows all 4 panels as specified
Resumed execution shows additional panels as specified
✓ OK Execution started message is printed

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Summary `agents plan execute <PLAN_ID>` rich output uses the generic `_print_lifecycle_plan()` function which renders a single flat panel. The specification requires a structured multi-panel layout with 4 distinct panels plus a `✓ OK Execution started` message. ## Expected Behavior (from spec §agents plan execute) The spec defines the following rich output panels: ``` ╭─ Execution ──────────────────────╮ │ Plan: 01HXM8C2ZK4Q7C2B3F2R4VYV6J │ │ Phase: execute │ │ Sandbox: git_worktree │ │ Worker: local/executor │ │ Started: 12:58:10 │ │ Attempt: 1 │ ╰──────────────────────────────────╯ ╭─ Sandbox ──────────────────────────────────╮ │ Strategy: git_worktree │ │ Path: /repos/api/.worktrees/plan-01HXM8 │ │ Branch: cleveragents/plan-01HXM8C2 │ │ Status: active │ ╰────────────────────────────────────────────╯ ╭─ Strategy Summary ─────────────────────╮ │ Decisions: 8 │ │ Invariants: 2 │ │ Planned Child Plans: 2+ │ │ Estimated Files: ~12 │ │ Risk: low │ ╰────────────────────────────────────────╯ ╭─ Progress ─────────╮ │ ⏳ Collect context │ │ • Run tools │ │ • Build changeset │ │ • Validate │ ╰────────────────────╯ ✓ OK Execution started ``` For a resumed execution, the spec shows additional panels: - `Execution Resumed` panel with Checkpoint and Resumed From fields - `Previous Progress` panel with step-by-step history - `Guidance Applied` panel (if guidance was provided) - `✓ OK Execution resumed from checkpoint cp_...` ## Actual Behavior The implementation calls `_print_lifecycle_plan(plan, title="Plan Executed")` which renders a single generic panel with all fields flattened. No separate Execution, Sandbox, Strategy Summary, or Progress panels. No `✓ OK Execution started` message — only a dim hint like `Plan execution completed (execute/complete). Run 'agents plan apply <id>' when ready.` ## Code Location `src/cleveragents/cli/commands/plan.py`, `execute_plan()` function (~line 2465): ```python _print_lifecycle_plan(plan, title="Plan Executed") ``` Note: The JSON/YAML output for `plan execute` already uses the correct `_execute_output_dict()` envelope — only the rich output is affected. ## Steps to Reproduce ```bash agents plan execute <PLAN_ID> ``` ## Impact Users cannot see sandbox details, strategy summary (decisions, invariants, risk), or execution progress at a glance. The UX is significantly degraded compared to the spec. ## Subtasks - [ ] Implement `_print_plan_executed_panels()` helper with execute-specific panel rendering - [ ] Show `Execution` panel (Plan, Phase, Sandbox, Worker, Started, Attempt) - [ ] Show `Sandbox` panel (Strategy, Path, Branch, Status) - [ ] Show `Strategy Summary` panel (Decisions, Invariants, Planned Child Plans, Estimated Files, Risk) - [ ] Show `Progress` panel with ⏳/• indicators - [ ] Show `Previous Progress` panel for resumed executions - [ ] Print `✓ OK Execution started` or `✓ OK Execution resumed from checkpoint ...` message ## Definition of Done - `agents plan execute` rich output shows all 4 panels as specified - Resumed execution shows additional panels as specified - `✓ OK Execution started` message is printed --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 15:24:18 +00:00

HAL9000 referenced this issue

2026-04-09 15:39:15 +00:00

[AUTO-UAT-POOL] UAT Testing Report (Cycle 1) #5750