Remove DisableHtmlWithExceptions and implement UseHardBreaks.

The `DisableHtmlWithExceptions` logic was removed and replaced with a simpler `UseHardBreaks` implementation for handling `<br>` tags. This refactor simplifies the markdown pipeline and improves maintainability while retaining support for hard breaks.

Author: Mpdreamz

src/Elastic.Markdown/Myst/Extensions/DisableHtmlWithExceptionsExtension.cs

@@ -0,0 +1,65 @@
+// Licensed to Elasticsearch B.V under one or more agreements.
+// 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 Markdig;
+using Markdig.Helpers;
+using Markdig.Parsers;
+using Markdig.Parsers.Inlines;
+using Markdig.Renderers;
+using Markdig.Renderers.Html;
+using Markdig.Renderers.Html.Inlines;
+using Markdig.Syntax.Inlines;
+
+namespace Elastic.Markdown.Myst.InlineParsers;
+
+public static class HardBreakBuilderExtensions
+{
+	public static MarkdownPipelineBuilder UseHardBreaks(this MarkdownPipelineBuilder pipeline)
+	{
+		pipeline.Extensions.AddIfNotAlready<HardBreakBuilderExtension>();
+		return pipeline;
+	}
+}
+
+public class HardBreakBuilderExtension : IMarkdownExtension
+{
+	public void Setup(MarkdownPipelineBuilder pipeline) =>
+		pipeline.InlineParsers.InsertBefore<EmphasisInlineParser>(new HardBreakParser());
+
+	public void Setup(MarkdownPipeline pipeline, IMarkdownRenderer renderer) =>
+		renderer.ObjectRenderers.InsertAfter<EmphasisInlineRenderer>(new HardBreakRenderer());
+}
+
+public class HardBreakParser : InlineParser
+{
+	public HardBreakParser() => OpeningCharacters = ['<'];
+
+	public override bool Match(InlineProcessor processor, ref StringSlice slice)
+	{
+		var span = slice.AsSpan();
+		if (!span.StartsWith("<br"))
+			return false;
+
+		var closingStart = span[3..].IndexOf('>');
+		// we allow
+		if (closingStart != 0)
+			return false;
+
+		processor.Inline = new HardBreak();
+
+		var sliceEnd = slice.Start + 4; //<br + >
+		while (slice.Start != sliceEnd)
+			slice.SkipChar();
+
+		return true;
+	}
+}
+
+public class HardBreak : LeafInline;
+
+public class HardBreakRenderer : HtmlObjectRenderer<HardBreak>
+{
+	protected override void Write(HtmlRenderer renderer, HardBreak obj) =>
+		renderer.Write("<br>");
+}

src/Elastic.Markdown/Myst/InlineParsers/HardBreakParser.cs

@@ -9,7 +9,6 @@
 using Elastic.Markdown.Myst.CodeBlocks;
 using Elastic.Markdown.Myst.Comments;
 using Elastic.Markdown.Myst.Directives;
-using Elastic.Markdown.Myst.Extensions;
 using Elastic.Markdown.Myst.FrontMatter;
 using Elastic.Markdown.Myst.InlineParsers;
 using Elastic.Markdown.Myst.Substitution;
@@ -53,7 +52,8 @@ public class MarkdownParser(
 			.UsePipeTables()
 			.UseDirectives()
 			.UseEnhancedCodeBlocks()
-			.DisableHtmlWithExceptions(["<br>"])
+			.DisableHtml()
+			.UseHardBreaks()
 			.Build();
 
 	public ConfigurationFile Configuration { get; } = configuration;

src/Elastic.Markdown/Myst/InlineParsers/RestrictedAutolinkInlineParser.cs

@@ -7,7 +7,7 @@
 
 namespace Elastic.Markdown.Tests.Inline;
 
-public class AllowBRTagTest(ITestOutputHelper output)
+public class AllowBrTagTest(ITestOutputHelper output)
 	: InlineTest(output,
 		"Hello,<br>World!")
 {
@@ -18,6 +18,17 @@ public void GeneratesHtml() =>
 		);
 }
 
+public class BrTagNeedsToBeExact(ITestOutputHelper output)
+	: InlineTest(output,
+		"Hello,<br >World<br />!")
+{
+	[Fact]
+	public void GeneratesHtml() =>
+		Html.Should().Contain(
+			"<p>Hello,&lt;br &gt;World&lt;br /&gt;!</p>"
+		);
+}
+
 public class DisallowSpanTag(ITestOutputHelper output)
 	: InlineTest(output,
 		"Hello,<span>World!</span>")