chore: update code based on review comment

Author: filzrev

src/BenchmarkDotNet/Loggers/CompositeLogger.cs

@@ -6,7 +6,7 @@
 
 namespace BenchmarkDotNet.Loggers
 {
-    internal class CompositeLogger : ILogger
+    internal class CompositeLogger : ILogger, ILinkLogger
     {
         private readonly ImmutableHashSet<ILogger> loggers;
 
@@ -61,11 +61,5 @@ public void Flush()
                 }
             }
         }
-
-        public bool TryGetConsoleLogger([NotNullWhen(true)] out IConsoleLogger? consoleLogger)
-        {
-            consoleLogger = loggers.OfType<IConsoleLogger>().SingleOrDefault();
-            return consoleLogger != null;
-        }
     }
 }

src/BenchmarkDotNet/Loggers/ConsoleLogger.cs

@@ -9,7 +9,7 @@
 
 namespace BenchmarkDotNet.Loggers
 {
-    public sealed class ConsoleLogger : ILogger, IConsoleLogger
+    public sealed class ConsoleLogger : ILogger, ILinkLogger
     {
         private const ConsoleColor DefaultColor = ConsoleColor.Gray;
 

src/BenchmarkDotNet/Loggers/IConsoleLogger.cs

@@ -0,0 +1,8 @@
+namespace BenchmarkDotNet.Loggers;
+
+/// <summary>
+/// Marker interface for a logger that supports clickable link with ANSI escape sequence.
+/// </summary>
+internal interface ILinkLogger : ILogger
+{
+}

src/BenchmarkDotNet/Loggers/ILinkLogger.cs

@@ -0,0 +1,45 @@
+using BenchmarkDotNet.Helpers;
+
+namespace BenchmarkDotNet.Loggers;
+
+#nullable enable
+
+public static class ILoggerExtensions
+{
+    /// <summary>
+    /// Write clickable link to logger.
+    /// If the logger doesn't implement <see cref="ILinkLogger"/>. It's written as plain text.
+    /// </summary>
+    public static void WriteLink(this ILogger logger, string link, string? linkCaption = null, LogKind logKind = LogKind.Info)
+    {
+        if (logger is ILinkLogger)
+        {
+            if (ConsoleHelper.TryGetClickableLink(link, linkCaption, out var clickableLink))
+                link = clickableLink;
+        }
+
+        logger.Write(logKind, link);
+    }
+
+    /// <summary>
+    /// Write clickable link to logger.
+    /// If the logger doesn't implement <see cref="ILinkLogger"/>. It's written as plain text.
+    /// </summary>
+    public static void WriteLineLink(this ILogger logger, string link, string? linkCaption = null, string prefixText = "", string suffixText = "", LogKind logKind = LogKind.Info)
+    {
+        if (logger is ILinkLogger)
+        {
+            if (ConsoleHelper.TryGetClickableLink(link, linkCaption, out var clickableLink))
+            {
+                link = clickableLink;
+
+                // Temporary workaround for Windows Terminal.
+                // To avoid link style corruption issue when output ends with a clickable link and window is resized.
+                if (ConsoleHelper.IsWindowsTerminal && suffixText == "")
+                    suffixText = " ";
+            }
+        }
+
+        logger.WriteLine(logKind, $"{prefixText}{link}{suffixText}");
+    }
+}

src/BenchmarkDotNet/Loggers/ILoggerExtensions.cs

@@ -194,17 +194,17 @@ internal static Summary[] Run(BenchmarkRunInfo[] benchmarkRunInfos)
 
                     // Output additional information to console.
                     var logFileEnabled = benchmarkRunInfos.All(info => !info.Config.Options.IsSet(ConfigOptions.DisableLogFile));
-                    if (logFileEnabled && compositeLogger.TryGetConsoleLogger(out var consoleLogger))
+                    if (logFileEnabled)
                     {
                         var artifactDirectoryFullPath = Path.GetFullPath(rootArtifactsFolderPath);
                         var logFileFullPath = Path.GetFullPath(logFilePath);
                         var logFileRelativePath = PathHelper.GetRelativePath(artifactDirectoryFullPath, logFileFullPath);
 
-                        consoleLogger.WriteLine();
-                        consoleLogger.WriteLineHeader("// * Benchmark LogFile *");
-                        consoleLogger.WriteLineLink(artifactDirectoryFullPath);
-                        consoleLogger.WriteLineLink(logFileFullPath, linkCaption: logFileRelativePath, prefixText: "  ");
-                        consoleLogger.Flush();
+                        compositeLogger.WriteLine();
+                        compositeLogger.WriteLineHeader("// * Benchmark LogFile *");
+                        compositeLogger.WriteLineLink(artifactDirectoryFullPath);
+                        compositeLogger.WriteLineLink(logFileFullPath, linkCaption: logFileRelativePath, prefixText: "  ");
+                        compositeLogger.Flush();
                     }
 
                     eventProcessor.OnEndRunStage();
@@ -768,7 +768,7 @@ private static StreamWriter GetLogFileStreamWriter(BenchmarkRunInfo[] benchmarkR
             return new StreamWriter(logFilePath, append: false);
         }
 
-        private static CompositeLogger CreateCompositeLogger(BenchmarkRunInfo[] benchmarkRunInfos, StreamLogger streamLogger)
+        private static ILogger CreateCompositeLogger(BenchmarkRunInfo[] benchmarkRunInfos, StreamLogger streamLogger)
         {
             var loggers = new Dictionary<string, ILogger>();