From 0f8d183ec71ca42fdbfef08f7e1edc7f822ef040 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 25 Feb 2018 14:24:25 -0500
Subject: bypass cache on log parser page

---
 src/SMAPI.Web/Views/LogParser/Index.cshtml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index be9f74a0..d9125954 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -17,10 +17,10 @@
 @using StardewModdingAPI.Web.Framework.LogParsing.Models
 @model StardewModdingAPI.Web.ViewModels.LogParserModel
 @section Head {
-    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=20180101" />
+    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=201802" />
     <script src="https://cdn.jsdelivr.net/npm/vue"></script>
     <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js" crossorigin="anonymous"></script>
-    <script src="~/Content/js/log-parser.js?r=20180101"></script>
+    <script src="~/Content/js/log-parser.js?r=201802"></script>
     <script>
         $(function() {
             smapi.logParser({
-- 
cgit 


From 212e85489a456acca6889faff5da835cbb707080 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 25 Feb 2018 23:27:44 -0500
Subject: fix log parser not correctly parsing mod list if a mod has no author
 name

---
 docs/release-notes.md                           | 4 ++++
 src/SMAPI.Common/SemanticVersionImpl.cs         | 5 ++++-
 src/SMAPI.Web/Framework/LogParsing/LogParser.cs | 4 +++-
 3 files changed, 11 insertions(+), 2 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 0da5220b..16284211 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,4 +1,8 @@
 # Release notes
+## 2.6 (upcoming)
+* For the [log parser][]:
+  * Fixed mod list not including all mods if at least one has no author name.
+
 ## 2.5.2
 * For modders:
   * Fixed issue where replacing an asset through `asset.AsImage()` or `asset.AsDictionary()` didn't take effect.
diff --git a/src/SMAPI.Common/SemanticVersionImpl.cs b/src/SMAPI.Common/SemanticVersionImpl.cs
index 1c713b47..084f56a3 100644
--- a/src/SMAPI.Common/SemanticVersionImpl.cs
+++ b/src/SMAPI.Common/SemanticVersionImpl.cs
@@ -22,6 +22,9 @@ namespace StardewModdingAPI.Common
         /// <summary>An optional prerelease tag.</summary>
         public string Tag { get; }
 
+        /// <summary>A regex pattern matching a version within a larger string.</summary>
+        internal const string UnboundedVersionPattern = @"(?>(?<major>0|[1-9]\d*))\.(?>(?<minor>0|[1-9]\d*))(?>(?:\.(?<patch>0|[1-9]\d*))?)(?:-(?<prerelease>(?>[a-z0-9]+[\-\.]?)+))?";
+
         /// <summary>A regular expression matching a semantic version string.</summary>
         /// <remarks>
         /// This pattern is derived from the BNF documentation in the <a href="https://github.com/mojombo/semver">semver repo</a>,
@@ -30,7 +33,7 @@ namespace StardewModdingAPI.Common
         /// - allows hyphens in prerelease tags as synonyms for dots (like "-unofficial-update.3");
         /// - doesn't allow '+build' suffixes.
         /// </remarks>
-        internal static readonly Regex Regex = new Regex(@"^(?>(?<major>0|[1-9]\d*))\.(?>(?<minor>0|[1-9]\d*))(?>(?:\.(?<patch>0|[1-9]\d*))?)(?:-(?<prerelease>(?>[a-z0-9]+[\-\.]?)+))?$", RegexOptions.CultureInvariant | RegexOptions.IgnoreCase | RegexOptions.Compiled | RegexOptions.ExplicitCapture);
+        internal static readonly Regex Regex = new Regex($@"^{SemanticVersionImpl.UnboundedVersionPattern}$", RegexOptions.CultureInvariant | RegexOptions.IgnoreCase | RegexOptions.Compiled | RegexOptions.ExplicitCapture);
 
         /*********
         ** Public methods
diff --git a/src/SMAPI.Web/Framework/LogParsing/LogParser.cs b/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
index 23a1baa4..385accf0 100644
--- a/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
+++ b/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
@@ -3,6 +3,7 @@ using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 using System.Text.RegularExpressions;
+using StardewModdingAPI.Common;
 using StardewModdingAPI.Web.Framework.LogParsing.Models;
 
 namespace StardewModdingAPI.Web.Framework.LogParsing
@@ -29,7 +30,8 @@ namespace StardewModdingAPI.Web.Framework.LogParsing
         private readonly Regex ModListStartPattern = new Regex(@"^Loaded \d+ mods:$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
 
         /// <summary>A regex pattern matching an entry in SMAPI's mod list.</summary>
-        private readonly Regex ModListEntryPattern = new Regex(@"^   (?<name>.+) (?<version>.+) by (?<author>.+) \| (?<description>.+)$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
+        /// <remarks>The author name and description are optional.</remarks>
+        private readonly Regex ModListEntryPattern = new Regex(@"^   (?<name>.+?) (?<version>" + SemanticVersionImpl.UnboundedVersionPattern + @")(?: by (?<author>[^\|]+))?(?: \| (?<description>.+))?$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
 
         /// <summary>A regex pattern matching the start of SMAPI's content pack list.</summary>
         private readonly Regex ContentPackListStartPattern = new Regex(@"^Loaded \d+ content packs:$", RegexOptions.Compiled | RegexOptions.IgnoreCase);
-- 
cgit 


From c984d5ad51c80a9ede1613c2bbbf51279966dd8b Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 25 Feb 2018 23:33:07 -0500
Subject: fix log filtering some mods incorrectly

---
 docs/release-notes.md                          |  1 +
 src/SMAPI.Web/Views/LogParser/Index.cshtml     | 12 ++++++------
 src/SMAPI.Web/wwwroot/Content/js/log-parser.js |  6 ++++++
 3 files changed, 13 insertions(+), 6 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 16284211..74db0256 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -2,6 +2,7 @@
 ## 2.6 (upcoming)
 * For the [log parser][]:
   * Fixed mod list not including all mods if at least one has no author name.
+  * Fixed some log entries being incorrectly filtered.
 
 ## 2.5.2
 * For modders:
diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index d9125954..39557d50 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -78,13 +78,13 @@
             <caption>
                 Installed mods:
                 <span class="notice txt"><i>click any mod to filter</i></span>
-                <span class="notice btn txt" v-on:click="showAllMods" v-if="stats.modsHidden > 0">show all</span>
-                <span class="notice btn txt" v-on:click="hideAllMods" v-if="stats.modsShown > 0 && stats.modsHidden > 0">hide all</span>
+                <span class="notice btn txt" v-on:click="showAllMods" v-show="stats.modsHidden > 0">show all</span>
+                <span class="notice btn txt" v-on:click="hideAllMods" v-show="stats.modsShown > 0 && stats.modsHidden > 0">hide all</span>
             </caption>
             @foreach (var mod in Model.ParsedLog.Mods.Where(p => p.ContentPackFor == null))
             {
                 <tr v-on:click="toggleMod('@GetSlug(mod.Name)')" class="mod-entry" v-bind:class="{ hidden: !showMods['@GetSlug(mod.Name)'] }">
-                    <td><input type="checkbox" v-bind:checked="showMods['@GetSlug(mod.Name)']" v-if="anyModsHidden" /></td>
+                    <td><input type="checkbox" v-bind:checked="showMods['@GetSlug(mod.Name)']" v-show="anyModsHidden" /></td>
                     <td>
                         @mod.Name
                         @if (contentPacks != null && contentPacks.TryGetValue(mod.Name, out LogModInfo[] contentPackList))
@@ -127,9 +127,9 @@
         <table id="log">
             @foreach (var message in Model.ParsedLog.Messages)
             {
-                string levelStr = @message.Level.ToString().ToLower();
+                string levelStr = message.Level.ToString().ToLower();
 
-                <tr class="@levelStr mod" v-if="showMods['@message.Mod'] && showLevels['@levelStr']">
+                <tr class="@levelStr mod" v-show="filtersAllow('@GetSlug(message.Mod)', '@levelStr')">
                     <td>@message.Time</td>
                     <td>@message.Level.ToString().ToUpper()</td>
                     <td data-title="@message.Mod">@message.Mod</td>
@@ -137,7 +137,7 @@
                 </tr>
                 if (message.Repeated > 0)
                 {
-                    <tr class="@levelStr mod mod-repeat" v-if="showMods['@message.Mod'] && showLevels['@levelStr']">
+                    <tr class="@levelStr mod mod-repeat" v-show="filtersAllow('@GetSlug(message.Mod)', '@levelStr')">
                         <td colspan="3"></td>
                         <td><i>repeats [@message.Repeated] times.</i></td>
                     </tr>
diff --git a/src/SMAPI.Web/wwwroot/Content/js/log-parser.js b/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
index 87a70391..38a75a80 100644
--- a/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
+++ b/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
@@ -62,6 +62,7 @@ smapi.logParser = function (data, sectionUrl) {
 
                 updateModFilters();
             },
+
             showAllMods: function () {
                 for (var key in this.showMods) {
                     if (this.showMods.hasOwnProperty(key)) {
@@ -70,6 +71,7 @@ smapi.logParser = function (data, sectionUrl) {
                 }
                 updateModFilters();
             },
+
             hideAllMods: function () {
                 for (var key in this.showMods) {
                     if (this.showMods.hasOwnProperty(key)) {
@@ -77,6 +79,10 @@ smapi.logParser = function (data, sectionUrl) {
                     }
                 }
                 updateModFilters();
+            },
+
+            filtersAllow: function(modId, level) {
+                return this.showMods[modId] !== false && this.showLevels[level] !== false;
             }
         }
     });
-- 
cgit 


From 5c1318431b716e13feb214abe8116cb1498047a5 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 25 Feb 2018 23:35:14 -0500
Subject: always include raw taxt in model

---
 src/SMAPI.Web/Framework/LogParsing/LogParser.cs        | 8 +++++---
 src/SMAPI.Web/Framework/LogParsing/Models/ParsedLog.cs | 4 ++--
 src/SMAPI.Web/Views/LogParser/Index.cshtml             | 2 +-
 3 files changed, 8 insertions(+), 6 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Framework/LogParsing/LogParser.cs b/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
index 385accf0..9e44f163 100644
--- a/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
+++ b/src/SMAPI.Web/Framework/LogParsing/LogParser.cs
@@ -55,6 +55,7 @@ namespace StardewModdingAPI.Web.Framework.LogParsing
                     return new ParsedLog
                     {
                         IsValid = false,
+                        RawText = logText,
                         Error = "The log is empty."
                     };
                 }
@@ -63,7 +64,8 @@ namespace StardewModdingAPI.Web.Framework.LogParsing
                 ParsedLog log = new ParsedLog
                 {
                     IsValid = true,
-                    Messages = this.CollapseRepeats(this.GetMessages(logText)).ToArray(),
+                    RawText = logText,
+                    Messages = this.CollapseRepeats(this.GetMessages(logText)).ToArray()
                 };
 
                 // parse log messages
@@ -154,7 +156,7 @@ namespace StardewModdingAPI.Web.Framework.LogParsing
                 {
                     IsValid = false,
                     Error = ex.Message,
-                    RawTextIfError = logText
+                    RawText = logText
                 };
             }
             catch (Exception ex)
@@ -163,7 +165,7 @@ namespace StardewModdingAPI.Web.Framework.LogParsing
                 {
                     IsValid = false,
                     Error = $"Parsing the log file failed. Technical details:\n{ex}",
-                    RawTextIfError = logText
+                    RawText = logText
                 };
             }
         }
diff --git a/src/SMAPI.Web/Framework/LogParsing/Models/ParsedLog.cs b/src/SMAPI.Web/Framework/LogParsing/Models/ParsedLog.cs
index 31ef2fe1..a82b6a1b 100644
--- a/src/SMAPI.Web/Framework/LogParsing/Models/ParsedLog.cs
+++ b/src/SMAPI.Web/Framework/LogParsing/Models/ParsedLog.cs
@@ -17,8 +17,8 @@ namespace StardewModdingAPI.Web.Framework.LogParsing.Models
         /// <summary>An error message indicating why the log file is invalid.</summary>
         public string Error { get; set; }
 
-        /// <summary>The raw text if <see cref="IsValid"/> is false.</summary>
-        public string RawTextIfError { get; set; }
+        /// <summary>The raw log text.</summary>
+        public string RawText { get; set; }
 
         /****
         ** Log data
diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index 39557d50..310277eb 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -155,7 +155,7 @@ else if (Model.ParsedLog?.IsValid == false)
     </div>
 
     <h3>Raw log</h3>
-    <pre>@Model.ParsedLog.RawTextIfError</pre>
+    <pre>@Model.ParsedLog.RawText</pre>
 }
 
 <div id="upload-area">
-- 
cgit 


From a82e0bb275f9088ba28f6b87daf1d13d1a5747a3 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Mon, 26 Feb 2018 00:28:32 -0500
Subject: bypass cache on log parser page

---
 src/SMAPI.Web/Views/LogParser/Index.cshtml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index 310277eb..35340aa5 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -17,10 +17,10 @@
 @using StardewModdingAPI.Web.Framework.LogParsing.Models
 @model StardewModdingAPI.Web.ViewModels.LogParserModel
 @section Head {
-    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=201802" />
+    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=20180225" />
     <script src="https://cdn.jsdelivr.net/npm/vue"></script>
     <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js" crossorigin="anonymous"></script>
-    <script src="~/Content/js/log-parser.js?r=201802"></script>
+    <script src="~/Content/js/log-parser.js?r=20180225"></script>
     <script>
         $(function() {
             smapi.logParser({
-- 
cgit 


From c7f2e53f209bce7fb06fd316c380bf6161689abd Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 27 Feb 2018 21:50:49 -0500
Subject: fix content pack list formatting

---
 src/SMAPI.Web/Views/LogParser/Index.cshtml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index 35340aa5..f622ccc4 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -92,7 +92,7 @@
                             <div class="content-packs">
                                 @foreach (var contentPack in contentPackList)
                                 {
-                                    <text>+@contentPack.Name @contentPack.Version</text>
+                                    <text>+ @contentPack.Name @contentPack.Version</text><br />
                                 }
                             </div>
                         }
-- 
cgit 


From 19570f43129dbd3dbfa77a9c62e135a72528a68e Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 01:07:55 -0500
Subject: simplify and always include default update URL, shorten
 no-longer-compatible skip messages

---
 docs/release-notes.md                         |   1 +
 src/SMAPI.Web/Startup.cs                      |   3 +-
 src/SMAPI/Framework/ModLoading/ModResolver.cs |   5 +-
 src/SMAPI/Program.cs                          |   5 +-
 src/SMAPI/StardewModdingAPI.config.json       | 201 +++++++++-----------------
 5 files changed, 77 insertions(+), 138 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index d20584af..fb056e79 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -2,6 +2,7 @@
 ## 2.5.3
 * For players:
   * Fixed some incompatible-mod errors not showing the mod URL.
+  * Simplified default mod update URL, which is now always included for incompatible mods.
   * Updated compatibility list.
 
 * For the [log parser][]:
diff --git a/src/SMAPI.Web/Startup.cs b/src/SMAPI.Web/Startup.cs
index bc3e9f7b..93135239 100644
--- a/src/SMAPI.Web/Startup.cs
+++ b/src/SMAPI.Web/Startup.cs
@@ -147,7 +147,8 @@ namespace StardewModdingAPI.Web
             ));
 
             // shortcut redirects
-            redirects.Add(new RedirectToUrlRule("^/docs$", "https://stardewvalleywiki.com/Modding:Index"));
+            redirects.Add(new RedirectToUrlRule(@"^/compat\.?$", "https://stardewvalleywiki.com/Modding:SMAPI_compatibility"));
+            redirects.Add(new RedirectToUrlRule(@"^/docs\.?$", "https://stardewvalleywiki.com/Modding:Index"));
 
             // redirect legacy canimod.com URLs
             var wikiRedirects = new Dictionary<string, string[]>
diff --git a/src/SMAPI/Framework/ModLoading/ModResolver.cs b/src/SMAPI/Framework/ModLoading/ModResolver.cs
index ba6dab1a..f878a1b9 100644
--- a/src/SMAPI/Framework/ModLoading/ModResolver.cs
+++ b/src/SMAPI/Framework/ModLoading/ModResolver.cs
@@ -98,7 +98,7 @@ namespace StardewModdingAPI.Framework.ModLoading
                     case ModStatus.AssumeBroken:
                         {
                             // get reason
-                            string reasonPhrase = mod.DataRecord.StatusReasonPhrase ?? "it's no longer compatible";
+                            string reasonPhrase = mod.DataRecord.StatusReasonPhrase ?? "it's outdated";
 
                             // get update URLs
                             List<string> updateUrls = new List<string>();
@@ -111,6 +111,9 @@ namespace StardewModdingAPI.Framework.ModLoading
                             if (mod.DataRecord.AlternativeUrl != null)
                                 updateUrls.Add(mod.DataRecord.AlternativeUrl);
 
+                            // default update URL
+                            updateUrls.Add("https://smapi.io/compat");
+
                             // build error
                             string error = $"{reasonPhrase}. Please check for a ";
                             if (mod.DataRecord.StatusUpperVersion == null || mod.Manifest.Version.Equals(mod.DataRecord.StatusUpperVersion))
diff --git a/src/SMAPI/Program.cs b/src/SMAPI/Program.cs
index 22bceafd..979f6328 100644
--- a/src/SMAPI/Program.cs
+++ b/src/SMAPI/Program.cs
@@ -747,8 +747,9 @@ namespace StardewModdingAPI
                     }
                     catch (IncompatibleInstructionException) // details already in trace logs
                     {
-                        string url = modDatabase.GetModPageUrlFor(metadata.Manifest.UniqueID);
-                        TrackSkip(metadata, $"it's no longer compatible. Please check for a newer version of the mod{(url != null ? $" at {url}" : "")}.");
+                        string[] updateUrls = new[] { modDatabase.GetModPageUrlFor(metadata.Manifest.UniqueID), "https://smapi.io/compat" }.Where(p => p != null).ToArray();
+
+                        TrackSkip(metadata, $"it's outdated. Please check for a new version at {string.Join(" or ", updateUrls)}.");
                         continue;
                     }
                     catch (SAssemblyLoadFailedException ex)
diff --git a/src/SMAPI/StardewModdingAPI.config.json b/src/SMAPI/StardewModdingAPI.config.json
index 101b9dac..437feea5 100644
--- a/src/SMAPI/StardewModdingAPI.config.json
+++ b/src/SMAPI/StardewModdingAPI.config.json
@@ -97,15 +97,13 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "AccessChestAnywhere",
       "MapLocalVersions": { "1.1-1078": "1.1" },
       "Default | UpdateKey": "Nexus:257",
-      "~1.1    | Status": "AssumeBroken",
-      "~1.1    | AlternativeUrl": "https://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1    | Status": "AssumeBroken"
     },
 
     "AdjustArtisanPrices": {
       "ID": "1e36d4ca-c7ef-4dfb-9927-d27a6c3c8bdc",
       "Default | UpdateKey": "Chucklefish:3532",
-      "~0.1    | Status": "AssumeBroken",
-      "~0.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.1    | Status": "AssumeBroken"
     },
 
     "Adjust Monster": {
@@ -127,8 +125,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "AgingMod": {
       "ID": "skn.AgingMod",
       "Default | UpdateKey": "Nexus:1129",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "All Crops All Seasons": {
@@ -173,8 +170,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "A Tapper's Dream": {
       "ID": "ddde5195-8f85-4061-90cc-0d4fd5459358",
       "Default | UpdateKey": "Nexus:260",
-      "~1.4    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.4    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.4    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Auto Animal Doors": {
@@ -247,16 +243,14 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Kithio:BetterShippingBox",
       "MapLocalVersions": { "1.0.1": "1.0.2" },
       "Default | UpdateKey": "Chucklefish:4302",
-      "~1.0.2  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Better Sprinklers": {
       "ID": "Speeder.BetterSprinklers",
       "FormerIDs": "SPDSprinklersMod", // changed in 2.3
       "Default                   | UpdateKey": "Nexus:41",
-      "~2.3.1-pathoschild-update | Status": "AssumeBroken", // broke in SDV 1.2
-      "~2.3.1-pathoschild-update | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~2.3.1-pathoschild-update | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Billboard Anywhere": {
@@ -269,8 +263,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "KathrynHazuka.BirthdayMail",
       "FormerIDs": "005e02dc-d900-425c-9c68-1ff55c5a295d", // changed in 1.2.3-pathoschild-update
       "Default | UpdateKey": "Nexus:276",
-      "~1.2.2  | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.2.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.2.2  | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Breed Like Rabbits": {
@@ -331,8 +324,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Speeder.ChestLabel",
       "FormerIDs": "SPDChestLabel", // changed in 1.5.1-pathoschild-update
       "Default | UpdateKey": "Nexus:242",
-      "~1.6    | Status": "AssumeBroken", // broke in SDV 1.1
-      "~1.6    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.6    | Status": "AssumeBroken" // broke in SDV 1.1
     },
 
     "Chest Pooling": {
@@ -352,8 +344,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Choose Baby Gender": {
       "FormerIDs": "{EntryDll: 'ChooseBabyGender.dll'}",
       "Default | UpdateKey": "Nexus:590",
-      "~1.0.2  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "CJB Automation": {
@@ -398,8 +389,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Cold Weather Haley": {
       "ID": "LordXamon.ColdWeatherHaleyPRO",
       "Default | UpdateKey": "Nexus:1169",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Colored Chests": {
@@ -411,8 +401,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Combat with Farm Implements": {
       "ID": "SPDFarmingImplementsInCombat",
       "Default | UpdateKey": "Nexus:313",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Community Bundle Item Tooltip": {
@@ -434,8 +423,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Configurable Shipping Dates": {
       "ID": "ConfigurableShippingDates",
       "Default | UpdateKey": "Nexus:675",
-      "~1.1.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Cooking Skill": {
@@ -515,8 +503,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Customizable Traveling Cart Days": {
       "ID": "TravelingCartYyeahdude",
       "Default | UpdateKey": "Nexus:567",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Custom Linens": {
@@ -566,8 +553,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Dynamic Checklist": {
       "ID": "gunnargolf.DynamicChecklist",
       "Default | UpdateKey": "Nexus:1145", // added in 1.0.1-pathoschild-update
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Dynamic Horses": {
@@ -580,8 +566,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "DynamicMachines",
       "MapLocalVersions": { "1.1": "1.1.1" },
       "Default | UpdateKey": "Nexus:374",
-      "~1.1.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Dynamic NPC Sprites": {
@@ -597,16 +582,14 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Empty Hands": {
       "ID": "QuicksilverFox.EmptyHands",
       "Default | UpdateKey": "Nexus:1176", // added in 1.0.1-pathoschild-update
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Enemy Health Bars": {
       "ID": "Speeder.HealthBars",
       "FormerIDs": "SPDHealthBar", // changed in 1.7.1-pathoschild-update
       "Default | UpdateKey": "Nexus:193",
-      "~1.7    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.7    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.7    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Entoarox Framework": {
@@ -636,15 +619,13 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Crystalmir.ExtendedFridge",
       "FormerIDs": "Mystra007ExtendedFridge", // changed in 1.0.1
       "Default | UpdateKey": "Nexus:485",
-      "~1.0    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Extended Greenhouse": {
       "ID": "ExtendedGreenhouse",
       "Default | UpdateKey": "Chucklefish:4303",
-      "~1.0.2  | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2  | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Extended Minecart": {
@@ -667,35 +648,30 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Farm Automation: Barn Door Automation": {
       "FormerIDs": "{EntryDll: 'FarmAutomation.BarnDoorAutomation.dll'}",
-      "~1.0 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Farm Automation: Item Collector": {
       "FormerIDs": "{EntryDll: 'FarmAutomation.ItemCollector.dll'}",
-      "~1.0 | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0 | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Farm Automation Unofficial: Item Collector": {
       "ID": "Maddy99.FarmAutomation.ItemCollector",
-      "~0.5 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~0.5 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.5 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Farm Expansion": {
       "ID": "Advize.FarmExpansion",
       "FormerIDs": "3888bdfd-73f6-4776-8bb7-8ad45aea1915 | AdvizeFarmExpansionMod-2-0 | AdvizeFarmExpansionMod-2-0-5", // changed in 2.0, 2.0.5, and 3.0
       "Default | UpdateKey": "Nexus:130",
-      "~2.0.5  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~2.0.5  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~2.0.5  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Farm Resource Generator": {
       "FormerIDs": "{EntryDll: 'FarmResourceGenerator.dll'}",
       "Default | UpdateKey": "Nexus:647",
-      "~1.0.4  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.4  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.4  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Fast Animations": {
@@ -718,8 +694,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "KathrynHazuka.FasterRun",
       "FormerIDs": "{EntryDll: 'FasterRun.dll'}", // changed in 1.1.1-pathoschild-update
       "Default | UpdateKey": "Nexus:733", // added in 1.1.1-pathoschild-update
-      "~1.1    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Fishing Adjust": {
@@ -741,8 +716,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "FormerIDs": "{EntryDll: 'FlorenceMod.dll'}",
       "MapLocalVersions": { "1.0.1": "1.1" },
       "Default | UpdateKey": "Nexus:591",
-      "~1.1    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Flower Color Picker": {
@@ -753,8 +727,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Forage at the Farm": {
       "ID": "ForageAtTheFarm",
       "Default | UpdateKey": "Nexus:673",
-      "~1.5.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.5.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.5.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Furniture Anywhere": {
@@ -812,8 +785,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Happy Animals": {
       "ID": "HappyAnimals",
-      "~1.0.3 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.3 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.3 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Happy Birthday (Omegasis)": {
@@ -841,8 +813,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Harvest With Scythe": {
       "ID": "965169fd-e1ed-47d0-9f12-b104535fb4bc",
       "Default | UpdateKey": "Nexus:236",
-      "~1.0.6  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.6  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.6  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Horse Whistle (icepuente)": {
@@ -858,8 +829,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Hunger for Food (Tigerle)": {
       "ID": "HungerForFoodByTigerle",
       "Default | UpdateKey": "Nexus:810",
-      "~0.1.2  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~0.1.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.1.2  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Hunger Mod (skn)": {
@@ -882,8 +852,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Instant Geode": {
       "ID": "InstantGeode",
-      "~1.12 | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.12 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.12 | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Instant Grow Trees": {
@@ -895,8 +864,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Interaction Helper": {
       "ID": "HammurabiInteractionHelper",
       "Default | UpdateKey": "Chucklefish:4640", // added in 1.0.4-pathoschild-update
-      "~1.0.3  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.3  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.3  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Item Auto Stacker": {
@@ -926,8 +894,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "BALANCEMOD_AntiExhaustion",
       "MapLocalVersions": { "0.0": "1.1" },
       "Default | UpdateKey": "Nexus:637",
-      "~1.1 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Level Extender": {
@@ -1005,8 +972,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Message Box [API]? (ChatMod)": {
       "ID": "Kithio:ChatMod",
       "Default | UpdateKey": "Chucklefish:4296",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Mining at the Farm": {
@@ -1040,8 +1006,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "More Artifact Spots": {
       "ID": "451",
       "Default | UpdateKey": "Nexus:451",
-      "~1.0.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "More Map Layers": {
@@ -1070,8 +1035,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "FileLoading",
       "MapLocalVersions": { "1.1": "1.12" },
       "Default | UpdateKey": "Nexus:1094",
-      "~1.12   | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.12   | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.12   | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Museum Rearranger": {
@@ -1089,8 +1053,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "New Machines": {
       "ID": "F70D4FAB-0AB2-4B78-9F1B-AF2CA2236A59",
       "Default   | UpdateKey": "Chucklefish:3683",
-      "~4.2.1343 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~4.2.1343 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~4.2.1343 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Night Owl": {
@@ -1131,8 +1094,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "No Soil Decay": {
       "ID": "289dee03-5f38-4d8e-8ffc-e440198e8610",
       "Default | UpdateKey": "Nexus:237",
-      "~0.5    | Status": "AssumeBroken", // broke in SDV 1.2 and uses Assembly.GetExecutingAssembly().Location
-      "~0.5    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.5    | Status": "AssumeBroken" // broke in SDV 1.2 and uses Assembly.GetExecutingAssembly().Location
     },
 
     "No Soil Decay Redux": {
@@ -1150,8 +1112,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "NPC Speak": {
       "FormerIDs": "{EntryDll: 'NpcEcho.dll'}",
       "Default | UpdateKey": "Nexus:694",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Object Time Left": {
@@ -1188,8 +1149,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "PelicanTTS": {
       "ID": "Platonymous.PelicanTTS",
       "Default | UpdateKey": "Nexus:1079", // added in 1.6.1
-      "~1.6    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.6    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.6    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Persia the Mermaid - Standalone Custom NPC": {
@@ -1205,8 +1165,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Persival's BundleMod": {
       "FormerIDs": "{EntryDll: 'BundleMod.dll'}",
       "Default | UpdateKey": "Nexus:438",
-      "~1.0    | Status": "AssumeBroken", // broke in SDV 1.1
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SDV 1.1
     },
 
     "Plant on Grass": {
@@ -1240,8 +1199,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Mucchan.PrairieKingMadeEasy",
       "FormerIDs": "{EntryDll: 'PrairieKingMadeEasy.dll'}", // changed in 1.0.1
       "Default | UpdateKey": "Chucklefish:3594",
-      "~1.0    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Quest Delay": {
@@ -1251,8 +1209,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Rain Randomizer": {
       "FormerIDs": "{EntryDll: 'RainRandomizer.dll'}",
-      "~1.0.3 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.3 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.3 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Recatch Legendary Fish": {
@@ -1274,16 +1231,14 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "RelationshipsEnhanced": {
       "ID": "relationshipsenhanced",
       "Default | UpdateKey": "Chucklefish:4435",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Relationship Status": {
       "ID": "relationshipstatus",
       "MapRemoteVersions": { "1.0.5": "1.0.4" }, // not updated in manifest
       "Default | UpdateKey": "Nexus:751",
-      "~1.0.5  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.5  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.5  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Rented Tools": {
@@ -1312,8 +1267,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Reusable Wallpapers and Floors (Wallpaper Retain)": {
       "ID": "dae1b553-2e39-43e7-8400-c7c5c836134b",
       "Default | UpdateKey": "Nexus:356",
-      "~1.5    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.5    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.5    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Ring of Fire": {
@@ -1395,16 +1349,14 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Shed Notifications (BuildingsNotifications)": {
       "ID": "TheCroak.BuildingsNotifications",
       "Default | UpdateKey": "Nexus:620",
-      "~0.4.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~0.4.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.4.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Shenandoah Project": {
       "ID": "Shenandoah Project",
       "MapRemoteVersions": { "1.1.1": "1.1" }, // not updated in manifest
       "Default | UpdateKey": "Nexus:756",
-      "~1.1.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Ship Anywhere": {
@@ -1415,8 +1367,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Shipment Tracker": {
       "ID": "7e474181-e1a0-40f9-9c11-d08a3dcefaf3",
       "Default | UpdateKey": "Nexus:321",
-      "~1.1    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Shop Expander": {
@@ -1430,8 +1381,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Igorious.Showcase",
       "MapLocalVersions": { "0.9-500": "0.9" },
       "Default | UpdateKey": "Chucklefish:4487",
-      "~0.9    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~0.9    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.9    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Shroom Spotter": {
@@ -1461,8 +1411,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "6266959802",
       "MapLocalVersions": { "0.0": "1.4" },
       "Default | UpdateKey": "Nexus:366",
-      "~1.2.2  | Status": "AssumeBroken", // broke in SMAPI 1.9 (has multiple Mod instances)
-      "~1.2.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.2.2  | Status": "AssumeBroken" // broke in SMAPI 1.9 (has multiple Mod instances)
     },
 
     "Skill Prestige": {
@@ -1506,14 +1455,12 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Speeder.SlowerFenceDecay",
       "FormerIDs": "SPDSlowFenceDecay", // changed in 0.5.2-pathoschild-update
       "Default | UpdateKey": "Nexus:252",
-      "~0.5.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~0.5.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~0.5.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Smart Mod": {
       "ID": "KuroBear.SmartMod",
-      "~2.2 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~2.2 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~2.2 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Solar Eclipse Event": {
@@ -1542,15 +1489,13 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Sprinkles": {
       "ID": "Platonymous.Sprinkles",
       "Default | UpdateKey": "Chucklefish:4592",
-      "~1.1.3  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1.3  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1.3  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Sprint and Dash": {
       "ID": "SPDSprintAndDash",
       "Default | UpdateKey": "Chucklefish:3531",
-      "~1.0    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Sprint and Dash Redux": {
@@ -1563,8 +1508,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "a10d3097-b073-4185-98ba-76b586cba00c",
       "MapLocalVersions": { "1.0": "2.1" }, // not updated in manifest
       "Default | UpdateKey": "GitHub:oliverpl/SprintingMod",
-      "~2.1    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~2.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~2.1    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "StackSplitX": {
@@ -1576,8 +1520,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "StaminaRegen": {
       "FormerIDs": "{EntryDll: 'StaminaRegen.dll'}",
-      "~1.0.3 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.3 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.3 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Stardew Config Menu": {
@@ -1599,8 +1542,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Stardew Notification": {
       "ID": "stardewnotification",
       "Default | UpdateKey": "GitHub:monopandora/StardewNotification",
-      "~1.7    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.7    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.7    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Stardew Symphony": {
@@ -1625,8 +1567,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "StashItemsToChest": {
       "ID": "BlueMod_StashItemsToChest",
       "Default | UpdateKey": "GitHub:lambui/StardewValleyMod_StashItemsToChest",
-      "~1.0.1  | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.1  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.1  | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Stephan's Lots of Crops": {
@@ -1650,8 +1591,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Super Greenhouse Warp Modifier": {
       "ID": "SuperGreenhouse",
       "Default | UpdateKey": "Chucklefish:4334",
-      "~1.0    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Swim Almost Anywhere / Swim Suit": {
@@ -1661,8 +1601,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Tainted Cellar": {
       "FormerIDs": "{EntryDll: 'TaintedCellar.dll'}",
-      "~1.0 | Status": "AssumeBroken", // broke in SDV 1.1 or 1.11
-      "~1.0 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0 | Status": "AssumeBroken" // broke in SDV 1.1 or 1.11
     },
 
     "Tapper Ready": {
@@ -1678,8 +1617,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
     "Teleporter": {
       "ID": "Teleporter",
       "Default | UpdateKey": "Chucklefish:4374",
-      "~1.0.2  | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0.2  | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2  | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "The Long Night": {
@@ -1738,8 +1676,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "ID": "Demiacle.UiModSuite",
       "MapLocalVersions": { "0.5": "1.0" }, // not updated in manifest
       "Default | UpdateKey": "Nexus:1023",
-      "~1.0    | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0    | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "Variable Grass": {
@@ -1754,15 +1691,13 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "WakeUp": {
       "FormerIDs": "{EntryDll: 'WakeUp.dll'}",
-      "~1.0.2 | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.0.2 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2 | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "Wallpaper Fix": {
       "FormerIDs": "{EntryDll: 'WallpaperFix.dll'}",
       "Default | UpdateKey": "Chucklefish:4211",
-      "~1.1    | Status": "AssumeBroken", // broke in SMAPI 2.0
-      "~1.1    | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.1    | Status": "AssumeBroken" // broke in SMAPI 2.0
     },
 
     "WarpAnimals": {
@@ -1772,8 +1707,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Weather Controller": {
       "FormerIDs": "{EntryDll: 'WeatherController.dll'}",
-      "~1.0.2 | Status": "AssumeBroken", // broke in SDV 1.2
-      "~1.0.2 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0.2 | Status": "AssumeBroken" // broke in SDV 1.2
     },
 
     "What Farm Cave / WhatAMush": {
@@ -1788,8 +1722,7 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
 
     "Wonderful Farm Life": {
       "FormerIDs": "{EntryDll: 'WonderfulFarmLife.dll'}",
-      "~1.0 | Status": "AssumeBroken", // broke in SDV 1.1 or 1.11
-      "~1.0 | AlternativeUrl": "http://stardewvalleywiki.com/Modding:SMAPI_2.0"
+      "~1.0 | Status": "AssumeBroken" // broke in SDV 1.1 or 1.11
     },
 
     "XmlSerializerRetool": {
-- 
cgit 


From ac6127c63e2978a5cf63ff4753991e5ab33db8c3 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 13:37:42 -0500
Subject: fix log parser error when mod names are duplicated

---
 docs/release-notes.md                      | 1 +
 src/SMAPI.Web/Views/LogParser/Index.cshtml | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index fb056e79..e1c262be 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -8,6 +8,7 @@
 * For the [log parser][]:
   * Fixed mod list not including all mods if at least one has no author name.
   * Fixed some log entries being incorrectly filtered.
+  * Fixed error when mods have duplicate names.
 
 * For SMAPI developers:
   * Internal changes to support the upcoming Stardew Valley 1.3 update.
diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index f622ccc4..d2d8004e 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -26,7 +26,7 @@
             smapi.logParser({
                 logStarted: new Date(@Json.Serialize(Model.ParsedLog?.Timestamp)),
                 showPopup: @Json.Serialize(Model.ParsedLog == null),
-                showMods: @Json.Serialize(Model.ParsedLog?.Mods?.ToDictionary(p => GetSlug(p.Name), p => true), new JsonSerializerSettings { Formatting = Formatting.None }),
+                showMods: @Json.Serialize(Model.ParsedLog?.Mods?.Select(p => GetSlug(p.Name)).Distinct().ToDictionary(slug => slug, slug => true), new JsonSerializerSettings { Formatting = Formatting.None }),
                 showLevels: {
                     trace: false,
                     debug: false,
-- 
cgit 


From cea113eb2908105c5b5d5ed86a8221e95ea92fb4 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 17:32:29 -0500
Subject: fetch mod info from Nexus website until we can use their API again
 (#454)

---
 .../Framework/Clients/Nexus/NexusClient.cs         |  2 +-
 src/SMAPI.Web/Framework/Clients/Nexus/NexusMod.cs  |  4 +
 .../Clients/Nexus/NexusWebScrapeClient.cs          | 93 ++++++++++++++++++++++
 .../Framework/ModRepositories/NexusRepository.cs   |  8 +-
 src/SMAPI.Web/Startup.cs                           |  9 ++-
 5 files changed, 110 insertions(+), 6 deletions(-)
 create mode 100644 src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Framework/Clients/Nexus/NexusClient.cs b/src/SMAPI.Web/Framework/Clients/Nexus/NexusClient.cs
index 1a7011e2..adec41be 100644
--- a/src/SMAPI.Web/Framework/Clients/Nexus/NexusClient.cs
+++ b/src/SMAPI.Web/Framework/Clients/Nexus/NexusClient.cs
@@ -3,7 +3,7 @@ using Pathoschild.Http.Client;
 
 namespace StardewModdingAPI.Web.Framework.Clients.Nexus
 {
-    /// <summary>An HTTP client for fetching mod metadata from Nexus Mods.</summary>
+    /// <summary>An HTTP client for fetching mod metadata from the Nexus Mods API.</summary>
     internal class NexusClient : INexusClient
     {
         /*********
diff --git a/src/SMAPI.Web/Framework/Clients/Nexus/NexusMod.cs b/src/SMAPI.Web/Framework/Clients/Nexus/NexusMod.cs
index 2b04104f..cd52c72b 100644
--- a/src/SMAPI.Web/Framework/Clients/Nexus/NexusMod.cs
+++ b/src/SMAPI.Web/Framework/Clients/Nexus/NexusMod.cs
@@ -17,5 +17,9 @@ namespace StardewModdingAPI.Web.Framework.Clients.Nexus
         /// <summary>The mod's web URL.</summary>
         [JsonProperty("mod_page_uri")]
         public string Url { get; set; }
+
+        /// <summary>A user-friendly error which indicates why fetching the mod info failed (if applicable).</summary>
+        [JsonIgnore]
+        public string Error { get; set; }
     }
 }
diff --git a/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs b/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs
new file mode 100644
index 00000000..1e941e3d
--- /dev/null
+++ b/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs
@@ -0,0 +1,93 @@
+using System;
+using System.Net;
+using System.Threading.Tasks;
+using HtmlAgilityPack;
+using Pathoschild.Http.Client;
+
+namespace StardewModdingAPI.Web.Framework.Clients.Nexus
+{
+    /// <summary>An HTTP client for fetching mod metadata from the Nexus website.</summary>
+    internal class NexusWebScrapeClient : INexusClient
+    {
+        /*********
+        ** Properties
+        *********/
+        /// <summary>The URL for a Nexus web page excluding the base URL, where {0} is the mod ID.</summary>
+        private readonly string ModUrlFormat;
+
+        /// <summary>The underlying HTTP client.</summary>
+        private readonly IClient Client;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Construct an instance.</summary>
+        /// <param name="userAgent">The user agent for the Nexus Mods API client.</param>
+        /// <param name="baseUrl">The base URL for the Nexus Mods site.</param>
+        /// <param name="modUrlFormat">The URL for a Nexus Mods web page excluding the <paramref name="baseUrl"/>, where {0} is the mod ID.</param>
+        public NexusWebScrapeClient(string userAgent, string baseUrl, string modUrlFormat)
+        {
+            this.ModUrlFormat = modUrlFormat;
+            this.Client = new FluentClient(baseUrl).SetUserAgent(userAgent);
+        }
+
+        /// <summary>Get metadata about a mod.</summary>
+        /// <param name="id">The Nexus mod ID.</param>
+        /// <returns>Returns the mod info if found, else <c>null</c>.</returns>
+        public async Task<NexusMod> GetModAsync(uint id)
+        {
+            // fetch HTML
+            string html;
+            try
+            {
+                html = await this.Client
+                    .GetAsync(string.Format(this.ModUrlFormat, id))
+                    .AsString();
+            }
+            catch (ApiException ex) when (ex.Status == HttpStatusCode.NotFound)
+            {
+                return null;
+            }
+
+            // parse HTML
+            var doc = new HtmlDocument();
+            doc.LoadHtml(html);
+
+            // handle Nexus error message
+            HtmlNode node = doc.DocumentNode.SelectSingleNode("//div[contains(@class, 'site-notice')][contains(@class, 'warning')]");
+            if (node != null)
+            {
+                string[] errorParts = node.InnerText.Trim().Split(new[] { '\n' }, 2, System.StringSplitOptions.RemoveEmptyEntries);
+                string errorCode = errorParts[0];
+                string errorText = errorParts.Length > 1 ? errorParts[1] : null;
+                switch (errorCode.Trim().ToLower())
+                {
+                    case "not found":
+                        return null;
+
+                    default:
+                        return new NexusMod { Error = $"Nexus error: {errorCode} ({errorText})." };
+                }
+            }
+
+            // extract mod info
+            string url = new UriBuilder(this.Client.BaseClient.BaseAddress) { Path = string.Format(this.ModUrlFormat, id) }.Uri.ToString();
+            string name = doc.DocumentNode.SelectSingleNode("//h1")?.InnerText.Trim();
+            string version = doc.DocumentNode.SelectSingleNode("//ul[contains(@class, 'stats')]//li[@class='stat-version']//div[@class='stat']")?.InnerText.Trim();
+
+            return new NexusMod
+            {
+                Name = name,
+                Version = version,
+                Url = url
+            };
+        }
+
+        /// <summary>Performs application-defined tasks associated with freeing, releasing, or resetting unmanaged resources.</summary>
+        public void Dispose()
+        {
+            this.Client?.Dispose();
+        }
+    }
+}
diff --git a/src/SMAPI.Web/Framework/ModRepositories/NexusRepository.cs b/src/SMAPI.Web/Framework/ModRepositories/NexusRepository.cs
index cfa757ab..e1dc0fcc 100644
--- a/src/SMAPI.Web/Framework/ModRepositories/NexusRepository.cs
+++ b/src/SMAPI.Web/Framework/ModRepositories/NexusRepository.cs
@@ -39,9 +39,11 @@ namespace StardewModdingAPI.Web.Framework.ModRepositories
             try
             {
                 NexusMod mod = await this.Client.GetModAsync(nexusID);
-                return mod != null
-                    ? new ModInfoModel(mod.Name, this.NormaliseVersion(mod.Version), mod.Url)
-                    : new ModInfoModel("Found no mod with this ID.");
+                if (mod == null)
+                    return new ModInfoModel("Found no mod with this ID.");
+                if (mod.Error != null)
+                    return new ModInfoModel(mod.Error);
+                return new ModInfoModel(mod.Name, this.NormaliseVersion(mod.Version), mod.Url);
             }
             catch (Exception ex)
             {
diff --git a/src/SMAPI.Web/Startup.cs b/src/SMAPI.Web/Startup.cs
index 93135239..d7d4d074 100644
--- a/src/SMAPI.Web/Startup.cs
+++ b/src/SMAPI.Web/Startup.cs
@@ -81,8 +81,13 @@ namespace StardewModdingAPI.Web
                     password: api.GitHubPassword
                 ));
 
-                services.AddSingleton<INexusClient>(new NexusClient(
-                    userAgent: api.NexusUserAgent,
+                //services.AddSingleton<INexusClient>(new NexusClient(
+                //    userAgent: api.NexusUserAgent,
+                //    baseUrl: api.NexusBaseUrl,
+                //    modUrlFormat: api.NexusModUrlFormat
+                //));
+                services.AddSingleton<INexusClient>(new NexusWebScrapeClient(
+                    userAgent: userAgent,
                     baseUrl: api.NexusBaseUrl,
                     modUrlFormat: api.NexusModUrlFormat
                 ));
-- 
cgit 


From 90c8593ba9a0828a4d3f563c6f08fc90933cd2ff Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 17:34:43 -0500
Subject: update SMAPI URL in user agent (#454)

---
 docs/release-notes.md          | 1 +
 src/SMAPI.Web/appsettings.json | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 37d3c7aa..93d07385 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -5,6 +5,7 @@
   * Updated compatibility list.
   * Fixed some incompatible-mod errors not showing the mod URL.
   * Fixed rare crash with a specific combination of `manifest.json` fields and internal SMAPI mod data.
+  * Fixed update checks not working for Nexus Mods due to a change in their API.
 
 * For the [log parser][]:
   * Fixed mod list not including all mods if at least one has no author name.
diff --git a/src/SMAPI.Web/appsettings.json b/src/SMAPI.Web/appsettings.json
index 9758f4a7..3cf72ddb 100644
--- a/src/SMAPI.Web/appsettings.json
+++ b/src/SMAPI.Web/appsettings.json
@@ -18,7 +18,7 @@
     "LogParserUrl": null // see top note
   },
   "ApiClients": {
-    "UserAgent": "SMAPI/{0} (+https://github.com/Pathoschild/SMAPI)",
+    "UserAgent": "SMAPI/{0} (+https://smapi.io)",
 
     "ChucklefishBaseUrl": "https://community.playstarbound.com",
     "ChucklefishModPageUrlFormat": "resources/{0}",
-- 
cgit 


From 99023f94871e1f9bad5ee5f5db0ff041a02e9ed5 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 18:46:05 -0500
Subject: add support for mapping non-semantic remote mod versions

---
 docs/release-notes.md                              |  1 +
 src/SMAPI.Common/Models/ModSeachModel.cs           |  7 ++++-
 src/SMAPI.Web/Controllers/ModsApiController.cs     | 24 ++++++++++++-----
 src/SMAPI/Framework/ModData/ModDataRecord.cs       | 11 +++++---
 src/SMAPI/Framework/ModData/ParsedModDataRecord.cs |  2 +-
 src/SMAPI/Framework/WebApiClient.cs                |  2 +-
 src/SMAPI/Program.cs                               | 31 +++++++++++-----------
 7 files changed, 50 insertions(+), 28 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 93d07385..903eaf6f 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -6,6 +6,7 @@
   * Fixed some incompatible-mod errors not showing the mod URL.
   * Fixed rare crash with a specific combination of `manifest.json` fields and internal SMAPI mod data.
   * Fixed update checks not working for Nexus Mods due to a change in their API.
+  * Fixed update checks failing for some older mods with non-standard versions.
 
 * For the [log parser][]:
   * Fixed mod list not including all mods if at least one has no author name.
diff --git a/src/SMAPI.Common/Models/ModSeachModel.cs b/src/SMAPI.Common/Models/ModSeachModel.cs
index 13b05d2d..3c33d0b6 100644
--- a/src/SMAPI.Common/Models/ModSeachModel.cs
+++ b/src/SMAPI.Common/Models/ModSeachModel.cs
@@ -12,6 +12,9 @@ namespace StardewModdingAPI.Common.Models
         /// <summary>The namespaced mod keys to search.</summary>
         public string[] ModKeys { get; set; }
 
+        /// <summary>Whether to allow non-semantic versions, instead of returning an error for those.</summary>
+        public bool AllowInvalidVersions { get; set; }
+
 
         /*********
         ** Public methods
@@ -24,9 +27,11 @@ namespace StardewModdingAPI.Common.Models
 
         /// <summary>Construct an instance.</summary>
         /// <param name="modKeys">The namespaced mod keys to search.</param>
-        public ModSearchModel(IEnumerable<string> modKeys)
+        /// <param name="allowInvalidVersions">Whether to allow non-semantic versions, instead of returning an error for those.</param>
+        public ModSearchModel(IEnumerable<string> modKeys, bool allowInvalidVersions)
         {
             this.ModKeys = modKeys.ToArray();
+            this.AllowInvalidVersions = allowInvalidVersions;
         }
     }
 }
diff --git a/src/SMAPI.Web/Controllers/ModsApiController.cs b/src/SMAPI.Web/Controllers/ModsApiController.cs
index dcb4ec52..abae7db7 100644
--- a/src/SMAPI.Web/Controllers/ModsApiController.cs
+++ b/src/SMAPI.Web/Controllers/ModsApiController.cs
@@ -64,14 +64,15 @@ namespace StardewModdingAPI.Web.Controllers
 
         /// <summary>Fetch version metadata for the given mods.</summary>
         /// <param name="modKeys">The namespaced mod keys to search as a comma-delimited array.</param>
+        /// <param name="allowInvalidVersions">Whether to allow non-semantic versions, instead of returning an error for those.</param>
         [HttpGet]
-        public async Task<IDictionary<string, ModInfoModel>> GetAsync(string modKeys)
+        public async Task<IDictionary<string, ModInfoModel>> GetAsync(string modKeys, bool allowInvalidVersions = false)
         {
             string[] modKeysArray = modKeys?.Split(',').ToArray();
             if (modKeysArray == null || !modKeysArray.Any())
                 return new Dictionary<string, ModInfoModel>();
 
-            return await this.PostAsync(new ModSearchModel(modKeysArray));
+            return await this.PostAsync(new ModSearchModel(modKeysArray, allowInvalidVersions));
         }
 
         /// <summary>Fetch version metadata for the given mods.</summary>
@@ -79,7 +80,8 @@ namespace StardewModdingAPI.Web.Controllers
         [HttpPost]
         public async Task<IDictionary<string, ModInfoModel>> PostAsync([FromBody] ModSearchModel search)
         {
-            // sort & filter keys
+            // parse model
+            bool allowInvalidVersions = search?.AllowInvalidVersions ?? false;
             string[] modKeys = (search?.ModKeys?.ToArray() ?? new string[0])
                 .Distinct(StringComparer.CurrentCultureIgnoreCase)
                 .OrderBy(p => p, StringComparer.CurrentCultureIgnoreCase)
@@ -106,12 +108,20 @@ namespace StardewModdingAPI.Web.Controllers
                 // fetch mod info
                 result[modKey] = await this.Cache.GetOrCreateAsync($"{repository.VendorKey}:{modID}".ToLower(), async entry =>
                 {
-                    entry.AbsoluteExpiration = DateTimeOffset.UtcNow.AddMinutes(this.CacheMinutes);
-
+                    // fetch info
                     ModInfoModel info = await repository.GetModInfoAsync(modID);
-                    if (info.Error == null && (info.Version == null || !Regex.IsMatch(info.Version, this.VersionRegex, RegexOptions.CultureInvariant | RegexOptions.IgnoreCase)))
-                        info = new ModInfoModel(info.Name, info.Version, info.Url, info.Version == null ? "Mod has no version number." : $"Mod has invalid semantic version '{info.Version}'.");
 
+                    // validate
+                    if (info.Error == null)
+                    {
+                        if (info.Version == null)
+                            info = new ModInfoModel(info.Name, info.Version, info.Url, "Mod has no version number.");
+                        if (!allowInvalidVersions && !Regex.IsMatch(info.Version, this.VersionRegex, RegexOptions.CultureInvariant | RegexOptions.IgnoreCase))
+                            info = new ModInfoModel(info.Name, info.Version, info.Url, $"Mod has invalid semantic version '{info.Version}'.");
+                    }
+
+                    // cache & return
+                    entry.AbsoluteExpiration = DateTimeOffset.UtcNow.AddMinutes(this.CacheMinutes);
                     return info;
                 });
             }
diff --git a/src/SMAPI/Framework/ModData/ModDataRecord.cs b/src/SMAPI/Framework/ModData/ModDataRecord.cs
index 79a954f7..56275f53 100644
--- a/src/SMAPI/Framework/ModData/ModDataRecord.cs
+++ b/src/SMAPI/Framework/ModData/ModDataRecord.cs
@@ -106,10 +106,10 @@ namespace StardewModdingAPI.Framework.ModData
 
         /// <summary>Get a semantic local version for update checks.</summary>
         /// <param name="version">The remote version to normalise.</param>
-        public string GetLocalVersionForUpdateChecks(string version)
+        public ISemanticVersion GetLocalVersionForUpdateChecks(ISemanticVersion version)
         {
-            return this.MapLocalVersions != null && this.MapLocalVersions.TryGetValue(version, out string newVersion)
-                ? newVersion
+            return this.MapLocalVersions != null && this.MapLocalVersions.TryGetValue(version.ToString(), out string newVersion)
+                ? new SemanticVersion(newVersion)
                 : version;
         }
 
@@ -117,6 +117,11 @@ namespace StardewModdingAPI.Framework.ModData
         /// <param name="version">The remote version to normalise.</param>
         public string GetRemoteVersionForUpdateChecks(string version)
         {
+            // normalise version if possible
+            if (SemanticVersion.TryParse(version, out ISemanticVersion parsed))
+                version = parsed.ToString();
+
+            // fetch remote version
             return this.MapRemoteVersions != null && this.MapRemoteVersions.TryGetValue(version, out string newVersion)
                 ? newVersion
                 : version;
diff --git a/src/SMAPI/Framework/ModData/ParsedModDataRecord.cs b/src/SMAPI/Framework/ModData/ParsedModDataRecord.cs
index 7f49790d..deb12bdc 100644
--- a/src/SMAPI/Framework/ModData/ParsedModDataRecord.cs
+++ b/src/SMAPI/Framework/ModData/ParsedModDataRecord.cs
@@ -33,7 +33,7 @@ namespace StardewModdingAPI.Framework.ModData
         *********/
         /// <summary>Get a semantic local version for update checks.</summary>
         /// <param name="version">The remote version to normalise.</param>
-        public string GetLocalVersionForUpdateChecks(string version)
+        public ISemanticVersion GetLocalVersionForUpdateChecks(ISemanticVersion version)
         {
             return this.DataRecord.GetLocalVersionForUpdateChecks(version);
         }
diff --git a/src/SMAPI/Framework/WebApiClient.cs b/src/SMAPI/Framework/WebApiClient.cs
index e78ac14b..7f0122cf 100644
--- a/src/SMAPI/Framework/WebApiClient.cs
+++ b/src/SMAPI/Framework/WebApiClient.cs
@@ -40,7 +40,7 @@ namespace StardewModdingAPI.Framework
         {
             return this.Post<ModSearchModel, Dictionary<string, ModInfoModel>>(
                 $"v{this.Version}/mods",
-                new ModSearchModel(modKeys)
+                new ModSearchModel(modKeys, allowInvalidVersions: true)
             );
         }
 
diff --git a/src/SMAPI/Program.cs b/src/SMAPI/Program.cs
index 979f6328..1a1b9645 100644
--- a/src/SMAPI/Program.cs
+++ b/src/SMAPI/Program.cs
@@ -604,29 +604,30 @@ namespace StardewModdingAPI
                         foreach (var result in results)
                         {
                             IModMetadata mod = result.Mod;
-                            ModInfoModel info = result.Info;
+                            ModInfoModel remoteInfo = result.Info;
 
                             // handle error
-                            if (info.Error != null)
+                            if (remoteInfo.Error != null)
                             {
-                                this.Monitor.Log($"   {mod.DisplayName} ({result.Key}): update error: {info.Error}", LogLevel.Trace);
+                                this.Monitor.Log($"   {mod.DisplayName} ({result.Key}): update error: {remoteInfo.Error}", LogLevel.Trace);
                                 continue;
                             }
 
-                            // track update
-                            ISemanticVersion localVersion = mod.DataRecord != null
-                                ? new SemanticVersion(mod.DataRecord.GetLocalVersionForUpdateChecks(mod.Manifest.Version.ToString()))
-                                : mod.Manifest.Version;
-                            ISemanticVersion latestVersion = new SemanticVersion(mod.DataRecord != null
-                                ? mod.DataRecord.GetRemoteVersionForUpdateChecks(new SemanticVersion(info.Version).ToString())
-                                : info.Version
-                            );
-                            bool isUpdate = latestVersion.IsNewerThan(localVersion);
-                            this.VerboseLog($"   {mod.DisplayName} ({result.Key}): {(isUpdate ? $"{mod.Manifest.Version}{(!localVersion.Equals(mod.Manifest.Version) ? $" [{localVersion}]" : "")} => {info.Version}{(!latestVersion.Equals(new SemanticVersion(info.Version)) ? $" [{latestVersion}]" : "")}" : "okay")}.");
+                            // normalise versions
+                            ISemanticVersion localVersion = mod.DataRecord?.GetLocalVersionForUpdateChecks(mod.Manifest.Version) ?? mod.Manifest.Version;
+                            if (!SemanticVersion.TryParse(mod.DataRecord?.GetRemoteVersionForUpdateChecks(remoteInfo.Version) ?? remoteInfo.Version, out ISemanticVersion remoteVersion))
+                            {
+                                this.Monitor.Log($"   {mod.DisplayName} ({result.Key}): update error: Mod has invalid version {remoteInfo.Version}", LogLevel.Trace);
+                                continue;
+                            }
+
+                            // compare versions
+                            bool isUpdate = remoteVersion.IsNewerThan(localVersion);
+                            this.VerboseLog($"   {mod.DisplayName} ({result.Key}): {(isUpdate ? $"{mod.Manifest.Version}{(!localVersion.Equals(mod.Manifest.Version) ? $" [{localVersion}]" : "")} => {remoteInfo.Version}{(!remoteVersion.Equals(new SemanticVersion(remoteInfo.Version)) ? $" [{remoteVersion}]" : "")}" : "okay")}.");
                             if (isUpdate)
                             {
-                                if (!updatesByMod.TryGetValue(mod, out ModInfoModel other) || latestVersion.IsNewerThan(other.Version))
-                                    updatesByMod[mod] = info;
+                                if (!updatesByMod.TryGetValue(mod, out ModInfoModel other) || remoteVersion.IsNewerThan(other.Version))
+                                    updatesByMod[mod] = remoteInfo;
                             }
                         }
 
-- 
cgit 


From 0a71285c631a6ef52ea381c35224f353a618e3fa Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 4 Mar 2018 21:05:50 -0500
Subject: fix broken Nexus mod URLs

---
 .../Clients/Chucklefish/ChucklefishClient.cs          | 19 ++++++++++++++-----
 .../Framework/Clients/Nexus/NexusWebScrapeClient.cs   | 15 ++++++++++++++-
 2 files changed, 28 insertions(+), 6 deletions(-)

(limited to 'src/SMAPI.Web')

diff --git a/src/SMAPI.Web/Framework/Clients/Chucklefish/ChucklefishClient.cs b/src/SMAPI.Web/Framework/Clients/Chucklefish/ChucklefishClient.cs
index 6772672c..029553ce 100644
--- a/src/SMAPI.Web/Framework/Clients/Chucklefish/ChucklefishClient.cs
+++ b/src/SMAPI.Web/Framework/Clients/Chucklefish/ChucklefishClient.cs
@@ -12,9 +12,6 @@ namespace StardewModdingAPI.Web.Framework.Clients.Chucklefish
         /*********
         ** Properties
         *********/
-        /// <summary>The base URL for the Chucklefish mod site.</summary>
-        private readonly string BaseUrl;
-
         /// <summary>The URL for a mod page excluding the base URL, where {0} is the mod ID.</summary>
         private readonly string ModPageUrlFormat;
 
@@ -31,7 +28,6 @@ namespace StardewModdingAPI.Web.Framework.Clients.Chucklefish
         /// <param name="modPageUrlFormat">The URL for a mod page excluding the <paramref name="baseUrl"/>, where {0} is the mod ID.</param>
         public ChucklefishClient(string userAgent, string baseUrl, string modPageUrlFormat)
         {
-            this.BaseUrl = baseUrl;
             this.ModPageUrlFormat = modPageUrlFormat;
             this.Client = new FluentClient(baseUrl).SetUserAgent(userAgent);
         }
@@ -59,7 +55,7 @@ namespace StardewModdingAPI.Web.Framework.Clients.Chucklefish
             doc.LoadHtml(html);
 
             // extract mod info
-            string url = new UriBuilder(new Uri(this.BaseUrl)) { Path = string.Format(this.ModPageUrlFormat, id) }.Uri.ToString();
+            string url = this.GetModUrl(id);
             string name = doc.DocumentNode.SelectSingleNode("//meta[@name='twitter:title']").Attributes["content"].Value;
             if (name.StartsWith("[SMAPI] "))
                 name = name.Substring("[SMAPI] ".Length);
@@ -79,5 +75,18 @@ namespace StardewModdingAPI.Web.Framework.Clients.Chucklefish
         {
             this.Client?.Dispose();
         }
+
+
+        /*********
+        ** Private methods
+        *********/
+        /// <summary>Get the full mod page URL for a given ID.</summary>
+        /// <param name="id">The mod ID.</param>
+        private string GetModUrl(uint id)
+        {
+            UriBuilder builder = new UriBuilder(this.Client.BaseClient.BaseAddress);
+            builder.Path += string.Format(this.ModPageUrlFormat, id);
+            return builder.Uri.ToString();
+        }
     }
 }
diff --git a/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs b/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs
index 1e941e3d..d0597965 100644
--- a/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs
+++ b/src/SMAPI.Web/Framework/Clients/Nexus/NexusWebScrapeClient.cs
@@ -72,7 +72,7 @@ namespace StardewModdingAPI.Web.Framework.Clients.Nexus
             }
 
             // extract mod info
-            string url = new UriBuilder(this.Client.BaseClient.BaseAddress) { Path = string.Format(this.ModUrlFormat, id) }.Uri.ToString();
+            string url = this.GetModUrl(id);
             string name = doc.DocumentNode.SelectSingleNode("//h1")?.InnerText.Trim();
             string version = doc.DocumentNode.SelectSingleNode("//ul[contains(@class, 'stats')]//li[@class='stat-version']//div[@class='stat']")?.InnerText.Trim();
 
@@ -89,5 +89,18 @@ namespace StardewModdingAPI.Web.Framework.Clients.Nexus
         {
             this.Client?.Dispose();
         }
+
+
+        /*********
+        ** Private methods
+        *********/
+        /// <summary>Get the full mod page URL for a given ID.</summary>
+        /// <param name="id">The mod ID.</param>
+        private string GetModUrl(uint id)
+        {
+            UriBuilder builder = new UriBuilder(this.Client.BaseClient.BaseAddress);
+            builder.Path += string.Format(this.ModUrlFormat, id);
+            return builder.Uri.ToString();
+        }
     }
 }
-- 
cgit