From 03bf288844541e3703c409b4cb84fe92e3a5120a Mon Sep 17 00:00:00 2001 From: Thorsten Sommer Date: Sun, 1 Jun 2025 20:46:45 +0200 Subject: [PATCH] Refactor TemporaryChatService for cleaner logging and improvements --- .../Tools/Services/TemporaryChatService.cs | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/app/MindWork AI Studio/Tools/Services/TemporaryChatService.cs b/app/MindWork AI Studio/Tools/Services/TemporaryChatService.cs index ea7a26ee..ab2f39e7 100644 --- a/app/MindWork AI Studio/Tools/Services/TemporaryChatService.cs +++ b/app/MindWork AI Studio/Tools/Services/TemporaryChatService.cs @@ -3,13 +3,11 @@ using AIStudio.Settings.DataModel; namespace AIStudio.Tools.Services; -public class TemporaryChatService(ILogger logger, SettingsManager settingsManager) : BackgroundService +public sealed class TemporaryChatService(ILogger logger, SettingsManager settingsManager) : BackgroundService { private static readonly TimeSpan CHECK_INTERVAL = TimeSpan.FromDays(1); private static bool IS_INITIALIZED; - private readonly ILogger logger = logger; - #region Overrides of BackgroundService protected override async Task ExecuteAsync(CancellationToken stoppingToken) @@ -17,12 +15,12 @@ public class TemporaryChatService(ILogger logger, Settings while (!stoppingToken.IsCancellationRequested && !IS_INITIALIZED) await Task.Delay(TimeSpan.FromSeconds(3), stoppingToken); - this.logger.LogInformation("The temporary chat maintenance service was initialized."); + logger.LogInformation("The temporary chat maintenance service was initialized."); await settingsManager.LoadSettings(); if(settingsManager.ConfigurationData.Workspace.StorageTemporaryMaintenancePolicy is WorkspaceStorageTemporaryMaintenancePolicy.NO_AUTOMATIC_MAINTENANCE) { - this.logger.LogWarning("Automatic maintenance of temporary chat storage is disabled. Exiting maintenance service."); + logger.LogWarning("Automatic maintenance of temporary chat storage is disabled. Exiting maintenance service."); return; } @@ -38,11 +36,11 @@ public class TemporaryChatService(ILogger logger, Settings private Task StartMaintenance() { - this.logger.LogInformation("Starting maintenance of temporary chat storage."); + logger.LogInformation("Starting maintenance of temporary chat storage."); var temporaryDirectories = Path.Join(SettingsManager.DataDirectory, "tempChats"); if(!Directory.Exists(temporaryDirectories)) { - this.logger.LogWarning("Temporary chat storage directory does not exist. End maintenance."); + logger.LogWarning("Temporary chat storage directory does not exist. End maintenance."); return Task.CompletedTask; } @@ -67,12 +65,12 @@ public class TemporaryChatService(ILogger logger, Settings if(deleteChat) { - this.logger.LogInformation($"Deleting temporary chat storage directory '{tempChatDirPath}' due to maintenance policy."); + logger.LogInformation($"Deleting temporary chat storage directory '{tempChatDirPath}' due to maintenance policy."); Directory.Delete(tempChatDirPath, true); } } - this.logger.LogInformation("Finished maintenance of temporary chat storage."); + logger.LogInformation("Finished maintenance of temporary chat storage."); return Task.CompletedTask; }