Setup config values at the start and pass as argument where possible

src/FaluCli/Extensions/IServiceCollectionExtensions.cs

@@ -11,15 +11,19 @@ internal static class IServiceCollectionExtensions
 {
     // services are registered as transit to allow for easier debugging because no scope is created by the parser
 
-    public static IServiceCollection AddFaluClientForCli(this IServiceCollection services)
+    public static IServiceCollection AddFaluClientForCli(this IServiceCollection services, ConfigValues configValues)
     {
-        // A dummy ApiKey is used so that the options validator can pass
-        services.AddFalu<FaluCliClient, FaluClientOptions>(o => o.ApiKey = "dummy")
-                .AddHttpMessageHandler<FaluCliClientHandler>()
-                .ConfigureHttpClientStandard();
+        var builder = services.AddFalu<FaluCliClient, FaluClientOptions>(options =>
+        {
+            // A dummy ApiKey is used so that the options validator can pass
+            options.ApiKey = "dummy";
+            options.Retries = configValues.Retries;
+        });
+
+        builder.AddHttpMessageHandler<FaluCliClientHandler>()
+               .ConfigureHttpClientStandard();
 
         services.AddTransient<FaluCliClientHandler>();
-        services.ConfigureOptions<FaluClientConfigureOptions>();
 
         return services;
     }
@@ -32,11 +36,6 @@ public static IServiceCollection AddUpdateChecker(this IServiceCollection servic
         return services.AddSingleton<IHostedService>(p => p.GetRequiredService<UpdateChecker>());
     }
 
-    public static IServiceCollection AddConfigValuesProvider(this IServiceCollection services)
-    {
-        return services.AddTransient<IConfigValuesProvider, ConfigValuesProvider>();
-    }
-
     public static IServiceCollection AddOpenIdProvider(this IServiceCollection services)
     {
         services.AddHttpClient<OidcProvider>(name: "Oidc")

src/FaluCli/Extensions/InvocationContextExtensions.cs

@@ -8,7 +8,6 @@ namespace System.CommandLine;
 internal static class InvocationContextExtensions
 {
     public static bool IsVerboseEnabled(this InvocationContext context) => context.ParseResult.ValueForOption<bool>("--verbose");
-    public static bool IsTelemetryDisabled(this InvocationContext context) => context.ParseResult.ValueForOption<bool>("--no-telemetry");
     public static string? GetWorkspaceId(this InvocationContext context) => context.ParseResult.ValueForOption<string>("--workspace");
     public static bool? GetLiveMode(this InvocationContext context) => context.ParseResult.ValueForOption<bool?>("--live");
 

src/FaluCli/Program.cs

@@ -7,6 +7,7 @@
 using Falu.Commands.Money.Statements;
 using Falu.Commands.RequestLogs;
 using Falu.Commands.Templates;
+using Falu.Config;
 using System.CommandLine.Builder;
 using System.CommandLine.Hosting;
 using Res = Falu.Properties.Resources;
@@ -72,15 +73,18 @@
 
 rootCommand.Description = "Official CLI tool for Falu.";
 rootCommand.AddGlobalOption(["-v", "--verbose"], "Whether to output verbosely.", false);
-rootCommand.AddGlobalOption<bool?>(["--skip-update-checks"], Res.OptionDescriptionSkipUpdateCheck); // nullable so as to allow checking if specified
+rootCommand.AddGlobalOption<bool?>(["--skip-update-checks"], Res.OptionDescriptionSkipUpdateCheck); // nullable so as to allow checking if not specified
+
+var configValuesProvider = new ConfigValuesProvider();
+var configValues = await configValuesProvider.GetConfigValuesAsync();
 
 var builder = new CommandLineBuilder(rootCommand)
     .UseHost(_ => Host.CreateDefaultBuilder(args), host =>
     {
         host.ConfigureAppConfiguration((context, builder) =>
         {
-            var iv = context.GetInvocationContext();
-            var verbose = iv.IsVerboseEnabled();
+            var invocation = context.GetInvocationContext();
+            var verbose = invocation.IsVerboseEnabled();
 
             builder.AddInMemoryCollection(new Dictionary<string, string?>
             {
@@ -113,9 +117,9 @@
         host.ConfigureServices((context, services) =>
         {
             var configuration = context.Configuration;
-            services.AddFaluClientForCli();
+            services.AddSingleton<IConfigValuesProvider>(configValuesProvider);
+            services.AddFaluClientForCli(configValues);
             services.AddUpdateChecker();
-            services.AddConfigValuesProvider();
             services.AddOpenIdProvider();
             services.AddTransient<WebsocketHandler>();
         });