refactored code

src/github.rs

@@ -7,7 +7,6 @@ use std::{
 };
 
 use anyhow::{Context, Result};
-use globset::{Glob, GlobSet, GlobSetBuilder};
 use indicatif::{ProgressBar, ProgressStyle};
 use octorust::{
     auth::Credentials,
@@ -119,83 +118,14 @@ fn parse_excluded_repo(raw: &str) -> Option<String> {
     parse_repo_name_from_path(trimmed)
 }
 
-struct ExcludeMatcher {
-    exact: HashSet<String>,
-    globs: Option<GlobSet>,
+use crate::git_host;
+
+fn build_exclude_matcher(exclude_repos: &[String]) -> git_host::ExcludeMatcher {
+    git_host::build_exclude_matcher(exclude_repos, |raw| parse_excluded_repo(raw), "GitHub")
 }
 
-impl ExcludeMatcher {
-    fn is_empty(&self) -> bool {
-        self.exact.is_empty() && self.globs.is_none()
-    }
-
-    fn matches(&self, name: &str) -> bool {
-        if self.exact.contains(name) {
-            return true;
-        }
-        if let Some(globs) = &self.globs {
-            return globs.is_match(name);
-        }
-        false
-    }
-}
-
-fn looks_like_glob(pattern: &str) -> bool {
-    pattern.contains('*') || pattern.contains('?') || pattern.contains('[')
-}
-
-fn build_exclude_matcher(exclude_repos: &[String]) -> ExcludeMatcher {
-    let mut exact = HashSet::new();
-    let mut glob_builder = GlobSetBuilder::new();
-    let mut has_glob = false;
-
-    for raw in exclude_repos {
-        match parse_excluded_repo(raw) {
-            Some(name) => {
-                if looks_like_glob(&name) {
-                    match Glob::new(&name) {
-                        Ok(glob) => {
-                            glob_builder.add(glob);
-                            has_glob = true;
-                        }
-                        Err(err) => {
-                            warn!("Ignoring invalid GitHub exclusion pattern '{raw}': {err}");
-                            exact.insert(name);
-                        }
-                    }
-                } else {
-                    exact.insert(name);
-                }
-            }
-            None => {
-                warn!("Ignoring invalid GitHub exclusion '{raw}' (expected owner/repo)");
-            }
-        }
-    }
-
-    let globs = if has_glob {
-        match glob_builder.build() {
-            Ok(set) => Some(set),
-            Err(err) => {
-                warn!("Failed to build GitHub exclusion patterns: {err}");
-                None
-            }
-        }
-    } else {
-        None
-    };
-
-    ExcludeMatcher { exact, globs }
-}
-
-fn should_exclude_repo(clone_url: &str, excludes: &ExcludeMatcher) -> bool {
-    if excludes.is_empty() {
-        return false;
-    }
-    if let Some(name) = parse_repo_name_from_url(clone_url) {
-        return excludes.matches(&name);
-    }
-    false
+fn should_exclude_repo(clone_url: &str, excludes: &git_host::ExcludeMatcher) -> bool {
+    git_host::should_exclude_repo(clone_url, excludes, parse_repo_name_from_url)
 }
 fn create_github_client(github_url: &url::Url, ignore_certs: bool) -> Result<Arc<Client>> {
     // Try personal access token