1 files changed, 11 insertions, 2 deletions
diff --git a/src/SMAPI.Toolkit/Framework/Clients/WebApi/ModSearchModel.cs b/src/SMAPI.Toolkit/Framework/Clients/WebApi/ModSearchModel.cs
index 73698173..3c74bab0 100644
--- a/src/SMAPI.Toolkit/Framework/Clients/WebApi/ModSearchModel.cs
+++ b/src/SMAPI.Toolkit/Framework/Clients/WebApi/ModSearchModel.cs
@@ -1,3 +1,5 @@
+using System;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using StardewModdingAPI.Toolkit.Utilities;
 
@@ -22,16 +24,23 @@ namespace StardewModdingAPI.Toolkit.Framework.Clients.WebApi
         public ISemanticVersion GameVersion { get; set; }
 
         /// <summary>The OS on which the player plays.</summary>
-        public Platform? Platform { get; set; }
+        public Platform Platform { get; set; }
 
 
         /*********
         ** Public methods
         *********/
         /// <summary>Construct an empty instance.</summary>
+        [Obsolete("This constructor only exists to support ASP.NET model binding, and shouldn't be used directly.")]
+        [SuppressMessage("ReSharper", "UnusedMember.Global", Justification = "Used by ASP.NET model binding.")]
         public ModSearchModel()
         {
-            // needed for JSON deserializing
+            // ASP.NET Web API needs a public empty constructor for top-level request models, and
+            // it'll fail if the other constructor is marked with [JsonConstructor]. Apparently
+            // it's fine with non-empty constructors in nested models like ModSearchEntryModel.
+            this.Mods = Array.Empty<ModSearchEntryModel>();
+            this.ApiVersion = null!;
+            this.GameVersion = null!;
         }
 
         /// <summary>Construct an instance.</summary>