💡 run distinct on configuration

* To account for any duplicate team names in the configuration
* Thanks PC for bringing this potential issue to my attention

Author: JoshuaTheMiller

source/Gttsb.Core/SyncInput.cs

@@ -1,6 +1,6 @@
 namespace Gttsb.Core
 {
-    public sealed class SyncInput
+    public sealed record SyncInput
     {
         public IEnumerable<string> GitHubTeamNames { get; init; } = Enumerable.Empty<string>();
         public string EmailAppend { get; init; } = String.Empty;

source/Gttsb.Gh/InstalledGitHubFacade.cs

@@ -1,4 +1,5 @@
 using Gttsb.Core;
+using Newtonsoft.Json;
 using Octokit;
 using Octokit.GraphQL;
 using YamlDotNet.Serialization;
@@ -163,9 +164,27 @@ public async Task<SyncInput> GetConfigurationForInstallationAsync()
             var configurationAsString = System.Text.Encoding.Default.GetString(configurationContent);
 
             var deserializer = new DeserializerBuilder().Build();
-            var yamlObject = deserializer.Deserialize<SyncInput>(configurationAsString);
 
-            return yamlObject;
+            // TODO: fix this nonsense
+            // https://github.com/aaubry/YamlDotNet/issues/571
+            var rawYamlObject = deserializer.Deserialize<object>(configurationAsString);
+            var syncInput = JsonConvert.DeserializeObject<SyncInput>(JsonConvert.SerializeObject(rawYamlObject));
+            
+            if(syncInput == null)
+            {
+                // TODO: throw proper custom exception
+                throw new Exception("Configuration appears to be null");
+            }
+
+            // One could argue that such logic is too many responsibilities for this method. I pose this is a fine tradeoff though...
+            // Once we get around to implementing config file status checks, I will most likely change my mind.
+            // This could also be turned into a Set.
+            syncInput = syncInput with
+            {
+                GitHubTeamNames = syncInput.GitHubTeamNames.Distinct().ToList()
+            };
+
+            return syncInput;
         }
     }
 }