Fix select clet: inline rendering, text output, theming, positional args

- Render input clets inline (AppModel.Inline) instead of fullscreen;
  viewers and --fullscreen flag use alt-screen. Matches gum/fzf UX.
- Return selected text instead of index (IClet<string?>) — every major
  CLI competitor (gum choose, fzf, inquirer) outputs text.
- Accept positional args as choices: `clet select a b c` works;
  --options flag remains as fallback. --initial matches by label text.
- Enable ConfigurationManager.Enable(ConfigLocations.All) for theming.
- Border thickness top-only for clean inline appearance.
- Dispose app before writing output (driver was swallowing stdout).
- Use platform default driver (not "ansi") for interactive sessions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: tig

CLAUDE.md

@@ -25,7 +25,7 @@ There is no separate lint step. CI runs on ubuntu-latest with `dotnet-quality: p
 
 Four projects in two repos (this repo only contains the `clet` side):
 
-- **`src/Clet/`** — The CLI executable (net10.0). Depends on `Terminal.Gui` v2 (preview NuGet, currently `2.0.2-develop.21` — pin tracked in `src/Clet/Clet.csproj`, must be replaced with a release tag before v0.5 schema-lock per spec §8 risks). All abstractions are `internal` (not published until v2 plugin system).
+- **`src/Clet/`** — The CLI executable (net10.0). Depends on `Terminal.Gui` v2 (preview NuGet, currently `2.0.2-develop.24` — pin tracked in `src/Clet/Clet.csproj`, must be replaced with a release tag before v0.5 schema-lock per spec §8 risks). All abstractions are `internal` (not published until v2 plugin system).
 - **`src/Clet.SourceGen/`** — Roslyn source generator for static clet registration (planned `[Clet]` attribute). Currently a placeholder; `BuiltInClets.RegisterAll` is hand-written until the generator earns its keep — see `specs/decisions.md` D-004.
 - **`tests/Clet.UnitTests/`** — Registry, JSON schema, host pipeline (CommandLineRoot, OutputFormatter, ExitCodes, BuiltInClets) tests.
 - **`tests/Clet.IntegrationTests/`** — In-process tests that init Terminal.Gui (`Application.Create()`, `app.Init("ansi")`).

src/Clet/Abstractions/CletRunOptions.cs

@@ -7,4 +7,5 @@ internal sealed record CletRunOptions
     public TimeSpan? Timeout { get; init; }
     public bool Fullscreen { get; init; }
     public IReadOnlyDictionary<string, string>? CletOptions { get; init; }
+    public IReadOnlyList<string>? Arguments { get; init; }
 }

src/Clet/Clets/Input/SelectClet.cs

@@ -5,20 +5,20 @@
 
 namespace Clet;
 
-internal sealed class SelectClet : IClet<int?>
+internal sealed class SelectClet : IClet<string?>
 {
     public string PrimaryAlias => "select";
     public IReadOnlyList<string> Aliases => ["select"];
-    public string Description => "Presents a list of options and returns the zero-based index of the selected item.";
+    public string Description => "Presents a list of options and returns the text of the selected item.";
     public CletKind Kind => CletKind.Input;
-    public Type ResultType => typeof (int?);
+    public Type ResultType => typeof (string);
 
     public IReadOnlyList<CletOptionDescriptor> Options =>
     [
         new ("options", "o", typeof (string), "Comma-separated list of options to display.", true, null),
     ];
 
-    public async Task<CletRunResult<int?>> RunAsync (
+    public async Task<CletRunResult<string?>> RunAsync (
         IApplication app,
         string? initial,
         CletRunOptions options,
@@ -29,19 +29,26 @@ internal sealed class SelectClet : IClet<int?>
             return new () { Status = CletRunStatus.Cancelled };
         }
 
-        string[] labels = options.CletOptions?.TryGetValue ("options", out string? optionsValue) == true
-            ? optionsValue?.Split (',') ?? []
-            : [];
+        string[] labels = options.Arguments is { Count: > 0 }
+            ? options.Arguments.ToArray ()
+            : options.CletOptions?.TryGetValue ("options", out string? optionsValue) == true
+                ? optionsValue?.Split (',') ?? []
+                : [];
 
         OptionSelector selector = new ()
         {
             Labels = labels,
             AssignHotKeys = true,
         };
 
-        if (int.TryParse (initial, out int initialIndex) && initialIndex >= 0 && initialIndex < labels.Length)
+        if (initial is not null)
         {
-            selector.Value = initialIndex;
+            int initialIdx = Array.FindIndex (labels, l => string.Equals (l, initial, StringComparison.OrdinalIgnoreCase));
+
+            if (initialIdx >= 0)
+            {
+                selector.Value = initialIdx;
+            }
         }
 
         RunnableWrapper<OptionSelector, int?> wrapper = new (selector)
@@ -50,6 +57,7 @@ internal sealed class SelectClet : IClet<int?>
             Width = Dim.Fill (),
             BorderStyle = LineStyle.Rounded,
         };
+        wrapper.Border.Thickness = new Thickness (0, 1, 0, 0);
 
         try
         {
@@ -60,8 +68,16 @@ internal sealed class SelectClet : IClet<int?>
             return new () { Status = CletRunStatus.Cancelled };
         }
 
-        return cancellationToken.IsCancellationRequested
-            ? new () { Status = CletRunStatus.Cancelled }
-            : new () { Status = CletRunStatus.Ok, Value = wrapper.Result };
+        if (cancellationToken.IsCancellationRequested)
+        {
+            return new () { Status = CletRunStatus.Cancelled };
+        }
+
+        int? selectedIndex = wrapper.Result;
+        string? selectedText = selectedIndex is >= 0 and var idx && idx < labels.Length
+            ? labels [idx]
+            : null;
+
+        return new () { Status = CletRunStatus.Ok, Value = selectedText };
     }
 }

src/Clet/Hosting/AliasDispatcher.cs

@@ -1,4 +1,5 @@
 using Terminal.Gui.App;
+using Terminal.Gui.Configuration;
 
 namespace Clet;
 
@@ -32,20 +33,27 @@ public async Task<int> DispatchAsync (
 
         BoxedCletResult result;
 
-        using IApplication app = Application.Create ();
-        app.Init ("ansi");
-
-        try
-        {
-            result = await clet.RunBoxedAsync (app, initial, options, linkedSource.Token);
-        }
-        catch (OperationCanceledException)
-        {
-            result = new (CletRunStatus.Cancelled, null, null, null);
-        }
-        catch (Exception ex)
         {
-            result = new (CletRunStatus.Error, null, "io", ex.Message);
+            ConfigurationManager.Enable (ConfigLocations.All);
+
+            bool useFullscreen = options.Fullscreen || clet.Kind == CletKind.Viewer;
+            Application.AppModel = useFullscreen ? AppModel.FullScreen : AppModel.Inline;
+
+            using IApplication app = Application.Create ();
+            app.Init ();
+
+            try
+            {
+                result = await clet.RunBoxedAsync (app, initial, options, linkedSource.Token);
+            }
+            catch (OperationCanceledException)
+            {
+                result = new (CletRunStatus.Cancelled, null, null, null);
+            }
+            catch (Exception ex)
+            {
+                result = new (CletRunStatus.Error, null, "io", ex.Message);
+            }
         }
 
         OutputFormatter.Write (result, options.JsonOutput, stdout, stderr);

src/Clet/Hosting/CommandLineRoot.cs

@@ -62,6 +62,7 @@ private async Task<int> DispatchAlias (
         bool fullscreen = false;
         TimeSpan? timeout = null;
         Dictionary<string, string> cletOptions = new (StringComparer.OrdinalIgnoreCase);
+        List<string> positionalArgs = [];
 
         for (int i = 1; i < args.Length; i++)
         {
@@ -130,16 +131,7 @@ private async Task<int> DispatchAlias (
                 continue;
             }
 
-            if (initial is null)
-            {
-                initial = arg;
-
-                continue;
-            }
-
-            stderr.WriteLine ($"error: unexpected positional argument '{arg}'.");
-
-            return ExitCodes.UsageError;
+            positionalArgs.Add (arg);
         }
 
         CletRunOptions options = new ()
@@ -148,6 +140,7 @@ private async Task<int> DispatchAlias (
             Fullscreen = fullscreen,
             Timeout = timeout,
             CletOptions = cletOptions,
+            Arguments = positionalArgs.Count > 0 ? positionalArgs : null,
         };
 
         return await _dispatcher.DispatchAsync (alias, initial, options, cancellationToken, stdout, stderr);

tests/Clet.IntegrationTests/SelectCletIntegrationTests.cs

@@ -23,7 +23,7 @@ public async Task RunAsync_CancellationToken_AlreadyCancelled_ReturnsCancelled (
         using CancellationTokenSource cts = new ();
         cts.Cancel ();
 
-        CletRunResult<int?> result = await clet.RunAsync (app, null, options, cts.Token);
+        CletRunResult<string?> result = await clet.RunAsync (app, null, options, cts.Token);
 
         Assert.Equal (CletRunStatus.Cancelled, result.Status);
         Assert.Null (result.Value);
@@ -45,7 +45,7 @@ public async Task RunAsync_WithStopAfterFirstIteration_ReturnsOk ()
 
         using CancellationTokenSource cts = new ();
 
-        CletRunResult<int?> result = await clet.RunAsync (app, null, options, cts.Token);
+        CletRunResult<string?> result = await clet.RunAsync (app, null, options, cts.Token);
 
         // Run returns after one iteration — result is Ok (value may be null since no input)
         Assert.Equal (CletRunStatus.Ok, result.Status);
@@ -67,7 +67,7 @@ public async Task RunAsync_WithInitialValue_SetsSelection ()
 
         using CancellationTokenSource cts = new ();
 
-        CletRunResult<int?> result = await clet.RunAsync (app, "2", options, cts.Token);
+        CletRunResult<string?> result = await clet.RunAsync (app, "Y", options, cts.Token);
 
         // Should complete without error
         Assert.Equal (CletRunStatus.Ok, result.Status);

tests/Clet.UnitTests/CommandLineRootTests.cs

@@ -92,7 +92,7 @@ public async Task List_Json_EmitsSchemaVersion1 ()
         Assert.Contains ("\"schemaVersion\":1", output);
         Assert.Contains ("\"alias\":\"select\"", output);
         Assert.Contains ("\"kind\":\"input\"", output);
-        Assert.Contains ("\"resultType\":\"int\"", output);
+        Assert.Contains ("\"resultType\":\"string\"", output);
     }
 
     [Theory]

tests/Clet.UnitTests/SelectCletTests.cs

@@ -25,7 +25,7 @@ public void ResultType_IsNullableInt ()
     {
         SelectClet clet = new ();
 
-        Assert.Equal (typeof (int?), clet.ResultType);
+        Assert.Equal (typeof (string), clet.ResultType);
     }
 
     [Fact]