UAT: `agents skill add` Tool Sources panel missing `Details` column #4531

New issue

Open

opened 2026-04-08 14:18:54 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-08 14:18:54 +00:00

Owner

Bug Report

Feature Area: Skills CLI — agents skill add rich output Tool Sources panel

Expected Behavior (from spec)

The spec (Section "agents skill add", lines 6395-6404) shows the Tool Sources panel with three columns: Source, Count, and Details:

╭─ Tool Sources ────────────────────────────────╮
│ Source         Count  Details                 │
│ ─────────────  ─────  ───────────────────     │
│ builtin        14     file, dir, git, shell   │
│ mcp            6      github (4), linear (2)  │
│ agent_skill    2      deploy, code-review     │
│ custom         1      run_migrations          │
│ ─────────────  ─────  ───────────────────     │
│ Total:         23                             │
╰───────────────────────────────────────────────╯

The Details column provides human-readable context about what tools are in each source category.

Actual Behavior (from code)

In src/cleveragents/cli/commands/skill.py, _print_skill_registered function (lines ~175-195):

source_table = Table(show_header=True, show_edge=False, pad_edge=False)
source_table.add_column("Source", style="cyan")
source_table.add_column("Count", justify="right")

Only two columns are created — Source and Count. The Details column is completely absent.

Steps to Reproduce

Create a skill YAML with multiple tool sources (builtin tools, MCP servers, inline tools)
Run: agents skill add --config my-skill.yaml
Observe the Tool Sources panel — it shows only Source and Count columns, no Details

Code Location

src/cleveragents/cli/commands/skill.py, _print_skill_registered function, lines ~175-195

Fix Suggestion

Add a Details column to the source table and populate it with tool names/counts per source:

source_table.add_column("Source", style="cyan")
source_table.add_column("Count", justify="right")
source_table.add_column("Details")

For builtin tools, list the tool short names. For MCP, show server_name (N tools). For agent_skill, show the skill names. For custom, show the inline tool names.

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

## Bug Report **Feature Area:** Skills CLI — `agents skill add` rich output Tool Sources panel ### Expected Behavior (from spec) The spec (Section "agents skill add", lines 6395-6404) shows the Tool Sources panel with **three columns**: `Source`, `Count`, and `Details`: ``` ╭─ Tool Sources ────────────────────────────────╮ │ Source Count Details │ │ ───────────── ───── ─────────────────── │ │ builtin 14 file, dir, git, shell │ │ mcp 6 github (4), linear (2) │ │ agent_skill 2 deploy, code-review │ │ custom 1 run_migrations │ │ ───────────── ───── ─────────────────── │ │ Total: 23 │ ╰───────────────────────────────────────────────╯ ``` The `Details` column provides human-readable context about what tools are in each source category. ### Actual Behavior (from code) In `src/cleveragents/cli/commands/skill.py`, `_print_skill_registered` function (lines ~175-195): ```python source_table = Table(show_header=True, show_edge=False, pad_edge=False) source_table.add_column("Source", style="cyan") source_table.add_column("Count", justify="right") ``` Only **two columns** are created — `Source` and `Count`. The `Details` column is completely absent. ### Steps to Reproduce 1. Create a skill YAML with multiple tool sources (builtin tools, MCP servers, inline tools) 2. Run: `agents skill add --config my-skill.yaml` 3. Observe the Tool Sources panel — it shows only `Source` and `Count` columns, no `Details` ### Code Location - `src/cleveragents/cli/commands/skill.py`, `_print_skill_registered` function, lines ~175-195 ### Fix Suggestion Add a `Details` column to the source table and populate it with tool names/counts per source: ```python source_table.add_column("Source", style="cyan") source_table.add_column("Count", justify="right") source_table.add_column("Details") ``` For builtin tools, list the tool short names. For MCP, show `server_name (N tools)`. For agent_skill, show the skill names. For custom, show the inline tool names. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 referenced this issue

2026-04-08 17:37:42 +00:00

UAT: agents skill add rich output "Tool Sources" table missing spec-required "Details" column; JSON output uses wrong data structure #4614

HAL9000 commented

2026-04-08 17:37:47 +00:00

Author

Owner

📝 Scope Extension Note

Issue #4614 (now closed as duplicate) also reported a related JSON output structure mismatch for agents skill add --format json:

data.skill.name expected but data.name returned (skill data not nested under skill key)
data.skill.config expected but data.config_path returned (wrong field name)
data.tool_sources (array with count/details) expected but data.sources (array of strings) returned
data.total_tools missing from JSON output
data.includes[].status not shown
messages[].text = "ok" instead of "Skill registered with N tools"
command field is empty string instead of actual command

This JSON structure issue is related to the same _skill_spec_dict() function in src/cleveragents/cli/commands/skill.py. Consider expanding the scope of this issue to cover both the rich output Details column AND the JSON structure fix.

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

## 📝 Scope Extension Note Issue #4614 (now closed as duplicate) also reported a related JSON output structure mismatch for `agents skill add --format json`: - `data.skill.name` expected but `data.name` returned (skill data not nested under `skill` key) - `data.skill.config` expected but `data.config_path` returned (wrong field name) - `data.tool_sources` (array with count/details) expected but `data.sources` (array of strings) returned - `data.total_tools` missing from JSON output - `data.includes[].status` not shown - `messages[].text` = "ok" instead of "Skill registered with N tools" - `command` field is empty string instead of actual command This JSON structure issue is related to the same `_skill_spec_dict()` function in `src/cleveragents/cli/commands/skill.py`. Consider expanding the scope of this issue to cover both the rich output Details column AND the JSON structure fix. --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 added this to the v3.5.0 milestone

2026-04-08 17:41:46 +00:00

HAL9000 added the