Nits

Author: marcarro

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/Razor/ExtractToNewComponentCodeActionProvider.cs

@@ -52,9 +52,8 @@ public Task<ImmutableArray<RazorVSInternalCodeAction>> ProvideAsync(RazorCodeAct
             return SpecializedTasks.EmptyImmutableArray<RazorVSInternalCodeAction>();
         }
 
-        var (startElementNode, endElementNode) = GetStartAndEndElements(context, syntaxTree, _logger);
-
         // Make sure the selection starts on an element tag
+        var (startElementNode, endElementNode) = GetStartAndEndElements(context, syntaxTree, _logger);
         if (startElementNode is null)
         {
             return SpecializedTasks.EmptyImmutableArray<RazorVSInternalCodeAction>();
@@ -72,7 +71,7 @@ public Task<ImmutableArray<RazorVSInternalCodeAction>> ProvideAsync(RazorCodeAct
         AddComponentDependenciesInRange(dependencyScanRoot,
                                         actionParams.ExtractStart,
                                         actionParams.ExtractEnd,
-                                        ref actionParams);
+                                        actionParams);
 
         if (IsMultiPointSelection(context.Request.Range))
         {
@@ -273,7 +272,7 @@ private static (SyntaxNode? Start, SyntaxNode? End) FindContainingSiblingPair(Sy
         return null;
     }
 
-    private static void AddComponentDependenciesInRange(SyntaxNode root, int extractStart, int extractEnd, ref ExtractToNewComponentCodeActionParams actionParams)
+    private static void AddComponentDependenciesInRange(SyntaxNode root, int extractStart, int extractEnd, ExtractToNewComponentCodeActionParams actionParams)
     {
         var components = new HashSet<string>();
         var extractSpan = new TextSpan(extractStart, extractEnd - extractStart);
@@ -285,7 +284,7 @@ private static void AddComponentDependenciesInRange(SyntaxNode root, int extract
                 var tagHelperInfo = GetTagHelperInfo(node);
                 if (tagHelperInfo != null)
                 {
-                    AddDependenciesFromTagHelperInfo(tagHelperInfo, components, ref actionParams);
+                    AddDependenciesFromTagHelperInfo(tagHelperInfo, components, actionParams);
                 }
             }
         }
@@ -307,7 +306,7 @@ private static bool IsMarkupTagHelperElement(SyntaxNode node, TextSpan extractSp
         return null;
     }
 
-    private static void AddDependenciesFromTagHelperInfo(TagHelperInfo tagHelperInfo, HashSet<string> components, ref ExtractToNewComponentCodeActionParams actionParams)
+    private static void AddDependenciesFromTagHelperInfo(TagHelperInfo tagHelperInfo, HashSet<string> components, ExtractToNewComponentCodeActionParams actionParams)
     {
         foreach (var descriptor in tagHelperInfo.BindingResult.Descriptors)
         {

src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/CodeActions/Razor/ExtractToNewComponentCodeActionProviderTest.cs

@@ -238,7 +238,7 @@ public async Task Handle_InProperMarkup_ReturnsEmpty()
         // Arrange
         var documentPath = "c:/Test.razor";
         var contents = """
-            page "/"
+            @page "/"
             
             <PageTitle>Home</PageTitle>