refactor: remove default initialization for boolean fields and improve config serialization

Author: chemodun

Config.cs

@@ -12,15 +12,16 @@ namespace X4LogWatcher
   public class Config : INotifyPropertyChanged
   {
     private static readonly int MaxRecentProfiles = 5;
-    private bool _isSaving = false; // Flag to prevent recursive saves
+    private bool _isSaving; // Flag to prevent recursive saves
 
     // Backing fields for properties
     private ObservableCollection<string> _recentProfiles = new ObservableCollection<string>();
     private string? _activeProfile;
     private string? _lastLogFolderPath;
     private string _logFileExtension = ".log";
     private bool _skipSignatureErrors = true;
-    private bool _realTimeStamping = false;
+    private bool _realTimeStamping;
+    private JsonSerializerOptions _jsonSerializerOptions = new JsonSerializerOptions { WriteIndented = true };
 
     // Properties with notification
     public ObservableCollection<string> RecentProfiles
@@ -88,7 +89,7 @@ public static void SaveConfig(Config config)
         };
 
         // Serialize the config to JSON and write to file
-        string jsonConfig = JsonSerializer.Serialize(configToSave, new JsonSerializerOptions { WriteIndented = true });
+        string jsonConfig = JsonSerializer.Serialize(configToSave, config._jsonSerializerOptions);
         File.WriteAllText(configPath, jsonConfig);
       }
       catch (Exception ex)
@@ -156,11 +157,8 @@ public void AddRecentProfile(string profilePath)
 
       try
       {
-        // Remove the profile if it already exists
-        if (RecentProfiles.Contains(profilePath))
-        {
-          RecentProfiles.Remove(profilePath);
-        }
+        // Remove the profile if it already exists (Remove returns true if found and removed)
+        RecentProfiles.Remove(profilePath);
 
         // Add the profile to the beginning of the list
         RecentProfiles.Insert(0, profilePath);

MenuAutoTabsHandlers.cs

@@ -23,13 +23,12 @@ private void MenuAutoTabs_Click(object sender, RoutedEventArgs e)
       {
         // Add the configs to the menu
         foreach (var config in autoTabConfigs)
-        {
-          // Create a display name that includes regex pattern details
+        { // Create a display name that includes regex pattern details
           string configName = config.PatternRegex;
 
           if (configName.Length > 80)
           {
-            configName = configName.Substring(0, 77) + "...";
+            configName = string.Concat(configName.AsSpan(0, 77), "...");
           }
 
           var configMenuItem = new MenuItem { Header = configName, Tag = config };

TabInfo.cs

@@ -17,7 +17,7 @@ public class TabInfo : INotifyPropertyChanged, IDisposable
   {
     public event PropertyChangedEventHandler? PropertyChanged;
 
-    private bool _disposed = false;
+    private bool _disposed;
 
     private string _regexPattern = string.Empty;
     public string RegexPattern
@@ -416,7 +416,7 @@ private bool IsUserAtBottom()
     /// <summary>
     /// Finds the ScrollViewer inside a control using visual tree walking
     /// </summary>
-    private ScrollViewer? FindScrollViewer(DependencyObject parent)
+    private static ScrollViewer? FindScrollViewer(DependencyObject parent)
     {
       if (parent == null)
         return null;