`automation-profile remove` rich output does not render "Profile Removed" panel as specified #9328

New issue

Open

opened 2026-04-14 14:56:26 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-14 14:56:26 +00:00

Owner

Metadata

Commit Message: fix(cli): align automation-profile remove rich output with spec
Branch: fix/automation-profile-remove-output

Background and Context

The agents automation-profile remove command's rich output does not match the specification. The specification (docs/specification.md §agents automation-profile remove) defines a "Profile Removed" panel showing the profile name, followed by a "✓ OK Profile removed" status line.

The current implementation outputs a plain text line ✓ Automation profile removed: {name} without a Rich panel.

Expected Behavior

Per the specification (§agents automation-profile remove, Rich output example), the output should be:

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

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

✓ OK Profile removed

Key differences:

A "Profile Removed" Rich panel should be rendered showing the profile name
A "✓ OK Profile removed" status line should follow the panel
The current plain text message should be replaced with the panel + status line

Current Behavior

Running agents automation-profile remove acme/strict --yes produces:

✓ Automation profile removed: acme/strict

Code location: src/cleveragents/cli/commands/automation_profile.py remove_profile() function (line 305):

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

Acceptance Criteria

agents automation-profile remove rich output shows a "Profile Removed" panel with the profile Name
A "✓ OK Profile removed" status line follows the panel
The plain text ✓ Automation profile removed: {name} message is replaced
BDD test scenario "Remove custom profile with confirmation" in features/automation_profile_cli.feature passes
All existing BDD tests continue to pass

Supporting Information

Spec reference: docs/specification.md §agents automation-profile remove (Rich output example, lines ~16964-16973)
Code location: src/cleveragents/cli/commands/automation_profile.py remove_profile() function (line 305)
Discovered during UAT testing of Automation Profile CLI feature area

Subtasks

Update remove_profile() in automation_profile.py to render a "Profile Removed" Rich panel
Panel should show Name: {profile_name}
Add "✓ OK Profile removed" status line after the panel
Remove the plain text ✓ Automation profile removed: {name} message
Update BDD tests if needed
Run nox -s unit_tests -- features/automation_profile_cli.feature and verify pass
Run nox (all default sessions), fix any errors
Verify coverage >=97% via nox -s coverage_report

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, followed by a blank line, then additional lines providing relevant details about the implementation.
The commit is pushed to the remote on the branch matching the Branch in Metadata exactly.
The commit is submitted as a pull request to master, reviewed, and merged before this issue is marked done.

Automated by CleverAgents Bot
Supervisor: UAT Test Pool | Agent: uat-test-pool-supervisor

## Metadata - **Commit Message**: `fix(cli): align automation-profile remove rich output with spec` - **Branch**: `fix/automation-profile-remove-output` ## Background and Context The `agents automation-profile remove` command's rich output does not match the specification. The specification (docs/specification.md §agents automation-profile remove) defines a "Profile Removed" panel showing the profile name, followed by a "✓ OK Profile removed" status line. The current implementation outputs a plain text line `✓ Automation profile removed: {name}` without a Rich panel. ## Expected Behavior Per the specification (§agents automation-profile remove, Rich output example), the output should be: ``` Remove automation profile local/careful-auto? [y/N]: y ╭─ Profile Removed ──────────╮ │ Name: local/careful-auto │ ╰────────────────────────────╯ ✓ OK Profile removed ``` Key differences: 1. A "Profile Removed" Rich panel should be rendered showing the profile name 2. A "✓ OK Profile removed" status line should follow the panel 3. The current plain text message should be replaced with the panel + status line ## Current Behavior Running `agents automation-profile remove acme/strict --yes` produces: ``` ✓ Automation profile removed: acme/strict ``` Code location: `src/cleveragents/cli/commands/automation_profile.py` `remove_profile()` function (line 305): ```python console.print(f"[green]✓[/green] Automation profile removed: {name}") ``` ## Acceptance Criteria - [ ] `agents automation-profile remove` rich output shows a "Profile Removed" panel with the profile Name - [ ] A "✓ OK Profile removed" status line follows the panel - [ ] The plain text `✓ Automation profile removed: {name}` message is replaced - [ ] BDD test scenario "Remove custom profile with confirmation" in `features/automation_profile_cli.feature` passes - [ ] All existing BDD tests continue to pass ## Supporting Information - Spec reference: `docs/specification.md` §agents automation-profile remove (Rich output example, lines ~16964-16973) - Code location: `src/cleveragents/cli/commands/automation_profile.py` `remove_profile()` function (line 305) - Discovered during UAT testing of Automation Profile CLI feature area ## Subtasks - [ ] Update `remove_profile()` in `automation_profile.py` to render a "Profile Removed" Rich panel - [ ] Panel should show `Name: {profile_name}` - [ ] Add "✓ OK Profile removed" status line after the panel - [ ] Remove the plain text `✓ Automation profile removed: {name}` message - [ ] Update BDD tests if needed - [ ] Run `nox -s unit_tests -- features/automation_profile_cli.feature` and verify pass - [ ] Run `nox` (all default sessions), fix any errors - [ ] Verify coverage >=97% via `nox -s coverage_report` ## 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, followed by a blank line, then additional lines providing relevant details about the implementation. - The commit is pushed to the remote on the branch matching the **Branch** in Metadata exactly. - The commit is submitted as a **pull request** to `master`, reviewed, and **merged** before this issue is marked done. --- **Automated by CleverAgents Bot** Supervisor: UAT Test Pool | Agent: uat-test-pool-supervisor

HAL9000 added the

labels

2026-04-14 14:58:12 +00:00

HAL9000 added this to the v3.5.0 milestone

2026-04-14 14:58:13 +00:00

HAL9000 commented

2026-04-14 14:58:13 +00:00

Author

Owner

✅ Triage: Verified [AUTO-OWNR-1]

Valid bug: agents automation-profile remove rich output renders a plain text message instead of the spec-required "Profile Removed" Rich panel followed by a "✓ OK Profile removed" status line. This is a spec compliance issue in the output format.

Assigning to v3.5.0 (Autonomy Hardening) as automation profiles are a core M6 feature. Priority Medium — output format issue, the command still functions.

MoSCoW: Should Have — spec-compliant output is important for user experience and consistency.

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

✅ **Triage: Verified** [AUTO-OWNR-1] Valid bug: `agents automation-profile remove` rich output renders a plain text message instead of the spec-required "Profile Removed" Rich panel followed by a "✓ OK Profile removed" status line. This is a spec compliance issue in the output format. Assigning to **v3.5.0** (Autonomy Hardening) as automation profiles are a core M6 feature. Priority **Medium** — output format issue, the command still functions. MoSCoW: **Should Have** — spec-compliant output is important for user experience and consistency. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor