remove DumpMetadata option

docs/release-notes.md

@@ -38,6 +38,7 @@
   * Added direct `Console` access to paranoid mode warnings.
   * Improved error messages for `TargetParameterCountException` when using the reflection API.
   * `helper.Read/WriteSaveData` can now be used while a save is being loaded (e.g. within a `Specialized.LoadStageChanged` event).
+  * Removed `DumpMetadata` option. It was only meant for specific debugging cases, but players would occasionally enable it incorrectly and then report crashes.
   * Fixed private textures loaded from content packs not having their `Name` field set.
 
 * For SMAPI developers:

src/SMAPI/Framework/Models/ModFolderExport.cs

@@ -1,21 +0,0 @@
-namespace StardewModdingAPI.Framework.Models
-{
-    /// <summary>Metadata exported to the mod folder.</summary>
-    internal class ModFolderExport
-    {
-        /// <summary>When the export was generated.</summary>
-        public string Exported { get; set; }
-
-        /// <summary>The absolute path of the mod folder.</summary>
-        public string ModFolderPath { get; set; }
-
-        /// <summary>The game version which last loaded the mods.</summary>
-        public string GameVersion { get; set; }
-
-        /// <summary>The SMAPI version which last loaded the mods.</summary>
-        public string ApiVersion { get; set; }
-
-        /// <summary>The detected mods.</summary>
-        public IModMetadata[] Mods { get; set; }
-    }
-}

src/SMAPI/Framework/Models/SConfig.cs

@@ -25,8 +25,7 @@ namespace StardewModdingAPI.Framework.Models
             [nameof(GitHubProjectName)] = "Pathoschild/SMAPI",
             [nameof(WebApiBaseUrl)] = "https://smapi.io/api/",
             [nameof(VerboseLogging)] = false,
-            [nameof(LogNetworkTraffic)] = false,
+            [nameof(LogNetworkTraffic)] = false
-            [nameof(DumpMetadata)] = false
         };
 
         /// <summary>The default values for <see cref="SuppressUpdateChecks"/>, to log changes if different.</summary>
@@ -64,9 +63,6 @@ namespace StardewModdingAPI.Framework.Models
         /// <summary>Whether SMAPI should log network traffic. Best combined with <see cref="VerboseLogging"/>, which includes network metadata.</summary>
         public bool LogNetworkTraffic { get; set; }
 
-        /// <summary>Whether to generate a file in the mods folder with detailed metadata about the detected mods.</summary>
-        public bool DumpMetadata { get; set; }
-
         /// <summary>The colors to use for text written to the SMAPI console.</summary>
         public ColorSchemeConfig ConsoleColors { get; set; }
 

src/SMAPI/Framework/SCore.cs

@@ -435,20 +435,6 @@ namespace StardewModdingAPI.Framework
                 mods = resolver.ProcessDependencies(mods, modDatabase).ToArray();
                 this.LoadMods(mods, this.Toolkit.JsonHelper, this.ContentCore, modDatabase);
 
-                // write metadata file
-                if (this.Settings.DumpMetadata)
-                {
-                    ModFolderExport export = new ModFolderExport
-                    {
-                        Exported = DateTime.UtcNow.ToString("O"),
-                        ApiVersion = Constants.ApiVersion.ToString(),
-                        GameVersion = Constants.GameVersion.ToString(),
-                        ModFolderPath = this.ModsPath,
-                        Mods = mods
-                    };
-                    this.Toolkit.JsonHelper.WriteJsonFile(Path.Combine(Constants.LogDir, $"{Constants.LogNamePrefix}metadata-dump.json"), export);
-                }
-
                 // check for updates
                 this.CheckForUpdatesAsync(mods);
             }

src/SMAPI/SMAPI.config.json

@@ -59,12 +59,6 @@ The default values are mirrored in StardewModdingAPI.Framework.Models.SConfig to
      */
     "LogNetworkTraffic": false,
 
-    /**
-     * Whether to generate a 'SMAPI-latest.metadata-dump.json' file in the logs folder with the full mod
-     * metadata for detected mods. This is only needed when troubleshooting some cases.
-     */
-    "DumpMetadata": false,
-
     /**
      * The colors to use for text written to the SMAPI console.
      *