Method commit_submodule_changes

src/git.rs:437–499

Commit any uncommitted changes in submodules before the superproject commit. This ensures submodule changes are captured and the superproject pointer is updated. Returns a tuple of (committed submodule paths, non-fatal warnings).

(
        &self,
        repo_path: &Path,
        message: &str,
    )

    /// This ensures submodule changes are captured and the superproject pointer is updated.
    /// Returns a tuple of (committed submodule paths, non-fatal warnings).
    async fn commit_submodule_changes(
        &self,
        repo_path: &Path,
        message: &str,
    ) -> Result<(Vec<String>, Vec<String>), String> {
        let submodules = self.parse_gitmodules(repo_path).await?;

        if submodules.is_empty() {
            return Ok((Vec::new(), Vec::new()));
        }

        let mut committed_submodules = Vec::new();
        let mut warnings = Vec::new();

        for submodule in submodules {
            let submodule_path = repo_path.join(&submodule.path);

            // Check if the submodule is a valid git repository
            if !git_operations::is_git_repository(&submodule_path)
                .await
                .unwrap_or(false)
            {
                continue;
            }

            // Check if submodule has uncommitted changes
            let status_output = match git_operations::get_status(&submodule_path).await {
                Ok(output) => output,
                Err(e) => {
                    warnings.push(format!(
                        "Failed to get status for submodule '{}': {}",
                        submodule.path, e
                    ));
                    continue;
                }
            };

            if status_output.trim().is_empty() {
                continue;
            }

            // Add and commit changes in the submodule
            if let Err(e) = git_operations::add_all(&submodule_path).await {
                warnings.push(format!(
                    "Failed to stage changes in submodule '{}': {}",
                    submodule.path, e
                ));
                continue;
            }

            if let Err(e) = git_operations::commit(&submodule_path, message).await {
                warnings.push(format!(
                    "Failed to commit changes in submodule '{}': {}",
                    submodule.path, e
                ));
                continue;
            }