Order Razor code actions by the Order property

Author: davidwengier

src/Razor/src/Microsoft.CodeAnalysis.Razor.Workspaces/CodeActions/CodeActionsService.cs

@@ -246,7 +246,7 @@ private static async Task<ImmutableArray<RazorVSInternalCodeAction>> Consolidate
     {
         var results = await Task.WhenAll(tasks).ConfigureAwait(false);
 
-        using var codeActions = new PooledArrayBuilder<RazorVSInternalCodeAction>();
+        using var codeActions = new PooledArrayBuilder<RazorVSInternalCodeAction>(capacity: tasks.Length);
 
         cancellationToken.ThrowIfCancellationRequested();
 
@@ -255,7 +255,7 @@ private static async Task<ImmutableArray<RazorVSInternalCodeAction>> Consolidate
             codeActions.AddRange(result);
         }
 
-        return codeActions.ToImmutable();
+        return codeActions.DrainToImmutableOrderedBy(static r => r.Order);
     }
 
     private static ImmutableHashSet<string> GetAllAvailableCodeActionNames()

src/Razor/src/Microsoft.CodeAnalysis.Razor.Workspaces/CodeActions/Models/RazorVSInternalCodeAction.cs

@@ -13,4 +13,10 @@ internal sealed class RazorVSInternalCodeAction : VSInternalCodeAction
     [JsonPropertyName("name")]
     [DataMember(Name = "name")]
     public string? Name { get; set; }
+
+    /// <summary>
+    /// The order code actions should appear. This is not serialized as its just used in the code actions service
+    /// </summary>
+    [JsonIgnore]
+    public int Order { get; set; }
 }

src/Razor/src/Microsoft.CodeAnalysis.Razor.Workspaces/CodeActions/Razor/RazorCodeActionFactory.cs

@@ -50,6 +50,8 @@ public static RazorVSInternalCodeAction CreateAddComponentUsing(string @namespac
             TelemetryId = s_addComponentUsingTelemetryId,
             Priority = VSInternalPriorityLevel.High,
             Name = LanguageServerConstants.CodeActions.AddUsing,
+            // Adding a using for an existing component should be first
+            Order = -1000,
         };
         return codeAction;
     }
@@ -63,6 +65,8 @@ public static RazorVSInternalCodeAction CreateFullyQualifyComponent(string fully
             TelemetryId = s_fullyQualifyComponentTelemetryId,
             Priority = VSInternalPriorityLevel.High,
             Name = LanguageServerConstants.CodeActions.FullyQualify,
+            // Fully qualifying an existing component should be very high, but not quite as high as Add Using
+            Order = -900,
         };
         return codeAction;
     }
@@ -102,6 +106,8 @@ public static RazorVSInternalCodeAction CreateExtractToComponent(RazorCodeAction
             Data = data,
             TelemetryId = s_createExtractToComponentTelemetryId,
             Name = LanguageServerConstants.CodeActions.ExtractToNewComponentAction,
+            // Since Extract to Component is offered basically everywhere, always offer it last
+            Order = 9999
         };
         return codeAction;
     }