@plainconceptsplatform/agent-harness 3.2.0 → 3.3.1

package/cli/steps/copy/fullstack-engineer.js

@@ -1,89 +1,89 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { success } from '../../utils/exec.js'
-import { agentColor, yamlColor } from '../../utils/agent-color.js'
-
-const FULLSTACK_FILE = 'fullstack-engineer.md'
-const FULLSTACK_DESCRIPTION = 'Default engineer that accumulates skills from all created persona engineers. Use as fallback when no specialist matches: but prefer spawning a specific engineer for deterministic results.'
-const FULLSTACK_IDENTITY = 'You are the default engineer, and your body is what the build and plan agents run. You are more complete but less accurate than specialized engineers, so prefer spawning a specialist when one matches the task domain.'
-
-// The reminder plugin owns ability bootstrap. Remove the old prompt-level
-// directive when regenerating so it cannot duplicate the plugin reminder.
-function stripStartupDirective(text) {
-  if (!text) return text
-  return text
-    .split(/\r?\n\r?\n+/)
-    .filter(paragraph => !paragraph.trim().startsWith('**Startup'))
-    .join('\n\n')
-    .trim()
-}
-
-export async function removeLegacyStartupDirectives({ cwd = process.cwd() } = {}) {
-  const agentsDir = path.join(cwd, '.opencode', 'agents')
-  if (!await fse.pathExists(agentsDir)) return 0
-
-  let updated = 0
-  for (const fileName of await fse.readdir(agentsDir)) {
-    if (!fileName.endsWith('.md')) continue
-    const filePath = path.join(agentsDir, fileName)
-    const content = await fse.readFile(filePath, 'utf-8')
-    const cleaned = stripStartupDirective(content)
-    if (cleaned === content.trim()) continue
-    await fse.writeFile(filePath, `${cleaned}\n`, 'utf-8')
-    updated++
-  }
-  return updated
-}
-
-export async function generateFullstackEngineer({ cwd = process.cwd(), updateMode = false } = {}) {
-  const agentsDir = path.join(cwd, '.opencode', 'agents')
-  await fse.ensureDir(agentsDir)
-  const filePath = path.join(agentsDir, FULLSTACK_FILE)
-
-  if (updateMode && await fse.pathExists(filePath)) {
-    return { generated: false, preserved: true }
-  }
-
-  let existingAbilities = null
-  let existingIdentity = null
-  if (await fse.pathExists(filePath)) {
-    const content = await fse.readFile(filePath, 'utf-8')
-    const abilitiesMatch = content.match(/## Abilities\n([\s\S]*?)$/)
-    if (abilitiesMatch) existingAbilities = abilitiesMatch[1].trim()
-    const identityMatch = content.match(/^---\n[\s\S]*?\n---\n\n(.*?)(?:\n\n## Abilities)/s)
-    if (identityMatch) existingIdentity = stripStartupDirective(identityMatch[1])
-  }
-
-  const frontmatter = [
-    '---',
-    `description: ${FULLSTACK_DESCRIPTION}`,
-    // subagent, not primary: build.md and plan.md are the primaries, and
-    // pc-subagent-tiers generates both from this file on every startup.
-    'mode: subagent',
-    // warning is reserved for plan, so fullstack takes its name-derived hex
-    // like every other subagent.
-    // Quoted: a bare #hex is a YAML comment, which opencode rejects.
-    `color: ${yamlColor(agentColor('fullstack-engineer'))}`,
-    'permission:',
-    '  edit: allow',
-    '  bash: allow',
-    '  read: allow',
-    '  glob: allow',
-    '  grep: allow',
-    '  question: allow',
-    '  todowrite: allow',
-    '---',
-  ]
-
-  const identity = existingIdentity ?? FULLSTACK_IDENTITY
-
-  const abilities = existingAbilities ?? [
-    '- Guardrails: @pc-guardrails-generic, @pc-guardrails-project',
-  ].join('\n')
-
-  const content = `${frontmatter.join('\n')}\n\n${identity}\n\n## Abilities\n${abilities}\n`
-
-  await fse.writeFile(filePath, content, 'utf-8')
-  success(`Generated ${FULLSTACK_FILE}`)
-  return { generated: true }
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { success } from '../../utils/exec.js'
+import { agentColor, yamlColor } from '../../utils/agent-color.js'
+
+const FULLSTACK_FILE = 'fullstack-engineer.md'
+const FULLSTACK_DESCRIPTION = 'Default engineer that accumulates skills from all created persona engineers. Use as fallback when no specialist matches: but prefer spawning a specific engineer for deterministic results.'
+const FULLSTACK_IDENTITY = 'You are the default engineer, and your body is what the build and plan agents run. You are more complete but less accurate than specialized engineers, so prefer spawning a specialist when one matches the task domain.'
+
+// The reminder plugin owns ability bootstrap. Remove the old prompt-level
+// directive when regenerating so it cannot duplicate the plugin reminder.
+function stripStartupDirective(text) {
+  if (!text) return text
+  return text
+    .split(/\r?\n\r?\n+/)
+    .filter(paragraph => !paragraph.trim().startsWith('**Startup'))
+    .join('\n\n')
+    .trim()
+}
+
+export async function removeLegacyStartupDirectives({ cwd = process.cwd() } = {}) {
+  const agentsDir = path.join(cwd, '.opencode', 'agents')
+  if (!await fse.pathExists(agentsDir)) return 0
+
+  let updated = 0
+  for (const fileName of await fse.readdir(agentsDir)) {
+    if (!fileName.endsWith('.md')) continue
+    const filePath = path.join(agentsDir, fileName)
+    const content = await fse.readFile(filePath, 'utf-8')
+    const cleaned = stripStartupDirective(content)
+    if (cleaned === content.trim()) continue
+    await fse.writeFile(filePath, `${cleaned}\n`, 'utf-8')
+    updated++
+  }
+  return updated
+}
+
+export async function generateFullstackEngineer({ cwd = process.cwd(), updateMode = false } = {}) {
+  const agentsDir = path.join(cwd, '.opencode', 'agents')
+  await fse.ensureDir(agentsDir)
+  const filePath = path.join(agentsDir, FULLSTACK_FILE)
+
+  if (updateMode && await fse.pathExists(filePath)) {
+    return { generated: false, preserved: true }
+  }
+
+  let existingAbilities = null
+  let existingIdentity = null
+  if (await fse.pathExists(filePath)) {
+    const content = await fse.readFile(filePath, 'utf-8')
+    const abilitiesMatch = content.match(/## Abilities\n([\s\S]*?)$/)
+    if (abilitiesMatch) existingAbilities = abilitiesMatch[1].trim()
+    const identityMatch = content.match(/^---\n[\s\S]*?\n---\n\n(.*?)(?:\n\n## Abilities)/s)
+    if (identityMatch) existingIdentity = stripStartupDirective(identityMatch[1])
+  }
+
+  const frontmatter = [
+    '---',
+    `description: ${FULLSTACK_DESCRIPTION}`,
+    // subagent, not primary: build.md and plan.md are the primaries, and
+    // pc-subagent-tiers generates both from this file on every startup.
+    'mode: subagent',
+    // warning is reserved for plan, so fullstack takes its name-derived hex
+    // like every other subagent.
+    // Quoted: a bare #hex is a YAML comment, which opencode rejects.
+    `color: ${yamlColor(agentColor('fullstack-engineer'))}`,
+    'permission:',
+    '  edit: allow',
+    '  bash: allow',
+    '  read: allow',
+    '  glob: allow',
+    '  grep: allow',
+    '  question: allow',
+    '  todowrite: allow',
+    '---',
+  ]
+
+  const identity = existingIdentity ?? FULLSTACK_IDENTITY
+
+  const abilities = existingAbilities ?? [
+    '- Guardrails: @pc-guardrails-generic, @pc-guardrails-project',
+  ].join('\n')
+
+  const content = `${frontmatter.join('\n')}\n\n${identity}\n\n## Abilities\n${abilities}\n`
+
+  await fse.writeFile(filePath, content, 'utf-8')
+  success(`Generated ${FULLSTACK_FILE}`)
+  return { generated: true }
+}

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)
+  }
+}