Command reference documents short flags that do not exist #52

Problem Statement

The command reference currently states that global short flags -v and -q are available, but the actual CLI only supports long-form --verbose and --quiet. This creates a documentation/runtime mismatch that causes avoidable command failures for users and agent workflows following the reference.

Impact: Commands like trop -v list and trop -q reserve fail at argument parsing instead of running, which can break scripts and automated usage based on the documented interface.

Affected Files / Lines

• plugins/trop/skills/trop-reference/references/cli-commands.md:57
  • Documents: --verbose / -v and --quiet / -q.
• trop-cli/src/cli.rs:22
  • verbose is defined as long-only: #[arg(long, global = true)].
• trop-cli/src/cli.rs:26
  • quiet is defined as long-only: #[arg(long, global = true)].

Additional source-of-truth context:

• reference/ImplementationSpecification.md:333-337 specifies long forms only (--verbose, --quiet, etc.).

Recommended Fix

Align the command reference with the implemented/spec'd CLI behavior by removing unsupported short aliases from the docs.

--- a/plugins/trop/skills/trop-reference/references/cli-commands.md
+++ b/plugins/trop/skills/trop-reference/references/cli-commands.md
@@
-All commands accept: `--verbose` / `-v`, `--quiet` / `-q`, `--data-dir <path>`, `--busy-timeout <seconds>`, `--help` / `-h`
+All commands accept: `--verbose`, `--quiet`, `--data-dir <path>`, `--busy-timeout <seconds>`, `--help` / `-h`

Optional hardening to prevent future drift: add tests that explicitly verify -v and -q are rejected unless short aliases are intentionally introduced later.

// trop-cli/tests/cli.rs
#[test]
fn test_cli_short_verbose_flag_not_supported() {
    let mut cmd = Command::cargo_bin("trop").unwrap();
    cmd.arg("-v").arg("list");
    cmd.assert().failure().stderr(predicate::str::contains("error:"));
}

#[test]
fn test_cli_short_quiet_flag_not_supported() {
    let mut cmd = Command::cargo_bin("trop").unwrap();
    cmd.arg("-q").arg("list");
    cmd.assert().failure().stderr(predicate::str::contains("error:"));
}

Validation Strategy

1. Docs validation

   • Confirm plugins/trop/skills/trop-reference/references/cli-commands.md no longer advertises -v/-q.

2. Manual CLI verification

   • trop --verbose list succeeds.
   • trop --quiet list succeeds.
   • trop -v list fails with argument parsing error.
   • trop -q list fails with argument parsing error.

3. Automated checks

   • Add/adjust tests in trop-cli/tests/cli.rs (or trop-cli/tests/global_options.rs) for unsupported short flags.
   • Run cargo test to ensure behavior is documented and enforced.

🤖 Discovered by Codex · expanded by Codex · filed by Claude Code