Address feedback

Author: captainsafia

src/Framework/AspNetCoreAnalyzers/src/Analyzers/DiagnosticDescriptors.cs

@@ -231,5 +231,6 @@ internal static class DiagnosticDescriptors
         "Usage",
         DiagnosticSeverity.Info,
         isEnabledByDefault: true,
-        helpLinkUri: "https://aka.ms/aspnet/analyzers");
+        helpLinkUri: "https://aka.ms/aspnet/analyzers",
+        customTags: WellKnownDiagnosticTags.Unnecessary);
 }

src/Framework/AspNetCoreAnalyzers/src/Analyzers/Resources.resx

@@ -322,9 +322,9 @@
     <value>[Authorize] overridden by [AllowAnonymous] from farther away</value>
   </data>
   <data name="Analyzer_PublicPartialProgramClass_Message" xml:space="preserve">
-    <value>Using public partial class Program { } to make generated Program class public is no longer required. See https://aka.ms/aspnetcore-warnings/ASP0027 for more details.</value>
+    <value>Using public partial class Program { } to make the generated Program class public is no longer required in ASP.NET Core apps. See https://aka.ms/aspnetcore-warnings/ASP0027 for more details.</value>
   </data>
   <data name="Analyzer_PublicPartialProgramClass_Title" xml:space="preserve">
-    <value>Explicit class declaration not required</value>
+    <value>Unnecessary public Program class declaration</value>
   </data>
-</root>
+</root>

src/Framework/AspNetCoreAnalyzers/src/Analyzers/WebApplicationBuilder/PublicPartialProgramClassAnalyzer.cs

@@ -2,6 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System.Collections.Immutable;
+using System.Linq;
 using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.CSharp;
 using Microsoft.CodeAnalysis.CSharp.Syntax;
@@ -23,7 +24,9 @@ public override void Initialize(AnalysisContext context)
             var syntaxNode = context.Node;
             if (IsPublicPartialClassProgram(syntaxNode))
             {
-                context.ReportDiagnostic(Diagnostic.Create(DiagnosticDescriptors.PublicPartialProgramClassNotRequired, syntaxNode.GetLocation()));
+                context.ReportDiagnostic(Diagnostic.Create(
+                    DiagnosticDescriptors.PublicPartialProgramClassNotRequired,
+                    syntaxNode.GetLocation()));
             }
         }, SyntaxKind.ClassDeclaration);
     }
@@ -34,6 +37,8 @@ private static bool IsPublicPartialClassProgram(SyntaxNode syntaxNode)
             && modifiers is { Count: > 1 }
             && modifiers.Any(SyntaxKind.PublicKeyword)
             && modifiers.Any(SyntaxKind.PartialKeyword)
-            && classDeclaration is { Identifier.ValueText: "Program" };
+            && classDeclaration is { Identifier.ValueText: "Program" }
+            && classDeclaration.Parent is CompilationUnitSyntax parentNode
+            && parentNode.DescendantNodes().Count() > 1;
     }
 }

src/Framework/AspNetCoreAnalyzers/src/CodeFixes/PublicPartialProgramClassFixer.cs

@@ -25,7 +25,7 @@ public sealed override Task RegisterCodeFixesAsync(CodeFixContext context)
         foreach (var diagnostic in context.Diagnostics)
         {
             context.RegisterCodeFix(
-                CodeAction.Create("Fix unnecessary public partial class Program",
+                CodeAction.Create("Remove unnecessary public partial class Program declaration",
                     async cancellationToken =>
                     {
                         var editor = await DocumentEditor.CreateAsync(context.Document, cancellationToken).ConfigureAwait(false);
@@ -41,7 +41,7 @@ public sealed override Task RegisterCodeFixesAsync(CodeFixContext context)
                         {
                             return context.Document;
                         }
-                        editor.RemoveNode(classDeclaration, SyntaxRemoveOptions.KeepNoTrivia);
+                        editor.RemoveNode(classDeclaration, SyntaxRemoveOptions.KeepExteriorTrivia);
                         return editor.GetChangedDocument();
                     },
                     equivalenceKey: DiagnosticDescriptors.PublicPartialProgramClassNotRequired.Id),

src/Framework/AspNetCoreAnalyzers/test/WebApplicationBuilder/PublicPartialProgramClassTest.cs

@@ -60,6 +60,46 @@ public async Task RemovesDeclarationIfItIsFound(string declarationStyle)
 
 app.Run();
 
+
+""";
+
+        // Assert
+        await VerifyCS.VerifyCodeFixAsync(source, [diagnostic], fixedSource);
+    }
+
+    [Fact]
+    public async Task RemovesDeclarationIfItIsFound_WithLeadingTrivia()
+    {
+        // Arrange
+        var source = """
+using Microsoft.AspNetCore.Builder;
+
+var app = WebApplication.Create();
+
+app.MapGet("/", () => "Hello, World!");
+
+app.Run();
+
+// This is a test
+
+{|#0:public partial class Program;|}
+""";
+
+        var diagnostic = new DiagnosticResult(DiagnosticDescriptors.PublicPartialProgramClassNotRequired)
+                .WithLocation(0);
+
+        var fixedSource = """
+using Microsoft.AspNetCore.Builder;
+
+var app = WebApplication.Create();
+
+app.MapGet("/", () => "Hello, World!");
+
+app.Run();
+
+// This is a test
+
+
 """;
 
         // Assert