cleveragents/cleveragents-core

Fork 3

UAT: `agents validation attach` output missing spec-required "Validation Attached" panel with Attachment ID, Mode, Resource, and Scope fields #4117

New issue

Open

opened 2026-04-06 10:28:22 +00:00 by freemo · 1 comment

freemo commented

2026-04-06 10:28:22 +00:00

Owner

Metadata

Branch: fix/cli/validation-attach-output-panel
Commit Message: fix(cli): add spec-required Validation Attached panel to agents validation attach output
Milestone: None (backlog)
Parent Epic: #3374

Summary

The agents validation attach command outputs a simple one-liner instead of the rich "Validation Attached" panel required by the specification. The panel must include Attachment ID, Validation name, Mode, Resource, and Scope fields.

What Was Tested

Reviewed src/cleveragents/cli/commands/validation.py, the attach() function (line 278), and compared its output against the specification (spec lines 9556–9570 and 9631–9650).

Expected Behavior (from spec)

The spec (lines 9556–9570) requires:

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

✓ OK Validation attached

Actual Behavior

The code at src/cleveragents/cli/commands/validation.py lines 344–348 outputs:

console.print(
    f"[green]Attached validation:[/green] {validation_name} -> "
    f"{resource} (id: {att_id})"
)

Which produces:

Attached validation: local/run-tests -> local/api-repo (id: 01HXM5A1B2C3D4E5F6G7H8J9K0)

Code Location

File: src/cleveragents/cli/commands/validation.py
Function: attach() at line 278
Output section: lines 340–348

Steps to Reproduce

Register a validation: agents validation add --config ./validations/unit-tests.yaml --required
Attach it: agents validation attach local/api-repo local/unit-tests
Observe the output — a one-liner is printed instead of a panel

Impact on Workflow Examples

This blocks visual verification of workflow examples that use agents validation attach. The command functions correctly (validation is attached), but the output does not match the spec.

Subtasks

Replace one-liner output with a "Validation Attached" panel
Panel must include: Attachment ID, Validation name, Mode, Resource, Scope (project/plan/none)
Add ✓ OK Validation attached confirmation line after the panel

Definition of Done

agents validation attach local/api-repo local/unit-tests outputs a "Validation Attached" panel matching the spec format
Scope field correctly shows "project ", "plan ", or "(none)" based on flags used
All nox stages pass
Coverage >= 97%

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

## Metadata - **Branch**: `fix/cli/validation-attach-output-panel` - **Commit Message**: `fix(cli): add spec-required Validation Attached panel to agents validation attach output` - **Milestone**: None (backlog) - **Parent Epic**: #3374 ## Summary The `agents validation attach` command outputs a simple one-liner instead of the rich "Validation Attached" panel required by the specification. The panel must include Attachment ID, Validation name, Mode, Resource, and Scope fields. ## What Was Tested Reviewed `src/cleveragents/cli/commands/validation.py`, the `attach()` function (line 278), and compared its output against the specification (spec lines 9556–9570 and 9631–9650). ## Expected Behavior (from spec) The spec (lines 9556–9570) requires: ``` ╭─ Validation Attached ──────────────────────────────────────╮ │ Attachment ID: 01HXM5A1B2C3D4E5F6G7H8J9K0 │ │ Validation: local/run-tests │ │ Mode: required │ │ Resource: local/api-repo │ │ Scope: project local/api-service │ ╰────────────────────────────────────────────────────────────╯ ✓ OK Validation attached ``` ## Actual Behavior The code at `src/cleveragents/cli/commands/validation.py` lines 344–348 outputs: ```python console.print( f"[green]Attached validation:[/green] {validation_name} -> " f"{resource} (id: {att_id})" ) ``` Which produces: ``` Attached validation: local/run-tests -> local/api-repo (id: 01HXM5A1B2C3D4E5F6G7H8J9K0) ``` ## Code Location - File: `src/cleveragents/cli/commands/validation.py` - Function: `attach()` at line 278 - Output section: lines 340–348 ## Steps to Reproduce 1. Register a validation: `agents validation add --config ./validations/unit-tests.yaml --required` 2. Attach it: `agents validation attach local/api-repo local/unit-tests` 3. Observe the output — a one-liner is printed instead of a panel ## Impact on Workflow Examples This blocks visual verification of workflow examples that use `agents validation attach`. The command functions correctly (validation is attached), but the output does not match the spec. ## Subtasks - [ ] Replace one-liner output with a "Validation Attached" panel - [ ] Panel must include: Attachment ID, Validation name, Mode, Resource, Scope (project/plan/none) - [ ] Add `✓ OK Validation attached` confirmation line after the panel ## Definition of Done - [ ] `agents validation attach local/api-repo local/unit-tests` outputs a "Validation Attached" panel matching the spec format - [ ] Scope field correctly shows "project <name>", "plan <id>", or "(none)" based on flags used - [ ] All nox stages pass - [ ] Coverage >= 97% > **Backlog note:** This issue was discovered during autonomous operation > on milestone None. 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: ca-new-issue-creator

freemo added a new dependency

2026-04-06 10:28:39 +00:00

#3374 Epic: E2E Workflow Specification Tests & Code Review Tool Examples

freemo added the

Points

label

2026-04-06 10:32:29 +00:00

freemo referenced this issue

2026-04-06 10:33:32 +00:00

[Automated] CleverAgents Build Session - 2026-04-06 #3775

freemo referenced this issue

2026-04-06 10:34:17 +00:00

[Automated] CleverAgents Build Session - 2026-04-06 #3775

freemo referenced this issue

2026-04-06 10:35:53 +00:00

[Automated] CleverAgents Build Session - 2026-04-06 #3775

freemo added this to the v3.6.0 milestone

2026-04-06 18:07:49 +00:00

freemo commented

2026-04-06 18:58:43 +00:00

Author

Owner

Milestone Triage Decision: Moved to Backlog (Belongs in Earlier Milestone)

This CLI output panel fix has been moved out of v3.6.0 during aggressive milestone triage. Basic CLI panel fixes belong in earlier milestones, not in Advanced Concepts.

Reasoning:

v3.6.0 focus: Advanced concepts that extend beyond core MVP
This issue: Validation attach output panel - basic CLI functionality
Impact: Basic CLI polish, not advanced conceptual capability

Should be addressed in v3.3.0-v3.4.0 alongside validation core functionality.

**Milestone Triage Decision: Moved to Backlog (Belongs in Earlier Milestone)** This CLI output panel fix has been moved out of v3.6.0 during aggressive milestone triage. Basic CLI panel fixes belong in earlier milestones, not in Advanced Concepts. **Reasoning:** - v3.6.0 focus: Advanced concepts that extend beyond core MVP - This issue: Validation attach output panel - basic CLI functionality - Impact: Basic CLI polish, not advanced conceptual capability Should be addressed in v3.3.0-v3.4.0 alongside validation core functionality.

freemo removed this from the v3.6.0 milestone

2026-04-06 20:39:46 +00:00