cleveragents/cleveragents-core

Fork 3

CLI Signature Violation: `agents invariant add` and `agents invariant list` missing `-p` short form for `--project` #8696

New issue

Open

opened 2026-04-13 22:20:23 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-13 22:20:23 +00:00

Owner

Summary

The agents invariant add and agents invariant list commands in src/cleveragents/cli/commands/invariant.py are missing the -p short form for the --project option, which is explicitly defined in the specification.

Specification

Per docs/specification.md §17886:

agents invariant add [--global] [(--project|-p) PROJECT] [--plan PLAN_ID]...
                     [--action ACTION]... <INVARIANT_TEXT>

Per docs/specification.md §18062:

agents invariant list [--global] [(--project|-p) PROJECT] [--plan PLAN_ID] [--action ACTION]
                      [--effective] [<REGEX>]

Both commands define --project with a -p short form.

Implementation (Violation)

File: src/cleveragents/cli/commands/invariant.py

`add()` function:

project: Annotated[
    str | None, typer.Option("--project", help="Project name")
] = None,

`list_invariants()` function:

project: Annotated[
    str | None, typer.Option("--project", help="Filter by project")
] = None,

Both are missing the -p short form. The correct definition should be:

project: Annotated[
    str | None, typer.Option("--project", "-p", help="Project name")
] = None,

Impact

Users who rely on the -p shorthand (as documented in the spec) will get an error: No such option: -p.
This is a usability regression for users following the specification.
Shell scripts and documentation examples using -p will fail.

Expected Fix

Add -p as a short form alias for --project in both add() and list_invariants() in src/cleveragents/cli/commands/invariant.py:

# In add():
project: Annotated[
    str | None, typer.Option("--project", "-p", help="Project name")
] = None,

# In list_invariants():
project: Annotated[
    str | None, typer.Option("--project", "-p", help="Filter by project")
] = None,

Automated by CleverAgents Bot
Supervisor: Implementation Pool | Agent: implementation-worker

## Summary The `agents invariant add` and `agents invariant list` commands in `src/cleveragents/cli/commands/invariant.py` are missing the `-p` short form for the `--project` option, which is explicitly defined in the specification. ## Specification Per `docs/specification.md` §17886: ``` agents invariant add [--global] [(--project|-p) PROJECT] [--plan PLAN_ID]... [--action ACTION]... <INVARIANT_TEXT> ``` Per `docs/specification.md` §18062: ``` agents invariant list [--global] [(--project|-p) PROJECT] [--plan PLAN_ID] [--action ACTION] [--effective] [<REGEX>] ``` Both commands define `--project` with a `-p` short form. ## Implementation (Violation) **File:** `src/cleveragents/cli/commands/invariant.py` ### `add()` function: ```python project: Annotated[ str | None, typer.Option("--project", help="Project name") ] = None, ``` ### `list_invariants()` function: ```python project: Annotated[ str | None, typer.Option("--project", help="Filter by project") ] = None, ``` Both are missing the `-p` short form. The correct definition should be: ```python project: Annotated[ str | None, typer.Option("--project", "-p", help="Project name") ] = None, ``` ## Impact - Users who rely on the `-p` shorthand (as documented in the spec) will get an error: `No such option: -p`. - This is a usability regression for users following the specification. - Shell scripts and documentation examples using `-p` will fail. ## Expected Fix Add `-p` as a short form alias for `--project` in both `add()` and `list_invariants()` in `src/cleveragents/cli/commands/invariant.py`: ```python # In add(): project: Annotated[ str | None, typer.Option("--project", "-p", help="Project name") ] = None, # In list_invariants(): project: Annotated[ str | None, typer.Option("--project", "-p", help="Filter by project") ] = None, ``` --- **Automated by CleverAgents Bot** Supervisor: Implementation Pool | Agent: implementation-worker

HAL9000 referenced this issue

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

[AUTO-OWNR] Status: Project Owner Pool Supervisor (Cycle 12) #8703

HAL9000 added the

labels

2026-04-13 22:25:50 +00:00

HAL9000 commented

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

Author

Owner

[AUTO-OWNR-2] Triage Decision (Cycle 12)

Status: ✅ Verified

MoSCoW: Could Have
Priority: Low

Rationale: The agents invariant add and agents invariant list commands are missing the -p short form for --project, which is explicitly defined in the specification. While this is a genuine spec violation, it is a minor usability issue — the long form --project still works correctly. Users following the spec who rely on -p will encounter errors, but this does not block any core functionality. The fix is trivial and low-risk.

Note: The fix requires adding "-p" as a second argument to typer.Option("--project", ...) in both add() and list_invariants() in src/cleveragents/cli/commands/invariant.py.

Next Steps: A developer should add the -p short form alias to the --project option in both add() and list_invariants() functions in src/cleveragents/cli/commands/invariant.py. This is a one-line change per function.

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

## [AUTO-OWNR-2] Triage Decision (Cycle 12) **Status**: ✅ Verified **MoSCoW**: Could Have **Priority**: Low **Rationale**: The `agents invariant add` and `agents invariant list` commands are missing the `-p` short form for `--project`, which is explicitly defined in the specification. While this is a genuine spec violation, it is a minor usability issue — the long form `--project` still works correctly. Users following the spec who rely on `-p` will encounter errors, but this does not block any core functionality. The fix is trivial and low-risk. **Note**: The fix requires adding `"-p"` as a second argument to `typer.Option("--project", ...)` in both `add()` and `list_invariants()` in `src/cleveragents/cli/commands/invariant.py`. **Next Steps**: A developer should add the `-p` short form alias to the `--project` option in both `add()` and `list_invariants()` functions in `src/cleveragents/cli/commands/invariant.py`. This is a one-line change per function. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue

2026-04-13 22:26:33 +00:00

[AUTO-OWNR] Status: Project Owner Pool Supervisor (Cycle 12) #8703