UAT: agents actor show missing spec-required detail panels and fields #2388

Open
opened 2026-04-03 17:28:18 +00:00 by freemo · 2 comments
Owner

Metadata

  • Branch: fix/actor-show-missing-panels
  • Commit Message: fix(cli): add missing panels and fields to agents actor show command
  • Milestone: v3.2.0
  • Parent Epic: #392

Background and Context

The agents actor show command was implemented with only a minimal "Actor Details" panel. According to the specification (§5535–5710), the command must render multiple additional panels and fields that are currently absent. This was discovered during UAT testing.

Current Behavior

The _print_actor() function (called with show_add_panels=False) renders a single panel containing only 6 fields: Name, Provider, Model, Default, Unsafe, and Type. The show command invokes _print_actor(actor, title="Actor details", fmt=fmt), which omits all additional panels and the success message.

Code locations:

  • src/cleveragents/cli/commands/actor.py_print_actor function (lines 339–416)
  • src/cleveragents/cli/commands/actor.pyshow command (lines 791–815)

Steps to reproduce:

  1. Add an actor: agents actor add --config examples/actors/simple_llm.yaml
  2. Run: agents actor show assistants/code_reviewer
  3. Observe: Only a basic panel is shown — missing Built-in, Created, Updated, Config, Config Hash fields, and all additional panels.

Expected Behavior (from spec §5535–5710)

Actor Details panel must include ALL of:

  • Name
  • Provider
  • Model
  • Default
  • Built-in ← MISSING
  • Unsafe
  • Type
  • Created ← MISSING
  • Updated ← MISSING
  • Config (file path) ← MISSING
  • Config Hash ← MISSING

Additional panels MISSING entirely:

  • Options panel: temperature, max_tokens, top_p, etc.
  • Graph Structure panel: Nodes count, Edges count, Entry node, Exit node(s)
  • Tools panel: tool name, read-only flag, safe flag (table format)
  • Access panel: Unsafe, Filesystem, Network
  • Usage panel: Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run

Success message MISSING: ✓ OK Actor loaded (Rich) / [OK] Actor loaded (Plain)

Acceptance Criteria

  • agents actor show <NAME> renders the Actor Details panel with all 11 fields: Name, Provider, Model, Default, Built-in, Unsafe, Type, Created, Updated, Config, Config Hash
  • Options panel is rendered when options are present on the actor
  • Graph Structure panel is rendered for graph-type actors (Nodes, Edges, Entry, Exit)
  • Tools panel is rendered as a table when tools are attached to the actor
  • Access panel is rendered (Unsafe, Filesystem, Network)
  • Usage panel is rendered (Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run)
  • Success message ✓ OK Actor loaded (Rich) / [OK] Actor loaded (Plain) / messages array (JSON/YAML) is emitted
  • JSON and YAML output formats include all fields per spec §5535–5710
  • All existing agents actor show tests continue to pass

Subtasks

  • Extend _print_actor() to include Built-in, Created, Updated, Config, and Config Hash fields in the Actor Details panel
  • Implement Options panel rendering in _print_actor() (shown when actor has options)
  • Implement Graph Structure panel rendering in _print_actor() (shown for graph-type actors)
  • Implement Tools panel rendering in _print_actor() (table: Tool, Read-Only, Safe)
  • Implement Access panel rendering in _print_actor() (Unsafe, Filesystem, Network)
  • Implement Usage panel rendering in _print_actor() (Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run)
  • Emit success message ✓ OK Actor loaded at end of show command output
  • Ensure JSON and YAML output formats include all spec-required fields
  • Tests (Behave): Add/update scenarios covering all new panels and fields for Rich, Plain, JSON, and YAML formats
  • Tests (Robot): Add/update integration test for agents actor show full output
  • Verify coverage >= 97% via nox -s coverage_report
  • Run nox (all default sessions), fix any errors

Definition of Done

This issue is complete when:

  • All subtasks above are completed and checked off.
  • A Git commit is created where the first line of the commit message matches the Commit Message in Metadata exactly (fix(cli): add missing panels and fields to agents actor show command), followed by a blank line, then additional lines providing relevant implementation details.
  • The commit is pushed to the remote on the branch matching the Branch in Metadata exactly (fix/actor-show-missing-panels).
  • The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.
  • All nox stages pass.
  • Coverage >= 97%.

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

## Metadata - **Branch**: `fix/actor-show-missing-panels` - **Commit Message**: `fix(cli): add missing panels and fields to agents actor show command` - **Milestone**: v3.2.0 - **Parent Epic**: #392 ## Background and Context The `agents actor show` command was implemented with only a minimal "Actor Details" panel. According to the specification (§5535–5710), the command must render multiple additional panels and fields that are currently absent. This was discovered during UAT testing. ## Current Behavior The `_print_actor()` function (called with `show_add_panels=False`) renders a single panel containing only 6 fields: Name, Provider, Model, Default, Unsafe, and Type. The `show` command invokes `_print_actor(actor, title="Actor details", fmt=fmt)`, which omits all additional panels and the success message. **Code locations:** - `src/cleveragents/cli/commands/actor.py` — `_print_actor` function (lines 339–416) - `src/cleveragents/cli/commands/actor.py` — `show` command (lines 791–815) **Steps to reproduce:** 1. Add an actor: `agents actor add --config examples/actors/simple_llm.yaml` 2. Run: `agents actor show assistants/code_reviewer` 3. Observe: Only a basic panel is shown — missing Built-in, Created, Updated, Config, Config Hash fields, and all additional panels. ## Expected Behavior (from spec §5535–5710) **Actor Details panel** must include ALL of: - Name - Provider - Model - Default - **Built-in** ← MISSING - Unsafe - Type - **Created** ← MISSING - **Updated** ← MISSING - **Config** (file path) ← MISSING - **Config Hash** ← MISSING **Additional panels MISSING entirely:** - **Options** panel: temperature, max_tokens, top_p, etc. - **Graph Structure** panel: Nodes count, Edges count, Entry node, Exit node(s) - **Tools** panel: tool name, read-only flag, safe flag (table format) - **Access** panel: Unsafe, Filesystem, Network - **Usage** panel: Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run **Success message MISSING:** `✓ OK Actor loaded` (Rich) / `[OK] Actor loaded` (Plain) ## Acceptance Criteria - [ ] `agents actor show <NAME>` renders the Actor Details panel with all 11 fields: Name, Provider, Model, Default, Built-in, Unsafe, Type, Created, Updated, Config, Config Hash - [ ] Options panel is rendered when options are present on the actor - [ ] Graph Structure panel is rendered for graph-type actors (Nodes, Edges, Entry, Exit) - [ ] Tools panel is rendered as a table when tools are attached to the actor - [ ] Access panel is rendered (Unsafe, Filesystem, Network) - [ ] Usage panel is rendered (Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run) - [ ] Success message `✓ OK Actor loaded` (Rich) / `[OK] Actor loaded` (Plain) / `messages` array (JSON/YAML) is emitted - [ ] JSON and YAML output formats include all fields per spec §5535–5710 - [ ] All existing `agents actor show` tests continue to pass ## Subtasks - [ ] Extend `_print_actor()` to include Built-in, Created, Updated, Config, and Config Hash fields in the Actor Details panel - [ ] Implement Options panel rendering in `_print_actor()` (shown when actor has options) - [ ] Implement Graph Structure panel rendering in `_print_actor()` (shown for graph-type actors) - [ ] Implement Tools panel rendering in `_print_actor()` (table: Tool, Read-Only, Safe) - [ ] Implement Access panel rendering in `_print_actor()` (Unsafe, Filesystem, Network) - [ ] Implement Usage panel rendering in `_print_actor()` (Referenced by Actions, Active in Sessions, Total Runs, Avg Cost/Run) - [ ] Emit success message `✓ OK Actor loaded` at end of `show` command output - [ ] Ensure JSON and YAML output formats include all spec-required fields - [ ] Tests (Behave): Add/update scenarios covering all new panels and fields for Rich, Plain, JSON, and YAML formats - [ ] Tests (Robot): Add/update integration test for `agents actor show` full output - [ ] Verify coverage >= 97% via `nox -s coverage_report` - [ ] Run `nox` (all default sessions), fix any errors ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - A Git commit is created where the **first line** of the commit message matches the Commit Message in Metadata exactly (`fix(cli): add missing panels and fields to agents actor show command`), followed by a blank line, then additional lines providing relevant implementation details. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly (`fix/actor-show-missing-panels`). - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done. - All nox stages pass. - Coverage >= 97%. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-uat-tester
freemo added this to the v3.2.0 milestone 2026-04-03 17:28:23 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: High (correct — CLI output completeness is important for v3.2.0 acceptance)
  • Milestone: v3.2.0 (correct)
  • MoSCoW: Should Have — The agents actor show command is part of the core CLI experience. Missing panels and fields mean the command doesn't meet spec requirements (§5535–5710). This should be fixed for milestone acceptance.
  • Parent Epic: #392

Well-documented with clear expected vs actual behavior and specific code locations.


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

Issue triaged by project owner: - **State**: Verified ✅ - **Priority**: High (correct — CLI output completeness is important for v3.2.0 acceptance) - **Milestone**: v3.2.0 (correct) - **MoSCoW**: Should Have — The `agents actor show` command is part of the core CLI experience. Missing panels and fields mean the command doesn't meet spec requirements (§5535–5710). This should be fixed for milestone acceptance. - **Parent Epic**: #392 Well-documented with clear expected vs actual behavior and specific code locations. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
freemo removed this from the v3.2.0 milestone 2026-04-06 22:30:25 +00:00
Author
Owner

This issue has been moved to the backlog as part of an aggressive grooming of the v3.2.0 milestone. It has been deemed non-critical for the minimal viability of the milestone and will be addressed in a future release.

This issue has been moved to the backlog as part of an aggressive grooming of the v3.2.0 milestone. It has been deemed non-critical for the minimal viability of the milestone and will be addressed in a future release.
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
#392 Epic: Actor YAML & Compiler
cleveragents/cleveragents-core
Reference
cleveragents/cleveragents-core#2388
No description provided.