feat: lint commit message from hash range

Cargo.toml

@@ -19,6 +19,7 @@ anyhow = "1.0"
 [dev-dependencies]
 assert_cmd = "2"
 predicates = "3"
+serial_test = "3"
 tempfile = "3"
 
 # The profile that 'dist' will build with

README.md

@@ -7,9 +7,37 @@ A Conventional Commitlint binary tool (x for executable).
 ## Development
 
 Run cocox
-
 ```shell
 cargo run -- "my commit message"
 
 cargo run -- --file "/foo/bar"
+
+
+# using commit hash
+cargo run -- --hash xxxx-xxxx
+
+# using  range of hashes: 
+cargo run -- --from-hash xxxx-xxxx --to-hash xxxx-xxxx
+
+# --to-hash points to HEAD by default
+cargo run -- --from-hash xxxx-xxxx 
+
 ```
+
+## Usage: 
+```shell
+A Conventional Commitlint binary tool
+
+Usage: cocox [OPTIONS] <MESSAGE|--file <FILE>|--hash <HASH>|--from-hash <FROM_HASH>>
+
+Arguments:
+  [MESSAGE]  
+
+Options:
+      --file <FILE>            
+      --hash <HASH>            
+      --from-hash <FROM_HASH>  
+      --to-hash <TO_HASH>      [default: HEAD]
+  -h, --help                   Print help
+  -V, --version                Print version
+```

src/cli.rs

@@ -6,7 +6,7 @@ use clap::{ArgGroup, Parser};
 #[command(version)]
 #[command(group(
     ArgGroup::new("input")
-        .args(["message", "file", "hash"])
+        .args(["message", "file", "hash", "from_hash"])
         .required(true)
         .multiple(false)))]
 pub struct Cli {
@@ -18,4 +18,14 @@ pub struct Cli {
 
     #[arg(long)]
     pub hash: Option<String>,
+
+    #[arg(long)]
+    pub from_hash: Option<String>,
+
+    #[arg(
+        long, requires = "from_hash",
+        conflicts_with_all = ["message", "file", "hash"],
+        default_value = "HEAD"
+    )]
+    pub to_hash: String,
 }

src/command.rs

@@ -1,8 +1,7 @@
 use crate::cli::Cli;
-use crate::git_helpers::get_commit_message_from_hash;
-use crate::linter::lint_commit_message;
+use crate::git_helpers::{get_commit_message_from_hash, get_commit_messages_from_hash_range};
+use crate::linter::{LintOutcome, lint_commit_message};
 use crate::messages::{VALIDATION_FAILED, VALIDATION_SUCCESSFUL};
-use crate::utils::{is_empty, is_ignored};
 use anyhow::{Context, Result};
 
 fn read_file(file: &String) -> Result<String> {
@@ -11,26 +10,43 @@ fn read_file(file: &String) -> Result<String> {
 }
 
 fn handle_commit_message(msg: &str) {
-    if is_empty(msg) {
-        eprintln!(
-            "{}: Aborting commit due to empty commit message",
-            VALIDATION_FAILED
-        );
-        std::process::exit(1);
+    match lint_commit_message(msg) {
+        LintOutcome::Empty => {
+            std::process::exit(1);
+        }
+        LintOutcome::Ignored => (),
+        LintOutcome::Valid => {
+            println!("{}\n", VALIDATION_SUCCESSFUL);
+        }
+        LintOutcome::Invalid => {
+            eprintln!("{}\n", VALIDATION_FAILED);
+            std::process::exit(1);
+        }
     }
+}
 
-    if is_ignored(msg) {
-        return;
+fn handle_multiple_commit_messages(messages: &[String]) {
+    let mut has_failure = false;
+
+    for msg in messages {
+        match lint_commit_message(msg) {
+            LintOutcome::Empty => {
+                std::process::exit(1);
+            }
+            LintOutcome::Ignored => (),
+            LintOutcome::Valid => (),
+            LintOutcome::Invalid => {
+                has_failure = true;
+            }
+        }
     }
 
-    let success = lint_commit_message(msg);
-    if success {
-        println!("{}", VALIDATION_SUCCESSFUL);
-        return;
+    if has_failure {
+        eprintln!("{}", VALIDATION_FAILED);
+        std::process::exit(1);
     }
 
-    eprintln!("{}", VALIDATION_FAILED);
-    std::process::exit(1);
+    println!("{}", VALIDATION_SUCCESSFUL);
 }
 
 pub fn run(args: Cli) -> Result<()> {
@@ -42,10 +58,16 @@ pub fn run(args: Cli) -> Result<()> {
         let msg = read_file(file)?;
         handle_commit_message(&msg);
     } else if let Some(hash) = &args.hash {
+        // commit hash
         let msg = get_commit_message_from_hash(hash)?;
         handle_commit_message(&msg);
+    } else if let Some(from_hash) = &args.from_hash {
+        // commit hash range
+        let to_hash = &args.to_hash;
+        let messages = get_commit_messages_from_hash_range(from_hash, to_hash)?;
+        handle_multiple_commit_messages(&messages);
     } else {
-        unreachable!("invalid option is handle by clap");
+        unreachable!("invalid option is handled by clap");
     }
 
     Ok(())