@plainconceptsplatform/agent-harness 3.4.1 → 3.5.0

package/cli/presets/browser.json

@@ -1,10 +1,10 @@
-{
-  "installer": {
-    "command": "npm",
-    "args": ["install", "-g", "agent-browser@0.36.0"]
-  },
-  "chromeSetup": {
-    "command": "agent-browser",
-    "args": ["install"]
-  }
-}
+{
+  "installer": {
+    "command": "npm",
+    "args": ["install", "-g", "agent-browser@0.36.0"]
+  },
+  "chromeSetup": {
+    "command": "agent-browser",
+    "args": ["install"]
+  }
+}

package/cli/steps/browser/index.js

@@ -1,48 +1,48 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import { header, info, success, warn, error } from '../../utils/exec.js'
-import os from 'os'
-import path from 'path'
-import { fileURLToPath } from 'url'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const BROWSER_PRESET_PATH = path.resolve(__dirname, '../../presets/browser.json')
-const browserPreset = await fse.readJson(BROWSER_PRESET_PATH)
-
-export async function installBrowser() {
-  header('Step 9, Installing agent-browser')
-
-  try {
-    const install = await execa(browserPreset.installer.command, browserPreset.installer.args, {
-      cwd: os.homedir(),
-      reject: false,
-    })
-
-    if (install.exitCode !== 0) {
-      warn('agent-browser npm install exited with non-zero code')
-      if (install.stderr) info(install.stderr)
-      return
-    }
-    success('agent-browser installed')
-  } catch (err) {
-    error(`Failed to install agent-browser: ${err.message}`)
-    return
-  }
-
-  try {
-    const chrome = await execa(browserPreset.chromeSetup.command, browserPreset.chromeSetup.args, {
-      cwd: os.homedir(),
-      reject: false,
-    })
-
-    if (chrome.exitCode === 0) {
-      success('agent-browser Chrome ready')
-      if (chrome.stdout) info(chrome.stdout.trim())
-    } else {
-      warn('agent-browser install (Chrome setup) exited with non-zero code')
-      if (chrome.stderr) info(chrome.stderr)
-    }
-  } catch (err) {
-    error(`Failed to set up Chrome for agent-browser: ${err.message}`)
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import { header, info, success, warn, error } from '../../utils/exec.js'
+import os from 'os'
+import path from 'path'
+import { fileURLToPath } from 'url'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const BROWSER_PRESET_PATH = path.resolve(__dirname, '../../presets/browser.json')
+const browserPreset = await fse.readJson(BROWSER_PRESET_PATH)
+
+export async function installBrowser() {
+  header('Step 9, Installing agent-browser')
+
+  try {
+    const install = await execa(browserPreset.installer.command, browserPreset.installer.args, {
+      cwd: os.homedir(),
+      reject: false,
+    })
+
+    if (install.exitCode !== 0) {
+      warn('agent-browser npm install exited with non-zero code')
+      if (install.stderr) info(install.stderr)
+      return
+    }
+    success('agent-browser installed')
+  } catch (err) {
+    error(`Failed to install agent-browser: ${err.message}`)
+    return
+  }
+
+  try {
+    const chrome = await execa(browserPreset.chromeSetup.command, browserPreset.chromeSetup.args, {
+      cwd: os.homedir(),
+      reject: false,
+    })
+
+    if (chrome.exitCode === 0) {
+      success('agent-browser Chrome ready')
+      if (chrome.stdout) info(chrome.stdout.trim())
+    } else {
+      warn('agent-browser install (Chrome setup) exited with non-zero code')
+      if (chrome.stderr) info(chrome.stderr)
+    }
+  } catch (err) {
+    error(`Failed to set up Chrome for agent-browser: ${err.message}`)
+  }
+}

package/cli/steps/copy/commands.js

@@ -1,79 +1,79 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { success } from '../../utils/exec.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-
-const _archive = {
-  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/az.md'), 'utf-8'),
-  github: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/gh.md'), 'utf-8'),
-  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/gl.md'), 'utf-8'),
-  none: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/none.md'), 'utf-8'),
-}
-
-const _ship = {
-  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/gh.md'), 'utf-8'),
-  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/az.md'), 'utf-8'),
-  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/gl.md'), 'utf-8'),
-}
-
-const _review = {
-  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/gh.md'), 'utf-8'),
-  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/az.md'), 'utf-8'),
-  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/gl.md'), 'utf-8'),
-}
-
-const _backlog = {
-  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/gh.md'), 'utf-8'),
-  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/az.md'), 'utf-8'),
-  jira: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/jira.md'), 'utf-8'),
-}
-
-// relativePath is POSIX-style relative to the project root; commands live in
-// .opencode/commands/, skill-backed procedures (plan-archive, ops-ship) in
-// .agents/skills/<name>/SKILL.md.
-function patchFile(relativePath, startMarker, endMarker, content, platform, cwd = process.cwd()) {
-  const targetPath = path.join(cwd, ...relativePath.split('/'))
-  if (!fse.pathExistsSync(targetPath)) return
-
-  let fileContent = fse.readFileSync(targetPath, 'utf-8')
-  if (!fileContent.includes(startMarker) || !fileContent.includes(endMarker)) return
-
-  const pattern = new RegExp(`${startMarker.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}[\\s\\S]*?${endMarker.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}`)
-  // Replacer function, not a replacement string: the injected fragments contain
-  // shell quoting like $'...' and a string replacement expands $' as "everything
-  // after the match", silently truncating the command. That is how the ops-ship
-  // screenshot comment lost its entire --body argument.
-  fileContent = fileContent.replace(pattern, () => `${startMarker}\n${content.trim()}\n${endMarker}`)
-  fse.writeFileSync(targetPath, `${fileContent.replace(/\s*$/, '')}\n`, 'utf-8')
-  success(`${relativePath} content injected for platform: ${platform}`)
-}
-
-export async function patchArchiveCommand(platform, cwd = process.cwd()) {
-  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? platform.backlogPlatform ?? 'github') : platform
-  const replacement = _archive[repoPlatform]
-  if (!replacement) return
-  patchFile('.agents/skills/pc-plan-archive/SKILL.md', '<!-- PC-PLATFORM-ARCHIVE-START -->', '<!-- PC-PLATFORM-ARCHIVE-END -->', replacement, repoPlatform, cwd)
-}
-
-export async function patchOpsShip(platform, cwd = process.cwd()) {
-  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? 'github') : platform
-  const replacement = _ship[repoPlatform]
-  if (!replacement) return
-  patchFile('.agents/skills/pc-ops-ship/SKILL.md', '<!-- PC-PLATFORM-SHIP-START -->', '<!-- PC-PLATFORM-SHIP-END -->', replacement, repoPlatform, cwd)
-}
-
-export async function patchOpsReview(platform, cwd = process.cwd()) {
-  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? 'github') : platform
-  const replacement = _review[repoPlatform]
-  if (!replacement) return
-  patchFile('.opencode/commands/ops-review.md', '<!-- PC-PLATFORM-REVIEW-START -->', '<!-- PC-PLATFORM-REVIEW-END -->', replacement, repoPlatform, cwd)
-}
-
-export async function patchOpsBacklog(platform, cwd = process.cwd()) {
-  const backlogPlatform = typeof platform === 'object' ? (platform.backlogPlatform ?? 'github') : platform
-  const replacement = _backlog[backlogPlatform]
-  if (!replacement) return
-  patchFile('.opencode/commands/ops-backlog.md', '<!-- PC-PLATFORM-BACKLOG-START -->', '<!-- PC-PLATFORM-BACKLOG-END -->', replacement, backlogPlatform, cwd)
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+import { success } from '../../utils/exec.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+
+const _archive = {
+  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/az.md'), 'utf-8'),
+  github: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/gh.md'), 'utf-8'),
+  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/gl.md'), 'utf-8'),
+  none: await fse.readFile(path.resolve(__dirname, '../../fragments/archive/none.md'), 'utf-8'),
+}
+
+const _ship = {
+  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/gh.md'), 'utf-8'),
+  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/az.md'), 'utf-8'),
+  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-ship/gl.md'), 'utf-8'),
+}
+
+const _review = {
+  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/gh.md'), 'utf-8'),
+  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/az.md'), 'utf-8'),
+  gitlab: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-review/gl.md'), 'utf-8'),
+}
+
+const _backlog = {
+  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/gh.md'), 'utf-8'),
+  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/az.md'), 'utf-8'),
+  jira: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-backlog/jira.md'), 'utf-8'),
+}
+
+// relativePath is POSIX-style relative to the project root; commands live in
+// .opencode/commands/, skill-backed procedures (plan-archive, ops-ship) in
+// .agents/skills/<name>/SKILL.md.
+function patchFile(relativePath, startMarker, endMarker, content, platform, cwd = process.cwd()) {
+  const targetPath = path.join(cwd, ...relativePath.split('/'))
+  if (!fse.pathExistsSync(targetPath)) return
+
+  let fileContent = fse.readFileSync(targetPath, 'utf-8')
+  if (!fileContent.includes(startMarker) || !fileContent.includes(endMarker)) return
+
+  const pattern = new RegExp(`${startMarker.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}[\\s\\S]*?${endMarker.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}`)
+  // Replacer function, not a replacement string: the injected fragments contain
+  // shell quoting like $'...' and a string replacement expands $' as "everything
+  // after the match", silently truncating the command. That is how the ops-ship
+  // screenshot comment lost its entire --body argument.
+  fileContent = fileContent.replace(pattern, () => `${startMarker}\n${content.trim()}\n${endMarker}`)
+  fse.writeFileSync(targetPath, `${fileContent.replace(/\s*$/, '')}\n`, 'utf-8')
+  success(`${relativePath} content injected for platform: ${platform}`)
+}
+
+export async function patchArchiveCommand(platform, cwd = process.cwd()) {
+  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? platform.backlogPlatform ?? 'github') : platform
+  const replacement = _archive[repoPlatform]
+  if (!replacement) return
+  patchFile('.agents/skills/pc-plan-archive/SKILL.md', '<!-- PC-PLATFORM-ARCHIVE-START -->', '<!-- PC-PLATFORM-ARCHIVE-END -->', replacement, repoPlatform, cwd)
+}
+
+export async function patchOpsShip(platform, cwd = process.cwd()) {
+  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? 'github') : platform
+  const replacement = _ship[repoPlatform]
+  if (!replacement) return
+  patchFile('.agents/skills/pc-ops-ship/SKILL.md', '<!-- PC-PLATFORM-SHIP-START -->', '<!-- PC-PLATFORM-SHIP-END -->', replacement, repoPlatform, cwd)
+}
+
+export async function patchOpsReview(platform, cwd = process.cwd()) {
+  const repoPlatform = typeof platform === 'object' ? (platform.repoPlatform ?? 'github') : platform
+  const replacement = _review[repoPlatform]
+  if (!replacement) return
+  patchFile('.opencode/commands/ops-review.md', '<!-- PC-PLATFORM-REVIEW-START -->', '<!-- PC-PLATFORM-REVIEW-END -->', replacement, repoPlatform, cwd)
+}
+
+export async function patchOpsBacklog(platform, cwd = process.cwd()) {
+  const backlogPlatform = typeof platform === 'object' ? (platform.backlogPlatform ?? 'github') : platform
+  const replacement = _backlog[backlogPlatform]
+  if (!replacement) return
+  patchFile('.opencode/commands/ops-backlog.md', '<!-- PC-PLATFORM-BACKLOG-START -->', '<!-- PC-PLATFORM-BACKLOG-END -->', replacement, backlogPlatform, cwd)
+}

package/cli/steps/copy/index.js

@@ -1,88 +1,88 @@
-import fse from "fs-extra"
-import path from "path"
-import { fileURLToPath } from "url"
-import { copyContent, recordManagedContent } from "../../utils/copy.js"
-import { error, header, success } from "../../utils/exec.js"
-import { exit } from "../../utils/process.js"
-import { patchAgentGuidance, patchAgentsMd } from "./agents.js"
-import { patchArchiveCommand, patchOpsShip, patchOpsReview, patchOpsBacklog } from "./commands.js"
-import { installSkills, removeRetiredFiles } from "./skills.js"
-import { generateFullstackEngineer, removeLegacyStartupDirectives } from "./fullstack-engineer.js"
-import { patchOpencodeJson, patchOpencodePackage } from "./opencode-json.js"
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const CONTENT_DIR = path.resolve(__dirname, "../../../harness")
-
-export async function copyContentStep(platform, ctx = {}) {
-  header("Step 5, Copying agent-harness files")
-
-  const dest = process.cwd()
-
-  // Support both old single-string platform and new { backlogPlatform, repoPlatform }
-  const backlogPlatform = typeof platform === 'object' ? platform.backlogPlatform : platform
-  const repoPlatform = typeof platform === 'object' ? platform.repoPlatform : platform
-
-  try {
-    await copyContent(CONTENT_DIR, dest, platform, ctx)
-
-    // .gitignore is stripped by npm during publish, so the source file is _gitignore.
-    // Also, fse.copy with overwrite:false won't overwrite an existing .gitignore,
-    // so merge manually to preserve both opencode's defaults and our additions.
-    const srcGitignore = path.join(CONTENT_DIR, ".opencode", "_gitignore")
-    const destGitignore = path.join(dest, ".opencode", ".gitignore")
-    if (await fse.pathExists(srcGitignore)) {
-      const srcLines = (await fse.readFile(srcGitignore, "utf-8")).split("\n").map(l => l.trim()).filter(Boolean)
-      const destLines = (await fse.pathExists(destGitignore))
-        ? (await fse.readFile(destGitignore, "utf-8")).split("\n").map(l => l.trim()).filter(Boolean)
-        : []
-      const merged = Array.from(new Set([...destLines, ...srcLines]))
-      await fse.writeFile(destGitignore, `${merged.join("\n")}\n`, "utf-8")
-    }
-
-    // Remove the _gitignore template from dest: it was only needed for the merge above.
-    const destGitignoreTemplate = path.join(dest, ".opencode", "_gitignore")
-    if (await fse.pathExists(destGitignoreTemplate)) {
-      await fse.remove(destGitignoreTemplate)
-    }
-
-    const rootsFile = path.join(dest, ".opencode", "source-roots.json")
-    if (!ctx.updateMode || !await fse.pathExists(rootsFile)) {
-      await fse.writeJson(
-        rootsFile,
-        {
-          mode: ctx.sourceMode || "current",
-          // An empty array is truthy, so `|| [dest]` never kicked in and reruns
-          // wrote `roots: []`, leaving agents with no source scope.
-          roots: ctx.sourceRoots?.length ? ctx.sourceRoots : [dest],
-        },
-        { spaces: 2 },
-      )
-    }
-
-    await patchAgentGuidance(backlogPlatform, repoPlatform)
-    await patchOpsReview({ backlogPlatform, repoPlatform })
-    await patchOpsBacklog({ backlogPlatform, repoPlatform })
-    await patchOpencodeJson()
-    await patchOpencodePackage()
-    if (!ctx.updateMode) await patchAgentsMd(ctx)
-
-    if (!ctx.skipSkills) {
-      await installSkills(backlogPlatform, repoPlatform, {
-        forceOverwrite: ctx.forceOverwrite,
-        updateMode: ctx.updateMode,
-      })
-    }
-    // These patch SKILL.md files (pc-plan-archive, pc-ops-ship),
-    // so they must run after installSkills has copied the skills into the project.
-    await patchArchiveCommand({ backlogPlatform, repoPlatform })
-    await patchOpsShip({ backlogPlatform, repoPlatform })
-    await removeRetiredFiles()
-    await generateFullstackEngineer({ updateMode: ctx.updateMode })
-    await removeLegacyStartupDirectives()
-    await recordManagedContent(CONTENT_DIR, dest, { updateMode: ctx.updateMode })
-    success("Files copied to project root")
-  } catch (err) {
-    error(`Failed to copy content: ${err.message}`)
-    exit(1)
-  }
-}
+import fse from "fs-extra"
+import path from "path"
+import { fileURLToPath } from "url"
+import { copyContent, recordManagedContent } from "../../utils/copy.js"
+import { error, header, success } from "../../utils/exec.js"
+import { exit } from "../../utils/process.js"
+import { patchAgentGuidance, patchAgentsMd } from "./agents.js"
+import { patchArchiveCommand, patchOpsShip, patchOpsReview, patchOpsBacklog } from "./commands.js"
+import { installSkills, removeRetiredFiles } from "./skills.js"
+import { generateFullstackEngineer, removeLegacyStartupDirectives } from "./fullstack-engineer.js"
+import { patchOpencodeJson, patchOpencodePackage } from "./opencode-json.js"
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const CONTENT_DIR = path.resolve(__dirname, "../../../harness")
+
+export async function copyContentStep(platform, ctx = {}) {
+  header("Step 5, Copying agent-harness files")
+
+  const dest = process.cwd()
+
+  // Support both old single-string platform and new { backlogPlatform, repoPlatform }
+  const backlogPlatform = typeof platform === 'object' ? platform.backlogPlatform : platform
+  const repoPlatform = typeof platform === 'object' ? platform.repoPlatform : platform
+
+  try {
+    await copyContent(CONTENT_DIR, dest, platform, ctx)
+
+    // .gitignore is stripped by npm during publish, so the source file is _gitignore.
+    // Also, fse.copy with overwrite:false won't overwrite an existing .gitignore,
+    // so merge manually to preserve both opencode's defaults and our additions.
+    const srcGitignore = path.join(CONTENT_DIR, ".opencode", "_gitignore")
+    const destGitignore = path.join(dest, ".opencode", ".gitignore")
+    if (await fse.pathExists(srcGitignore)) {
+      const srcLines = (await fse.readFile(srcGitignore, "utf-8")).split("\n").map(l => l.trim()).filter(Boolean)
+      const destLines = (await fse.pathExists(destGitignore))
+        ? (await fse.readFile(destGitignore, "utf-8")).split("\n").map(l => l.trim()).filter(Boolean)
+        : []
+      const merged = Array.from(new Set([...destLines, ...srcLines]))
+      await fse.writeFile(destGitignore, `${merged.join("\n")}\n`, "utf-8")
+    }
+
+    // Remove the _gitignore template from dest: it was only needed for the merge above.
+    const destGitignoreTemplate = path.join(dest, ".opencode", "_gitignore")
+    if (await fse.pathExists(destGitignoreTemplate)) {
+      await fse.remove(destGitignoreTemplate)
+    }
+
+    const rootsFile = path.join(dest, ".opencode", "source-roots.json")
+    if (!ctx.updateMode || !await fse.pathExists(rootsFile)) {
+      await fse.writeJson(
+        rootsFile,
+        {
+          mode: ctx.sourceMode || "current",
+          // An empty array is truthy, so `|| [dest]` never kicked in and reruns
+          // wrote `roots: []`, leaving agents with no source scope.
+          roots: ctx.sourceRoots?.length ? ctx.sourceRoots : [dest],
+        },
+        { spaces: 2 },
+      )
+    }
+
+    await patchAgentGuidance(backlogPlatform, repoPlatform)
+    await patchOpsReview({ backlogPlatform, repoPlatform })
+    await patchOpsBacklog({ backlogPlatform, repoPlatform })
+    await patchOpencodeJson()
+    await patchOpencodePackage()
+    if (!ctx.updateMode) await patchAgentsMd(ctx)
+
+    if (!ctx.skipSkills) {
+      await installSkills(backlogPlatform, repoPlatform, {
+        forceOverwrite: ctx.forceOverwrite,
+        updateMode: ctx.updateMode,
+      })
+    }
+    // These patch SKILL.md files (pc-plan-archive, pc-ops-ship),
+    // so they must run after installSkills has copied the skills into the project.
+    await patchArchiveCommand({ backlogPlatform, repoPlatform })
+    await patchOpsShip({ backlogPlatform, repoPlatform })
+    await removeRetiredFiles()
+    await generateFullstackEngineer({ updateMode: ctx.updateMode })
+    await removeLegacyStartupDirectives()
+    await recordManagedContent(CONTENT_DIR, dest, { updateMode: ctx.updateMode })
+    success("Files copied to project root")
+  } catch (err) {
+    error(`Failed to copy content: ${err.message}`)
+    exit(1)
+  }
+}