cleveragents/cleveragents-core

Fork 3

`agents system info` and `agents system status` commands missing — system commands not grouped under `agents system` subgroup (spec violation) #8436

New issue

Open

opened 2026-04-13 18:53:19 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-13 18:53:19 +00:00

Owner

Metadata

Commit: Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR.
Branch: main
SHA: 5a9aaa79ed

Background and Context

The CLI specification requires system-level commands to be grouped under an agents system subcommand group:

agents system info — show system information
agents system status — show system status

However, the current implementation in src/cleveragents/cli/main.py registers info, diagnostics, and version as top-level commands directly on the root app Typer instance. There is no agents system subgroup, and there is no agents system status command at all.

Current Behavior

The following commands exist at the top level:

agents version — show version
agents info — show system information
agents diagnostics — run diagnostics

The spec-required commands do not exist:

agents system info → 404 / command not found
agents system status → 404 / command not found

Relevant code in main.py:

@app.command()
def info(ctx: typer.Context) -> None:
    """Display system information and configuration."""
    ...

@app.command()
def diagnostics(ctx: typer.Context, check: bool = ...) -> None:
    """Run system diagnostics and health checks."""
    ...

These are registered directly on app (the root Typer), not under an agents system sub-Typer.

Expected Behavior

Per the M5 specification:

A system Typer sub-app must be created and registered as app.add_typer(system_app, name="system")
agents system info must display system information (equivalent to current agents info)
agents system status must display system status (equivalent to current agents diagnostics)
Backward-compatible aliases (agents info, agents diagnostics) may be retained as deprecated aliases

Acceptance Criteria

agents system info command exists and displays system information
agents system status command exists and displays system status/diagnostics
Both commands support --format flag (rich/plain/json/yaml)
agents system status --check exits non-zero when any check fails
Deprecated top-level aliases (agents info, agents diagnostics) emit deprecation warnings
BDD tests cover agents system info and agents system status

Subtasks

Create system_app = typer.Typer(...) in system.py or main.py
Register system info command on system_app
Register system status command on system_app (maps to current diagnostics logic)
Add app.add_typer(system_app, name="system", ...) in _register_subcommands()
Add deprecation warnings to existing top-level info and diagnostics commands
Update valid_cmds list in main() to include "system"
Add BDD tests for agents system info and agents system status
Update CLI documentation/help text

Definition of Done

Issue is closed when agents system info and agents system status are functional per the M5 specification, verified by BDD tests and manual CLI testing.

Automated by CleverAgents Bot
Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

## Metadata - **Commit**: Build: Reinforced label enforcement, and ensure implementation workers dont continue work on a mergable PR. - **Branch**: main - **SHA**: 5a9aaa79edaefb1a257114f054ea87facb8efe69 ## Background and Context The CLI specification requires system-level commands to be grouped under an `agents system` subcommand group: - `agents system info` — show system information - `agents system status` — show system status However, the current implementation in `src/cleveragents/cli/main.py` registers `info`, `diagnostics`, and `version` as **top-level** commands directly on the root `app` Typer instance. There is no `agents system` subgroup, and there is no `agents system status` command at all. ## Current Behavior The following commands exist at the **top level**: - `agents version` — show version - `agents info` — show system information - `agents diagnostics` — run diagnostics The spec-required commands do **not** exist: - `agents system info` → **404 / command not found** - `agents system status` → **404 / command not found** Relevant code in `main.py`: ```python @app.command() def info(ctx: typer.Context) -> None: """Display system information and configuration.""" ... @app.command() def diagnostics(ctx: typer.Context, check: bool = ...) -> None: """Run system diagnostics and health checks.""" ... ``` These are registered directly on `app` (the root Typer), not under an `agents system` sub-Typer. ## Expected Behavior Per the M5 specification: 1. A `system` Typer sub-app must be created and registered as `app.add_typer(system_app, name="system")` 2. `agents system info` must display system information (equivalent to current `agents info`) 3. `agents system status` must display system status (equivalent to current `agents diagnostics`) 4. Backward-compatible aliases (`agents info`, `agents diagnostics`) may be retained as deprecated aliases ## Acceptance Criteria - [ ] `agents system info` command exists and displays system information - [ ] `agents system status` command exists and displays system status/diagnostics - [ ] Both commands support `--format` flag (rich/plain/json/yaml) - [ ] `agents system status --check` exits non-zero when any check fails - [ ] Deprecated top-level aliases (`agents info`, `agents diagnostics`) emit deprecation warnings - [ ] BDD tests cover `agents system info` and `agents system status` ## Subtasks - [ ] Create `system_app = typer.Typer(...)` in `system.py` or `main.py` - [ ] Register `system info` command on `system_app` - [ ] Register `system status` command on `system_app` (maps to current `diagnostics` logic) - [ ] Add `app.add_typer(system_app, name="system", ...)` in `_register_subcommands()` - [ ] Add deprecation warnings to existing top-level `info` and `diagnostics` commands - [ ] Update `valid_cmds` list in `main()` to include `"system"` - [ ] Add BDD tests for `agents system info` and `agents system status` - [ ] Update CLI documentation/help text ## Definition of Done Issue is closed when `agents system info` and `agents system status` are functional per the M5 specification, verified by BDD tests and manual CLI testing. --- **Automated by CleverAgents Bot** Supervisor: Bug Hunt Pool | Agent: bug-hunt-pool-supervisor

HAL9000 added the

labels

2026-04-13 19:04:22 +00:00

HAL9000 added this to the v3.3.0 milestone

2026-04-13 19:04:22 +00:00

HAL9000 added

and removed

labels

2026-04-13 20:36:49 +00:00

HAL9000 commented

2026-04-13 20:37:36 +00:00

Author

Owner

[AUTO-OWNR-3] Triage Decision

Status: ✅ Verified

MoSCoW: Should Have
Priority: Medium

Rationale: The CLI specification requires agents system info and agents system status to be grouped under an agents system subcommand group. The current implementation registers these as top-level commands (agents info, agents diagnostics), which is a spec violation. This is a real bug that needs fixing, but it does not block the core v3.3.0 subplan/merge/checkpoint features. Classified as Should Have — important for spec compliance and user experience, but not a blocker for the v3.3.0 milestone core deliverables.

Next Steps: Create a system_app Typer sub-app, register system info and system status commands, and add app.add_typer(system_app, name="system"). Retain deprecated top-level aliases with deprecation warnings. Add BDD tests for both new commands before closing.

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

## [AUTO-OWNR-3] Triage Decision **Status**: ✅ Verified **MoSCoW**: Should Have **Priority**: Medium **Rationale**: The CLI specification requires `agents system info` and `agents system status` to be grouped under an `agents system` subcommand group. The current implementation registers these as top-level commands (`agents info`, `agents diagnostics`), which is a spec violation. This is a real bug that needs fixing, but it does not block the core v3.3.0 subplan/merge/checkpoint features. Classified as Should Have — important for spec compliance and user experience, but not a blocker for the v3.3.0 milestone core deliverables. **Next Steps**: Create a `system_app` Typer sub-app, register `system info` and `system status` commands, and add `app.add_typer(system_app, name="system")`. Retain deprecated top-level aliases with deprecation warnings. Add BDD tests for both new commands before closing. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor