Merge pull request #2375 from microsoft/copilot/fix-2374

Remove KeyComparer property from OpenApiWriterSettings

Author: baywet

src/Microsoft.OpenApi/Extensions/CollectionExtensions.cs

@@ -419,13 +419,6 @@ private static void WriteCollectionInternal<T>(
             writer.WriteStartArray();
             if (elements != null)
             {
-                var settings = writer.GetSettings();
-
-                if (settings?.KeyComparer is IComparer<T> typedComparer)
-                {
-                    elements = elements.Sort(typedComparer);
-                }
-
                 foreach (var item in elements)
                 {
                     if (item != null)
@@ -464,12 +457,6 @@ private static void WriteMapInternal<T>(
 
             if (elements != null)
             {
-                var settings = writer.GetSettings();
-                if (settings?.KeyComparer != null)
-                {
-                    elements = elements.Sort(settings.KeyComparer); // sort using custom comparer
-                }
-
                 foreach (var item in elements)
                 {
                     writer.WritePropertyName(item.Key);

src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs

@@ -1,6 +1,4 @@
-using System.Collections.Generic;
-
-namespace Microsoft.OpenApi
+namespace Microsoft.OpenApi
 {
     /// <summary>
     /// Configuration settings to control how OpenAPI documents are written
@@ -23,10 +21,5 @@ internal bool ShouldInlineReference(OpenApiReference reference)
             return (reference.IsLocal && InlineLocalReferences)
                              || (reference.IsExternal && InlineExternalReferences);
         }
-
-        /// <summary>  
-        /// Specifies a comparer used to sort string-based collection keys, such as components or tags.
-        /// </summary>  
-        public IComparer<string>? KeyComparer { get; set; }
     }
 }