commandmate 0.30.2 → 0.31.0

package/dist/server/src/lib/cli-tools/start-availability.js

@@ -64,7 +64,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.reportSessionStartFailure = reportSessionStartFailure;
 exports.assertToolStartable = assertToolStartable;
 exports.reportToolUnavailable = reportToolUnavailable;
-const session_start_error_1 = require("../session/session-start-error");
+const install_hints_1 = require("./install-hints");
 /**
  * Report a session-start failure. Fire-and-forget; never throws.
  *
@@ -99,11 +99,16 @@ function reportSessionStartFailure(report, error) {
  * the route reports `status: 'ready'`, and the honest place for that answer is
  * the tool object rather than a hand-rolled check at the HTTP layer.
  *
- * The wording is {@link SessionStartUnavailableError}'s class default — the
- * sentence that error was built to replace the HTTP layer's copy with. A tool
- * whose launch path has something more useful to say (copilot ships an install
- * hint) still says it there; this function is reached only by callers that never
- * get that far.
+ * The wording is {@link missingToolError}'s — the same sentence the tool's own
+ * launch path throws. It used to be `SessionStartUnavailableError`'s class
+ * default instead, on the reasoning that "a tool whose launch path has
+ * something more useful to say (copilot ships an install hint) still says it
+ * there". That reasoning held only while copilot was the sole tool with a hint:
+ * for the other seven the caller that never reaches `launchSession` got a
+ * strictly worse sentence than the caller that does, for no reason a reader of
+ * the message could see. Issue #2301 gave all eight a hint and pointed both
+ * paths at the one builder, so which path found the missing binary no longer
+ * changes what the operator is told.
  *
  * @param tool - The tool being asked to start
  * @param target - Who the refusal belongs to, and how to address it
@@ -112,7 +117,7 @@ function reportSessionStartFailure(report, error) {
 async function assertToolStartable(tool, target) {
     if (await tool.isInstalled())
         return;
-    const error = new session_start_error_1.SessionStartUnavailableError(tool.name);
+    const error = (0, install_hints_1.missingToolError)(tool);
     reportSessionStartFailure({
         worktreeId: target.worktreeId,
         cliToolId: tool.id,
@@ -130,7 +135,9 @@ async function assertToolStartable(tool, target) {
  * label `CLI_TOOL_DISPLAY_NAMES` carries ("Claude"). Both are legitimate
  * spellings and the repository uses each in its own place, but ONE missing
  * binary must not read differently depending on which of the two paths found
- * it, and the start gate above reads the tool object. So this one does too.
+ * it, and the start gate above reads the tool object. So this one does too —
+ * and since Issue #2301 the whole sentence, install hint included, comes from
+ * {@link missingToolError} for the same reason.
  *
  * `await import('./manager')` because a static import would be a module-scope
  * cycle: the manager constructs all seven tools, every one of them extends
@@ -147,14 +154,14 @@ async function assertToolStartable(tool, target) {
  */
 function reportToolUnavailable(cliToolId, target) {
     void Promise.resolve().then(() => __importStar(require('./manager'))).then(({ CLIToolManager }) => {
-        const toolName = CLIToolManager.getInstance().getTool(cliToolId).name;
+        const tool = CLIToolManager.getInstance().getTool(cliToolId);
         reportSessionStartFailure({
             worktreeId: target.worktreeId,
             cliToolId,
             instanceId: target.instanceId,
-            toolName,
+            toolName: tool.name,
             subject: target.subject,
-        }, new session_start_error_1.SessionStartUnavailableError(toolName));
+        }, (0, install_hints_1.missingToolError)(tool));
     })
         .catch(() => { });
 }

package/dist/server/src/lib/cli-tools/types.js

@@ -24,7 +24,7 @@ exports.isImageCapableCLITool = isImageCapableCLITool;
  * T2.1: Single source of truth for CLI tool IDs
  * CLIToolType is derived from this constant (DRY principle)
  */
-exports.CLI_TOOL_IDS = ['claude', 'codex', 'gemini', 'vibe-local', 'opencode', 'copilot', 'antigravity'];
+exports.CLI_TOOL_IDS = ['claude', 'codex', 'gemini', 'vibe-local', 'opencode', 'copilot', 'antigravity', 'command-code'];
 /**
  * CLI tools that render inside the terminal's alternate screen (Issue #1268).
  *
@@ -44,6 +44,10 @@ const ALTERNATE_SCREEN_CLI_TOOLS = new Set([
     'claude',
     'opencode',
     'copilot',
+    // Issue #2250: `command-code` is deliberately absent. Command Code renders
+    // INLINE -- measured on 1.40.1, `#{alternate_on}` is 0 and the pane keeps its
+    // scrollback -- so its captured line count really is a monotonic cursor, the
+    // way codex's and agy's are.
 ]);
 /**
  * Whether a CLI tool renders in the terminal's alternate screen, making the
@@ -190,6 +194,7 @@ exports.CLI_TOOL_DISPLAY_NAMES = {
     opencode: 'OpenCode',
     copilot: 'Copilot',
     antigravity: 'Antigravity',
+    'command-code': 'Command Code',
 };
 /**
  * Check if a string is a valid CLIToolType

package/dist/server/src/lib/cli-tools/vibe-local.js

@@ -19,7 +19,7 @@ const db_instance_1 = require("../db/db-instance");
 const db_1 = require("../db");
 const logger_1 = require("../../lib/logger");
 const cli_tool_timing_config_1 = require("../../config/cli-tool-timing-config");
-const session_start_error_1 = require("../session/session-start-error");
+const install_hints_1 = require("./install-hints");
 const logger = (0, logger_1.createLogger)('cli-tools/vibe-local');
 /**
  * Extract error message from unknown error type (DRY)
@@ -57,7 +57,7 @@ class VibeLocalTool extends base_1.BaseCLITool {
     async launchSession(worktreeId, worktreePath, instanceId) {
         const vibeLocalAvailable = await this.isInstalled();
         if (!vibeLocalAvailable) {
-            throw new session_start_error_1.SessionStartUnavailableError(this.name, 'vibe-local is not installed or not in PATH');
+            throw (0, install_hints_1.missingToolError)(this);
         }
         const sessionName = this.getSessionName(worktreeId, instanceId);
         const exists = await (0, tmux_1.hasSession)(sessionName);

package/dist/server/src/lib/cmate-parser.js

@@ -224,6 +224,12 @@ function parseSchedulesSection(rows) {
             case 'antigravity':
                 allowedValues = schedule_config_1.ANTIGRAVITY_PERMISSIONS;
                 break;
+            // Issue #2250: its own case, not a fall-through. #1914's rule is that a
+            // tool with a permission vocabulary names itself here; inheriting the
+            // no-flag branch would silently blank a valid `plan` / `auto-accept` cell.
+            case 'command-code':
+                allowedValues = schedule_config_1.COMMAND_CODE_PERMISSIONS;
+                break;
             case 'gemini':
             case 'vibe-local':
             case 'opencode':