fix: use delegating console for SpectreLogger configuration

Create DelegatingAnsiConsole wrapper that forwards to the current
AnsiConsole.Console instance. This fixes the conflict where:
- DI setup was configuring width on initial AnsiConsole.Console
- ConsoleCoordinator later replaces AnsiConsole.Console entirely
- SpectreLogger was left using the old console instance

Now SpectreLogger uses DelegatingAnsiConsole.Instance which always
delegates to the current AnsiConsole.Console, ensuring the console
replacement by ConsoleCoordinator works correctly.

Console width configuration is now solely handled by
ConsoleCoordinator.ConfigureConsoleWidth().

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: thomhurst

src/ModularPipelines/Console/DelegatingAnsiConsole.cs

@@ -0,0 +1,34 @@
+using Spectre.Console;
+using Spectre.Console.Rendering;
+
+namespace ModularPipelines.Console;
+
+/// <summary>
+/// A delegating wrapper that always forwards to the current AnsiConsole.Console.
+/// This allows the console instance to be replaced (e.g., by ConsoleCoordinator)
+/// while loggers continue to use the correct console.
+/// </summary>
+internal sealed class DelegatingAnsiConsole : IAnsiConsole
+{
+    public static DelegatingAnsiConsole Instance { get; } = new();
+
+    private DelegatingAnsiConsole()
+    {
+    }
+
+    private static IAnsiConsole Console => AnsiConsole.Console;
+
+    public Profile Profile => Console.Profile;
+
+    public IAnsiConsoleCursor Cursor => Console.Cursor;
+
+    public IAnsiConsoleInput Input => Console.Input;
+
+    public IExclusivityMode ExclusivityMode => Console.ExclusivityMode;
+
+    public RenderPipeline Pipeline => Console.Pipeline;
+
+    public void Clear(bool home) => Console.Clear(home);
+
+    public void Write(IRenderable renderable) => Console.Write(renderable);
+}

src/ModularPipelines/DependencyInjection/DependencyInjectionSetup.cs

@@ -31,7 +31,7 @@
 using ModularPipelines.Options.Validators;
 using ModularPipelines.Engine.State;
 using ModularPipelines.Validation;
-using Spectre.Console;
+using ModularPipelines.Console;
 using Vertical.SpectreLogger;
 using Vertical.SpectreLogger.Options;
 
@@ -79,15 +79,11 @@ private static void RegisterBundledServices(IServiceCollection services)
 
                 builder.AddSpectreConsole(config =>
                 {
-                    // Configure console width for CI environments
-                    // When output is redirected (common in CI), Spectre defaults to 80 chars
-                    // We expand this to 160 for better readability in CI logs
-                    if (System.Console.IsOutputRedirected)
-                    {
-                        AnsiConsole.Console.Profile.Width = 160;
-                    }
-
-                    config.UseConsole(AnsiConsole.Console);
+                    // Use delegating console that forwards to current AnsiConsole.Console
+                    // This allows ConsoleCoordinator to replace the console instance later
+                    // while SpectreLogger continues to use the correct one
+                    // Note: Console width is configured by ConsoleCoordinator.ConfigureConsoleWidth()
+                    config.UseConsole(DelegatingAnsiConsole.Instance);
 
                     // Configure output templates without timestamps
                     // Command logging already includes precise timestamps ([HH:mm:ss.fff])