Updated code and tests.

Author: MattFromRVA

StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1514UnitTests.cs

@@ -1169,5 +1169,64 @@ public class TestClass
 
             await VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
         }
+
+        [Fact]
+        public async Task TestClassInGlobalNamespaceWithoutNewlineAsync()
+        {
+            var testCode = @"/// <summary>
+/// X.
+/// </summary>
+public class TestClass
+{
+}
+";
+
+            var expected = DiagnosticResult.EmptyDiagnosticResults;
+
+            await VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
+        }
+
+        [Fact]
+        public async Task TestClassInGlobalNamespaceWithCommentAsync()
+        {
+            var testCode = @"
+// Normal comment
+{|#0:///|} <summary>
+/// X.
+/// </summary>
+public class TestClass
+{
+}
+";
+
+            var expected = new[]
+            {
+                Diagnostic().WithLocation(0).WithArguments(" not", "preceded"),
+            };
+
+            await VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
+        }
+
+        [Fact]
+        public async Task TestClassInGlobalNamespaceWithPreprocessorDirectiveAsync()
+        {
+            var testCode = @"
+#if DEBUG
+#endif
+{|#0:///|} <summary>
+/// X.
+/// </summary>
+public class TestClass
+{
+}
+";
+
+            var expected = new[]
+            {
+        Diagnostic().WithLocation(0).WithArguments(" not", "preceded"),
+            };
+
+            await VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
+        }
     }
 }

StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1514ElementDocumentationHeaderMustBePrecededByBlankLine.cs

@@ -174,13 +174,8 @@ private static void HandleDeclaration(SyntaxNodeAnalysisContext context)
                 // Logic to handle global namespace case
                 if (prevToken.IsKind(SyntaxKind.None))
                 {
-                    // Check if the node is the first non-trivia element in the file
-                    var firstToken = context.Node.SyntaxTree.GetRoot().GetFirstToken();
-                    if (firstToken == context.Node.GetFirstToken())
-                    {
-                        // Node is the first element in the global namespace
-                        return;
-                    }
+                    // Node is the first element in the global namespace
+                    return;
                 }
             }