UAT: agents automation-profile remove rich output missing spec-required "Profile Removed" panel — only prints plain checkmark message #2966

Closed
opened 2026-04-05 02:58:36 +00:00 by freemo · 2 comments
Owner

Metadata

  • Branch: fix/automation-profile-remove-rich-output-panel
  • Commit Message: fix(cli): render "Profile Removed" panel in automation-profile remove command
  • Milestone: v3.7.0
  • Parent Epic: #362

Summary

The agents automation-profile remove command's rich output does not render the "Profile Removed" panel specified in the documentation. Instead, it only prints a plain green checkmark message.

Expected Behavior (from spec)

From docs/specification.md section "agents automation-profile remove", the rich output should render:

Remove automation profile local/careful-auto? [y/N]: y

╭─ Profile Removed ──────────╮
│ Name: local/careful-auto   │
╰────────────────────────────╯

✓ OK Profile removed

Actual Behavior

The implementation in src/cleveragents/cli/commands/automation_profile.py (line 305) only prints:

console.print(f"[green]✓[/green] Automation profile removed: {name}")

No "Profile Removed" panel is rendered. The profile name is included in the plain message but not in a structured panel.

Code Location

src/cleveragents/cli/commands/automation_profile.py:

  • Line 305: console.print(f"[green]✓[/green] Automation profile removed: {name}") — missing panel rendering

Steps to Reproduce

agents automation-profile remove acme/strict --yes

Observe that no "Profile Removed" panel is shown, only a plain message.

Impact

Minor visual inconsistency — the remove command output does not match the spec's documented format.

Subtasks

  • Add a Panel render to automation_profile.py remove command handler displaying the removed profile name under the title "Profile Removed"
  • Replace or supplement the plain console.print checkmark message with the spec-required panel output
  • Write Behave unit tests covering the rich output rendering for automation-profile remove
  • Write Robot Framework integration test verifying the panel is rendered on agents automation-profile remove

Definition of Done

  • agents automation-profile remove <name> --yes renders a ╭─ Profile Removed ─╮ Rich panel containing the profile name
  • The ✓ OK Profile removed success message is still printed after the panel
  • Behave unit tests added and passing for the rich output path
  • Robot Framework integration test added and passing
  • All nox stages pass
  • Coverage >= 97%

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

## Metadata - **Branch**: `fix/automation-profile-remove-rich-output-panel` - **Commit Message**: `fix(cli): render "Profile Removed" panel in automation-profile remove command` - **Milestone**: v3.7.0 - **Parent Epic**: #362 ## Summary The `agents automation-profile remove` command's rich output does not render the "Profile Removed" panel specified in the documentation. Instead, it only prints a plain green checkmark message. ## Expected Behavior (from spec) From `docs/specification.md` section "agents automation-profile remove", the rich output should render: ``` Remove automation profile local/careful-auto? [y/N]: y ╭─ Profile Removed ──────────╮ │ Name: local/careful-auto │ ╰────────────────────────────╯ ✓ OK Profile removed ``` ## Actual Behavior The implementation in `src/cleveragents/cli/commands/automation_profile.py` (line 305) only prints: ```python console.print(f"[green]✓[/green] Automation profile removed: {name}") ``` No "Profile Removed" panel is rendered. The profile name is included in the plain message but not in a structured panel. ## Code Location `src/cleveragents/cli/commands/automation_profile.py`: - Line 305: `console.print(f"[green]✓[/green] Automation profile removed: {name}")` — missing panel rendering ## Steps to Reproduce ```bash agents automation-profile remove acme/strict --yes ``` Observe that no "Profile Removed" panel is shown, only a plain message. ## Impact Minor visual inconsistency — the remove command output does not match the spec's documented format. ## Subtasks - [x] Add a `Panel` render to `automation_profile.py` `remove` command handler displaying the removed profile name under the title "Profile Removed" - [x] Replace or supplement the plain `console.print` checkmark message with the spec-required panel output - [x] Write Behave unit tests covering the rich output rendering for `automation-profile remove` - [x] Write Robot Framework integration test verifying the panel is rendered on `agents automation-profile remove` ## Definition of Done - [x] `agents automation-profile remove <name> --yes` renders a `╭─ Profile Removed ─╮` Rich panel containing the profile name - [x] The `✓ OK Profile removed` success message is still printed after the panel - [x] Behave unit tests added and passing for the rich output path - [x] Robot Framework integration test added and passing - All nox stages pass - Coverage >= 97% --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-new-issue-creator
freemo added this to the v3.7.0 milestone 2026-04-05 02:58:41 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: Low (confirmed)
  • MoSCoW: Could Have — output panel cosmetic issue

Valid finding verified during batch triage.


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

Issue triaged by project owner: - **State**: Verified - **Priority**: Low (confirmed) - **MoSCoW**: Could Have — output panel cosmetic issue Valid finding verified during batch triage. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
Author
Owner

PR #3293 created on branch fix/automation-profile-remove-rich-output-panel. PR review and merge handled by continuous review stream.

Implementation summary:

  • Added spec-required Panel render to automation_profile.py remove command handler displaying the removed profile name under the title "Profile Removed"
  • Replaced plain ✓ Automation profile removed: {name} with Panel + ✓ OK Profile removed per spec
  • Added Behave unit test scenarios covering the rich output panel rendering
  • Updated Robot Framework helper to verify panel presence

All quality gates passed: lint , typecheck , security scan , dead code


Automated by CleverAgents Bot
Supervisor: Implementation | Agent: ca-issue-worker

PR #3293 created on branch `fix/automation-profile-remove-rich-output-panel`. PR review and merge handled by continuous review stream. **Implementation summary:** - Added spec-required `Panel` render to `automation_profile.py` `remove` command handler displaying the removed profile name under the title "Profile Removed" - Replaced plain `✓ Automation profile removed: {name}` with Panel + `✓ OK Profile removed` per spec - Added Behave unit test scenarios covering the rich output panel rendering - Updated Robot Framework helper to verify panel presence All quality gates passed: lint ✅, typecheck ✅, security scan ✅, dead code ✅ --- **Automated by CleverAgents Bot** Supervisor: Implementation | Agent: ca-issue-worker
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.

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