[worker:doc-drift] correct bare min no-subcommand behavior in CLI reference - #1190
Conversation
📝 WalkthroughWalkthroughThe CLI reference updates the documented behavior of bare ChangesBare
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/reference/cli-min.md`:
- Around line 18-19: Update the read-only state report description in the CLI
documentation to say it “exits successfully” instead of merely “exits,”
documenting the zero exit status while leaving failure behavior unchanged.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 5e751398-b2c5-4a22-bc28-45b85744f70f
📒 Files selected for processing (1)
docs/reference/cli-min.md
|
Commit pushed:
|
Resolves documentation drift in
docs/reference/cli-min.md.Problem
The CLI reference (line 14) stated that running bare
minwith no subcommand "prints this help and exits." That does not match the implementation (crates/minimal/src/lib.rs,cmd_bare, lines 860-872):minroutes into a session using the smart-resolution rules ofmin session attachwith no argument (cwd match → attach, only session → attach, ambiguity → picker), and creates one from the current directory and attaches when no sessions exist.min --helpis what prints the help text.Fix
Updated the prose in
cli-min.mdto describe the actual TTY / non-TTY behavior of baremin, and to point atmin --helpfor the help output. Docs-only change; no source orCargo.tomltouched.Verification
This is a Markdown-documentation-only change (no
.rsorCargo.tomledits), so no compilation is affected. The host backstop still gates the PR withcargo fmt --all --check,cargo build --workspace --all-targets --locked,cargo clippy --workspace --all-targets --locked -- -D warnings, andcargo test --workspace --all-targets --lockedbefore it opens.Closes #1187
Note
Correct bare
minno-subcommand behavior in CLI reference docsUpdates cli-min.md to accurately describe what happens when
minis run with no subcommand. Previously documented as printing help and exiting; the correct behavior is:min session attach(cwd match → attach, only session → attach, ambiguity → picker), creating and attaching a new session from cwd if none existmin --helpis now the correct way to print helpMacroscope summarized c4163ef.
Summary by CodeRabbit
minin an interactive terminal now opens or attaches to a session automatically.minprints a read-only state report instead of displaying help and exiting.