Add pre-push subcommand and update dependencies

- Implement `pre-push` subcommand that runs targeted checks on modified crates
  - Detects changed files since merge-base with origin/main
  - Identifies affected crates from file paths
  - Runs clippy, tests, and doc-tests only for affected crates
  - Provides clear, colorized output for check results

- Update dependencies:
  - fs-err: 3.1.0 -> 3.2.0
  - log: 0.4.27 -> 0.4.29
  - owo-colors: 4.2.1 -> 4.2.3

- Update GitHub Actions:
  - actions/checkout: v5 -> v6
  - github/codeql-action: v3 -> v4

Resolves facet-rs/facet#1247

Author: fasterthanlime

Cargo.lock

@@ -9,9 +9,9 @@ rust-version = "1.87"
 license = "MIT OR Apache-2.0"
 
 [dependencies]
-fs-err = "3.1.0"
-log = { version = "0.4.27", features = ["std"] }
-owo-colors = "4.2.1"
+fs-err = "3.2.0"
+log = { version = "0.4.29", features = ["std"] }
+owo-colors = "4.2.3"
 
 [dev-dependencies]
 cargo-husky = { version = "1.5.0", default-features = false, features = [

Cargo.toml

@@ -16,7 +16,7 @@ jobs:
     container:
       image: ghcr.io/facet-rs/facet-ci:latest-amd64
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -47,7 +47,7 @@ jobs:
     container:
       image: ghcr.io/facet-rs/facet-ci:latest-amd64
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -68,7 +68,7 @@ jobs:
     container:
       image: ghcr.io/facet-rs/facet-ci:latest-amd64
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -92,7 +92,7 @@ jobs:
     container:
       image: ghcr.io/facet-rs/facet-ci:latest-amd64
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -111,7 +111,7 @@ jobs:
     container:
       image: ghcr.io/facet-rs/facet-ci:latest-amd64
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -128,7 +128,7 @@ jobs:
     permissions:
       security-events: write # to upload sarif results
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
 
       - uses: Swatinem/rust-cache@v2
 
@@ -145,7 +145,7 @@ jobs:
         continue-on-error: true
 
       - name: Upload SARIF results
-        uses: github/codeql-action/upload-sarif@v3
+        uses: github/codeql-action/upload-sarif@v4
         with:
           sarif_file: clippy-results.sarif
           wait-for-processing: true

src/.github/workflows/test.yml

@@ -380,6 +380,205 @@ fn enqueue_cargo_husky_precommit_hook_jobs(sender: std::sync::mpsc::Sender<Job>)
     }
 }
 
+fn run_pre_push() {
+    use std::collections::{BTreeSet, HashSet};
+
+    println!("{}", "Running pre-push checks...".cyan().bold());
+
+    // Find the merge base with origin/main
+    let merge_base_output = Command::new("git")
+        .args(["merge-base", "HEAD", "origin/main"])
+        .output();
+
+    let merge_base = match merge_base_output {
+        Ok(output) if output.status.success() => {
+            String::from_utf8_lossy(&output.stdout).trim().to_string()
+        }
+        _ => {
+            warn!("Failed to find merge base with origin/main, using HEAD");
+            "HEAD".to_string()
+        }
+    };
+
+    // Get the list of changed files
+    let diff_output = Command::new("git")
+        .args(["diff", "--name-only", &format!("{}...HEAD", merge_base)])
+        .output();
+
+    let changed_files = match diff_output {
+        Ok(output) if output.status.success() => String::from_utf8_lossy(&output.stdout)
+            .lines()
+            .map(|s| s.to_string())
+            .collect::<Vec<_>>(),
+        Err(e) => {
+            error!("Failed to get changed files: {}", e);
+            std::process::exit(1);
+        }
+        Ok(output) => {
+            error!(
+                "git diff failed: {}",
+                String::from_utf8_lossy(&output.stderr)
+            );
+            std::process::exit(1);
+        }
+    };
+
+    if changed_files.is_empty() {
+        println!("{}", "No changes detected".green().bold());
+        std::process::exit(0);
+    }
+
+    // Find which crates are affected
+    let mut affected_crates = HashSet::new();
+
+    for file in &changed_files {
+        let path = Path::new(file);
+
+        // Find the crate directory by looking for Cargo.toml
+        let mut current = path;
+        while let Some(parent) = current.parent() {
+            let cargo_toml = if parent.as_os_str().is_empty() {
+                PathBuf::from("Cargo.toml")
+            } else {
+                parent.join("Cargo.toml")
+            };
+
+            if cargo_toml.exists() {
+                // Read Cargo.toml to get the package name
+                if let Ok(content) = fs::read_to_string(&cargo_toml) {
+                    // Simple parsing: look for [package] section and name field
+                    let mut in_package = false;
+                    for line in content.lines() {
+                        let trimmed = line.trim();
+                        if trimmed == "[package]" {
+                            in_package = true;
+                        } else if trimmed.starts_with('[') {
+                            in_package = false;
+                        } else if in_package && trimmed.starts_with("name") {
+                            if let Some(name_part) = trimmed.split('=').nth(1) {
+                                let name = name_part.trim().trim_matches('"').trim_matches('\'');
+                                affected_crates.insert(name.to_string());
+                                break;
+                            }
+                        }
+                    }
+                }
+                break;
+            }
+
+            if parent.as_os_str().is_empty() {
+                break;
+            }
+            current = parent;
+        }
+    }
+
+    if affected_crates.is_empty() {
+        println!("{}", "No crates affected by changes".yellow());
+        std::process::exit(0);
+    }
+
+    // Sort for consistent output
+    let affected_crates: BTreeSet<_> = affected_crates.into_iter().collect();
+
+    println!(
+        "{} Affected crates: {}",
+        "🔍".cyan(),
+        affected_crates
+            .iter()
+            .map(|s| s.as_str())
+            .collect::<Vec<_>>()
+            .join(", ")
+            .yellow()
+    );
+
+    let mut all_passed = true;
+
+    for crate_name in &affected_crates {
+        println!(
+            "\n{} Checking crate: {}",
+            "📦".cyan(),
+            crate_name.yellow().bold()
+        );
+
+        // Run clippy
+        print!("  {} Running clippy... ", "🔍".cyan());
+        io::stdout().flush().unwrap();
+        let clippy_status = Command::new("cargo")
+            .args(["clippy", "-p", crate_name, "--", "-D", "warnings"])
+            .status();
+
+        match clippy_status {
+            Ok(status) if status.success() => {
+                println!("{}", "passed".green());
+            }
+            _ => {
+                println!("{}", "failed".red());
+                all_passed = false;
+            }
+        }
+
+        // Run tests
+        print!("  {} Running tests... ", "🧪".cyan());
+        io::stdout().flush().unwrap();
+        let test_status = Command::new("cargo")
+            .args(["test", "-p", crate_name])
+            .stdout(Stdio::null())
+            .stderr(Stdio::null())
+            .status();
+
+        match test_status {
+            Ok(status) if status.success() => {
+                println!("{}", "passed".green());
+            }
+            _ => {
+                println!("{}", "failed".red());
+                all_passed = false;
+            }
+        }
+
+        // Run doc tests
+        print!("  {} Running doc tests... ", "📚".cyan());
+        io::stdout().flush().unwrap();
+        let doctest_status = Command::new("cargo")
+            .args(["test", "--doc", "-p", crate_name])
+            .stdout(Stdio::null())
+            .stderr(Stdio::null())
+            .status();
+
+        match doctest_status {
+            Ok(status) if status.success() => {
+                println!("{}", "passed".green());
+            }
+            Ok(status) if status.code() == Some(101) => {
+                // Exit code 101 often means "no tests to run"
+                println!("{}", "skipped (no lib)".yellow());
+            }
+            _ => {
+                println!("{}", "failed".red());
+                all_passed = false;
+            }
+        }
+    }
+
+    println!();
+    if all_passed {
+        println!(
+            "{} {}",
+            "✅".green(),
+            "All pre-push checks passed!".green().bold()
+        );
+        std::process::exit(0);
+    } else {
+        println!(
+            "{} {}",
+            "❌".red(),
+            "Some pre-push checks failed".red().bold()
+        );
+        std::process::exit(1);
+    }
+}
+
 fn show_and_apply_jobs(jobs: &mut [Job]) {
     use std::io::{self, Write};
 
@@ -456,6 +655,13 @@ fn main() {
         }
     }
 
+    // Parse CLI arguments
+    let args: Vec<String> = std::env::args().collect();
+    if args.len() > 1 && args[1] == "pre-push" {
+        run_pre_push();
+        return;
+    }
+
     let staged_files = match collect_staged_files() {
         Ok(sf) => sf,
         Err(e) => {