Warn of files with no title

Author: Mpdreamz

src/Elastic.Markdown/Diagnostics/DiagnosticsChannel.cs

@@ -138,4 +138,26 @@ public virtual async Task StopAsync(CancellationToken cancellationToken)
 			await _started;
 		await Channel.Reader.Completion;
 	}
+
+
+	public void EmitError(string file, string message)
+	{
+		var d = new Diagnostic
+		{
+			Severity = Severity.Error,
+			File = file,
+			Message = message,
+		};
+		Channel.Write(d);
+	}
+	public void EmitWarning(string file, string message)
+	{
+		var d = new Diagnostic
+		{
+			Severity = Severity.Warning,
+			File = file,
+			Message = message,
+		};
+		Channel.Write(d);
+	}
 }

src/Elastic.Markdown/IO/MarkdownFile.cs

@@ -2,6 +2,7 @@
 // Elasticsearch B.V licenses this file to you under the Apache 2.0 License.
 // See the LICENSE file in the project root for more information
 using System.IO.Abstractions;
+using Elastic.Markdown.Diagnostics;
 using Elastic.Markdown.Myst;
 using Elastic.Markdown.Myst.Directives;
 using Elastic.Markdown.Slices;
@@ -23,8 +24,11 @@ public MarkdownFile(IFileInfo sourceFile, IDirectoryInfo rootPath, MarkdownParse
 		FileName = sourceFile.Name;
 		UrlPathPrefix = context.UrlPathPrefix;
 		MarkdownParser = parser;
+		Collector = context.Collector;
 	}
 
+	public DiagnosticsCollector Collector { get; }
+
 	public string? UrlPathPrefix { get; }
 	private MarkdownParser MarkdownParser { get; }
 	public YamlFrontMatter? YamlFrontMatter { get; private set; }
@@ -60,6 +64,8 @@ public async Task<MarkdownDocument> ParseFullAsync(Cancel ctx)
 			await MinimalParse(ctx);
 
 		var document = await MarkdownParser.ParseAsync(SourceFile, YamlFrontMatter, ctx);
+		if (Title == RelativePath)
+			Collector.EmitWarning(SourceFile.FullName, "Missing yaml front-matter block defining a title or a level 1 header");
 		return document;
 	}
 

tests/Elastic.Markdown.Tests/Directives/YamlFrontMatterTests.cs

@@ -32,3 +32,17 @@ public void ReadsSubstitutions()
 			.And.ContainKey("key");
 	}
 }
+
+public class EmptyFileWarnsNeedingATitle(ITestOutputHelper output) : DirectiveTest(output, "")
+{
+	[Fact]
+	public void ReadsTitle() => File.Title.Should().Be("index.md");
+
+	[Fact]
+	public void ReadsNavigationTitle() => File.NavigationTitle.Should().Be("index.md");
+
+	[Fact]
+	public void WarnsOfNoTitle() =>
+		Collector.Diagnostics.Should().NotBeEmpty()
+			.And.Contain(d=>d.Message.Contains("Missing yaml front-matter block defining a title"));
+}