feat: wip on remove config command

src/AltGen.Console.Tests/Unit/AddConfigCommandTests.cs

@@ -0,0 +1,6 @@
+namespace AltGen.Console.Tests.Unit;
+
+public class AddConfigCommandTests
+{
+
+}

src/AltGen.Console.Tests/Unit/AppSettingsTests.cs

@@ -0,0 +1,5 @@
+namespace AltGen.Console.Tests.Unit;
+
+public class AppSettingsTests
+{
+}

src/AltGen.Console.Tests/Unit/RemoveConfigCommandTests.cs

@@ -0,0 +1,5 @@
+namespace AltGen.Console.Tests.Unit;
+
+public class RemoveConfigCommandTests
+{
+}

src/AltGen.Console/Common/HostBuilderExtensions.cs

@@ -7,7 +7,16 @@ static class HostBuilderExtensions
     var registrar = new TypeRegistrar(builder);
     var app = new CommandApp<GenerateCommand>(registrar);
 
-    app.Configure(static c => c.AddCommand<ConfigCommand>("config"));
+    app.Configure(static c =>
+    {
+      c.PropagateExceptions();
+
+      c.AddBranch("config", static c =>
+      {
+        c.AddCommand<AddConfigCommand>("add");
+        c.AddCommand<RemoveConfigCommand>("remove");
+      });
+    });
 
     return app;
   }

src/AltGen.Console/Config/AddConfigCommand.cs

@@ -1,19 +1,9 @@
-
-using Microsoft.Extensions.Configuration;
-
 namespace AltGen.Console.Config;
 
-// TODO: We should refactor this so that
-// we have a config add and a config remove command
-// the config add will have basically the same code
-// as below but the config remove will just
-// accept a provider identifier and remove it
-
-sealed class ConfigCommand(
+sealed class AddConfigCommand(
   IAnsiConsole console,
-  IFileSystem fileSystem,
-  IConfiguration config
-) : AsyncCommand<ConfigCommand.Settings>
+  IFileSystem fileSystem
+) : AsyncCommand<AddConfigCommand.Settings>
 {
   static readonly JsonSerializerOptions JsonOptions = new()
   {
@@ -23,7 +13,6 @@ sealed class ConfigCommand(
   };
   readonly IAnsiConsole _console = console;
   readonly IFileSystem _fileSystem = fileSystem;
-  readonly IConfiguration _config = config;
 
   public class Settings(IFileSystem filesystem) : CommandSettings
   {
@@ -63,47 +52,10 @@ sealed class ConfigCommand(
     var existingJson = await _fileSystem.File.ReadAllTextAsync(settings.SettingsPath);
     var existingAppSettings = JsonSerializer.Deserialize<AppSettings>(existingJson, JsonOptions)
       ?? throw new ConfigException("Failed to deserialize settings.");
-    var updatedAppSettings = existingAppSettings.Update(settings);
+    var updatedAppSettings = existingAppSettings.AddOrUpdateProvider(settings);
     var udpatedJson = JsonSerializer.Serialize(updatedAppSettings, JsonOptions);
     await _fileSystem.File.WriteAllTextAsync(settings.SettingsPath, udpatedJson);
     _console.MarkupLine($"[bold]{settings.Provider}[/] has been configured.");
     return 0;
   }
-}
-
-record AppSettings(ProviderSettings[] Providers)
-{
-  public AppSettings Update(ConfigCommand.Settings settings)
-  {
-    var updatedProviders = Providers.Select(p =>
-    {
-      if (p.Provider == settings.Provider)
-      {
-        return p with { Key = settings.Key, Default = settings.Default };
-      }
-
-      if (p.Provider != settings.Provider && settings.Default)
-      {
-        return p with { Default = false };
-      }
-
-      return p;
-    });
-
-    var provider = Providers.FirstOrDefault(p => p.Provider == settings.Provider);
-
-    if (provider is null)
-    {
-      var newProvider = new ProviderSettings(settings.Provider, settings.Key, settings.Default);
-      return this with { Providers = [.. updatedProviders, newProvider] };
-    }
-
-    return this with { Providers = [.. updatedProviders] };
-  }
-}
-
-record ProviderSettings(string Provider, string Key, bool Default);
-
-class ConfigException(string message) : Exception(message)
-{
 }

src/AltGen.Console/Config/AppSettings.cs

@@ -0,0 +1,38 @@
+namespace AltGen.Console.Config;
+
+record AppSettings(ProviderSettings[] Providers)
+{
+  public AppSettings AddOrUpdateProvider(AddConfigCommand.Settings settings)
+  {
+    var updatedProviders = Providers.Select(p =>
+    {
+      if (p.Provider == settings.Provider)
+      {
+        return p with { Key = settings.Key, Default = settings.Default };
+      }
+
+      if (p.Provider != settings.Provider && settings.Default)
+      {
+        return p with { Default = false };
+      }
+
+      return p;
+    });
+
+    var provider = Providers.FirstOrDefault(p => p.Provider == settings.Provider);
+
+    if (provider is null)
+    {
+      var newProvider = new ProviderSettings(settings.Provider, settings.Key, settings.Default);
+      return this with { Providers = [.. updatedProviders, newProvider] };
+    }
+
+    return this with { Providers = [.. updatedProviders] };
+  }
+
+  public AppSettings RemoveProvider(RemoveConfigCommand.Settings settings)
+  {
+    var updatedProviders = Providers.Where(p => p.Provider != settings.Provider);
+    return this with { Providers = [.. updatedProviders] };
+  }
+}

src/AltGen.Console/Config/ConfigException.cs

@@ -0,0 +1,5 @@
+namespace AltGen.Console.Config;
+
+class ConfigException(string message) : Exception(message)
+{
+}

src/AltGen.Console/Config/ProviderSettings.cs

@@ -0,0 +1,3 @@
+namespace AltGen.Console.Config;
+
+record ProviderSettings(string Provider, string Key, bool Default);

src/AltGen.Console/Config/RemoveConfigCommand.cs

@@ -1,16 +1,40 @@
 namespace AltGen.Console.Config;
 
-sealed class RemoveConfigCommand() : AsyncCommand<RemoveConfigCommand.Settings>
+sealed class RemoveConfigCommand(
+  IFileSystem fileSystem,
+  IAnsiConsole console
+) : AsyncCommand<RemoveConfigCommand.Settings>
 {
-  public class Settings : CommandSettings
+  readonly IFileSystem _fileSystem = fileSystem;
+  readonly IAnsiConsole _console = console;
+
+  public class Settings(IFileSystem fileSystem) : CommandSettings
   {
+    readonly IFileSystem _fileSystem = fileSystem;
+
     [CommandArgument(1, "<provider>")]
     [Description("The provider to remove.")]
     public string Provider { get; init; } = string.Empty;
+
+    public string SettingsPath => _fileSystem.Path.Combine(AppContext.BaseDirectory, "appsettings.json");
   }
 
   public override async Task<int> ExecuteAsync(CommandContext context, Settings settings)
   {
+    var exists = _fileSystem.Path.Exists(settings.SettingsPath);
+
+    if (exists is false)
+    {
+      _console.MarkupLine("[bold]No existing settings found.[/]");
+      return 1;
+    }
+
+    var settingsJson = await _fileSystem.File.ReadAllTextAsync(settings.SettingsPath);
+    var appSettings = JsonSerializer.Deserialize<AppSettings>(settingsJson) ?? throw new ConfigException("Failed to deserialize app settings.");
+    var updatedAppSettings = appSettings.RemoveProvider(settings);
+    var updatedJson = JsonSerializer.Serialize(updatedAppSettings);
+    await _fileSystem.File.WriteAllTextAsync(settings.SettingsPath, updatedJson);
+    _console.MarkupLine($"[bold]{settings.Provider}[/] has been removed.");
     return 0;
   }
 }