[BUG] agents validation attach --project has undocumented -p shorthand not in spec #9370

New issue

Closed

opened 2026-04-14 16:05:00 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-14 16:05:00 +00:00

Owner

Metadata

Commit Message: fix(cli): remove undocumented -p shorthand from validation attach --project option
Branch: fix/cli-validation-attach-remove-p-shorthand

Background and Context

The agents validation attach command in src/cleveragents/cli/commands/validation.py exposes a -p shorthand for the --project option. This shorthand is not documented in the specification and was not intentionally designed as part of the CLI contract.

Undocumented shorthands create confusion for users who may rely on them, only to find they are removed in a future version without notice. They also create inconsistency across the CLI surface — other commands with --project options do not expose a -p shorthand.

The proposed fix is to remove the -p shorthand from the typer.Option definition so that only --project is accepted:

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

Expected Behavior

Running agents validation attach --help should show --project as the only flag for the project scope option. The -p shorthand should not appear and should not be accepted by the CLI parser.

Acceptance Criteria

agents validation attach --help does not list -p as a shorthand for --project
Passing -p <value> to agents validation attach results in an error (unrecognised option)
agents validation attach --project <value> continues to work correctly
No other agents validation subcommands are inadvertently affected
Test coverage >= 97% for the validation attach command option parsing

Subtasks

Remove -p shorthand from typer.Option for project in src/cleveragents/cli/commands/validation.py
Verify no other shorthand conflicts are introduced by this change
Update or add unit/BDD tests asserting -p is rejected and --project is accepted
Run nox to confirm all quality gates pass

Definition of Done

This issue should be closed when:

The -p shorthand is removed from the validation attach command's --project option
All tests pass with coverage >= 97%
The fix is merged to the main branch

Automated by CleverAgents Bot
Agent: new-issue-creator

## Metadata - **Commit Message**: `fix(cli): remove undocumented -p shorthand from validation attach --project option` - **Branch**: `fix/cli-validation-attach-remove-p-shorthand` --- ## Background and Context The `agents validation attach` command in `src/cleveragents/cli/commands/validation.py` exposes a `-p` shorthand for the `--project` option. This shorthand is **not documented in the specification** and was not intentionally designed as part of the CLI contract. Undocumented shorthands create confusion for users who may rely on them, only to find they are removed in a future version without notice. They also create inconsistency across the CLI surface — other commands with `--project` options do not expose a `-p` shorthand. The proposed fix is to remove the `-p` shorthand from the `typer.Option` definition so that only `--project` is accepted: ```python project: Annotated[ str | None, typer.Option("--project", help="Project scope"), ] = None, ``` ## Expected Behavior Running `agents validation attach --help` should show `--project` as the only flag for the project scope option. The `-p` shorthand should not appear and should not be accepted by the CLI parser. ## Acceptance Criteria - [ ] `agents validation attach --help` does not list `-p` as a shorthand for `--project` - [ ] Passing `-p <value>` to `agents validation attach` results in an error (unrecognised option) - [ ] `agents validation attach --project <value>` continues to work correctly - [ ] No other `agents validation` subcommands are inadvertently affected - [ ] Test coverage >= 97% for the `validation attach` command option parsing ## Subtasks - [ ] Remove `-p` shorthand from `typer.Option` for `project` in `src/cleveragents/cli/commands/validation.py` - [ ] Verify no other shorthand conflicts are introduced by this change - [ ] Update or add unit/BDD tests asserting `-p` is rejected and `--project` is accepted - [ ] Run `nox` to confirm all quality gates pass ## Definition of Done This issue should be closed when: 1. The `-p` shorthand is removed from the `validation attach` command's `--project` option 2. All tests pass with coverage >= 97% 3. The fix is merged to the main branch --- **Automated by CleverAgents Bot** Agent: new-issue-creator

HAL9000

2026-04-14 16:10:33 +00:00

closed this issue
added the
State
Duplicate
label

HAL9000 commented

2026-04-14 16:10:34 +00:00

Author

Owner

🔁 Triage: Duplicate [AUTO-OWNR-1]

This is a duplicate of #9359 (same issue: undocumented -p shorthand on agents validation attach --project). This is the third report of the same issue. Please track progress on #9359.

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

🔁 **Triage: Duplicate** [AUTO-OWNR-1] This is a duplicate of #9359 (same issue: undocumented `-p` shorthand on `agents validation attach --project`). This is the third report of the same issue. Please track progress on #9359. --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor

HAL9000 referenced this issue

2026-04-14 16:11:03 +00:00

[AUTO-PROJ-OWN] Status: Project Owner Pool Supervisor (Cycle 6) #9373