Remove TODOs related to AllowOutOfOrderMetadataProperty workarounds. (#977)

Author: eiriktsarpalis

src/ModelContextProtocol.Core/Protocol/ContentBlock.cs

@@ -25,7 +25,7 @@ namespace ModelContextProtocol.Protocol;
 /// See the <see href="https://github.com/modelcontextprotocol/specification/blob/main/schema/">schema</see> for more details.
 /// </para>
 /// </remarks>
-[JsonConverter(typeof(Converter))] // TODO: This converter exists due to the lack of downlevel support for AllowOutOfOrderMetadataProperties.
+[JsonConverter(typeof(Converter))]
 public abstract class ContentBlock
 {
     /// <summary>Prevent external derivations.</summary>
@@ -55,6 +55,8 @@ private protected ContentBlock()
     /// <summary>
     /// Provides a <see cref="JsonConverter"/> for <see cref="ContentBlock"/>.
     /// </summary>
+    /// Provides a polymorphic converter for the <see cref="ContentBlock"/> class that doesn't  require
+    /// setting <see cref="JsonSerializerOptions.AllowOutOfOrderMetadataProperties"/> explicitly.
     [EditorBrowsable(EditorBrowsableState.Never)]
     public class Converter : JsonConverter<ContentBlock>
     {

src/ModelContextProtocol.Core/Protocol/ElicitRequestParams.cs

@@ -61,7 +61,7 @@ public IDictionary<string, PrimitiveSchemaDefinition> Properties
     /// Represents restricted subset of JSON Schema: 
     /// <see cref="StringSchema"/>, <see cref="NumberSchema"/>, <see cref="BooleanSchema"/>, or <see cref="EnumSchema"/>.
     /// </summary>
-    [JsonConverter(typeof(Converter))] // TODO: This converter exists due to the lack of downlevel support for AllowOutOfOrderMetadataProperties.
+    [JsonConverter(typeof(Converter))]
     public abstract class PrimitiveSchemaDefinition
     {
         /// <summary>Prevent external derivations.</summary>
@@ -84,6 +84,8 @@ protected private PrimitiveSchemaDefinition()
         /// <summary>
         /// Provides a <see cref="JsonConverter"/> for <see cref="ResourceContents"/>.
         /// </summary>
+        /// Provides a polymorphic converter for the <see cref="PrimitiveSchemaDefinition"/> class that doesn't  require
+        /// setting <see cref="JsonSerializerOptions.AllowOutOfOrderMetadataProperties"/> explicitly.
         [EditorBrowsable(EditorBrowsableState.Never)]
         public class Converter : JsonConverter<PrimitiveSchemaDefinition>
         {

src/ModelContextProtocol.Core/Protocol/Reference.cs

@@ -39,6 +39,10 @@ private protected Reference()
     /// <summary>
     /// Provides a <see cref="JsonConverter"/> for <see cref="Reference"/>.
     /// </summary>
+    /// <remarks>
+    /// Provides a polymorphic converter for the <see cref="Reference"/> class that doesn't  require
+    /// setting <see cref="JsonSerializerOptions.AllowOutOfOrderMetadataProperties"/> explicitly.
+    /// </remarks>
     [EditorBrowsable(EditorBrowsableState.Never)]
     public sealed class Converter : JsonConverter<Reference>
     {
@@ -95,8 +99,6 @@ public sealed class Converter : JsonConverter<Reference>
                 }
             }
 
-            // TODO: This converter exists due to the lack of downlevel support for AllowOutOfOrderMetadataProperties.
-
             switch (type)
             {
                 case "ref/prompt":