Migrate logic over

Author: CyrusNajmabadi

src/Features/Core/Portable/Diagnostics/Service/DiagnosticAnalyzerService.HostAnalyzerInfo.cs

@@ -72,10 +72,16 @@ private static int GetPriority(DiagnosticAnalyzer state)
     /// Return <see cref="DiagnosticAnalyzer"/>s for the given <see cref="Project"/>. 
     /// </summary>
     internal ImmutableArray<DiagnosticAnalyzer> GetProjectAnalyzers(Project project)
+    {
+        var (hostAnalyzers, projectAnalyzers) = GetProjectAnalyzersPair(project);
+        return [.. hostAnalyzers, .. projectAnalyzers];
+    }
+
+    private (ImmutableArray<DiagnosticAnalyzer> hostAnalyzers, ImmutableHashSet<DiagnosticAnalyzer> projectAnalyzers) GetProjectAnalyzersPair(Project project)
     {
         var hostAnalyzerInfo = GetOrCreateHostAnalyzerInfo(project);
         var projectAnalyzerInfo = GetOrCreateProjectAnalyzerInfo(project);
-        return hostAnalyzerInfo.OrderedAllAnalyzers.AddRange(projectAnalyzerInfo.Analyzers);
+        return (hostAnalyzerInfo.OrderedAllAnalyzers, projectAnalyzerInfo.Analyzers);
     }
 
     private HostAnalyzerInfo GetOrCreateHostAnalyzerInfo(Project project)

src/Features/Core/Portable/Diagnostics/Service/DiagnosticAnalyzerService.cs

@@ -13,9 +13,11 @@
 using Microsoft.CodeAnalysis.Host;
 using Microsoft.CodeAnalysis.Host.Mef;
 using Microsoft.CodeAnalysis.Options;
+using Microsoft.CodeAnalysis.Shared.Extensions;
 using Microsoft.CodeAnalysis.Shared.TestHooks;
 using Microsoft.CodeAnalysis.SolutionCrawler;
 using Microsoft.CodeAnalysis.Workspaces.Diagnostics;
+using Roslyn.Utilities;
 
 namespace Microsoft.CodeAnalysis.Diagnostics;
 
@@ -124,20 +126,40 @@ private ImmutableArray<DiagnosticAnalyzer> GetDiagnosticAnalyzers(
         ImmutableHashSet<string>? diagnosticIds,
         Func<DiagnosticAnalyzer, bool>? shouldIncludeAnalyzer)
     {
-        var analyzersForProject = GetProjectAnalyzers(project);
-        var analyzers = analyzersForProject.WhereAsArray(a => ShouldIncludeAnalyzer(project, a));
+        var (hostAnalyzers, projectAnalyzers) = GetProjectAnalyzersPair(project);
+        return [
+            .. hostAnalyzers.WhereAsArray(a => ShouldIncludeAnalyzer(project, a, isHostAnalyzer: true)),
+            .. projectAnalyzers.WhereAsArray(a => ShouldIncludeAnalyzer(project, a, isHostAnalyzer: false))];
 
-        return analyzers;
-
-        bool ShouldIncludeAnalyzer(Project project, DiagnosticAnalyzer analyzer)
+        bool ShouldIncludeAnalyzer(Project project, DiagnosticAnalyzer analyzer, bool isHostAnalyzer)
         {
             if (!DocumentAnalysisExecutor.IsAnalyzerEnabledForProject(analyzer, project, this._globalOptions))
                 return false;
 
             if (shouldIncludeAnalyzer != null && !shouldIncludeAnalyzer(analyzer))
                 return false;
 
-            if (diagnosticIds != null && _analyzerInfoCache.GetDiagnosticDescriptors(analyzer).All(d => !diagnosticIds.Contains(d.Id)))
+            var descriptors = _analyzerInfoCache.GetDiagnosticDescriptors(analyzer);
+
+            if (diagnosticIds != null && descriptors.All(d => !diagnosticIds.Contains(d.Id)))
+                return false;
+
+            var analyzerConfigOptions = project.GetAnalyzerConfigOptions();
+
+            var allDescriptorsAreHidden = project.CompilationOptions != null && descriptors.All(static (d, arg) =>
+                {
+                    var effectiveSeverity = d.GetEffectiveSeverity(
+                        arg.CompilationOptions,
+                        arg.isHostAnalyzer
+                            ? arg.analyzerConfigOptions?.ConfigOptionsWithFallback
+                            : arg.analyzerConfigOptions?.ConfigOptionsWithoutFallback,
+                        arg.analyzerConfigOptions?.TreeOptions);
+
+                    return effectiveSeverity == ReportDiagnostic.Hidden;
+                },
+                (project.CompilationOptions, isHostAnalyzer, analyzerConfigOptions));
+
+            if (allDescriptorsAreHidden)
                 return false;
 
             return true;