UAT: agents resource add output missing spec-required "Auto-discovered Children" panel with status column #2478

Open
opened 2026-04-03 18:36:51 +00:00 by freemo · 1 comment
Owner

Metadata

  • Branch: fix/resource-add-autodiscovered-children-panel
  • Commit Message: fix(cli): display auto-discovered children panel and warning on resource add
  • Milestone: v3.6.0
  • Parent Epic: #825

Background

When agents resource add registers a resource (e.g., git-checkout or fs-directory) that has auto-discovered children (including devcontainer-instance), the spec requires displaying an "Auto-discovered Children" panel showing each child's ID, Type, and Status. The current code only prints a single line: "Added resource: {name} (id: {id})".

Spec reference (lines 10722–10736): Shows a rich panel with auto-discovered children including devcontainer-instance with status "detected (not built)".

Code location: src/cleveragents/cli/commands/resource.py lines 714–731 (the resource_add function's output section)

Actual behavior: Only prints "Added resource: local/api-repo (id: 01HXR...)"

Expected behavior: Should display:

  • A resource details panel (Name, ID, Type, Physical/Virtual, Path, Branch, Created)
  • An "Auto-discovered Children" panel with columns: ID, Type, Status
  • For devcontainer-instance children: status "detected (not built)"
  • A warning message: "⚠ Devcontainer detected at .devcontainer/devcontainer.json — Container will be built lazily on first access."
  • A summary: "✓ OK Resource registered (N child resources discovered)"

Subtasks

  • Update resource_add output section in src/cleveragents/cli/commands/resource.py (lines 714–731) to render a resource details panel (Name, ID, Type, Physical/Virtual, Path, Branch, Created)
  • After registration, query discovered children from the resource handler and render an "Auto-discovered Children" panel with columns: ID, Type, Status
  • Set status "detected (not built)" for devcontainer-instance children that have not yet been built
  • Emit the spec-required warning message when a devcontainer is detected: "⚠ Devcontainer detected at .devcontainer/devcontainer.json — Container will be built lazily on first access."
  • Emit the spec-required summary line: "✓ OK Resource registered (N child resources discovered)"
  • Handle the case where no children are discovered (omit the children panel gracefully)
  • Tests (Behave): Add scenarios verifying the full output format for resource add with and without auto-discovered children
  • Tests (Behave): Add scenario verifying the devcontainer warning message is shown when applicable
  • Tests (Robot): Add integration test for resource add output format with a real git-checkout resource containing a .devcontainer/ directory
  • 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, 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.
  • All nox stages pass.
  • Coverage ≥ 97%.

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

## Metadata - **Branch**: `fix/resource-add-autodiscovered-children-panel` - **Commit Message**: `fix(cli): display auto-discovered children panel and warning on resource add` - **Milestone**: v3.6.0 - **Parent Epic**: #825 ## Background When `agents resource add` registers a resource (e.g., `git-checkout` or `fs-directory`) that has auto-discovered children (including `devcontainer-instance`), the spec requires displaying an "Auto-discovered Children" panel showing each child's ID, Type, and Status. The current code only prints a single line: `"Added resource: {name} (id: {id})"`. **Spec reference (lines 10722–10736):** Shows a rich panel with auto-discovered children including `devcontainer-instance` with status `"detected (not built)"`. **Code location:** `src/cleveragents/cli/commands/resource.py` lines 714–731 (the `resource_add` function's output section) **Actual behavior:** Only prints `"Added resource: local/api-repo (id: 01HXR...)"` **Expected behavior:** Should display: - A resource details panel (Name, ID, Type, Physical/Virtual, Path, Branch, Created) - An "Auto-discovered Children" panel with columns: ID, Type, Status - For `devcontainer-instance` children: status `"detected (not built)"` - A warning message: `"⚠ Devcontainer detected at .devcontainer/devcontainer.json — Container will be built lazily on first access."` - A summary: `"✓ OK Resource registered (N child resources discovered)"` ## Subtasks - [ ] Update `resource_add` output section in `src/cleveragents/cli/commands/resource.py` (lines 714–731) to render a resource details panel (Name, ID, Type, Physical/Virtual, Path, Branch, Created) - [ ] After registration, query discovered children from the resource handler and render an "Auto-discovered Children" panel with columns: ID, Type, Status - [ ] Set status `"detected (not built)"` for `devcontainer-instance` children that have not yet been built - [ ] Emit the spec-required warning message when a devcontainer is detected: `"⚠ Devcontainer detected at .devcontainer/devcontainer.json — Container will be built lazily on first access."` - [ ] Emit the spec-required summary line: `"✓ OK Resource registered (N child resources discovered)"` - [ ] Handle the case where no children are discovered (omit the children panel gracefully) - [ ] Tests (Behave): Add scenarios verifying the full output format for `resource add` with and without auto-discovered children - [ ] Tests (Behave): Add scenario verifying the devcontainer warning message is shown when applicable - [ ] Tests (Robot): Add integration test for `resource add` output format with a real `git-checkout` resource containing a `.devcontainer/` directory - [ ] 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, 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. - All nox stages pass. - Coverage ≥ 97%. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: ca-uat-tester
freemo added this to the v3.6.0 milestone 2026-04-03 18:37:24 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • MoSCoW: Should Have — Spec compliance or quality improvement that should be included in the milestone.

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

Issue triaged by project owner: - **State**: Verified - **MoSCoW**: Should Have — Spec compliance or quality improvement that should be included in the milestone. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: ca-project-owner
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
Reference
cleveragents/cleveragents-core#2478
No description provided.