UAT: agents actor list rich output missing spec-required Summary panel and ✓ OK N actors listed success message #1525

Closed
opened 2026-04-02 20:14:30 +00:00 by freemo · 2 comments
Owner

Metadata

  • Branch: fix/actor-list-missing-summary-panel
  • Commit Message: fix(cli): add Summary panel and success message to actor list rich output
  • Milestone: v3.5.0
  • Parent Epic: #936

Description

The specification defines the following rich output structure for agents actor list:

╭─ Actors ──────────────────────────────────────────────────────────────╮
│ Name            Provider   Model       Default  Built-in  Unsafe      │
│ ──────────────  ─────────  ──────────  ───────  ────────  ──────      │
│ local/reviewer  openai     gpt-4       ✓                  no          │
│ openai/gpt-4    openai     gpt-4                ✓         no          │
│ anthropic/3.5   anthropic  claude-3.5           ✓         no          │
╰───────────────────────────────────────────────────────────────────────╯

╭─ Summary ──────────────╮
│ Total: 3               │
│ Built-in: 2            │
│ Custom: 1              │
│ Unsafe: 0              │
│ Providers Used: 2      │
╰────────────────────────╯

✓ OK 3 actors listed

Actual behaviour: list_actors() in src/cleveragents/cli/commands/actor.py (lines ~615–665) renders only the actors table (titled "Actors (N total)") but omits:

  1. The Summary panel (Total, Built-in, Custom, Unsafe, Providers Used counts)
  2. The ✓ OK N actors listed success message

Severity: Low — the command is functionally correct and the actor table is rendered; only the summary statistics panel and the success status line are absent.

Code location: src/cleveragents/cli/commands/actor.py, list_actors() function (~lines 615–665)

Subtasks

  • Audit list_actors() in actor.py against the spec-defined rich output structure
  • Implement the Summary panel (Total, Built-in, Custom, Unsafe, Providers Used) via the OutputSession/Rich materializer
  • Implement the ✓ OK N actors listed success status message via StatusHandle
  • Ensure the panel and status message are suppressed in --plain / --json / --yaml output modes (spec-compliant format gating)
  • Add/update Behave scenario for agents actor list rich output to assert the Summary panel and success message are present
  • Run nox (all default sessions) and fix any errors

Definition of Done

  • agents actor list rich output includes the Summary panel with correct counts (Total, Built-in, Custom, Unsafe, Providers Used)
  • agents actor list rich output includes the ✓ OK N actors listed success message
  • Non-rich output modes (--plain, --json, --yaml) are unaffected
  • Behave scenario covers the Summary panel and success message assertions
  • All nox stages pass
  • Coverage >= 97%

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

## Metadata - **Branch**: `fix/actor-list-missing-summary-panel` - **Commit Message**: `fix(cli): add Summary panel and success message to actor list rich output` - **Milestone**: v3.5.0 - **Parent Epic**: #936 ## Description The specification defines the following rich output structure for `agents actor list`: ``` ╭─ Actors ──────────────────────────────────────────────────────────────╮ │ Name Provider Model Default Built-in Unsafe │ │ ────────────── ───────── ────────── ─────── ──────── ────── │ │ local/reviewer openai gpt-4 ✓ no │ │ openai/gpt-4 openai gpt-4 ✓ no │ │ anthropic/3.5 anthropic claude-3.5 ✓ no │ ╰───────────────────────────────────────────────────────────────────────╯ ╭─ Summary ──────────────╮ │ Total: 3 │ │ Built-in: 2 │ │ Custom: 1 │ │ Unsafe: 0 │ │ Providers Used: 2 │ ╰────────────────────────╯ ✓ OK 3 actors listed ``` **Actual behaviour**: `list_actors()` in `src/cleveragents/cli/commands/actor.py` (lines ~615–665) renders only the actors table (titled "Actors (N total)") but omits: 1. The `Summary` panel (Total, Built-in, Custom, Unsafe, Providers Used counts) 2. The `✓ OK N actors listed` success message **Severity**: Low — the command is functionally correct and the actor table is rendered; only the summary statistics panel and the success status line are absent. **Code location**: `src/cleveragents/cli/commands/actor.py`, `list_actors()` function (~lines 615–665) ## Subtasks - [ ] Audit `list_actors()` in `actor.py` against the spec-defined rich output structure - [ ] Implement the `Summary` panel (Total, Built-in, Custom, Unsafe, Providers Used) via the OutputSession/Rich materializer - [ ] Implement the `✓ OK N actors listed` success status message via `StatusHandle` - [ ] Ensure the panel and status message are suppressed in `--plain` / `--json` / `--yaml` output modes (spec-compliant format gating) - [ ] Add/update Behave scenario for `agents actor list` rich output to assert the Summary panel and success message are present - [ ] Run `nox` (all default sessions) and fix any errors ## Definition of Done - [ ] `agents actor list` rich output includes the `Summary` panel with correct counts (Total, Built-in, Custom, Unsafe, Providers Used) - [ ] `agents actor list` rich output includes the `✓ OK N actors listed` success message - [ ] Non-rich output modes (`--plain`, `--json`, `--yaml`) are unaffected - [ ] Behave scenario covers the Summary panel and success message assertions - [ ] All nox stages pass - [ ] Coverage >= 97% --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-new-issue-creator
freemo added this to the v3.5.0 milestone 2026-04-02 20:22:34 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: Medium — The command works functionally but is missing the Summary panel (Total, Built-in, Custom, Unsafe, Providers Used) and the success message. These provide useful at-a-glance statistics.
  • Milestone: v3.5.0 (already assigned)
  • MoSCoW: Should Have — The spec defines the Summary panel and success message as part of the standard rich output structure. While the actor listing works, the missing summary statistics reduce the output's usefulness. This is part of the broader CLI output spec-alignment effort under Epic #936.
  • Parent Epic: #936

Valid UAT bug. Well-structured with all required sections. Part of the same pattern as #1522, #1511, #1508 — CLI commands missing spec-required panels.


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

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — The command works functionally but is missing the Summary panel (Total, Built-in, Custom, Unsafe, Providers Used) and the success message. These provide useful at-a-glance statistics. - **Milestone**: v3.5.0 (already assigned) - **MoSCoW**: Should Have — The spec defines the Summary panel and success message as part of the standard rich output structure. While the actor listing works, the missing summary statistics reduce the output's usefulness. This is part of the broader CLI output spec-alignment effort under Epic #936. - **Parent Epic**: #936 Valid UAT bug. Well-structured with all required sections. Part of the same pattern as #1522, #1511, #1508 — CLI commands missing spec-required panels. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
freemo self-assigned this 2026-04-02 20:59:00 +00:00
Author
Owner

Closing — work completed in PR #1562 (merged). The PR added Summary panel and success message to agents actor list rich output.


Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: ca-backlog-groomer

Closing — work completed in PR #1562 (merged). The PR added Summary panel and success message to `agents actor list` rich output. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: ca-backlog-groomer
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.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core#1525
No description provided.