syntaur 0.16.0 → 0.16.2

package/dist/launch/index.d.ts

@@ -1,6 +1,8 @@
 import { SpawnOptions, ChildProcess } from 'node:child_process';
 
-type OpenUrlErrorCode = 'bad-scheme' | 'bad-host' | 'missing-id' | 'both-ids' | 'malformed' | 'duplicate-param' | 'bad-mode';
+type TerminalChoice = 'terminal-app' | 'iterm' | 'ghostty' | 'alacritty' | 'warp' | 'kitty';
+
+type OpenUrlErrorCode = 'bad-scheme' | 'bad-host' | 'missing-id' | 'both-ids' | 'malformed' | 'duplicate-param' | 'bad-terminal' | 'bad-mode';
 declare class OpenUrlError extends Error {
     readonly code: OpenUrlErrorCode;
     constructor(code: OpenUrlErrorCode, message: string);
@@ -9,6 +11,13 @@ type SessionMode = 'resume' | 'fork';
 interface ParsedOpenUrl {
     kind: 'assignment' | 'session';
     id: string;
+    /**
+     * Optional one-shot terminal override. When present, the launch plan uses
+     * this instead of the configured `terminal:`. The dashboard's
+     * missing-terminal fallback dialog sets this so a confirm-to-fallback flow
+     * doesn't require mutating user config.
+     */
+    terminal?: TerminalChoice;
     /**
      * Only set when `kind === 'session'`. Defaults to `'resume'` when the URL
      * has no `mode` query param. Distinguishes "continue this session under the
@@ -27,6 +36,8 @@ interface ParsedOpenUrl {
  * - exactly one of `assignment` or `session` query params must be present
  * - neither param may be duplicated
  * - when `session` is present, optional `mode=resume|fork` (default `resume`)
+ * - optional `terminal=<choice>` one-shot override (validated against
+ *   `TERMINAL_CHOICES`)
  *
  * Throws OpenUrlError with a structured code on any failure.
  */
@@ -116,7 +127,7 @@ interface ThemeConfig {
 interface HotkeyBindingsConfig {
     bindings: Partial<Record<BindableActionKind, string>>;
 }
-type TerminalChoice = 'terminal-app' | 'iterm' | 'ghostty' | 'alacritty' | 'warp' | 'kitty';
+
 interface SyntaurConfig {
     version: string;
     defaultProjectDir: string;
@@ -181,6 +192,13 @@ interface ResolveLaunchPlanInput {
     config: SyntaurConfig;
     projectsDir: string;
     assignmentsDir: string;
+    /**
+     * One-shot terminal override. When set, used in place of
+     * `getTerminal(config)`. Wired through from `?terminal=<choice>` on the
+     * incoming `syntaur://` URL so the dashboard's missing-terminal fallback
+     * dialog can confirm a different terminal without mutating user config.
+     */
+    terminalOverride?: TerminalChoice;
 }
 /**
  * Pick the agent the "Open in agent" flow should use. Order of preference:

package/dist/launch/index.js

@@ -3,6 +3,22 @@ var __esm = (fn, res) => function __init() {
   return fn && (res = (0, fn[__getOwnPropNames(fn)[0]])(fn = 0)), res;
 };
 
+// src/utils/terminal-schema.ts
+var TERMINAL_CHOICES;
+var init_terminal_schema = __esm({
+  "src/utils/terminal-schema.ts"() {
+    "use strict";
+    TERMINAL_CHOICES = [
+      "terminal-app",
+      "iterm",
+      "ghostty",
+      "alacritty",
+      "warp",
+      "kitty"
+    ];
+  }
+});
+
 // src/utils/paths.ts
 import { homedir } from "os";
 import { resolve } from "path";
@@ -265,6 +281,7 @@ var init_agents_schema = __esm({
 
 // src/utils/config.ts
 import { readFile as readFile2 } from "fs/promises";
+import { spawnSync } from "child_process";
 import { resolve as resolve3, isAbsolute } from "path";
 function parseAgentCommand(value, agentId) {
   if (typeof value !== "string" || value.trim() === "") {
@@ -889,9 +906,20 @@ function parseTerminalConfig(value) {
   return trimmed;
 }
 function getTerminal(config) {
-  return config.terminal ?? "terminal-app";
+  if (config.terminal) return config.terminal;
+  if (process.platform === "darwin") return "terminal-app";
+  if (process.platform === "linux") {
+    const order = ["kitty", "alacritty", "warp"];
+    for (const candidate of order) {
+      const result = spawnSync("which", [candidate], { encoding: "utf-8" });
+      if (result.status === 0 && result.stdout.trim().length > 0) {
+        return candidate;
+      }
+    }
+  }
+  return "terminal-app";
 }
-var TERMINAL_CHOICES, DEFAULT_CONFIG, AUTO_CREATE_WORKTREE_VALUES, AgentConfigError, KNOWN_AGENT_SCALAR_FIELDS, migratedConfigPaths, TerminalConfigError;
+var DEFAULT_CONFIG, AUTO_CREATE_WORKTREE_VALUES, AgentConfigError, KNOWN_AGENT_SCALAR_FIELDS, migratedConfigPaths, TerminalConfigError;
 var init_config2 = __esm({
   "src/utils/config.ts"() {
     "use strict";
@@ -901,14 +929,7 @@ var init_config2 = __esm({
     init_fs_migration();
     init_hotkeysCatalog();
     init_agents_schema();
-    TERMINAL_CHOICES = [
-      "terminal-app",
-      "iterm",
-      "ghostty",
-      "alacritty",
-      "warp",
-      "kitty"
-    ];
+    init_terminal_schema();
     DEFAULT_CONFIG = {
       version: "2.0",
       defaultProjectDir: defaultProjectDir(),
@@ -1001,6 +1022,12 @@ function parseListField(frontmatter, fieldName) {
   }
   return results;
 }
+function unquoteYamlString(value) {
+  if (value.startsWith('"') && value.endsWith('"') || value.startsWith("'") && value.endsWith("'")) {
+    return value.slice(1, -1);
+  }
+  return value;
+}
 function parseProject(fileContent) {
   const [fm, body] = extractFrontmatter(fileContent);
   const slug = getField(fm, "slug") ?? getField(fm, "mission") ?? "";
@@ -1016,6 +1043,7 @@ function parseProject(fileContent) {
     updated: getField(fm, "updated") ?? "",
     tags: parseListField(fm, "tags"),
     workspace: getField(fm, "workspace"),
+    repositories: parseListField(fm, "repositories").map(unquoteYamlString),
     externalIds: parseExternalIds(fm),
     body
   };
@@ -2464,6 +2492,7 @@ var init_api = __esm({
 });
 
 // src/launch/url.ts
+init_terminal_schema();
 var OpenUrlError = class extends Error {
   code;
   constructor(code, message) {
@@ -2515,6 +2544,24 @@ function parseOpenUrl(input) {
       "URL has both `assignment` and `session` query params \u2014 only one is allowed"
     );
   }
+  const terminalVals = url.searchParams.getAll("terminal");
+  if (terminalVals.length > 1) {
+    throw new OpenUrlError(
+      "duplicate-param",
+      "URL has more than one `terminal` query param"
+    );
+  }
+  let terminal;
+  if (terminalVals.length === 1 && terminalVals[0].trim() !== "") {
+    const candidate = terminalVals[0];
+    if (!TERMINAL_CHOICES.includes(candidate)) {
+      throw new OpenUrlError(
+        "bad-terminal",
+        `\`terminal\` query param must be one of: ${TERMINAL_CHOICES.join(", ")}`
+      );
+    }
+    terminal = candidate;
+  }
   if (assignmentVals.length === 1) {
     const id = assignmentVals[0];
     if (id.trim() === "") {
@@ -2523,7 +2570,7 @@ function parseOpenUrl(input) {
         "`assignment` query param is empty"
       );
     }
-    return { kind: "assignment", id };
+    return { kind: "assignment", id, ...terminal ? { terminal } : {} };
   }
   if (sessionVals.length === 1) {
     const id = sessionVals[0];
@@ -2548,7 +2595,7 @@ function parseOpenUrl(input) {
       }
       mode = raw;
     }
-    return { kind: "session", id, mode };
+    return { kind: "session", id, mode, ...terminal ? { terminal } : {} };
   }
   throw new OpenUrlError(
     "missing-id",
@@ -2671,7 +2718,7 @@ function pickAgent(config) {
   return agents.find((a) => a.default) ?? agents[0];
 }
 async function resolveLaunchPlan(input) {
-  const terminal = getTerminal(input.config);
+  const terminal = input.terminalOverride ?? getTerminal(input.config);
   if (input.kind === "assignment") {
     return resolveAssignmentPlan(input, terminal);
   }
@@ -2914,19 +2961,19 @@ function buildTerminalInvocation(plan) {
         command: "osascript",
         args: [
           "-e",
-          'tell application "Ghostty"',
+          'tell application "Ghostty" to activate',
           "-e",
-          "activate",
+          "delay 0.3",
           "-e",
-          "set newWin to (new window)",
+          'tell application "System Events"',
           "-e",
-          "delay 0.2",
+          'keystroke "n" using command down',
           "-e",
-          "set t to terminal 1 of selected tab of newWin",
+          "delay 0.4",
           "-e",
-          `input text ${appleScriptString(cdAndRun)} to t`,
+          `keystroke ${appleScriptString(cdAndRun)}`,
           "-e",
-          'send key "enter" to t',
+          "key code 36",
           "-e",
           "end tell"
         ]