Run phases of code cleanup in parallel

src/Features/CSharpTest/IntroduceVariable/IntroduceVariableTests.cs

@@ -5977,7 +5977,7 @@ void M()
             {
                 int x = 1;
                 int {|Rename:y1|} = C.y;
-                var t = new { y= y1, y= y1 }; // this is an error already
+                var t = new { y = y1, y = y1 }; // this is an error already
             }
         }
         """;

src/Features/Core/Portable/IntroduceVariable/AbstractIntroduceVariableService.AbstractIntroduceVariableCodeAction.cs

@@ -16,7 +16,7 @@ namespace Microsoft.CodeAnalysis.IntroduceVariable;
 
 internal partial class AbstractIntroduceVariableService<TService, TExpressionSyntax, TTypeSyntax, TTypeDeclarationSyntax, TQueryExpressionSyntax, TNameSyntax>
 {
-    internal abstract class AbstractIntroduceVariableCodeAction : CodeAction
+    private abstract class AbstractIntroduceVariableCodeAction : CodeAction
     {
         private readonly bool _allOccurrences;
         private readonly bool _isConstant;

src/Features/Core/Portable/IntroduceVariable/AbstractIntroduceVariableService.CodeAction.cs

@@ -2,27 +2,22 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
-#nullable disable
-
 using Microsoft.CodeAnalysis.CodeCleanup;
 
 namespace Microsoft.CodeAnalysis.IntroduceVariable;
 
 internal partial class AbstractIntroduceVariableService<TService, TExpressionSyntax, TTypeSyntax, TTypeDeclarationSyntax, TQueryExpressionSyntax, TNameSyntax>
 {
-    private class IntroduceVariableCodeAction : AbstractIntroduceVariableCodeAction
+    private sealed class IntroduceVariableCodeAction(
+        TService service,
+        SemanticDocument document,
+        CodeCleanupOptions options,
+        TExpressionSyntax expression,
+        bool allOccurrences,
+        bool isConstant,
+        bool isLocal,
+        bool isQueryLocal) : AbstractIntroduceVariableCodeAction(
+            service, document, options, expression, allOccurrences, isConstant, isLocal, isQueryLocal)
     {
-        internal IntroduceVariableCodeAction(
-            TService service,
-            SemanticDocument document,
-            CodeCleanupOptions options,
-            TExpressionSyntax expression,
-            bool allOccurrences,
-            bool isConstant,
-            bool isLocal,
-            bool isQueryLocal)
-            : base(service, document, options, expression, allOccurrences, isConstant, isLocal, isQueryLocal)
-        {
-        }
     }
 }

src/Features/Core/Portable/IntroduceVariable/AbstractIntroduceVariableService.IntroduceVariableAllOccurrenceCodeAction.cs

@@ -2,40 +2,22 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
-#nullable disable
-
-using System.Threading;
-using System.Threading.Tasks;
-using Microsoft.CodeAnalysis.CaseCorrection;
 using Microsoft.CodeAnalysis.CodeCleanup;
-using Microsoft.CodeAnalysis.Formatting;
-using Microsoft.CodeAnalysis.Simplification;
 
 namespace Microsoft.CodeAnalysis.IntroduceVariable;
 
 internal partial class AbstractIntroduceVariableService<TService, TExpressionSyntax, TTypeSyntax, TTypeDeclarationSyntax, TQueryExpressionSyntax, TNameSyntax>
 {
-    private class IntroduceVariableAllOccurrenceCodeAction : AbstractIntroduceVariableCodeAction
+    private class IntroduceVariableAllOccurrenceCodeAction(
+        TService service,
+        SemanticDocument document,
+        CodeCleanupOptions options,
+        TExpressionSyntax expression,
+        bool allOccurrences,
+        bool isConstant,
+        bool isLocal,
+        bool isQueryLocal) : AbstractIntroduceVariableCodeAction(
+            service, document, options, expression, allOccurrences, isConstant, isLocal, isQueryLocal)
     {
-        internal IntroduceVariableAllOccurrenceCodeAction(
-            TService service,
-            SemanticDocument document,
-            CodeCleanupOptions options,
-            TExpressionSyntax expression,
-            bool allOccurrences,
-            bool isConstant,
-            bool isLocal,
-            bool isQueryLocal)
-            : base(service, document, options, expression, allOccurrences, isConstant, isLocal, isQueryLocal)
-        {
-        }
-
-        protected override async Task<Document> PostProcessChangesAsync(Document document, CancellationToken cancellationToken)
-        {
-            document = await Simplifier.ReduceAsync(document, Simplifier.Annotation, Options.SimplifierOptions, cancellationToken).ConfigureAwait(false);
-            document = await Formatter.FormatAsync(document, Formatter.Annotation, Options.FormattingOptions, cancellationToken).ConfigureAwait(false);
-            document = await CaseCorrector.CaseCorrectAsync(document, CaseCorrector.Annotation, cancellationToken).ConfigureAwait(false);
-            return document;
-        }
     }
 }

src/Features/Core/Portable/IntroduceVariable/IntroduceVariableCodeRefactoringProvider.cs

@@ -15,16 +15,11 @@ namespace Microsoft.CodeAnalysis.IntroduceVariable;
 [ExtensionOrder(After = PredefinedCodeRefactoringProviderNames.ConvertAnonymousTypeToClass)]
 [ExtensionOrder(After = PredefinedCodeRefactoringProviderNames.InvertConditional)]
 [ExtensionOrder(After = PredefinedCodeRefactoringProviderNames.InvertLogical)]
-[ExportCodeRefactoringProvider(LanguageNames.CSharp, LanguageNames.VisualBasic,
-    Name = PredefinedCodeRefactoringProviderNames.IntroduceVariable), Shared]
-internal class IntroduceVariableCodeRefactoringProvider : CodeRefactoringProvider
+[ExportCodeRefactoringProvider(LanguageNames.CSharp, LanguageNames.VisualBasic, Name = PredefinedCodeRefactoringProviderNames.IntroduceVariable), Shared]
+[method: ImportingConstructor]
+[method: SuppressMessage("RoslynDiagnosticsReliability", "RS0033:Importing constructor should be [Obsolete]", Justification = "Used in test code: https://github.com/dotnet/roslyn/issues/42814")]
+internal sealed class IntroduceVariableCodeRefactoringProvider() : CodeRefactoringProvider
 {
-    [ImportingConstructor]
-    [SuppressMessage("RoslynDiagnosticsReliability", "RS0033:Importing constructor should be [Obsolete]", Justification = "Used in test code: https://github.com/dotnet/roslyn/issues/42814")]
-    public IntroduceVariableCodeRefactoringProvider()
-    {
-    }
-
     public override async Task ComputeRefactoringsAsync(CodeRefactoringContext context)
     {
         var (document, textSpan, cancellationToken) = context;