Make IConsoleDriver public and add ConsoleDriverFactory for extensibility

[Copilot]

IConsoleDriver and DefaultConsoleDriver were internal, blocking custom console driver implementations and making forms untestable without a real interactive console.

Changes

• IConsoleDriver — promoted from internal to public
• DefaultConsoleDriver — promoted from internal sealed to public (unsealed to allow subclassing)
• Prompt.ConsoleDriverFactory — new Func<IConsoleDriver> property on Prompt, defaults to () => new DefaultConsoleDriver(); setter guards against null
• FormBase — constructor now calls Prompt.ConsoleDriverFactory() instead of hardcoding new DefaultConsoleDriver()
• ConsoleDriverFactoryTests — new test class covering: default factory is non-null, custom factory assignment, null factory throws ArgumentNullException, factory reset, and custom IConsoleDriver implementation
• IConsoleDriverTests — new test class covering all 17 interface members via a RecordingConsoleDriver stub: all methods (Beep, Reset, ClearLine, ReadKey, Write, WriteLine, SetCursorPosition, Dispose), all properties (KeyAvailable, CursorVisible, CursorLeft, CursorTop, BufferWidth, BufferHeight, WindowWidth, WindowHeight), and CancellationCallback

Usage

// Inject a custom or mock driver globally before running prompts
Prompt.ConsoleDriverFactory = () => new MyCustomConsoleDriver();

Fully backward-compatible — no changes required for existing consumers.

Original prompt

全体的なアーキテクチャ設計を見直してみてください

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[shibayan]

@copilot Please add test cases

[Copilot]

@copilot Please add test cases

Added ConsoleDriverFactoryTests in commit 975d9f1 covering:

• Default factory is non-null
• Setting a custom factory stores and invokes it correctly
• Setting null throws ArgumentNullException
• Factory can be reset to a previous value
• A custom IConsoleDriver implementation can be assigned and retrieved

[shibayan]

@copilot Please add IConsoleDriver test cases

[Copilot]

@copilot Please add IConsoleDriver test cases

Added IConsoleDriverTests in commit 93afb8c covering all 17 interface members via a RecordingConsoleDriver stub:

• Methods: Beep, Reset, ClearLine (with argument verification), ReadKey (return value), Write (value + color), WriteLine, SetCursorPosition (coordinates), Dispose
• Properties: KeyAvailable, CursorVisible (set), CursorLeft, CursorTop, BufferWidth, BufferHeight, WindowWidth, WindowHeight
• CancellationCallback (set + invoke)

[Copilot]

Pull request overview

This PR makes the console driver abstraction extensible by exposing IConsoleDriver publicly and introducing Prompt.ConsoleDriverFactory so consumers (and tests) can inject custom console implementations instead of hardwiring DefaultConsoleDriver.

Changes:

• Promote IConsoleDriver and DefaultConsoleDriver to public and add Prompt.ConsoleDriverFactory (with null-guard on setter).
• Update FormBase to construct its driver via Prompt.ConsoleDriverFactory() and set CancellationCallback.
• Add new unit tests covering the factory behavior and a stub implementation exercising the interface members.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
Sharprompt/Prompt.Configuration.cs	Adds global ConsoleDriverFactory configuration for driver injection.
Sharprompt/Forms/FormBase.cs	Uses Prompt.ConsoleDriverFactory() instead of directly instantiating DefaultConsoleDriver.
Sharprompt/Drivers/IConsoleDriver.cs	Makes the console driver interface public for external implementations/mocks.
Sharprompt/Drivers/DefaultConsoleDriver.cs	Makes the default driver public (and no longer sealed).
Sharprompt.Tests/ConsoleDriverFactoryTests.cs	Verifies default/custom factory assignment and null-guard behavior.
Sharprompt.Tests/IConsoleDriverTests.cs	Adds a recording stub to exercise the public interface contract.

Comments suppressed due to low confidence (1)

Sharprompt/Drivers/DefaultConsoleDriver.cs:17

• DefaultConsoleDriver is now public and the type initializer throws when IO is redirected. Exceptions from static constructors are surfaced as TypeInitializationException, which is a confusing failure mode for public consumers. Consider moving the environment checks into the instance constructor (or a factory method) so callers get the intended InvalidOperationException directly, and avoid making type-loading fail for merely referencing the type.

public class DefaultConsoleDriver : IConsoleDriver
{
    static DefaultConsoleDriver()
    {
        if (Console.IsInputRedirected || Console.IsOutputRedirected)
        {
            throw new InvalidOperationException(Resource.Message_NotSupportedEnvironment);
        }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.