feat(samoid): implement hook execution runtime (#3)

samoid/Cargo.toml

@@ -3,6 +3,14 @@ name = "samoid"
 version = "0.1.0"
 edition = "2024"
 
+[[bin]]
+name = "samoid"
+path = "src/main.rs"
+
+[[bin]]
+name = "samoid-hook"
+path = "src/hook_runner.rs"
+
 [dependencies]
 clap = { version = "4.5", features = ["derive"] }
 toml = "0.8"

samoid/src/config.rs

@@ -69,10 +69,7 @@ impl SamoidConfig {
 
         // Add pre-push hook if the project type has a default command
         if let Some(pre_push_cmd) = project_type.default_pre_push_command() {
-            hooks.insert(
-                "pre-push".to_string(),
-                pre_push_cmd.to_string(),
-            );
+            hooks.insert("pre-push".to_string(), pre_push_cmd.to_string());
         }
 
         Self {
@@ -243,7 +240,7 @@ mod tests {
         assert!(is_valid_git_hook("post-merge"));
         assert!(is_valid_git_hook("pre-push"));
         assert!(is_valid_git_hook("pre-auto-gc"));
-        
+
         // Test invalid hook names
         assert!(!is_valid_git_hook("invalid-hook"));
         assert!(!is_valid_git_hook(""));
@@ -310,16 +307,16 @@ fail_fast = false
     fn test_validate_method_coverage() {
         // Test various validation scenarios to ensure complete coverage
         let mut config = SamoidConfig::default_for_project_type(&ProjectType::Rust);
-        
+
         // Test valid configuration
         assert!(config.validate().is_ok());
-        
+
         // Test validation with custom settings that should pass
         config.settings.debug = true;
         config.settings.skip_hooks = true;
         config.settings.fail_fast = false;
         assert!(config.validate().is_ok());
-        
+
         // Test validation ensures proper error handling paths are covered
         config.settings.hook_directory = "valid-dir".to_string();
         assert!(config.validate().is_ok());
@@ -332,57 +329,81 @@ fail_fast = false
             hooks: std::collections::HashMap::new(),
             settings: SamoidSettings::default(),
         };
-        
+
         // Add a valid hook to test hook validation logic
-        config.hooks.insert("pre-commit".to_string(), "echo test".to_string());
+        config
+            .hooks
+            .insert("pre-commit".to_string(), "echo test".to_string());
         assert!(config.validate().is_ok());
-        
+
         // Test with multiple hooks to cover iteration logic
-        config.hooks.insert("post-commit".to_string(), "echo post".to_string());
-        config.hooks.insert("prepare-commit-msg".to_string(), "echo prepare".to_string());
+        config
+            .hooks
+            .insert("post-commit".to_string(), "echo post".to_string());
+        config
+            .hooks
+            .insert("prepare-commit-msg".to_string(), "echo prepare".to_string());
         assert!(config.validate().is_ok());
-        
+
         // Test hook command trimming logic
-        config.hooks.insert("pre-push".to_string(), "  echo with spaces  ".to_string());
+        config
+            .hooks
+            .insert("pre-push".to_string(), "  echo with spaces  ".to_string());
         assert!(config.validate().is_ok());
-        
+
         // Test validation with all valid hook types to ensure complete coverage
         for hook_name in [
-            "pre-commit", "pre-merge-commit", "prepare-commit-msg", "commit-msg",
-            "post-commit", "applypatch-msg", "pre-applypatch", "post-applypatch",
-            "pre-rebase", "post-rewrite", "post-checkout", "post-merge",
-            "pre-push", "pre-auto-gc"
+            "pre-commit",
+            "pre-merge-commit",
+            "prepare-commit-msg",
+            "commit-msg",
+            "post-commit",
+            "applypatch-msg",
+            "pre-applypatch",
+            "post-applypatch",
+            "pre-rebase",
+            "post-rewrite",
+            "post-checkout",
+            "post-merge",
+            "pre-push",
+            "pre-auto-gc",
         ] {
             let mut test_config = SamoidConfig {
                 hooks: std::collections::HashMap::new(),
                 settings: SamoidSettings::default(),
             };
-            test_config.hooks.insert(hook_name.to_string(), "test command".to_string());
-            assert!(test_config.validate().is_ok(), "Failed validation for hook: {}", hook_name);
+            test_config
+                .hooks
+                .insert(hook_name.to_string(), "test command".to_string());
+            assert!(
+                test_config.validate().is_ok(),
+                "Failed validation for hook: {}",
+                hook_name
+            );
         }
     }
 
     #[test]
     fn test_settings_validation_edge_cases() {
         // Test edge cases in settings validation
         let mut config = SamoidConfig::default_for_project_type(&ProjectType::Rust);
-        
+
         // Test hook_directory validation with valid paths
         config.settings.hook_directory = ".custom".to_string();
         assert!(config.validate().is_ok());
-        
+
         config.settings.hook_directory = "hooks-dir".to_string();
         assert!(config.validate().is_ok());
-        
+
         config.settings.hook_directory = "very/deep/nested/path".to_string();
         assert!(config.validate().is_ok());
-        
+
         // Test with all boolean combinations
         config.settings.debug = true;
         config.settings.fail_fast = false;
         config.settings.skip_hooks = true;
         assert!(config.validate().is_ok());
-        
+
         config.settings.debug = false;
         config.settings.fail_fast = true;
         config.settings.skip_hooks = false;
@@ -394,10 +415,10 @@ fail_fast = false
         // Test the default functions to ensure they're covered
         let default_dir = super::default_hook_directory();
         assert_eq!(default_dir, ".samoid");
-        
+
         let default_fail_fast = super::default_fail_fast();
         assert_eq!(default_fail_fast, true);
-        
+
         // Test Settings construction with defaults
         let settings = SamoidSettings {
             hook_directory: default_hook_directory(),
@@ -414,14 +435,22 @@ fail_fast = false
         // Test that all project types can generate configs and validate successfully
         for project_type in [
             ProjectType::Rust,
-            ProjectType::Go, 
+            ProjectType::Go,
             ProjectType::Node,
             ProjectType::Python,
             ProjectType::Unknown,
         ] {
             let config = SamoidConfig::default_for_project_type(&project_type);
-            assert!(config.validate().is_ok(), "Failed validation for project type: {:?}", project_type);
-            assert!(!config.hooks.is_empty(), "No hooks generated for project type: {:?}", project_type);
+            assert!(
+                config.validate().is_ok(),
+                "Failed validation for project type: {:?}",
+                project_type
+            );
+            assert!(
+                !config.hooks.is_empty(),
+                "No hooks generated for project type: {:?}",
+                project_type
+            );
         }
     }
 }