UAT: `agents plan diff --correction` is a stub — returns placeholder text instead of actual correction diff #4414

New issue

Open

opened 2026-04-08 12:25:04 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-08 12:25:04 +00:00

Owner

Bug Report

Feature Area: Plan lifecycle — agents plan diff
Severity: Medium
Found by: UAT tester worker (plan lifecycle phases)

What Was Tested

The agents plan diff command with the --correction flag, as specified in the spec:

agents plan diff (--correction <CORRECTION_ATTEMPT_ID>|<PLAN_ID>)

Expected Behavior (from spec)

When --correction <CORRECTION_ATTEMPT_ID> is provided, agents plan diff should show the diff for a specific correction attempt — displaying the files changed, insertions/deletions, and patch preview for that correction attempt.

Actual Behavior

The implementation is a stub that returns a placeholder message:

# From src/cleveragents/cli/commands/plan.py, lines 2949-2960
if correction:
    # Show correction-specific diff (stub: shows info panel)
    console.print(
        Panel(
            f"[bold]Correction Attempt:[/bold] {correction}\n"
            f"[bold]Plan:[/bold] {plan_id}\n\n"
            "[dim]Correction diff will be available after M4.2.[/dim]",
            title="Correction Diff",
            expand=False,
        )
    )
    return

The output is:

╭─ Correction Diff ─────────────────────────────────────────╮
│ Correction Attempt: <id>                                   │
│ Plan: <plan_id>                                            │
│                                                            │
│ Correction diff will be available after M4.2.             │
╰────────────────────────────────────────────────────────────╯

Steps to Reproduce

agents plan diff <PLAN_ID> --correction <CORRECTION_ATTEMPT_ID>

Code Location

src/cleveragents/cli/commands/plan.py, lines 2949–2960 in the plan_diff function.

Impact

Users cannot inspect the diff for a specific correction attempt. The --correction flag is documented in the spec but non-functional.

Definition of Done

agents plan diff --correction <CORRECTION_ATTEMPT_ID> returns the actual diff for the correction attempt
Output matches the spec format (Diff Summary, Files, Patch Preview, Risk Assessment panels)
JSON/YAML output includes the structured diff data

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

## Bug Report **Feature Area:** Plan lifecycle — `agents plan diff` **Severity:** Medium **Found by:** UAT tester worker (plan lifecycle phases) ### What Was Tested The `agents plan diff` command with the `--correction` flag, as specified in the spec: ``` agents plan diff (--correction <CORRECTION_ATTEMPT_ID>|<PLAN_ID>) ``` ### Expected Behavior (from spec) When `--correction <CORRECTION_ATTEMPT_ID>` is provided, `agents plan diff` should show the diff for a specific correction attempt — displaying the files changed, insertions/deletions, and patch preview for that correction attempt. ### Actual Behavior The implementation is a **stub** that returns a placeholder message: ```python # From src/cleveragents/cli/commands/plan.py, lines 2949-2960 if correction: # Show correction-specific diff (stub: shows info panel) console.print( Panel( f"[bold]Correction Attempt:[/bold] {correction}\n" f"[bold]Plan:[/bold] {plan_id}\n\n" "[dim]Correction diff will be available after M4.2.[/dim]", title="Correction Diff", expand=False, ) ) return ``` The output is: ``` ╭─ Correction Diff ─────────────────────────────────────────╮ │ Correction Attempt: <id> │ │ Plan: <plan_id> │ │ │ │ Correction diff will be available after M4.2. │ ╰────────────────────────────────────────────────────────────╯ ``` ### Steps to Reproduce ```bash agents plan diff <PLAN_ID> --correction <CORRECTION_ATTEMPT_ID> ``` ### Code Location `src/cleveragents/cli/commands/plan.py`, lines 2949–2960 in the `plan_diff` function. ### Impact Users cannot inspect the diff for a specific correction attempt. The `--correction` flag is documented in the spec but non-functional. ### Definition of Done - `agents plan diff --correction <CORRECTION_ATTEMPT_ID>` returns the actual diff for the correction attempt - Output matches the spec format (Diff Summary, Files, Patch Preview, Risk Assessment panels) - JSON/YAML output includes the structured diff data --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added this to the v3.5.0 milestone

2026-04-08 17:42:52 +00:00

HAL9000 added the