UAT: agents plan list rich output has extra columns (Name, Invariants) not in spec, and Summary panel missing Cancelled count #3686

Open
opened 2026-04-05 21:36:32 +00:00 by freemo · 1 comment
Owner

Metadata

  • Branch: fix/plan-list-columns-and-summary-panel
  • Commit Message: fix(cli): remove extra columns and add Cancelled count to plan list output
  • Milestone: None (backlog)
  • Parent Epic: #394

Description

The agents plan list command's rich output has two deviations from the specification (docs/specification.md lines 12126–12272):

  1. Extra columns: The implementation renders ID, Name, Phase, State, Action, Invariants, Project, Elapsed columns, but the spec only requires ID, Phase, State, Action, Project, Elapsed. The Name and Invariants columns are not in the spec.

  2. Missing Cancelled count in Summary panel: The spec shows a Cancelled: N row in the Summary panel, but the implementation only shows Total, Processing, Completed, Errored.

Expected behavior (per spec lines 12126–12272)

Table columns: ID, Phase, State, Action, Project, Elapsed (6 columns)

Summary panel:

╭─ Summary ─────────────╮
│ Total: 5              │
│ Processing: 2         │
│ Completed: 1          │
│ Errored: 1            │
│ Cancelled: 1          │
╰───────────────────────╯

Actual behavior

Table columns: ID, Name, Phase, State, Action, Invariants, Project, Elapsed (8 columns — 2 extra)

Summary panel (lines 2546–2551 of plan.py):

summary_text = (
    f"[yellow bold]Total:[/yellow bold] {total_plans}\n"
    f"[blue bold]Processing:[/blue bold] {processing_count}\n"
    f"[green bold]Completed:[/green bold] {completed_count}\n"
    f"[red bold]Errored:[/red bold] {errored_count}"
)

Missing Cancelled count.

Affected code

  • src/cleveragents/cli/commands/plan.pylifecycle_list_plans():
    • Table column definitions (lines 2456–2464): remove Name and Invariants columns
    • Table row building (lines 2490–2499): remove str(plan.namespaced_name) and invariant_count from row
    • Summary panel (lines 2546–2551): add cancelled_count and Cancelled row

Steps to reproduce

  1. Create one or more plans (including at least one cancelled plan)
  2. Run agents plan list
  3. Observe that the table has Name and Invariants columns not in the spec
  4. Observe that the Summary panel does not show a Cancelled count

Subtasks

  • Remove Name column from lifecycle_list_plans() table definition and row building
  • Remove Invariants column from lifecycle_list_plans() table definition and row building
  • Add cancelled_count calculation and Cancelled row to the Summary panel in lifecycle_list_plans()
  • Add/update Behave unit tests in features/ covering the correct column set and Summary panel content
  • Verify nox -e typecheck passes
  • Verify nox -e coverage_report >= 97%

Definition of Done

  • agents plan list renders a table with exactly ID, Phase, State, Action, Project, Elapsed columns
  • agents plan list Summary panel shows Total, Processing, Completed, Errored, Cancelled counts
  • All nox stages pass (nox -e lint, nox -e typecheck, nox -e unit_tests, nox -e integration_tests)
  • Coverage >= 97%

Backlog note: This issue was discovered during autonomous operation
on milestone v3.3.0. 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

## Metadata - **Branch**: `fix/plan-list-columns-and-summary-panel` - **Commit Message**: `fix(cli): remove extra columns and add Cancelled count to plan list output` - **Milestone**: None (backlog) - **Parent Epic**: #394 ## Description The `agents plan list` command's rich output has two deviations from the specification (`docs/specification.md` lines 12126–12272): 1. **Extra columns**: The implementation renders `ID, Name, Phase, State, Action, Invariants, Project, Elapsed` columns, but the spec only requires `ID, Phase, State, Action, Project, Elapsed`. The `Name` and `Invariants` columns are not in the spec. 2. **Missing `Cancelled` count in Summary panel**: The spec shows a `Cancelled: N` row in the Summary panel, but the implementation only shows `Total`, `Processing`, `Completed`, `Errored`. ### Expected behavior (per spec lines 12126–12272) Table columns: `ID, Phase, State, Action, Project, Elapsed` (6 columns) Summary panel: ``` ╭─ Summary ─────────────╮ │ Total: 5 │ │ Processing: 2 │ │ Completed: 1 │ │ Errored: 1 │ │ Cancelled: 1 │ ╰───────────────────────╯ ``` ### Actual behavior Table columns: `ID, Name, Phase, State, Action, Invariants, Project, Elapsed` (8 columns — 2 extra) Summary panel (lines 2546–2551 of `plan.py`): ```python summary_text = ( f"[yellow bold]Total:[/yellow bold] {total_plans}\n" f"[blue bold]Processing:[/blue bold] {processing_count}\n" f"[green bold]Completed:[/green bold] {completed_count}\n" f"[red bold]Errored:[/red bold] {errored_count}" ) ``` Missing `Cancelled` count. ### Affected code - `src/cleveragents/cli/commands/plan.py` — `lifecycle_list_plans()`: - Table column definitions (lines 2456–2464): remove `Name` and `Invariants` columns - Table row building (lines 2490–2499): remove `str(plan.namespaced_name)` and `invariant_count` from row - Summary panel (lines 2546–2551): add `cancelled_count` and `Cancelled` row ### Steps to reproduce 1. Create one or more plans (including at least one cancelled plan) 2. Run `agents plan list` 3. Observe that the table has `Name` and `Invariants` columns not in the spec 4. Observe that the Summary panel does not show a `Cancelled` count ## Subtasks - [ ] Remove `Name` column from `lifecycle_list_plans()` table definition and row building - [ ] Remove `Invariants` column from `lifecycle_list_plans()` table definition and row building - [ ] Add `cancelled_count` calculation and `Cancelled` row to the Summary panel in `lifecycle_list_plans()` - [ ] Add/update Behave unit tests in `features/` covering the correct column set and Summary panel content - [ ] Verify `nox -e typecheck` passes - [ ] Verify `nox -e coverage_report` >= 97% ## Definition of Done - [ ] `agents plan list` renders a table with exactly `ID, Phase, State, Action, Project, Elapsed` columns - [ ] `agents plan list` Summary panel shows `Total`, `Processing`, `Completed`, `Errored`, `Cancelled` counts - [ ] All nox stages pass (`nox -e lint`, `nox -e typecheck`, `nox -e unit_tests`, `nox -e integration_tests`) - [ ] Coverage >= 97% > **Backlog note:** This issue was discovered during autonomous operation > on milestone v3.3.0. 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
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: Backlog — CLI output cosmetic deviation; does not block functionality.
  • Milestone: None (backlog — no milestone assigned; will be picked up when CLI polish work is prioritized)
  • Story Points: 2 — S — Remove 2 columns and add 1 count to summary panel. Straightforward code changes in a single file.
  • MoSCoW: Could Have — The extra columns provide additional information (Name, Invariants) that may actually be useful. The spec defines the minimum required columns, and the missing Cancelled count is a minor omission. This is polish work.
  • Parent Epic: #394

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Backlog — CLI output cosmetic deviation; does not block functionality. - **Milestone**: None (backlog — no milestone assigned; will be picked up when CLI polish work is prioritized) - **Story Points**: 2 — S — Remove 2 columns and add 1 count to summary panel. Straightforward code changes in a single file. - **MoSCoW**: Could Have — The extra columns provide additional information (Name, Invariants) that may actually be useful. The spec defines the minimum required columns, and the missing Cancelled count is a minor omission. This is polish work. - **Parent Epic**: #394 --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#394 Epic: Decision Framework
cleveragents/cleveragents-core
Reference
cleveragents/cleveragents-core#3686
No description provided.