cleveragents/cleveragents-core

Fork 3

UAT: `agents resource inspect --tree` shows DAG child resources instead of spec-required filesystem file tree #4795

New issue

Open

opened 2026-04-08 18:59:48 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-08 18:59:48 +00:00

Owner

Bug Report

Feature Area: Resource registry — agents resource inspect
Severity: Medium
Found by: UAT tester instance uat-worker-resource-registry
Spec reference: docs/specification.md §agents resource inspect (lines ~11383–11676)

What Was Tested

Code-level analysis of src/cleveragents/cli/commands/resource.py — resource_inspect() function (lines ~1082–1187).

Expected Behavior (from spec)

The spec (line 11383–11428) defines agents resource inspect --tree as showing the filesystem file tree of the resource's location:

$ agents resource inspect local/api-repo --tree

╭─ Resource Tree: local/api-repo ──────────────────────────────────────╮
│                                                                      │
│ /home/user/projects/api-service                                      │
│ ├── src/                                                             │
│ │   ├── api/                                                         │
│ │   │   ├── __init__.py                                              │
│ │   │   ├── auth/                                                    │
│ │   │   │   ├── auth_middleware.py                                   │
│ │   │   │   ├── token_service.py                                     │
│ │   │   │   └── rbac.py                                              │
│ │   │   ├── routes/                                                  │
│ │   │   └── models/                                                  │
│ │   └── utils/                                                       │
│ ├── tests/                                                           │
│ ├── README.md                                                        │
│ └── pyproject.toml                                                   │
│                                                                      │
╰──────────────────────────────────────────────────────────────────────╯

╭─ Summary ──────────────╮
│ Directories: 8         │
│ Files: 41              │
│ Total Size: 284 KB     │
╰────────────────────────╯

The --tree flag is meant to show the physical filesystem contents of the resource (directories and files), not the DAG child resources.

Actual Behavior

The resource_inspect() function (lines ~1124–1130) uses service.get_resource_tree() which returns DAG child resources (other registered resources that are children in the resource DAG), not the filesystem file tree:

# resource.py lines ~1124-1130
if tree:
    tree_data = service.get_resource_tree(resource, depth=-1)
    data["children"] = _tree_to_dict(
        tree_data[0].get("children", [])
        if tree_data
        else []
    )

get_resource_tree() queries ResourceLinkModel (DAG edges between registered resources), not the filesystem. The rich display also uses _print_tree_rich() which renders resource names and types, not file paths.

Spec vs Implementation Comparison

Behavior	Spec	Implementation
`--tree` output	Filesystem file/directory tree	DAG child resource tree
Tree nodes	Files and directories with paths	Registered resource names and types
Summary	Directories count, Files count, Total Size	(none)
Panel title	"Resource Tree: {name}"	"Children:" heading

Code Location

src/cleveragents/cli/commands/resource.py — resource_inspect() (lines ~1082–1187)
src/cleveragents/application/services/_resource_registry_dag.py — get_resource_tree() (returns DAG children, not filesystem tree)
Missing: filesystem tree walker for git-checkout and fs-directory resources

Impact

agents resource inspect local/my-repo --tree shows registered child resources (git, fs-directory, etc.) instead of the actual file tree
Users cannot use inspect --tree to browse the filesystem contents of a resource
The Summary panel (Directories, Files, Total Size) is never shown

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

## Bug Report **Feature Area:** Resource registry — `agents resource inspect` **Severity:** Medium **Found by:** UAT tester instance `uat-worker-resource-registry` **Spec reference:** `docs/specification.md` §`agents resource inspect` (lines ~11383–11676) --- ### What Was Tested Code-level analysis of `src/cleveragents/cli/commands/resource.py` — `resource_inspect()` function (lines ~1082–1187). ### Expected Behavior (from spec) The spec (line 11383–11428) defines `agents resource inspect --tree` as showing the **filesystem file tree** of the resource's location: ``` $ agents resource inspect local/api-repo --tree ╭─ Resource Tree: local/api-repo ──────────────────────────────────────╮ │ │ │ /home/user/projects/api-service │ │ ├── src/ │ │ │ ├── api/ │ │ │ │ ├── __init__.py │ │ │ │ ├── auth/ │ │ │ │ │ ├── auth_middleware.py │ │ │ │ │ ├── token_service.py │ │ │ │ │ └── rbac.py │ │ │ │ ├── routes/ │ │ │ │ └── models/ │ │ │ └── utils/ │ │ ├── tests/ │ │ ├── README.md │ │ └── pyproject.toml │ │ │ ╰──────────────────────────────────────────────────────────────────────╯ ╭─ Summary ──────────────╮ │ Directories: 8 │ │ Files: 41 │ │ Total Size: 284 KB │ ╰────────────────────────╯ ``` The `--tree` flag is meant to show the **physical filesystem contents** of the resource (directories and files), not the DAG child resources. ### Actual Behavior The `resource_inspect()` function (lines ~1124–1130) uses `service.get_resource_tree()` which returns **DAG child resources** (other registered resources that are children in the resource DAG), not the filesystem file tree: ```python # resource.py lines ~1124-1130 if tree: tree_data = service.get_resource_tree(resource, depth=-1) data["children"] = _tree_to_dict( tree_data[0].get("children", []) if tree_data else [] ) ``` `get_resource_tree()` queries `ResourceLinkModel` (DAG edges between registered resources), not the filesystem. The rich display also uses `_print_tree_rich()` which renders resource names and types, not file paths. ### Spec vs Implementation Comparison | Behavior | Spec | Implementation | |----------|------|----------------| | `--tree` output | Filesystem file/directory tree | DAG child resource tree | | Tree nodes | Files and directories with paths | Registered resource names and types | | Summary | Directories count, Files count, Total Size | (none) | | Panel title | "Resource Tree: {name}" | "Children:" heading | ### Code Location - `src/cleveragents/cli/commands/resource.py` — `resource_inspect()` (lines ~1082–1187) - `src/cleveragents/application/services/_resource_registry_dag.py` — `get_resource_tree()` (returns DAG children, not filesystem tree) - Missing: filesystem tree walker for `git-checkout` and `fs-directory` resources ### Impact - `agents resource inspect local/my-repo --tree` shows registered child resources (git, fs-directory, etc.) instead of the actual file tree - Users cannot use `inspect --tree` to browse the filesystem contents of a resource - The Summary panel (Directories, Files, Total Size) is never shown --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 referenced this issue

2026-04-08 19:46:19 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 referenced this issue

2026-04-08 20:08:52 +00:00

[Automated] CleverAgents Build Session - 2026-04-08 #4799

HAL9000 added the

labels

2026-04-08 23:00:09 +00:00

HAL9000 commented

2026-04-09 03:03:46 +00:00

Author

Owner

Issue triaged by project owner:

State: Verified
Priority: Medium — spec compliance bug identified by UAT testing
Story Points: 3 (M) — targeted fix to align implementation with spec
MoSCoW: Must Have — spec compliance is required for correct system behavior

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

Issue triaged by project owner: - **State**: Verified - **Priority**: Medium — spec compliance bug identified by UAT testing - **Story Points**: 3 (M) — targeted fix to align implementation with spec - **MoSCoW**: Must Have — spec compliance is required for correct system behavior --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner

HAL9000 added

and removed

labels

2026-04-09 03:03:58 +00:00

HAL9000 added this to the v3.5.0 milestone