@plainconceptsplatform/agent-harness 3.5.0 → 3.7.0

package/cli/steps/metadata/index.js

@@ -1,115 +1,115 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import path from 'path'
-import { createRequire } from 'node:module'
-import { header, success, warn } from '../../utils/exec.js'
-import { configPath } from '../../utils/paths.js'
-
-const require = createRequire(import.meta.url)
-const { version: harnessVersion } = require('../../../package.json')
-
-function clampConcurrency(n) {
-  const v = Number(n)
-  if (!Number.isFinite(v)) return 3
-  return Math.min(5, Math.max(1, Math.round(v)))
-}
-
-async function detectOpencodeVersion() {
-  try {
-    const result = await execa('opencode', ['--version'], { reject: false })
-    if (result.exitCode !== 0) return null
-    // `opencode --version` prints "opencode v2.0.22" on some builds and "2.0.22" on others;
-    // record the bare version so the field does not churn between machines.
-    const output = (result.stdout || result.stderr || '').trim()
-    return output.match(/\d+\.\d+\.\d+[\w.-]*/)?.[0] ?? (output || null)
-  } catch {
-    return null
-  }
-}
-
-export async function writeHarnessConfig(data) {
-  header('Step 10, Writing onboarding metadata')
-
-  const opencodeVersion = await detectOpencodeVersion()
-  const cwd = data.cwd ?? process.cwd()
-  const target = configPath(cwd)
-
-  // Read the existing file so load-bearing config (models, maxConcurrent)
-  // is preserved across metadata refreshes when not explicitly provided.
-  const existing = await fse.readJson(target).catch(() => null)
-
-  const selectedModels = Object.fromEntries(
-    Object.entries({
-      plan: data.planModel,
-      build: data.buildModel,
-      fast: data.fastModel,
-    }).filter(([, value]) => value)
-  )
-  const models = Object.keys(selectedModels).length > 0
-    ? { ...existing?.models, ...selectedModels }
-    : existing?.models ?? null
-  const maxConcurrent = clampConcurrency(data.maxConcurrentAgents ?? existing?.agents?.maxConcurrent ?? 3)
-
-  const optionalTools = data.optionalTools ?? existing?.tools ?? {}
-  const { caveman: _caveman, ...existingTools } = existing?.tools ?? {}
-  const tools = {
-    rtk: !!(optionalTools.rtk?.optedIn ?? optionalTools.rtk),
-    quota: !!(optionalTools.quota?.optedIn ?? optionalTools.quota),
-    simpleEnglish: !!(
-      optionalTools.simpleEnglish?.optedIn ?? optionalTools.simpleEnglish ??
-      optionalTools.caveman?.optedIn ?? optionalTools.caveman
-    ),
-    codegraph: !!(optionalTools.codegraph?.optedIn ?? optionalTools.codegraph),
-    memory: !!(optionalTools.memory?.optedIn ?? optionalTools.memory),
-    // humanizer was missing here, so a selection never survived a metadata
-    // write and patchGuardrails always saw it as off, leaving its marker
-    // section empty however the project was actually set up.
-    humanizer: !!(optionalTools.humanizer?.optedIn ?? optionalTools.humanizer),
-  }
-
-  const payload = {
-    ...existing,
-    // Never lower a schema version a repository already records.
-    version: Math.max(2, Number(existing?.version) || 0),
-    generatedAt: new Date().toISOString(),
-    harnessVersion,
-    opencodeVersion,
-
-    platform: {
-      ...existing?.platform,
-      backlog: data.backlogPlatform ?? 'none',
-      repo: data.repoPlatform ?? 'none',
-    },
-
-    ...(models ? { models } : {}),
-
-    agents: {
-      ...existing?.agents,
-      maxConcurrent,
-    },
-
-    source: {
-      ...existing?.source,
-      mode: data.sourceMode ?? 'current',
-      roots: data.sourceRoots ?? [],
-    },
-
-    tools: { ...existingTools, ...tools },
-
-    preexisting: {
-      ...existing?.preexisting,
-      design: !!data.hasDesign,
-      architecture: !!data.hasArchitecture,
-      openspec: !!data.hasOpenspec,
-    },
-  }
-
-  try {
-    await fse.ensureDir(path.dirname(target))
-    await fse.writeJson(target, payload, { spaces: 2 })
-    success('Wrote .opencode/harness.json')
-    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
-  } catch (err) {
-    warn(`Could not write onboarding metadata: ${err.message}`)
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import path from 'path'
+import { createRequire } from 'node:module'
+import { header, success, warn } from '../../utils/exec.js'
+import { configPath } from '../../utils/paths.js'
+
+const require = createRequire(import.meta.url)
+const { version: harnessVersion } = require('../../../package.json')
+
+function clampConcurrency(n) {
+  const v = Number(n)
+  if (!Number.isFinite(v)) return 3
+  return Math.min(5, Math.max(1, Math.round(v)))
+}
+
+async function detectOpencodeVersion() {
+  try {
+    const result = await execa('opencode', ['--version'], { reject: false })
+    if (result.exitCode !== 0) return null
+    // `opencode --version` prints "opencode v2.0.22" on some builds and "2.0.22" on others;
+    // record the bare version so the field does not churn between machines.
+    const output = (result.stdout || result.stderr || '').trim()
+    return output.match(/\d+\.\d+\.\d+[\w.-]*/)?.[0] ?? (output || null)
+  } catch {
+    return null
+  }
+}
+
+export async function writeHarnessConfig(data) {
+  header('Step 10, Writing onboarding metadata')
+
+  const opencodeVersion = await detectOpencodeVersion()
+  const cwd = data.cwd ?? process.cwd()
+  const target = configPath(cwd)
+
+  // Read the existing file so load-bearing config (models, maxConcurrent)
+  // is preserved across metadata refreshes when not explicitly provided.
+  const existing = await fse.readJson(target).catch(() => null)
+
+  const selectedModels = Object.fromEntries(
+    Object.entries({
+      plan: data.planModel,
+      build: data.buildModel,
+      fast: data.fastModel,
+    }).filter(([, value]) => value)
+  )
+  const models = Object.keys(selectedModels).length > 0
+    ? { ...existing?.models, ...selectedModels }
+    : existing?.models ?? null
+  const maxConcurrent = clampConcurrency(data.maxConcurrentAgents ?? existing?.agents?.maxConcurrent ?? 3)
+
+  const optionalTools = data.optionalTools ?? existing?.tools ?? {}
+  const { caveman: _caveman, ...existingTools } = existing?.tools ?? {}
+  const tools = {
+    rtk: !!(optionalTools.rtk?.optedIn ?? optionalTools.rtk),
+    quota: !!(optionalTools.quota?.optedIn ?? optionalTools.quota),
+    simpleEnglish: !!(
+      optionalTools.simpleEnglish?.optedIn ?? optionalTools.simpleEnglish ??
+      optionalTools.caveman?.optedIn ?? optionalTools.caveman
+    ),
+    codegraph: !!(optionalTools.codegraph?.optedIn ?? optionalTools.codegraph),
+    memory: !!(optionalTools.memory?.optedIn ?? optionalTools.memory),
+    // humanizer was missing here, so a selection never survived a metadata
+    // write and patchGuardrails always saw it as off, leaving its marker
+    // section empty however the project was actually set up.
+    humanizer: !!(optionalTools.humanizer?.optedIn ?? optionalTools.humanizer),
+  }
+
+  const payload = {
+    ...existing,
+    // Never lower a schema version a repository already records.
+    version: Math.max(2, Number(existing?.version) || 0),
+    generatedAt: new Date().toISOString(),
+    harnessVersion,
+    opencodeVersion,
+
+    platform: {
+      ...existing?.platform,
+      backlog: data.backlogPlatform ?? 'none',
+      repo: data.repoPlatform ?? 'none',
+    },
+
+    ...(models ? { models } : {}),
+
+    agents: {
+      ...existing?.agents,
+      maxConcurrent,
+    },
+
+    source: {
+      ...existing?.source,
+      mode: data.sourceMode ?? 'current',
+      roots: data.sourceRoots ?? [],
+    },
+
+    tools: { ...existingTools, ...tools },
+
+    preexisting: {
+      ...existing?.preexisting,
+      design: !!data.hasDesign,
+      architecture: !!data.hasArchitecture,
+      openspec: !!data.hasOpenspec,
+    },
+  }
+
+  try {
+    await fse.ensureDir(path.dirname(target))
+    await fse.writeJson(target, payload, { spaces: 2 })
+    success('Wrote .opencode/harness.json')
+    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
+  } catch (err) {
+    warn(`Could not write onboarding metadata: ${err.message}`)
+  }
+}

package/cli/steps/models/format.js

@@ -1,88 +1,88 @@
-import { search, select } from '@inquirer/prompts'
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const MODELS_PRESET_PATH = path.resolve(__dirname, '../../presets/models.json');
-const modelsPreset = await fse.readJson(MODELS_PRESET_PATH);
-
-function costTier(input) {
-  if (input === undefined || input === null) return '';
-  const tier = modelsPreset.costTiers.find(t => t.max === undefined || input <= t.max);
-  return tier ? ` ${tier.label}` : '';
-}
-
-function costTierDisplay(cost, canonicalCost) {
-  return costTier(canonicalCost !== undefined ? canonicalCost : cost);
-}
-
-function formatPrice(price) {
-  if (price === undefined || price === null) return '?';
-  if (price === 0) return '$0 (subscription)';
-  return `$${price}/M`;
-}
-
-export function buildDisplayModels(rawModels) {
-  return rawModels.map(m => {
-    const priceStr = formatPrice(m.cost);
-    const canonicalNote = m.canonicalCost !== undefined
-      ? ` · official price: ${formatPrice(m.canonicalCost)}/M`
-      : '';
-    const context = m.context ? `${m.context / 1000}k` : '?';
-    return {
-      ...m,
-      label: `${m.name}${costTierDisplay(m.cost, m.canonicalCost)}, ${m.id}`,
-      description: `${priceStr}${canonicalNote} · context: ${context}`,
-    };
-  });
-}
-
-export function buildModelChoices(input, models) {
-  const q = (input || '').toLowerCase();
-  const filtered = q
-    ? models.filter(m =>
-        m.label.toLowerCase().includes(q) ||
-        m.id.toLowerCase().includes(q)
-      )
-    : models;
-
-  return [
-    {
-      name: 'None',
-      value: null,
-      description: 'Leave this model unset',
-    },
-    {
-      name: '← Change provider',
-      value: '__back__',
-      description: 'Go back to provider selection',
-    },
-    ...filtered.slice(0, 50).map(m => ({
-      name: m.label,
-      value: m.id,
-      description: m.description,
-    })),
-  ];
-}
-
-export function pickModel(message, models) {
-  return search({
-    message,
-    source: input => buildModelChoices(input, models),
-  });
-}
-
-export function filterModelsByProvider(models, provider) {
-  if (!provider) return models
-  return models.filter(m => m.id.startsWith(`${provider}/`))
-}
-
-export function pickProvider(message) {
-  return select({
-    message,
-    choices: modelsPreset.providers.map(p => ({ value: p.value, name: p.name })),
-  })
-}
-
-export { modelsPreset };
+import { search, select } from '@inquirer/prompts'
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const MODELS_PRESET_PATH = path.resolve(__dirname, '../../presets/models.json');
+const modelsPreset = await fse.readJson(MODELS_PRESET_PATH);
+
+function costTier(input) {
+  if (input === undefined || input === null) return '';
+  const tier = modelsPreset.costTiers.find(t => t.max === undefined || input <= t.max);
+  return tier ? ` ${tier.label}` : '';
+}
+
+function costTierDisplay(cost, canonicalCost) {
+  return costTier(canonicalCost !== undefined ? canonicalCost : cost);
+}
+
+function formatPrice(price) {
+  if (price === undefined || price === null) return '?';
+  if (price === 0) return '$0 (subscription)';
+  return `$${price}/M`;
+}
+
+export function buildDisplayModels(rawModels) {
+  return rawModels.map(m => {
+    const priceStr = formatPrice(m.cost);
+    const canonicalNote = m.canonicalCost !== undefined
+      ? ` · official price: ${formatPrice(m.canonicalCost)}/M`
+      : '';
+    const context = m.context ? `${m.context / 1000}k` : '?';
+    return {
+      ...m,
+      label: `${m.name}${costTierDisplay(m.cost, m.canonicalCost)}, ${m.id}`,
+      description: `${priceStr}${canonicalNote} · context: ${context}`,
+    };
+  });
+}
+
+export function buildModelChoices(input, models) {
+  const q = (input || '').toLowerCase();
+  const filtered = q
+    ? models.filter(m =>
+        m.label.toLowerCase().includes(q) ||
+        m.id.toLowerCase().includes(q)
+      )
+    : models;
+
+  return [
+    {
+      name: 'None',
+      value: null,
+      description: 'Leave this model unset',
+    },
+    {
+      name: '← Change provider',
+      value: '__back__',
+      description: 'Go back to provider selection',
+    },
+    ...filtered.slice(0, 50).map(m => ({
+      name: m.label,
+      value: m.id,
+      description: m.description,
+    })),
+  ];
+}
+
+export function pickModel(message, models) {
+  return search({
+    message,
+    source: input => buildModelChoices(input, models),
+  });
+}
+
+export function filterModelsByProvider(models, provider) {
+  if (!provider) return models
+  return models.filter(m => m.id.startsWith(`${provider}/`))
+}
+
+export function pickProvider(message) {
+  return select({
+    message,
+    choices: modelsPreset.providers.map(p => ({ value: p.value, name: p.name })),
+  })
+}
+
+export { modelsPreset };

package/cli/steps/models/index.js

@@ -1,64 +1,64 @@
-import { header, info, prompt, success, warn } from '../../utils/exec.js'
-import { fetchModels } from '../../utils/models-cache.js'
-import { buildDisplayModels, filterModelsByProvider, modelsPreset, pickModel, pickProvider } from './format.js'
-import { writeModelsToConfigs } from './write.js'
-
-async function pickRoleModel(role, allModels) {
-  const rolePreset = modelsPreset.roles[role]
-  const [first, ...rest] = rolePreset.info
-  prompt(first)
-  for (const line of rest) info(line)
-  console.log()
-
-  while (true) {
-    const provider = await pickProvider('Provider:')
-    const filtered = filterModelsByProvider(allModels, provider)
-    const available = filtered.length > 0 ? filtered : allModels
-
-    const suggestion = rolePreset.suggestions?.[provider]
-    if (suggestion) info(`  Suggested: ${suggestion}`)
-    if (filtered.length === 0 && provider) warn(`  No ${provider} models found: showing all`)
-
-    const model = await pickModel(rolePreset.prompt, available)
-    if (model !== '__back__') return model
-  }
-}
-
-export async function chooseModels() {
-  header('Step 7, Choose models');
-
-  info('Fetching available models from models.dev...');
-  const { models: rawModels, source } = await fetchModels();
-
-  if (!rawModels) {
-    warn('Could not fetch models (offline and no cache). Skipping model selection.');
-    warn('Set models later in opencode.jsonc');
-    return;
-  }
-
-  if (source === 'stale-cache') {
-    warn('Network unavailable, using cached model list (may be outdated).');
-  } else if (source === 'cache') {
-    info('Using cached model list (refreshes weekly).');
-  }
-
-  const models = buildDisplayModels(rawModels);
-  success(`${models.length} models available`);
-  console.log();
-  info('Three models: main session, engineers, and light commands.');
-  info('Cost indicators: [$] cheap  [$$] mid  [$$$] expensive');
-  info('Pick provider first, then search by name. Change later in opencode.jsonc');
-  console.log();
-
-  const planModel  = await pickRoleModel('plan',  models); console.log();
-  const buildModel = await pickRoleModel('build', models); console.log();
-  const fastModel  = await pickRoleModel('fast',  models); console.log();
-
-  await writeModelsToConfigs({ planModel, buildModel, fastModel });
-
-  console.log();
-  warn('Make sure you have API access to the selected models.');
-  warn('Change them anytime in opencode.jsonc');
-
-  return { planModel, buildModel, fastModel };
-}
+import { header, info, prompt, success, warn } from '../../utils/exec.js'
+import { fetchModels } from '../../utils/models-cache.js'
+import { buildDisplayModels, filterModelsByProvider, modelsPreset, pickModel, pickProvider } from './format.js'
+import { writeModelsToConfigs } from './write.js'
+
+async function pickRoleModel(role, allModels) {
+  const rolePreset = modelsPreset.roles[role]
+  const [first, ...rest] = rolePreset.info
+  prompt(first)
+  for (const line of rest) info(line)
+  console.log()
+
+  while (true) {
+    const provider = await pickProvider('Provider:')
+    const filtered = filterModelsByProvider(allModels, provider)
+    const available = filtered.length > 0 ? filtered : allModels
+
+    const suggestion = rolePreset.suggestions?.[provider]
+    if (suggestion) info(`  Suggested: ${suggestion}`)
+    if (filtered.length === 0 && provider) warn(`  No ${provider} models found: showing all`)
+
+    const model = await pickModel(rolePreset.prompt, available)
+    if (model !== '__back__') return model
+  }
+}
+
+export async function chooseModels() {
+  header('Step 7, Choose models');
+
+  info('Fetching available models from models.dev...');
+  const { models: rawModels, source } = await fetchModels();
+
+  if (!rawModels) {
+    warn('Could not fetch models (offline and no cache). Skipping model selection.');
+    warn('Set models later in opencode.jsonc');
+    return;
+  }
+
+  if (source === 'stale-cache') {
+    warn('Network unavailable, using cached model list (may be outdated).');
+  } else if (source === 'cache') {
+    info('Using cached model list (refreshes weekly).');
+  }
+
+  const models = buildDisplayModels(rawModels);
+  success(`${models.length} models available`);
+  console.log();
+  info('Three models: main session, engineers, and light commands.');
+  info('Cost indicators: [$] cheap  [$$] mid  [$$$] expensive');
+  info('Pick provider first, then search by name. Change later in opencode.jsonc');
+  console.log();
+
+  const planModel  = await pickRoleModel('plan',  models); console.log();
+  const buildModel = await pickRoleModel('build', models); console.log();
+  const fastModel  = await pickRoleModel('fast',  models); console.log();
+
+  await writeModelsToConfigs({ planModel, buildModel, fastModel });
+
+  console.log();
+  warn('Make sure you have API access to the selected models.');
+  warn('Change them anytime in opencode.jsonc');
+
+  return { planModel, buildModel, fastModel };
+}

package/cli/steps/models/write.js

@@ -1,38 +1,38 @@
-import fse from 'fs-extra'
-import { applyEdits, modify, parse } from 'jsonc-parser'
-import path from 'path'
-import { success, warn } from '../../utils/exec.js'
-
-/**
- * Writes the selected build model as the default model in opencode.jsonc.
- * Per-tier models (plan/build/fast) are recorded in harness.json (models)
- * by the metadata step, then consumed by the agent-variant generator and /plan-apply.
- *
- * opencode tolerates JSONC (comments, trailing commas) in its config, so this
- * must not use readJson/writeJson: a strict parse crashes on valid configs and
- * a strict rewrite deletes user comments. jsonc-parser edits only the `model`
- * key and preserves the rest of the file byte-for-byte.
- */
-export async function writeModelsToConfigs({ buildModel, keepExisting = false, cwd = process.cwd() }) {
-  const opencodeJsonPath = path.join(cwd, 'opencode.jsonc')
-  if (!await fse.pathExists(opencodeJsonPath)) return
-
-  const text = await fse.readFile(opencodeJsonPath, 'utf-8')
-  const errors = []
-  parse(text, errors)
-  if (errors.length > 0) {
-    warn('opencode.jsonc could not be parsed: leaving it untouched. Set "model" manually.')
-    return
-  }
-
-  // A model the repository already chose is its decision, not the wizard's: `update` used to
-  // reset a deliberately pinned default to the build tier in every consumer it ran in.
-  if (keepExisting && typeof parse(text)?.model === 'string') return
-
-  // `undefined` removes the key; a string value sets it.
-  const edits = modify(text, ['model'], buildModel || undefined, {
-    formattingOptions: { insertSpaces: true, tabSize: 2 },
-  })
-  await fse.writeFile(opencodeJsonPath, applyEdits(text, edits), 'utf-8')
-  if (buildModel) success(`default model -> ${buildModel} (written to opencode.jsonc)`)
-}
+import fse from 'fs-extra'
+import { applyEdits, modify, parse } from 'jsonc-parser'
+import path from 'path'
+import { success, warn } from '../../utils/exec.js'
+
+/**
+ * Writes the selected build model as the default model in opencode.jsonc.
+ * Per-tier models (plan/build/fast) are recorded in harness.json (models)
+ * by the metadata step, then consumed by the agent-variant generator and /plan-apply.
+ *
+ * opencode tolerates JSONC (comments, trailing commas) in its config, so this
+ * must not use readJson/writeJson: a strict parse crashes on valid configs and
+ * a strict rewrite deletes user comments. jsonc-parser edits only the `model`
+ * key and preserves the rest of the file byte-for-byte.
+ */
+export async function writeModelsToConfigs({ buildModel, keepExisting = false, cwd = process.cwd() }) {
+  const opencodeJsonPath = path.join(cwd, 'opencode.jsonc')
+  if (!await fse.pathExists(opencodeJsonPath)) return
+
+  const text = await fse.readFile(opencodeJsonPath, 'utf-8')
+  const errors = []
+  parse(text, errors)
+  if (errors.length > 0) {
+    warn('opencode.jsonc could not be parsed: leaving it untouched. Set "model" manually.')
+    return
+  }
+
+  // A model the repository already chose is its decision, not the wizard's: `update` used to
+  // reset a deliberately pinned default to the build tier in every consumer it ran in.
+  if (keepExisting && typeof parse(text)?.model === 'string') return
+
+  // `undefined` removes the key; a string value sets it.
+  const edits = modify(text, ['model'], buildModel || undefined, {
+    formattingOptions: { insertSpaces: true, tabSize: 2 },
+  })
+  await fse.writeFile(opencodeJsonPath, applyEdits(text, edits), 'utf-8')
+  if (buildModel) success(`default model -> ${buildModel} (written to opencode.jsonc)`)
+}