Repo stats log options

AGENTS.md

@@ -3,6 +3,7 @@
 Whenever an automated agent lands code in this repository, it must also:
 
 - Bump the crate version in `Cargo.toml` before shipping.
+- Run `cargo build` so `Cargo.lock` is updated, and commit any lockfile changes.
 - Choose the bump size using semantic versioning rules:
   - Increment the **major** version (`x.0.0`) for breaking changes.
   - Increment the **minor** version when adding backward-compatible functionality.

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "loki-cli"
-version = "1.9.0"
+version = "2.0.0"
 authors = ["Kyle W. Rader"]
 description = "Loki: 🚀 A Git productivity tool"
 homepage = "https://github.com/kyle-rader/loki-cli"

README.md

@@ -95,7 +95,7 @@ Analyze commits reachable from HEAD to see who has been landing work in a reposi
 
 - `--name` filters by author display name (repeatable, case-insensitive).
 - `--email` filters by author email (repeatable, case-insensitive).
-- `--first-parent` limits the analysis to first-parent commits.
+- `--all` includes all commits (default is first-parent only).
 
 #### Example
 ```

src/main.rs

@@ -73,9 +73,9 @@ struct RepoStatsOptions {
     #[clap(long, default_value_t = 20)]
     top: usize,
 
-    /// Only include first-parent commits.
+    /// Include all commits (disables first-parent-only filtering).
     #[clap(long, default_value = "false")]
-    first_parent: bool,
+    all: bool,
 
     /// Only include commits authored by these names (repeatable, case-insensitive fuzzy match).
     #[clap(long = "name", value_name = "NAME")]
@@ -236,7 +236,7 @@ fn repo_stats(options: &RepoStatsOptions) -> Result<(), String> {
         options.emails.iter().map(|s| s.to_lowercase()).collect();
 
     let mut git_args: Vec<String> = vec!["log".to_string()];
-    if options.first_parent {
+    if !options.all {
         git_args.push("--first-parent".to_string());
     }
     git_args.push("--pretty=format:%ct%x09%an%x09%ae".to_string());
@@ -343,14 +343,14 @@ fn repo_stats(options: &RepoStatsOptions) -> Result<(), String> {
     progress.finish();
 
     if totals.is_empty() {
-        if options.first_parent {
+        if options.all {
             println!(
-                "No first-parent commits found between {} and {}.",
+                "No commits found between {} and {}.",
                 range.start_label, range.end_label
             );
         } else {
             println!(
-                "No commits found between {} and {}.",
+                "No first-parent commits found between {} and {}.",
                 range.start_label, range.end_label
             );
         }