From e98cd922f4d8292215c7b69e19931a8dfb0b3d61 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 26 Dec 2017 16:16:01 -0500
Subject: [PATCH 01/21] fix typo on download page

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

diff --git a/src/SMAPI.Web/Views/Index/Index.cshtml b/src/SMAPI.Web/Views/Index/Index.cshtml
index 2d76cc151..ad58898e1 100644
--- a/src/SMAPI.Web/Views/Index/Index.cshtml
+++ b/src/SMAPI.Web/Views/Index/Index.cshtml
@@ -62,7 +62,7 @@
 
 <h2>For mod creators</h2>
 <ul>
-    <li><a href="@Model.DevDownloadUrl">SMAPI 2.2 for developers</a> (includes <a href="https://docs.microsoft.com/en-us/visualstudio/ide/using-intellisense">intellisense</a> and full console output)</li>
+    <li><a href="@Model.DevDownloadUrl">SMAPI @Model.LatestVersion for developers</a> (includes <a href="https://docs.microsoft.com/en-us/visualstudio/ide/using-intellisense">intellisense</a> and full console output)</li>
     <li><a href="https://stardewvalleywiki.com/Modding:Index">Modding documentation</a></li>
     <li>Need help? Come <a href="https://stardewvalleywiki.com/Modding:Community#Discord">chat on Discord</a>.</li>
 </ul>

From d1a829497292ea4c72da5f9bbfb3b50404e06022 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Thu, 28 Dec 2017 00:16:54 -0500
Subject: [PATCH 02/21] fix issue where assemblies could be incorrectly
 reloaded

---
 docs/release-notes.md                            | 4 ++++
 src/SMAPI/Framework/ModLoading/AssemblyLoader.cs | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 165e7d4ef..aaa6d3ccc 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,4 +1,8 @@
 # Release notes
+## 2.4
+* For players:
+  * Fixed rare issues caused by assembly references being incorrectly loaded twice.
+
 ## 2.3
 * For players:
   * Added a user-friendly [download page](https://smapi.io).
diff --git a/src/SMAPI/Framework/ModLoading/AssemblyLoader.cs b/src/SMAPI/Framework/ModLoading/AssemblyLoader.cs
index 1e3c4a05a..3a7b214aa 100644
--- a/src/SMAPI/Framework/ModLoading/AssemblyLoader.cs
+++ b/src/SMAPI/Framework/ModLoading/AssemblyLoader.cs
@@ -162,7 +162,11 @@ private IEnumerable<AssemblyParseResult> GetReferencedLocalAssemblies(FileInfo f
 
             // skip if already visited
             if (visitedAssemblyNames.Contains(assembly.Name.Name))
+            {
                 yield return new AssemblyParseResult(file, null, AssemblyLoadStatus.AlreadyLoaded);
+                yield break;
+            }
+
             visitedAssemblyNames.Add(assembly.Name.Name);
 
             // yield referenced assemblies

From d471ac296e2e6ca737781f80794a521b74ac75fb Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 2 Jan 2018 16:48:33 -0500
Subject: [PATCH 03/21] fix parse error for logs with zero installed mods

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

diff --git a/docs/release-notes.md b/docs/release-notes.md
index aaa6d3ccc..a7a3c97fb 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -3,6 +3,9 @@
 * For players:
   * Fixed rare issues caused by assembly references being incorrectly loaded twice.
 
+* For the [log parser][]:
+  * Fixed parse error for logs with zero installed mods.
+
 ## 2.3
 * For players:
   * Added a user-friendly [download page](https://smapi.io).
diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index b7724c697..1659de8fb 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -3,9 +3,9 @@
 }
 @model StardewModdingAPI.Web.ViewModels.LogParserModel
 @section Head {
-    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=20171202" />
+    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=20180101" />
     <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=20171202"></script>
+    <script src="~/Content/js/log-parser.js?r=20180101"></script>
     <style type="text/css" id="modflags"></style>
     <script>
         $(function() {
diff --git a/src/SMAPI.Web/wwwroot/Content/js/log-parser.js b/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
index 6cce1ce96..914863f68 100644
--- a/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
+++ b/src/SMAPI.Web/wwwroot/Content/js/log-parser.js
@@ -175,7 +175,7 @@ smapi.logParser = function(sectionUrl, pasteID) {
 
         }
         var dataInfo = regexInfo.exec(data) || regexInfo.exec(data) || regexInfo.exec(data),
-            dataMods = regexMods.exec(data) || regexMods.exec(data) || regexMods.exec(data),
+            dataMods = regexMods.exec(data) || regexMods.exec(data) || regexMods.exec(data) || [""],
             dataDate = regexDate.exec(data) || regexDate.exec(data) || regexDate.exec(data),
             dataPath = regexPath.exec(data) || regexPath.exec(data) || regexPath.exec(data),
             match;

From d8a3ed36c849fdeb1a2d9ca9f55682dc8e5d21fb Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 7 Jan 2018 18:47:07 -0500
Subject: [PATCH 04/21] update ReSharper settings

---
 src/SMAPI.sln.DotSettings | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/SMAPI.sln.DotSettings b/src/SMAPI.sln.DotSettings
index d16ef6846..68d7c72bf 100644
--- a/src/SMAPI.sln.DotSettings
+++ b/src/SMAPI.sln.DotSettings
@@ -6,6 +6,7 @@
 	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpCodeStyle/StaticQualifier/STATIC_MEMBERS_QUALIFY_MEMBERS/@EntryValue">Field, Property, Event, Method</s:String>
 	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpCodeStyle/ThisQualifier/INSTANCE_MEMBERS_QUALIFY_MEMBERS/@EntryValue">Field, Property, Event, Method</s:String>
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/LINE_FEED_AT_FILE_END/@EntryValue">True</s:Boolean>
+	<s:String x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/PLACE_ACCESSORHOLDER_ATTRIBUTE_ON_SAME_LINE_EX/@EntryValue">NEVER</s:String>
 	<s:Boolean x:Key="/Default/CodeStyle/CodeFormatting/CSharpFormat/WRAP_LINES/@EntryValue">False</s:Boolean>
 	<s:String x:Key="/Default/CodeStyle/CSharpVarKeywordUsage/ForBuiltInTypes/@EntryValue">UseVarWhenEvident</s:String>
 	<s:String x:Key="/Default/CodeStyle/CSharpVarKeywordUsage/ForSimpleTypes/@EntryValue">UseExplicitType</s:String>
@@ -13,7 +14,12 @@
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/PredefinedNamingRules/=PrivateInstanceFields/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="AaBb"&gt;&lt;ExtraRule Prefix="_" Suffix="" Style="aaBb" /&gt;&lt;/Policy&gt;</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/PredefinedNamingRules/=PrivateStaticFields/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="AaBb"&gt;&lt;ExtraRule Prefix="_" Suffix="" Style="aaBb" /&gt;&lt;/Policy&gt;</s:String>
 	<s:String x:Key="/Default/CodeStyle/Naming/CSharpNaming/PredefinedNamingRules/=PrivateStaticReadonly/@EntryIndexedValue">&lt;Policy Inspect="True" Prefix="" Suffix="" Style="AaBb"&gt;&lt;ExtraRule Prefix="_" Suffix="" Style="aaBb" /&gt;&lt;/Policy&gt;</s:String>
+	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpAttributeForSingleLineMethodUpgrade/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpKeepExistingMigration/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpPlaceEmbeddedOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpRenamePlacementToArrangementMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EAddAccessorOwnerDeclarationBracesMigration/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002ECSharpPlaceAttributeOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateBlankLinesAroundFieldToBlankLinesAroundProperty/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateThisQualifierSettings/@EntryIndexedValue">True</s:Boolean>
 </wpf:ResourceDictionary>
\ No newline at end of file

From dc2ceb39f31c35752c943b5052d5abaa7b6494fa Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Fri, 12 Jan 2018 01:11:16 -0500
Subject: [PATCH 05/21] fix curly quotes in config.json automatically if
 possible (#412)

---
 docs/release-notes.md                         |  1 +
 .../Framework/Serialisation/JsonHelper.cs     | 44 +++++++++++++++----
 2 files changed, 36 insertions(+), 9 deletions(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index a7a3c97fb..87ef671ed 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,6 +1,7 @@
 # Release notes
 ## 2.4
 * For players:
+  * SMAPI now fixes curly quotes in `config.json` if possible.
   * Fixed rare issues caused by assembly references being incorrectly loaded twice.
 
 * For the [log parser][]:
diff --git a/src/SMAPI/Framework/Serialisation/JsonHelper.cs b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
index d923ec0cf..7c4e3ee33 100644
--- a/src/SMAPI/Framework/Serialisation/JsonHelper.cs
+++ b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
@@ -55,18 +55,15 @@ public TModel ReadJsonFile<TModel>(string fullPath)
             // deserialise model
             try
             {
-                return JsonConvert.DeserializeObject<TModel>(json, this.JsonSettings);
+                return this.Deserialise<TModel>(json);
             }
             catch (JsonReaderException ex)
             {
-                string message = $"The file at {fullPath} doesn't seem to be valid JSON.";
-
-                string text = File.ReadAllText(fullPath);
-                if (text.Contains("“") || text.Contains("”"))
-                    message += " Found curly quotes in the text; note that only straight quotes are allowed in JSON.";
-
-                message += $"\nTechnical details: {ex.Message}";
-                throw new JsonReaderException(message);
+                string error = $"The file at {fullPath} doesn't seem to be valid JSON.";
+                if (json.Contains("“") || json.Contains("”"))
+                    error += " Found curly quotes in the text; note that only straight quotes are allowed in JSON.";
+                error += $"\nTechnical details: {ex.Message}";
+                throw new JsonReaderException(error);
             }
         }
 
@@ -93,5 +90,34 @@ public void WriteJsonFile<TModel>(string fullPath, TModel model)
             string json = JsonConvert.SerializeObject(model, this.JsonSettings);
             File.WriteAllText(fullPath, json);
         }
+
+
+        /*********
+        ** Private methods
+        *********/
+        /// <summary>Deserialize JSON text if possible.</summary>
+        /// <typeparam name="TModel">The model type.</typeparam>
+        /// <param name="json">The raw JSON text.</param>
+        private TModel Deserialise<TModel>(string json)
+        {
+            try
+            {
+                return JsonConvert.DeserializeObject<TModel>(json, this.JsonSettings);
+            }
+            catch (JsonReaderException)
+            {
+                // try replacing curly quotes
+                if (json.Contains("“") || json.Contains("”"))
+                {
+                    try
+                    {
+                        return JsonConvert.DeserializeObject<TModel>(json.Replace('“', '"').Replace('”', '"'), this.JsonSettings);
+                    }
+                    catch { /* rethrow original error */ }
+                }
+
+                throw;
+            }
+        }
     }
 }

From 0ad9fbddddbf9edfd847c507d70e10d2f8ce559b Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Fri, 12 Jan 2018 01:24:49 -0500
Subject: [PATCH 06/21] fix semantic versions always ignoring `-0` tag (#421)

---
 docs/release-notes.md                         |  1 +
 src/SMAPI.Common/SemanticVersionImpl.cs       |  4 +--
 .../Utilities/SemanticVersionTests.cs         | 17 ++++++++++++
 src/SMAPI/Framework/LegacyManifestVersion.cs  | 26 +++++++++++++++++++
 .../Serialisation/SFieldConverter.cs          |  2 +-
 src/SMAPI/StardewModdingAPI.csproj            |  1 +
 6 files changed, 47 insertions(+), 4 deletions(-)
 create mode 100644 src/SMAPI/Framework/LegacyManifestVersion.cs

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 87ef671ed..4b6a7ba5e 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -2,6 +2,7 @@
 ## 2.4
 * For players:
   * SMAPI now fixes curly quotes in `config.json` if possible.
+  * Fixed semantic versions always ignoring `-0` tag.
   * Fixed rare issues caused by assembly references being incorrectly loaded twice.
 
 * For the [log parser][]:
diff --git a/src/SMAPI.Common/SemanticVersionImpl.cs b/src/SMAPI.Common/SemanticVersionImpl.cs
index 53cf5a214..1c713b471 100644
--- a/src/SMAPI.Common/SemanticVersionImpl.cs
+++ b/src/SMAPI.Common/SemanticVersionImpl.cs
@@ -190,9 +190,7 @@ internal static bool TryParse(string version, out SemanticVersionImpl parsed)
         private string GetNormalisedTag(string tag)
         {
             tag = tag?.Trim();
-            if (string.IsNullOrWhiteSpace(tag) || tag == "0") // '0' from incorrect examples in old SMAPI documentation
-                return null;
-            return tag;
+            return !string.IsNullOrWhiteSpace(tag) ? tag : null;
         }
     }
 }
diff --git a/src/SMAPI.Tests/Utilities/SemanticVersionTests.cs b/src/SMAPI.Tests/Utilities/SemanticVersionTests.cs
index d3e0988ee..f1a720121 100644
--- a/src/SMAPI.Tests/Utilities/SemanticVersionTests.cs
+++ b/src/SMAPI.Tests/Utilities/SemanticVersionTests.cs
@@ -33,6 +33,7 @@ public string Constructor_FromString(string input)
         [TestCase(3000, 4000, 5000, null, ExpectedResult = "3000.4000.5000")]
         [TestCase(1, 2, 3, "", ExpectedResult = "1.2.3")]
         [TestCase(1, 2, 3, "    ", ExpectedResult = "1.2.3")]
+        [TestCase(1, 2, 3, "0", ExpectedResult = "1.2.3-0")]
         [TestCase(1, 2, 3, "some-tag.4", ExpectedResult = "1.2.3-some-tag.4")]
         [TestCase(1, 2, 3, "some-tag.4   ", ExpectedResult = "1.2.3-some-tag.4")]
         public string Constructor_FromParts(int major, int minor, int patch, string tag)
@@ -270,6 +271,22 @@ public void GameVersion(string versionStr)
             Assert.IsTrue(version.IsOlderThan(new SemanticVersion("1.2.30")), "The game version should be considered older than the later semantic versions.");
         }
 
+        /****
+        ** LegacyManifestVersion
+        ****/
+        [Test(Description = "Assert that the LegacyManifestVersion subclass correctly parses legacy manifest versions.")]
+        [TestCase(1, 0, 0, null, ExpectedResult = "1.0")]
+        [TestCase(3000, 4000, 5000, null, ExpectedResult = "3000.4000.5000")]
+        [TestCase(1, 2, 3, "", ExpectedResult = "1.2.3")]
+        [TestCase(1, 2, 3, "    ", ExpectedResult = "1.2.3")]
+        [TestCase(1, 2, 3, "0", ExpectedResult = "1.2.3")]  // special case: drop '0' tag for legacy manifest versions
+        [TestCase(1, 2, 3, "some-tag.4", ExpectedResult = "1.2.3-some-tag.4")]
+        [TestCase(1, 2, 3, "some-tag.4   ", ExpectedResult = "1.2.3-some-tag.4")]
+        public string LegacyManifestVersion(int major, int minor, int patch, string tag)
+        {
+            return new LegacyManifestVersion(major, minor, patch, tag).ToString();
+        }
+
 
         /*********
         ** Private methods
diff --git a/src/SMAPI/Framework/LegacyManifestVersion.cs b/src/SMAPI/Framework/LegacyManifestVersion.cs
new file mode 100644
index 000000000..454b9137c
--- /dev/null
+++ b/src/SMAPI/Framework/LegacyManifestVersion.cs
@@ -0,0 +1,26 @@
+using Newtonsoft.Json;
+
+namespace StardewModdingAPI.Framework
+{
+    /// <summary>An implementation of <see cref="ISemanticVersion"/> that hamdles the legacy <see cref="IManifest"/> version format.</summary>
+    internal class LegacyManifestVersion : SemanticVersion
+    {
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Construct an instance.</summary>
+        /// <param name="majorVersion">The major version incremented for major API changes.</param>
+        /// <param name="minorVersion">The minor version incremented for backwards-compatible changes.</param>
+        /// <param name="patchVersion">The patch version for backwards-compatible bug fixes.</param>
+        /// <param name="build">An optional build tag.</param>
+        [JsonConstructor]
+        public LegacyManifestVersion(int majorVersion, int minorVersion, int patchVersion, string build = null)
+            : base(
+                majorVersion,
+                minorVersion,
+                patchVersion,
+                build != "0" ? build : null // '0' from incorrect examples in old SMAPI documentation
+            )
+        { }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/SFieldConverter.cs b/src/SMAPI/Framework/Serialisation/SFieldConverter.cs
index 917c950d9..6e0685997 100644
--- a/src/SMAPI/Framework/Serialisation/SFieldConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/SFieldConverter.cs
@@ -51,7 +51,7 @@ public override object ReadJson(JsonReader reader, Type objectType, object exist
                             int minor = obj.Value<int>(nameof(ISemanticVersion.MinorVersion));
                             int patch = obj.Value<int>(nameof(ISemanticVersion.PatchVersion));
                             string build = obj.Value<string>(nameof(ISemanticVersion.Build));
-                            return new SemanticVersion(major, minor, patch, build);
+                            return new LegacyManifestVersion(major, minor, patch, build);
                         }
 
                     case JTokenType.String:
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index f76ac439e..c9c302f58 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -86,6 +86,7 @@
       <Link>Properties\GlobalAssemblyInfo.cs</Link>
     </Compile>
     <Compile Include="Framework\Content\ContentCache.cs" />
+    <Compile Include="Framework\LegacyManifestVersion.cs" />
     <Compile Include="Framework\Models\ModCompatibility.cs" />
     <Compile Include="Framework\ModLoading\Finders\EventFinder.cs" />
     <Compile Include="Framework\ModLoading\Finders\FieldFinder.cs" />

From b87d6da6c514c40f01396bfd8e403eb0a5f2e25c Mon Sep 17 00:00:00 2001
From: EnderHDMC <EnderHDMC@gmail.com>
Date: Mon, 15 Jan 2018 23:33:43 +0200
Subject: [PATCH 07/21] Update unix-launcher.sh

---
 src/SMAPI/unix-launcher.sh | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/SMAPI/unix-launcher.sh b/src/SMAPI/unix-launcher.sh
index 70f1873a4..30021470f 100644
--- a/src/SMAPI/unix-launcher.sh
+++ b/src/SMAPI/unix-launcher.sh
@@ -63,7 +63,11 @@ else
 
     # open SMAPI in terminal
     if $COMMAND x-terminal-emulator 2>/dev/null; then
-        x-terminal-emulator -e "$LAUNCHER"
+        if [[ " $(readlink $(readlink /usr/bin/x-terminal-emulator))" == *"/terminator" ]]; then
+            terminator -e "$LAUNCHER"
+        else
+            x-terminal-emulator -e "$LAUNCHER"
+        fi
     elif $COMMAND xfce4-terminal 2>/dev/null; then
         xfce4-terminal -e "$LAUNCHER"
     elif $COMMAND gnome-terminal 2>/dev/null; then

From 0122abaf203c1efa70e5e48336cc4800d4f01cc0 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 16 Jan 2018 19:16:58 -0500
Subject: [PATCH 08/21] add JSON converters for crossplatform-incompatible
 types (#423)

---
 docs/release-notes.md                         |  7 +-
 .../Framework/Serialisation/ColorConverter.cs | 82 ++++++++++++++++++
 .../Framework/Serialisation/JsonHelper.cs     |  8 +-
 .../Framework/Serialisation/PointConverter.cs | 79 +++++++++++++++++
 .../Serialisation/RectangleConverter.cs       | 84 +++++++++++++++++++
 src/SMAPI/StardewModdingAPI.csproj            |  3 +
 6 files changed, 260 insertions(+), 3 deletions(-)
 create mode 100644 src/SMAPI/Framework/Serialisation/ColorConverter.cs
 create mode 100644 src/SMAPI/Framework/Serialisation/PointConverter.cs
 create mode 100644 src/SMAPI/Framework/Serialisation/RectangleConverter.cs

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 4b6a7ba5e..c4a96b95e 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,9 +1,12 @@
 # Release notes
 ## 2.4
 * For players:
-  * SMAPI now fixes curly quotes in `config.json` if possible.
+  * Fixed error parsing `config.json` files containing curly quotes.
+  * Fixed error parsing JSON files generated on another platform.
+  * Fixed rare issues caused by mods reloading core assemblies, which is no longer allowed.
+
+* For mod authors:
   * Fixed semantic versions always ignoring `-0` tag.
-  * Fixed rare issues caused by assembly references being incorrectly loaded twice.
 
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.
diff --git a/src/SMAPI/Framework/Serialisation/ColorConverter.cs b/src/SMAPI/Framework/Serialisation/ColorConverter.cs
new file mode 100644
index 000000000..d29f73b8c
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/ColorConverter.cs
@@ -0,0 +1,82 @@
+using System;
+using Microsoft.Xna.Framework;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Exceptions;
+
+namespace StardewModdingAPI.Framework.Serialisation
+{
+    /// <summary>Handles deserialisation of <see cref="Color"/> for crossplatform compatibility.</summary>
+    internal class ColorConverter : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(Color);
+        }
+
+        /// <summary>Reads the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            //    Linux/Mac: { "B": 76, "G": 51, "R": 25, "A": 102 }
+            //    Windows:   "26, 51, 76, 102"
+            JToken token = JToken.Load(reader);
+            switch (token.Type)
+            {
+                case JTokenType.Object:
+                    {
+                        JObject obj = (JObject)token;
+                        int r = obj.Value<int>(nameof(Color.R));
+                        int g = obj.Value<int>(nameof(Color.G));
+                        int b = obj.Value<int>(nameof(Color.B));
+                        int a = obj.Value<int>(nameof(Color.A));
+                        return new Color(r, g, b, a);
+                    }
+
+                case JTokenType.String:
+                    {
+                        string str = token.Value<string>();
+                        if (string.IsNullOrWhiteSpace(str))
+                            return null;
+
+                        string[] parts = str.Split(',');
+                        if (parts.Length != 4)
+                            throw new SParseException($"Can't parse {typeof(Color).Name} from {token.Path}, invalid value '{str}'.");
+
+                        int r = Convert.ToInt32(parts[0]);
+                        int g = Convert.ToInt32(parts[1]);
+                        int b = Convert.ToInt32(parts[2]);
+                        int a = Convert.ToInt32(parts[3]);
+                        return new Color(r, g, b, a);
+                    }
+
+                default:
+                    throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, must be an object or string.");
+            }
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/JsonHelper.cs b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
index 7c4e3ee33..f66f9dfb0 100644
--- a/src/SMAPI/Framework/Serialisation/JsonHelper.cs
+++ b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
@@ -19,9 +19,15 @@ internal class JsonHelper
             ObjectCreationHandling = ObjectCreationHandling.Replace, // avoid issue where default ICollection<T> values are duplicated each time the config is loaded
             Converters = new List<JsonConverter>
             {
+                // enums
                 new StringEnumConverter<Buttons>(),
                 new StringEnumConverter<Keys>(),
-                new StringEnumConverter<SButton>()
+                new StringEnumConverter<SButton>(),
+
+                // crossplatform compatibility
+                new ColorConverter(),
+                new PointConverter(),
+                new RectangleConverter()
             }
         };
 
diff --git a/src/SMAPI/Framework/Serialisation/PointConverter.cs b/src/SMAPI/Framework/Serialisation/PointConverter.cs
new file mode 100644
index 000000000..d35660bef
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/PointConverter.cs
@@ -0,0 +1,79 @@
+using System;
+using Microsoft.Xna.Framework;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Exceptions;
+
+namespace StardewModdingAPI.Framework.Serialisation
+{
+    /// <summary>Handles deserialisation of <see cref="PointConverter"/> for crossplatform compatibility.</summary>
+    internal class PointConverter : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(Point);
+        }
+
+        /// <summary>Reads the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            // point
+            //    Linux/Mac: { "X": 1, "Y": 2 }
+            //    Windows:   "1, 2"
+            JToken token = JToken.Load(reader);
+            switch (token.Type)
+            {
+                case JTokenType.Object:
+                    {
+                        JObject obj = (JObject)token;
+                        int x = obj.Value<int>(nameof(Point.X));
+                        int y = obj.Value<int>(nameof(Point.Y));
+                        return new Point(x, y);
+                    }
+
+                case JTokenType.String:
+                    {
+                        string str = token.Value<string>();
+                        if (string.IsNullOrWhiteSpace(str))
+                            return null;
+
+                        string[] parts = str.Split(',');
+                        if (parts.Length != 2)
+                            throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, invalid value '{str}'.");
+
+                        int x = Convert.ToInt32(parts[0]);
+                        int y = Convert.ToInt32(parts[1]);
+                        return new Point(x, y);
+                    }
+
+                default:
+                    throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, must be an object or string.");
+            }
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/RectangleConverter.cs b/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
new file mode 100644
index 000000000..74df54e2b
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
@@ -0,0 +1,84 @@
+using System;
+using System.Text.RegularExpressions;
+using Microsoft.Xna.Framework;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Exceptions;
+
+namespace StardewModdingAPI.Framework.Serialisation
+{
+    /// <summary>Handles deserialisation of <see cref="Rectangle"/> for crossplatform compatibility.</summary>
+    internal class RectangleConverter : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(Rectangle);
+        }
+
+        /// <summary>Reads the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            //    Linux/Mac: { "X": 1, "Y": 2, "Width": 3, "Height": 4 }
+            //    Windows:   "{X:1 Y:2 Width:3 Height:4}"
+            JToken token = JToken.Load(reader);
+            switch (token.Type)
+            {
+                case JTokenType.Object:
+                    {
+                        JObject obj = (JObject)token;
+                        int x = obj.Value<int>(nameof(Rectangle.X));
+                        int y = obj.Value<int>(nameof(Rectangle.Y));
+                        int width = obj.Value<int>(nameof(Rectangle.Width));
+                        int height = obj.Value<int>(nameof(Rectangle.Height));
+                        return new Rectangle(x, y, width, height);
+                    }
+
+                case JTokenType.String:
+                    {
+                        string str = token.Value<string>();
+                        if (string.IsNullOrWhiteSpace(str))
+                            return Rectangle.Empty;
+
+                        var match = Regex.Match(str, @"^\{X:(?<x>\d+) Y:(?<y>\d+) Width:(?<width>\d+) Height:(?<height>\d+)\}$");
+                        if (!match.Success)
+                            throw new SParseException($"Can't parse {typeof(Rectangle).Name} from {reader.Path}, invalid string format.");
+
+                        int x = Convert.ToInt32(match.Groups["x"].Value);
+                        int y = Convert.ToInt32(match.Groups["y"].Value);
+                        int width = Convert.ToInt32(match.Groups["width"].Value);
+                        int height = Convert.ToInt32(match.Groups["height"].Value);
+
+                        return new Rectangle(x, y, width, height);
+                    }
+
+                default:
+                    throw new SParseException($"Can't parse {typeof(Rectangle).Name} from {reader.Path}, must be an object or string.");
+            }
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+    }
+}
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index c9c302f58..0625711c6 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -108,6 +108,9 @@
     <Compile Include="Framework\Exceptions\SAssemblyLoadFailedException.cs" />
     <Compile Include="Framework\ModLoading\AssemblyLoadStatus.cs" />
     <Compile Include="Framework\Reflection\InterfaceProxyBuilder.cs" />
+    <Compile Include="Framework\Serialisation\RectangleConverter.cs" />
+    <Compile Include="Framework\Serialisation\ColorConverter.cs" />
+    <Compile Include="Framework\Serialisation\PointConverter.cs" />
     <Compile Include="Framework\Utilities\ContextHash.cs" />
     <Compile Include="IReflectedField.cs" />
     <Compile Include="IReflectedMethod.cs" />

From 3df62fa856d241df4e7b30a48c77dab184edba4e Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Tue, 16 Jan 2018 19:27:39 -0500
Subject: [PATCH 09/21] fix typo

---
 src/SMAPI.ModBuildConfig/build/smapi.targets | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/SMAPI.ModBuildConfig/build/smapi.targets b/src/SMAPI.ModBuildConfig/build/smapi.targets
index c0319e225..83f0dcbd3 100644
--- a/src/SMAPI.ModBuildConfig/build/smapi.targets
+++ b/src/SMAPI.ModBuildConfig/build/smapi.targets
@@ -122,8 +122,8 @@
     <Error Condition="'$(OS)' != 'OSX' AND '$(OS)' != 'Unix' AND '$(OS)' != 'Windows_NT'" Text="The mod build package doesn't recognise OS type '$(OS)'." />
 
     <Error Condition="!Exists('$(GamePath)')" Text="The mod build package can't find your game folder. You can specify where to find it; see details at https://github.com/Pathoschild/SMAPI/blob/develop/docs/mod-build-config.md#game-path." />
-    <Error Condition="'$(OS)' == 'Windows_NT' AND !Exists('$(GamePath)\Stardew Valley.exe')" Text="The mod build package found a a game folder at $(GamePath), but it doesn't contain the Stardew Valley.exe file. If this folder is invalid, delete it and the package will autodetect another game install path." />
-    <Error Condition="'$(OS)' != 'Windows_NT' AND !Exists('$(GamePath)\StardewValley.exe')" Text="The mod build package found a a game folder at $(GamePath), but it doesn't contain the StardewValley.exe file. If this folder is invalid, delete it and the package will autodetect another game install path." />
+    <Error Condition="'$(OS)' == 'Windows_NT' AND !Exists('$(GamePath)\Stardew Valley.exe')" Text="The mod build package found a game folder at $(GamePath), but it doesn't contain the Stardew Valley.exe file. If this folder is invalid, delete it and the package will autodetect another game install path." />
+    <Error Condition="'$(OS)' != 'Windows_NT' AND !Exists('$(GamePath)\StardewValley.exe')" Text="The mod build package found a game folder at $(GamePath), but it doesn't contain the StardewValley.exe file. If this folder is invalid, delete it and the package will autodetect another game install path." />
     <Error Condition="!Exists('$(GamePath)\StardewModdingAPI.exe')" Text="The mod build package found a game folder at $(GamePath), but it doesn't contain SMAPI. You need to install SMAPI before building the mod." />
   </Target>
 

From a59572ee4eb64b075836247b92401c0fb554b6f0 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Wed, 17 Jan 2018 00:20:24 -0500
Subject: [PATCH 10/21] overhaul input handling (#422)

---
 docs/release-notes.md                    |  12 +-
 src/SMAPI/Framework/Input/InputState.cs  | 163 ++++++++++++++++++++
 src/SMAPI/Framework/Input/InputStatus.cs |  29 ++++
 src/SMAPI/Framework/SGame.cs             | 186 ++++++-----------------
 src/SMAPI/SButton.cs                     |  15 ++
 src/SMAPI/StardewModdingAPI.csproj       |   2 +
 6 files changed, 266 insertions(+), 141 deletions(-)
 create mode 100644 src/SMAPI/Framework/Input/InputState.cs
 create mode 100644 src/SMAPI/Framework/Input/InputStatus.cs

diff --git a/docs/release-notes.md b/docs/release-notes.md
index c4a96b95e..dd9f03525 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -5,12 +5,18 @@
   * Fixed error parsing JSON files generated on another platform.
   * Fixed rare issues caused by mods reloading core assemblies, which is no longer allowed.
 
-* For mod authors:
-  * Fixed semantic versions always ignoring `-0` tag.
-
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.
 
+* For modders:
+  * Added `SButton` `IsActionButton()` and `IsUseToolButton()` extensions.
+  * Fixed input events not recognising controller input as an action or use-tool button.
+  * Fixed input events setting the same `IsActionButton` and `IsUseToolButton` values for all buttons pressed in an update tick.
+  * Fixed semantic versions always ignoring `-0` tag.
+
+* For SMAPI developers:
+  * Overhauled input handling to support future input events.
+
 ## 2.3
 * For players:
   * Added a user-friendly [download page](https://smapi.io).
diff --git a/src/SMAPI/Framework/Input/InputState.cs b/src/SMAPI/Framework/Input/InputState.cs
new file mode 100644
index 000000000..8b0108ae7
--- /dev/null
+++ b/src/SMAPI/Framework/Input/InputState.cs
@@ -0,0 +1,163 @@
+using System.Collections.Generic;
+using System.Linq;
+using Microsoft.Xna.Framework;
+using Microsoft.Xna.Framework.Input;
+using StardewValley;
+
+namespace StardewModdingAPI.Framework.Input
+{
+    /// <summary>A summary of input changes during an update frame.</summary>
+    internal class InputState
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>The underlying controller state.</summary>
+        public GamePadState ControllerState { get; }
+
+        /// <summary>The underlying keyboard state.</summary>
+        public KeyboardState KeyboardState { get; }
+
+        /// <summary>The underlying mouse state.</summary>
+        public MouseState MouseState { get; }
+
+        /// <summary>The mouse position on the screen adjusted for the zoom level.</summary>
+        public Point MousePosition { get; }
+
+        /// <summary>The buttons which were pressed, held, or released.</summary>
+        public IDictionary<SButton, InputStatus> ActiveButtons { get; } = new Dictionary<SButton, InputStatus>();
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Construct an empty instance.</summary>
+        public InputState() { }
+
+        /// <summary>Construct an instance.</summary>
+        /// <param name="previousState">The previous input state.</param>
+        /// <param name="controllerState">The current controller state.</param>
+        /// <param name="keyboardState">The current keyboard state.</param>
+        /// <param name="mouseState">The current mouse state.</param>
+        public InputState(InputState previousState, GamePadState controllerState, KeyboardState keyboardState, MouseState mouseState)
+        {
+            // init properties
+            this.ControllerState = controllerState;
+            this.KeyboardState = keyboardState;
+            this.MouseState = mouseState;
+            this.MousePosition = new Point((int)(mouseState.X * (1.0 / Game1.options.zoomLevel)), (int)(mouseState.Y * (1.0 / Game1.options.zoomLevel))); // derived from Game1::getMouseX
+
+            // get button states
+            SButton[] down = InputState.GetPressedButtons(keyboardState, mouseState, controllerState).ToArray();
+            foreach (SButton button in down)
+                this.ActiveButtons[button] = this.GetStatus(previousState.GetStatus(button), isDown: true);
+            foreach (KeyValuePair<SButton, InputStatus> prev in previousState.ActiveButtons)
+            {
+                if (prev.Value.IsDown() && !this.ActiveButtons.ContainsKey(prev.Key))
+                    this.ActiveButtons[prev.Key] = InputStatus.Released;
+            }
+        }
+
+        /// <summary>Get the status of a button.</summary>
+        /// <param name="button">The button to check.</param>
+        public InputStatus GetStatus(SButton button)
+        {
+            return this.ActiveButtons.TryGetValue(button, out InputStatus status) ? status : InputStatus.None;
+        }
+
+        /// <summary>Get whether a given button was pressed or held.</summary>
+        /// <param name="button">The button to check.</param>
+        public bool IsDown(SButton button)
+        {
+            return this.GetStatus(button).IsDown();
+        }
+
+        /// <summary>Get the current input state.</summary>
+        /// <param name="previousState">The previous input state.</param>
+        public static InputState GetState(InputState previousState)
+        {
+            GamePadState controllerState = GamePad.GetState(PlayerIndex.One);
+            KeyboardState keyboardState = Keyboard.GetState();
+            MouseState mouseState = Mouse.GetState();
+
+            return new InputState(previousState, controllerState, keyboardState, mouseState);
+        }
+
+        /*********
+        ** Private methods
+        *********/
+        /// <summary>Get the status of a button.</summary>
+        /// <param name="oldStatus">The previous button status.</param>
+        /// <param name="isDown">Whether the button is currently down.</param>
+        public InputStatus GetStatus(InputStatus oldStatus, bool isDown)
+        {
+            if (isDown && oldStatus.IsDown())
+                return InputStatus.Held;
+            if (isDown)
+                return InputStatus.Pressed;
+            return InputStatus.Released;
+        }
+
+        /// <summary>Get the buttons pressed in the given stats.</summary>
+        /// <param name="keyboard">The keyboard state.</param>
+        /// <param name="mouse">The mouse state.</param>
+        /// <param name="controller">The controller state.</param>
+        private static IEnumerable<SButton> GetPressedButtons(KeyboardState keyboard, MouseState mouse, GamePadState controller)
+        {
+            // keyboard
+            foreach (Keys key in keyboard.GetPressedKeys())
+                yield return key.ToSButton();
+
+            // mouse
+            if (mouse.LeftButton == ButtonState.Pressed)
+                yield return SButton.MouseLeft;
+            if (mouse.RightButton == ButtonState.Pressed)
+                yield return SButton.MouseRight;
+            if (mouse.MiddleButton == ButtonState.Pressed)
+                yield return SButton.MouseMiddle;
+            if (mouse.XButton1 == ButtonState.Pressed)
+                yield return SButton.MouseX1;
+            if (mouse.XButton2 == ButtonState.Pressed)
+                yield return SButton.MouseX2;
+
+            // controller
+            if (controller.IsConnected)
+            {
+                if (controller.Buttons.A == ButtonState.Pressed)
+                    yield return SButton.ControllerA;
+                if (controller.Buttons.B == ButtonState.Pressed)
+                    yield return SButton.ControllerB;
+                if (controller.Buttons.Back == ButtonState.Pressed)
+                    yield return SButton.ControllerBack;
+                if (controller.Buttons.BigButton == ButtonState.Pressed)
+                    yield return SButton.BigButton;
+                if (controller.Buttons.LeftShoulder == ButtonState.Pressed)
+                    yield return SButton.LeftShoulder;
+                if (controller.Buttons.LeftStick == ButtonState.Pressed)
+                    yield return SButton.LeftStick;
+                if (controller.Buttons.RightShoulder == ButtonState.Pressed)
+                    yield return SButton.RightShoulder;
+                if (controller.Buttons.RightStick == ButtonState.Pressed)
+                    yield return SButton.RightStick;
+                if (controller.Buttons.Start == ButtonState.Pressed)
+                    yield return SButton.ControllerStart;
+                if (controller.Buttons.X == ButtonState.Pressed)
+                    yield return SButton.ControllerX;
+                if (controller.Buttons.Y == ButtonState.Pressed)
+                    yield return SButton.ControllerY;
+                if (controller.DPad.Up == ButtonState.Pressed)
+                    yield return SButton.DPadUp;
+                if (controller.DPad.Down == ButtonState.Pressed)
+                    yield return SButton.DPadDown;
+                if (controller.DPad.Left == ButtonState.Pressed)
+                    yield return SButton.DPadLeft;
+                if (controller.DPad.Right == ButtonState.Pressed)
+                    yield return SButton.DPadRight;
+                if (controller.Triggers.Left > 0.2f)
+                    yield return SButton.LeftTrigger;
+                if (controller.Triggers.Right > 0.2f)
+                    yield return SButton.RightTrigger;
+            }
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Input/InputStatus.cs b/src/SMAPI/Framework/Input/InputStatus.cs
new file mode 100644
index 000000000..99b0006cc
--- /dev/null
+++ b/src/SMAPI/Framework/Input/InputStatus.cs
@@ -0,0 +1,29 @@
+namespace StardewModdingAPI.Framework.Input
+{
+    /// <summary>The input status for a button during an update frame.</summary>
+    internal enum InputStatus
+    {
+        /// <summary>The button was neither pressed, held, nor released.</summary>
+        None,
+
+        /// <summary>The button was pressed in this frame.</summary>
+        Pressed,
+
+        /// <summary>The button has been held since the last frame.</summary>
+        Held,
+
+        /// <summary>The button was released in this frame.</summary>
+        Released
+    }
+
+    /// <summary>Extension methods for <see cref="InputStatus"/>.</summary>
+    internal static class InputStatusExtensions
+    {
+        /// <summary>Whether the button was pressed or held.</summary>
+        /// <param name="status">The button status.</param>
+        public static bool IsDown(this InputStatus status)
+        {
+            return status == InputStatus.Held || status == InputStatus.Pressed;
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/SGame.cs b/src/SMAPI/Framework/SGame.cs
index 0a614f17b..4a17926ce 100644
--- a/src/SMAPI/Framework/SGame.cs
+++ b/src/SMAPI/Framework/SGame.cs
@@ -10,6 +10,7 @@
 using Microsoft.Xna.Framework.Graphics;
 using Microsoft.Xna.Framework.Input;
 using StardewModdingAPI.Events;
+using StardewModdingAPI.Framework.Input;
 using StardewModdingAPI.Framework.Reflection;
 using StardewModdingAPI.Framework.Utilities;
 using StardewValley;
@@ -53,20 +54,8 @@ internal class SGame : Game1
         /****
         ** Game state
         ****/
-        /// <summary>A record of the buttons pressed as of the previous tick.</summary>
-        private SButton[] PreviousPressedButtons = new SButton[0];
-
-        /// <summary>A record of the keyboard state (i.e. the up/down state for each button) as of the previous tick.</summary>
-        private KeyboardState PreviousKeyState;
-
-        /// <summary>A record of the controller state (i.e. the up/down state for each button) as of the previous tick.</summary>
-        private GamePadState PreviousControllerState;
-
-        /// <summary>A record of the mouse state (i.e. the cursor position, scroll amount, and the up/down state for each button) as of the previous tick.</summary>
-        private MouseState PreviousMouseState;
-
-        /// <summary>The previous mouse position on the screen adjusted for the zoom level.</summary>
-        private Point PreviousMousePosition;
+        /// <summary>The player input as of the previous tick.</summary>
+        private InputState PreviousInput = new InputState();
 
         /// <summary>The window size value at last check.</summary>
         private Point PreviousWindowSize;
@@ -348,34 +337,17 @@ protected override void Update(GameTime gameTime)
                 *********/
                 if (Game1.game1.IsActive)
                 {
-                    // get latest state
-                    KeyboardState keyState;
-                    GamePadState controllerState;
-                    MouseState mouseState;
-                    Point mousePosition;
+                    // get input state
+                    InputState inputState;
                     try
                     {
-                        keyState = Keyboard.GetState();
-                        controllerState = GamePad.GetState(PlayerIndex.One);
-                        mouseState = Mouse.GetState();
-                        mousePosition = new Point(Game1.getMouseX(), Game1.getMouseY());
+                        inputState = InputState.GetState(this.PreviousInput);
                     }
                     catch (InvalidOperationException) // GetState() may crash for some players if window doesn't have focus but game1.IsActive == true
                     {
-                        keyState = this.PreviousKeyState;
-                        controllerState = this.PreviousControllerState;
-                        mouseState = this.PreviousMouseState;
-                        mousePosition = this.PreviousMousePosition;
+                        inputState = this.PreviousInput;
                     }
 
-                    // analyse state
-                    SButton[] currentlyPressedKeys = this.GetPressedButtons(keyState, mouseState, controllerState).ToArray();
-                    SButton[] previousPressedKeys = this.PreviousPressedButtons;
-                    SButton[] framePressedKeys = currentlyPressedKeys.Except(previousPressedKeys).ToArray();
-                    SButton[] frameReleasedKeys = previousPressedKeys.Except(currentlyPressedKeys).ToArray();
-                    bool isUseToolButton = Game1.options.useToolButton.Any(p => framePressedKeys.Contains(p.ToSButton()));
-                    bool isActionButton = !isUseToolButton && Game1.options.actionButton.Any(p => framePressedKeys.Contains(p.ToSButton()));
-
                     // get cursor position
                     ICursorPosition cursor;
                     {
@@ -388,60 +360,58 @@ protected override void Update(GameTime gameTime)
                         cursor = new CursorPosition(screenPixels, tile, grabTile);
                     }
 
-                    // raise button pressed
-                    foreach (SButton button in framePressedKeys)
+                    // raise input events
+                    foreach (var pair in inputState.ActiveButtons)
                     {
-                        InputEvents.InvokeButtonPressed(this.Monitor, button, cursor, isActionButton, isUseToolButton);
+                        SButton button = pair.Key;
+                        InputStatus status = pair.Value;
 
-                        // legacy events
-                        if (button.TryGetKeyboard(out Keys key))
-                        {
-                            if (key != Keys.None)
-                                ControlEvents.InvokeKeyPressed(this.Monitor, key);
-                        }
-                        else if (button.TryGetController(out Buttons controllerButton))
+                        if (status == InputStatus.Pressed)
                         {
-                            if (controllerButton == Buttons.LeftTrigger || controllerButton == Buttons.RightTrigger)
-                                ControlEvents.InvokeTriggerPressed(this.Monitor, controllerButton, controllerButton == Buttons.LeftTrigger ? controllerState.Triggers.Left : controllerState.Triggers.Right);
-                            else
-                                ControlEvents.InvokeButtonPressed(this.Monitor, controllerButton);
-                        }
-                    }
+                            InputEvents.InvokeButtonPressed(this.Monitor, button, cursor, button.IsActionButton(), button.IsUseToolButton());
 
-                    // raise button released
-                    foreach (SButton button in frameReleasedKeys)
-                    {
-                        bool wasUseToolButton = (from opt in Game1.options.useToolButton let optButton = opt.ToSButton() where optButton == button && framePressedKeys.Contains(optButton) select optButton).Any();
-                        bool wasActionButton = !wasUseToolButton && (from opt in Game1.options.actionButton let optButton = opt.ToSButton() where optButton == button && framePressedKeys.Contains(optButton) select optButton).Any();
-                        InputEvents.InvokeButtonReleased(this.Monitor, button, cursor, wasActionButton, wasUseToolButton);
-
-                        // legacy events
-                        if (button.TryGetKeyboard(out Keys key))
-                        {
-                            if (key != Keys.None)
-                                ControlEvents.InvokeKeyReleased(this.Monitor, key);
+                            // legacy events
+                            if (button.TryGetKeyboard(out Keys key))
+                            {
+                                if (key != Keys.None)
+                                    ControlEvents.InvokeKeyPressed(this.Monitor, key);
+                            }
+                            else if (button.TryGetController(out Buttons controllerButton))
+                            {
+                                if (controllerButton == Buttons.LeftTrigger || controllerButton == Buttons.RightTrigger)
+                                    ControlEvents.InvokeTriggerPressed(this.Monitor, controllerButton, controllerButton == Buttons.LeftTrigger ? inputState.ControllerState.Triggers.Left : inputState.ControllerState.Triggers.Right);
+                                else
+                                    ControlEvents.InvokeButtonPressed(this.Monitor, controllerButton);
+                            }
                         }
-                        else if (button.TryGetController(out Buttons controllerButton))
+                        else if (status == InputStatus.Released)
                         {
-                            if (controllerButton == Buttons.LeftTrigger || controllerButton == Buttons.RightTrigger)
-                                ControlEvents.InvokeTriggerReleased(this.Monitor, controllerButton, controllerButton == Buttons.LeftTrigger ? controllerState.Triggers.Left : controllerState.Triggers.Right);
-                            else
-                                ControlEvents.InvokeButtonReleased(this.Monitor, controllerButton);
+                            InputEvents.InvokeButtonReleased(this.Monitor, button, cursor, button.IsActionButton(), button.IsUseToolButton());
+
+                            // legacy events
+                            if (button.TryGetKeyboard(out Keys key))
+                            {
+                                if (key != Keys.None)
+                                    ControlEvents.InvokeKeyReleased(this.Monitor, key);
+                            }
+                            else if (button.TryGetController(out Buttons controllerButton))
+                            {
+                                if (controllerButton == Buttons.LeftTrigger || controllerButton == Buttons.RightTrigger)
+                                    ControlEvents.InvokeTriggerReleased(this.Monitor, controllerButton, controllerButton == Buttons.LeftTrigger ? inputState.ControllerState.Triggers.Left : inputState.ControllerState.Triggers.Right);
+                                else
+                                    ControlEvents.InvokeButtonReleased(this.Monitor, controllerButton);
+                            }
                         }
                     }
 
                     // raise legacy state-changed events
-                    if (keyState != this.PreviousKeyState)
-                        ControlEvents.InvokeKeyboardChanged(this.Monitor, this.PreviousKeyState, keyState);
-                    if (mouseState != this.PreviousMouseState)
-                        ControlEvents.InvokeMouseChanged(this.Monitor, this.PreviousMouseState, mouseState, this.PreviousMousePosition, mousePosition);
+                    if (inputState.KeyboardState != this.PreviousInput.KeyboardState)
+                        ControlEvents.InvokeKeyboardChanged(this.Monitor, this.PreviousInput.KeyboardState, inputState.KeyboardState);
+                    if (inputState.MouseState != this.PreviousInput.MouseState)
+                        ControlEvents.InvokeMouseChanged(this.Monitor, this.PreviousInput.MouseState, inputState.MouseState, this.PreviousInput.MousePosition, inputState.MousePosition);
 
                     // track state
-                    this.PreviousMouseState = mouseState;
-                    this.PreviousMousePosition = mousePosition;
-                    this.PreviousKeyState = keyState;
-                    this.PreviousControllerState = controllerState;
-                    this.PreviousPressedButtons = currentlyPressedKeys;
+                    this.PreviousInput = inputState;
                 }
 
                 /*********
@@ -1304,67 +1274,7 @@ private void CleanupAfterReturnToTitle()
             this.PreviousSaveID = 0;
         }
 
-        /// <summary>Get the buttons pressed in the given stats.</summary>
-        /// <param name="keyboard">The keyboard state.</param>
-        /// <param name="mouse">The mouse state.</param>
-        /// <param name="controller">The controller state.</param>
-        private IEnumerable<SButton> GetPressedButtons(KeyboardState keyboard, MouseState mouse, GamePadState controller)
-        {
-            // keyboard
-            foreach (Keys key in keyboard.GetPressedKeys())
-                yield return key.ToSButton();
-
-            // mouse
-            if (mouse.LeftButton == ButtonState.Pressed)
-                yield return SButton.MouseLeft;
-            if (mouse.RightButton == ButtonState.Pressed)
-                yield return SButton.MouseRight;
-            if (mouse.MiddleButton == ButtonState.Pressed)
-                yield return SButton.MouseMiddle;
-            if (mouse.XButton1 == ButtonState.Pressed)
-                yield return SButton.MouseX1;
-            if (mouse.XButton2 == ButtonState.Pressed)
-                yield return SButton.MouseX2;
-
-            // controller
-            if (controller.IsConnected)
-            {
-                if (controller.Buttons.A == ButtonState.Pressed)
-                    yield return SButton.ControllerA;
-                if (controller.Buttons.B == ButtonState.Pressed)
-                    yield return SButton.ControllerB;
-                if (controller.Buttons.Back == ButtonState.Pressed)
-                    yield return SButton.ControllerBack;
-                if (controller.Buttons.BigButton == ButtonState.Pressed)
-                    yield return SButton.BigButton;
-                if (controller.Buttons.LeftShoulder == ButtonState.Pressed)
-                    yield return SButton.LeftShoulder;
-                if (controller.Buttons.LeftStick == ButtonState.Pressed)
-                    yield return SButton.LeftStick;
-                if (controller.Buttons.RightShoulder == ButtonState.Pressed)
-                    yield return SButton.RightShoulder;
-                if (controller.Buttons.RightStick == ButtonState.Pressed)
-                    yield return SButton.RightStick;
-                if (controller.Buttons.Start == ButtonState.Pressed)
-                    yield return SButton.ControllerStart;
-                if (controller.Buttons.X == ButtonState.Pressed)
-                    yield return SButton.ControllerX;
-                if (controller.Buttons.Y == ButtonState.Pressed)
-                    yield return SButton.ControllerY;
-                if (controller.DPad.Up == ButtonState.Pressed)
-                    yield return SButton.DPadUp;
-                if (controller.DPad.Down == ButtonState.Pressed)
-                    yield return SButton.DPadDown;
-                if (controller.DPad.Left == ButtonState.Pressed)
-                    yield return SButton.DPadLeft;
-                if (controller.DPad.Right == ButtonState.Pressed)
-                    yield return SButton.DPadRight;
-                if (controller.Triggers.Left > 0.2f)
-                    yield return SButton.LeftTrigger;
-                if (controller.Triggers.Right > 0.2f)
-                    yield return SButton.RightTrigger;
-            }
-        }
+
 
         /// <summary>Get the player inventory changes between two states.</summary>
         /// <param name="current">The player's current inventory.</param>
diff --git a/src/SMAPI/SButton.cs b/src/SMAPI/SButton.cs
index bd6635c70..3f95169ae 100644
--- a/src/SMAPI/SButton.cs
+++ b/src/SMAPI/SButton.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Linq;
 using Microsoft.Xna.Framework.Input;
 using StardewValley;
 
@@ -683,5 +684,19 @@ public static bool TryGetStardewInput(this SButton input, out InputButton button
             button = default(InputButton);
             return false;
         }
+
+        /// <summary>Get whether the given button is equivalent to <see cref="Options.useToolButton"/>.</summary>
+        /// <param name="input">The button.</param>
+        public static bool IsUseToolButton(this SButton input)
+        {
+            return input == SButton.ControllerX || Game1.options.useToolButton.Any(p => p.ToSButton() == input);
+        }
+
+        /// <summary>Get whether the given button is equivalent to <see cref="Options.actionButton"/>.</summary>
+        /// <param name="input">The button.</param>
+        public static bool IsActionButton(this SButton input)
+        {
+            return input == SButton.ControllerA || Game1.options.actionButton.Any(p => p.ToSButton() == input);
+        }
     }
 }
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index 0625711c6..e02e1ab4d 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -86,6 +86,8 @@
       <Link>Properties\GlobalAssemblyInfo.cs</Link>
     </Compile>
     <Compile Include="Framework\Content\ContentCache.cs" />
+    <Compile Include="Framework\Input\InputState.cs" />
+    <Compile Include="Framework\Input\InputStatus.cs" />
     <Compile Include="Framework\LegacyManifestVersion.cs" />
     <Compile Include="Framework\Models\ModCompatibility.cs" />
     <Compile Include="Framework\ModLoading\Finders\EventFinder.cs" />

From 568ba2757e0b2947a8578128ff8f0a70eb075b38 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Fri, 19 Jan 2018 21:13:23 -0500
Subject: [PATCH 11/21] fix events being raised while the game is loading a
 save (#424)

---
 docs/release-notes.md        | 4 +++-
 src/SMAPI/Framework/SGame.cs | 7 +++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index dd9f03525..d55ba5a14 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,15 +1,17 @@
 # Release notes
 ## 2.4
 * For players:
+  * Fixed graphical corruption in rare cases.
   * Fixed error parsing `config.json` files containing curly quotes.
   * Fixed error parsing JSON files generated on another platform.
-  * Fixed rare issues caused by mods reloading core assemblies, which is no longer allowed.
+  * Fixed error parsing some JSON files after mods reload core assemblies, which is no longer allowed.
 
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.
 
 * For modders:
   * Added `SButton` `IsActionButton()` and `IsUseToolButton()` extensions.
+  * Fixed events being raised while the game is loading a save file.
   * Fixed input events not recognising controller input as an action or use-tool button.
   * Fixed input events setting the same `IsActionButton` and `IsUseToolButton` values for all buttons pressed in an update tick.
   * Fixed semantic versions always ignoring `-0` tag.
diff --git a/src/SMAPI/Framework/SGame.cs b/src/SMAPI/Framework/SGame.cs
index 4a17926ce..2eb2da99d 100644
--- a/src/SMAPI/Framework/SGame.cs
+++ b/src/SMAPI/Framework/SGame.cs
@@ -229,6 +229,13 @@ protected override void Update(GameTime gameTime)
                     return;
                 }
 
+                // game is asynchronously loading a save, block mod events to avoid conflicts
+                if (Game1.gameMode == Game1.loadingMode)
+                {
+                    base.Update(gameTime);
+                    return;
+                }
+
                 /*********
                 ** Save events + suppress events during save
                 *********/

From 32defd60e2e1ce4d153c567f75343111e4597684 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Fri, 19 Jan 2018 21:24:22 -0500
Subject: [PATCH 12/21] tweak new script, document workaround, update release
 notes

---
 docs/release-notes.md      | 1 +
 src/SMAPI/unix-launcher.sh | 5 ++++-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 4b6a7ba5e..2f136509b 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -4,6 +4,7 @@
   * SMAPI now fixes curly quotes in `config.json` if possible.
   * Fixed semantic versions always ignoring `-0` tag.
   * Fixed rare issues caused by assembly references being incorrectly loaded twice.
+  * Fixed error on Linux loading SMAPI when the default terminal is set to Terminator.
 
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.
diff --git a/src/SMAPI/unix-launcher.sh b/src/SMAPI/unix-launcher.sh
index 30021470f..2542a2863 100644
--- a/src/SMAPI/unix-launcher.sh
+++ b/src/SMAPI/unix-launcher.sh
@@ -63,7 +63,10 @@ else
 
     # open SMAPI in terminal
     if $COMMAND x-terminal-emulator 2>/dev/null; then
-        if [[ " $(readlink $(readlink /usr/bin/x-terminal-emulator))" == *"/terminator" ]]; then
+        # Terminator converts -e to -x when used through x-terminal-emulator for some reason (per
+        # `man terminator`), which causes an "unable to find shell" error. If x-terminal-emulator
+        # is mapped to Terminator, invoke it directly instead.
+        if [[ "$(readlink -e $(which x-terminal-emulator))" == *"/terminator" ]]; then
             terminator -e "$LAUNCHER"
         else
             x-terminal-emulator -e "$LAUNCHER"

From 9636d5b3aac99459e5933bc4fa6ddb8ca84917af Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 21:26:21 -0500
Subject: [PATCH 13/21] encapsulate common JSON converter code, improve parse
 errors (#423)

---
 docs/release-notes.md                         |  1 +
 .../Framework/Serialisation/ColorConverter.cs | 92 ++++++-------------
 .../Framework/Serialisation/JsonHelper.cs     | 13 ++-
 .../Framework/Serialisation/PointConverter.cs | 83 +++++------------
 .../Serialisation/RectangleConverter.cs       | 91 ++++++------------
 .../Serialisation/SimpleReadOnlyConverter.cs  | 77 ++++++++++++++++
 src/SMAPI/StardewModdingAPI.csproj            |  1 +
 7 files changed, 168 insertions(+), 190 deletions(-)
 create mode 100644 src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 27110bff6..31ceb7dec 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -12,6 +12,7 @@
 
 * For modders:
   * Added `SButton` `IsActionButton()` and `IsUseToolButton()` extensions.
+  * Improved JSON parse errors to provide more useful info for troubleshooting.
   * Fixed events being raised while the game is loading a save file.
   * Fixed input events not recognising controller input as an action or use-tool button.
   * Fixed input events setting the same `IsActionButton` and `IsUseToolButton` values for all buttons pressed in an update tick.
diff --git a/src/SMAPI/Framework/Serialisation/ColorConverter.cs b/src/SMAPI/Framework/Serialisation/ColorConverter.cs
index d29f73b8c..d2315a004 100644
--- a/src/SMAPI/Framework/Serialisation/ColorConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/ColorConverter.cs
@@ -1,82 +1,46 @@
 using System;
 using Microsoft.Xna.Framework;
-using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
 namespace StardewModdingAPI.Framework.Serialisation
 {
     /// <summary>Handles deserialisation of <see cref="Color"/> for crossplatform compatibility.</summary>
-    internal class ColorConverter : JsonConverter
+    /// <remarks>
+    /// - Linux/Mac format: { "B": 76, "G": 51, "R": 25, "A": 102 }
+    /// - Windows format:   "26, 51, 76, 102"
+    /// </remarks>
+    internal class ColorConverter : SimpleReadOnlyConverter<Color>
     {
         /*********
-        ** Accessors
+        ** Protected methods
         *********/
-        /// <summary>Whether this converter can write JSON.</summary>
-        public override bool CanWrite => false;
-
-
-        /*********
-        ** Public methods
-        *********/
-        /// <summary>Get whether this instance can convert the specified object type.</summary>
-        /// <param name="objectType">The object type.</param>
-        public override bool CanConvert(Type objectType)
+        /// <summary>Read a JSON object.</summary>
+        /// <param name="obj">The JSON object to read.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Color ReadObject(JObject obj, string path)
         {
-            return objectType == typeof(Color);
-        }
-
-        /// <summary>Reads the JSON representation of the object.</summary>
-        /// <param name="reader">The JSON reader.</param>
-        /// <param name="objectType">The object type.</param>
-        /// <param name="existingValue">The object being read.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
-        {
-            //    Linux/Mac: { "B": 76, "G": 51, "R": 25, "A": 102 }
-            //    Windows:   "26, 51, 76, 102"
-            JToken token = JToken.Load(reader);
-            switch (token.Type)
-            {
-                case JTokenType.Object:
-                    {
-                        JObject obj = (JObject)token;
-                        int r = obj.Value<int>(nameof(Color.R));
-                        int g = obj.Value<int>(nameof(Color.G));
-                        int b = obj.Value<int>(nameof(Color.B));
-                        int a = obj.Value<int>(nameof(Color.A));
-                        return new Color(r, g, b, a);
-                    }
-
-                case JTokenType.String:
-                    {
-                        string str = token.Value<string>();
-                        if (string.IsNullOrWhiteSpace(str))
-                            return null;
-
-                        string[] parts = str.Split(',');
-                        if (parts.Length != 4)
-                            throw new SParseException($"Can't parse {typeof(Color).Name} from {token.Path}, invalid value '{str}'.");
-
-                        int r = Convert.ToInt32(parts[0]);
-                        int g = Convert.ToInt32(parts[1]);
-                        int b = Convert.ToInt32(parts[2]);
-                        int a = Convert.ToInt32(parts[3]);
-                        return new Color(r, g, b, a);
-                    }
-
-                default:
-                    throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, must be an object or string.");
-            }
+            int r = obj.Value<int>(nameof(Color.R));
+            int g = obj.Value<int>(nameof(Color.G));
+            int b = obj.Value<int>(nameof(Color.B));
+            int a = obj.Value<int>(nameof(Color.A));
+            return new Color(r, g, b, a);
         }
 
-        /// <summary>Writes the JSON representation of the object.</summary>
-        /// <param name="writer">The JSON writer.</param>
-        /// <param name="value">The value.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Color ReadString(string str, string path)
         {
-            throw new InvalidOperationException("This converter does not write JSON.");
+            string[] parts = str.Split(',');
+            if (parts.Length != 4)
+                throw new SParseException($"Can't parse {typeof(Color).Name} from invalid value '{str}' (path: {path}).");
+
+            int r = Convert.ToInt32(parts[0]);
+            int g = Convert.ToInt32(parts[1]);
+            int b = Convert.ToInt32(parts[2]);
+            int a = Convert.ToInt32(parts[3]);
+            return new Color(r, g, b, a);
         }
     }
 }
diff --git a/src/SMAPI/Framework/Serialisation/JsonHelper.cs b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
index f66f9dfb0..90a6d2584 100644
--- a/src/SMAPI/Framework/Serialisation/JsonHelper.cs
+++ b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
@@ -63,11 +63,16 @@ public TModel ReadJsonFile<TModel>(string fullPath)
             {
                 return this.Deserialise<TModel>(json);
             }
-            catch (JsonReaderException ex)
+            catch (Exception ex)
             {
-                string error = $"The file at {fullPath} doesn't seem to be valid JSON.";
-                if (json.Contains("“") || json.Contains("”"))
-                    error += " Found curly quotes in the text; note that only straight quotes are allowed in JSON.";
+                string error = $"Can't parse JSON file at {fullPath}.";
+
+                if (ex is JsonReaderException)
+                {
+                    error += " This doesn't seem to be valid JSON.";
+                    if (json.Contains("“") || json.Contains("”"))
+                        error += " Found curly quotes in the text; note that only straight quotes are allowed in JSON.";
+                }
                 error += $"\nTechnical details: {ex.Message}";
                 throw new JsonReaderException(error);
             }
diff --git a/src/SMAPI/Framework/Serialisation/PointConverter.cs b/src/SMAPI/Framework/Serialisation/PointConverter.cs
index d35660bef..bdcefaa56 100644
--- a/src/SMAPI/Framework/Serialisation/PointConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/PointConverter.cs
@@ -1,79 +1,42 @@
 using System;
 using Microsoft.Xna.Framework;
-using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
 namespace StardewModdingAPI.Framework.Serialisation
 {
     /// <summary>Handles deserialisation of <see cref="PointConverter"/> for crossplatform compatibility.</summary>
-    internal class PointConverter : JsonConverter
+    /// <remarks>
+    /// - Linux/Mac format: { "X": 1, "Y": 2 }
+    /// - Windows format:   "1, 2"
+    /// </remarks>
+    internal class PointConverter : SimpleReadOnlyConverter<Point>
     {
         /*********
-        ** Accessors
+        ** Protected methods
         *********/
-        /// <summary>Whether this converter can write JSON.</summary>
-        public override bool CanWrite => false;
-
-
-        /*********
-        ** Public methods
-        *********/
-        /// <summary>Get whether this instance can convert the specified object type.</summary>
-        /// <param name="objectType">The object type.</param>
-        public override bool CanConvert(Type objectType)
+        /// <summary>Read a JSON object.</summary>
+        /// <param name="obj">The JSON object to read.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Point ReadObject(JObject obj, string path)
         {
-            return objectType == typeof(Point);
+            int x = obj.Value<int>(nameof(Point.X));
+            int y = obj.Value<int>(nameof(Point.Y));
+            return new Point(x, y);
         }
 
-        /// <summary>Reads the JSON representation of the object.</summary>
-        /// <param name="reader">The JSON reader.</param>
-        /// <param name="objectType">The object type.</param>
-        /// <param name="existingValue">The object being read.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Point ReadString(string str, string path)
         {
-            // point
-            //    Linux/Mac: { "X": 1, "Y": 2 }
-            //    Windows:   "1, 2"
-            JToken token = JToken.Load(reader);
-            switch (token.Type)
-            {
-                case JTokenType.Object:
-                    {
-                        JObject obj = (JObject)token;
-                        int x = obj.Value<int>(nameof(Point.X));
-                        int y = obj.Value<int>(nameof(Point.Y));
-                        return new Point(x, y);
-                    }
-
-                case JTokenType.String:
-                    {
-                        string str = token.Value<string>();
-                        if (string.IsNullOrWhiteSpace(str))
-                            return null;
+            string[] parts = str.Split(',');
+            if (parts.Length != 2)
+                throw new SParseException($"Can't parse {typeof(Point).Name} from invalid value '{str}' (path: {path}).");
 
-                        string[] parts = str.Split(',');
-                        if (parts.Length != 2)
-                            throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, invalid value '{str}'.");
-
-                        int x = Convert.ToInt32(parts[0]);
-                        int y = Convert.ToInt32(parts[1]);
-                        return new Point(x, y);
-                    }
-
-                default:
-                    throw new SParseException($"Can't parse {typeof(Point).Name} from {token.Path}, must be an object or string.");
-            }
-        }
-
-        /// <summary>Writes the JSON representation of the object.</summary>
-        /// <param name="writer">The JSON writer.</param>
-        /// <param name="value">The value.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
-        {
-            throw new InvalidOperationException("This converter does not write JSON.");
+            int x = Convert.ToInt32(parts[0]);
+            int y = Convert.ToInt32(parts[1]);
+            return new Point(x, y);
         }
     }
 }
diff --git a/src/SMAPI/Framework/Serialisation/RectangleConverter.cs b/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
index 74df54e2b..bbf60435c 100644
--- a/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
@@ -1,84 +1,51 @@
 using System;
 using System.Text.RegularExpressions;
 using Microsoft.Xna.Framework;
-using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
 namespace StardewModdingAPI.Framework.Serialisation
 {
     /// <summary>Handles deserialisation of <see cref="Rectangle"/> for crossplatform compatibility.</summary>
-    internal class RectangleConverter : JsonConverter
+    /// <remarks>
+    /// - Linux/Mac format: { "X": 1, "Y": 2, "Width": 3, "Height": 4 }
+    /// - Windows format:   "{X:1 Y:2 Width:3 Height:4}"
+    /// </remarks>
+    internal class RectangleConverter : SimpleReadOnlyConverter<Rectangle>
     {
         /*********
-        ** Accessors
+        ** Protected methods
         *********/
-        /// <summary>Whether this converter can write JSON.</summary>
-        public override bool CanWrite => false;
-
-
-        /*********
-        ** Public methods
-        *********/
-        /// <summary>Get whether this instance can convert the specified object type.</summary>
-        /// <param name="objectType">The object type.</param>
-        public override bool CanConvert(Type objectType)
+        /// <summary>Read a JSON object.</summary>
+        /// <param name="obj">The JSON object to read.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Rectangle ReadObject(JObject obj, string path)
         {
-            return objectType == typeof(Rectangle);
+            int x = obj.Value<int>(nameof(Rectangle.X));
+            int y = obj.Value<int>(nameof(Rectangle.Y));
+            int width = obj.Value<int>(nameof(Rectangle.Width));
+            int height = obj.Value<int>(nameof(Rectangle.Height));
+            return new Rectangle(x, y, width, height);
         }
 
-        /// <summary>Reads the JSON representation of the object.</summary>
-        /// <param name="reader">The JSON reader.</param>
-        /// <param name="objectType">The object type.</param>
-        /// <param name="existingValue">The object being read.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override Rectangle ReadString(string str, string path)
         {
-            //    Linux/Mac: { "X": 1, "Y": 2, "Width": 3, "Height": 4 }
-            //    Windows:   "{X:1 Y:2 Width:3 Height:4}"
-            JToken token = JToken.Load(reader);
-            switch (token.Type)
-            {
-                case JTokenType.Object:
-                    {
-                        JObject obj = (JObject)token;
-                        int x = obj.Value<int>(nameof(Rectangle.X));
-                        int y = obj.Value<int>(nameof(Rectangle.Y));
-                        int width = obj.Value<int>(nameof(Rectangle.Width));
-                        int height = obj.Value<int>(nameof(Rectangle.Height));
-                        return new Rectangle(x, y, width, height);
-                    }
-
-                case JTokenType.String:
-                    {
-                        string str = token.Value<string>();
-                        if (string.IsNullOrWhiteSpace(str))
-                            return Rectangle.Empty;
+            if (string.IsNullOrWhiteSpace(str))
+                return Rectangle.Empty;
 
-                        var match = Regex.Match(str, @"^\{X:(?<x>\d+) Y:(?<y>\d+) Width:(?<width>\d+) Height:(?<height>\d+)\}$");
-                        if (!match.Success)
-                            throw new SParseException($"Can't parse {typeof(Rectangle).Name} from {reader.Path}, invalid string format.");
+            var match = Regex.Match(str, @"^\{X:(?<x>\d+) Y:(?<y>\d+) Width:(?<width>\d+) Height:(?<height>\d+)\}$");
+            if (!match.Success)
+                throw new SParseException($"Can't parse {typeof(Rectangle).Name} from invalid value '{str}' (path: {path}).");
 
-                        int x = Convert.ToInt32(match.Groups["x"].Value);
-                        int y = Convert.ToInt32(match.Groups["y"].Value);
-                        int width = Convert.ToInt32(match.Groups["width"].Value);
-                        int height = Convert.ToInt32(match.Groups["height"].Value);
+            int x = Convert.ToInt32(match.Groups["x"].Value);
+            int y = Convert.ToInt32(match.Groups["y"].Value);
+            int width = Convert.ToInt32(match.Groups["width"].Value);
+            int height = Convert.ToInt32(match.Groups["height"].Value);
 
-                        return new Rectangle(x, y, width, height);
-                    }
-
-                default:
-                    throw new SParseException($"Can't parse {typeof(Rectangle).Name} from {reader.Path}, must be an object or string.");
-            }
-        }
-
-        /// <summary>Writes the JSON representation of the object.</summary>
-        /// <param name="writer">The JSON writer.</param>
-        /// <param name="value">The value.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
-        {
-            throw new InvalidOperationException("This converter does not write JSON.");
+            return new Rectangle(x, y, width, height);
         }
     }
 }
diff --git a/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs b/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs
new file mode 100644
index 000000000..9308456ba
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs
@@ -0,0 +1,77 @@
+using System;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Exceptions;
+
+namespace StardewModdingAPI.Framework.Serialisation
+{
+    /// <summary>The base implementation for simplified converters which deserialise <typeparamref name="T"/> without overriding serialisation.</summary>
+    /// <typeparam name="T">The type to deserialise.</typeparam>
+    internal abstract class SimpleReadOnlyConverter<T> : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(T);
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+
+        /// <summary>Reads the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            string path = reader.Path;
+            switch (reader.TokenType)
+            {
+                case JsonToken.StartObject:
+                    return this.ReadObject(JObject.Load(reader), path);
+                case JsonToken.String:
+                    return this.ReadString(JToken.Load(reader).Value<string>(), path);
+                default:
+                    throw new SParseException($"Can't parse {typeof(T).Name} from {reader.TokenType} (path: {reader.Path}).");
+            }
+        }
+
+
+        /*********
+        ** Protected methods
+        *********/
+        /// <summary>Read a JSON object.</summary>
+        /// <param name="obj">The JSON object to read.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected virtual T ReadObject(JObject obj, string path)
+        {
+            throw new SParseException($"Can't parse {typeof(T).Name} from object (path: {path}).");
+        }
+
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected virtual T ReadString(string str, string path)
+        {
+            throw new SParseException($"Can't parse {typeof(T).Name} from string (path: {path}).");
+        }
+    }
+}
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index e02e1ab4d..dd4f61343 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -110,6 +110,7 @@
     <Compile Include="Framework\Exceptions\SAssemblyLoadFailedException.cs" />
     <Compile Include="Framework\ModLoading\AssemblyLoadStatus.cs" />
     <Compile Include="Framework\Reflection\InterfaceProxyBuilder.cs" />
+    <Compile Include="Framework\Serialisation\SimpleReadOnlyConverter.cs" />
     <Compile Include="Framework\Serialisation\RectangleConverter.cs" />
     <Compile Include="Framework\Serialisation\ColorConverter.cs" />
     <Compile Include="Framework\Serialisation\PointConverter.cs" />

From 894fd25a18cd5ddf31a905860aca95c438894efd Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 21:29:47 -0500
Subject: [PATCH 14/21] move converters into namespace

---
 src/SMAPI/Framework/Models/Manifest.cs                 |  1 +
 src/SMAPI/Framework/Models/ModDataRecord.cs            |  1 +
 .../Serialisation/{ => Converters}/ColorConverter.cs   |  2 +-
 .../Serialisation/{ => Converters}/PointConverter.cs   |  2 +-
 .../{ => Converters}/RectangleConverter.cs             |  2 +-
 .../Serialisation/{ => Converters}/SFieldConverter.cs  |  2 +-
 .../{ => Converters}/StringEnumConverter.cs            |  2 +-
 src/SMAPI/Framework/Serialisation/JsonHelper.cs        |  1 +
 src/SMAPI/StardewModdingAPI.csproj                     | 10 +++++-----
 9 files changed, 13 insertions(+), 10 deletions(-)
 rename src/SMAPI/Framework/Serialisation/{ => Converters}/ColorConverter.cs (96%)
 rename src/SMAPI/Framework/Serialisation/{ => Converters}/PointConverter.cs (96%)
 rename src/SMAPI/Framework/Serialisation/{ => Converters}/RectangleConverter.cs (97%)
 rename src/SMAPI/Framework/Serialisation/{ => Converters}/SFieldConverter.cs (98%)
 rename src/SMAPI/Framework/Serialisation/{ => Converters}/StringEnumConverter.cs (91%)

diff --git a/src/SMAPI/Framework/Models/Manifest.cs b/src/SMAPI/Framework/Models/Manifest.cs
index b85787e50..c362be9b3 100644
--- a/src/SMAPI/Framework/Models/Manifest.cs
+++ b/src/SMAPI/Framework/Models/Manifest.cs
@@ -1,6 +1,7 @@
 using System.Collections.Generic;
 using Newtonsoft.Json;
 using StardewModdingAPI.Framework.Serialisation;
+using StardewModdingAPI.Framework.Serialisation.Converters;
 
 namespace StardewModdingAPI.Framework.Models
 {
diff --git a/src/SMAPI/Framework/Models/ModDataRecord.cs b/src/SMAPI/Framework/Models/ModDataRecord.cs
index c6a12188a..fa6fc68df 100644
--- a/src/SMAPI/Framework/Models/ModDataRecord.cs
+++ b/src/SMAPI/Framework/Models/ModDataRecord.cs
@@ -2,6 +2,7 @@
 using System.Linq;
 using Newtonsoft.Json;
 using StardewModdingAPI.Framework.Serialisation;
+using StardewModdingAPI.Framework.Serialisation.Converters;
 
 namespace StardewModdingAPI.Framework.Models
 {
diff --git a/src/SMAPI/Framework/Serialisation/ColorConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs
similarity index 96%
rename from src/SMAPI/Framework/Serialisation/ColorConverter.cs
rename to src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs
index d2315a004..7f830a0eb 100644
--- a/src/SMAPI/Framework/Serialisation/ColorConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs
@@ -3,7 +3,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation
+namespace StardewModdingAPI.Framework.Serialisation.Converters
 {
     /// <summary>Handles deserialisation of <see cref="Color"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/PointConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs
similarity index 96%
rename from src/SMAPI/Framework/Serialisation/PointConverter.cs
rename to src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs
index bdcefaa56..87d0ff343 100644
--- a/src/SMAPI/Framework/Serialisation/PointConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs
@@ -3,7 +3,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation
+namespace StardewModdingAPI.Framework.Serialisation.Converters
 {
     /// <summary>Handles deserialisation of <see cref="PointConverter"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/RectangleConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs
similarity index 97%
rename from src/SMAPI/Framework/Serialisation/RectangleConverter.cs
rename to src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs
index bbf60435c..7cf912a7b 100644
--- a/src/SMAPI/Framework/Serialisation/RectangleConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs
@@ -4,7 +4,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation
+namespace StardewModdingAPI.Framework.Serialisation.Converters
 {
     /// <summary>Handles deserialisation of <see cref="Rectangle"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/SFieldConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
similarity index 98%
rename from src/SMAPI/Framework/Serialisation/SFieldConverter.cs
rename to src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
index 6e0685997..693534919 100644
--- a/src/SMAPI/Framework/Serialisation/SFieldConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
@@ -5,7 +5,7 @@
 using StardewModdingAPI.Framework.Exceptions;
 using StardewModdingAPI.Framework.Models;
 
-namespace StardewModdingAPI.Framework.Serialisation
+namespace StardewModdingAPI.Framework.Serialisation.Converters
 {
     /// <summary>Overrides how SMAPI reads and writes <see cref="ISemanticVersion"/> and <see cref="IManifestDependency"/> fields.</summary>
     internal class SFieldConverter : JsonConverter
diff --git a/src/SMAPI/Framework/Serialisation/StringEnumConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs
similarity index 91%
rename from src/SMAPI/Framework/Serialisation/StringEnumConverter.cs
rename to src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs
index 7afe86cdf..0a612c74d 100644
--- a/src/SMAPI/Framework/Serialisation/StringEnumConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs
@@ -1,7 +1,7 @@
 using System;
 using Newtonsoft.Json.Converters;
 
-namespace StardewModdingAPI.Framework.Serialisation
+namespace StardewModdingAPI.Framework.Serialisation.Converters
 {
     /// <summary>A variant of <see cref="StringEnumConverter"/> which only converts a specified enum.</summary>
     /// <typeparam name="T">The enum type.</typeparam>
diff --git a/src/SMAPI/Framework/Serialisation/JsonHelper.cs b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
index 90a6d2584..1253f242d 100644
--- a/src/SMAPI/Framework/Serialisation/JsonHelper.cs
+++ b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
@@ -3,6 +3,7 @@
 using System.IO;
 using Microsoft.Xna.Framework.Input;
 using Newtonsoft.Json;
+using StardewModdingAPI.Framework.Serialisation.Converters;
 
 namespace StardewModdingAPI.Framework.Serialisation
 {
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index dd4f61343..f5060e2db 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -111,9 +111,9 @@
     <Compile Include="Framework\ModLoading\AssemblyLoadStatus.cs" />
     <Compile Include="Framework\Reflection\InterfaceProxyBuilder.cs" />
     <Compile Include="Framework\Serialisation\SimpleReadOnlyConverter.cs" />
-    <Compile Include="Framework\Serialisation\RectangleConverter.cs" />
-    <Compile Include="Framework\Serialisation\ColorConverter.cs" />
-    <Compile Include="Framework\Serialisation\PointConverter.cs" />
+    <Compile Include="Framework\Serialisation\Converters\RectangleConverter.cs" />
+    <Compile Include="Framework\Serialisation\Converters\ColorConverter.cs" />
+    <Compile Include="Framework\Serialisation\Converters\PointConverter.cs" />
     <Compile Include="Framework\Utilities\ContextHash.cs" />
     <Compile Include="IReflectedField.cs" />
     <Compile Include="IReflectedMethod.cs" />
@@ -181,8 +181,8 @@
     <Compile Include="Framework\SContentManager.cs" />
     <Compile Include="Framework\Exceptions\SParseException.cs" />
     <Compile Include="Framework\Serialisation\JsonHelper.cs" />
-    <Compile Include="Framework\Serialisation\StringEnumConverter.cs" />
-    <Compile Include="Framework\Serialisation\SFieldConverter.cs" />
+    <Compile Include="Framework\Serialisation\Converters\StringEnumConverter.cs" />
+    <Compile Include="Framework\Serialisation\Converters\SFieldConverter.cs" />
     <Compile Include="IAssetEditor.cs" />
     <Compile Include="IAssetInfo.cs" />
     <Compile Include="IAssetLoader.cs" />

From 454a2de2949deae90de4a936fb8fa7341d4a2990 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 21:45:21 -0500
Subject: [PATCH 15/21] split SFieldConverter into separate converters

---
 src/SMAPI/Framework/Models/Manifest.cs        |   9 +-
 src/SMAPI/Framework/Models/ModDataRecord.cs   |   7 +-
 .../Converters/SFieldConverter.cs             | 121 ------------------
 .../ColorConverter.cs                         |   2 +-
 .../PointConverter.cs                         |   2 +-
 .../RectangleConverter.cs                     |   2 +-
 .../Framework/Serialisation/JsonHelper.cs     |   3 +-
 .../ManifestDependencyArrayConverter.cs       |  60 +++++++++
 .../ModCompatibilityArrayConverter.cs         |  61 +++++++++
 .../SmapiConverters/ModDataIdConverter.cs     |  19 +++
 .../SemanticVersionConverter.cs               |  36 ++++++
 .../StringEnumConverter.cs                    |   2 +-
 src/SMAPI/StardewModdingAPI.csproj            |  13 +-
 13 files changed, 197 insertions(+), 140 deletions(-)
 delete mode 100644 src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
 rename src/SMAPI/Framework/Serialisation/{Converters => CrossplatformConverters}/ColorConverter.cs (95%)
 rename src/SMAPI/Framework/Serialisation/{Converters => CrossplatformConverters}/PointConverter.cs (95%)
 rename src/SMAPI/Framework/Serialisation/{Converters => CrossplatformConverters}/RectangleConverter.cs (96%)
 create mode 100644 src/SMAPI/Framework/Serialisation/SmapiConverters/ManifestDependencyArrayConverter.cs
 create mode 100644 src/SMAPI/Framework/Serialisation/SmapiConverters/ModCompatibilityArrayConverter.cs
 create mode 100644 src/SMAPI/Framework/Serialisation/SmapiConverters/ModDataIdConverter.cs
 create mode 100644 src/SMAPI/Framework/Serialisation/SmapiConverters/SemanticVersionConverter.cs
 rename src/SMAPI/Framework/Serialisation/{Converters => SmapiConverters}/StringEnumConverter.cs (91%)

diff --git a/src/SMAPI/Framework/Models/Manifest.cs b/src/SMAPI/Framework/Models/Manifest.cs
index c362be9b3..f97624069 100644
--- a/src/SMAPI/Framework/Models/Manifest.cs
+++ b/src/SMAPI/Framework/Models/Manifest.cs
@@ -1,7 +1,6 @@
 using System.Collections.Generic;
 using Newtonsoft.Json;
-using StardewModdingAPI.Framework.Serialisation;
-using StardewModdingAPI.Framework.Serialisation.Converters;
+using StardewModdingAPI.Framework.Serialisation.SmapiConverters;
 
 namespace StardewModdingAPI.Framework.Models
 {
@@ -21,18 +20,18 @@ internal class Manifest : IManifest
         public string Author { get; set; }
 
         /// <summary>The mod version.</summary>
-        [JsonConverter(typeof(SFieldConverter))]
+        [JsonConverter(typeof(SemanticVersionConverter))]
         public ISemanticVersion Version { get; set; }
 
         /// <summary>The minimum SMAPI version required by this mod, if any.</summary>
-        [JsonConverter(typeof(SFieldConverter))]
+        [JsonConverter(typeof(SemanticVersionConverter))]
         public ISemanticVersion MinimumApiVersion { get; set; }
 
         /// <summary>The name of the DLL in the directory that has the <see cref="IMod.Entry"/> method.</summary>
         public string EntryDll { get; set; }
 
         /// <summary>The other mods that must be loaded before this mod.</summary>
-        [JsonConverter(typeof(SFieldConverter))]
+        [JsonConverter(typeof(ManifestDependencyArrayConverter))]
         public IManifestDependency[] Dependencies { get; set; }
 
         /// <summary>The namespaced mod IDs to query for updates (like <c>Nexus:541</c>).</summary>
diff --git a/src/SMAPI/Framework/Models/ModDataRecord.cs b/src/SMAPI/Framework/Models/ModDataRecord.cs
index fa6fc68df..580acb706 100644
--- a/src/SMAPI/Framework/Models/ModDataRecord.cs
+++ b/src/SMAPI/Framework/Models/ModDataRecord.cs
@@ -1,8 +1,7 @@
 using System.Collections.Generic;
 using System.Linq;
 using Newtonsoft.Json;
-using StardewModdingAPI.Framework.Serialisation;
-using StardewModdingAPI.Framework.Serialisation.Converters;
+using StardewModdingAPI.Framework.Serialisation.SmapiConverters;
 
 namespace StardewModdingAPI.Framework.Models
 {
@@ -13,7 +12,7 @@ internal class ModDataRecord
         ** Accessors
         *********/
         /// <summary>The unique mod identifier.</summary>
-        [JsonConverter(typeof(SFieldConverter))]
+        [JsonConverter(typeof(ModDataIdConverter))]
         public ModDataID ID { get; set; }
 
         /// <summary>A value to inject into <see cref="IManifest.UpdateKeys"/> field if it's not already set.</summary>
@@ -23,7 +22,7 @@ internal class ModDataRecord
         public string AlternativeUrl { get; set; }
 
         /// <summary>The compatibility of given mod versions (if any).</summary>
-        [JsonConverter(typeof(SFieldConverter))]
+        [JsonConverter(typeof(ModCompatibilityArrayConverter))]
         public ModCompatibility[] Compatibility { get; set; } = new ModCompatibility[0];
 
         /// <summary>Map local versions to a semantic version for update checks.</summary>
diff --git a/src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs b/src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
deleted file mode 100644
index 693534919..000000000
--- a/src/SMAPI/Framework/Serialisation/Converters/SFieldConverter.cs
+++ /dev/null
@@ -1,121 +0,0 @@
-using System;
-using System.Collections.Generic;
-using Newtonsoft.Json;
-using Newtonsoft.Json.Linq;
-using StardewModdingAPI.Framework.Exceptions;
-using StardewModdingAPI.Framework.Models;
-
-namespace StardewModdingAPI.Framework.Serialisation.Converters
-{
-    /// <summary>Overrides how SMAPI reads and writes <see cref="ISemanticVersion"/> and <see cref="IManifestDependency"/> fields.</summary>
-    internal class SFieldConverter : JsonConverter
-    {
-        /*********
-        ** Accessors
-        *********/
-        /// <summary>Whether this converter can write JSON.</summary>
-        public override bool CanWrite => false;
-
-
-        /*********
-        ** Public methods
-        *********/
-        /// <summary>Get whether this instance can convert the specified object type.</summary>
-        /// <param name="objectType">The object type.</param>
-        public override bool CanConvert(Type objectType)
-        {
-            return
-                objectType == typeof(ISemanticVersion)
-                || objectType == typeof(IManifestDependency[])
-                || objectType == typeof(ModDataID)
-                || objectType == typeof(ModCompatibility[]);
-        }
-
-        /// <summary>Reads the JSON representation of the object.</summary>
-        /// <param name="reader">The JSON reader.</param>
-        /// <param name="objectType">The object type.</param>
-        /// <param name="existingValue">The object being read.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
-        {
-            // semantic version
-            if (objectType == typeof(ISemanticVersion))
-            {
-                JToken token = JToken.Load(reader);
-                switch (token.Type)
-                {
-                    case JTokenType.Object:
-                        {
-                            JObject obj = (JObject)token;
-                            int major = obj.Value<int>(nameof(ISemanticVersion.MajorVersion));
-                            int minor = obj.Value<int>(nameof(ISemanticVersion.MinorVersion));
-                            int patch = obj.Value<int>(nameof(ISemanticVersion.PatchVersion));
-                            string build = obj.Value<string>(nameof(ISemanticVersion.Build));
-                            return new LegacyManifestVersion(major, minor, patch, build);
-                        }
-
-                    case JTokenType.String:
-                        {
-                            string str = token.Value<string>();
-                            if (string.IsNullOrWhiteSpace(str))
-                                return null;
-                            if (!SemanticVersion.TryParse(str, out ISemanticVersion version))
-                                throw new SParseException($"Can't parse semantic version from invalid value '{str}', should be formatted like 1.2, 1.2.30, or 1.2.30-beta.");
-                            return version;
-                        }
-
-                    default:
-                        throw new SParseException($"Can't parse semantic version from {token.Type}, must be an object or string.");
-                }
-            }
-
-            // manifest dependencies
-            if (objectType == typeof(IManifestDependency[]))
-            {
-                List<IManifestDependency> result = new List<IManifestDependency>();
-                foreach (JObject obj in JArray.Load(reader).Children<JObject>())
-                {
-                    string uniqueID = obj.Value<string>(nameof(IManifestDependency.UniqueID));
-                    string minVersion = obj.Value<string>(nameof(IManifestDependency.MinimumVersion));
-                    bool required = obj.Value<bool?>(nameof(IManifestDependency.IsRequired)) ?? true;
-                    result.Add(new ManifestDependency(uniqueID, minVersion, required));
-                }
-                return result.ToArray();
-            }
-
-            // mod data ID
-            if (objectType == typeof(ModDataID))
-            {
-                JToken token = JToken.Load(reader);
-                return new ModDataID(token.Value<string>());
-            }
-
-            // mod compatibility records
-            if (objectType == typeof(ModCompatibility[]))
-            {
-                List<ModCompatibility> result = new List<ModCompatibility>();
-                foreach (JProperty property in JObject.Load(reader).Properties())
-                {
-                    string range = property.Name;
-                    ModStatus status = (ModStatus)Enum.Parse(typeof(ModStatus), property.Value.Value<string>(nameof(ModCompatibility.Status)));
-                    string reasonPhrase = property.Value.Value<string>(nameof(ModCompatibility.ReasonPhrase));
-
-                    result.Add(new ModCompatibility(range, status, reasonPhrase));
-                }
-                return result.ToArray();
-            }
-
-            // unknown
-            throw new NotSupportedException($"Unknown type '{objectType?.FullName}'.");
-        }
-
-        /// <summary>Writes the JSON representation of the object.</summary>
-        /// <param name="writer">The JSON writer.</param>
-        /// <param name="value">The value.</param>
-        /// <param name="serializer">The calling serializer.</param>
-        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
-        {
-            throw new InvalidOperationException("This converter does not write JSON.");
-        }
-    }
-}
diff --git a/src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/ColorConverter.cs
similarity index 95%
rename from src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs
rename to src/SMAPI/Framework/Serialisation/CrossplatformConverters/ColorConverter.cs
index 7f830a0eb..f4a2a26ed 100644
--- a/src/SMAPI/Framework/Serialisation/Converters/ColorConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/ColorConverter.cs
@@ -3,7 +3,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation.Converters
+namespace StardewModdingAPI.Framework.Serialisation.CrossplatformConverters
 {
     /// <summary>Handles deserialisation of <see cref="Color"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/PointConverter.cs
similarity index 95%
rename from src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs
rename to src/SMAPI/Framework/Serialisation/CrossplatformConverters/PointConverter.cs
index 87d0ff343..84c709891 100644
--- a/src/SMAPI/Framework/Serialisation/Converters/PointConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/PointConverter.cs
@@ -3,7 +3,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation.Converters
+namespace StardewModdingAPI.Framework.Serialisation.CrossplatformConverters
 {
     /// <summary>Handles deserialisation of <see cref="PointConverter"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/RectangleConverter.cs
similarity index 96%
rename from src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs
rename to src/SMAPI/Framework/Serialisation/CrossplatformConverters/RectangleConverter.cs
index 7cf912a7b..b89551e3c 100644
--- a/src/SMAPI/Framework/Serialisation/Converters/RectangleConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/CrossplatformConverters/RectangleConverter.cs
@@ -4,7 +4,7 @@
 using Newtonsoft.Json.Linq;
 using StardewModdingAPI.Framework.Exceptions;
 
-namespace StardewModdingAPI.Framework.Serialisation.Converters
+namespace StardewModdingAPI.Framework.Serialisation.CrossplatformConverters
 {
     /// <summary>Handles deserialisation of <see cref="Rectangle"/> for crossplatform compatibility.</summary>
     /// <remarks>
diff --git a/src/SMAPI/Framework/Serialisation/JsonHelper.cs b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
index 1253f242d..2e2a666e2 100644
--- a/src/SMAPI/Framework/Serialisation/JsonHelper.cs
+++ b/src/SMAPI/Framework/Serialisation/JsonHelper.cs
@@ -3,7 +3,8 @@
 using System.IO;
 using Microsoft.Xna.Framework.Input;
 using Newtonsoft.Json;
-using StardewModdingAPI.Framework.Serialisation.Converters;
+using StardewModdingAPI.Framework.Serialisation.CrossplatformConverters;
+using StardewModdingAPI.Framework.Serialisation.SmapiConverters;
 
 namespace StardewModdingAPI.Framework.Serialisation
 {
diff --git a/src/SMAPI/Framework/Serialisation/SmapiConverters/ManifestDependencyArrayConverter.cs b/src/SMAPI/Framework/Serialisation/SmapiConverters/ManifestDependencyArrayConverter.cs
new file mode 100644
index 000000000..6352e3672
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/SmapiConverters/ManifestDependencyArrayConverter.cs
@@ -0,0 +1,60 @@
+using System;
+using System.Collections.Generic;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Models;
+
+namespace StardewModdingAPI.Framework.Serialisation.SmapiConverters
+{
+    /// <summary>Handles deserialisation of <see cref="IManifestDependency"/> arrays.</summary>
+    internal class ManifestDependencyArrayConverter : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(IManifestDependency[]);
+        }
+
+
+        /*********
+        ** Protected methods
+        *********/
+        /// <summary>Read the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            List<IManifestDependency> result = new List<IManifestDependency>();
+            foreach (JObject obj in JArray.Load(reader).Children<JObject>())
+            {
+                string uniqueID = obj.Value<string>(nameof(IManifestDependency.UniqueID));
+                string minVersion = obj.Value<string>(nameof(IManifestDependency.MinimumVersion));
+                bool required = obj.Value<bool?>(nameof(IManifestDependency.IsRequired)) ?? true;
+                result.Add(new ManifestDependency(uniqueID, minVersion, required));
+            }
+            return result.ToArray();
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/SmapiConverters/ModCompatibilityArrayConverter.cs b/src/SMAPI/Framework/Serialisation/SmapiConverters/ModCompatibilityArrayConverter.cs
new file mode 100644
index 000000000..3232dde4a
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/SmapiConverters/ModCompatibilityArrayConverter.cs
@@ -0,0 +1,61 @@
+using System;
+using System.Collections.Generic;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Models;
+
+namespace StardewModdingAPI.Framework.Serialisation.SmapiConverters
+{
+    /// <summary>Handles deserialisation of <see cref="ModCompatibility"/> arrays.</summary>
+    internal class ModCompatibilityArrayConverter : JsonConverter
+    {
+        /*********
+        ** Accessors
+        *********/
+        /// <summary>Whether this converter can write JSON.</summary>
+        public override bool CanWrite => false;
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get whether this instance can convert the specified object type.</summary>
+        /// <param name="objectType">The object type.</param>
+        public override bool CanConvert(Type objectType)
+        {
+            return objectType == typeof(ModCompatibility[]);
+        }
+
+
+        /*********
+        ** Protected methods
+        *********/
+        /// <summary>Read the JSON representation of the object.</summary>
+        /// <param name="reader">The JSON reader.</param>
+        /// <param name="objectType">The object type.</param>
+        /// <param name="existingValue">The object being read.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override object ReadJson(JsonReader reader, Type objectType, object existingValue, JsonSerializer serializer)
+        {
+            List<ModCompatibility> result = new List<ModCompatibility>();
+            foreach (JProperty property in JObject.Load(reader).Properties())
+            {
+                string range = property.Name;
+                ModStatus status = (ModStatus)Enum.Parse(typeof(ModStatus), property.Value.Value<string>(nameof(ModCompatibility.Status)));
+                string reasonPhrase = property.Value.Value<string>(nameof(ModCompatibility.ReasonPhrase));
+
+                result.Add(new ModCompatibility(range, status, reasonPhrase));
+            }
+            return result.ToArray();
+        }
+
+        /// <summary>Writes the JSON representation of the object.</summary>
+        /// <param name="writer">The JSON writer.</param>
+        /// <param name="value">The value.</param>
+        /// <param name="serializer">The calling serializer.</param>
+        public override void WriteJson(JsonWriter writer, object value, JsonSerializer serializer)
+        {
+            throw new InvalidOperationException("This converter does not write JSON.");
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/SmapiConverters/ModDataIdConverter.cs b/src/SMAPI/Framework/Serialisation/SmapiConverters/ModDataIdConverter.cs
new file mode 100644
index 000000000..8a10db47d
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/SmapiConverters/ModDataIdConverter.cs
@@ -0,0 +1,19 @@
+using StardewModdingAPI.Framework.Models;
+
+namespace StardewModdingAPI.Framework.Serialisation.SmapiConverters
+{
+    /// <summary>Handles deserialisation of <see cref="ModDataID"/>.</summary>
+    internal class ModDataIdConverter : SimpleReadOnlyConverter<ModDataID>
+    {
+        /*********
+        ** Protected methods
+        *********/
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override ModDataID ReadString(string str, string path)
+        {
+            return new ModDataID(str);
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/SmapiConverters/SemanticVersionConverter.cs b/src/SMAPI/Framework/Serialisation/SmapiConverters/SemanticVersionConverter.cs
new file mode 100644
index 000000000..50181809e
--- /dev/null
+++ b/src/SMAPI/Framework/Serialisation/SmapiConverters/SemanticVersionConverter.cs
@@ -0,0 +1,36 @@
+using Newtonsoft.Json.Linq;
+using StardewModdingAPI.Framework.Exceptions;
+
+namespace StardewModdingAPI.Framework.Serialisation.SmapiConverters
+{
+    /// <summary>Handles deserialisation of <see cref="SemanticVersion"/>.</summary>
+    internal class SemanticVersionConverter : SimpleReadOnlyConverter<ISemanticVersion>
+    {
+        /*********
+        ** Protected methods
+        *********/
+        /// <summary>Read a JSON object.</summary>
+        /// <param name="obj">The JSON object to read.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override ISemanticVersion ReadObject(JObject obj, string path)
+        {
+            int major = obj.Value<int>(nameof(ISemanticVersion.MajorVersion));
+            int minor = obj.Value<int>(nameof(ISemanticVersion.MinorVersion));
+            int patch = obj.Value<int>(nameof(ISemanticVersion.PatchVersion));
+            string build = obj.Value<string>(nameof(ISemanticVersion.Build));
+            return new LegacyManifestVersion(major, minor, patch, build);
+        }
+
+        /// <summary>Read a JSON string.</summary>
+        /// <param name="str">The JSON string value.</param>
+        /// <param name="path">The path to the current JSON node.</param>
+        protected override ISemanticVersion ReadString(string str, string path)
+        {
+            if (string.IsNullOrWhiteSpace(str))
+                return null;
+            if (!SemanticVersion.TryParse(str, out ISemanticVersion version))
+                throw new SParseException($"Can't parse semantic version from invalid value '{str}', should be formatted like 1.2, 1.2.30, or 1.2.30-beta (path: {path}).");
+            return version;
+        }
+    }
+}
diff --git a/src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs b/src/SMAPI/Framework/Serialisation/SmapiConverters/StringEnumConverter.cs
similarity index 91%
rename from src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs
rename to src/SMAPI/Framework/Serialisation/SmapiConverters/StringEnumConverter.cs
index 0a612c74d..c88ac834f 100644
--- a/src/SMAPI/Framework/Serialisation/Converters/StringEnumConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/SmapiConverters/StringEnumConverter.cs
@@ -1,7 +1,7 @@
 using System;
 using Newtonsoft.Json.Converters;
 
-namespace StardewModdingAPI.Framework.Serialisation.Converters
+namespace StardewModdingAPI.Framework.Serialisation.SmapiConverters
 {
     /// <summary>A variant of <see cref="StringEnumConverter"/> which only converts a specified enum.</summary>
     /// <typeparam name="T">The enum type.</typeparam>
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index f5060e2db..ca05399d8 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -110,10 +110,14 @@
     <Compile Include="Framework\Exceptions\SAssemblyLoadFailedException.cs" />
     <Compile Include="Framework\ModLoading\AssemblyLoadStatus.cs" />
     <Compile Include="Framework\Reflection\InterfaceProxyBuilder.cs" />
+    <Compile Include="Framework\Serialisation\SmapiConverters\ModCompatibilityArrayConverter.cs" />
+    <Compile Include="Framework\Serialisation\SmapiConverters\ManifestDependencyArrayConverter.cs" />
+    <Compile Include="Framework\Serialisation\SmapiConverters\ModDataIdConverter.cs" />
+    <Compile Include="Framework\Serialisation\SmapiConverters\SemanticVersionConverter.cs" />
     <Compile Include="Framework\Serialisation\SimpleReadOnlyConverter.cs" />
-    <Compile Include="Framework\Serialisation\Converters\RectangleConverter.cs" />
-    <Compile Include="Framework\Serialisation\Converters\ColorConverter.cs" />
-    <Compile Include="Framework\Serialisation\Converters\PointConverter.cs" />
+    <Compile Include="Framework\Serialisation\CrossplatformConverters\RectangleConverter.cs" />
+    <Compile Include="Framework\Serialisation\CrossplatformConverters\ColorConverter.cs" />
+    <Compile Include="Framework\Serialisation\CrossplatformConverters\PointConverter.cs" />
     <Compile Include="Framework\Utilities\ContextHash.cs" />
     <Compile Include="IReflectedField.cs" />
     <Compile Include="IReflectedMethod.cs" />
@@ -181,8 +185,7 @@
     <Compile Include="Framework\SContentManager.cs" />
     <Compile Include="Framework\Exceptions\SParseException.cs" />
     <Compile Include="Framework\Serialisation\JsonHelper.cs" />
-    <Compile Include="Framework\Serialisation\Converters\StringEnumConverter.cs" />
-    <Compile Include="Framework\Serialisation\Converters\SFieldConverter.cs" />
+    <Compile Include="Framework\Serialisation\SmapiConverters\StringEnumConverter.cs" />
     <Compile Include="IAssetEditor.cs" />
     <Compile Include="IAssetInfo.cs" />
     <Compile Include="IAssetLoader.cs" />

From b10a4b410bb2d50245928bdb88a9efc05aadbe72 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 22:18:11 -0500
Subject: [PATCH 16/21] tweak JSON error messages

---
 .../Framework/Serialisation/SimpleReadOnlyConverter.cs      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs b/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs
index 9308456ba..5765ad96c 100644
--- a/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs
+++ b/src/SMAPI/Framework/Serialisation/SimpleReadOnlyConverter.cs
@@ -50,7 +50,7 @@ public override object ReadJson(JsonReader reader, Type objectType, object exist
                 case JsonToken.String:
                     return this.ReadString(JToken.Load(reader).Value<string>(), path);
                 default:
-                    throw new SParseException($"Can't parse {typeof(T).Name} from {reader.TokenType} (path: {reader.Path}).");
+                    throw new SParseException($"Can't parse {typeof(T).Name} from {reader.TokenType} node (path: {reader.Path}).");
             }
         }
 
@@ -63,7 +63,7 @@ public override object ReadJson(JsonReader reader, Type objectType, object exist
         /// <param name="path">The path to the current JSON node.</param>
         protected virtual T ReadObject(JObject obj, string path)
         {
-            throw new SParseException($"Can't parse {typeof(T).Name} from object (path: {path}).");
+            throw new SParseException($"Can't parse {typeof(T).Name} from object node (path: {path}).");
         }
 
         /// <summary>Read a JSON string.</summary>
@@ -71,7 +71,7 @@ protected virtual T ReadObject(JObject obj, string path)
         /// <param name="path">The path to the current JSON node.</param>
         protected virtual T ReadString(string str, string path)
         {
-            throw new SParseException($"Can't parse {typeof(T).Name} from string (path: {path}).");
+            throw new SParseException($"Can't parse {typeof(T).Name} from string node (path: {path}).");
         }
     }
 }

From 9eae0df9f08d19646b3060f79a2a7301d6c038c6 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 22:56:03 -0500
Subject: [PATCH 17/21] update Json.NET to 11.0.1-beta3 to fix parsing error
 with JSON comments in arrays (#423)

---
 .../StardewModdingAPI.Mods.ConsoleCommands.csproj             | 4 ++--
 src/SMAPI.Mods.ConsoleCommands/packages.config                | 2 +-
 src/SMAPI.Tests/StardewModdingAPI.Tests.csproj                | 4 ++--
 src/SMAPI.Tests/packages.config                               | 2 +-
 src/SMAPI/StardewModdingAPI.csproj                            | 4 ++--
 src/SMAPI/packages.config                                     | 2 +-
 6 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/SMAPI.Mods.ConsoleCommands/StardewModdingAPI.Mods.ConsoleCommands.csproj b/src/SMAPI.Mods.ConsoleCommands/StardewModdingAPI.Mods.ConsoleCommands.csproj
index a65ad72c4..a5b89a336 100644
--- a/src/SMAPI.Mods.ConsoleCommands/StardewModdingAPI.Mods.ConsoleCommands.csproj
+++ b/src/SMAPI.Mods.ConsoleCommands/StardewModdingAPI.Mods.ConsoleCommands.csproj
@@ -36,8 +36,8 @@
     <PlatformTarget>x86</PlatformTarget>
   </PropertyGroup>
   <ItemGroup>
-    <Reference Include="Newtonsoft.Json, Version=10.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
-      <HintPath>..\packages\Newtonsoft.Json.10.0.3\lib\net45\Newtonsoft.Json.dll</HintPath>
+    <Reference Include="Newtonsoft.Json, Version=11.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
+      <HintPath>..\packages\Newtonsoft.Json.11.0.1-beta3\lib\net45\Newtonsoft.Json.dll</HintPath>
       <Private>False</Private>
     </Reference>
     <Reference Include="System" />
diff --git a/src/SMAPI.Mods.ConsoleCommands/packages.config b/src/SMAPI.Mods.ConsoleCommands/packages.config
index ee51c2373..a0f76c34a 100644
--- a/src/SMAPI.Mods.ConsoleCommands/packages.config
+++ b/src/SMAPI.Mods.ConsoleCommands/packages.config
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
-  <package id="Newtonsoft.Json" version="10.0.3" targetFramework="net45" />
+  <package id="Newtonsoft.Json" version="11.0.1-beta3" targetFramework="net45" />
 </packages>
\ No newline at end of file
diff --git a/src/SMAPI.Tests/StardewModdingAPI.Tests.csproj b/src/SMAPI.Tests/StardewModdingAPI.Tests.csproj
index c7a673067..6e7fa1f0c 100644
--- a/src/SMAPI.Tests/StardewModdingAPI.Tests.csproj
+++ b/src/SMAPI.Tests/StardewModdingAPI.Tests.csproj
@@ -36,8 +36,8 @@
     <Reference Include="Moq, Version=4.7.142.0, Culture=neutral, PublicKeyToken=69f491c39445e920, processorArchitecture=MSIL">
       <HintPath>..\packages\Moq.4.7.142\lib\net45\Moq.dll</HintPath>
     </Reference>
-    <Reference Include="Newtonsoft.Json, Version=10.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
-      <HintPath>..\packages\Newtonsoft.Json.10.0.3\lib\net45\Newtonsoft.Json.dll</HintPath>
+    <Reference Include="Newtonsoft.Json, Version=11.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
+      <HintPath>..\packages\Newtonsoft.Json.11.0.1-beta3\lib\net45\Newtonsoft.Json.dll</HintPath>
     </Reference>
     <Reference Include="nunit.framework, Version=3.8.1.0, Culture=neutral, PublicKeyToken=2638cd05610744eb, processorArchitecture=MSIL">
       <HintPath>..\packages\NUnit.3.8.1\lib\net45\nunit.framework.dll</HintPath>
diff --git a/src/SMAPI.Tests/packages.config b/src/SMAPI.Tests/packages.config
index 7a91e807d..498325d67 100644
--- a/src/SMAPI.Tests/packages.config
+++ b/src/SMAPI.Tests/packages.config
@@ -2,6 +2,6 @@
 <packages>
   <package id="Castle.Core" version="4.2.1" targetFramework="net45" />
   <package id="Moq" version="4.7.142" targetFramework="net45" />
-  <package id="Newtonsoft.Json" version="10.0.3" targetFramework="net45" />
+  <package id="Newtonsoft.Json" version="11.0.1-beta3" targetFramework="net45" />
   <package id="NUnit" version="3.8.1" targetFramework="net45" />
 </packages>
\ No newline at end of file
diff --git a/src/SMAPI/StardewModdingAPI.csproj b/src/SMAPI/StardewModdingAPI.csproj
index ca05399d8..cd6c4ac32 100644
--- a/src/SMAPI/StardewModdingAPI.csproj
+++ b/src/SMAPI/StardewModdingAPI.csproj
@@ -65,8 +65,8 @@
       <HintPath>..\packages\Mono.Cecil.0.9.6.4\lib\net45\Mono.Cecil.Pdb.dll</HintPath>
       <Private>True</Private>
     </Reference>
-    <Reference Include="Newtonsoft.Json, Version=10.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
-      <HintPath>..\packages\Newtonsoft.Json.10.0.3\lib\net45\Newtonsoft.Json.dll</HintPath>
+    <Reference Include="Newtonsoft.Json, Version=11.0.0.0, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed, processorArchitecture=MSIL">
+      <HintPath>..\packages\Newtonsoft.Json.11.0.1-beta3\lib\net45\Newtonsoft.Json.dll</HintPath>
     </Reference>
     <Reference Include="System" />
     <Reference Include="System.Core" />
diff --git a/src/SMAPI/packages.config b/src/SMAPI/packages.config
index 98d742c77..1a0b78fa9 100644
--- a/src/SMAPI/packages.config
+++ b/src/SMAPI/packages.config
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
   <package id="Mono.Cecil" version="0.9.6.4" targetFramework="net45" />
-  <package id="Newtonsoft.Json" version="10.0.3" targetFramework="net45" />
+  <package id="Newtonsoft.Json" version="11.0.1-beta3" targetFramework="net45" />
 </packages>
\ No newline at end of file

From b3318af7d0e40462563e6d98c0c15fac48bb6770 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sat, 20 Jan 2018 23:14:39 -0500
Subject: [PATCH 18/21] update release notes

---
 docs/release-notes.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 31ceb7dec..38fb680be 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -6,6 +6,7 @@
   * Fixed error parsing JSON files generated on another platform.
   * Fixed error parsing some JSON files after mods reload core assemblies, which is no longer allowed.
   * Fixed error on Linux loading SMAPI when the default terminal is set to Terminator.
+  * Fixed issue where Custom Farm Types couldn't initialise a custom farm.
 
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.
@@ -17,6 +18,7 @@
   * Fixed input events not recognising controller input as an action or use-tool button.
   * Fixed input events setting the same `IsActionButton` and `IsUseToolButton` values for all buttons pressed in an update tick.
   * Fixed semantic versions always ignoring `-0` tag.
+  * Updated Json.NET to 11.0.1-beta3 (needed to avoid parser edge case with the new converters).
 
 * For SMAPI developers:
   * Overhauled input handling to support future input events.

From d76476ca687a4936cc610380e2b3902db137bced Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 21 Jan 2018 02:14:28 -0500
Subject: [PATCH 19/21] add before/after save creation events (#429)

This supports mods like Custom Farm Types that need to intercept the very first save, which doesn't raise the normal save events since the world isn't fully initialised yet.
---
 docs/release-notes.md          |  1 +
 src/SMAPI/Events/SaveEvents.cs | 22 +++++++++++++++++++++-
 src/SMAPI/Framework/SGame.cs   | 18 ++++++++++++++++++
 3 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 38fb680be..afd89c680 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -12,6 +12,7 @@
   * Fixed parse error for logs with zero installed mods.
 
 * For modders:
+  * Added `SaveEvents.BeforeCreate` and `AfterCreate` events.
   * Added `SButton` `IsActionButton()` and `IsUseToolButton()` extensions.
   * Improved JSON parse errors to provide more useful info for troubleshooting.
   * Fixed events being raised while the game is loading a save file.
diff --git a/src/SMAPI/Events/SaveEvents.cs b/src/SMAPI/Events/SaveEvents.cs
index 50e6d729c..99b6c8d2b 100644
--- a/src/SMAPI/Events/SaveEvents.cs
+++ b/src/SMAPI/Events/SaveEvents.cs
@@ -1,4 +1,4 @@
-using System;
+using System;
 using StardewModdingAPI.Framework;
 
 namespace StardewModdingAPI.Events
@@ -9,6 +9,12 @@ public static class SaveEvents
         /*********
         ** Events
         *********/
+        /// <summary>Raised before the game creates the save file.</summary>
+        public static event EventHandler BeforeCreate;
+
+        /// <summary>Raised after the game finishes creating the save file.</summary>
+        public static event EventHandler AfterCreate;
+
         /// <summary>Raised before the game begins writes data to the save file.</summary>
         public static event EventHandler BeforeSave;
 
@@ -25,6 +31,20 @@ public static class SaveEvents
         /*********
         ** Internal methods
         *********/
+        /// <summary>Raise a <see cref="BeforeCreate"/> event.</summary>
+        /// <param name="monitor">Encapsulates monitoring and logging.</param>
+        internal static void InvokeBeforeCreate(IMonitor monitor)
+        {
+            monitor.SafelyRaisePlainEvent($"{nameof(SaveEvents)}.{nameof(SaveEvents.BeforeCreate)}", SaveEvents.BeforeCreate?.GetInvocationList(), null, EventArgs.Empty);
+        }
+
+        /// <summary>Raise a <see cref="AfterCreate"/> event.</summary>
+        /// <param name="monitor">Encapsulates monitoring and logging.</param>
+        internal static void InvokeAfterCreated(IMonitor monitor)
+        {
+            monitor.SafelyRaisePlainEvent($"{nameof(SaveEvents)}.{nameof(SaveEvents.AfterCreate)}", SaveEvents.AfterCreate?.GetInvocationList(), null, EventArgs.Empty);
+        }
+
         /// <summary>Raise a <see cref="BeforeSave"/> event.</summary>
         /// <param name="monitor">Encapsulates monitoring and logging.</param>
         internal static void InvokeBeforeSave(IMonitor monitor)
diff --git a/src/SMAPI/Framework/SGame.cs b/src/SMAPI/Framework/SGame.cs
index 2eb2da99d..e82ee778c 100644
--- a/src/SMAPI/Framework/SGame.cs
+++ b/src/SMAPI/Framework/SGame.cs
@@ -51,6 +51,9 @@ internal class SGame : Game1
         /// <summary>Whether the game is saving and SMAPI has already raised <see cref="SaveEvents.BeforeSave"/>.</summary>
         private bool IsBetweenSaveEvents;
 
+        /// <summary>Whether the game is creating the save file and SMAPI has already raised <see cref="SaveEvents.BeforeCreate"/>.</summary>
+        private bool IsBetweenCreateEvents;
+
         /****
         ** Game state
         ****/
@@ -246,6 +249,14 @@ protected override void Update(GameTime gameTime)
                 // opened (since the save hasn't started yet), but all other events should be suppressed.
                 if (Context.IsSaving)
                 {
+                    // raise before-create
+                    if (!Context.IsWorldReady && !this.IsBetweenCreateEvents)
+                    {
+                        this.IsBetweenCreateEvents = true;
+                        this.Monitor.Log("Context: before save creation.", LogLevel.Trace);
+                        SaveEvents.InvokeBeforeCreate(this.Monitor);
+                    }
+                    
                     // raise before-save
                     if (Context.IsWorldReady && !this.IsBetweenSaveEvents)
                     {
@@ -258,6 +269,13 @@ protected override void Update(GameTime gameTime)
                     base.Update(gameTime);
                     return;
                 }
+                if (this.IsBetweenCreateEvents)
+                {
+                    // raise after-create
+                    this.IsBetweenCreateEvents = false;
+                    this.Monitor.Log($"Context: after save creation, starting {Game1.currentSeason} {Game1.dayOfMonth} Y{Game1.year}.", LogLevel.Trace);
+                    SaveEvents.InvokeAfterCreated(this.Monitor);
+                }
                 if (this.IsBetweenSaveEvents)
                 {
                     // raise after-save

From 274139eafef34866f8e7d404a52db85c8d89a624 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 21 Jan 2018 11:48:12 -0500
Subject: [PATCH 20/21] rm release note about Custom Farm Types (needs a
 separate update anyway)

---
 docs/release-notes.md | 1 -
 1 file changed, 1 deletion(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index afd89c680..51ac8c5ad 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -6,7 +6,6 @@
   * Fixed error parsing JSON files generated on another platform.
   * Fixed error parsing some JSON files after mods reload core assemblies, which is no longer allowed.
   * Fixed error on Linux loading SMAPI when the default terminal is set to Terminator.
-  * Fixed issue where Custom Farm Types couldn't initialise a custom farm.
 
 * For the [log parser][]:
   * Fixed parse error for logs with zero installed mods.

From 5c96a10da5801049ee17ffa185dbf19e6d8a2306 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <github@jplamondonw.com>
Date: Sun, 21 Jan 2018 22:22:44 -0500
Subject: [PATCH 21/21] update for release

---
 build/GlobalAssemblyInfo.cs                  |  4 ++--
 docs/release-notes.md                        | 15 ++++++++-------
 src/SMAPI.Mods.ConsoleCommands/manifest.json |  7 +------
 src/SMAPI/Constants.cs                       |  2 +-
 4 files changed, 12 insertions(+), 16 deletions(-)

diff --git a/build/GlobalAssemblyInfo.cs b/build/GlobalAssemblyInfo.cs
index bcfdc1242..79d473e67 100644
--- a/build/GlobalAssemblyInfo.cs
+++ b/build/GlobalAssemblyInfo.cs
@@ -2,5 +2,5 @@
 using System.Runtime.InteropServices;
 
 [assembly: ComVisible(false)]
-[assembly: AssemblyVersion("2.3.0.0")]
-[assembly: AssemblyFileVersion("2.3.0.0")]
+[assembly: AssemblyVersion("2.4.0.0")]
+[assembly: AssemblyFileVersion("2.4.0.0")]
diff --git a/docs/release-notes.md b/docs/release-notes.md
index 51ac8c5ad..9c1fbf93b 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -1,14 +1,15 @@
 # Release notes
 ## 2.4
 * For players:
-  * Fixed graphical corruption in rare cases.
-  * Fixed error parsing `config.json` files containing curly quotes.
-  * Fixed error parsing JSON files generated on another platform.
+  * Fixed visual map glitch in rare cases.
+  * Fixed error parsing JSON files which have curly quotes.
+  * Fixed error parsing some JSON files generated on another system.
   * Fixed error parsing some JSON files after mods reload core assemblies, which is no longer allowed.
-  * Fixed error on Linux loading SMAPI when the default terminal is set to Terminator.
+  * Fixed intermittent errors (e.g. 'collection has been modified') with some mods when loading a save.
+  * Fixed compatibility with Linux Terminator terminal.
 
 * For the [log parser][]:
-  * Fixed parse error for logs with zero installed mods.
+  * Fixed error parsing logs with zero installed mods.
 
 * For modders:
   * Added `SaveEvents.BeforeCreate` and `AfterCreate` events.
@@ -17,8 +18,8 @@
   * Fixed events being raised while the game is loading a save file.
   * Fixed input events not recognising controller input as an action or use-tool button.
   * Fixed input events setting the same `IsActionButton` and `IsUseToolButton` values for all buttons pressed in an update tick.
-  * Fixed semantic versions always ignoring `-0` tag.
-  * Updated Json.NET to 11.0.1-beta3 (needed to avoid parser edge case with the new converters).
+  * Fixed semantic versions ignoring `-0` as a prerelease tag.
+  * Updated Json.NET to 11.0.1-beta3 (needed to avoid a parser edge case).
 
 * For SMAPI developers:
   * Overhauled input handling to support future input events.
diff --git a/src/SMAPI.Mods.ConsoleCommands/manifest.json b/src/SMAPI.Mods.ConsoleCommands/manifest.json
index 06ab1b541..fc5ce35d1 100644
--- a/src/SMAPI.Mods.ConsoleCommands/manifest.json
+++ b/src/SMAPI.Mods.ConsoleCommands/manifest.json
@@ -1,12 +1,7 @@
 {
   "Name": "Console Commands",
   "Author": "SMAPI",
-  "Version": {
-    "MajorVersion": 2,
-    "MinorVersion": 3,
-    "PatchVersion": 0,
-    "Build": null
-  },
+  "Version": "2.4.0",
   "Description": "Adds SMAPI console commands that let you manipulate the game.",
   "UniqueID": "SMAPI.ConsoleCommands",
   "EntryDll": "ConsoleCommands.dll"
diff --git a/src/SMAPI/Constants.cs b/src/SMAPI/Constants.cs
index 786ae32b1..515e98702 100644
--- a/src/SMAPI/Constants.cs
+++ b/src/SMAPI/Constants.cs
@@ -29,7 +29,7 @@ public static class Constants
         ** Public
         ****/
         /// <summary>SMAPI's current semantic version.</summary>
-        public static ISemanticVersion ApiVersion { get; } = new SemanticVersion("2.3");
+        public static ISemanticVersion ApiVersion { get; } = new SemanticVersion("2.4.0");
 
         /// <summary>The minimum supported version of Stardew Valley.</summary>
         public static ISemanticVersion MinimumGameVersion { get; } = new SemanticVersion("1.2.30");