Refactor method

Author: theletterf

src/Elastic.Markdown/Myst/Renderers/LlmMarkdown/LlmBlockRenderers.cs

@@ -42,16 +42,26 @@ public static void RenderBlockWithIndentation(LlmMarkdownRenderer renderer, Mark
 	/// Converts relative URLs to absolute URLs using BuildContext.CanonicalBaseUrl for better LLM consumption
 	/// </summary>
 	public static string? MakeAbsoluteUrl(LlmMarkdownRenderer renderer, string? url)
+	{
+		if (renderer.BuildContext.CanonicalBaseUrl == null)
+			return url;
+
+		return MakeAbsoluteUrl(renderer.BuildContext.CanonicalBaseUrl, url);
+	}
+
+	/// <summary>
+	/// Converts relative URLs to absolute URLs for LLM consumption
+	/// </summary>
+	public static string? MakeAbsoluteUrl(Uri? baseUri, string? url)
 	{
 		if (
 			string.IsNullOrEmpty(url)
-			|| renderer.BuildContext.CanonicalBaseUrl == null
+			|| baseUri == null
 			|| Uri.IsWellFormedUriString(url, UriKind.Absolute)
 			|| !Uri.IsWellFormedUriString(url, UriKind.Relative))
 			return url;
 		try
 		{
-			var baseUri = renderer.BuildContext.CanonicalBaseUrl;
 			var absoluteUri = new Uri(baseUri, url);
 			return absoluteUri.ToString();
 		}
@@ -62,24 +72,28 @@ public static void RenderBlockWithIndentation(LlmMarkdownRenderer renderer, Mark
 	}
 
 	/// <summary>
-	/// Converts documentation URLs to absolute markdown URLs specifically for LLM consumption.
-	/// Transforms HTML paths to .md file paths and ensures they're absolute URLs.
+	/// Converts documentation URLs to absolute markdown URLs for LLM consumption
 	/// </summary>
-	/// <param name="url">The documentation URL to convert (e.g., "/docs/solutions/search/")</param>
-	/// <param name="baseUrl">The base URL to use (defaults to "https://www.elastic.co")</param>
-	/// <returns>Absolute markdown URL (e.g., "https://www.elastic.co/docs/solutions/search.md")</returns>
-	public static string ConvertToAbsoluteMarkdownUrl(string url, string? baseUrl = null)
+	public static string MakeAbsoluteMarkdownUrl(Uri baseUri, string? url)
 	{
 		if (string.IsNullOrEmpty(url))
-			return url;
+			return string.Empty;
+
+		// Convert to .md URL for LLM consumption
+		var markdownUrl = ConvertToMarkdownUrl(url);
 
-		baseUrl ??= "https://www.elastic.co";
+		// Use the existing absolute URL logic
+		return MakeAbsoluteUrl(baseUri, markdownUrl) ?? markdownUrl;
+	}
 
-		// Convert HTML URLs to .md URLs for LLM consumption
-		// e.g., "/docs/solutions/search/" -> "https://www.elastic.co/docs/solutions/search.md"
+	/// <summary>
+	/// Converts documentation URL paths to .md file paths for LLM consumption
+	/// </summary>
+	private static string ConvertToMarkdownUrl(string url)
+	{
 		var cleanUrl = url.TrimStart('/');
 
-		// Remove "docs/" prefix if present for the markdown filename
+		// Remove "docs/" prefix if present for the markdown filename, then add it back
 		var markdownPath = cleanUrl;
 		if (markdownPath.StartsWith("docs/", StringComparison.Ordinal))
 			markdownPath = markdownPath.Substring(5);
@@ -90,9 +104,9 @@ public static string ConvertToAbsoluteMarkdownUrl(string url, string? baseUrl =
 		else if (!markdownPath.EndsWith(".md", StringComparison.Ordinal))
 			markdownPath += ".md";
 
-		// Make absolute URL using the canonical base URL
-		return $"{baseUrl.TrimEnd('/')}/docs/{markdownPath}";
+		return $"/docs/{markdownPath}";
 	}
+
 }
 
 /// <summary>

src/services/Elastic.Documentation.Assembler/Building/AssemblerBuildService.cs

@@ -117,7 +117,9 @@ private static async Task EnhanceLlmsTxtFile(AssembleContext context, GlobalNavi
 			return; // No llms.txt file to enhance
 
 		var existingContent = await readFs.File.ReadAllTextAsync(llmsTxtPath, ctx);
-		var navigationSections = enhancer.GenerateNavigationSections(navigation);
+		// Assembler always uses the production URL as canonical base URL
+		var canonicalBaseUrl = new Uri("https://www.elastic.co");
+		var navigationSections = enhancer.GenerateNavigationSections(navigation, canonicalBaseUrl);
 
 		// Append the navigation sections to the existing boilerplate
 		var enhancedContent = existingContent + Environment.NewLine + navigationSections;

src/services/Elastic.Documentation.Assembler/Navigation/LlmsNavigationEnhancer.cs

@@ -20,7 +20,7 @@ namespace Elastic.Documentation.Assembler.Navigation;
 /// </summary>
 public class LlmsNavigationEnhancer
 {
-	public string GenerateNavigationSections(GlobalNavigation navigation)
+	public string GenerateNavigationSections(GlobalNavigation navigation, Uri canonicalBaseUrl)
 	{
 		var content = new StringBuilder();
 
@@ -45,7 +45,7 @@ public string GenerateNavigationSections(GlobalNavigation navigation)
 				foreach (var child in firstLevelChildren)
 				{
 					var title = GetBestTitle(child);
-					var url = LlmRenderingHelpers.ConvertToAbsoluteMarkdownUrl(child.Url);
+					var url = LlmRenderingHelpers.MakeAbsoluteMarkdownUrl(canonicalBaseUrl, child.Url);
 					var description = GetDescription(child);
 
 					_ = !string.IsNullOrEmpty(description)