Also change links in source file

Author: reakaleek

src/docs-builder/Cli/Commands.cs

@@ -5,7 +5,6 @@
 using System.IO.Abstractions;
 using Actions.Core.Services;
 using ConsoleAppFramework;
-using Documentation.Builder.Diagnostics;
 using Documentation.Builder.Diagnostics.Console;
 using Documentation.Builder.Http;
 using Elastic.Markdown;
@@ -129,7 +128,7 @@ public async Task<int> Move(
 		};
 		var set = new DocumentationSet(context);
 
-		var moveCommand = new MoveCommand(fileSystem, set, logger);
+		var moveCommand = new Move(fileSystem, set, logger);
 		return await moveCommand.Execute(source, target, dryRun ?? false, ctx);
 	}
 }

src/docs-builder/Cli/MoveCommand.cs renamed to src/docs-builder/Cli/Move.cs

@@ -1,16 +1,19 @@
 // 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 System.IO.Abstractions;
 using Elastic.Markdown.IO;
 using Microsoft.Extensions.Logging;
+using System.Text.RegularExpressions;
 
 namespace Documentation.Builder.Cli;
 
-internal class MoveCommand(IFileSystem fileSystem, DocumentationSet documentationSet, ILoggerFactory loggerFactory)
+internal class Move(IFileSystem fileSystem, DocumentationSet documentationSet, ILoggerFactory loggerFactory)
 {
-	private readonly ILogger _logger = loggerFactory.CreateLogger<MoveCommand>();
+	private readonly ILogger _logger = loggerFactory.CreateLogger<Move>();
 	private readonly List<(string filePath, string originalContent,string newContent)> _changes = [];
+	private const string ChangeFormatString = "Change \e[31m{0}\e[0m to \e[32m{1}\e[0m at \e[34m{2}:{3}:{4}\e[0m";
 
 	public async Task<int> Execute(string? source, string? target, bool isDryRun, Cancel ctx = default)
 	{
@@ -25,6 +28,42 @@ public async Task<int> Execute(string? source, string? target, bool isDryRun, Ca
 		var sourcePath = Path.GetFullPath(source!);
 		var targetPath = Path.GetFullPath(target!);
 
+		var (_, sourceMarkdownFile) = documentationSet.MarkdownFiles.Single(i => i.Value.FilePath == sourcePath);
+
+		var soureContent = await fileSystem.File.ReadAllTextAsync(sourceMarkdownFile.FilePath, ctx);
+
+		var markdownLinkRegex = new Regex(@"\[([^\]]*)\]\(((?:\.{0,2}\/)?[^:)]+\.md(?:#[^)]*)?)\)", RegexOptions.Compiled);
+		var matches = markdownLinkRegex.Matches(soureContent);
+
+		var change = Regex.Replace(soureContent, markdownLinkRegex.ToString(), match =>
+		{
+			var originalPath = match.Value.Substring(match.Value.IndexOf('(') + 1, match.Value.LastIndexOf(')') - match.Value.IndexOf('(') - 1);
+			// var anchor = originalPath.Contains('#') ? originalPath[originalPath.IndexOf('#')..] : "";
+
+			string newPath;
+
+			var isAbsoluteStylePath = originalPath.StartsWith('/');
+			if (isAbsoluteStylePath)
+			{
+				newPath = originalPath;
+			} else {
+				var targetDirectory = Path.GetDirectoryName(targetPath)!;
+				var sourceDirectory = Path.GetDirectoryName(sourceMarkdownFile.FilePath)!;
+				var fullPath = Path.GetFullPath(Path.Combine(sourceDirectory, originalPath));
+				var relativePath = Path.GetRelativePath(targetDirectory, fullPath);
+				newPath = relativePath;
+			}
+			var newLink = $"[{match.Groups[1].Value}]({newPath})";
+			var lineNumber = soureContent.Substring(0, match.Index).Count(c => c == '\n') + 1;
+			var columnNumber = match.Index - soureContent.LastIndexOf('\n', match.Index);
+			_logger.LogInformation(string.Format(ChangeFormatString, match.Value, newLink,
+				sourceMarkdownFile.SourceFile.FullName, lineNumber, columnNumber));
+
+			return newLink;
+		});
+
+		_changes.Add((sourceMarkdownFile.FilePath, soureContent, change));
+
 		foreach (var (_, markdownFile) in documentationSet.MarkdownFiles)
 		{
 			await ProcessMarkdownFile(
@@ -53,9 +92,6 @@ await ProcessMarkdownFile(
             throw;
         }
 
-
-
-
 		return 0;
 	}
 
@@ -168,12 +204,6 @@ MarkdownFile value
 					// Absolute style link
 					newLink = $"[{match.Groups[1].Value}]({absoluteStyleTarget}{anchor})";
 				}
-				// else if (originalPath.StartsWith("./"))
-				// {
-				// 	// Relative link with ./ prefix
-				// 	var relativeTarget = Path.Combine(".", Path.GetRelativePath(Path.GetDirectoryName(value.FilePath)!, target)).Replace('\\', '/');
-				// 	newLink = $"[{match.Groups[1].Value}]({relativeTarget}{anchor})";
-				// }
 				else
 				{
 					// Relative link
@@ -182,7 +212,9 @@ MarkdownFile value
 				}
 
 				var lineNumber = content.Substring(0, match.Index).Count(c => c == '\n') + 1;
-				_logger.LogInformation($"Change \e[31m{match.Value}\e[0m to \e[32m{newLink}\e[0m at {value.SourceFile.FullName}:{lineNumber}");
+				var columnNumber = match.Index - content.LastIndexOf('\n', match.Index);
+				_logger.LogInformation(string.Format(ChangeFormatString, match.Value, newLink, value.SourceFile.FullName,
+					lineNumber, columnNumber));
 				return newLink;
 			});
 	}

src/docs-builder/docs-builder.csproj

@@ -32,5 +32,4 @@
   <ItemGroup>
     <ProjectReference Include="..\Elastic.Markdown\Elastic.Markdown.csproj" />
   </ItemGroup>
-  
-</Project>
+</Project>