submod 0.3.0

// SPDX-FileCopyrightText: 2025 Adam Poulemanos <89049923+bashandbone@users.noreply.github.com>
//
// SPDX-License-Identifier: LicenseRef-PlainMIT OR MIT
// TODO: This module is very not-DRY...but it's low priority right now.
use anyhow::{Context, Result};
use gix::bstr::ByteSlice;
use std::collections::HashMap;
use std::path::Path;

use crate::git_ops::simple_gix::fetch_repo;

/// Parse a gix config file from raw bytes
fn gix_file_from_bytes(bytes: Vec<u8>) -> Result<gix::config::File<'static>> {
    let mut owned_bytes: Vec<u8> = bytes;
    gix::config::File::from_bytes_owned(
        &mut owned_bytes,
        gix::config::file::Metadata::from(gix::config::Source::Local),
        Default::default(),
    )
    .map_err(|e| anyhow::anyhow!("Failed to parse gix config file: {e}"))
}

use super::{DetailedSubmoduleStatus, GitConfig, GitOperations, SubmoduleStatusFlags};
use crate::config::{SubmoduleAddOptions, SubmoduleEntries, SubmoduleUpdateOptions};
use crate::options::{ConfigLevel, GitmodulesConvert};
use crate::utilities;

/// Primary implementation using gix (gitoxide)
#[derive(Debug, Clone, PartialEq)]
pub struct GixOperations {
    repo: gix::Repository,
}
impl GixOperations {
    /// Create a new `GixOperations` instance
    pub fn new(repo_path: Option<&Path>) -> Result<Self> {
        let repo = match repo_path {
            Some(path) => gix::open(path)
                .with_context(|| format!("Failed to open repository at {}", path.display()))?,
            None => gix::discover(".")
                .with_context(|| "Failed to discover repository in current directory")?,
        };
        Ok(Self { repo })
    }

    /// Try to perform operation with gix, return error if not supported
    fn try_gix_operation<T, F>(&self, operation: F) -> Result<T>
    where
        F: FnOnce(&gix::Repository) -> Result<T>,
    {
        operation(&self.repo)
    }

    /// Try to perform ops with gix using a mutable reference
    fn try_gix_operation_mut<T, F>(&mut self, operation: F) -> Result<T>
    where
        F: FnOnce(&mut gix::Repository) -> Result<T>,
    {
        operation(&mut self.repo)
    }

    /// Convert gix submodule file to `SubmoduleEntries`
    fn convert_gitmodules_to_entries(
        &self,
        gitmodules: gix_submodule::File,
    ) -> Result<SubmoduleEntries> {
        let as_config_file = gitmodules.into_config();
        let mut sections_map = std::collections::HashMap::new();
        for section in as_config_file.sections() {
            // we need to convert everything to String and add to map
            let mut section_entries = std::collections::HashMap::new();
            let name = if let Some(subsection) = section.header().subsection_name() {
                subsection.to_string()
            } else {
                section.header().name().to_string()
            };
            let body_entries = section
                .body()
                .clone()
                .into_iter()
                .collect::<HashMap<_, _>>();
            for (key, value) in body_entries {
                section_entries.insert(key.to_string().clone(), value.to_string().clone());
            }
            sections_map.insert(name, section_entries);
        }
        let submodule_entries = crate::config::SubmoduleEntries::from_gitmodules(sections_map);

        Ok(submodule_entries)
    }
    /// Get the name of the current branch in the superproject
    fn get_superproject_branch(&self) -> Result<String> {
        self.repo
            .head_ref()
            .map_err(|e| anyhow::anyhow!("Failed to get HEAD reference: {e}"))?
            .map(|r| r.name().shorten().to_string())
            .ok_or_else(|| anyhow::anyhow!("HEAD is detached, not on a branch"))
    }

    /// Convert gix submodule status to our status flags
    #[allow(dead_code)]
    fn convert_gix_status_to_flags(&self, status: &gix::submodule::Status) -> SubmoduleStatusFlags {
        let mut flags = SubmoduleStatusFlags::empty();
        // Map gix status to our flags
        // Note: This is a simplified mapping as gix status structure may differ
        if status.is_dirty() == Some(true) {
            flags |= SubmoduleStatusFlags::WD_WD_MODIFIED;
        }
        // Add more mappings as needed based on gix::submodule::Status structure
        flags
    }
}

impl GitOperations for GixOperations {
    /// Read the .gitmodules file and convert it to `SubmoduleEntries`
    fn read_gitmodules(&self) -> Result<SubmoduleEntries> {
        let mutable_self = self.clone();
        mutable_self.try_gix_operation(|repo| {
            let gitmodules_path = repo
                .workdir()
                .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?
                .join(".gitmodules");

            if !gitmodules_path.exists() {
                return Ok(SubmoduleEntries::default());
            }

            let content = std::fs::read(&gitmodules_path)?;
            let config = repo.config_snapshot();
            let submodule_file =
                gix_submodule::File::from_bytes(&content, Some(gitmodules_path), &config)?;

            mutable_self.convert_gitmodules_to_entries(submodule_file)
        })
    }

    /// Write the submodule entries to the .gitmodules file
    fn write_gitmodules(&mut self, config: &SubmoduleEntries) -> Result<()> {
        self.try_gix_operation(|repo| {
            let mut git_config = gix::config::File::new(gix::config::file::Metadata::api());

            // Convert SubmoduleEntries to gix config format
            for (name, entry) in config.submodule_iter() {
                let subsection_name = name.as_bytes().as_bstr();

                if let Some(path) = &entry.path {
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "path",
                        path.as_bytes().as_bstr(),
                    )?;
                }
                if let Some(url) = &entry.url {
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "url",
                        url.as_bytes().as_bstr(),
                    )?;
                }
                if let Some(branch) = &entry.branch {
                    let value = branch.to_string();
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "branch",
                        value.as_bytes().as_bstr(),
                    )?;
                }
                if let Some(update) = &entry.update {
                    let value = update.to_gitmodules();
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "update",
                        value.as_bytes().as_bstr(),
                    )?;
                }
                if let Some(ignore) = &entry.ignore {
                    let value = ignore.to_gitmodules();
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "ignore",
                        value.as_bytes().as_bstr(),
                    )?;
                }
                if let Some(fetch_recurse) = &entry.fetch_recurse {
                    let value = fetch_recurse.to_gitmodules();
                    git_config.set_raw_value_by(
                        "submodule",
                        Some(subsection_name),
                        "fetchRecurseSubmodules",
                        value.as_bytes().as_bstr(),
                    )?;
                }
            }

            // Write to .gitmodules file
            let gitmodules_path = repo
                .workdir()
                .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?
                .join(".gitmodules");

            let mut file = std::fs::File::create(&gitmodules_path)?;
            git_config.write_to(&mut file)?;
            Ok(())
        })
    }

    /// Read the Git configuration at the specified level
    fn read_git_config(&self, level: ConfigLevel) -> Result<GitConfig> {
        self.clone().try_gix_operation_mut(|repo| {
            let config_snapshot = repo.config_snapshot();
            let mut entries = HashMap::new();

            // Filter by configuration level
            let source_filter = match level {
                ConfigLevel::System => gix::config::Source::System,
                ConfigLevel::Global => gix::config::Source::User,
                ConfigLevel::Local => gix::config::Source::Local,
                ConfigLevel::Worktree => gix::config::Source::Worktree,
            };

            // Extract entries from the specified level
            for section in config_snapshot.sections() {
                if section.meta().source == source_filter {
                    let section_name = section.header().name();
                    let body_iter = section.body().clone().into_iter();
                    for (key, value) in body_iter {
                        entries.insert(format!("{section_name}.{key}"), value.to_string());
                    }
                }
            }

            Ok(GitConfig { entries })
        })
    }

    /// Write the Git configuration to the repository
    fn write_git_config(&self, config: &GitConfig, level: ConfigLevel) -> Result<()> {
        // gix::config::File<'static> requires 'static lifetimes for all string arguments
        // passed to set_raw_value_by (Key: TryFrom<&'static str>, subsection: &'static BStr).
        // We use Box::leak to produce 'static references. Memory leaked is minimal
        // (a few bytes per key) and acceptable in this WIP implementation.
        // TODO: Replace with a gix API that accepts owned data when available.
        let parsed: Vec<(
            &'static str,
            Option<&'static gix::bstr::BStr>,
            &'static str,
            Vec<u8>,
        )> = config
            .entries
            .iter()
            .map(|(key, value)| {
                let mut parts = key.splitn(3, '.');
                let section: &'static str =
                    Box::leak(parts.next().unwrap_or("").to_owned().into_boxed_str());
                let subsection: Option<&'static gix::bstr::BStr> = parts.next().map(|s| {
                    let bytes: &'static [u8] = Box::leak(s.as_bytes().to_vec().into_boxed_slice());
                    bytes.as_bstr()
                });
                let name: &'static str =
                    Box::leak(parts.next().unwrap_or("").to_owned().into_boxed_str());
                (section, subsection, name, value.as_bytes().to_vec())
            })
            .collect();

        self.try_gix_operation(|repo| {
            let config_path = match level {
                ConfigLevel::Local | ConfigLevel::Worktree => repo.git_dir().join("config"),
                _ => {
                    return Err(anyhow::anyhow!(
                        "Only local config writing is supported with gix"
                    ));
                }
            };
            let bytes = if config_path.exists() {
                std::fs::read(&config_path)?
            } else {
                Vec::new()
            };
            let mut config_file = gix_file_from_bytes(bytes).with_context(|| {
                format!("Failed to read config file at {}", config_path.display())
            })?;
            for (section, subsection, name, value) in &parsed {
                config_file.set_raw_value_by(*section, *subsection, *name, value.as_bstr())?;
            }
            let mut output = std::fs::File::create(&config_path)?;
            config_file.write_to(&mut output)?;
            Ok(())
        })
    }

    /// Set a configuration value in the repository
    fn set_config_value(&self, key: &str, value: &str, level: ConfigLevel) -> Result<()> {
        let mut entries = HashMap::new();
        entries.insert(key.to_string(), value.to_string());
        // Merge with existing config
        let existing = self.read_git_config(level)?;
        let mut merged = existing.entries;
        merged.insert(key.to_string(), value.to_string());
        let merged_config = GitConfig { entries: merged };
        self.write_git_config(&merged_config, level)
    }

    /// Add a new submodule to the repository
    fn add_submodule(&mut self, opts: &SubmoduleAddOptions) -> Result<()> {
        // gix does not support cloning in add_submodule; fall through to git2/CLI.
        Err(anyhow::anyhow!(
            "gix add_submodule not implemented: use git2 or CLI fallback for '{}'",
            opts.name
        ))
    }

    /// Initialize a submodule by reading its configuration and setting it up
    fn init_submodule(&mut self, path: &str) -> Result<()> {
        // 1. Read .gitmodules to get submodule configuration
        let entries = self.read_gitmodules()?;

        // 2. Find the submodule entry by path
        let submodule_entry = entries
            .submodule_iter()
            .find(|(_, entry)| entry.path.as_ref() == Some(&path.to_string()))
            .ok_or_else(|| anyhow::anyhow!("Submodule '{path}' not found in .gitmodules"))?;

        let (name, entry) = submodule_entry;
        let url = entry
            .url
            .as_ref()
            .ok_or_else(|| anyhow::anyhow!("Submodule '{name}' has no URL configured"))?;

        self.try_gix_operation(|repo| {
            // 3. Set up submodule configuration in .git/config
            let config_snapshot = repo.config_snapshot();
            let mut config_file = config_snapshot.to_owned();

            // Set submodule URL in local config
            let _url_key = format!("submodule.{name}.url");
            config_file.set_raw_value_by(
                "submodule",
                Some(name.as_bytes().as_bstr()),
                "url",
                url.as_bytes().as_bstr(),
            )?;

            // Set submodule active flag
            let _active_key = format!("submodule.{name}.active");
            config_file.set_raw_value_by(
                "submodule",
                Some(name.as_bytes().as_bstr()),
                "active",
                b"true".as_bstr(),
            )?;

            // 4. Check if submodule directory exists and is empty
            let workdir = repo
                .workdir()
                .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?;
            let submodule_path = workdir.join(path);

            if !submodule_path.exists() {
                std::fs::create_dir_all(&submodule_path)?;
            } else if submodule_path.read_dir()?.next().is_some() {
                // Directory exists and is not empty - this is fine for init
                // (unlike clone which would fail)
            }

            // 5. Clone the submodule if it doesn't exist yet
            if !submodule_path.join(".git").exists() {
                // Clone the submodule repository using gix
                let mut prepare = gix::prepare_clone(url.clone(), &submodule_path)?;
                if entry.shallow == Some(true) {
                    prepare = prepare
                        .with_shallow(gix::remote::fetch::Shallow::DepthAtRemote(1.try_into()?));
                }
                let should_interrupt =
                    std::sync::Arc::new(std::sync::atomic::AtomicBool::new(false));
                let progress = gix::progress::Discard;
                let (_checkout, _outcome) =
                    prepare.fetch_then_checkout(progress, &should_interrupt)?;
            }

            Ok(())
        })
    }

    /// Update a submodule to the latest commit in its remote repository
    fn update_submodule(&mut self, path: &str, opts: &SubmoduleUpdateOptions) -> Result<()> {
        let entries = self.read_gitmodules()?;
        let submodule_entry = entries
            .submodule_iter()
            .find(|(_, entry)| entry.path.as_ref() == Some(&path.to_string()))
            .ok_or_else(|| anyhow::anyhow!("Submodule '{path}' not found in .gitmodules"))?;
        let (name, entry) = submodule_entry;
        let url = entry
            .url
            .as_ref()
            .ok_or_else(|| anyhow::anyhow!("Submodule '{name}' has no URL configured"))?;
        let workdir = self
            .repo
            .workdir()
            .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?;
        let submodule_path = workdir.join(path);

        if !submodule_path.exists() || !submodule_path.join(".git").exists() {
            // Use gix::prepare_clone for proper remote operations
            let mut prepare = gix::prepare_clone(url.clone(), &submodule_path)?;
            if entry.shallow == Some(true) {
                prepare =
                    prepare.with_shallow(gix::remote::fetch::Shallow::DepthAtRemote(1.try_into()?));
            }
            let should_interrupt = std::sync::Arc::new(std::sync::atomic::AtomicBool::new(false));
            let progress = gix::progress::Discard;
            let (checkout, _outcome) = prepare.fetch_then_checkout(progress, &should_interrupt)?;
            if let Some(branch) = &entry.branch {
                let mut config_file = checkout.repo().config_snapshot().to_owned();
                match branch {
                    crate::options::SerializableBranch::Name(branch_name) => {
                        config_file.set_raw_value_by(
                            "branch",
                            Some(branch_name.as_bytes().as_bstr()),
                            "remote",
                            b"origin".as_bstr(),
                        )?;
                        config_file.set_raw_value_by(
                            "branch",
                            Some(branch_name.as_bytes().as_bstr()),
                            "merge",
                            format!("refs/heads/{branch_name}").as_bytes().as_bstr(),
                        )?;
                    }
                    crate::options::SerializableBranch::CurrentInSuperproject => {
                        // Set branch to current branch in superproject
                        let superproject_branch = self.get_superproject_branch()?;
                        config_file.set_raw_value_by(
                            "branch",
                            Some(superproject_branch.as_bytes().as_bstr()),
                            "remote",
                            b"origin".as_bstr(),
                        )?;
                        config_file.set_raw_value_by(
                            "branch",
                            Some(superproject_branch.as_bytes().as_bstr()),
                            "merge",
                            format!("refs/heads/{superproject_branch}")
                                .as_bytes()
                                .as_bstr(),
                        )?;
                    }
                }
            }
        } else {
            // Submodule exists — fetch updates using sync fetch_repo
            // Pass None to let gix resolve the default remote (which has refspecs configured).
            // Passing the URL string would create a bare remote without refspecs.
            let submodule_repo = gix::open(&submodule_path)?;
            fetch_repo(submodule_repo, None, entry.shallow == Some(true))
                .map_err(|e| anyhow::anyhow!("Failed to fetch submodule: {e}"))?;
            match opts.strategy {
                crate::options::SerializableUpdate::Checkout
                | crate::options::SerializableUpdate::Unspecified => {
                    // Fetch complete above
                }
                crate::options::SerializableUpdate::Merge => {
                    return Err(anyhow::anyhow!(
                        "Merge strategy not yet implemented with gix"
                    ));
                }
                crate::options::SerializableUpdate::Rebase => {
                    return Err(anyhow::anyhow!(
                        "Rebase strategy not yet implemented with gix"
                    ));
                }
                crate::options::SerializableUpdate::None => {
                    // No update
                }
            }
        }
        Ok(())
    }

    /// Delete a submodule by removing its configuration and content
    fn delete_submodule(&mut self, path: &str) -> Result<()> {
        // 1. Read .gitmodules to get submodule configuration (outside closure)
        let mut entries = self.read_gitmodules()?;

        // 2. Find the submodule entry by path
        let submodule_name = entries
            .submodule_iter()
            .find(|(_, entry)| entry.path.as_ref() == Some(&path.to_string()))
            .map(|(name, _)| name.to_string())
            .ok_or_else(|| anyhow::anyhow!("Submodule '{path}' not found in .gitmodules"))?;

        // 3. Remove from .gitmodules
        entries.remove_submodule(&submodule_name);
        self.write_gitmodules(&entries)?;

        self.try_gix_operation_mut(|repo| {
            // 4. Remove from git index using gix (fixed API usage)
            let index_path = repo.git_dir().join("index");
            if index_path.exists() {
                let mut index = gix::index::File::at(
                    &index_path,
                    gix::hash::Kind::Sha1,
                    false,
                    gix::index::decode::Options::default(),
                )?;
                // Remove all entries matching the submodule path prefix
                let remove_prefix = path;
                index.remove_entries(|_idx, path, _entry| {
                    let path_str = std::str::from_utf8(path).unwrap_or("");
                    path_str.starts_with(remove_prefix)
                });
                let mut index_file = std::fs::OpenOptions::new()
                    .write(true)
                    .truncate(true)
                    .open(&index_path)?;
                index.write_to(&mut index_file, gix::index::write::Options::default())?;
                let mut index_file = std::fs::OpenOptions::new()
                    .write(true)
                    .truncate(true)
                    .open(&index_path)?;
                index.write_to(&mut index_file, gix::index::write::Options::default())?;
            }

            // 5. Remove submodule configuration from .git/config
            let config_snapshot = repo.config_snapshot();
            let _config_file = config_snapshot.to_owned();
            let _config_file = config_snapshot.to_owned();

            // Remove all submodule.{name}.* entries
            let _section_name = format!("submodule.{submodule_name}");
            let _section_name = format!("submodule.{submodule_name}");
            // Note: gix config API for removing sections is complex
            // For now, we'll fall back to manual removal or git2 for this part
            // This is acceptable as it's a less common operation

            // 6. Remove the submodule directory from working tree
            let workdir = repo
                .workdir()
                .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?;
            let submodule_path = workdir.join(path);

            if submodule_path.exists() {
                std::fs::remove_dir_all(&submodule_path).with_context(|| {
                    format!(
                        "Failed to remove submodule directory at {}",
                        submodule_path.display()
                    )
                })?;
            }

            // 7. Remove .git/modules/{name} directory if it exists
            let modules_path = repo.git_dir().join("modules").join(&submodule_name);
            if modules_path.exists() {
                std::fs::remove_dir_all(&modules_path).with_context(|| {
                    format!(
                        "Failed to remove submodule git directory at {}",
                        modules_path.display()
                    )
                })?;
            }

            Ok(())
        })
    }

    /// Deinitialize a submodule, removing its configuration and content
    fn deinit_submodule(&mut self, path: &str, force: bool) -> Result<()> {
        let entries = self.read_gitmodules()?;
        let submodule_name = entries
            .submodule_iter()
            .find(|(_, entry)| entry.path.as_ref() == Some(&path.to_string()))
            .map(|(name, _)| name.to_string())
            .ok_or_else(|| anyhow::anyhow!("Submodule '{path}' not found in .gitmodules"))?;
        self.clone().try_gix_operation_mut(|repo| {
            // 1. Get the submodule directory
            let workdir = repo.workdir()
                .ok_or_else(|| anyhow::anyhow!("Repository has no working directory"))?;
            let submodule_path = workdir.join(path);

            // 2. Check if submodule has uncommitted changes (unless force is true)
            if !force && submodule_path.exists() && submodule_path.join(".git").exists() {
                if let Ok(submodule_repo) = gix::open(&submodule_path) {
                    // Check for uncommitted changes using gix.
                    // The is_dirty() method will return true if there are uncommitted changes,
                    // including untracked files and modifications to tracked files.
                    match submodule_repo.is_dirty() {
                        Ok(is_dirty) => {
                            if is_dirty {
                                return Err(anyhow::anyhow!(
                                    "Submodule '{path}' has uncommitted changes. Use force=true to override."
                                ));
                            }
                        }
                        Err(err) => {
                            // If we can't determine dirty status reliably, assume it might have changes
                            return Err(anyhow::anyhow!(
                                "Submodule '{path}' might have uncommitted changes. Use force=true to override.\nError: {err}"
                            ));
                        }
                    }
                } else {
                    return Err(anyhow::anyhow!(
                        "Submodule '{path}' might have uncommitted changes. Use force=true to override."
                    ));
                }
            }

            // 4. Remove submodule configuration from .git/config
            let config_snapshot = repo.config_snapshot();
            let _config_file = config_snapshot.to_owned();
            let _config_file = config_snapshot.to_owned();

            // Remove submodule.{name}.url and submodule.{name}.active
            // Note: gix config API for removing specific keys is complex
            // For a complete implementation, we might need to fall back to git2
            // or implement more sophisticated config manipulation

            // 5. Clear the submodule working directory
            if submodule_path.exists() {
                if force {
                    // Force removal of all content
                    std::fs::remove_dir_all(&submodule_path)
                        .with_context(|| format!("Failed to remove submodule directory at {}", submodule_path.display()))?;

                    // Recreate empty directory to maintain the path structure
                    std::fs::create_dir_all(&submodule_path)?;
                } else {
                    // Only remove .git directory and tracked files, preserve untracked files
                    let git_dir = submodule_path.join(".git");
                    if git_dir.exists() {
                        if git_dir.is_dir() {
                            std::fs::remove_dir_all(&git_dir)?;
                        } else {
                            // .git is a file (gitdir reference)
                            std::fs::remove_file(&git_dir)?;
                        }
                    }

                    // Remove tracked files by checking out empty tree
                    // This is complex to implement properly with gix
                    // For now, we'll do a simple approach by removing all files
                    // except untracked ones (which is hard to determine without proper status)
                    // We'll just remove common git-tracked file patterns
                    for entry in std::fs::read_dir(&submodule_path)? {
                        let entry = entry?;
                        let path = entry.path();
                        if path.is_file() {
                            std::fs::remove_file(&path).ok(); // Ignore errors for individual files
                        }
                    }
                }
            }

            // 6. Remove .git/modules/{name} directory if it exists
            let modules_path = repo.git_dir().join("modules").join(&submodule_name);
            if modules_path.exists() {
                std::fs::remove_dir_all(&modules_path)
                    .with_context(|| format!("Failed to remove submodule git directory at {}", modules_path.display()))?;
            }

            Ok(())
        })
    }
    /// Get the status of a submodule
    fn get_submodule_status(&self, _path: &str) -> Result<DetailedSubmoduleStatus> {
        Err(anyhow::anyhow!(
            "get_submodule_status not yet implemented with gix"
        ))
    }
    fn list_submodules(&self) -> Result<Vec<String>> {
        self.try_gix_operation(|repo| {
            let mut submodule_paths = Vec::new();
            if let Some(submodule_iter) = repo.submodules()? {
                for submodule in submodule_iter {
                    let path = submodule.path()?.to_string();
                    submodule_paths.push(path);
                }
            }
            Ok(submodule_paths)
        })
    }
    fn fetch_submodule(&self, _path: &str) -> Result<()> {
        // Pass None to let gix resolve the default remote (which has refspecs configured).
        let submodule_repo = utilities::repo_from_path(&std::path::PathBuf::from(_path))?;
        fetch_repo(submodule_repo, None, false)
            .map_err(|e| anyhow::anyhow!("Failed to fetch submodule: {e}"))
    }

    fn reset_submodule(&self, _path: &str, _hard: bool) -> Result<()> {
        // gix doesn't support submodule reset yet
        Err(anyhow::anyhow!(
            "gix submodule reset not yet supported, falling back to git2"
        ))
    }
    fn clean_submodule(&self, _path: &str, _force: bool, _remove_directories: bool) -> Result<()> {
        // gix doesn't support submodule cleaning yet
        Err(anyhow::anyhow!(
            "gix submodule cleaning not yet supported, falling back to git2"
        ))
    }
    fn stash_submodule(&self, _path: &str, _include_untracked: bool) -> Result<()> {
        // gix doesn't support stashing yet
        Err(anyhow::anyhow!(
            "gix stashing not yet supported, falling back to git2"
        ))
    }
    fn enable_sparse_checkout(&self, _path: &str) -> Result<()> {
        // Defer to git2 which correctly handles submodule paths
        Err(anyhow::anyhow!(
            "gix sparse checkout setup not implemented for submodule paths, falling back to git2"
        ))
    }
    fn set_sparse_patterns(&self, _path: &str, _patterns: &[String]) -> Result<()> {
        // Defer to git2 which correctly handles submodule paths
        Err(anyhow::anyhow!(
            "gix sparse patterns not implemented for submodule paths, falling back to git2"
        ))
    }
    fn get_sparse_patterns(&self, _path: &str) -> Result<Vec<String>> {
        // Defer to git2 which correctly handles submodule paths
        Err(anyhow::anyhow!(
            "gix get sparse patterns not implemented for submodule paths, falling back to git2"
        ))
    }
    fn apply_sparse_checkout(&self, _path: &str) -> Result<()> {
        self.try_gix_operation(|repo| {
            // Get sparse checkout patterns
            let patterns = self.get_sparse_patterns(_path)?;
            if patterns.is_empty() {
                return Ok(()); // No patterns to apply
            }

            // Load the index
            let index_path = repo.git_dir().join("index");
            let _index = gix::index::File::at(
                &index_path,
                gix::hash::Kind::Sha1,
                false,
                gix::index::decode::Options::default(),
            )?;

            // Use a simpler approach since remove_entries closure signature is complex
            // Fall back to git2 for now for sparse checkout application
            Err(anyhow::anyhow!(
                "gix sparse checkout application is complex, falling back to git2"
            ))
        })
    }
}

impl From<super::GitOpsManager> for GixOperations {
    fn from(git_ops: super::GitOpsManager) -> Self {
        git_ops
            .gix_ops
            
            .expect("GixOperations should always be initialized")
    }
}