@open-code-review/cli 2.2.0 → 2.2.1

package/dist/index.js

@@ -16034,6 +16034,12 @@ function execBinary(binary, args, opts) {
     shell: isWindows
   });
 }
+async function execBinaryAsync(binary, args, opts) {
+  return execFilePromise(binary, args, {
+    ...opts,
+    shell: isWindows
+  });
+}
 function isProcessAlive(pid) {
   try {
     process.kill(pid, 0);
@@ -30086,7 +30092,7 @@ ${hint}
 }
 
 // src/lib/version.ts
-var CLI_VERSION = true ? "2.2.0" : createRequire(import.meta.url)("../../package.json").version;
+var CLI_VERSION = true ? "2.2.1" : createRequire(import.meta.url)("../../package.json").version;
 
 // src/lib/deps.ts
 init_src();
@@ -35520,23 +35526,62 @@ var sessionCommand = new Command("session").description("Manage agent-CLI sessio
 
 // src/lib/models.ts
 init_src();
-var BUNDLED_CLAUDE_MODELS = [
-  { id: "claude-opus-4-7", displayName: "Claude Opus 4.7" },
-  { id: "claude-sonnet-4-6", displayName: "Claude Sonnet 4.6" },
-  { id: "claude-haiku-4-5-20251001", displayName: "Claude Haiku 4.5" }
-];
-var BUNDLED_OPENCODE_MODELS = [
-  { id: "anthropic/claude-opus-4-7", provider: "anthropic" },
-  { id: "anthropic/claude-sonnet-4-6", provider: "anthropic" },
-  { id: "anthropic/claude-haiku-4-5-20251001", provider: "anthropic" }
-];
-function detectActiveVendor() {
-  for (const vendor of ["claude", "opencode"]) {
+function parseOpenCodeModelList(stdout) {
+  const models = [];
+  for (const rawLine of stdout.split(/\r?\n/)) {
+    const line = rawLine.trim();
+    if (!/^[^\s:]+\/\S+$/.test(line)) continue;
+    const provider = line.slice(0, line.indexOf("/"));
+    models.push({ id: line, provider });
+  }
+  return models.length > 0 ? models : null;
+}
+var VENDOR_MODEL_STRATEGIES = {
+  claude: {
+    displayName: "Claude Code",
+    native: {
+      // Verified against Claude Code 2.1.x: the CLI has no model-listing
+      // subcommand (`claude models --json` → "unknown option"). Revisit if
+      // a future release adds one.
+      unavailableReason: "Claude Code does not provide a model-listing command; showing its documented model aliases instead"
+    },
+    // Vendor-documented aliases that always track the latest generation —
+    // dated ids here would go stale by construction (the exact bug class of
+    // issue #39). Pinned dated ids remain available via free-text entry.
+    bundled: [
+      { id: "opus", displayName: "Claude Opus (latest)" },
+      { id: "sonnet", displayName: "Claude Sonnet (latest)" },
+      { id: "haiku", displayName: "Claude Haiku (latest)" }
+    ]
+  },
+  opencode: {
+    displayName: "OpenCode",
+    native: {
+      // Plain `opencode models` — newline-delimited ids. (`--json` is not a
+      // real flag, and `--verbose` interleaves JSON metadata blocks that
+      // defeat line parsing.)
+      args: ["models"],
+      parse: parseOpenCodeModelList
+    },
+    bundled: [
+      { id: "anthropic/claude-opus-4-8", provider: "anthropic" },
+      { id: "anthropic/claude-sonnet-4-6", provider: "anthropic" },
+      { id: "anthropic/claude-haiku-4-5", provider: "anthropic" }
+    ]
+  }
+};
+var SUPPORTED_VENDORS = Object.keys(
+  VENDOR_MODEL_STRATEGIES
+);
+function isModelVendor(value) {
+  return Object.hasOwn(VENDOR_MODEL_STRATEGIES, value);
+}
+async function detectActiveVendor() {
+  for (const vendor of SUPPORTED_VENDORS) {
     try {
-      execBinary(vendor, ["--version"], {
+      await execBinaryAsync(vendor, ["--version"], {
         encoding: "utf-8",
-        timeout: 3e3,
-        stdio: ["ignore", "pipe", "ignore"]
+        timeout: 3e3
       });
       return vendor;
     } catch {
@@ -35544,68 +35589,97 @@ function detectActiveVendor() {
   }
   return null;
 }
-function tryNativeEnumeration(vendor) {
+function describeProbeFailure(vendor, args, err) {
+  const command = `${vendor} ${args.join(" ")}`;
+  const e = err;
+  if (e.code === "ENOENT") {
+    return `\`${vendor}\` is not installed or not on PATH`;
+  }
+  if (e.killed) {
+    return `\`${command}\` timed out or exceeded output limits`;
+  }
+  const stderr = typeof e.stderr === "string" ? e.stderr.trim() : "";
+  const firstLine = (stderr.split(/\r?\n/)[0] ?? "").replace(/\u001b\[[0-9;]*[A-Za-z]/g, "").replace(/[\u0000-\u001f\u007f]/g, "").slice(0, 200);
+  const detail = firstLine ? `: ${firstLine}` : "";
+  const exit = typeof e.code === "number" ? ` with exit code ${e.code}` : "";
+  return `\`${command}\` failed${exit}${detail}`;
+}
+async function tryNativeEnumeration(vendor, probe) {
+  let stdout;
   try {
-    const output = execBinary(vendor, ["models", "--json"], {
+    const result = await execBinaryAsync(vendor, probe.args, {
       encoding: "utf-8",
-      timeout: 5e3,
-      stdio: ["ignore", "pipe", "ignore"]
+      timeout: 5e3
     });
-    const parsed = JSON.parse(output);
-    if (!Array.isArray(parsed)) return null;
-    const models = [];
-    for (const item of parsed) {
-      if (typeof item === "string") {
-        models.push({ id: item });
-      } else if (typeof item === "object" && item !== null && "id" in item && typeof item.id === "string") {
-        const obj = item;
-        const desc = { id: obj.id };
-        if (typeof obj.displayName === "string") desc.displayName = obj.displayName;
-        if (typeof obj.provider === "string") desc.provider = obj.provider;
-        if (Array.isArray(obj.tags)) {
-          desc.tags = obj.tags.filter((t) => typeof t === "string");
-        }
-        models.push(desc);
-      }
-    }
-    return models.length > 0 ? models : null;
-  } catch {
-    return null;
+    stdout = result.stdout;
+  } catch (err) {
+    return { models: null, reason: describeProbeFailure(vendor, probe.args, err) };
   }
+  const models = probe.parse(stdout);
+  if (!models) {
+    return {
+      models: null,
+      reason: `\`${vendor} ${probe.args.join(" ")}\` output did not contain any model identifiers`
+    };
+  }
+  return { models };
 }
-function bundledForVendor(vendor) {
-  if (vendor === "claude") return BUNDLED_CLAUDE_MODELS;
-  return BUNDLED_OPENCODE_MODELS;
-}
-function listModelsForVendor(vendor) {
-  const native = tryNativeEnumeration(vendor);
-  if (native) {
-    return { vendor, source: "native", models: native };
+var SUCCESS_TTL_MS = 6e4;
+var FAILURE_TTL_MS = 1e4;
+var cache = /* @__PURE__ */ new Map();
+async function listModelsForVendor(vendor) {
+  const cached = cache.get(vendor);
+  if (cached && cached.expiresAt > Date.now()) {
+    return cached.result;
+  }
+  const strategy = VENDOR_MODEL_STRATEGIES[vendor];
+  if (!strategy) {
+    throw new Error(`Unknown vendor: ${vendor}`);
   }
-  return { vendor, source: "bundled", models: bundledForVendor(vendor) };
+  let result;
+  if ("unavailableReason" in strategy.native) {
+    result = {
+      vendor,
+      source: "bundled",
+      models: strategy.bundled,
+      nativeUnavailableReason: strategy.native.unavailableReason
+    };
+  } else {
+    const native = await tryNativeEnumeration(vendor, strategy.native);
+    result = native.models ? { vendor, source: "native", models: native.models } : {
+      vendor,
+      source: "bundled",
+      models: strategy.bundled,
+      nativeUnavailableReason: native.reason
+    };
+  }
+  const ttl = result.source === "native" ? SUCCESS_TTL_MS : FAILURE_TTL_MS;
+  cache.set(vendor, { result, expiresAt: Date.now() + ttl });
+  return result;
 }
 
 // src/commands/models.ts
-var listSubcommand2 = new Command("list").description("List models the active AI CLI is willing to accept").option(
-  "--vendor <vendor>",
-  "Override autodetection (claude | opencode)"
-).option("--json", "Emit JSON for programmatic consumption").action(async (options) => {
+var vendorList = SUPPORTED_VENDORS.join(" | ");
+var listSubcommand2 = new Command("list").description("List models the active AI CLI is willing to accept").option("--vendor <vendor>", `Override autodetection (${vendorList})`).option("--json", "Emit JSON for programmatic consumption").action(async (options) => {
   let vendor;
   if (options.vendor) {
-    if (options.vendor !== "claude" && options.vendor !== "opencode") {
+    const requested = options.vendor.toLowerCase();
+    if (!isModelVendor(requested)) {
       console.error(
         source_default.red(
-          `Invalid --vendor: "${options.vendor}". Must be "claude" or "opencode".`
+          `Invalid --vendor: "${options.vendor}". Must be one of: ${vendorList}.`
         )
       );
       process.exit(1);
     }
-    vendor = options.vendor;
+    vendor = requested;
   } else {
-    vendor = detectActiveVendor();
+    vendor = await detectActiveVendor();
     if (!vendor) {
       if (options.json) {
-        console.log("[]");
+        console.log(
+          JSON.stringify({ vendor: null, source: null, models: [] }, null, 2)
+        );
         return;
       }
       console.error(
@@ -35616,16 +35690,18 @@ var listSubcommand2 = new Command("list").description("List models the active AI
       process.exit(1);
     }
   }
-  const { source, models } = listModelsForVendor(vendor);
+  const result = await listModelsForVendor(vendor);
   if (options.json) {
-    console.log(JSON.stringify(models, null, 2));
+    console.log(JSON.stringify(result, null, 2));
     return;
   }
+  const { source, models, nativeUnavailableReason } = result;
   console.log(source_default.bold(`Models for ${vendor} (${source})`));
   if (source === "bundled") {
+    const reason = nativeUnavailableReason ? ` \u2014 ${nativeUnavailableReason}` : "";
     console.log(
       source_default.dim(
-        "  Note: bundled fallback list \u2014 may be stale. Free-text input is always accepted."
+        `  Note: bundled fallback list${reason}. Free-text input is always accepted.`
       )
     );
   }
@@ -36844,10 +36920,10 @@ function readCache(cacheFile) {
     return null;
   }
 }
-function writeCache(cacheFile, cache) {
+function writeCache(cacheFile, cache2) {
   try {
     mkdirSync8(join28(cacheFile, ".."), { recursive: true });
-    writeFileSync10(cacheFile, JSON.stringify(cache));
+    writeFileSync10(cacheFile, JSON.stringify(cache2));
   } catch {
   }
 }
@@ -36869,14 +36945,14 @@ async function checkForUpdate(currentVersion, options) {
   }
   const cacheFile = join28(options?.cacheDir ?? CACHE_DIR2, "update-check.json");
   try {
-    const cache = readCache(cacheFile);
-    if (cache && Date.now() - cache.lastCheck < CHECK_INTERVAL_MS) {
-      if (!cache.latestVersion) return null;
-      if (!isNewer(cache.latestVersion, currentVersion)) return null;
+    const cache2 = readCache(cacheFile);
+    if (cache2 && Date.now() - cache2.lastCheck < CHECK_INTERVAL_MS) {
+      if (!cache2.latestVersion) return null;
+      if (!isNewer(cache2.latestVersion, currentVersion)) return null;
       return {
         updateAvailable: true,
         currentVersion,
-        latestVersion: cache.latestVersion,
+        latestVersion: cache2.latestVersion,
         updateCommand: detectUpdateCommand()
       };
     }

package/dist/lib/models.js

@@ -7,31 +7,70 @@ import {
 import { promisify } from "node:util";
 var execFilePromise = promisify(execFile);
 var isWindows = process.platform === "win32";
-function execBinary(binary, args, opts) {
-  return execFileSync(binary, args, {
+async function execBinaryAsync(binary, args, opts) {
+  return execFilePromise(binary, args, {
     ...opts,
     shell: isWindows
   });
 }
 
 // src/lib/models.ts
-var BUNDLED_CLAUDE_MODELS = [
-  { id: "claude-opus-4-7", displayName: "Claude Opus 4.7" },
-  { id: "claude-sonnet-4-6", displayName: "Claude Sonnet 4.6" },
-  { id: "claude-haiku-4-5-20251001", displayName: "Claude Haiku 4.5" }
-];
-var BUNDLED_OPENCODE_MODELS = [
-  { id: "anthropic/claude-opus-4-7", provider: "anthropic" },
-  { id: "anthropic/claude-sonnet-4-6", provider: "anthropic" },
-  { id: "anthropic/claude-haiku-4-5-20251001", provider: "anthropic" }
-];
-function detectActiveVendor() {
-  for (const vendor of ["claude", "opencode"]) {
+function parseOpenCodeModelList(stdout) {
+  const models = [];
+  for (const rawLine of stdout.split(/\r?\n/)) {
+    const line = rawLine.trim();
+    if (!/^[^\s:]+\/\S+$/.test(line)) continue;
+    const provider = line.slice(0, line.indexOf("/"));
+    models.push({ id: line, provider });
+  }
+  return models.length > 0 ? models : null;
+}
+var VENDOR_MODEL_STRATEGIES = {
+  claude: {
+    displayName: "Claude Code",
+    native: {
+      // Verified against Claude Code 2.1.x: the CLI has no model-listing
+      // subcommand (`claude models --json` → "unknown option"). Revisit if
+      // a future release adds one.
+      unavailableReason: "Claude Code does not provide a model-listing command; showing its documented model aliases instead"
+    },
+    // Vendor-documented aliases that always track the latest generation —
+    // dated ids here would go stale by construction (the exact bug class of
+    // issue #39). Pinned dated ids remain available via free-text entry.
+    bundled: [
+      { id: "opus", displayName: "Claude Opus (latest)" },
+      { id: "sonnet", displayName: "Claude Sonnet (latest)" },
+      { id: "haiku", displayName: "Claude Haiku (latest)" }
+    ]
+  },
+  opencode: {
+    displayName: "OpenCode",
+    native: {
+      // Plain `opencode models` — newline-delimited ids. (`--json` is not a
+      // real flag, and `--verbose` interleaves JSON metadata blocks that
+      // defeat line parsing.)
+      args: ["models"],
+      parse: parseOpenCodeModelList
+    },
+    bundled: [
+      { id: "anthropic/claude-opus-4-8", provider: "anthropic" },
+      { id: "anthropic/claude-sonnet-4-6", provider: "anthropic" },
+      { id: "anthropic/claude-haiku-4-5", provider: "anthropic" }
+    ]
+  }
+};
+var SUPPORTED_VENDORS = Object.keys(
+  VENDOR_MODEL_STRATEGIES
+);
+function isModelVendor(value) {
+  return Object.hasOwn(VENDOR_MODEL_STRATEGIES, value);
+}
+async function detectActiveVendor() {
+  for (const vendor of SUPPORTED_VENDORS) {
     try {
-      execBinary(vendor, ["--version"], {
+      await execBinaryAsync(vendor, ["--version"], {
         encoding: "utf-8",
-        timeout: 3e3,
-        stdio: ["ignore", "pipe", "ignore"]
+        timeout: 3e3
       });
       return vendor;
     } catch {
@@ -39,47 +78,83 @@ function detectActiveVendor() {
   }
   return null;
 }
-function tryNativeEnumeration(vendor) {
+function describeProbeFailure(vendor, args, err) {
+  const command = `${vendor} ${args.join(" ")}`;
+  const e = err;
+  if (e.code === "ENOENT") {
+    return `\`${vendor}\` is not installed or not on PATH`;
+  }
+  if (e.killed) {
+    return `\`${command}\` timed out or exceeded output limits`;
+  }
+  const stderr = typeof e.stderr === "string" ? e.stderr.trim() : "";
+  const firstLine = (stderr.split(/\r?\n/)[0] ?? "").replace(/\u001b\[[0-9;]*[A-Za-z]/g, "").replace(/[\u0000-\u001f\u007f]/g, "").slice(0, 200);
+  const detail = firstLine ? `: ${firstLine}` : "";
+  const exit = typeof e.code === "number" ? ` with exit code ${e.code}` : "";
+  return `\`${command}\` failed${exit}${detail}`;
+}
+async function tryNativeEnumeration(vendor, probe) {
+  let stdout;
   try {
-    const output = execBinary(vendor, ["models", "--json"], {
+    const result = await execBinaryAsync(vendor, probe.args, {
       encoding: "utf-8",
-      timeout: 5e3,
-      stdio: ["ignore", "pipe", "ignore"]
+      timeout: 5e3
     });
-    const parsed = JSON.parse(output);
-    if (!Array.isArray(parsed)) return null;
-    const models = [];
-    for (const item of parsed) {
-      if (typeof item === "string") {
-        models.push({ id: item });
-      } else if (typeof item === "object" && item !== null && "id" in item && typeof item.id === "string") {
-        const obj = item;
-        const desc = { id: obj.id };
-        if (typeof obj.displayName === "string") desc.displayName = obj.displayName;
-        if (typeof obj.provider === "string") desc.provider = obj.provider;
-        if (Array.isArray(obj.tags)) {
-          desc.tags = obj.tags.filter((t) => typeof t === "string");
-        }
-        models.push(desc);
-      }
-    }
-    return models.length > 0 ? models : null;
-  } catch {
-    return null;
+    stdout = result.stdout;
+  } catch (err) {
+    return { models: null, reason: describeProbeFailure(vendor, probe.args, err) };
   }
+  const models = probe.parse(stdout);
+  if (!models) {
+    return {
+      models: null,
+      reason: `\`${vendor} ${probe.args.join(" ")}\` output did not contain any model identifiers`
+    };
+  }
+  return { models };
 }
-function bundledForVendor(vendor) {
-  if (vendor === "claude") return BUNDLED_CLAUDE_MODELS;
-  return BUNDLED_OPENCODE_MODELS;
+var SUCCESS_TTL_MS = 6e4;
+var FAILURE_TTL_MS = 1e4;
+var cache = /* @__PURE__ */ new Map();
+function clearModelListCache() {
+  cache.clear();
 }
-function listModelsForVendor(vendor) {
-  const native = tryNativeEnumeration(vendor);
-  if (native) {
-    return { vendor, source: "native", models: native };
+async function listModelsForVendor(vendor) {
+  const cached = cache.get(vendor);
+  if (cached && cached.expiresAt > Date.now()) {
+    return cached.result;
+  }
+  const strategy = VENDOR_MODEL_STRATEGIES[vendor];
+  if (!strategy) {
+    throw new Error(`Unknown vendor: ${vendor}`);
+  }
+  let result;
+  if ("unavailableReason" in strategy.native) {
+    result = {
+      vendor,
+      source: "bundled",
+      models: strategy.bundled,
+      nativeUnavailableReason: strategy.native.unavailableReason
+    };
+  } else {
+    const native = await tryNativeEnumeration(vendor, strategy.native);
+    result = native.models ? { vendor, source: "native", models: native.models } : {
+      vendor,
+      source: "bundled",
+      models: strategy.bundled,
+      nativeUnavailableReason: native.reason
+    };
   }
-  return { vendor, source: "bundled", models: bundledForVendor(vendor) };
+  const ttl = result.source === "native" ? SUCCESS_TTL_MS : FAILURE_TTL_MS;
+  cache.set(vendor, { result, expiresAt: Date.now() + ttl });
+  return result;
 }
 export {
+  SUPPORTED_VENDORS,
+  VENDOR_MODEL_STRATEGIES,
+  clearModelListCache,
   detectActiveVendor,
-  listModelsForVendor
+  isModelVendor,
+  listModelsForVendor,
+  parseOpenCodeModelList
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-code-review/cli",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "CLI for Open Code Review - Multi-environment setup and progress tracking",
   "type": "module",
   "bin": {
@@ -80,7 +80,7 @@
     "ora": "^8.1.1",
     "socket.io": "^4.8",
     "yaml": "^2.8.3",
-    "@open-code-review/agents": "2.2.0"
+    "@open-code-review/agents": "2.2.1"
   },
   "publishConfig": {
     "access": "public"

package/dist/dashboard/client/assets/channel-rgw7C1e7.js

@@ -1 +0,0 @@
-import{ap as o,aq as n}from"./mermaid-renderer-D2jYNs7K.js";const t=(a,r)=>o.lang.round(n.parse(a)[r]);export{t as c};

package/dist/dashboard/client/assets/classDiagram-2ON5EDUG-DTGi7d9X.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BSRfOovX.js";import{_ as i}from"./mermaid-renderer-D2jYNs7K.js";import"./chunk-FMBD7UC4-5xWP6GRj.js";import"./chunk-55IACEB6-C8xpXw9G.js";import"./chunk-QN33PNHL-DfNCVcy8.js";import"./index-C3NEq704.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/dashboard/client/assets/classDiagram-v2-WZHVMYZB-DTGi7d9X.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-B4BG7PRW-BSRfOovX.js";import{_ as i}from"./mermaid-renderer-D2jYNs7K.js";import"./chunk-FMBD7UC4-5xWP6GRj.js";import"./chunk-55IACEB6-C8xpXw9G.js";import"./chunk-QN33PNHL-DfNCVcy8.js";import"./index-C3NEq704.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/dashboard/client/assets/clone-Cz7hswqi.js

@@ -1 +0,0 @@
-import{b as r}from"./_baseUniq-CFHdos6T.js";var e=4;function a(o){return r(o,e)}export{a as c};