feat: add -O/-F short flags and hidden --force alias - #52
Conversation
-O aliases --output-dir, -F aliases --force-export. --force is a hidden alias for --force-export, kept out of --help since --force-export names the exact behavior (bypass smart export filtering).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📜 Recent review details⏰ Context from checks skipped due to timeout. (1)
🧰 Additional context used📓 Path-based instructions (3)Maintain CLI source as `src/main.rs` and library core as `src/lib.rs`📄 CodeRabbit inference engine (AGENTS.md) Files:
CLI (`src/main.rs`) uses library export functions (`export_to_csv`, `export_to_gpx`, `export_to_event`) with CLI-specific status messages as thin wrapper with zero public functions📄 CodeRabbit inference engine (AGENTS.md) Files:
Do not remove or modify comments unless the related code is changed; only add comments that explain code functionality, no AI instructional comments Never embed or call external binaries from Rust code Ensure `cargo build --release` has no...📄 CodeRabbit inference engine (AGENTS.md) Files:
🔇 Additional comments (1)
📝 WalkthroughWalkthroughThe CLI adds ChangesCLI option aliases
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a localized CLI compatibility change adding short flags and a hidden alias, with no actionable merge-blocking risk remaining after normal checks and review. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai — AI generated comment Analysis requested on this small CLI change:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
echo '== src/main.rs outline =='
ast-grep outline src/main.rs --items all --type function --view expanded || true
echo '== clap dependency declarations =='
rg -n -C 2 '(^clap\s*=|name = "clap"|clap\s*=)' Cargo.toml Cargo.lock 2>/dev/null || true
echo '== argument and short/alias declarations in src/main.rs =='
rg -n -C 3 '\.(short|long|alias|visible_alias|version|help)\s*\(' src/main.rsLength of output: 4431 🏁 Script executed: #!/bin/bash
set -euo pipefail
echo '== build_command =='
sed -n '266,355p' src/main.rs
echo '== CLI references in source, tests, and docs =='
rg -n -i -C 2 --glob '!Cargo.lock' \
'(^|[^[:alnum:]_-])(output-dir|force-export|--force|-O\b|-F\b)([^[:alnum:]_-]|$)|get_one\s*::<.*>\s*\(\s*"(output-dir|force-export|force)"' \
.Length of output: 5451
No change is required. You are interacting with an AI system. |
|
AI Generated comment Multi-tool review summary for this change (-O/-F short flags, hidden --force alias):
No actionable findings from any tool. |
1 participant
AI Generated pull-request
Summary
-Oshort flag for--output-dir-Fshort flag for--force-export--forcehidden long alias for--force-export(works, not listed in--help)Test plan
cargo test --verbosepasses (53+11+8+3 tests)cargo test --features=cli --verbosepassescargo build --releasepasses with no errors/warningscargo fmt --all -- --checkpassescargo clippy --all-targets --all-features -- -D warnings— pre-existing failure unrelated to this change (src/parser/frame.rs:719,needless_range_loop); reproduces identically on unmodifiedmasterSummary by CodeRabbit
-Oas a shortcut for specifying the output directory.-Fas a shortcut for enabling forced exports.forceas an alias for the force-export option.