perf boost

Author: perryqh

src/runner/api.rs

@@ -4,7 +4,7 @@ use std::path::Path;
 use crate::ownership::FileOwner;
 use crate::project::Team;
 
-use super::{Error, ForFileResult, RunConfig, RunResult, Runner, config_from_path, run};
+use super::{Error, ForFileResult, RunConfig, RunResult, config_from_path, run};
 
 pub fn for_file(run_config: &RunConfig, file_path: &str, from_codeowners: bool, json: bool) -> RunResult {
     if from_codeowners {
@@ -37,10 +37,17 @@ pub fn crosscheck_owners(run_config: &RunConfig) -> RunResult {
     run(run_config, |runner| runner.crosscheck_owners())
 }
 
+// Returns all owners for a file without creating a Runner (performance optimized)
+pub fn owners_for_file(run_config: &RunConfig, file_path: &str) -> error_stack::Result<Vec<FileOwner>, Error> {
+    let config = config_from_path(&run_config.config_path)?;
+    use crate::ownership::file_owner_resolver::find_file_owners;
+    let owners = find_file_owners(&run_config.project_root, &config, std::path::Path::new(file_path)).map_err(Error::Io)?;
+    Ok(owners)
+}
+
 // Returns the highest priority owner for a file. More to come here.
 pub fn file_owner_for_file(run_config: &RunConfig, file_path: &str) -> error_stack::Result<Option<FileOwner>, Error> {
-    let runner = Runner::new(run_config)?;
-    let owners = runner.owners_for_file(file_path)?;
+    let owners = owners_for_file(run_config, file_path)?;
     Ok(owners.first().cloned())
 }