refactor(cli): migrate to typed .argument() for positional args

Move positional args from .command() string to .argument() calls,
unifying the action callback to receive a single typed opts object.

Author: beorn

packages/admin/src/index.ts

@@ -55,50 +55,54 @@ probe.addHelpSection("Examples:", [
 // ── probe termless ──
 
 probe
-  .command("termless [selectors...]")
+  .command("termless")
+  .argument("[selectors...]", "Backend selectors to probe")
   .description("Headless library probes via Termless backends")
   .option("--all", "Probe all backends")
   .option("-f, --force", "Re-run even if cached")
-  .action(async (selectors: string[], opts: { all?: boolean; force?: boolean }) => {
+  .action(async (opts: { selectors: string[]; all?: boolean; force?: boolean }) => {
     const { runTermlessProbes } = await import("./termless.ts")
-    await runTermlessProbes(opts.all ? [] : selectors, opts)
+    await runTermlessProbes(opts.all ? [] : opts.selectors, opts)
   })
 
 // ── probe server ──
 
 probe
-  .command("server [daemon]")
+  .command("server")
+  .argument("[daemon]", "Daemon name to probe")
   .description("Probe running daemon servers")
   .option("--start", "Start daemon in this terminal")
   .option("-p, --port <port>", "Port for --start", parseInt)
   .option("--all", "Probe all running daemons")
-  .action(async (daemon: string | undefined, opts: { start?: boolean; port?: number; all?: boolean }) => {
+  .action(async (opts: { daemon?: string; start?: boolean; port?: number; all?: boolean }) => {
     const { handleServer } = await import("./server.ts")
-    await handleServer(daemon, opts)
+    await handleServer(opts.daemon, opts)
   })
 
 // ── probe app ──
 
 probe
-  .command("app [terminal]")
+  .command("app")
+  .argument("[terminal]", "Terminal app to probe")
   .description("Launch and probe macOS terminal apps")
   .option("--all", "Probe all installed terminals")
   .option("-f, --force", "Re-run even if cached")
-  .action(async (terminal: string | undefined, opts: { all?: boolean; force?: boolean }) => {
+  .action(async (opts: { terminal?: string; all?: boolean; force?: boolean }) => {
     const { handleApp } = await import("./app.ts")
-    await handleApp(terminal, opts)
+    await handleApp(opts.terminal, opts)
   })
 
 // ── probe mux ──
 
 probe
-  .command("mux [multiplexer]")
+  .command("mux")
+  .argument("[multiplexer]", "Multiplexer to probe (tmux, screen)")
   .description("Probe through terminal multiplexers (tmux, screen)")
   .option("--all", "Probe through all installed multiplexers")
   .option("-f, --force", "Re-run even if cached")
-  .action(async (multiplexer: string | undefined, opts: { all?: boolean; force?: boolean }) => {
+  .action(async (opts: { multiplexer?: string; all?: boolean; force?: boolean }) => {
     const { handleMux } = await import("./mux.ts")
-    await handleMux(multiplexer, opts)
+    await handleMux(opts.multiplexer, opts)
   })
 
 // ── probe here ──

packages/terminfo.dev/src/index.tsx

@@ -221,13 +221,14 @@ program.addHelpSection("Examples:", [
 // ── test ──
 
 program
-  .command("test [daemon]")
+  .command("test")
+  .argument("[daemon]", "Daemon name to test")
   .description("Test this terminal's feature support")
   .option("--json", "Output results as JSON")
   .option("--serve", "Start daemon for remote testing")
   .option("-p, --port <port>", "Port for --serve", uint)
   .option("--all", "Test all running daemons")
-  .action(async (daemon: string | undefined, opts) => {
+  .action(async (opts: { daemon?: string; json?: boolean; serve?: boolean; port?: number; all?: boolean }) => {
     // --serve: start daemon mode
     if (opts.serve) {
       const { startDaemon } = await import("./serve.ts")
@@ -236,25 +237,25 @@ program
     }
 
     // --all or specific daemon: test remote daemons
-    if (opts.all || daemon) {
+    if (opts.all || opts.daemon) {
       const { listDaemons } = await import("./serve.ts")
       const daemons = listDaemons()
 
       let targets = daemons
-      if (daemon) {
+      if (opts.daemon) {
         targets = daemons.filter(
           (d) =>
-            d.terminal.toLowerCase() === daemon.toLowerCase() ||
-            d.terminal.toLowerCase().includes(daemon.toLowerCase()),
+            d.terminal.toLowerCase() === opts.daemon!.toLowerCase() ||
+            d.terminal.toLowerCase().includes(opts.daemon!.toLowerCase()),
         )
         if (targets.length === 0) {
-          console.error(`No daemon found matching "${daemon}".`)
+          console.error(`No daemon found matching "${opts.daemon}".`)
           if (daemons.length > 0) {
             console.error(`Running: ${daemons.map((d) => d.terminal).join(", ")}`)
           } else {
             console.error(`No daemons running. Start one: terminfo test --serve`)
           }
-          throw new Error(`No daemon found matching "${daemon}"`)
+          throw new Error(`No daemon found matching "${opts.daemon}"`)
         }
       }