avoiding panics

Author: perryqh

dev/run_benchmarks_for_file.sh

@@ -11,5 +11,4 @@ echo "To run these benchmarks on your application, you can place this repo next
 
 hyperfine --warmup=2 --runs=3 --export-markdown tmp/codeowners_for_file_benchmarks.md \
   "../rubyatscale/codeowners-rs/target/release/codeowners for-file \"$1\"" \
-  "bin/codeowners for_file \"$1\"" \
   "bin/codeownership for_file \"$1\"" 

dev/run_benchmarks_for_gv.sh

@@ -9,5 +9,4 @@ echo "To run these benchmarks on your application, you can place this repo next
 
 hyperfine --warmup=2 --runs=3 --export-markdown tmp/codeowners_benchmarks_gv.md \
   '../rubyatscale/codeowners-rs/target/release/codeowners gv' \
-  'bin/codeowners validate' \
   'bin/codeownership validate' 

src/ownership/parser.rs

@@ -59,17 +59,23 @@ impl Parser {
 fn teams_by_github_team_name(team_file_glob: Vec<String>) -> HashMap<String, Team> {
     let mut teams = HashMap::new();
     for glob in team_file_glob {
-        let paths = glob::glob(&glob).expect("Failed to read glob pattern").filter_map(Result::ok);
-
-        for path in paths {
-            let team = match Team::from_team_file_path(path) {
-                Ok(team) => team,
-                Err(e) => {
-                    eprintln!("Error parsing team file: {}", e);
-                    continue;
+        match glob::glob(&glob) {
+            Ok(paths) => {
+                for path in paths.filter_map(Result::ok) {
+                    let team = match Team::from_team_file_path(path) {
+                        Ok(team) => team,
+                        Err(e) => {
+                            eprintln!("Error parsing team file: {}", e);
+                            continue;
+                        }
+                    };
+                    teams.insert(team.github_team.clone(), team);
                 }
-            };
-            teams.insert(team.github_team.clone(), team);
+            }
+            Err(e) => {
+                eprintln!("Failed to read glob pattern '{}': {}", glob, e);
+                continue;
+            }
         }
     }
 

src/project_builder.rs

@@ -71,8 +71,25 @@ impl<'a> ProjectBuilder<'a> {
             })
         });
 
-        // Process sequentially with &mut self
-        let collected_entries = Arc::try_unwrap(collected).unwrap().into_inner().unwrap();
+        // Process sequentially with &mut self without panicking on Arc/Mutex unwraps
+        let collected_entries = match Arc::try_unwrap(collected) {
+            // We are the sole owner of the Arc
+            Ok(mutex) => match mutex.into_inner() {
+                // Mutex not poisoned
+                Ok(entries) => entries,
+                // Recover entries even if the mutex was poisoned
+                Err(poisoned) => poisoned.into_inner(),
+            },
+            // There are still other Arc references; lock and take the contents
+            Err(arc) => match arc.lock() {
+                Ok(mut guard) => std::mem::take(&mut *guard),
+                // Recover guard even if poisoned, then take contents
+                Err(poisoned) => {
+                    let mut guard = poisoned.into_inner();
+                    std::mem::take(&mut *guard)
+                }
+            },
+        };
         for entry in collected_entries {
             entry_types.push(self.build_entry_type(entry)?);
         }
@@ -89,11 +106,10 @@ impl<'a> ProjectBuilder<'a> {
         if is_dir {
             return Ok(EntryType::Directory(absolute_path.to_owned(), relative_path.to_owned()));
         }
-        let file_name = relative_path
-            .file_name()
-            .expect("expected a file_name")
-            .to_string_lossy()
-            .to_lowercase();
+        let file_name = match relative_path.file_name() {
+            Some(name) => name.to_string_lossy().to_lowercase(),
+            None => return Ok(EntryType::NullEntry()),
+        };
 
         match file_name.as_str() {
             name if name == "package.yml"
@@ -142,11 +158,14 @@ impl<'a> ProjectBuilder<'a> {
                         }
                         EntryType::Directory(absolute_path, relative_path) => {
                             if relative_path.parent() == Some(Path::new(&self.config.vendored_gems_path)) {
-                                let file_name = relative_path.file_name().expect("expected a file_name");
-                                gems.push(VendoredGem {
-                                    path: absolute_path,
-                                    name: file_name.to_string_lossy().to_string(),
-                                });
+                                if let Some(file_name) = relative_path.file_name() {
+                                    gems.push(VendoredGem {
+                                        path: absolute_path,
+                                        name: file_name.to_string_lossy().to_string(),
+                                    });
+                                } else {
+                                    warn!("Vendored gem path without file name: {:?}", relative_path);
+                                }
                             }
                         }
                         EntryType::RubyPackage(absolute_path, relative_path) => {