commandmate 0.17.0 → 0.19.0

package/dist/cli/commands/auto-yes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"auto-yes.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/auto-yes.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAQpC,wBAAgB,oBAAoB,IAAI,OAAO,CA6F9C"}
+{"version":3,"file":"auto-yes.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/auto-yes.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AASpC,wBAAgB,oBAAoB,IAAI,OAAO,CAoG9C"}

package/dist/cli/commands/auto-yes.js

@@ -11,6 +11,7 @@ const api_client_1 = require("../utils/api-client");
 const command_helpers_1 = require("../utils/command-helpers");
 const duration_constants_1 = require("../config/duration-constants");
 const cli_tool_ids_1 = require("../config/cli-tool-ids");
+const instances_1 = require("./instances");
 function createAutoYesCommand() {
     const cmd = new commander_1.Command('auto-yes');
     cmd
@@ -71,8 +72,14 @@ function createAutoYesCommand() {
                     body.stopPattern = options.stopPattern;
                 }
             }
-            if (options.agent) {
-                body.cliToolId = options.agent;
+            // Issue #1629: the auto-yes poller keys on (worktree, cliTool, instance)
+            // and watches the session cliTool names, so the instance must be paired
+            // with the CLI tool its roster entry declares.
+            const agent = options.instance
+                ? await (0, instances_1.resolveInstanceCliTool)(client, worktreeId, options.instance, options.agent)
+                : options.agent;
+            if (agent) {
+                body.cliToolId = agent;
             }
             // Issue #896: per-instance auto-yes
             if (options.instance) {

package/dist/cli/commands/capture.d.ts

@@ -1,6 +1,7 @@
 /**
  * capture Command - Get current terminal output
  * Issue #518: [DR1-08] Factory pattern
+ * Issue #1623: `--pane` reads the raw tmux pane as a transcript (案B)
  */
 import { Command } from 'commander';
 export declare function createCaptureCommand(): Command;

package/dist/cli/commands/capture.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"capture.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/capture.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAgBpC,wBAAgB,oBAAoB,IAAI,OAAO,CAsD9C"}
+{"version":3,"file":"capture.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/capture.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA6HpC,wBAAgB,oBAAoB,IAAI,OAAO,CA+E9C"}

package/dist/cli/commands/capture.js

@@ -2,6 +2,7 @@
 /**
  * capture Command - Get current terminal output
  * Issue #518: [DR1-08] Factory pattern
+ * Issue #1623: `--pane` reads the raw tmux pane as a transcript (案B)
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createCaptureCommand = createCaptureCommand;
@@ -10,6 +11,19 @@ const types_1 = require("../types");
 const api_client_1 = require("../utils/api-client");
 const command_helpers_1 = require("../utils/command-helpers");
 const cli_tool_ids_1 = require("../config/cli-tool-ids");
+const instances_1 = require("./instances");
+const pager_1 = require("../utils/pager");
+const transcript_squeeze_1 = require("../../lib/tmux/transcript-squeeze");
+/**
+ * Rows of pane history the `--pane` viewer asks for.
+ *
+ * Deliberately NOT configurable and deliberately equal to the capture route's
+ * own default: Issue #1623 has to leave the detection pipeline's view of the
+ * pane untouched, and the cheapest way to guarantee that is for the viewer to
+ * make exactly the request everything else already makes. `--tail` narrows the
+ * RESULT instead (see {@link CaptureOptions.tail}).
+ */
+const PANE_CAPTURE_LINES = 1000;
 /**
  * Format capture output as JSON (excluding fullOutput for size).
  */
@@ -17,12 +31,91 @@ function formatJson(data) {
     const { fullOutput: _fullOutput, ...rest } = data;
     return JSON.stringify(rest, null, 2);
 }
+/**
+ * Parse and validate `--tail N`.
+ *
+ * @param raw - Raw option value, if the user passed one
+ * @returns The parsed count, or undefined when the option was absent
+ */
+function parseTail(raw) {
+    if (raw === undefined)
+        return undefined;
+    const value = Number(raw);
+    if (!Number.isInteger(value) || value < 1) {
+        console.error('Error: --tail must be a positive integer.');
+        process.exit(types_1.ExitCode.CONFIG_ERROR);
+    }
+    return value;
+}
+/**
+ * Resolve which CLI tool's session the pane should be read from.
+ *
+ * POST /capture demands an explicit `cliToolId`, while GET /current-output falls
+ * back to the worktree's own default server-side. Mirroring that fallback here
+ * (`worktree.cliToolId || 'claude'`, matching the route) keeps bare
+ * `capture <id> --pane` pointed at the same session the rest of the CLI uses.
+ *
+ * @param client - API client
+ * @param worktreeId - Worktree ID
+ * @param agent - Value of `--agent`, already validated
+ * @param instance - Value of `--instance`, already validated
+ * @returns CLI tool ID to capture
+ */
+async function resolvePaneCliTool(client, worktreeId, agent, instance) {
+    if (instance) {
+        const resolved = await (0, instances_1.resolveInstanceCliTool)(client, worktreeId, instance, agent);
+        if (resolved)
+            return resolved;
+    }
+    if (agent)
+        return agent;
+    const worktree = await client.get(`/api/worktrees/${worktreeId}`);
+    return worktree.cliToolId || 'claude';
+}
+/**
+ * `--pane` path: read the raw tmux pane and render it as a readable transcript.
+ *
+ * The squeeze happens CLIENT-side on purpose. The server keeps returning the
+ * unmodified frame, so nothing that consumes `/capture` — detection, Auto-Yes,
+ * the response saver — sees a different payload because this command exists.
+ */
+async function capturePane(worktreeId, options) {
+    const client = new api_client_1.ApiClient({ token: options.token });
+    const tail = parseTail(options.tail);
+    const cliToolId = await resolvePaneCliTool(client, worktreeId, options.agent, options.instance);
+    const data = await client.post(`/api/worktrees/${worktreeId}/capture`, {
+        cliToolId,
+        lines: PANE_CAPTURE_LINES,
+        ...(options.instance ? { instanceId: options.instance } : {}),
+    });
+    const raw = data.output ?? '';
+    const squeezed = (0, transcript_squeeze_1.squeezeTranscript)(raw, { tail });
+    const text = options.raw ? raw : squeezed.text;
+    if (options.json) {
+        console.log(JSON.stringify({
+            cliToolId,
+            instanceId: options.instance ?? null,
+            output: text,
+            lines: options.raw ? squeezed.rawLines : squeezed.lines,
+            rawLines: squeezed.rawLines,
+            squeezed: !options.raw,
+            tailed: !options.raw && squeezed.tailed,
+        }, null, 2));
+        return;
+    }
+    (0, pager_1.printMaybePaged)(text);
+}
 function createCaptureCommand() {
     const cmd = new commander_1.Command('capture');
     cmd
         .description('Capture current terminal output from a worktree')
         .argument('<worktree-id>', 'Worktree ID')
         .option('--json', 'JSON output (excludes fullOutput)')
+        // Issue #1623: reading mode. Works without attaching and without tmux 3.2+,
+        // which is why it is the fallback the popup binding degrades to.
+        .option('--pane', 'Read the raw tmux pane as a transcript, with blank layout rows squeezed')
+        .option('--tail <n>', 'With --pane: keep only the last N lines OF THE SQUEEZED transcript')
+        .option('--raw', 'With --pane: skip the squeeze and print the pane verbatim')
         .option('--agent <agent>', 'CLI tool agent (claude, codex, gemini, vibe-local, opencode, copilot, antigravity)')
         .option('--instance <id>', 'Agent instance ID: <agent> or <agent>-<n> (e.g. claude-2). Defaults to the agent\'s primary instance.')
         .option('--token <token>', command_helpers_1.TOKEN_WARNING)
@@ -42,11 +135,28 @@ function createCaptureCommand() {
                 console.error('Error: Invalid --instance. Must be an alphanumeric/underscore/hyphen identifier (max 64 chars).');
                 process.exit(types_1.ExitCode.CONFIG_ERROR);
             }
+            // Issue #1623: --tail / --raw only mean anything for the pane viewer.
+            // Silently ignoring them would look like the flag had no effect.
+            if (!options.pane && (options.tail !== undefined || options.raw)) {
+                console.error('Error: --tail and --raw require --pane.');
+                process.exit(types_1.ExitCode.CONFIG_ERROR);
+            }
+            if (options.pane) {
+                await capturePane(worktreeId, options);
+                return;
+            }
             const client = new api_client_1.ApiClient({ token: options.token });
+            // Issue #1629: /current-output takes the CLI tool at face value and
+            // otherwise falls back to the worktree default, so `--instance codex`
+            // alone captured the wrong (claude-named) session. Resolve the tool the
+            // instance is registered under before asking.
+            const agent = options.instance
+                ? await (0, instances_1.resolveInstanceCliTool)(client, worktreeId, options.instance, options.agent)
+                : options.agent;
             // Build path with optional cliTool/instance query parameters
             const query = new URLSearchParams();
-            if (options.agent) {
-                query.set('cliTool', options.agent);
+            if (agent) {
+                query.set('cliTool', agent);
             }
             if (options.instance) {
                 query.set('instance', options.instance);

package/dist/cli/commands/instances.d.ts

@@ -9,5 +9,29 @@
  *   commandmate instances <worktree-id> kill <instance-id>
  */
 import { Command } from 'commander';
+import { ApiClient } from '../utils/api-client';
+/**
+ * Resolve which CLI tool backs `instanceId`, from the worktree's roster
+ * (Issue #1629). Shared by every command that targets an instance: `send`,
+ * `respond`, `capture` and `auto-yes`.
+ *
+ * `--instance <id>` on its own carries no CLI tool, so the server used to fall
+ * back to the worktree default. Since the tool id is part of the tmux session
+ * name, `send --instance codex` against a claude-default worktree started
+ * Claude in `mcbd-claude-<worktree>-codex`. Resolving here means the resolved
+ * tool is sent explicitly to every endpoint the command touches — including
+ * `/tasks`, `/current-output` and `/prompt-response`, which have no other way
+ * to learn it.
+ *
+ * The roster wins over `--agent`: it is the user-maintained declaration of what
+ * a named instance is, so a contradiction is an error rather than a session
+ * whose name disagrees with the agent inside it. An instance the roster does
+ * not know (the ad-hoc `send --instance <new-id>` flow) keeps `--agent` as
+ * given, and an unreadable roster degrades to a warning so an older daemon
+ * behaves as it did before.
+ *
+ * @returns the CLI tool to send, or undefined to let the server decide
+ */
+export declare function resolveInstanceCliTool(client: ApiClient, worktreeId: string, instanceId: string, requestedAgent: string | undefined): Promise<string | undefined>;
 export declare function createInstancesCommand(): Command;
 //# sourceMappingURL=instances.d.ts.map

package/dist/cli/commands/instances.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"instances.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/instances.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA6NpC,wBAAgB,sBAAsB,IAAI,OAAO,CAqEhD"}
+{"version":3,"file":"instances.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/instances.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAIpC,OAAO,EAAE,SAAS,EAAwC,MAAM,qBAAqB,CAAC;AAmDtF;;;;;;;;;;;;;;;;;;;;;GAqBG;AACH,wBAAsB,sBAAsB,CAC1C,MAAM,EAAE,SAAS,EACjB,UAAU,EAAE,MAAM,EAClB,UAAU,EAAE,MAAM,EAClB,cAAc,EAAE,MAAM,GAAG,SAAS,GACjC,OAAO,CAAC,MAAM,GAAG,SAAS,CAAC,CA0B7B;AAwKD,wBAAgB,sBAAsB,IAAI,OAAO,CAqEhD"}

package/dist/cli/commands/instances.js

@@ -10,6 +10,7 @@
  *   commandmate instances <worktree-id> kill <instance-id>
  */
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.resolveInstanceCliTool = resolveInstanceCliTool;
 exports.createInstancesCommand = createInstancesCommand;
 const commander_1 = require("commander");
 const types_1 = require("../types");
@@ -38,6 +39,49 @@ function formatInstancesTable(rows) {
     const dataLines = dataRows.map(r => r.map((cell, i) => cell.padEnd(colWidths[i])).join('  '));
     return [headerLine, separator, ...dataLines].join('\n');
 }
+/**
+ * Resolve which CLI tool backs `instanceId`, from the worktree's roster
+ * (Issue #1629). Shared by every command that targets an instance: `send`,
+ * `respond`, `capture` and `auto-yes`.
+ *
+ * `--instance <id>` on its own carries no CLI tool, so the server used to fall
+ * back to the worktree default. Since the tool id is part of the tmux session
+ * name, `send --instance codex` against a claude-default worktree started
+ * Claude in `mcbd-claude-<worktree>-codex`. Resolving here means the resolved
+ * tool is sent explicitly to every endpoint the command touches — including
+ * `/tasks`, `/current-output` and `/prompt-response`, which have no other way
+ * to learn it.
+ *
+ * The roster wins over `--agent`: it is the user-maintained declaration of what
+ * a named instance is, so a contradiction is an error rather than a session
+ * whose name disagrees with the agent inside it. An instance the roster does
+ * not know (the ad-hoc `send --instance <new-id>` flow) keeps `--agent` as
+ * given, and an unreadable roster degrades to a warning so an older daemon
+ * behaves as it did before.
+ *
+ * @returns the CLI tool to send, or undefined to let the server decide
+ */
+async function resolveInstanceCliTool(client, worktreeId, instanceId, requestedAgent) {
+    let registered;
+    try {
+        const instances = await (0, agent_instances_1.fetchAgentInstances)(client, worktreeId);
+        registered = instances.find((inst) => inst.id === instanceId);
+    }
+    catch (error) {
+        console.error(`Warning: could not read the agent-instance roster (${(0, types_1.getErrorMessage)(error)}); using --agent as given.`);
+        return requestedAgent;
+    }
+    if (!registered) {
+        return requestedAgent;
+    }
+    if (requestedAgent && requestedAgent !== registered.cliTool) {
+        console.error(`Error: instance '${instanceId}' is registered as ${registered.cliTool}, but --agent ${requestedAgent} was given. `
+            + `Drop --agent, pass --agent ${registered.cliTool}, or re-register the instance.`);
+        process.exit(types_1.ExitCode.CONFIG_ERROR);
+        return requestedAgent;
+    }
+    return registered.cliTool;
+}
 /**
  * List action: roster + live running/auto-yes status per instance.
  * Probes GET .../current-output?cliTool=&instance= per instance (same

package/dist/cli/commands/respond.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"respond.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/respond.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAQpC,wBAAgB,oBAAoB,IAAI,OAAO,CAiE9C"}
+{"version":3,"file":"respond.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/respond.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AASpC,wBAAgB,oBAAoB,IAAI,OAAO,CAyE9C"}

package/dist/cli/commands/respond.js

@@ -12,6 +12,7 @@ const types_1 = require("../types");
 const api_client_1 = require("../utils/api-client");
 const command_helpers_1 = require("../utils/command-helpers");
 const cli_tool_ids_1 = require("../config/cli-tool-ids");
+const instances_1 = require("./instances");
 function createRespondCommand() {
     const cmd = new commander_1.Command('respond');
     cmd
@@ -44,10 +45,17 @@ function createRespondCommand() {
                 process.exit(types_1.ExitCode.CONFIG_ERROR);
             }
             const client = new api_client_1.ApiClient({ token: options.token });
+            // Issue #1629: /prompt-response derives the session name from cliTool
+            // and falls back to the worktree default, so `--instance codex` alone
+            // answered into a session that was never started. Resolve the tool the
+            // instance is registered under first.
+            const agent = options.instance
+                ? await (0, instances_1.resolveInstanceCliTool)(client, worktreeId, options.instance, options.agent)
+                : options.agent;
             // [DR2-06] Use prompt-response API with cliTool (not cliToolId)
             const body = { answer };
-            if (options.agent) {
-                body.cliTool = options.agent;
+            if (agent) {
+                body.cliTool = agent;
             }
             // Issue #868: target a specific agent instance
             if (options.instance) {

package/dist/cli/commands/send.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"send.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/send.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AA8IpC,wBAAgB,iBAAiB,IAAI,OAAO,CAmJ3C"}
+{"version":3,"file":"send.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/send.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAgKpC,wBAAgB,iBAAiB,IAAI,OAAO,CAoK3C"}

package/dist/cli/commands/send.js

@@ -13,28 +13,41 @@ const duration_constants_1 = require("../config/duration-constants");
 const cli_tool_ids_1 = require("../config/cli-tool-ids");
 const model_validation_1 = require("../config/model-validation");
 const agent_instances_1 = require("../utils/agent-instances");
+const instances_1 = require("./instances");
+/** Auto-yes duration used when --duration is omitted. */
+const DEFAULT_AUTO_YES_DURATION = '1h';
 /**
- * Build auto-yes request body and send it to the API (DRY extraction).
- * Validates duration and exits on invalid input.
+ * Resolve --duration to milliseconds, exiting on an invalid value.
  *
- * @param client - API client instance
- * @param worktreeId - Target worktree ID
- * @param options - Send command options containing auto-yes settings
+ * Issue #1608: this is called from the option-validation block at the top of
+ * the action, not from enableAutoYes(). enableAutoYes() runs after --contract
+ * has already created the task row, so validating there left a `pending` task
+ * behind for a message that was never sent.
  */
-async function enableAutoYes(client, worktreeId, options) {
-    const durationMs = options.duration
-        ? (0, duration_constants_1.parseDurationToMs)(options.duration)
-        : (0, duration_constants_1.parseDurationToMs)('1h'); // default 1h
+function resolveAutoYesDurationMs(duration) {
+    const durationMs = (0, duration_constants_1.parseDurationToMs)(duration ?? DEFAULT_AUTO_YES_DURATION);
     if (durationMs === null) {
         console.error(`Error: Invalid duration. Must be one of: ${duration_constants_1.ALLOWED_DURATIONS.join(', ')}`);
         process.exit(types_1.ExitCode.CONFIG_ERROR);
     }
+    return durationMs;
+}
+/**
+ * Build auto-yes request body and send it to the API (DRY extraction).
+ *
+ * @param client - API client instance
+ * @param worktreeId - Target worktree ID
+ * @param options - Send command options containing auto-yes settings
+ * @param durationMs - Duration already validated by resolveAutoYesDurationMs()
+ * @param agent - CLI tool resolved for --instance (Issue #1629), else --agent
+ */
+async function enableAutoYes(client, worktreeId, options, durationMs, agent) {
     const autoYesBody = {
         enabled: true,
         duration: durationMs,
     };
-    if (options.agent) {
-        autoYesBody.cliToolId = options.agent;
+    if (agent) {
+        autoYesBody.cliToolId = agent;
     }
     // Issue #896: per-instance auto-yes. When --instance is given, the poller keys
     // on worktreeId:cliToolId:instanceId so the targeted instance is auto-answered
@@ -74,10 +87,10 @@ async function registerInstance(client, worktreeId, instanceId, cliTool) {
  * from that worktree's verify.yaml. Every violation the server found is printed
  * before exiting, so a broken contract is fixed in one pass (Issue #1545).
  */
-async function createContractTask(client, worktreeId, options) {
+async function createContractTask(client, worktreeId, options, agent) {
     const body = { contractPath: options.contract };
-    if (options.agent) {
-        body.cliToolId = options.agent;
+    if (agent) {
+        body.cliToolId = agent;
     }
     if (options.instance) {
         body.instanceId = options.instance;
@@ -187,14 +200,29 @@ function createSendCommand() {
                     process.exit(types_1.ExitCode.CONFIG_ERROR);
                 }
             }
+            // Issue #1608: every option that can be judged from its own value is
+            // judged here, before the first side effect. --duration used to be the
+            // exception: enableAutoYes() validated it, and that runs after
+            // --contract has already created the task row, so `--duration 2h` left
+            // a `pending` task for a message that was never sent. Validated
+            // unconditionally, like --stop-pattern and --model above: a value the
+            // CLI cannot honour is an error whether or not --auto-yes accompanies it.
+            const autoYesDurationMs = resolveAutoYesDurationMs(options.duration);
             const client = new api_client_1.ApiClient({ token: options.token });
+            // Issue #1629: --instance names an agent instance, not a CLI tool, and
+            // the roster is the only place that pairs the two. Resolve it once, up
+            // front, so the task row, the send and auto-yes all name the same tool
+            // as the session that actually starts.
+            const agent = options.instance
+                ? await (0, instances_1.resolveInstanceCliTool)(client, worktreeId, options.instance, options.agent)
+                : options.agent;
             // Issue #1545: resolve the contract before anything with a side effect,
             // so an invalid contract cannot leave auto-yes enabled for a message
             // that was never sent.
             let taskId;
             let content = message;
             if (options.contract) {
-                const task = await createContractTask(client, worktreeId, options);
+                const task = await createContractTask(client, worktreeId, options, agent);
                 taskId = task.taskId;
                 content = task.message;
                 console.error(`Task created: ${taskId}`);
@@ -202,12 +230,12 @@ function createSendCommand() {
             }
             // --auto-yes: enable auto-yes first (unless --model is specified, then after send) [DR2-02]
             if (options.autoYes && !options.model) {
-                await enableAutoYes(client, worktreeId, options);
+                await enableAutoYes(client, worktreeId, options, autoYesDurationMs, agent);
             }
             // [DR2-05] Send API uses "content" not "message"
             const sendBody = { content };
-            if (options.agent) {
-                sendBody.cliToolId = options.agent;
+            if (agent) {
+                sendBody.cliToolId = agent;
             }
             // Issue #868: Include instance ID in send body
             if (options.instance) {
@@ -240,7 +268,7 @@ function createSendCommand() {
             // Issue #576: Enable auto-yes AFTER send when --model is specified
             // This avoids auto-yes interfering with the /model command interaction
             if (options.autoYes && options.model) {
-                await enableAutoYes(client, worktreeId, options);
+                await enableAutoYes(client, worktreeId, options, autoYesDurationMs, agent);
             }
         }
         catch (error) {

package/dist/cli/commands/wait.d.ts

@@ -4,10 +4,13 @@
  *
  * Exit codes [DR1-03]:
  * - 0: SUCCESS (agent completed)
- * - 10: PROMPT_DETECTED (agent waiting for user input)
+ * - 10: PROMPT_DETECTED (agent waiting for user input, including arrow-key
+ *       selection lists — Issue #1628)
  * - 124: TIMEOUT (--timeout exceeded)
  * Issue #1544 adds --verify / --require-work, which can turn a detected
  * completion into 20 (VERIFY_FAILED) or 21 (NOT_STARTED).
+ * Issue #1628 also returns 21 without --verify when the session was never
+ * running: a wait with nothing to wait for must not report success.
  * Infrastructure errors use ExitCode (1, 2, 99)
  */
 import { Command } from 'commander';

package/dist/cli/commands/wait.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"wait.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/wait.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAkKpC,wBAAgB,iBAAiB,IAAI,OAAO,CA6F3C"}
+{"version":3,"file":"wait.d.ts","sourceRoot":"","sources":["../../../src/cli/commands/wait.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAwSpC,wBAAgB,iBAAiB,IAAI,OAAO,CA6G3C"}