Refactor logging system and remove unused dependencies

Replaced `FlowSettings.LogLevel` with `PublicApi.Instance.GetLogLevel()` for dynamic log level retrieval. Introduced a centralized `LOGLEVEL` enum in the `Flow.Launcher.Plugin` namespace to standardize log level definitions. Added `GetLogLevel()` to the `IPublicAPI` interface for plugin access.

Removed the dependency on `CommunityToolkit.Mvvm.DependencyInjection` and unused namespaces across multiple files. Updated `PublicAPIInstance` to use a `Lock` type for `_saveSettingsLock`. Introduced a new `LogLevel.cs` file for the `LOGLEVEL` enum and removed the old enum from `Flow.Launcher.Infrastructure.Logger`.

Improved modularity, maintainability, and extensibility of the logging system while streamlining the codebase.

Author: Jack251970

Flow.Launcher.Core/Plugin/PluginManager.cs

@@ -6,12 +6,10 @@
 using System.Text.Json;
 using System.Threading;
 using System.Threading.Tasks;
-using CommunityToolkit.Mvvm.DependencyInjection;
 using Flow.Launcher.Core.ExternalPlugins;
 using Flow.Launcher.Core.Resource;
 using Flow.Launcher.Infrastructure;
 using Flow.Launcher.Infrastructure.DialogJump;
-using Flow.Launcher.Infrastructure.Logger;
 using Flow.Launcher.Infrastructure.UserSettings;
 using Flow.Launcher.Plugin;
 using Flow.Launcher.Plugin.SharedCommands;
@@ -34,7 +32,6 @@ public static class PluginManager
         private static readonly ConcurrentDictionary<string, PluginPair> _nonGlobalPlugins = [];
 
         private static PluginsSettings Settings;
-        private static readonly Settings FlowSettings = Ioc.Default.GetRequiredService<Settings>();
         private static readonly ConcurrentBag<string> ModifiedPlugins = [];
 
         private static readonly ConcurrentBag<PluginPair> _contextMenuPlugins = [];
@@ -280,14 +277,14 @@ public static async Task InitializePluginsAsync(IResultUpdateRegister register)
                     {
                         // If this plugin is already disabled, do not show error message again
                         // Or else it will be shown every time
-                        if (FlowSettings.LogLevel == LOGLEVEL.DEBUG)
+                        if (PublicApi.Instance.GetLogLevel() == LOGLEVEL.DEBUG)
                             PublicApi.Instance.LogDebug(ClassName, $"Skipped init for <{pair.Metadata.Name}> due to error");
                     }
                     else
                     {
                         pair.Metadata.Disabled = true;
                         pair.Metadata.HomeDisabled = true;
-                        if (FlowSettings.LogLevel == LOGLEVEL.DEBUG)
+                        if (PublicApi.Instance.GetLogLevel() == LOGLEVEL.DEBUG)
                             PublicApi.Instance.LogDebug(ClassName, $"Disable plugin <{pair.Metadata.Name}> because init failed");
                     }
 

Flow.Launcher.Core/Plugin/PluginsLoader.cs

@@ -2,7 +2,6 @@
 using System.Collections.Generic;
 using System.Linq;
 using System.Reflection;
-using CommunityToolkit.Mvvm.DependencyInjection;
 using Flow.Launcher.Core.ExternalPlugins.Environments;
 #pragma warning disable IDE0005
 using Flow.Launcher.Infrastructure.Logger;
@@ -15,7 +14,6 @@ namespace Flow.Launcher.Core.Plugin
     public static class PluginsLoader
     {
         private static readonly string ClassName = nameof(PluginsLoader);
-        private static readonly Settings FlowSettings = Ioc.Default.GetRequiredService<Settings>();
 
         public static List<PluginPair> Plugins(List<PluginMetadata> metadatas, PluginsSettings settings)
         {
@@ -110,7 +108,7 @@ private static List<PluginPair> DotNetPlugins(List<PluginMetadata> source)
                 });
 
                 metadata.InitTime += milliseconds;
-                if (FlowSettings.LogLevel == LOGLEVEL.DEBUG)
+                if (PublicApi.Instance.GetLogLevel() == LOGLEVEL.DEBUG)
                     PublicApi.Instance.LogDebug(ClassName, $"Constructor cost for <{metadata.Name}> is <{metadata.InitTime}ms>");
             }
 

Flow.Launcher.Infrastructure/Http/Http.cs

@@ -6,6 +6,7 @@
 using System.Threading.Tasks;
 using Flow.Launcher.Infrastructure.Logger;
 using Flow.Launcher.Infrastructure.UserSettings;
+using Flow.Launcher.Plugin;
 using JetBrains.Annotations;
 
 namespace Flow.Launcher.Infrastructure.Http

Flow.Launcher.Infrastructure/Logger/Log.cs

@@ -3,6 +3,7 @@
 using System.Runtime.CompilerServices;
 using System.Runtime.ExceptionServices;
 using Flow.Launcher.Infrastructure.UserSettings;
+using Flow.Launcher.Plugin;
 using NLog;
 using NLog.Config;
 using NLog.Targets;
@@ -167,12 +168,4 @@ public static void Warn(string className, string message, [CallerMemberName] str
             LogInternal(NLog.LogLevel.Warn, className, message, methodName);
         }
     }
-
-    public enum LOGLEVEL
-    {
-        NONE,
-        ERROR,
-        INFO,
-        DEBUG
-    }
 }

Flow.Launcher.Infrastructure/Stopwatch.cs

@@ -2,6 +2,7 @@
 using System.Runtime.CompilerServices;
 using System.Threading.Tasks;
 using Flow.Launcher.Infrastructure.Logger;
+using Flow.Launcher.Plugin;
 
 namespace Flow.Launcher.Infrastructure
 {

Flow.Launcher.Plugin/Interfaces/IPublicAPI.cs

@@ -637,5 +637,11 @@ public interface IPublicAPI
         /// </summary>
         /// <returns></returns>
         string GetLogDirectory();
+
+        /// <summary>
+        /// Get the current log level of Flow Launcher.
+        /// </summary>
+        /// <returns></returns>
+        LOGLEVEL GetLogLevel();
     }
 }

Flow.Launcher.Plugin/LogLevel.cs

@@ -0,0 +1,27 @@
+namespace Flow.Launcher.Plugin;
+
+/// <summary>
+/// Log level enum used in Flow
+/// </summary>
+public enum LOGLEVEL
+{
+    /// <summary>
+    /// No log will be produced
+    /// </summary>
+    NONE,
+
+    /// <summary>
+    /// Only error log will be produced
+    /// </summary>
+    ERROR,
+
+    /// <summary>
+    /// Info and error log will be produced
+    /// </summary>
+    INFO,
+
+    /// <summary>
+    /// Debug, info and error log will be produced
+    /// </summary>
+    DEBUG
+}

Flow.Launcher/PublicAPIInstance.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.Collections.Specialized;
@@ -52,7 +52,7 @@ public class PublicAPIInstance : IPublicAPI, IRemovable
         private Updater _updater;
         private Updater Updater => _updater ??= Ioc.Default.GetRequiredService<Updater>();
 
-        private readonly object _saveSettingsLock = new();
+        private readonly Lock _saveSettingsLock = new();
 
         #region Constructor
 
@@ -623,6 +623,8 @@ public event ActualApplicationThemeChangedEventHandler ActualApplicationThemeCha
 
         public string GetLogDirectory() => DataLocation.VersionLogDirectory;
 
+        public LOGLEVEL GetLogLevel() => Log.LogLevel;
+
         #endregion
 
         #region Private Methods