limit admonitions to attention, caution, note & tip (#64)

Author: Mpdreamz

src/Elastic.Markdown/Myst/Directives/AdmonitionBlock.cs

@@ -7,6 +7,9 @@ public class AdmonitionBlock(DirectiveBlockParser parser, string admonition, Dic
 	: DirectiveBlock(parser, properties)
 {
 	public string Admonition => admonition == "admonition" ? Classes?.Trim() ?? "note" : admonition;
+
+	public override string Directive => Admonition;
+
 	public string? Classes { get; protected set; }
 	public string? CrossReferenceName  { get; private set; }
 	public bool? DropdownOpen  { get; private set; }
@@ -15,7 +18,7 @@ public string Title
 	{
 		get
 		{
-			var t = Admonition == "seealso" ? "see also" : Admonition;
+			var t = Admonition;
 			var title = Thread.CurrentThread.CurrentCulture.TextInfo.ToTitleCase(t);
 			if (admonition is "admonition" && !string.IsNullOrEmpty(Arguments))
 				title = Arguments;

src/Elastic.Markdown/Myst/Directives/CodeBlock.cs

@@ -6,7 +6,7 @@ namespace Elastic.Markdown.Myst.Directives;
 public class CodeBlock(DirectiveBlockParser parser, string directive, Dictionary<string, string> properties)
 	: DirectiveBlock(parser, properties)
 {
-	public string Directive => directive;
+	public override string Directive => directive;
 	public string? Caption { get; private set; }
 	public string? CrossReferenceName { get; private set; }
 

src/Elastic.Markdown/Myst/Directives/DirectiveBlock.cs

@@ -5,6 +5,7 @@
 // This file is licensed under the BSD-Clause 2 license.
 // See the license.txt file in the project root for more information.
 
+using Elastic.Markdown.Diagnostics;
 using Markdig.Helpers;
 using Markdig.Syntax;
 
@@ -24,7 +25,6 @@ namespace Elastic.Markdown.Myst.Directives;
 public abstract class DirectiveBlock(DirectiveBlockParser parser, Dictionary<string, string> properties)
 	: ContainerBlock(parser), IFencedBlock
 {
-
 	public IReadOnlyDictionary<string, string> Properties { get; } = properties;
 
 	/// <inheritdoc />
@@ -89,5 +89,10 @@ protected bool PropBool(params string[] keys)
 		return default;
 	}
 
+	public abstract string Directive { get; }
+
+	protected void EmitError(ParserContext context, string message) =>
+		context.EmitError(Line + 1, 1, Directive.Length + 4 , message);
+
 
 }

src/Elastic.Markdown/Myst/Directives/DirectiveBlockParser.cs

@@ -30,9 +30,7 @@ public DirectiveBlockParser()
 
 	private Dictionary<string, string> _admonitionData = new();
 
-	private readonly string[] _admonitions = [
-		"admonition", "attention", "caution", "danger", "error", "hint", "important", "note", "tip", "seealso"
-	];
+	private readonly string[] _admonitions = [ "attention", "caution", "note", "tip" ];
 
 	private readonly string[] _versionBlocks = [ "versionadded", "versionchanged", "versionremoved", "deprecated" ];
 
@@ -59,7 +57,13 @@ public DirectiveBlockParser()
 		{ "sidebar", 4 },
 		{ "code-cell", 8 },
 
-
+		{ "admonition", 3 },
+		{ "attention", 3 },
+		{ "danger", 3 },
+		{ "error", 3 },
+		{ "hint", 3 },
+		{ "important", 3 },
+		{ "seealso", 3 }
 	}.ToFrozenDictionary();
 
     protected override DirectiveBlock CreateFencedBlock(BlockProcessor processor)

src/Elastic.Markdown/Myst/Directives/ImageBlock.cs

@@ -6,6 +6,8 @@ namespace Elastic.Markdown.Myst.Directives;
 public class ImageBlock(DirectiveBlockParser parser, Dictionary<string, string> properties, ParserContext context)
 	: DirectiveBlock(parser, properties)
 {
+	public override string Directive => "image";
+
 	public BuildContext Build { get; } = context.Build;
 
 	/// <summary>

src/Elastic.Markdown/Myst/Directives/IncludeBlock.cs

@@ -9,6 +9,8 @@ namespace Elastic.Markdown.Myst.Directives;
 public class IncludeBlock(DirectiveBlockParser parser, Dictionary<string, string> properties, ParserContext context)
 	: DirectiveBlock(parser, properties)
 {
+	public override string Directive => "include";
+
 	public BuildContext Build { get; } = context.Build;
 
 	public IFileSystem FileSystem { get; } = context.Build.ReadFileSystem;
@@ -21,8 +23,6 @@ public class IncludeBlock(DirectiveBlockParser parser, Dictionary<string, string
 
 	public bool Found { get; private set; }
 
-	protected virtual string Directive { get; } = "include";
-
 	public bool Literal { get; protected set; }
 	public string? Language { get; private set; }
 	public string? Caption { get; private set; }
@@ -54,7 +54,7 @@ public override void FinalizeAndValidate(ParserContext context)
 		if (FileSystem.File.Exists(IncludePath))
 			Found = true;
 		else
-			context.EmitError(Line, Column, "```{include}".Length , $"`{IncludePath}` does not exist.");
+			EmitError(context, $"`{IncludePath}` does not exist.");
 
 
 	}
@@ -66,5 +66,6 @@ public class LiteralIncludeBlock : IncludeBlock
 	public LiteralIncludeBlock(DirectiveBlockParser parser, Dictionary<string, string> properties, ParserContext context)
 		: base(parser, properties, context) => Literal = true;
 
-	protected override string Directive { get; } = "literalinclude";
+	public override string Directive => "literalinclude";
+
 }

src/Elastic.Markdown/Myst/Directives/MermaidBlock.cs

@@ -6,6 +6,8 @@ namespace Elastic.Markdown.Myst.Directives;
 public class MermaidBlock(DirectiveBlockParser parser, Dictionary<string, string> properties)
 	: DirectiveBlock(parser, properties)
 {
+	public override string Directive => "mermaid";
+
 	public override void FinalizeAndValidate(ParserContext context)
 	{
 	}

src/Elastic.Markdown/Myst/Directives/TabSetBlock.cs

@@ -9,6 +9,8 @@ namespace Elastic.Markdown.Myst.Directives;
 public class TabSetBlock(DirectiveBlockParser parser, Dictionary<string, string> properties)
 	: DirectiveBlock(parser, properties)
 {
+	public override string Directive => "tab-set";
+
 	public int Index { get; set; }
 	public override void FinalizeAndValidate(ParserContext context) => Index = FindIndex();
 
@@ -24,6 +26,8 @@ public int FindIndex()
 public class TabItemBlock(DirectiveBlockParser parser, Dictionary<string, string> properties)
 	: DirectiveBlock(parser, properties)
 {
+	public override string Directive => "tab-set-item";
+
 	public string Title { get; set; } = default!;
 	public int Index { get; set; }
 	public int TabSetIndex { get; set; }

src/Elastic.Markdown/Myst/Directives/UnknownDirectiveBlock.cs

@@ -7,7 +7,7 @@ namespace Elastic.Markdown.Myst.Directives;
 public class UnknownDirectiveBlock(DirectiveBlockParser parser, string directive, Dictionary<string, string> properties)
 	: DirectiveBlock(parser, properties)
 {
-	public string Directive => directive;
+	public override string Directive => directive;
 
 	public override void FinalizeAndValidate(ParserContext context)
 	{

src/Elastic.Markdown/Myst/Directives/UnsupportedDirectiveBlock.cs

@@ -9,7 +9,7 @@ namespace Elastic.Markdown.Myst.Directives;
 public class UnsupportedDirectiveBlock(DirectiveBlockParser parser, string directive, Dictionary<string, string> properties, int issueId)
 	: DirectiveBlock(parser, properties)
 {
-	public string Directive => directive;
+	public override string Directive => directive;
 
 	public string IssueUrl => $"https://github.com/elastic/docs-builder/issues/{issueId}";