refactor: Copilot review

Author: massimopaganigh

src/Commands/SharedIssueTracker.cs

@@ -58,6 +58,30 @@ public SharedIssueTracker(string repo)
             return rs;
         }
 
+        public async Task<bool> HasAnyAsync()
+        {
+            if (!File.Exists(_file))
+                return false;
+
+            Args = $"config -f {_file.Quoted()} -l";
+
+            var output = await ReadToEndAsync().ConfigureAwait(false);
+
+            if (output.IsSuccess)
+                foreach (var line in output.StdOut.Split(['\r', '\n'], StringSplitOptions.RemoveEmptyEntries))
+                {
+                    var parts = line.Split('=', 2);
+
+                    if (parts.Length < 2)
+                        continue;
+
+                    if (parts[0].StartsWith("issuetracker.", StringComparison.Ordinal))
+                        return true;
+                }
+
+            return false;
+        }
+
         public async Task<bool> AddAsync(Models.IssueTrackerRule rule)
         {
             Args = $"config -f {_file.Quoted()} issuetracker.{rule.Name.Quoted()}.regex {rule.RegexString.Quoted()}";

src/ViewModels/RepositoryConfigure.cs

@@ -314,16 +314,15 @@ public async Task ChangeIssueTrackerShareModeAsync()
             {
                 await sharedTracker.RemoveAsync(rule);
 
-                if ((await sharedTracker.ReadAllAsync()).Count != 0)
+                if (await sharedTracker.HasAnyAsync())
                     return;
-                
+
                 var filePath = Path.Combine(_repo.FullPath, ".issuetracker");
 
-                if (!string.IsNullOrEmpty(await File.ReadAllTextAsync(filePath)))
+                if (!File.Exists(filePath) || !string.IsNullOrEmpty(await File.ReadAllTextAsync(filePath)))
                     return;
-                
-                if (File.Exists(filePath))
-                    File.Delete(filePath);
+
+                File.Delete(filePath);
             }
         }