UAT: `agents validation detach` rich output is a plain text line instead of spec-required structured panel #4482

New issue

Open

opened 2026-04-08 13:26:27 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-08 13:26:27 +00:00

Owner

Metadata

Branch: fix/validation-detach-rich-panel-output
Commit Message: fix(cli): replace plain text detach output with spec-required rich panel
Milestone: v3.8.0
Parent Epic: #4182

Bug Report

Feature Area: Validation CLI — agents validation detach

Expected Behavior (from spec)

The spec (CLI Commands > agents validation detach) shows that agents validation detach should output a rich panel:

╭─ Validation Detached ───────────────────────────────────────╮
│ Attachment ID: 01HXM5A1B2C3D4E5F6G7H8J9K0                   │
│ Validation:    local/run-tests                               │
│ Resource:      local/api-repo                                │
│ Scope:         project local/api-service                     │
╰─────────────────────────────────────────────────────────────╯

✓ OK Validation detached

The panel must include:

Attachment ID (the ULID that was detached)
Validation (the validation name)
Resource (the resource it was attached to)
Scope (the scope it was attached under)

Actual Behavior

The implementation in src/cleveragents/cli/commands/validation.py (detach command) outputs a single plain text line:

console.print(f"[green]Detached validation:[/green] {attachment_id}")

This produces output like:

Detached validation: 01HXM5A1B2C3D4E5F6G7H8J9K0

This is missing:

The structured ╭─ Validation Detached ─╮ panel
The Validation name (which validation was detached)
The Resource field (which resource it was detached from)
The Scope field (what scope was removed)
The ✓ OK Validation detached success indicator

Code Location

src/cleveragents/cli/commands/validation.py — detach() function, the rich output branch.

Impact

After detaching a validation, users cannot confirm which validation was removed from which resource at which scope. The output only shows the ULID, which is not human-readable. The spec explicitly shows a structured panel with full context.

Steps to Reproduce

Attach a validation and note the attachment ULID
Run: agents validation detach <ATTACHMENT_ULID>
Observe the output is a plain text line, not the spec-required panel

Subtasks

Locate the detach() function in src/cleveragents/cli/commands/validation.py
Retrieve the full attachment record (validation name, resource, scope) from the detach response or service layer
Replace the plain console.print line with a rich.panel.Panel matching the spec layout
Add the ✓ OK Validation detached success line below the panel
Write/update Behave unit tests in features/ covering the new rich panel output
Write/update Robot Framework integration tests in robot/ verifying the panel fields
Ensure all nox quality gates pass (nox -e lint, nox -e typecheck, nox -e unit_tests, nox -e integration_tests)
Verify coverage remains >= 97%

Definition of Done

agents validation detach <ULID> outputs a ╭─ Validation Detached ─╮ rich panel
Panel includes Attachment ID, Validation, Resource, and Scope fields
✓ OK Validation detached success line is printed below the panel
Output matches the spec layout exactly
Behave unit tests cover the rich panel rendering
Robot Framework integration tests verify end-to-end panel output
All nox stages pass
Coverage >= 97%

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

## Metadata - **Branch**: `fix/validation-detach-rich-panel-output` - **Commit Message**: `fix(cli): replace plain text detach output with spec-required rich panel` - **Milestone**: v3.8.0 - **Parent Epic**: #4182 ## Bug Report **Feature Area:** Validation CLI — `agents validation detach` ### Expected Behavior (from spec) The spec (CLI Commands > agents validation detach) shows that `agents validation detach` should output a rich panel: ``` ╭─ Validation Detached ───────────────────────────────────────╮ │ Attachment ID: 01HXM5A1B2C3D4E5F6G7H8J9K0 │ │ Validation: local/run-tests │ │ Resource: local/api-repo │ │ Scope: project local/api-service │ ╰─────────────────────────────────────────────────────────────╯ ✓ OK Validation detached ``` The panel must include: - `Attachment ID` (the ULID that was detached) - `Validation` (the validation name) - `Resource` (the resource it was attached to) - `Scope` (the scope it was attached under) ### Actual Behavior The implementation in `src/cleveragents/cli/commands/validation.py` (`detach` command) outputs a single plain text line: ```python console.print(f"[green]Detached validation:[/green] {attachment_id}") ``` This produces output like: ``` Detached validation: 01HXM5A1B2C3D4E5F6G7H8J9K0 ``` This is missing: 1. The structured `╭─ Validation Detached ─╮` panel 2. The `Validation` name (which validation was detached) 3. The `Resource` field (which resource it was detached from) 4. The `Scope` field (what scope was removed) 5. The `✓ OK Validation detached` success indicator ### Code Location `src/cleveragents/cli/commands/validation.py` — `detach()` function, the rich output branch. ### Impact After detaching a validation, users cannot confirm which validation was removed from which resource at which scope. The output only shows the ULID, which is not human-readable. The spec explicitly shows a structured panel with full context. ### Steps to Reproduce 1. Attach a validation and note the attachment ULID 2. Run: `agents validation detach <ATTACHMENT_ULID>` 3. Observe the output is a plain text line, not the spec-required panel ## Subtasks - [ ] Locate the `detach()` function in `src/cleveragents/cli/commands/validation.py` - [ ] Retrieve the full attachment record (validation name, resource, scope) from the detach response or service layer - [ ] Replace the plain `console.print` line with a `rich.panel.Panel` matching the spec layout - [ ] Add the `✓ OK Validation detached` success line below the panel - [ ] Write/update Behave unit tests in `features/` covering the new rich panel output - [ ] Write/update Robot Framework integration tests in `robot/` verifying the panel fields - [ ] Ensure all nox quality gates pass (`nox -e lint`, `nox -e typecheck`, `nox -e unit_tests`, `nox -e integration_tests`) - [ ] Verify coverage remains >= 97% ## Definition of Done - [ ] `agents validation detach <ULID>` outputs a `╭─ Validation Detached ─╮` rich panel - [ ] Panel includes `Attachment ID`, `Validation`, `Resource`, and `Scope` fields - [ ] `✓ OK Validation detached` success line is printed below the panel - [ ] Output matches the spec layout exactly - [ ] Behave unit tests cover the rich panel rendering - [ ] Robot Framework integration tests verify end-to-end panel output - [ ] All nox stages pass - [ ] Coverage >= 97% > **Backlog note:** This issue was discovered during autonomous operation > on milestone v3.8.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: new-issue-creator

HAL9000 added this to the v3.8.0 milestone

2026-04-08 13:28:26 +00:00

HAL9000 added a new dependency

2026-04-08 13:28:30 +00:00

#4182 Epic: Legacy CLI command removal and full verification of CLI interface and workflows

HAL9000 added the