fix: remove nonexistent ssh-forward from cli-min.md completions list - #1059
Conversation
The completions section listed `ssh-forward` as accepting UUID or session-name completions, but the SshForward subcommand is gated behind the remote-access feature (off by default), so `min ssh-forward` is unrecognized in a standard build. Remove it from the list; the four remaining commands all exist and take a session argument. Refs: #1050
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 52 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Comment |
Fixes #1050
Routing-Key: inbox-route/I_kwDOSUhdos8AAAABKrYUHg
The completions section of
docs/reference/cli-min.mdlistedssh-forwardamong the commands that complete a "UUID or session name" argument, but itsSshForwardsubcommand is gated behind theremote-accessfeature (off by default incrates/minimal/Cargo.toml), somin ssh-forwardis unrecognized in a standard build — exactly the reporter's symptom. This reference documents only default-build commands (the equally feature-gatedmeshis absent entirely andremote-accessis never mentioned), so the consistent minimal fix is to dropssh-forwardfrom the list; the four remaining commands (session attach/destroy/rename/policy) all exist in a default build and carry the session-name completer. The cosmetic missing### session policysection is left out of scope (informed by #438, the prior cli-reference drift fix).Verification
cargo fmt --all --check --manifest-path target/Cargo.toml — clean, exit 0
cargo clippy --workspace --locked --manifest-path target/Cargo.toml -- -D warnings — Finished, no warnings, exit 0
cargo build --workspace --locked --manifest-path target/Cargo.toml — Finished in 4m 33s, exit 0
cargo test --workspace --locked --manifest-path target/Cargo.toml — all suites ok, 0 failed, exit 0
Note
Remove nonexistent
ssh-forwardcommand from completions list incli-min.mdRemoves
ssh-forwardfrom the list of commands that support 'UUID or session name' completion in cli-min.md, as that command does not exist. Also updates two phrases in the--no-promptflag description to use contractions.Macroscope summarized 73c8408.