Skip validate-link-reference on ci if its run against a repo with no docs

Author: Mpdreamz

src/Elastic.Documentation.Configuration/BuildContext.cs

@@ -77,7 +77,7 @@ public BuildContext(
 			? ReadFileSystem.DirectoryInfo.New(source)
 			: ReadFileSystem.DirectoryInfo.New(Path.Combine(Paths.WorkingDirectoryRoot.FullName));
 
-		(DocumentationSourceDirectory, ConfigurationPath) = FindDocsFolderFromRoot(rootFolder);
+		(DocumentationSourceDirectory, ConfigurationPath) = Paths.FindDocsFolderFromRoot(ReadFileSystem, rootFolder);
 
 		DocumentationCheckoutDirectory = Paths.DetermineSourceDirectoryRoot(DocumentationSourceDirectory);
 
@@ -96,29 +96,4 @@ public BuildContext(
 		};
 	}
 
-	private (IDirectoryInfo, IFileInfo) FindDocsFolderFromRoot(IDirectoryInfo rootPath)
-	{
-		string[] files = ["docset.yml", "_docset.yml"];
-		string[] knownFolders = [rootPath.FullName, Path.Combine(rootPath.FullName, "docs")];
-		var mostLikelyTargets =
-			from file in files
-			from folder in knownFolders
-			select Path.Combine(folder, file);
-
-		var knownConfigPath = mostLikelyTargets.FirstOrDefault(ReadFileSystem.File.Exists);
-		var configurationPath = knownConfigPath is null ? null : ReadFileSystem.FileInfo.New(knownConfigPath);
-		if (configurationPath is not null)
-			return (configurationPath.Directory!, configurationPath);
-
-		configurationPath = rootPath
-			.EnumerateFiles("*docset.yml", SearchOption.AllDirectories)
-			.FirstOrDefault()
-			?? throw new Exception($"Can not locate docset.yml file in '{rootPath}'");
-
-		var docsFolder = configurationPath.Directory
-			?? throw new Exception($"Can not locate docset.yml file in '{rootPath}'");
-
-		return (docsFolder, configurationPath);
-	}
-
 }

src/Elastic.Documentation.Configuration/Paths.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.Diagnostics.CodeAnalysis;
 using System.IO.Abstractions;
 
 namespace Elastic.Documentation.Configuration;
@@ -50,4 +51,47 @@ private static DirectoryInfo GetApplicationFolder()
 		return new DirectoryInfo(elasticPath);
 	}
 
+	public static (IDirectoryInfo, IFileInfo) FindDocsFolderFromRoot(IFileSystem readFileSystem, IDirectoryInfo rootPath)
+	{
+		string[] files = ["docset.yml", "_docset.yml"];
+		string[] knownFolders = [rootPath.FullName, Path.Combine(rootPath.FullName, "docs")];
+		var mostLikelyTargets =
+			from file in files
+			from folder in knownFolders
+			select Path.Combine(folder, file);
+
+		var knownConfigPath = mostLikelyTargets.FirstOrDefault(readFileSystem.File.Exists);
+		var configurationPath = knownConfigPath is null ? null : readFileSystem.FileInfo.New(knownConfigPath);
+		if (configurationPath is not null)
+			return (configurationPath.Directory!, configurationPath);
+
+		configurationPath = rootPath
+			.EnumerateFiles("*docset.yml", SearchOption.AllDirectories)
+			.FirstOrDefault()
+		?? throw new Exception($"Can not locate docset.yml file in '{rootPath}'");
+
+		var docsFolder = configurationPath.Directory ?? throw new Exception($"Can not locate docset.yml file in '{rootPath}'");
+
+		return (docsFolder, configurationPath);
+	}
+
+	public static bool TryFindDocsFolderFromRoot(
+		IFileSystem readFileSystem,
+		IDirectoryInfo rootPath,
+		[NotNullWhen(true)] out IDirectoryInfo? docDirectory,
+		[NotNullWhen(true)] out IFileInfo? configurationPath
+	)
+	{
+		docDirectory = null;
+		configurationPath = null;
+		try
+		{
+			(docDirectory, configurationPath) = FindDocsFolderFromRoot(readFileSystem, rootPath);
+			return true;
+		}
+		catch
+		{
+			return false;
+		}
+	}
 }

src/tooling/docs-assembler/Cli/InboundLinkCommands.cs

@@ -17,13 +17,13 @@ namespace Documentation.Assembler.Cli;
 internal sealed class InboundLinkCommands(ILoggerFactory logger, ICoreService githubActionsService)
 {
 	private readonly LinkIndexLinkChecker _linkIndexLinkChecker = new(logger);
+	private readonly ILogger<Program> _log = logger.CreateLogger<Program>();
 
 	[SuppressMessage("Usage", "CA2254:Template should be a static expression")]
 	private void AssignOutputLogger()
 	{
-		var log = logger.CreateLogger<Program>();
-		ConsoleApp.Log = msg => log.LogInformation(msg);
-		ConsoleApp.LogError = msg => log.LogError(msg);
+		ConsoleApp.Log = msg => _log.LogInformation(msg);
+		ConsoleApp.LogError = msg => _log.LogError(msg);
 	}
 
 	/// <summary> Validate all published cross_links in all published links.json files. </summary>
@@ -64,19 +64,30 @@ public async Task<int> ValidateRepoInboundLinks(string? from = null, string? to
 	/// Validate a locally published links.json file against all published links.json files in the registry
 	/// </summary>
 	/// <param name="file">Path to `links.json` defaults to '.artifacts/docs/html/links.json'</param>
+	/// <param name="path"> -p, Defaults to the `{pwd}` folder</param>
 	/// <param name="ctx"></param>
 	[Command("validate-link-reference")]
-	public async Task<int> ValidateLocalLinkReference([Argument] string? file = null, Cancel ctx = default)
+	public async Task<int> ValidateLocalLinkReference(string? file = null, string? path = null, Cancel ctx = default)
 	{
 		AssignOutputLogger();
 		file ??= ".artifacts/docs/html/links.json";
 		var fs = new FileSystem();
-		var root = fs.DirectoryInfo.New(Paths.WorkingDirectoryRoot.FullName);
+		var root = !string.IsNullOrEmpty(path) ? fs.DirectoryInfo.New(path) : fs.DirectoryInfo.New(Paths.WorkingDirectoryRoot.FullName);
 		var repository = GitCheckoutInformation.Create(root, new FileSystem(), logger.CreateLogger(nameof(GitCheckoutInformation))).RepositoryName
 						?? throw new Exception("Unable to determine repository name");
 
+		var runningOnCi = !string.IsNullOrEmpty(Environment.GetEnvironmentVariable("GITHUB_ACTIONS"));
+		if (runningOnCi && !Paths.TryFindDocsFolderFromRoot(fs, root, out _, out _))
+		{
+			_log.LogInformation("Running in CI on a folder with no docset.yml file in {Directory}, skipping the validation", root.FullName);
+			return 0;
+		}
+
+		var resolvedFile = Path.Combine(root.FullName, file);
+		_log.LogInformation("Validating {File} in {Directory}", file, root.FullName);
+
 		await using var collector = new ConsoleDiagnosticsCollector(logger, githubActionsService).StartAsync(ctx);
-		await _linkIndexLinkChecker.CheckWithLocalLinksJson(collector, repository, file, ctx);
+		await _linkIndexLinkChecker.CheckWithLocalLinksJson(collector, repository, resolvedFile, ctx);
 		await collector.StopAsync(ctx);
 		return collector.Errors;
 	}

src/tooling/docs-builder/Cli/Commands.cs

@@ -125,8 +125,8 @@ public async Task<int> Generate(
 				CanonicalBaseUrl = canonicalBaseUri
 			};
 		}
-		// On CI, we are running on merge commit which may have changes against an older
-		// docs folder (this can happen on out of date PR's).
+		// On CI, we are running on a merge commit which may have changes against an older
+		// docs folder (this can happen on out-of-date PR's).
 		// At some point in the future we can remove this try catch
 		catch (Exception e) when (runningOnCi && e.Message.StartsWith("Can not locate docset.yml file in"))
 		{

src/tooling/docs-builder/Cli/InboundLinkCommands.cs

@@ -83,6 +83,13 @@ public async Task<int> ValidateLocalLinkReference(string? file = null, string? p
 		var repository = GitCheckoutInformation.Create(root, new FileSystem(), logger.CreateLogger(nameof(GitCheckoutInformation))).RepositoryName
 						?? throw new Exception("Unable to determine repository name");
 
+		var runningOnCi = !string.IsNullOrEmpty(Environment.GetEnvironmentVariable("GITHUB_ACTIONS"));
+		if (runningOnCi && !Paths.TryFindDocsFolderFromRoot(fs, root, out _, out _))
+		{
+			_log.LogInformation("Running in CI on a folder with no docset.yml file in {Directory}, skipping the validation", root.FullName);
+			return 0;
+		}
+
 		var resolvedFile = Path.Combine(root.FullName, file);
 		_log.LogInformation("Validating {File} in {Directory}", file, root.FullName);