@monster-orange/den 0.4.2 → 0.4.4

package/src/shared/agents/catalog-runtime.ts

@@ -0,0 +1,178 @@
+/**
+ * A runtime from a catalog entry (AG-0.5, catalog.ts): found by AG-0.1's
+ * search with the entry's binary names and install directories, and
+ * launched as its interactive CLI in the agent's directory (the entry's
+ * launch argv, "{bin}" the path found).
+ *
+ * These don't get Den's tools yet: no MCP config, no permission mode
+ * (AG-12 and AG-13 wire them per runtime). Lists tag them "Den tools:
+ * coming" (denTools: false). Claude Code is den-claude, which takes its
+ * search from the catalog and keeps its own launch.
+ *
+ * checkRuntimeBin is what "Set path…" runs: the entry's version command,
+ * and the entry's identity (its output names it, it resolves into the
+ * vendor's npm package, or under the vendor's install directory), so a
+ * path that isn't that runtime is refused.
+ */
+import * as fs from "fs";
+import * as path from "path";
+import { configuredRuntimeBin } from "../den-config.js";
+import { entryDirs, type CatalogEntry } from "./catalog.js";
+import {
+  LaunchError,
+  type DetectOptions,
+  type LaunchOutput,
+  type Runtime,
+  type RuntimeDetect,
+} from "./runtime.js";
+import {
+  locateRuntime,
+  parseVersion,
+  systemBinDirs,
+  versionOutput,
+  type RuntimeFind,
+} from "./runtime-search.js";
+
+/** An entry as AG-0.1's search takes it. */
+export function catalogFind(e: CatalogEntry): RuntimeFind {
+  return {
+    bins: e.bins,
+    dirs: (home, env) => entryDirs(e, home, env, systemBinDirs),
+    versionArgs: e.version.args,
+  };
+}
+
+/** Find an entry's CLI: a setting (workspace or global), else the search. */
+export function detectEntry(
+  e: CatalogEntry,
+  find: RuntimeFind,
+  env: NodeJS.ProcessEnv,
+  opts: DetectOptions,
+): RuntimeDetect {
+  const setting =
+    opts.bin === undefined ? configuredRuntimeBin(e.id) : opts.bin;
+  const found = locateRuntime(e.id, find, env, setting);
+  if (!found.available || !found.path)
+    return {
+      available: false,
+      why: found.why ?? `${e.bins.join(" or ")} not found`,
+      ...(found.source ? { source: found.source } : {}),
+      searched: found.searched,
+    };
+  return {
+    available: true,
+    path: found.path,
+    ...(found.source ? { source: found.source } : {}),
+    searched: found.searched,
+  };
+}
+
+/** The launch argv: the entry's template, "{bin}" the path found. */
+export function launchArgv(e: CatalogEntry, bin: string): string[] {
+  return e.launch.argv.map((a) => a.split("{bin}").join(bin));
+}
+
+/** A runtime for a catalog entry other than Claude Code (den-claude). */
+export function catalogRuntime(e: CatalogEntry): Runtime {
+  const find = catalogFind(e);
+  const detect = (
+    env: NodeJS.ProcessEnv = process.env,
+    opts: DetectOptions = {},
+  ) => detectEntry(e, find, env, opts);
+  return {
+    id: e.id,
+    label: e.name,
+    vendor: e.vendor,
+    catalog: e,
+    find,
+    denTools: false,
+    detect,
+    launch(input): LaunchOutput {
+      let bin = input.bin;
+      if (!bin) {
+        const d = detect(input.env);
+        if (!d.available || !d.path)
+          throw new LaunchError(`${e.id}: ${d.why ?? "not found"}`);
+        bin = d.path;
+      }
+      return {
+        argv: launchArgv(e, bin),
+        env: {},
+        warnings: [
+          `${e.name} runs as its own CLI in ${input.worktree}: Den's tools (MCP) and permission modes are coming for it`,
+        ],
+      };
+    },
+    approvals: "terminal",
+    scrubEnv: [],
+  };
+}
+
+/** The npm package a file resolves into (the nearest package.json's name). */
+function packageOf(real: string): string | undefined {
+  for (let dir = path.dirname(real), i = 0; i < 8; i++) {
+    try {
+      const pkg = JSON.parse(
+        fs.readFileSync(path.join(dir, "package.json"), "utf-8"),
+      ) as { name?: unknown };
+      if (typeof pkg.name === "string") return pkg.name;
+    } catch {
+      /* not here */
+    }
+    const up = path.dirname(dir);
+    if (up === dir) break;
+    dir = up;
+  }
+  return undefined;
+}
+
+export type BinCheck =
+  | { ok: true; path: string; version: string }
+  | { ok: false; why: string };
+
+/**
+ * Whether `bin` is entry `e`'s runtime: an absolute path to an executable
+ * file whose version command exits 0 and prints a version, and whose
+ * output, npm package or real location names it. Runs only that command
+ * (stdin closed, under a timeout). Never throws.
+ */
+export async function checkRuntimeBin(
+  e: CatalogEntry,
+  bin: string,
+  env: NodeJS.ProcessEnv,
+): Promise<BinCheck> {
+  const p = bin.trim();
+  const shown = p.replace(/[^\x20-\x7e]/g, "?").slice(0, 200);
+  if (!path.isAbsolute(p))
+    return { ok: false, why: `"${shown}" is not an absolute path` };
+  let real: string;
+  try {
+    const st = fs.statSync(p);
+    fs.accessSync(p, fs.constants.X_OK);
+    if (!st.isFile()) throw new Error("not a file");
+    real = fs.realpathSync(p);
+  } catch {
+    return { ok: false, why: `${shown} is not an executable file` };
+  }
+  const cmd = `${path.basename(p)} ${e.version.args.join(" ")}`;
+  const r = await versionOutput(p, e.version.args, env);
+  if (!r.ok) return { ok: false, why: `\`${cmd}\` failed: not ${e.name}` };
+  const version = parseVersion(r.out);
+  if (!version)
+    return { ok: false, why: `\`${cmd}\` printed no version: not ${e.name}` };
+  const idn = e.version.identity;
+  const home = env.HOME && path.isAbsolute(env.HOME) ? env.HOME : undefined;
+  const under = (idn.realpath_under ?? []).flatMap((d) =>
+    d.startsWith("~/") ? (home ? [path.join(home, d.slice(2))] : []) : [d],
+  );
+  const named =
+    (idn.output !== undefined && new RegExp(idn.output, "i").test(r.out)) ||
+    (idn.package !== undefined && packageOf(real) === idn.package) ||
+    under.some((d) => real === d || real.startsWith(d + path.sep));
+  if (!named)
+    return {
+      ok: false,
+      why: `${shown} is not ${e.name}: its version output, package and location don't name it`,
+    };
+  return { ok: true, path: p, version };
+}

package/src/shared/agents/catalog.ts

@@ -0,0 +1,343 @@
+/**
+ * The runtime catalog (AG-0.5): every vendor CLI Den knows how to find,
+ * shipped as data with each release. An entry says:
+ *
+ *   id        the runtime's id ("den-claude", "codex")
+ *   vendor    who makes it ("Anthropic")
+ *   name      its name for people ("Claude Code")
+ *   bins      its executable names, in order
+ *   version   args: what prints its version ("--version"); identity: how
+ *             a path is known to be this runtime, not just any program that
+ *             prints a version (its output names it, it lives in the
+ *             vendor's npm package, or under the vendor's install dir).
+ *             A path someone sets is refused unless one of these holds.
+ *   dirs      its well-known install directories: "~/…", an absolute
+ *             path, "$NAME" (an install-dir variable the vendor's installer
+ *             honours; skipped when unset) or "{system}" (Homebrew's,
+ *             /usr/local/bin and npm's global bin)
+ *   launch    argv: its interactive CLI, "{bin}" first (the path found)
+ *
+ * Each built-in entry was checked against the vendor's own docs or
+ * installer (the URLs beside it). A vendor with no official CLI is left
+ * out (docs/distributions.md, Runtimes, says which, and why); nothing is
+ * guessed.
+ *
+ * A distribution (`agents.runtime_catalog` in its config) and a person
+ * (`agents.runtime_catalog` in config/local/user.yml) add entries, checked
+ * by the same validator (validateCatalogEntry). An entry can't replace a
+ * built-in one: a path is set with agents.runtimes.<id>.bin instead.
+ */
+import * as path from "path";
+
+export interface CatalogIdentity {
+  /** A case-insensitive pattern the version command's output matches. */
+  output?: string;
+  /** The npm package the binary resolves into (its package.json name). */
+  package?: string;
+  /** Directories ("~/…" or absolute) the binary resolves under. */
+  realpath_under?: string[];
+}
+
+export interface CatalogEntry {
+  id: string;
+  vendor: string;
+  name: string;
+  bins: string[];
+  version: { args: string[]; identity: CatalogIdentity };
+  dirs: string[];
+  launch: { argv: string[] };
+}
+
+/** The keys an entry takes (the validator refuses any other). */
+const ENTRY_KEYS = [
+  "id",
+  "vendor",
+  "name",
+  "bins",
+  "version",
+  "dirs",
+  "launch",
+] as const;
+
+/** Den's built-in catalog, in the order lists show them. */
+export const BUILTIN_CATALOG: readonly CatalogEntry[] = [
+  // Claude Code. https://code.claude.com/docs/en/setup: the native
+  // installer's launcher is ~/.local/bin/claude; `claude --version` prints
+  // "2.1.211 (Claude Code)"; npm package @anthropic-ai/claude-code;
+  // Homebrew cask claude-code. ~/.claude/local is the older local install.
+  {
+    id: "den-claude",
+    vendor: "Anthropic",
+    name: "Claude Code",
+    bins: ["claude"],
+    version: {
+      args: ["--version"],
+      identity: {
+        output: "\\(Claude Code\\)",
+        package: "@anthropic-ai/claude-code",
+        realpath_under: ["~/.local/share/claude"],
+      },
+    },
+    dirs: ["~/.local/bin", "~/.claude/local", "{system}"],
+    launch: { argv: ["{bin}"] },
+  },
+  // Codex CLI. https://github.com/openai/codex: binary `codex`, npm
+  // @openai/codex, Homebrew cask codex. The standalone installer
+  // (https://chatgpt.com/codex/install.sh → releases.openai.com) installs
+  // to ${CODEX_INSTALL_DIR:-$HOME/.local/bin}/codex and reads the version
+  // from `codex --version`; the CLI declares clap's version flag
+  // (codex-rs/cli/src/main.rs, bin_name "codex").
+  {
+    id: "codex",
+    vendor: "OpenAI",
+    name: "Codex CLI",
+    bins: ["codex"],
+    version: {
+      args: ["--version"],
+      identity: { output: "codex", package: "@openai/codex" },
+    },
+    dirs: ["$CODEX_INSTALL_DIR", "~/.local/bin", "{system}"],
+    launch: { argv: ["{bin}"] },
+  },
+  // Gemini CLI. https://github.com/google-gemini/gemini-cli: binary
+  // `gemini`, npm @google/gemini-cli, Homebrew formula gemini-cli;
+  // docs/cli/cli-reference.md: `--version`/`-v` "Show CLI version number
+  // and exit" (the number only, so the package names it), and no
+  // arguments starts the interactive REPL.
+  {
+    id: "gemini",
+    vendor: "Google",
+    name: "Gemini CLI",
+    bins: ["gemini"],
+    version: {
+      args: ["--version"],
+      identity: { output: "gemini", package: "@google/gemini-cli" },
+    },
+    dirs: ["{system}"],
+    launch: { argv: ["{bin}"] },
+  },
+  // Grok Build. https://docs.x.ai/build/overview: install with
+  // https://x.ai/cli/install.sh, run `grok` in a project directory. The
+  // installer puts the binary in ~/.grok/downloads, links it as
+  // ${GROK_BIN_DIR:-$HOME/.grok/bin}/grok (and into ~/.local/bin or
+  // /usr/local/bin when that dir isn't on PATH), and checks it with
+  // `grok --version`.
+  {
+    id: "grok",
+    vendor: "xAI",
+    name: "Grok Build",
+    bins: ["grok"],
+    version: {
+      args: ["--version"],
+      identity: { output: "grok", realpath_under: ["~/.grok"] },
+    },
+    dirs: ["$GROK_BIN_DIR", "~/.grok/bin", "~/.local/bin", "/usr/local/bin"],
+    launch: { argv: ["{bin}"] },
+  },
+];
+
+/** Den's own runtime ids that a catalog entry can't take. */
+const RESERVED_IDS = ["generic"];
+const ID_RE = /^[a-z][a-z0-9-]{0,31}$/;
+const BIN_RE = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/;
+const ENV_DIR_RE = /^\$[A-Z_][A-Z0-9_]*$/;
+const PKG_RE = /^(@[a-z0-9-~][a-z0-9-._~]*\/)?[a-z0-9-~][a-z0-9-._~]*$/;
+
+const isMapping = (v: unknown): v is Record<string, unknown> =>
+  v !== null && typeof v === "object" && !Array.isArray(v);
+const shown = (v: unknown) =>
+  JSON.stringify(v)
+    ?.replace(/[^\x20-\x7e]/g, "?")
+    .slice(0, 60) ?? "undefined";
+
+function text(where: string, v: unknown, max = 60): string {
+  if (typeof v !== "string" || !v.trim() || v.length > max)
+    throw new Error(`${where}: must be a name of 1-${max} characters`);
+  return v.trim();
+}
+
+function list(
+  where: string,
+  v: unknown,
+  each: (x: unknown, at: string) => string,
+  { min = 1, max = 16 } = {},
+): string[] {
+  if (!Array.isArray(v) || v.length < min || v.length > max)
+    throw new Error(`${where}: must be a list of ${min}-${max} entries`);
+  return v.map((x, i) => each(x, `${where}[${i}]`));
+}
+
+function dirOk(at: string, x: unknown): string {
+  if (
+    typeof x === "string" &&
+    (x === "{system}" ||
+      ENV_DIR_RE.test(x) ||
+      (x.startsWith("~/") && !x.includes("..")) ||
+      (path.isAbsolute(x) && !x.includes("..")))
+  )
+    return x;
+  throw new Error(
+    `${at}: ${shown(x)} must be "~/…", an absolute path, "$NAME" or "{system}"`,
+  );
+}
+
+/**
+ * One catalog entry, checked; throws naming the key (`where`, e.g.
+ * "agents.runtime_catalog[0]"). Never guesses a missing field.
+ */
+export function validateCatalogEntry(
+  raw: unknown,
+  where: string,
+): CatalogEntry {
+  if (!isMapping(raw)) throw new Error(`${where}: must be a mapping`);
+  for (const k of Object.keys(raw))
+    if (!(ENTRY_KEYS as readonly string[]).includes(k))
+      throw new Error(
+        `${where}.${k}: unknown key (an entry takes ${ENTRY_KEYS.join(", ")})`,
+      );
+  const id = raw.id;
+  if (typeof id !== "string" || !ID_RE.test(id) || RESERVED_IDS.includes(id))
+    throw new Error(
+      `${where}.id: ${shown(id)} must be lowercase letters, digits and "-" (1-32), not ${RESERVED_IDS.join(", ")}`,
+    );
+  const bins = list(`${where}.bins`, raw.bins, (x, at) => {
+    if (typeof x === "string" && BIN_RE.test(x)) return x;
+    throw new Error(`${at}: ${shown(x)} must be an executable's name, no path`);
+  });
+  const version = raw.version;
+  if (!isMapping(version))
+    throw new Error(`${where}.version: must be a mapping { args, identity }`);
+  for (const k of Object.keys(version))
+    if (k !== "args" && k !== "identity")
+      throw new Error(
+        `${where}.version.${k}: unknown key (version takes args, identity)`,
+      );
+  const args = list(
+    `${where}.version.args`,
+    version.args,
+    (x, at) => {
+      if (typeof x === "string" && x.length <= 40 && !/[\0\n]/.test(x))
+        return x;
+      throw new Error(`${at}: must be an argument of up to 40 characters`);
+    },
+    { min: 1, max: 4 },
+  );
+  const idn = version.identity;
+  if (!isMapping(idn))
+    throw new Error(
+      `${where}.version.identity: must be a mapping with output, package or realpath_under`,
+    );
+  const identity: CatalogIdentity = {};
+  for (const [k, v] of Object.entries(idn)) {
+    const at = `${where}.version.identity.${k}`;
+    if (k === "output") {
+      if (typeof v !== "string" || !v || v.length > 100)
+        throw new Error(`${at}: must be a pattern of 1-100 characters`);
+      try {
+        new RegExp(v, "i");
+      } catch {
+        throw new Error(`${at}: ${shown(v)} is not a valid pattern`);
+      }
+      identity.output = v;
+    } else if (k === "package") {
+      if (typeof v !== "string" || !PKG_RE.test(v))
+        throw new Error(`${at}: ${shown(v)} is not an npm package name`);
+      identity.package = v;
+    } else if (k === "realpath_under") {
+      identity.realpath_under = list(at, v, (x, a) => dirOk(a, x)).map((d) => {
+        if (d === "{system}" || d.startsWith("$"))
+          throw new Error(`${at}: ${shown(d)} must be "~/…" or absolute`);
+        return d;
+      });
+    } else
+      throw new Error(
+        `${at}: unknown key (identity takes output, package, realpath_under)`,
+      );
+  }
+  if (!identity.output && !identity.package && !identity.realpath_under)
+    throw new Error(
+      `${where}.version.identity: needs output, package or realpath_under (how a path is known to be ${id})`,
+    );
+  const dirs = list(`${where}.dirs`, raw.dirs, (x, at) => dirOk(at, x), {
+    min: 0,
+    max: 16,
+  });
+  const launch = raw.launch;
+  if (!isMapping(launch) || Object.keys(launch).some((k) => k !== "argv"))
+    throw new Error(`${where}.launch: must be a mapping { argv: [...] }`);
+  const argv = list(
+    `${where}.launch.argv`,
+    launch.argv,
+    (x, at) => {
+      if (typeof x === "string" && x.length <= 200 && !/[\0\n]/.test(x))
+        return x;
+      throw new Error(`${at}: must be an argument of up to 200 characters`);
+    },
+    { min: 1, max: 16 },
+  );
+  if (argv[0] !== "{bin}")
+    throw new Error(
+      `${where}.launch.argv[0]: must be "{bin}" (the path Den found)`,
+    );
+  return {
+    id,
+    vendor: text(`${where}.vendor`, raw.vendor),
+    name: text(`${where}.name`, raw.name),
+    bins,
+    version: { args, identity },
+    dirs,
+    launch: { argv },
+  };
+}
+
+/**
+ * Extra entries (a distribution's, then user.yml's), checked; an id that
+ * is already in the catalog (a built-in, or an earlier entry) is refused.
+ */
+export function validateCatalog(
+  raw: unknown,
+  where: string,
+  taken: readonly string[] = BUILTIN_CATALOG.map((e) => e.id),
+): CatalogEntry[] {
+  if (raw === undefined || raw === null) return [];
+  if (!Array.isArray(raw))
+    throw new Error(`${where}: must be a list of runtime entries`);
+  const ids = [...taken];
+  return raw.map((x, i) => {
+    const e = validateCatalogEntry(x, `${where}[${i}]`);
+    if (ids.includes(e.id))
+      throw new Error(
+        `${where}[${i}].id: "${e.id}" is already in the catalog (set its path with agents.runtimes.${e.id}.bin instead)`,
+      );
+    ids.push(e.id);
+    return e;
+  });
+}
+
+/**
+ * An entry's install directories on this machine: "~/" from HOME, "$NAME"
+ * from the env (left out when unset), "{system}" as `system` gives them.
+ */
+export function entryDirs(
+  e: Pick<CatalogEntry, "dirs">,
+  home: string | undefined,
+  env: NodeJS.ProcessEnv,
+  system: (env: NodeJS.ProcessEnv) => string[],
+): string[] {
+  const out: string[] = [];
+  for (const d of e.dirs) {
+    if (d === "{system}") out.push(...system(env));
+    else if (d.startsWith("$")) {
+      const v = env[d.slice(1)];
+      if (v && path.isAbsolute(v)) out.push(v);
+    } else if (d.startsWith("~/")) {
+      if (home) out.push(path.join(home, d.slice(2)));
+    } else out.push(d);
+  }
+  return out;
+}
+
+/** A built-in entry by id. */
+export function builtinEntry(id: string): CatalogEntry | undefined {
+  return BUILTIN_CATALOG.find((e) => e.id === id);
+}

package/src/shared/agents/den-claude.ts

@@ -28,27 +28,43 @@
  *   session Den starts (an agent is not a child of the session that
  *   started it).
  *
- * detect runs nothing: `claude` on PATH, and its version read from where it
- * is installed (a native install's versions/<version>, or the npm package's
- * package.json).
+ * detect runs nothing (AG-0.1, runtime-search.ts): agents.runtimes.den-claude.bin
+ * in user.yml when set, else `claude` on the login shell's PATH, the
+ * caller's PATH, then its install directories (~/.local/bin, the native
+ * installer's; ~/.claude/local; /opt/homebrew/bin; /usr/local/bin; npm's
+ * global bin). Its version is read from where it is installed (a native
+ * install's versions/<version>, or the npm package's package.json); the
+ * Start dialog and `agents doctor` ask `claude --version` once as well.
  */
 import { randomUUID } from "crypto";
 import * as fs from "fs";
 import * as path from "path";
+import { configuredRuntimeBin } from "../den-config.js";
 import { rolePrompt } from "./personas.js";
 import {
   LaunchError,
   mcpConfigJson,
   mcpConfigPath,
-  whichBin,
+  type DetectOptions,
   type LaunchInput,
   type LaunchOutput,
   type Runtime,
   type RuntimeDetect,
 } from "./runtime.js";
+import { builtinEntry } from "./catalog.js";
+import { catalogFind } from "./catalog-runtime.js";
+import { locateRuntime, type RuntimeFind } from "./runtime-search.js";
 
 export const DEN_CLAUDE_ID = "den-claude";
 const BIN = "claude";
+
+/**
+ * Where Claude Code is found, and how its version is asked: its catalog
+ * entry (catalog.ts): the native installer's ~/.local/bin, the older
+ * local install's ~/.claude/local, Homebrew's, /usr/local/bin, npm's.
+ */
+export const DEN_CLAUDE_CATALOG = builtinEntry(DEN_CLAUDE_ID)!;
+export const DEN_CLAUDE_FIND: RuntimeFind = catalogFind(DEN_CLAUDE_CATALOG);
 const PACKAGE = "@anthropic-ai/claude-code";
 const SEMVER = /^\d+\.\d+\.\d+(?:[-+][0-9A-Za-z.-]+)?$/;
 
@@ -97,11 +113,29 @@ export function installedVersion(bin: string): string | undefined {
   return undefined;
 }
 
-function detect(env: NodeJS.ProcessEnv = process.env): RuntimeDetect {
-  const bin = whichBin(BIN, env);
-  if (!bin) return { available: false, why: `${BIN} is not on PATH` };
-  const version = installedVersion(bin);
-  return version ? { available: true, version } : { available: true };
+function detect(
+  env: NodeJS.ProcessEnv = process.env,
+  opts: DetectOptions = {},
+): RuntimeDetect {
+  const setting =
+    opts.bin === undefined ? configuredRuntimeBin(DEN_CLAUDE_ID) : opts.bin;
+  const found = locateRuntime(DEN_CLAUDE_ID, DEN_CLAUDE_FIND, env, setting);
+  if (!found.available || !found.path)
+    return {
+      available: false,
+      why: found.why ?? `${BIN} not found`,
+      // "setting": a bad agents.runtimes.<id>.bin is the reason, not the search.
+      ...(found.source ? { source: found.source } : {}),
+      searched: found.searched,
+    };
+  const version = installedVersion(found.path);
+  return {
+    available: true,
+    ...(version ? { version } : {}),
+    path: found.path,
+    ...(found.source ? { source: found.source } : {}),
+    searched: found.searched,
+  };
 }
 
 /**
@@ -231,13 +265,20 @@ function common(input: LaunchInput): {
 }
 
 function binFor(input: LaunchInput): string {
-  const bin = whichBin(BIN, input.env);
-  if (!bin) throw new LaunchError(`${DEN_CLAUDE_ID}: ${BIN} is not on PATH`);
-  return bin;
+  if (input.bin) return input.bin;
+  const d = detect(input.env);
+  if (!d.available || !d.path)
+    throw new LaunchError(`${DEN_CLAUDE_ID}: ${d.why ?? `${BIN} not found`}`);
+  return d.path;
 }
 
 export const denClaude: Runtime = {
   id: DEN_CLAUDE_ID,
+  label: DEN_CLAUDE_CATALOG.name,
+  vendor: DEN_CLAUDE_CATALOG.vendor,
+  catalog: DEN_CLAUDE_CATALOG,
+  denTools: true,
+  find: DEN_CLAUDE_FIND,
   detect,
   launch(input): LaunchOutput {
     const bin = binFor(input);

package/src/shared/agents/generic.ts

@@ -20,6 +20,7 @@ export const GENERIC_ID = "generic";
 
 export const generic: Runtime = {
   id: GENERIC_ID,
+  label: "Generic command",
   // The CLI is the persona's; there is nothing to look for until launch.
   detect: () => ({ available: true }),
   scrubEnv: [],