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

New issue

Open

opened 2026-04-09 14:28:15 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-09 14:28:15 +00:00

Owner

Summary

agents plan apply <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 5 distinct panels plus a success/error message.

Expected Behavior (from spec §agents plan apply)

The spec defines the following rich output panels for a successful apply:

Apply changes for plan 01HXM8C2ZK4Q7C2B3F2R4VYV6J? [y/N]: y

╭─ Apply Summary ─────────────────────╮
│ Plan: 01HXM8C2ZK4Q7C2B3F2R4VYV6J    │
│ Artifacts: 6 files updated          │
│ Changes: 42 insertions, 9 deletions │
│ Project: local/api-service          │
│ Applied At: 2026-02-08 13:04        │
╰─────────────────────────────────────╯

╭─ Validation (from Execute) ────╮
│ Tests: passed (24/24)          │
│ Lint: passed (0 warnings)      │
│ Type Check: passed (0 errors)  │
│ Duration: 12.4s                │
╰────────────────────────────────╯

╭─ Sandbox Cleanup ─────────╮
│ Worktree: removed         │
│ Branch: merged to main    │
│ Checkpoint: archived      │
╰───────────────────────────╯

╭─ Plan Lifecycle ────────────────────────╮
│ Phase: apply                            │
│ State: applied                          │
│ Total Duration: 00:06:14                │
│ Total Cost: $0.0847                     │
│ Decisions Made: 8                       │
│ Child Plans: 2 (completed)              │
╰─────────────────────────────────────────╯

╭─ Next Steps ──────╮
│ - Review git diff │
│ - Commit changes  │
╰───────────────────╯

✓ OK Changes applied

For a failed apply (validation failures), the spec shows:

Apply Summary panel
Validation panel with ✗ failures listed
Sandbox Status panel (preserved, not cleaned up)
Recovery Options panel
✗ ERROR Apply refused — N required Execute-phase validations did not pass

Actual Behavior

The implementation calls _print_lifecycle_plan(plan, title="Plan Applied") which renders a single generic panel. No separate Apply Summary, Validation, Sandbox Cleanup, Plan Lifecycle, or Next Steps panels. No ✓ OK Changes applied message.

Code Location

src/cleveragents/cli/commands/plan.py, lifecycle_apply_plan() function (~line 2325):

_print_lifecycle_plan(plan, title=title)

Steps to Reproduce

agents plan apply --yes <PLAN_ID>

Impact

Users cannot see artifact counts, validation results, sandbox cleanup status, or total cost/duration at a glance after applying a plan. The UX is significantly degraded compared to the spec.

Subtasks

Implement _print_plan_applied_panels() helper with apply-specific panel rendering
Show Apply Summary panel (artifacts, changes, project, applied_at)
Show Validation panel with test/lint/typecheck results
Show Sandbox Cleanup panel (worktree, branch, checkpoint status)
Show Plan Lifecycle panel (phase, state, duration, cost, decisions, child plans)
Show Next Steps panel
Show Recovery Options panel when apply fails
Print ✓ OK Changes applied or ✗ ERROR Apply refused message

Definition of Done

agents plan apply rich output shows all 5 panels as specified for success
agents plan apply rich output shows Recovery Options panel for failures
Appropriate ✓ OK or ✗ ERROR message printed

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

## Summary `agents plan apply <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 5 distinct panels plus a success/error message. ## Expected Behavior (from spec §agents plan apply) The spec defines the following rich output panels for a successful apply: ``` Apply changes for plan 01HXM8C2ZK4Q7C2B3F2R4VYV6J? [y/N]: y ╭─ Apply Summary ─────────────────────╮ │ Plan: 01HXM8C2ZK4Q7C2B3F2R4VYV6J │ │ Artifacts: 6 files updated │ │ Changes: 42 insertions, 9 deletions │ │ Project: local/api-service │ │ Applied At: 2026-02-08 13:04 │ ╰─────────────────────────────────────╯ ╭─ Validation (from Execute) ────╮ │ Tests: passed (24/24) │ │ Lint: passed (0 warnings) │ │ Type Check: passed (0 errors) │ │ Duration: 12.4s │ ╰────────────────────────────────╯ ╭─ Sandbox Cleanup ─────────╮ │ Worktree: removed │ │ Branch: merged to main │ │ Checkpoint: archived │ ╰───────────────────────────╯ ╭─ Plan Lifecycle ────────────────────────╮ │ Phase: apply │ │ State: applied │ │ Total Duration: 00:06:14 │ │ Total Cost: $0.0847 │ │ Decisions Made: 8 │ │ Child Plans: 2 (completed) │ ╰─────────────────────────────────────────╯ ╭─ Next Steps ──────╮ │ - Review git diff │ │ - Commit changes │ ╰───────────────────╯ ✓ OK Changes applied ``` For a failed apply (validation failures), the spec shows: - `Apply Summary` panel - `Validation` panel with ✗ failures listed - `Sandbox Status` panel (preserved, not cleaned up) - `Recovery Options` panel - `✗ ERROR Apply refused — N required Execute-phase validations did not pass` ## Actual Behavior The implementation calls `_print_lifecycle_plan(plan, title="Plan Applied")` which renders a single generic panel. No separate Apply Summary, Validation, Sandbox Cleanup, Plan Lifecycle, or Next Steps panels. No `✓ OK Changes applied` message. ## Code Location `src/cleveragents/cli/commands/plan.py`, `lifecycle_apply_plan()` function (~line 2325): ```python _print_lifecycle_plan(plan, title=title) ``` ## Steps to Reproduce ```bash agents plan apply --yes <PLAN_ID> ``` ## Impact Users cannot see artifact counts, validation results, sandbox cleanup status, or total cost/duration at a glance after applying a plan. The UX is significantly degraded compared to the spec. ## Subtasks - [ ] Implement `_print_plan_applied_panels()` helper with apply-specific panel rendering - [ ] Show `Apply Summary` panel (artifacts, changes, project, applied_at) - [ ] Show `Validation` panel with test/lint/typecheck results - [ ] Show `Sandbox Cleanup` panel (worktree, branch, checkpoint status) - [ ] Show `Plan Lifecycle` panel (phase, state, duration, cost, decisions, child plans) - [ ] Show `Next Steps` panel - [ ] Show `Recovery Options` panel when apply fails - [ ] Print `✓ OK Changes applied` or `✗ ERROR Apply refused` message ## Definition of Done - `agents plan apply` rich output shows all 5 panels as specified for success - `agents plan apply` rich output shows Recovery Options panel for failures - Appropriate `✓ OK` or `✗ ERROR` message printed --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 14:32:52 +00:00

HAL9000 commented

2026-04-09 15:44:14 +00:00

Author

Owner

MoSCoW classification: MoSCoW/Should have

Rationale: agents plan apply using a generic single panel instead of the spec-required multi-panel layout is a UI completeness issue. The spec defines a structured 5-panel layout for the apply output. The command works correctly but the output is less informative than specified. Should be fixed for spec compliance.

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner

MoSCoW classification: **MoSCoW/Should have** Rationale: `agents plan apply` using a generic single panel instead of the spec-required multi-panel layout is a UI completeness issue. The spec defines a structured 5-panel layout for the apply output. The command works correctly but the output is less informative than specified. Should be fixed for spec compliance. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added a new dependency

2026-04-09 17:52:43 +00:00

#5203 EPIC: CLI Output Format Compliance — Global Flags, JSON Envelope & Error Responses

HAL9000 added this to the v3.2.0 milestone

2026-04-09 21:18:54 +00:00