feat: add verbose logging

Author: amustaque97

sync-team/src/github/mod.rs

@@ -510,10 +510,11 @@ impl SyncGitHub {
                     // No change needed
                     continue;
                 }
-                Some(_actual_env) => {
+                Some(actual_env) => {
                     // Environment exists but branches differ - update it
                     environment_diffs.push(EnvironmentDiff::Update(
                         env_name.clone(),
+                        actual_env.branches.clone(),
                         expected_env.branches.clone(),
                     ));
                 }
@@ -966,7 +967,7 @@ struct UpdateRepoDiff {
 #[derive(Debug)]
 enum EnvironmentDiff {
     Create(String, Vec<String>),
-    Update(String, Vec<String>),
+    Update(String, Vec<String>, Vec<String>),
     Delete(String),
 }
 
@@ -1030,8 +1031,8 @@ impl UpdateRepoDiff {
                 EnvironmentDiff::Create(name, branches) => {
                     sync.create_environment(&self.org, &self.name, name, branches)?;
                 }
-                EnvironmentDiff::Update(name, branches) => {
-                    sync.update_environment(&self.org, &self.name, name, branches)?;
+                EnvironmentDiff::Update(name, _old_branches, new_branches) => {
+                    sync.update_environment(&self.org, &self.name, name, new_branches)?;
                 }
                 EnvironmentDiff::Delete(name) => {
                     sync.delete_environment(&self.org, &self.name, name)?;
@@ -1118,10 +1119,28 @@ impl std::fmt::Display for UpdateRepoDiff {
                             writeln!(f, "        Branches: {}", branches.join(", "))?;
                         }
                     }
-                    EnvironmentDiff::Update(name, branches) => {
+                    EnvironmentDiff::Update(name, old_branches, new_branches) => {
                         writeln!(f, "    🔄 Update: {name}")?;
-                        if !branches.is_empty() {
-                            writeln!(f, "        Branches: {}", branches.join(", "))?;
+                        let old_set: HashSet<_> = old_branches.iter().collect();
+                        let new_set: HashSet<_> = new_branches.iter().collect();
+
+                        let mut added: Vec<_> =
+                            new_set.difference(&old_set).map(|s| s.as_str()).collect();
+                        let mut removed: Vec<_> =
+                            old_set.difference(&new_set).map(|s| s.as_str()).collect();
+
+                        // Sort for deterministic output
+                        added.sort();
+                        removed.sort();
+
+                        if !added.is_empty() {
+                            writeln!(f, "        Adding branches: {}", added.join(", "))?;
+                        }
+                        if !removed.is_empty() {
+                            writeln!(f, "        Removing branches: {}", removed.join(", "))?;
+                        }
+                        if added.is_empty() && removed.is_empty() {
+                            writeln!(f, "        No branch changes")?;
                         }
                     }
                     EnvironmentDiff::Delete(name) => writeln!(f, "    ❌ Delete: {name}")?,

sync-team/src/github/tests/mod.rs

@@ -1108,3 +1108,62 @@ fn repo_environment_update() {
     ]
     "#);
 }
+
+#[test]
+fn repo_environment_update_branches() {
+    let mut model = DataModel::default();
+    model.create_repo(
+        RepoData::new("repo1").environment_with_branches("production", &["main", "release/*"]),
+    );
+    let gh = model.gh_model();
+
+    // Update branches for production environment
+    model.get_repo("repo1").environments.insert(
+        "production".to_string(),
+        v1::Environment {
+            branches: vec!["main".to_string(), "stable".to_string()],
+        },
+    );
+
+    let diff = model.diff_repos(gh);
+    insta::assert_debug_snapshot!(diff, @r#"
+    [
+        Update(
+            UpdateRepoDiff {
+                org: "rust-lang",
+                name: "repo1",
+                repo_node_id: "0",
+                settings_diff: (
+                    RepoSettings {
+                        description: "",
+                        homepage: None,
+                        archived: false,
+                        auto_merge_enabled: false,
+                    },
+                    RepoSettings {
+                        description: "",
+                        homepage: None,
+                        archived: false,
+                        auto_merge_enabled: false,
+                    },
+                ),
+                permission_diffs: [],
+                branch_protection_diffs: [],
+                environment_diffs: [
+                    Update(
+                        "production",
+                        [
+                            "main",
+                            "release/*",
+                        ],
+                        [
+                            "main",
+                            "stable",
+                        ],
+                    ),
+                ],
+            },
+        ),
+    ]
+    "#);
+}