@plainconceptsplatform/agent-harness 3.8.2 → 3.9.0

package/cli/utils/exec.js

@@ -1,134 +1,134 @@
-import chalk from 'chalk'
-import { execa } from 'execa'
-import ora from 'ora'
-import { appendLine, redraw, rotateStep, startSpinner, stopSpinner } from './exec-spinner.js'
-import { MARKERS } from './terminal.js';
-
-// ── Public API ───────────────────────────────────────────────────────────────
-
-/**
- * Run a shell command with a spinner.
- * Returns { success, stdout, stderr }
- */
-export async function run(command, args = [], { label, cwd = process.cwd() } = {}) {
-  const spinner = ora(label ?? `${command} ${args.join(' ')}`).start();
-  try {
-    const result = await execa(command, args, { cwd, reject: false });
-    if (result.exitCode === 0) {
-      spinner.succeed();
-    } else {
-      spinner.fail();
-    }
-    return { success: result.exitCode === 0, stdout: result.stdout, stderr: result.stderr };
-  } catch (err) {
-    spinner.fail();
-    return { success: false, stdout: '', stderr: err.message };
-  }
-}
-
-/**
- * Check if a command is available on PATH.
- * Returns true/false.
- */
-export async function commandExists(command) {
-  try {
-    const result = await execa(command, ['--version'], { reject: false });
-    return result.exitCode === 0;
-  } catch {
-    return false;
-  }
-}
-
-/**
- * Print a section header, clears screen, shows previous step dimmed, starts new step.
- */
-export function header(text) {
-  rotateStep();
-
-  const line1 = '';
-  const line2 = chalk.bold.hex('#fe3d57')(`━━ ${text}`);
-  const line3 = '';
-
-  appendLine(line1);
-  appendLine(line2);
-  appendLine(line3);
-
-  redraw();
-
-  startSpinner('working...');
-}
-
-/**
- * Restart the step spinner after prompts or logs.
- */
-export function loading(text = 'working...') {
-  startSpinner(text);
-}
-
-/**
- * Print a success line.
- */
-export function success(text) {
-  stopSpinner();
-  const line = chalk.green(`${MARKERS.OK_PREFIX}${text}`);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print a warning line.
- */
-export function warn(text) {
-  stopSpinner();
-  const line = chalk.yellow(`${MARKERS.WARN_PREFIX}${text}`);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an error line.
- */
-export function error(text) {
-  stopSpinner();
-  const line = chalk.red(`${MARKERS.ERROR_PREFIX}${text}`);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an info line.
- */
-export function info(text) {
-  stopSpinner();
-  const line = chalk.dim(`${MARKERS.EMPTY}${text}`);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print an action prompt line (white bold, requires user interaction).
- */
-export function prompt(text) {
-  stopSpinner();
-  const line = chalk.bold(`${MARKERS.EMPTY}${text}`);
-  appendLine(line);
-  console.log(line);
-}
-
-/**
- * Print a code block.
- */
-export function code(lines) {
-  stopSpinner();
-  appendLine('');
-  console.log();
-  for (const line of lines) {
-    const formatted = chalk.bgGray.white(
-      `${MARKERS.EMPTY}${line}${MARKERS.EMPTY}`,
-    );
-    appendLine(formatted);
-    console.log(formatted);
-  }
-  appendLine('');
-  console.log();
-}
+import chalk from 'chalk'
+import { execa } from 'execa'
+import ora from 'ora'
+import { appendLine, redraw, rotateStep, startSpinner, stopSpinner } from './exec-spinner.js'
+import { MARKERS } from './terminal.js';
+
+// ── Public API ───────────────────────────────────────────────────────────────
+
+/**
+ * Run a shell command with a spinner.
+ * Returns { success, stdout, stderr }
+ */
+export async function run(command, args = [], { label, cwd = process.cwd() } = {}) {
+  const spinner = ora(label ?? `${command} ${args.join(' ')}`).start();
+  try {
+    const result = await execa(command, args, { cwd, reject: false });
+    if (result.exitCode === 0) {
+      spinner.succeed();
+    } else {
+      spinner.fail();
+    }
+    return { success: result.exitCode === 0, stdout: result.stdout, stderr: result.stderr };
+  } catch (err) {
+    spinner.fail();
+    return { success: false, stdout: '', stderr: err.message };
+  }
+}
+
+/**
+ * Check if a command is available on PATH.
+ * Returns true/false.
+ */
+export async function commandExists(command) {
+  try {
+    const result = await execa(command, ['--version'], { reject: false });
+    return result.exitCode === 0;
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Print a section header, clears screen, shows previous step dimmed, starts new step.
+ */
+export function header(text) {
+  rotateStep();
+
+  const line1 = '';
+  const line2 = chalk.bold.hex('#fe3d57')(`━━ ${text}`);
+  const line3 = '';
+
+  appendLine(line1);
+  appendLine(line2);
+  appendLine(line3);
+
+  redraw();
+
+  startSpinner('working...');
+}
+
+/**
+ * Restart the step spinner after prompts or logs.
+ */
+export function loading(text = 'working...') {
+  startSpinner(text);
+}
+
+/**
+ * Print a success line.
+ */
+export function success(text) {
+  stopSpinner();
+  const line = chalk.green(`${MARKERS.OK_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print a warning line.
+ */
+export function warn(text) {
+  stopSpinner();
+  const line = chalk.yellow(`${MARKERS.WARN_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an error line.
+ */
+export function error(text) {
+  stopSpinner();
+  const line = chalk.red(`${MARKERS.ERROR_PREFIX}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an info line.
+ */
+export function info(text) {
+  stopSpinner();
+  const line = chalk.dim(`${MARKERS.EMPTY}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print an action prompt line (white bold, requires user interaction).
+ */
+export function prompt(text) {
+  stopSpinner();
+  const line = chalk.bold(`${MARKERS.EMPTY}${text}`);
+  appendLine(line);
+  console.log(line);
+}
+
+/**
+ * Print a code block.
+ */
+export function code(lines) {
+  stopSpinner();
+  appendLine('');
+  console.log();
+  for (const line of lines) {
+    const formatted = chalk.bgGray.white(
+      `${MARKERS.EMPTY}${line}${MARKERS.EMPTY}`,
+    );
+    appendLine(formatted);
+    console.log(formatted);
+  }
+  appendLine('');
+  console.log();
+}

package/cli/utils/legacy-check.js

@@ -1,30 +1,30 @@
-import fse from 'fs-extra'
-import path from 'node:path'
-import { CONFIG_FILE, LEGACY_CONFIG_FILES, OPENCODE_DIR } from './paths.js'
-
-/**
- * Detect a repo onboarded by opencode-onboard v1.x.
- *
- * v2 renamed both the state files and the `ob-` skill prefix, and it does not
- * migrate: see the rename plan. Running against a v1 repo would otherwise fail
- * silently in two places, because `patchFile` returns without logging when its
- * markers are absent and the skill sync only removes stale directories under
- * `forceOverwrite && !updateMode`. The result is a half-patched harness that
- * still reports success, so refuse the run instead.
- */
-export async function findLegacyInstall(cwd = process.cwd()) {
-  const opencodeDir = path.join(cwd, OPENCODE_DIR)
-  if (await fse.pathExists(path.join(opencodeDir, CONFIG_FILE))) return null
-
-  const found = []
-  for (const file of LEGACY_CONFIG_FILES) {
-    if (await fse.pathExists(path.join(opencodeDir, file))) found.push(`${OPENCODE_DIR}/${file}`)
-  }
-
-  const skillsDir = path.join(cwd, '.agents', 'skills')
-  const legacySkills = (await fse.readdir(skillsDir).catch(() => []))
-    .filter(entry => entry.startsWith('ob-'))
-
-  if (found.length === 0 && legacySkills.length === 0) return null
-  return { files: found, skills: legacySkills }
-}
+import fse from 'fs-extra'
+import path from 'node:path'
+import { CONFIG_FILE, LEGACY_CONFIG_FILES, OPENCODE_DIR } from './paths.js'
+
+/**
+ * Detect a repo onboarded by opencode-onboard v1.x.
+ *
+ * v2 renamed both the state files and the `ob-` skill prefix, and it does not
+ * migrate: see the rename plan. Running against a v1 repo would otherwise fail
+ * silently in two places, because `patchFile` returns without logging when its
+ * markers are absent and the skill sync only removes stale directories under
+ * `forceOverwrite && !updateMode`. The result is a half-patched harness that
+ * still reports success, so refuse the run instead.
+ */
+export async function findLegacyInstall(cwd = process.cwd()) {
+  const opencodeDir = path.join(cwd, OPENCODE_DIR)
+  if (await fse.pathExists(path.join(opencodeDir, CONFIG_FILE))) return null
+
+  const found = []
+  for (const file of LEGACY_CONFIG_FILES) {
+    if (await fse.pathExists(path.join(opencodeDir, file))) found.push(`${OPENCODE_DIR}/${file}`)
+  }
+
+  const skillsDir = path.join(cwd, '.agents', 'skills')
+  const legacySkills = (await fse.readdir(skillsDir).catch(() => []))
+    .filter(entry => entry.startsWith('ob-'))
+
+  if (found.length === 0 && legacySkills.length === 0) return null
+  return { files: found, skills: legacySkills }
+}

package/cli/utils/models-cache.js

@@ -1,58 +1,58 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { parseModels } from './models-pricing.js'
-import { modelsCacheDir } from './paths.js'
-
-const CACHE_DIR = modelsCacheDir();
-const CACHE_FILE = path.join(CACHE_DIR, 'models-cache.json');
-const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 7 days
-const MODELS_URL = 'https://models.dev/api.json';
-
-async function loadCache() {
-  try {
-    if (!await fse.pathExists(CACHE_FILE)) return null;
-    const cache = await fse.readJson(CACHE_FILE);
-    if (!cache.timestamp || !cache.models) return null;
-    const age = Date.now() - cache.timestamp;
-    if (age > CACHE_TTL_MS) return null; // expired
-    return cache.models;
-  } catch {
-    return null;
-  }
-}
-
-async function saveCache(models) {
-  try {
-    await fse.ensureDir(CACHE_DIR);
-    await fse.writeJson(CACHE_FILE, { timestamp: Date.now(), models });
-  } catch {
-    // cache write failure is non-fatal
-  }
-}
-
-export async function fetchModels() {
-  // 1. Try cache first (fresh)
-  const cached = await loadCache();
-  if (cached) return { models: cached, source: 'cache' };
-
-  // 2. Try network
-  try {
-    const res = await fetch(MODELS_URL, { signal: AbortSignal.timeout(8000) });
-    if (!res.ok) throw new Error(`HTTP ${res.status}`);
-    const data = await res.json();
-    const models = parseModels(data);
-    await saveCache(models);
-    return { models, source: 'network' };
-  } catch {
-    // 3. Network failed, fall back to stale cache if available
-    try {
-      if (await fse.pathExists(CACHE_FILE)) {
-        const cache = await fse.readJson(CACHE_FILE);
-        if (cache.models?.length) return { models: cache.models, source: 'stale-cache' };
-      }
-    } catch {
-      // ignore
-    }
-    return { models: null, source: 'unavailable' };
-  }
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { parseModels } from './models-pricing.js'
+import { modelsCacheDir } from './paths.js'
+
+const CACHE_DIR = modelsCacheDir();
+const CACHE_FILE = path.join(CACHE_DIR, 'models-cache.json');
+const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 7 days
+const MODELS_URL = 'https://models.dev/api.json';
+
+async function loadCache() {
+  try {
+    if (!await fse.pathExists(CACHE_FILE)) return null;
+    const cache = await fse.readJson(CACHE_FILE);
+    if (!cache.timestamp || !cache.models) return null;
+    const age = Date.now() - cache.timestamp;
+    if (age > CACHE_TTL_MS) return null; // expired
+    return cache.models;
+  } catch {
+    return null;
+  }
+}
+
+async function saveCache(models) {
+  try {
+    await fse.ensureDir(CACHE_DIR);
+    await fse.writeJson(CACHE_FILE, { timestamp: Date.now(), models });
+  } catch {
+    // cache write failure is non-fatal
+  }
+}
+
+export async function fetchModels() {
+  // 1. Try cache first (fresh)
+  const cached = await loadCache();
+  if (cached) return { models: cached, source: 'cache' };
+
+  // 2. Try network
+  try {
+    const res = await fetch(MODELS_URL, { signal: AbortSignal.timeout(8000) });
+    if (!res.ok) throw new Error(`HTTP ${res.status}`);
+    const data = await res.json();
+    const models = parseModels(data);
+    await saveCache(models);
+    return { models, source: 'network' };
+  } catch {
+    // 3. Network failed, fall back to stale cache if available
+    try {
+      if (await fse.pathExists(CACHE_FILE)) {
+        const cache = await fse.readJson(CACHE_FILE);
+        if (cache.models?.length) return { models: cache.models, source: 'stale-cache' };
+      }
+    } catch {
+      // ignore
+    }
+    return { models: null, source: 'unavailable' };
+  }
+}

package/cli/utils/models-pricing.js

@@ -1,42 +1,42 @@
-// Providers considered "canonical" for reference pricing, in priority order.
-// When a model's own provider has no cost (e.g. github-copilot shows $0),
-// we look up the same model name in these providers and attach canonicalCost.
-export const CANONICAL_PROVIDERS = ['anthropic', 'openai', 'google', 'mistral', 'meta', 'cohere'];
-
-export function parseModels(data) {
-  // Build name → canonical cost lookup from authoritative providers first
-  const canonicalCostByName = new Map();
-  for (const providerId of CANONICAL_PROVIDERS) {
-    const provider = data[providerId];
-    if (!provider?.models) continue;
-    for (const model of Object.values(provider.models)) {
-      if (!model.tool_call) continue;
-      const name = model.name;
-      if (name && model.cost?.input !== undefined && !canonicalCostByName.has(name)) {
-        canonicalCostByName.set(name, model.cost.input);
-      }
-    }
-  }
-
-  const models = [];
-  for (const [providerId, provider] of Object.entries(data)) {
-    if (!provider.models) continue;
-    for (const [modelId, model] of Object.entries(provider.models)) {
-      if (!model.tool_call) continue;
-      const name = model.name || modelId;
-      const cost = model.cost?.input;
-      const canonicalCost = canonicalCostByName.get(name);
-      models.push({
-        id: `${providerId}/${modelId}`,
-        name,
-        cost,
-        // canonicalCost: cost from the authoritative provider for this model name.
-        // Defined when cost !== canonicalCost (different provider, reseller, or $0 subscription).
-        canonicalCost: canonicalCost !== undefined && canonicalCost !== cost ? canonicalCost : undefined,
-        context: model.limit?.context,
-      });
-    }
-  }
-  models.sort((a, b) => (a.cost ?? Infinity) - (b.cost ?? Infinity));
-  return models;
-}
+// Providers considered "canonical" for reference pricing, in priority order.
+// When a model's own provider has no cost (e.g. github-copilot shows $0),
+// we look up the same model name in these providers and attach canonicalCost.
+export const CANONICAL_PROVIDERS = ['anthropic', 'openai', 'google', 'mistral', 'meta', 'cohere'];
+
+export function parseModels(data) {
+  // Build name → canonical cost lookup from authoritative providers first
+  const canonicalCostByName = new Map();
+  for (const providerId of CANONICAL_PROVIDERS) {
+    const provider = data[providerId];
+    if (!provider?.models) continue;
+    for (const model of Object.values(provider.models)) {
+      if (!model.tool_call) continue;
+      const name = model.name;
+      if (name && model.cost?.input !== undefined && !canonicalCostByName.has(name)) {
+        canonicalCostByName.set(name, model.cost.input);
+      }
+    }
+  }
+
+  const models = [];
+  for (const [providerId, provider] of Object.entries(data)) {
+    if (!provider.models) continue;
+    for (const [modelId, model] of Object.entries(provider.models)) {
+      if (!model.tool_call) continue;
+      const name = model.name || modelId;
+      const cost = model.cost?.input;
+      const canonicalCost = canonicalCostByName.get(name);
+      models.push({
+        id: `${providerId}/${modelId}`,
+        name,
+        cost,
+        // canonicalCost: cost from the authoritative provider for this model name.
+        // Defined when cost !== canonicalCost (different provider, reseller, or $0 subscription).
+        canonicalCost: canonicalCost !== undefined && canonicalCost !== cost ? canonicalCost : undefined,
+        context: model.limit?.context,
+      });
+    }
+  }
+  models.sort((a, b) => (a.cost ?? Infinity) - (b.cost ?? Infinity));
+  return models;
+}