UAT: `agents plan diff --correction <CORRECTION_ATTEMPT_ID>` is a stub — correction diff not implemented #4691

New issue

Open

opened 2026-04-08 18:01:28 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-08 18:01:28 +00:00

Owner

Bug Report

Feature Area: Decision tree and correction model — agents plan diff --correction
Severity: Medium — the correction diff feature is completely unimplemented (stub)
Source: src/cleveragents/cli/commands/plan.py lines 2948–2960

What Was Tested

Code-level analysis of the plan_diff CLI command to verify that the --correction option is implemented.

Expected Behavior (from spec §agents plan diff, line 343)

The spec defines:

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

When --correction <CORRECTION_ATTEMPT_ID> is provided, the command should show the diff between the original decision subtree and the corrected subtree — i.e., what changed as a result of the correction.

Additionally, the spec signature shows --correction and <PLAN_ID> as alternatives (either/or), meaning plan_id should be optional when --correction is provided.

Actual Behavior

The implementation (lines 2948–2960) is a stub:

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

Deviations:

Not implemented: Returns a stub message "Correction diff will be available after M4.2."
Wrong signature: plan_id is a required positional argument even when --correction is provided. The spec shows them as alternatives.
No actual diff: Does not compare original vs corrected subtree state.

Steps to Reproduce

agents plan diff --correction <correction_attempt_id> <plan_id>

Expected: Shows unified diff of what changed due to the correction.
Actual: Shows stub panel "Correction diff will be available after M4.2."

Code Location

src/cleveragents/cli/commands/plan.py lines 2918–2960

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

## Bug Report **Feature Area:** Decision tree and correction model — `agents plan diff --correction` **Severity:** Medium — the correction diff feature is completely unimplemented (stub) **Source:** `src/cleveragents/cli/commands/plan.py` lines 2948–2960 --- ## What Was Tested Code-level analysis of the `plan_diff` CLI command to verify that the `--correction` option is implemented. ## Expected Behavior (from spec §agents plan diff, line 343) The spec defines: ``` agents plan diff (--correction <CORRECTION_ATTEMPT_ID>|<PLAN_ID>) ``` When `--correction <CORRECTION_ATTEMPT_ID>` is provided, the command should show the diff between the original decision subtree and the corrected subtree — i.e., what changed as a result of the correction. Additionally, the spec signature shows `--correction` and `<PLAN_ID>` as **alternatives** (either/or), meaning `plan_id` should be optional when `--correction` is provided. ## Actual Behavior The implementation (lines 2948–2960) is a stub: ```python 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 ``` ### Deviations: 1. **Not implemented**: Returns a stub message "Correction diff will be available after M4.2." 2. **Wrong signature**: `plan_id` is a required positional argument even when `--correction` is provided. The spec shows them as alternatives. 3. **No actual diff**: Does not compare original vs corrected subtree state. ## Steps to Reproduce ```bash agents plan diff --correction <correction_attempt_id> <plan_id> ``` Expected: Shows unified diff of what changed due to the correction. Actual: Shows stub panel "Correction diff will be available after M4.2." ## Code Location - `src/cleveragents/cli/commands/plan.py` lines 2918–2960 --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added this to the v3.5.0 milestone

2026-04-08 18:05:37 +00:00