From 2a152880f42422c2821d070105378008d9f27d07 Mon Sep 17 00:00:00 2001
From: Thorsten Sommer <mail@tsommer.org>
Date: Sun, 19 May 2024 16:09:52 +0200
Subject: [PATCH] Fixed handling of instance names while provider creation

---
 app/MindWork AI Studio/Components/Pages/Settings.razor.cs | 4 +---
 app/MindWork AI Studio/Provider/Providers.cs              | 7 ++++---
 app/MindWork AI Studio/Settings/ProviderDialog.razor.cs   | 7 ++-----
 3 files changed, 7 insertions(+), 11 deletions(-)

diff --git a/app/MindWork AI Studio/Components/Pages/Settings.razor.cs b/app/MindWork AI Studio/Components/Pages/Settings.razor.cs
index 94cc597..b97490d 100644
--- a/app/MindWork AI Studio/Components/Pages/Settings.razor.cs	
+++ b/app/MindWork AI Studio/Components/Pages/Settings.razor.cs	
@@ -88,9 +88,7 @@ public partial class Settings : ComponentBase
         if (dialogResult.Canceled)
             return;
         
-        var providerInstance = provider.UsedProvider.CreateProvider();
-        providerInstance.InstanceName = provider.InstanceName;
-        
+        var providerInstance = provider.UsedProvider.CreateProvider(provider.InstanceName);
         var deleteSecretResponse = await this.SettingsManager.DeleteAPIKey(this.JsRuntime, providerInstance);
         if(deleteSecretResponse.Success)
         {
diff --git a/app/MindWork AI Studio/Provider/Providers.cs b/app/MindWork AI Studio/Provider/Providers.cs
index 48c6638..0c93f42 100644
--- a/app/MindWork AI Studio/Provider/Providers.cs	
+++ b/app/MindWork AI Studio/Provider/Providers.cs	
@@ -28,15 +28,16 @@ public static class ExtensionsProvider
         Providers.NONE => "No provider selected",
         _ => "Unknown",
     };
-    
+
     /// <summary>
     /// Creates a new provider instance based on the provider value.
     /// </summary>
     /// <param name="provider">The provider value.</param>
+    /// <param name="instanceName">The used instance name.</param>
     /// <returns>The provider instance.</returns>
-    public static IProvider CreateProvider(this Providers provider) => provider switch
+    public static IProvider CreateProvider(this Providers provider, string instanceName) => provider switch
     {
-        Providers.OPEN_AI => new ProviderOpenAI(),
+        Providers.OPEN_AI => new ProviderOpenAI { InstanceName = instanceName },
         
         _ => new NoProvider(),
     };
diff --git a/app/MindWork AI Studio/Settings/ProviderDialog.razor.cs b/app/MindWork AI Studio/Settings/ProviderDialog.razor.cs
index 51e602f..91cf2b3 100644
--- a/app/MindWork AI Studio/Settings/ProviderDialog.razor.cs	
+++ b/app/MindWork AI Studio/Settings/ProviderDialog.razor.cs	
@@ -72,12 +72,10 @@ public partial class ProviderDialog : ComponentBase
         if(this.IsEditing)
         {
             this.dataEditingPreviousInstanceName = this.DataInstanceName.ToLowerInvariant();
-            var provider = this.DataProvider.CreateProvider();
+            var provider = this.DataProvider.CreateProvider(this.DataInstanceName);
             if(provider is NoProvider)
                 return;
             
-            provider.InstanceName = this.DataInstanceName;
-            
             // Load the API key:
             var requestedSecret = await this.SettingsManager.GetAPIKey(this.JsRuntime, provider);
             if(requestedSecret.Success)
@@ -124,8 +122,7 @@ public partial class ProviderDialog : ComponentBase
         };
         
         // We need to instantiate the provider to store the API key:
-        var provider = this.DataProvider.CreateProvider();
-        provider.InstanceName = this.DataInstanceName;
+        var provider = this.DataProvider.CreateProvider(this.DataInstanceName);
             
         // Store the API key in the OS secure storage:
         var storeResponse = await this.SettingsManager.SetAPIKey(this.JsRuntime, provider, this.dataAPIKey);