@plainconceptsplatform/agent-harness 3.5.0 → 3.7.0

package/cli/commands/shared.js

@@ -1,27 +1,27 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import { configPath } from '../utils/paths.js'
-
-export async function readHarnessConfig() {
-  const cfgPath = configPath()
-  if (!await fse.pathExists(cfgPath)) return null
-  try {
-    return await fse.readJson(cfgPath)
-  } catch {
-    return null
-  }
-}
-
-export async function ensureGitLongpaths(cwd = process.cwd()) {
-  // core.longpaths only matters on Windows; don't touch repo config elsewhere.
-  if (process.platform !== 'win32') return false
-  try {
-    const repoCheck = await execa('git', ['rev-parse', '--is-inside-work-tree'], { cwd, reject: false })
-    if (repoCheck.exitCode !== 0 || repoCheck.stdout.trim() !== 'true') return false
-
-    const configResult = await execa('git', ['config', 'core.longpaths', 'true'], { cwd, reject: false })
-    return configResult.exitCode === 0
-  } catch {
-    return false
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import { configPath } from '../utils/paths.js'
+
+export async function readHarnessConfig() {
+  const cfgPath = configPath()
+  if (!await fse.pathExists(cfgPath)) return null
+  try {
+    return await fse.readJson(cfgPath)
+  } catch {
+    return null
+  }
+}
+
+export async function ensureGitLongpaths(cwd = process.cwd()) {
+  // core.longpaths only matters on Windows; don't touch repo config elsewhere.
+  if (process.platform !== 'win32') return false
+  try {
+    const repoCheck = await execa('git', ['rev-parse', '--is-inside-work-tree'], { cwd, reject: false })
+    if (repoCheck.exitCode !== 0 || repoCheck.stdout.trim() !== 'true') return false
+
+    const configResult = await execa('git', ['config', 'core.longpaths', 'true'], { cwd, reject: false })
+    return configResult.exitCode === 0
+  } catch {
+    return false
+  }
+}

package/cli/commands/single.js

@@ -1,79 +1,79 @@
-import fse from 'fs-extra'
-import path from 'node:path'
-import { fileURLToPath } from 'url'
-import { cleanAiFiles } from '../steps/clean/index.js'
-import { copyContentStep } from '../steps/copy/index.js'
-import { chooseModels } from '../steps/models/index.js'
-import { initOpenspec } from '../steps/openspec/index.js'
-import { tokenOptimizationStep } from '../steps/optimization/index.js'
-import { choosePlatform } from '../steps/platform/index.js'
-import { installBrowser } from '../steps/browser/index.js'
-import { writeHarnessConfig } from '../steps/metadata/index.js'
-import { readHarnessConfig } from './shared.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
-// platforms.json is the single source of truth for valid platform values.
-// Hardcoding a subset here silently rewrote jira/gitlab/browser projects to github.
-const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
-
-export function resolvePlatform(value, fallback = 'github') {
-  return VALID_PLATFORMS.has(value) ? value : fallback
-}
-
-export async function runSingleCommand(command) {
-  const saved = await readHarnessConfig()
-  const ctx = {
-    hasDesign: !!saved?.preexisting?.design,
-    hasArchitecture: !!saved?.preexisting?.architecture,
-    hasOpenspec: !!saved?.preexisting?.openspec,
-    sourceMode: saved?.source?.mode ?? 'current',
-    sourceRoots: Array.isArray(saved?.source?.roots) ? saved.source.roots : [],
-    maxConcurrentAgents: saved?.agents?.maxConcurrent ?? 3,
-  }
-  const backlogPlatform = saved?.platform?.backlog
-  const repoPlatform = saved?.platform?.repo
-  const resolvedBacklog = resolvePlatform(backlogPlatform)
-  const resolvedRepo = resolvePlatform(repoPlatform)
-
-  const handlers = {
-    clean: async () => {
-      await cleanAiFiles()
-    },
-    platform: async () => {
-      await choosePlatform()
-    },
-    copy: async () => {
-      await copyContentStep({ backlogPlatform: resolvedBacklog, repoPlatform: resolvedRepo }, ctx)
-    },
-    openspec: async () => {
-      await initOpenspec()
-    },
-    models: async () => {
-      await chooseModels()
-    },
-    optimization: async () => {
-      await tokenOptimizationStep({ ctx })
-    },
-    browser: async () => {
-      await installBrowser()
-    },
-    metadata: async () => {
-      await writeHarnessConfig({
-        ...ctx,
-        backlogPlatform: resolvedBacklog,
-        repoPlatform: resolvedRepo,
-        maxConcurrentAgents: saved?.agents?.maxConcurrent ?? 3,
-        planModel: saved?.models?.plan ?? null,
-        buildModel: saved?.models?.build ?? null,
-        fastModel: saved?.models?.fast ?? null,
-        optionalTools: saved?.tools ?? null,
-      })
-    },
-  }
-
-  const handler = handlers[command]
-  if (!handler) return false
-  await handler()
-  return true
-}
+import fse from 'fs-extra'
+import path from 'node:path'
+import { fileURLToPath } from 'url'
+import { cleanAiFiles } from '../steps/clean/index.js'
+import { copyContentStep } from '../steps/copy/index.js'
+import { chooseModels } from '../steps/models/index.js'
+import { initOpenspec } from '../steps/openspec/index.js'
+import { tokenOptimizationStep } from '../steps/optimization/index.js'
+import { choosePlatform } from '../steps/platform/index.js'
+import { installBrowser } from '../steps/browser/index.js'
+import { writeHarnessConfig } from '../steps/metadata/index.js'
+import { readHarnessConfig } from './shared.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
+// platforms.json is the single source of truth for valid platform values.
+// Hardcoding a subset here silently rewrote jira/gitlab/browser projects to github.
+const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
+
+export function resolvePlatform(value, fallback = 'github') {
+  return VALID_PLATFORMS.has(value) ? value : fallback
+}
+
+export async function runSingleCommand(command) {
+  const saved = await readHarnessConfig()
+  const ctx = {
+    hasDesign: !!saved?.preexisting?.design,
+    hasArchitecture: !!saved?.preexisting?.architecture,
+    hasOpenspec: !!saved?.preexisting?.openspec,
+    sourceMode: saved?.source?.mode ?? 'current',
+    sourceRoots: Array.isArray(saved?.source?.roots) ? saved.source.roots : [],
+    maxConcurrentAgents: saved?.agents?.maxConcurrent ?? 3,
+  }
+  const backlogPlatform = saved?.platform?.backlog
+  const repoPlatform = saved?.platform?.repo
+  const resolvedBacklog = resolvePlatform(backlogPlatform)
+  const resolvedRepo = resolvePlatform(repoPlatform)
+
+  const handlers = {
+    clean: async () => {
+      await cleanAiFiles()
+    },
+    platform: async () => {
+      await choosePlatform()
+    },
+    copy: async () => {
+      await copyContentStep({ backlogPlatform: resolvedBacklog, repoPlatform: resolvedRepo }, ctx)
+    },
+    openspec: async () => {
+      await initOpenspec()
+    },
+    models: async () => {
+      await chooseModels()
+    },
+    optimization: async () => {
+      await tokenOptimizationStep({ ctx })
+    },
+    browser: async () => {
+      await installBrowser()
+    },
+    metadata: async () => {
+      await writeHarnessConfig({
+        ...ctx,
+        backlogPlatform: resolvedBacklog,
+        repoPlatform: resolvedRepo,
+        maxConcurrentAgents: saved?.agents?.maxConcurrent ?? 3,
+        planModel: saved?.models?.plan ?? null,
+        buildModel: saved?.models?.build ?? null,
+        fastModel: saved?.models?.fast ?? null,
+        optionalTools: saved?.tools ?? null,
+      })
+    },
+  }
+
+  const handler = handlers[command]
+  if (!handler) return false
+  await handler()
+  return true
+}

package/cli/commands/update.js

@@ -1,121 +1,121 @@
-import chalk from 'chalk'
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import path from 'node:path'
-import { fileURLToPath } from 'url'
-import { readHarnessConfig } from './shared.js'
-import { copyContentStep } from '../steps/copy/index.js'
-import { writeModelsToConfigs } from '../steps/models/write.js'
-import { patchGuardrails } from '../steps/optimization/patch-guardrails.js'
-import { reconcileTools } from '../steps/optimization/detect.js'
-import { writeHarnessConfig } from '../steps/metadata/index.js'
-import { info } from '../utils/exec.js'
-import { exit } from '../utils/process.js'
-
-const SIMPLE_ENGLISH_ENTRY = {
-  source: 'AminBlg/SimpleEnglish',
-  sourceType: 'github',
-  skillPath: 'skills/simple-english/SKILL.md',
-}
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
-const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
-
-function resolvePlatform(value, fallback = 'github') {
-  return VALID_PLATFORMS.has(value) ? value : fallback
-}
-
-export async function runUpdate() {
-  const saved = await readHarnessConfig()
-  if (!saved?.platform) {
-    console.log(chalk.red('No agent-harness config found. Run the wizard first.'))
-    exit(1)
-    return
-  }
-
-  const backlogPlatform = resolvePlatform(saved.platform.backlog)
-  const repoPlatform = resolvePlatform(saved.platform.repo)
-
-  const ctx = {
-    hasDesign: !!saved.preexisting?.design,
-    hasArchitecture: !!saved.preexisting?.architecture,
-    hasOpenspec: !!saved.preexisting?.openspec,
-    sourceMode: saved.source?.mode ?? 'current',
-    sourceRoots: Array.isArray(saved.source?.roots) ? saved.source.roots : [],
-    maxConcurrentAgents: saved.agents?.maxConcurrent ?? 3,
-    installScope: 'local',
-    skipSkills: false,
-    updateMode: true,
-  }
-
-  console.log()
-  console.log(chalk.bold('Updating project from saved config'))
-  console.log(chalk.dim(`  backlog: ${backlogPlatform}  repo: ${repoPlatform}  agents: ${ctx.maxConcurrentAgents}`))
-  console.log()
-
-  await copyContentStep({ backlogPlatform, repoPlatform }, ctx)
-
-  if (saved.models) {
-    await writeModelsToConfigs({ buildModel: saved.models.build, keepExisting: true })
-  }
-
-  // Reconcile against what is actually installed before patching guardrails.
-  // The saved flags drift: a project can be running the codegraph and
-  // agentmemory MCP servers with harness.json still recording them as false,
-  // and patchGuardrails would then empty those marker sections and strip the
-  // guidance for tools the project genuinely uses.
-  const tools = reconcileTools(saved.tools ?? {})
-  const gained = Object.keys(tools).filter(k => tools[k] && !saved.tools?.[k])
-  if (gained.length > 0) {
-    info(`Detected installed tools missing from harness.json: ${gained.join(', ')}`)
-  }
-
-  await migrateCavemanSkill(tools)
-  await patchGuardrails({
-    rtk: !!tools.rtk,
-    simpleEnglish: !!(tools.simpleEnglish || tools.caveman),
-    codegraph: !!tools.codegraph,
-    memory: !!tools.memory,
-    humanizer: !!tools.humanizer,
-  })
-
-  await writeHarnessConfig({
-    ...ctx,
-    backlogPlatform,
-    repoPlatform,
-    maxConcurrentAgents: ctx.maxConcurrentAgents,
-    planModel: saved.models?.plan ?? null,
-    buildModel: saved.models?.build ?? null,
-    fastModel: saved.models?.fast ?? null,
-    optionalTools: tools,
-  })
-
-  console.log()
-  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
-  console.log(chalk.bold.green('  Update complete!'))
-  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
-  console.log()
-}
-
-async function migrateCavemanSkill(tools) {
-  if (!tools.caveman || tools.simpleEnglish) return
-
-  const lockPath = path.join(process.cwd(), 'skills-lock.json')
-  const lock = await fse.readJson(lockPath).catch(() => ({ version: 1, skills: {} }))
-  lock.skills ??= {}
-  delete lock.skills.caveman
-  lock.skills['simple-english'] = SIMPLE_ENGLISH_ENTRY
-  await fse.writeJson(lockPath, lock, { spaces: 2 })
-  await fse.remove(path.join(process.cwd(), '.agents', 'skills', 'caveman'))
-
-  const result = await execa('npx', ['skills', 'experimental_install', '--yes'], {
-    cwd: process.cwd(),
-    reject: false,
-    stdio: 'pipe',
-    timeout: 600000,
-  })
-  if (result.exitCode !== 0) {
-    console.log(chalk.yellow('Simple English was added to skills-lock.json but could not be installed. Run npx skills experimental_install --yes.'))
-  }
-}
+import chalk from 'chalk'
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import path from 'node:path'
+import { fileURLToPath } from 'url'
+import { readHarnessConfig } from './shared.js'
+import { copyContentStep } from '../steps/copy/index.js'
+import { writeModelsToConfigs } from '../steps/models/write.js'
+import { patchGuardrails } from '../steps/optimization/patch-guardrails.js'
+import { reconcileTools } from '../steps/optimization/detect.js'
+import { writeHarnessConfig } from '../steps/metadata/index.js'
+import { info } from '../utils/exec.js'
+import { exit } from '../utils/process.js'
+
+const SIMPLE_ENGLISH_ENTRY = {
+  source: 'AminBlg/SimpleEnglish',
+  sourceType: 'github',
+  skillPath: 'skills/simple-english/SKILL.md',
+}
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
+const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
+
+function resolvePlatform(value, fallback = 'github') {
+  return VALID_PLATFORMS.has(value) ? value : fallback
+}
+
+export async function runUpdate() {
+  const saved = await readHarnessConfig()
+  if (!saved?.platform) {
+    console.log(chalk.red('No agent-harness config found. Run the wizard first.'))
+    exit(1)
+    return
+  }
+
+  const backlogPlatform = resolvePlatform(saved.platform.backlog)
+  const repoPlatform = resolvePlatform(saved.platform.repo)
+
+  const ctx = {
+    hasDesign: !!saved.preexisting?.design,
+    hasArchitecture: !!saved.preexisting?.architecture,
+    hasOpenspec: !!saved.preexisting?.openspec,
+    sourceMode: saved.source?.mode ?? 'current',
+    sourceRoots: Array.isArray(saved.source?.roots) ? saved.source.roots : [],
+    maxConcurrentAgents: saved.agents?.maxConcurrent ?? 3,
+    installScope: 'local',
+    skipSkills: false,
+    updateMode: true,
+  }
+
+  console.log()
+  console.log(chalk.bold('Updating project from saved config'))
+  console.log(chalk.dim(`  backlog: ${backlogPlatform}  repo: ${repoPlatform}  agents: ${ctx.maxConcurrentAgents}`))
+  console.log()
+
+  await copyContentStep({ backlogPlatform, repoPlatform }, ctx)
+
+  if (saved.models) {
+    await writeModelsToConfigs({ buildModel: saved.models.build, keepExisting: true })
+  }
+
+  // Reconcile against what is actually installed before patching guardrails.
+  // The saved flags drift: a project can be running the codegraph and
+  // agentmemory MCP servers with harness.json still recording them as false,
+  // and patchGuardrails would then empty those marker sections and strip the
+  // guidance for tools the project genuinely uses.
+  const tools = reconcileTools(saved.tools ?? {})
+  const gained = Object.keys(tools).filter(k => tools[k] && !saved.tools?.[k])
+  if (gained.length > 0) {
+    info(`Detected installed tools missing from harness.json: ${gained.join(', ')}`)
+  }
+
+  await migrateCavemanSkill(tools)
+  await patchGuardrails({
+    rtk: !!tools.rtk,
+    simpleEnglish: !!(tools.simpleEnglish || tools.caveman),
+    codegraph: !!tools.codegraph,
+    memory: !!tools.memory,
+    humanizer: !!tools.humanizer,
+  })
+
+  await writeHarnessConfig({
+    ...ctx,
+    backlogPlatform,
+    repoPlatform,
+    maxConcurrentAgents: ctx.maxConcurrentAgents,
+    planModel: saved.models?.plan ?? null,
+    buildModel: saved.models?.build ?? null,
+    fastModel: saved.models?.fast ?? null,
+    optionalTools: tools,
+  })
+
+  console.log()
+  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
+  console.log(chalk.bold.green('  Update complete!'))
+  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
+  console.log()
+}
+
+async function migrateCavemanSkill(tools) {
+  if (!tools.caveman || tools.simpleEnglish) return
+
+  const lockPath = path.join(process.cwd(), 'skills-lock.json')
+  const lock = await fse.readJson(lockPath).catch(() => ({ version: 1, skills: {} }))
+  lock.skills ??= {}
+  delete lock.skills.caveman
+  lock.skills['simple-english'] = SIMPLE_ENGLISH_ENTRY
+  await fse.writeJson(lockPath, lock, { spaces: 2 })
+  await fse.remove(path.join(process.cwd(), '.agents', 'skills', 'caveman'))
+
+  const result = await execa('npx', ['skills', 'experimental_install', '--yes'], {
+    cwd: process.cwd(),
+    reject: false,
+    stdio: 'pipe',
+    timeout: 600000,
+  })
+  if (result.exitCode !== 0) {
+    console.log(chalk.yellow('Simple English was added to skills-lock.json but could not be installed. Run npx skills experimental_install --yes.'))
+  }
+}