@plainconceptsplatform/agent-harness 2.0.0 → 2.1.0

package/src/steps/models/write.js

@@ -1,34 +1,34 @@
-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, 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
-  }
-
-  // `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, 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
+  }
+
+  // `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)`)
+}

package/src/steps/optimization/patch-guardrails.js

@@ -1,108 +1,108 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { info, success } from '../../utils/exec.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const GUARDRAILS_FRAGMENT_DIR = path.resolve(__dirname, '../../fragments/guardrails')
-
-const MARKER_SECTIONS = {
-  rtk: 'RTK',
-  codegraph: 'CODEGRAPH',
-  memory: 'MEMORY',
-  simpleEnglish: 'SIMPLE-ENGLISH',
-  humanizer: 'HUMANIZER',
-}
-
-const OPTIONAL_TOOL_GUIDANCE = {
-  codegraph: 'Use `codegraph_explore` to refine relevant symbols and file-disjointness for the current wave.',
-  memory: 'Use Agentmemory MCP tools for cross-session context and task-result notes.',
-}
-
-function replaceMarkerSection(content, startMarker, endMarker, replacement) {
-  if (!content.includes(startMarker) || !content.includes(endMarker)) return content
-  const escape = value => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-  const pattern = new RegExp(`${escape(startMarker)}[\\s\\S]*?${escape(endMarker)}`)
-  return content.replace(pattern, `${startMarker}\n${replacement}\n${endMarker}`)
-}
-
-async function patchOptionalToolGuidance(destSkillsDir, selections) {
-  const files = []
-
-  async function collect(dir) {
-    for (const entry of await fse.readdir(dir, { withFileTypes: true })) {
-      const entryPath = path.join(dir, entry.name)
-      if (entry.isDirectory()) await collect(entryPath)
-      else if (entry.name.endsWith('.md')) files.push(entryPath)
-    }
-  }
-
-  if (!await fse.pathExists(destSkillsDir)) return 0
-  await collect(destSkillsDir)
-
-  let patched = 0
-  for (const filePath of files) {
-    let content = await fse.readFile(filePath, 'utf-8')
-    const original = content
-    for (const [key, guidance] of Object.entries(OPTIONAL_TOOL_GUIDANCE)) {
-      const suffix = key.toUpperCase()
-      content = replaceMarkerSection(
-        content,
-        `<!-- PC-OPTIMIZATION-${suffix}-START -->`,
-        `<!-- PC-OPTIMIZATION-${suffix}-END -->`,
-        selections[key] ? guidance : '',
-      )
-    }
-    if (content !== original) {
-      await fse.writeFile(filePath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
-      patched++
-    }
-  }
-  return patched
-}
-
-export async function patchGuardrails(selections = {}, { cwd = process.cwd() } = {}) {
-  const destSkillsDir = path.join(cwd, '.agents', 'skills')
-  const guardrailsPath = path.join(destSkillsDir, 'pc-guardrails-generic', 'SKILL.md')
-  let patchedCount = 0
-
-  if (await fse.pathExists(guardrailsPath)) {
-    let content = await fse.readFile(guardrailsPath, 'utf-8')
-    content = replaceMarkerSection(
-      content,
-      '<!-- PC-GUARDRAILS-CAVEMAN-START -->',
-      '<!-- PC-GUARDRAILS-CAVEMAN-END -->',
-      '',
-    ).replace(
-      '<!-- PC-GUARDRAILS-CAVEMAN-START -->\n\n<!-- PC-GUARDRAILS-CAVEMAN-END -->',
-      '<!-- PC-GUARDRAILS-SIMPLE-ENGLISH-START -->\n<!-- PC-GUARDRAILS-SIMPLE-ENGLISH-END -->',
-    ).replace(
-      '1. Load ALL skills listed under your own `## Abilities` now (Guardrails first, then the rest), by calling the `skill` tool once per `@skill-name`.',
-      '1. The `pc-system-reminders` plugin has already loaded the skills listed under your `## Abilities`, guardrails first.',
-    )
-    for (const [key, markerSuffix] of Object.entries(MARKER_SECTIONS)) {
-      let sectionContent = ''
-      if (selections[key]) {
-        const presetName = key === 'simpleEnglish' ? 'simple-english' : key
-        const presetPath = path.join(GUARDRAILS_FRAGMENT_DIR, `${presetName}.md`)
-        if (await fse.pathExists(presetPath)) {
-          sectionContent = (await fse.readFile(presetPath, 'utf-8')).trim()
-        }
-      }
-      content = replaceMarkerSection(
-        content,
-        `<!-- PC-GUARDRAILS-${markerSuffix}-START -->`,
-        `<!-- PC-GUARDRAILS-${markerSuffix}-END -->`,
-        sectionContent,
-      )
-      if (selections[key]) patchedCount++
-    }
-    await fse.writeFile(guardrailsPath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
-  } else {
-    info('pc-guardrails-generic not found, skipping guardrails patching')
-  }
-
-  const optionalPatched = await patchOptionalToolGuidance(destSkillsDir, selections)
-  success(`Guardrails patched: ${patchedCount} sections injected; ${optionalPatched} optional tool file(s) updated`)
-  return { patched: true, count: patchedCount, optionalPatched }
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+import { info, success } from '../../utils/exec.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const GUARDRAILS_FRAGMENT_DIR = path.resolve(__dirname, '../../fragments/guardrails')
+
+const MARKER_SECTIONS = {
+  rtk: 'RTK',
+  codegraph: 'CODEGRAPH',
+  memory: 'MEMORY',
+  simpleEnglish: 'SIMPLE-ENGLISH',
+  humanizer: 'HUMANIZER',
+}
+
+const OPTIONAL_TOOL_GUIDANCE = {
+  codegraph: 'Use `codegraph_explore` to refine relevant symbols and file-disjointness for the current wave.',
+  memory: 'Use Agentmemory MCP tools for cross-session context and task-result notes.',
+}
+
+function replaceMarkerSection(content, startMarker, endMarker, replacement) {
+  if (!content.includes(startMarker) || !content.includes(endMarker)) return content
+  const escape = value => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+  const pattern = new RegExp(`${escape(startMarker)}[\\s\\S]*?${escape(endMarker)}`)
+  return content.replace(pattern, `${startMarker}\n${replacement}\n${endMarker}`)
+}
+
+async function patchOptionalToolGuidance(destSkillsDir, selections) {
+  const files = []
+
+  async function collect(dir) {
+    for (const entry of await fse.readdir(dir, { withFileTypes: true })) {
+      const entryPath = path.join(dir, entry.name)
+      if (entry.isDirectory()) await collect(entryPath)
+      else if (entry.name.endsWith('.md')) files.push(entryPath)
+    }
+  }
+
+  if (!await fse.pathExists(destSkillsDir)) return 0
+  await collect(destSkillsDir)
+
+  let patched = 0
+  for (const filePath of files) {
+    let content = await fse.readFile(filePath, 'utf-8')
+    const original = content
+    for (const [key, guidance] of Object.entries(OPTIONAL_TOOL_GUIDANCE)) {
+      const suffix = key.toUpperCase()
+      content = replaceMarkerSection(
+        content,
+        `<!-- PC-OPTIMIZATION-${suffix}-START -->`,
+        `<!-- PC-OPTIMIZATION-${suffix}-END -->`,
+        selections[key] ? guidance : '',
+      )
+    }
+    if (content !== original) {
+      await fse.writeFile(filePath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
+      patched++
+    }
+  }
+  return patched
+}
+
+export async function patchGuardrails(selections = {}, { cwd = process.cwd() } = {}) {
+  const destSkillsDir = path.join(cwd, '.agents', 'skills')
+  const guardrailsPath = path.join(destSkillsDir, 'pc-guardrails-generic', 'SKILL.md')
+  let patchedCount = 0
+
+  if (await fse.pathExists(guardrailsPath)) {
+    let content = await fse.readFile(guardrailsPath, 'utf-8')
+    content = replaceMarkerSection(
+      content,
+      '<!-- PC-GUARDRAILS-CAVEMAN-START -->',
+      '<!-- PC-GUARDRAILS-CAVEMAN-END -->',
+      '',
+    ).replace(
+      '<!-- PC-GUARDRAILS-CAVEMAN-START -->\n\n<!-- PC-GUARDRAILS-CAVEMAN-END -->',
+      '<!-- PC-GUARDRAILS-SIMPLE-ENGLISH-START -->\n<!-- PC-GUARDRAILS-SIMPLE-ENGLISH-END -->',
+    ).replace(
+      '1. Load ALL skills listed under your own `## Abilities` now (Guardrails first, then the rest), by calling the `skill` tool once per `@skill-name`.',
+      '1. The `pc-system-reminders` plugin has already loaded the skills listed under your `## Abilities`, guardrails first.',
+    )
+    for (const [key, markerSuffix] of Object.entries(MARKER_SECTIONS)) {
+      let sectionContent = ''
+      if (selections[key]) {
+        const presetName = key === 'simpleEnglish' ? 'simple-english' : key
+        const presetPath = path.join(GUARDRAILS_FRAGMENT_DIR, `${presetName}.md`)
+        if (await fse.pathExists(presetPath)) {
+          sectionContent = (await fse.readFile(presetPath, 'utf-8')).trim()
+        }
+      }
+      content = replaceMarkerSection(
+        content,
+        `<!-- PC-GUARDRAILS-${markerSuffix}-START -->`,
+        `<!-- PC-GUARDRAILS-${markerSuffix}-END -->`,
+        sectionContent,
+      )
+      if (selections[key]) patchedCount++
+    }
+    await fse.writeFile(guardrailsPath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
+  } else {
+    info('pc-guardrails-generic not found, skipping guardrails patching')
+  }
+
+  const optionalPatched = await patchOptionalToolGuidance(destSkillsDir, selections)
+  success(`Guardrails patched: ${patchedCount} sections injected; ${optionalPatched} optional tool file(s) updated`)
+  return { patched: true, count: patchedCount, optionalPatched }
+}

package/src/utils/copy.js

@@ -1,108 +1,108 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { canUpdateManagedFile, hashFile, readUpdateManifest, recordManagedFile, writeUpdateManifest } from './update-manifest.js'
-
-// Folders never copied (skills handled separately by installSkills, .bootstrap is internal tooling)
-const ALWAYS_EXCLUDE = ['.bootstrap', 'skills', 'node_modules']
-
-// Files never overwritten even with forceOverwrite: user owns these.
-// The CLI ships templates, but init / the wizard / the user populate them
-// with project-specific content. Overwriting on update would destroy user work.
-const NEVER_OVERWRITE = [
-  `openspec${path.sep}config.yaml`,
-  'opencode.jsonc',
-]
-
-const MARKER_COMMANDS = new Set(['.opencode/commands/ops-review.md', '.opencode/commands/ops-backlog.md'])
-
-function isTemplateTest(relativePath) {
-  return relativePath.split(path.sep).join('/').startsWith('.opencode/plugins/') && relativePath.endsWith('.test.js')
-}
-
-export function isManagedContentPath(relativePath) {
-  const normalized = relativePath.split(path.sep).join('/')
-  if (normalized.startsWith('.opencode/plugins/')) return true
-  if (normalized === '.opencode/tui/pc-subagents.tsx') return true
-  if (normalized.startsWith('.opencode/commands/') && !MARKER_COMMANDS.has(normalized)) return true
-  return normalized === 'openspec/specs/.gitkeep' || normalized === 'openspec/changes/archive/.gitkeep'
-}
-
-/**
- * Copy content/ directory to destination.
- * Excludes:
- *   - .agents/skills and .opencode/skills (handled separately)
- *   - .bootstrap (internal tooling)
- *   - node_modules
- *   - DESIGN.md and ARCHITECTURE.md if ctx says they already exist (preserve user's files)
- * @param {string} contentDir - absolute path to content/
- * @param {string} destDir - absolute path to destination (project root)
- * @param {'azure'|'github'} platform
- * @param {{ hasDesign?: boolean, hasArchitecture?: boolean, updateMode?: boolean }} ctx
- */
-export async function copyContent(contentDir, destDir, platform, ctx = {}) {
-  const manifest = ctx.updateMode ? await readUpdateManifest(destDir) : null
-  await fse.copy(contentDir, destDir, {
-    overwrite: ctx.updateMode || ctx.forceOverwrite || false,
-    filter: async (src) => {
-      const rel = path.relative(contentDir, src)
-      const parts = rel.split(path.sep)
-      if (parts.some(part => ALWAYS_EXCLUDE.includes(part))) return false
-      if (isTemplateTest(rel)) return false
-      if (ctx.hasDesign && rel === 'DESIGN.md') return false
-      if (ctx.hasArchitecture && rel === 'ARCHITECTURE.md') return false
-      // User-owned config files are never overwritten, even with forceOverwrite.
-      // The update command calls writeModelsToConfigs separately to set the
-      // model field in opencode.jsonc without destroying user additions.
-      if (NEVER_OVERWRITE.includes(rel)) return false
-      if (!ctx.updateMode) return true
-
-      const stat = await fse.stat(src)
-      if (stat.isDirectory()) return true
-      if (!isManagedContentPath(rel)) {
-        return !(await fse.pathExists(path.join(destDir, rel)))
-      }
-      return canUpdateManagedFile(rel, destDir, manifest)
-    },
-  })
-}
-
-export async function recordManagedContent(contentDir, destDir, { updateMode = false } = {}) {
-  const manifest = await readUpdateManifest(destDir)
-
-  async function walk(directory) {
-    for (const entry of await fse.readdir(directory, { withFileTypes: true })) {
-      const sourcePath = path.join(directory, entry.name)
-      const relativePath = path.relative(contentDir, sourcePath)
-      if (entry.isDirectory()) {
-        await walk(sourcePath)
-        continue
-      }
-      if (isTemplateTest(relativePath)) continue
-      if (!isManagedContentPath(relativePath)) continue
-
-      const destinationPath = path.join(destDir, relativePath)
-      if (!await fse.pathExists(destinationPath)) continue
-      const sourceHash = await hashFile(sourcePath)
-      const destinationHash = await hashFile(destinationPath)
-      const manifestPath = relativePath.split(path.sep).join('/')
-      const previousHash = manifest.files?.[manifestPath]
-      if (!updateMode || destinationHash === sourceHash || destinationHash === previousHash) {
-        await recordManagedFile(manifest, relativePath, sourcePath)
-      }
-    }
-  }
-
-  await walk(contentDir)
-  await writeUpdateManifest(manifest, destDir)
-}
-
-export async function findAiFiles(dir, files) {
-  const found = []
-  for (const file of files) {
-    const fullPath = path.join(dir, file)
-    if (await fse.pathExists(fullPath)) {
-      found.push(fullPath)
-    }
-  }
-  return found
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { canUpdateManagedFile, hashFile, readUpdateManifest, recordManagedFile, writeUpdateManifest } from './update-manifest.js'
+
+// Folders never copied (skills handled separately by installSkills, .bootstrap is internal tooling)
+const ALWAYS_EXCLUDE = ['.bootstrap', 'skills', 'node_modules']
+
+// Files never overwritten even with forceOverwrite: user owns these.
+// The CLI ships templates, but init / the wizard / the user populate them
+// with project-specific content. Overwriting on update would destroy user work.
+const NEVER_OVERWRITE = [
+  `openspec${path.sep}config.yaml`,
+  'opencode.jsonc',
+]
+
+const MARKER_COMMANDS = new Set(['.opencode/commands/ops-review.md', '.opencode/commands/ops-backlog.md'])
+
+function isTemplateTest(relativePath) {
+  return relativePath.split(path.sep).join('/').startsWith('.opencode/plugins/') && relativePath.endsWith('.test.js')
+}
+
+export function isManagedContentPath(relativePath) {
+  const normalized = relativePath.split(path.sep).join('/')
+  if (normalized.startsWith('.opencode/plugins/')) return true
+  if (normalized === '.opencode/tui/pc-subagents.tsx') return true
+  if (normalized.startsWith('.opencode/commands/') && !MARKER_COMMANDS.has(normalized)) return true
+  return normalized === 'openspec/specs/.gitkeep' || normalized === 'openspec/changes/archive/.gitkeep'
+}
+
+/**
+ * Copy content/ directory to destination.
+ * Excludes:
+ *   - .agents/skills and .opencode/skills (handled separately)
+ *   - .bootstrap (internal tooling)
+ *   - node_modules
+ *   - DESIGN.md and ARCHITECTURE.md if ctx says they already exist (preserve user's files)
+ * @param {string} contentDir - absolute path to content/
+ * @param {string} destDir - absolute path to destination (project root)
+ * @param {'azure'|'github'} platform
+ * @param {{ hasDesign?: boolean, hasArchitecture?: boolean, updateMode?: boolean }} ctx
+ */
+export async function copyContent(contentDir, destDir, platform, ctx = {}) {
+  const manifest = ctx.updateMode ? await readUpdateManifest(destDir) : null
+  await fse.copy(contentDir, destDir, {
+    overwrite: ctx.updateMode || ctx.forceOverwrite || false,
+    filter: async (src) => {
+      const rel = path.relative(contentDir, src)
+      const parts = rel.split(path.sep)
+      if (parts.some(part => ALWAYS_EXCLUDE.includes(part))) return false
+      if (isTemplateTest(rel)) return false
+      if (ctx.hasDesign && rel === 'DESIGN.md') return false
+      if (ctx.hasArchitecture && rel === 'ARCHITECTURE.md') return false
+      // User-owned config files are never overwritten, even with forceOverwrite.
+      // The update command calls writeModelsToConfigs separately to set the
+      // model field in opencode.jsonc without destroying user additions.
+      if (NEVER_OVERWRITE.includes(rel)) return false
+      if (!ctx.updateMode) return true
+
+      const stat = await fse.stat(src)
+      if (stat.isDirectory()) return true
+      if (!isManagedContentPath(rel)) {
+        return !(await fse.pathExists(path.join(destDir, rel)))
+      }
+      return canUpdateManagedFile(rel, destDir, manifest)
+    },
+  })
+}
+
+export async function recordManagedContent(contentDir, destDir, { updateMode = false } = {}) {
+  const manifest = await readUpdateManifest(destDir)
+
+  async function walk(directory) {
+    for (const entry of await fse.readdir(directory, { withFileTypes: true })) {
+      const sourcePath = path.join(directory, entry.name)
+      const relativePath = path.relative(contentDir, sourcePath)
+      if (entry.isDirectory()) {
+        await walk(sourcePath)
+        continue
+      }
+      if (isTemplateTest(relativePath)) continue
+      if (!isManagedContentPath(relativePath)) continue
+
+      const destinationPath = path.join(destDir, relativePath)
+      if (!await fse.pathExists(destinationPath)) continue
+      const sourceHash = await hashFile(sourcePath)
+      const destinationHash = await hashFile(destinationPath)
+      const manifestPath = relativePath.split(path.sep).join('/')
+      const previousHash = manifest.files?.[manifestPath]
+      if (!updateMode || destinationHash === sourceHash || destinationHash === previousHash) {
+        await recordManagedFile(manifest, relativePath, sourcePath)
+      }
+    }
+  }
+
+  await walk(contentDir)
+  await writeUpdateManifest(manifest, destDir)
+}
+
+export async function findAiFiles(dir, files) {
+  const found = []
+  for (const file of files) {
+    const fullPath = path.join(dir, file)
+    if (await fse.pathExists(fullPath)) {
+      found.push(fullPath)
+    }
+  }
+  return found
+}

package/src/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 }
+}