UAT: `agents plan correct` rich output is a single line — spec requires 4-5 structured panels (Correction, Affected Subtree, Sandbox Rollback, Recompute, History) #3119

New issue

Open

opened 2026-04-05 06:33:06 +00:00 by freemo · 0 comments

freemo commented

2026-04-05 06:33:06 +00:00

Owner

Summary

The agents plan correct command's rich (default) output is a single line "✓ Correction applied: <id>" with optional reverted/new decision lists. The spec defines a rich multi-panel output with Correction, Affected Subtree, Sandbox Rollback, Recompute, and History panels.

Metadata

Branch: fix/plan-correct-rich-output-panels
Commit Message: fix(cli): implement spec-compliant rich output panels for plan correct command
Milestone: None
Parent Epic: #394

Subtasks

Implement "Correction" panel (Mode, Impact summary, New Decision, Corrects, Attempt)
Implement "Affected Subtree" panel (Decisions Invalidated, Child Plans Rolled Back, Artifacts Archived, Unaffected Decisions)
Implement "Sandbox Rollback" panel (Checkpoint, Files Reverted, Status) — shown only for revert mode
Implement "Recompute" panel (Queued, ETA) — shown only for revert mode
Implement "History" panel (superseded decision note, diff hint)
Implement "Append Detail" panel for append mode (Original decision preserved, Existing artifacts kept, Additional work)
Add Behave unit tests for rich output rendering
Add Robot Framework integration test verifying panel presence

Definition of Done

agents plan correct <id> --mode revert --guidance "..." --yes shows all 5 spec-defined panels
agents plan correct <id> --mode append --guidance "..." shows Correction + Append Detail + Queued panels
All existing tests pass
All nox stages pass
Coverage >= 97%

Bug Details

What was tested

Code analysis of src/cleveragents/cli/commands/plan.py, function correct_decision(), lines 3124–3131

Expected behavior (from spec § agents plan correct, Rich tab)

╭─ Correction ────────────────────────────────────────╮
│ Mode: revert                                        │
│ Impact: 3 decisions, 2 child plans, 5 artifacts     │
│ New Decision: 01HXM9B7Z3Q1Q8K2E9H7K3W2M8            │
│ Corrects: 01HXM9A1C2Q7W3R5G8Z0P4Q1X9                │
│ Attempt: 2                                          │
╰─────────────────────────────────────────────────────╯

╭─ Affected Subtree ──────────────╮
│ Decisions Invalidated: 3        │
│ Child Plans Rolled Back: 2      │
│ Artifacts Archived: 5           │
│ Unaffected Decisions: 2         │
╰─────────────────────────────────╯

╭─ Sandbox Rollback ─────────────╮
│ Checkpoint: cp_01HXM8C2        │
│ Files Reverted: 5              │
│ Status: restored               │
╰────────────────────────────────╯

╭─ Recompute ──────────────╮
│ Queued: 2 child plans    │
│ ETA: 4m                  │
╰──────────────────────────╯

╭─ History ───────────────────────────────────────────╮
│ - Original decision superseded                      │
│ - Prior artifacts archived for comparison           │
│ - agents plan diff --correction 01HXM9B7Z3Q1Q8K2..  │
╰─────────────────────────────────────────────────────╯

✓ OK Correction applied

Actual behavior

# src/cleveragents/cli/commands/plan.py lines 3124-3131
console.print(
    f"[green]✓[/green] Correction applied: {result.correction_id}"
)
if result.reverted_decisions:
    console.print(f"  Reverted: {', '.join(result.reverted_decisions)}")
if result.new_decisions:
    console.print(f"  New decisions: {', '.join(result.new_decisions)}")

A single line with optional decision lists — no panels, no impact summary, no sandbox rollback info, no recompute ETA.

Code location

src/cleveragents/cli/commands/plan.py, function correct_decision(), lines 3124–3131

Backlog note: This issue was discovered during autonomous operation on milestone v3.5.0 (Corrections + Context). It does not block milestone completion and has been placed in the backlog for human review and future milestone assignment.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: ca-new-issue-creator

## Summary The `agents plan correct` command's rich (default) output is a single line `"✓ Correction applied: <id>"` with optional reverted/new decision lists. The spec defines a rich multi-panel output with Correction, Affected Subtree, Sandbox Rollback, Recompute, and History panels. ## Metadata - **Branch**: `fix/plan-correct-rich-output-panels` - **Commit Message**: `fix(cli): implement spec-compliant rich output panels for plan correct command` - **Milestone**: None - **Parent Epic**: #394 ## Subtasks - [ ] Implement "Correction" panel (Mode, Impact summary, New Decision, Corrects, Attempt) - [ ] Implement "Affected Subtree" panel (Decisions Invalidated, Child Plans Rolled Back, Artifacts Archived, Unaffected Decisions) - [ ] Implement "Sandbox Rollback" panel (Checkpoint, Files Reverted, Status) — shown only for revert mode - [ ] Implement "Recompute" panel (Queued, ETA) — shown only for revert mode - [ ] Implement "History" panel (superseded decision note, diff hint) - [ ] Implement "Append Detail" panel for append mode (Original decision preserved, Existing artifacts kept, Additional work) - [ ] Add Behave unit tests for rich output rendering - [ ] Add Robot Framework integration test verifying panel presence ## Definition of Done - [ ] `agents plan correct <id> --mode revert --guidance "..." --yes` shows all 5 spec-defined panels - [ ] `agents plan correct <id> --mode append --guidance "..."` shows Correction + Append Detail + Queued panels - [ ] All existing tests pass - [ ] All nox stages pass - [ ] Coverage >= 97% ## Bug Details ### What was tested Code analysis of `src/cleveragents/cli/commands/plan.py`, function `correct_decision()`, lines 3124–3131 ### Expected behavior (from spec § agents plan correct, Rich tab) ``` ╭─ Correction ────────────────────────────────────────╮ │ Mode: revert │ │ Impact: 3 decisions, 2 child plans, 5 artifacts │ │ New Decision: 01HXM9B7Z3Q1Q8K2E9H7K3W2M8 │ │ Corrects: 01HXM9A1C2Q7W3R5G8Z0P4Q1X9 │ │ Attempt: 2 │ ╰─────────────────────────────────────────────────────╯ ╭─ Affected Subtree ──────────────╮ │ Decisions Invalidated: 3 │ │ Child Plans Rolled Back: 2 │ │ Artifacts Archived: 5 │ │ Unaffected Decisions: 2 │ ╰─────────────────────────────────╯ ╭─ Sandbox Rollback ─────────────╮ │ Checkpoint: cp_01HXM8C2 │ │ Files Reverted: 5 │ │ Status: restored │ ╰────────────────────────────────╯ ╭─ Recompute ──────────────╮ │ Queued: 2 child plans │ │ ETA: 4m │ ╰──────────────────────────╯ ╭─ History ───────────────────────────────────────────╮ │ - Original decision superseded │ │ - Prior artifacts archived for comparison │ │ - agents plan diff --correction 01HXM9B7Z3Q1Q8K2.. │ ╰─────────────────────────────────────────────────────╯ ✓ OK Correction applied ``` ### Actual behavior ```python # src/cleveragents/cli/commands/plan.py lines 3124-3131 console.print( f"[green]✓[/green] Correction applied: {result.correction_id}" ) if result.reverted_decisions: console.print(f" Reverted: {', '.join(result.reverted_decisions)}") if result.new_decisions: console.print(f" New decisions: {', '.join(result.new_decisions)}") ``` A single line with optional decision lists — no panels, no impact summary, no sandbox rollback info, no recompute ETA. ### Code location `src/cleveragents/cli/commands/plan.py`, function `correct_decision()`, lines 3124–3131 > **Backlog note:** This issue was discovered during autonomous operation on milestone v3.5.0 (Corrections + Context). It does not block milestone completion and has been placed in the backlog for human review and future milestone assignment. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-new-issue-creator

freemo added a new dependency

2026-04-05 06:34:03 +00:00

#394 Epic: Decision Framework

freemo added this to the v3.2.0 milestone

2026-04-05 06:42:55 +00:00

freemo referenced this issue

2026-04-05 06:43:52 +00:00

[CA-AUTO] UAT Pool Supervisor — v3.2.0–v3.7.0 — Session Tracker #3094

freemo referenced this issue

2026-04-05 06:53:54 +00:00

[CA-AUTO] UAT Pool Supervisor — v3.2.0–v3.7.0 — Session Tracker #3094

freemo referenced this issue