Cannot delete some providers (#56)

Fixed the bug where users cannot delete a self-hosted provider when an API token was never entered

app/MindWork AI Studio/Settings/SettingsManager.cs

@@ -73,9 +73,10 @@ public sealed class SettingsManager
     /// <summary>
     /// Data structure for deleting a secret response.
     /// </summary>
-    /// <param name="Success">True, when the secret was successfully deleted.</param>
+    /// <param name="Success">True, when the secret was successfully deleted or not found.</param>
     /// <param name="Issue">The issue, when the secret could not be deleted.</param>
-    public readonly record struct DeleteSecretResponse(bool Success, string Issue);
+    /// <param name="WasEntryFound">True, when the entry was found and deleted.</param>
+    public readonly record struct DeleteSecretResponse(bool Success, string Issue, bool WasEntryFound);
     
     /// <summary>
     /// Tries to delete the API key for the given provider.

app/MindWork AI Studio/wwwroot/changelog/v0.8.5.md

@@ -6,6 +6,7 @@
 - Improved readability of the issue component
 - Improved switches: when an option is enabled, the switch is using a different color
 - Fixed the applying of spellchecking settings to the single-line dialog
+- Fixed the bug where users cannot delete a self-hosted provider when an API token was never entered
 - Restructured the layout of the settings page 
 - Refactored the settings data model
 - Upgraded to Rust 1.80.0

runtime/src/main.rs

@@ -15,6 +15,7 @@ use tauri::api::process::{Command, CommandChild, CommandEvent};
 use tauri::utils::config::AppUrl;
 use tokio::time;
 use flexi_logger::{AdaptiveFormat, Logger};
+use keyring::error::Error::NoEntry;
 use log::{debug, error, info, warn};
 use tauri::updater::UpdateResponse;
 
@@ -454,19 +455,31 @@ fn delete_secret(destination: String, user_name: String) -> DeleteSecretResponse
     let service = format!("mindwork-ai-studio::{}", destination);
     let entry = Entry::new(service.as_str(), user_name.as_str()).unwrap();
     let result = entry.delete_password();
+
     match result {
         Ok(_) => {
             warn!("Secret for {service} and user {user_name} was deleted successfully.");
             DeleteSecretResponse {
                 success: true,
+                was_entry_found: true,
                 issue: String::from(""),
             }
         },
 
+        Err(NoEntry) => {
+            warn!("No secret for {service} and user {user_name} was found.");
+            DeleteSecretResponse {
+                success: true,
+                was_entry_found: false,
+                issue: String::from(""),
+            }
+        }
+        
         Err(e) => {
             error!("Failed to delete secret for {service} and user {user_name}: {e}.");
             DeleteSecretResponse {
                 success: false,
+                was_entry_found: false,
                 issue: e.to_string(),
             }
         },
@@ -476,6 +489,7 @@ fn delete_secret(destination: String, user_name: String) -> DeleteSecretResponse
 #[derive(Serialize)]
 struct DeleteSecretResponse {
     success: bool,
+    was_entry_found: bool,
     issue: String,
 }