Rename executable_path to executable_name throughout code and tests

Author: perryqh

src/cli.rs

@@ -114,7 +114,7 @@ pub fn cli() -> Result<RunResult, RunnerError> {
         codeowners_file_path,
         project_root,
         no_cache: args.no_cache,
-        executable_path: None,
+        executable_name: None,
     };
 
     let runner_result = match args.command {

src/runner.rs

@@ -47,7 +47,7 @@ where
 pub(crate) fn config_from_run_config(run_config: &RunConfig) -> Result<Config, Error> {
     match crate::config::Config::load_from_path(&run_config.config_path) {
         Ok(mut c) => {
-            if let Some(executable_name) = &run_config.executable_path {
+            if let Some(executable_name) = &run_config.executable_name {
                 c.executable_name = executable_name.clone();
             }
             Ok(c)

src/runner/types.rs

@@ -17,7 +17,7 @@ pub struct RunConfig {
     pub codeowners_file_path: PathBuf,
     pub config_path: PathBuf,
     pub no_cache: bool,
-    pub executable_path: Option<String>,
+    pub executable_name: Option<String>,
 }
 
 #[derive(Debug, Serialize)]

tests/common/mod.rs

@@ -157,7 +157,7 @@ pub fn build_run_config(project_root: &Path, codeowners_rel_path: &str) -> RunCo
         codeowners_file_path,
         config_path,
         no_cache: true,
-        executable_path: None,
+        executable_name: None,
     }
 }
 

tests/run_config_executable_override_test.rs

@@ -18,7 +18,7 @@ fn test_run_config_executable_path_overrides_config_file() -> Result<(), Box<dyn
 
     let mut run_config = build_run_config(project_path, ".github/CODEOWNERS");
     // Use a relative path that gets displayed as-is in error messages
-    run_config.executable_path = Some("my-wrapper-tool".to_string());
+    run_config.executable_name = Some("my-wrapper-tool".to_string());
 
     let result = validate(&run_config, vec![]);
 
@@ -50,7 +50,7 @@ fn test_run_config_without_executable_path_uses_config_file() -> Result<(), Box<
     // This fixture has executable_name: "bin/codeownership" in config
 
     let mut run_config = build_run_config(project_path, ".github/CODEOWNERS");
-    run_config.executable_path = None; // Explicitly no override
+    run_config.executable_name = None; // Explicitly no override
 
     let result = validate(&run_config, vec![]);
 
@@ -78,7 +78,7 @@ fn test_run_config_executable_path_overrides_default() -> Result<(), Box<dyn Err
     // This fixture has NO executable_name in config (uses default "codeowners")
 
     let mut run_config = build_run_config(project_path, ".github/CODEOWNERS");
-    run_config.executable_path = Some("custom-command".to_string());
+    run_config.executable_name = Some("custom-command".to_string());
 
     let result = validate(&run_config, vec![]);
 

tests/runner_api.rs

@@ -38,7 +38,7 @@ team_file_glob:
         codeowners_file_path: temp_dir.path().join(".github/CODEOWNERS").to_path_buf(),
         config_path: temp_dir.path().join("config/code_ownership.yml").to_path_buf(),
         no_cache: true,
-        executable_path: None,
+        executable_name: None,
     };
 
     let file_owner = runner::file_owner_for_file(&run_config, "app/consumers/deep/nesting/nestdir/deep_file.rb")
@@ -64,7 +64,7 @@ fn test_teams_for_files_from_codeowners() {
         codeowners_file_path: project_root.join(".github/CODEOWNERS").to_path_buf(),
         config_path: project_root.join("config/code_ownership.yml").to_path_buf(),
         no_cache: true,
-        executable_path: None,
+        executable_name: None,
     };
     let teams =
         runner::teams_for_files_from_codeowners(&run_config, &file_paths.iter().map(|s| s.to_string()).collect::<Vec<String>>()).unwrap();
@@ -131,7 +131,7 @@ javascript_package_paths:
         codeowners_file_path: td.path().join(".github/CODEOWNERS"),
         config_path: td.path().join("config/code_ownership.yml"),
         no_cache: true,
-        executable_path: None,
+        executable_name: None,
     };
 
     // Ensure CODEOWNERS file matches generator output to avoid out-of-date errors
@@ -173,7 +173,7 @@ javascript_package_paths:
         codeowners_file_path: td.path().join(".github/CODEOWNERS"),
         config_path: td.path().join("config/code_ownership.yml"),
         no_cache: true,
-        executable_path: None,
+        executable_name: None,
     };
 
     let gv = runner::generate_and_validate(&rc, vec![], true);