add status reason details to past mods, clean out some old blacklist entries

1 files changed, 63 insertions, 120 deletions

diff --git a/src/SMAPI.Web/wwwroot/SMAPI.metadata.json b/src/SMAPI.Web/wwwroot/SMAPI.metadata.json
index b570a322..06762a3f 100644
--- a/src/SMAPI.Web/wwwroot/SMAPI.metadata.json
+++ b/src/SMAPI.Web/wwwroot/SMAPI.metadata.json
@@ -79,8 +79,7 @@
 
         "JSON Assets": {
             "ID": "spacechase0.JsonAssets",
-            "Default | UpdateKey": "Nexus:1720",
-            "1.3.1 | Status": "AssumeBroken" // causes runtime crashes
+            "Default | UpdateKey": "Nexus:1720"
         },
 
         "Mail Framework": {
@@ -90,8 +89,7 @@
 
         "MTN": {
             "ID": "SgtPickles.MTN",
-            "Default | UpdateKey": "Nexus:2256",
-            "~1.2.6 | Status": "AssumeBroken" // replaces Game1.multiplayer, which breaks SMAPI's multiplayer API.
+            "Default | UpdateKey": "Nexus:2256"
         },
 
         "PyTK": {
@@ -164,38 +162,45 @@
         *********/
         "Audio Devices": {
             "ID": "maxvollmer.audiodevices",
-            "~2.0.0 | Status": "AssumeBroken" // causes crash to desktop when starting the game
+            "~2.0.0 | Status": "AssumeBroken",
+            "~2.0.0 | StatusReasonDetails": "causes crash to desktop when starting the game"
         },
 
         "Custom Localization": {
             "ID": "ZaneYork.CustomLocalization",
             "FormerIDs": "SMAPI.CustomLocalization", // changed in 1.0.1
-            "~1.1 | Status": "AssumeBroken" // reflection error for _localizedAssets field
+            "~1.1 | Status": "AssumeBroken",
+            "~1.1 | StatusReasonDetails": "reflection error due to renamed _localizedAssets field"
         },
 
         "Mod Settings Tab": {
             "ID": "GilarF.ModSettingsTab",
-            "~0.2.1 | Status": "AssumeBroken" // fails extending title menu
+            "~0.2.1 | Status": "AssumeBroken",
+            "~0.2.1 | StatusReasonDetails": "fails extending title menu"
         },
 
         "More Grass": {
             "ID": "EpicBellyFlop45.MoreGrass",
-            "~1.0.8 | Status": "AssumeBroken" // crashes save load 
+            "~1.0.8 | Status": "AssumeBroken",
+            "~1.0.8 | StatusReasonDetails": "crashes on save load"
         },
 
         "Movement Speed": {
             "ID": "bcmpinc.MovementSpeed",
-            "~3.0.0 | Status": "AssumeBroken" // transpiler errors
+            "~3.0.0 | Status": "AssumeBroken",
+            "~3.0.0 | StatusReasonDetails": "broken due to transpiler errors"
         },
 
         "Tree Spread": {
             "ID": "bcmpinc.TreeSpread",
-            "~3.0.0 | Status": "AssumeBroken" // transpiler errors 
+            "~3.0.0 | Status": "AssumeBroken",
+            "~3.0.0 | StatusReasonDetails": "broken due to transpiler errors"
         },
 
         "TreeTransplant": {
             "ID": "TreeTransplant",
-            "~1.0.9 | Status": "AssumeBroken" // causes AccessViolationException which prevents game launch
+            "~1.0.9 | Status": "AssumeBroken",
+            "~3.0.0 | StatusReasonDetails": "breaks game launch due to AccessViolationException"
         },
 
         /*********
@@ -203,17 +208,14 @@
         *********/
         "Auto Quality Patch": {
             "ID": "SilentOak.AutoQualityPatch",
-            "~2.1.3-unofficial.7-mizzion | Status": "AssumeBroken" // runtime errors
+            "~2.1.3-unofficial.7-mizzion | Status": "AssumeBroken",
+            "~2.1.3-unofficial.7-mizzion | StatusReasonDetails": "broken due to runtime errors"
         },
 
         "Fix Dice": {
             "ID": "ashley.fixdice",
-            "~1.1.2 | Status": "AssumeBroken" // crashes game on startup
-        },
-
-        "Grass Growth": {
-            "ID": "bcmpinc.GrassGrowth",
-            "~1.0 | Status": "AssumeBroken"
+            "~1.1.2 | Status": "AssumeBroken",
+            "~1.1.2 | StatusReasonDetails": "crashes game on startup"
         },
 
         "Invite Code Mod": {
@@ -241,11 +243,6 @@
             "~2.15 | Status": "AssumeBroken"
         },
 
-        "Yet Another Harvest With Scythe Mod": {
-            "ID": "bcmpinc.HarvestWithScythe",
-            "~1.1 | Status": "AssumeBroken"
-        },
-
         /*********
         ** Broke in SMAPI 3.0 (runtime errors due to lifecycle changes)
         *********/
@@ -254,16 +251,6 @@
             "~1.0.0 | Status": "AssumeBroken"
         },
 
-        "Arcade 2048": {
-            "ID": "Platonymous.2048",
-            "~1.0.6 | Status": "AssumeBroken" // possibly due to PyTK
-        },
-
-        "Arcade Snake": {
-            "ID": "Platonymous.Snake",
-            "~1.1.0 | Status": "AssumeBroken" // possibly due to PyTK
-        },
-
         "Better Sprinklers": {
             "ID": "Speeder.BetterSprinklers",
             "~2.3.1-unofficial.7-pathoschild | Status": "AssumeBroken"
@@ -281,12 +268,8 @@
 
         "Decrafting Mod": {
             "ID": "MSCFC.DecraftingMod",
-            "~1.0 | Status": "AssumeBroken" // NRE in ModEntry
-        },
-
-        "JoJaBan - Arcade Sokoban": {
-            "ID": "Platonymous.JoJaBan",
-            "~0.4.3 | Status": "AssumeBroken" // possibly due to PyTK
+            "~1.0 | Status": "AssumeBroken",
+            "~1.0 | StatusReasonDetails": "fails due to NullReferenceException in ModEntry"
         },
 
         "Level Extender": {
@@ -304,11 +287,6 @@
             "~1.5 | Status": "AssumeBroken"
         },
 
-        "Seed Bag": {
-            "ID": "Platonymous.SeedBag",
-            "~1.2.7 | Status": "AssumeBroken" // possibly due to PyTK
-        },
-
         "Stardew Valley ESP": {
             "ID": "reimu.sdv-helper",
             "~1.1 | Status": "AssumeBroken"
@@ -316,12 +294,14 @@
 
         "Underdark Krobus": {
             "ID": "melnoelle.underdarkkrobus",
-            "~1.0.0 | Status": "AssumeBroken" // NRE in ModEntry
+            "~1.0 | Status": "AssumeBroken",
+            "~1.0 | StatusReasonDetails": "fails due to NullReferenceException in ModEntry"
         },
 
         "Underdark Sewer": {
             "ID": "melnoelle.underdarksewer",
-            "~1.1.0 | Status": "AssumeBroken" // NRE in ModEntry
+            "~1.1.0 | Status": "AssumeBroken",
+            "~1.1.0 | StatusReasonDetails": "fails due to NullReferenceException in ModEntry"
         },
 
         /*********
@@ -330,60 +310,70 @@
         "2cute FarmCave": {
             "ID": "taintedwheat.2CuteFarmCave",
             "Default | UpdateKey": "Nexus:843",
-            "~2.0 | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~2.0    | Status": "AssumeBroken",
+            "~2.0    | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Ace's Expanded Caves - Default Cave": {
             "ID": "Acerbicon.AECdefault",
             "Default | UpdateKey": "Nexus:2131",
-            "~1.2.2  | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~1.2.2  | Status": "AssumeBroken",
+            "~1.2.2  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Ace's Expanded Caves - Desert Cave": {
             "ID": "Acerbicon.AECdesert",
             "Default | UpdateKey": "Nexus:2131",
-            "~1.2.2  | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~1.2.2  | Status": "AssumeBroken",
+            "~1.2.2  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Ace's Expanded Caves - Ice Cave": {
             "ID": "Acerbicon.AECice",
             "Default | UpdateKey": "Nexus:2131",
-            "~1.2.2  | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~1.2.2  | Status": "AssumeBroken",
+            "~1.2.2  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Ace's Expanded Caves - Lava Cave": {
             "ID": "Acerbicon.AEClava",
             "Default | UpdateKey": "Nexus:2131",
-            "~1.2.2  | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~1.2.2  | Status": "AssumeBroken",
+            "~1.2.2  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Ace's Expanded Caves - Slime Cave": {
             "ID": "Acerbicon.AECslime",
             "Default | UpdateKey": "Nexus:2131",
-            "~1.2.2  | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~1.2.2  | Status": "AssumeBroken",
+            "~1.2.2  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Green Pastures Farm": {
             "ID": "bugbuddy.GreenPasturesFarm",
             "Default | UpdateKey": "Nexus:2326",
-            "~1.0    | Status": "AssumeBroken" // references deleted Content/weapons.xnb
+            "~1.0    | Status": "AssumeBroken",
+            "~1.0    | StatusReasonDetails": "references the deleted Content/weapons asset"
         },
 
         "Immersive Farm 2": {
             "ID": "zander.immersivefarm2",
-            "~2.0.1 | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~2.0.1 | Status": "AssumeBroken",
+            "~2.0.1 | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         "Karmylla's Immersive Map Edits": {
             "ID": "Karmylla.ImmersiveMapEdits",
             "Default | UpdateKey": "Nexus:1149",
-            "~2.4    | Status": "AssumeBroken" // references deleted Content/weapons.xnb
+            "~2.4    | Status": "AssumeBroken",
+            "~2.4    | StatusReasonDetails": "references the deleted Content/weapons asset"
         },
 
         "Secret Gardens Greenhouse": {
             "ID": "jessebot.secretgardens",
             "Default | UpdateKey": "Nexus:3067",
-            "~2.0.1 | Status": "AssumeBroken" // references deleted Content/Mine.xnb
+            "~2.0.1  | Status": "AssumeBroken",
+            "~2.0.1  | StatusReasonDetails": "references the deleted Content/Mine asset"
         },
 
         /*********
@@ -391,102 +381,55 @@
         *********/
         "Canon-Friendly Dialogue Expansion": {
             "ID": "gizzymo.canonfriendlyexpansion",
-            "~1.1.1 | Status": "AssumeBroken" // causes a save crash on certain dates
+            "~1.1.1 | Status": "AssumeBroken",
+            "~1.1.1 | StatusReasonDetails": "causes a save crash on certain dates"
         },
 
         "Everytime Submarine": {
             "ID": "MustafaDemirel.EverytimeSubmarine",
-            "~1.0.0 | Status": "AssumeBroken" // breaks player saves if their beach bridge is fixed
-        },
-
-        "Always Scroll Map": {
-            "ID": "bcmpinc.AlwaysScrollMap",
-            "~0.6 | Status": "AssumeBroken" // breaks newer versions of bcmpinc mods (per bcmpinc's request)
-        },
-
-        "Arcade Pong": {
-            "ID": "Platonymous.ArcadePong",
-            "~1.0.2 | Status": "AssumeBroken" // broke in SMAPI 2.6-beta.16 due to reflection into SMAPI internals
+            "~1.0.0 | Status": "AssumeBroken",
+            "~1.1.1 | StatusReasonDetails": "breaks player saves if their beach bridge is fixed"
         },
 
         "BJS Night Sounds": {
             "ID": "BunnyJumps.BJSNightSounds",
-            "~1.0.0 | Status": "AssumeBroken" // runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+
-        },
-
-        "Craft Counter": {
-            "ID": "bcmpinc.CraftCounter",
-            "~0.6 | Status": "AssumeBroken" // breaks newer versions of bcmpinc mods (per bcmpinc's request)
+            "~1.0.0 | Status": "AssumeBroken",
+            "~1.0.0 | StatusReasonDetails": "runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+"
         },
 
         "Fishing Adjust": {
             "ID": "shuaiz.FishingAdjustMod",
-            "~2.0.1 | Status": "AssumeBroken" // Method not found: 'Void Harmony.HarmonyInstance.Patch(System.Reflection.MethodBase, Harmony.HarmonyMethod, Harmony.HarmonyMethod, Harmony.HarmonyMethod)'
+            "~2.0.1 | Status": "AssumeBroken",
+            "~2.0.1 | StatusReasonDetails":  "fails with 'method not found' error for 'Void Harmony.HarmonyInstance.Patch(System.Reflection.MethodBase, Harmony.HarmonyMethod, Harmony.HarmonyMethod, Harmony.HarmonyMethod)'"
         },
 
         "Fishing Automaton": {
             "ID": "Drynwynn.FishingAutomaton",
-            "~1.1 | Status": "AssumeBroken" // runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+
-        },
-
-        "Fix Animal Tools": {
-            "ID": "bcmpinc.FixAnimalTools",
-            "~0.6 | Status": "AssumeBroken" // breaks newer versions of bcmpinc mods (per bcmpinc's request)
-        },
-
-        "Fix Scythe Exp": {
-            "ID": "bcmpinc.FixScytheExp",
-            "~0.3 | Status": "AssumeBroken" // broke in 1.3: Exception from HarmonyInstance "bcmpinc.FixScytheExp" [...] Bad label content in ILGenerator.
+            "~1.1 | Status": "AssumeBroken",
+            "~1.1 | StatusReasonDetails":  "runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+"
         },
 
         "More Silo Storage": {
             "ID": "OrneryWalrus.MoreSiloStorage",
-            "~1.0.1 | Status": "AssumeBroken" // broke in SDV 1.3
+            "~1.0.1 | Status": "AssumeBroken"
         },
 
         "No Added Flying Mine Monsters": {
             "ID": "Drynwynn.NoAddedFlyingMineMonsters",
-            "~1.1 | Status": "AssumeBroken" // runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+
+            "~1.1 | Status": "AssumeBroken",
+            "~1.1 | StatusReasonDetails": "runtime errors with Harmony 1.2.0.1 in SMAPI 2.8+"
         },
 
         "Server Bookmarker": {
             "ID": "Ilyaki.ServerBookmarker",
-            "~1.0.0 | Status": "AssumeBroken" // broke in Stardew Valley 1.3.29 (runtime errors)
-        },
-
-        "Skull Cave Saver": {
-            "ID": "cantorsdust.SkullCaveSaver",
-            "FormerIDs": "8ac06349-26f7-4394-806c-95d48fd35774 | community.SkullCaveSaver", // changed in 1.1 and 1.2.2
-            "1.3-beta | Status": "AssumeBroken" // doesn't work in multiplayer, no longer maintained
-        },
-
-        "Stardew Hack": {
-            "ID": "bcmpinc.StardewHack",
-            "~0.6 | Status": "AssumeBroken" // breaks newer versions of bcmpinc mods (per bcmpinc's request)
+            "~1.0.0 | Status": "AssumeBroken",
+            "~1.0.0 | StatusReasonDetails": "runtime errors in Stardew Valley 1.3.29"
         },
 
         "Stephan's Lots of Crops": {
             "ID": "stephansstardewcrops",
-            "~1.1    | Status": "AssumeBroken" // broke in SDV 1.3 (overwrites vanilla items)
-        },
-
-        "Summit Reborn": {
-            "ID": "KoihimeNakamura.summitreborn",
-            "FormerIDs": "emissaryofinfinity.summitreborn", // changed in 1.0.2
-            "~1.0.2 | Status": "AssumeBroken" // broke in SDV 1.3 (runtime errors)
-        },
-
-        "Tilled Soil Decay": {
-            "ID": "bcmpinc.TilledSoilDecay",
-            "~0.6 | Status": "AssumeBroken" // breaks newer versions of bcmpinc mods (per bcmpinc's request)
-        },
-
-        /*********
-        ** Broke circa SDV 1.2
-        *********/
-        "Move Faster": {
-            "ID": "shuaiz.MoveFasterMod",
-            "~1.0.1   | Status": "AssumeBroken" // doesn't do anything as of SDV 1.2.33 (bad Harmony patch?)
+            "~1.1 | Status": "AssumeBroken",
+            "~1.1 | StatusReasonDetails": "causes errors due to overwritten Stardew Valley 1.3 items"
         }
     }
 }