update Linux/macOS installer color scheme logic for 4.5.0+ changes

The selected color scheme should now be set in `config.user.json` instead of `config.json`, and the field was renamed.

Author: Pathoschild

docs/release-notes.md

@@ -3,6 +3,7 @@
 # Release notes
 ## Upcoming release
 * For players:
+  * Fixed the Linux/macOS installer not saving the color scheme correctly in 4.5.0+.
   * Fixed typo in config UI text (thanks to QuentiumYT!).
   * Improved translations. Thanks to dewanggatrustha (updated Indonesian), QuentiumYT (updated French), and  Timur13240
  (updated Russian)!

src/SMAPI.Installer/InteractiveInstaller.cs

@@ -6,13 +6,15 @@
 using System.IO;
 using System.Linq;
 using System.Reflection;
+using Newtonsoft.Json.Linq;
 using StardewModdingApi.Installer.Enums;
 using StardewModdingAPI.Installer.Framework;
 using StardewModdingAPI.Internal.ConsoleWriting;
 using StardewModdingAPI.Toolkit;
 using StardewModdingAPI.Toolkit.Framework;
 using StardewModdingAPI.Toolkit.Framework.GameScanning;
 using StardewModdingAPI.Toolkit.Framework.ModScanning;
+using StardewModdingAPI.Toolkit.Serialization;
 using StardewModdingAPI.Toolkit.Utilities;
 
 namespace StardewModdingApi.Installer;
@@ -501,10 +503,10 @@ public void Run(string[] args)
                 // set SMAPI's color scheme if defined
                 if (scheme != MonitorColorScheme.AutoDetect)
                 {
-                    string text = File
-                        .ReadAllText(paths.ApiConfigPath)
-                        .Replace(@"""UseScheme"": ""AutoDetect""", $@"""UseScheme"": ""{scheme}""");
-                    File.WriteAllText(paths.ApiConfigPath, text);
+                    this.SaveUserSettings(paths.ApiUserConfigPath, new Dictionary<string, object>
+                    {
+                        ["ConsoleColorScheme"] = scheme.ToString()
+                    });
                 }
             }
         }
@@ -559,6 +561,33 @@ private string GetDisplayText(MonitorColorScheme scheme)
         }
     }
 
+    /// <summary>Save the given options to a SMAPI internal config file.</summary>
+    /// <param name="filePath">The file path to edit.</param>
+    /// <param name="settings">The settings to add or overwrite.</param>
+    [SuppressMessage("Trimming", "IL2026:Members annotated with 'RequiresUnreferencedCodeAttribute' require dynamic access otherwise can break functionality when trimming application code", Justification = "This isn't an issue here, since we're only using JToken and object.")]
+    private void SaveUserSettings(string filePath, Dictionary<string, object> settings)
+    {
+        JsonHelper jsonHelper = new();
+
+        // merge into existing settings
+        Dictionary<string, JToken> saveSettings = settings.ToDictionary(p => p.Key, p => JToken.FromObject(p.Value));
+        try
+        {
+            if (jsonHelper.ReadJsonFileIfExists(filePath, out Dictionary<string, JToken>? fileSettings))
+            {
+                foreach ((string key, JToken value) in fileSettings)
+                    saveSettings.TryAdd(key, value);
+            }
+        }
+        catch (Exception ex)
+        {
+            this.PrintWarning($"Couldn't parse your SMAPI settings file. Replacing it with a default version.\n\nTechnical details:\n{ex}");
+        }
+
+        // save file
+        jsonHelper.WriteJsonFile(filePath, saveSettings);
+    }
+
     /// <summary>Print a message without formatting.</summary>
     /// <param name="text">The text to print.</param>
     private void PrintPlain(string text)