@plainconceptsplatform/agent-harness 2.2.0 → 2.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plainconceptsplatform/agent-harness",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Installs the Plain Concepts Platform Harness into any codebase, and keeps it up to date. Wires OpenCode, OpenSpec, codegraph, and agentmemory into a multi-agent workflow that runs on native parallel subagents.",
   "keywords": [
     "opencode",

package/src/commands/migrate.js

@@ -235,7 +235,11 @@ export async function runMigrate({ cwd = process.cwd(), force = false, dryRun =
   // 4. Rewrite identifiers everywhere that survives, so preserved skills and
   //    custom agents stop pointing at names that no longer exist.
   let rewritten = 0
-  for (const dir of [skillsDir, opencodeDir, path.join(cwd, 'openspec')]) {
+  // .github/workflows is included because GitHub Agentic Workflow definitions
+  // call the skills by name (skill("pc-repo-audit")), so a stale ob- name there
+  // fails at run time in CI rather than locally. Their compiled .lock.yml files
+  // embed the same prompt text, so recompile with `gh aw compile` afterwards.
+  for (const dir of [skillsDir, opencodeDir, path.join(cwd, 'openspec'), path.join(cwd, '.github', 'workflows')]) {
     if (!await fse.pathExists(dir)) continue
     for (const file of await collectTextFiles(dir)) {
       const before = await fse.readFile(file, 'utf-8')
@@ -281,6 +285,18 @@ export async function runMigrate({ cwd = process.cwd(), force = false, dryRun =
   console.log(chalk.bold('Next: run update to install the current harness.'))
   console.log(chalk.dim('  npx @plainconceptsplatform/agent-harness@latest update'))
 
+  // The .md definitions were rewritten above, but the compiled .lock.yml files
+  // embed the same prompt text and are what CI actually runs.
+  if (await fse.pathExists(path.join(cwd, '.github', 'workflows'))) {
+    const locks = (await fse.readdir(path.join(cwd, '.github', 'workflows')).catch(() => []))
+      .filter(f => f.endsWith('.lock.yml'))
+    if (locks.length > 0) {
+      console.log()
+      console.log(chalk.bold(`Then recompile ${locks.length} agentic workflow(s), whose .lock.yml still holds the old skill names:`))
+      console.log(chalk.dim('  gh aw compile'))
+    }
+  }
+
   return { migrated: true, plan, rewritten, demoted }
 }
 

package/src/steps/copy/agents.js

@@ -1,118 +1,119 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { info, success, warn } from '../../utils/exec.js'
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const agentsContent = await fse.readJson(path.resolve(__dirname, '../../presets/agents-content.json'))
-
-// Steps are matched by their title text, not by exact heading string:
-// heading level (###/####) and step numbers have drifted before and silently
-// broke removal. Matching `^#{3,4} Step N, <title>` survives both.
-const HISTORY_STEP_TITLE = 'Archive project history'
-const CHAIN_STEP_TITLE = 'Chain make commands'
-
-const CHAIN_CONFIRM_LINE = '- ARCHITECTURE.md generated'
-
-function stepHeadingPattern(title) {
-  const escaped = title.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
-  return new RegExp(`^#{2,4} Step \\d+, ${escaped}`)
-}
-
-// The step heading is kept and its body replaced with an explicit skip note.
-// Removing the block and renumbering the remaining steps (the old approach)
-// went stale against prose cross-references like "Skip steps 2, 3, and 4";
-// stable numbering plus a visible reason is unambiguous for the agent.
-export function skipStepBlock(content, title, note) {
-  const lines = content.split('\n')
-  const pattern = stepHeadingPattern(title)
-  const start = lines.findIndex(l => pattern.test(l.trim()))
-  if (start === -1) return { content, matched: false }
-
-  let end = lines.length
-  for (let i = start + 1; i < lines.length; i++) {
-    if (lines[i].trim() === '---') { end = i; break }
-  }
-
-  lines.splice(start + 1, end - start - 1, '', `> ${note}`, '')
-  return { content: lines.join('\n'), matched: true }
-}
-
-function removeConfirmLine(content, line) {
-  return content.split('\n').filter(l => l.trim() !== line.trim()).join('\n')
-}
-
-const PLATFORM_WORKFLOW_START = '<!-- PC-PLATFORM-WORKFLOW-START -->'
-const PLATFORM_WORKFLOW_END = '<!-- PC-PLATFORM-WORKFLOW-END -->'
-const PLATFORM_SKILLS_GUIDE_START = '<!-- PC-PLATFORM-SKILLS-GUIDE-START -->'
-const PLATFORM_SKILLS_GUIDE_END = '<!-- PC-PLATFORM-SKILLS-GUIDE-END -->'
-
-function platformContent(platform, key) {
-  const p = agentsContent.platform[platform] ?? agentsContent.platform.github
-  return p[key] ?? ''
-}
-
-function mixedPlatformContent(backlogPlatform, repoPlatform, key) {
-  if (backlogPlatform === repoPlatform) {
-    return platformContent(backlogPlatform, key)
-  }
-  const backlog = agentsContent.platform[backlogPlatform] ?? agentsContent.platform.github
-  const repo = agentsContent.platform[repoPlatform] ?? agentsContent.platform.github
-  const mixed = agentsContent.platform.mixed ?? {}
-  const mixedKey = `${backlogPlatform}-${repoPlatform}`
-  return mixed[mixedKey]?.[key] ?? mixed.default?.[key]?.replace('{backlog}', backlog[key] ?? '').replace('{repo}', repo[key] ?? '') ?? platformContent(repoPlatform, key)
-}
-
-function replaceBetween(content, start, end, replacement) {
-  if (!content.includes(start) || !content.includes(end)) return content
-  const pattern = new RegExp(`${start}[\\s\\S]*?${end}`)
-  return content.replace(pattern, `${start}\n${replacement.trim()}\n${end}`)
-}
-
-export async function patchAgentsMd(ctx) {
-  const obInitSkillPath = path.join(process.cwd(), '.agents', 'skills', 'pc-repo-initialize', 'SKILL.md')
-  if (!await fse.pathExists(obInitSkillPath)) return
-
-  let content = await fse.readFile(obInitSkillPath, 'utf-8')
-  const patches = []
-
-  const skips = [
-    [ctx.hasOpenspec, HISTORY_STEP_TITLE, null,
-      'Skipped during onboarding: this project already had an openspec/ history. Do not archive again; continue with the next step.'],
-    [ctx.hasDesign || ctx.hasArchitecture, CHAIN_STEP_TITLE, CHAIN_CONFIRM_LINE,
-      'Skipped during onboarding: project files already exist. Run /make-architecture or /make-design individually to regenerate.'],
-  ]
-
-  for (const [enabled, title, confirmLine, note] of skips) {
-    if (!enabled) continue
-    const result = skipStepBlock(content, title, note)
-    if (!result.matched) {
-      warn(`pc-repo-initialize SKILL.md step "${title}" not found: template drift? Skipping this patch.`)
-      continue
-    }
-    content = result.content
-    if (confirmLine) content = removeConfirmLine(content, confirmLine)
-    patches.push(`Step "${title}" marked as skipped, file already exists`)
-  }
-
-  if (patches.length > 0) {
-    await fse.writeFile(obInitSkillPath, content, 'utf-8')
-    for (const msg of patches) info(msg)
-    success('pc-repo-initialize SKILL.md patched for existing project state')
-  }
-}
-
-export async function patchAgentGuidance(backlogPlatform, repoPlatform, cwd = process.cwd()) {
-  const agentsMdPath = path.join(cwd, 'AGENTS.md')
-  if (await fse.pathExists(agentsMdPath)) {
-    const repo = repoPlatform ?? backlogPlatform ?? 'github'
-    let content = await fse.readFile(agentsMdPath, 'utf-8')
-    content = replaceBetween(content, PLATFORM_WORKFLOW_START, PLATFORM_WORKFLOW_END, mixedPlatformContent(backlogPlatform, repo, 'workflow'))
-    content = replaceBetween(content, PLATFORM_SKILLS_GUIDE_START, PLATFORM_SKILLS_GUIDE_END, mixedPlatformContent(backlogPlatform, repo, 'skillsGuide'))
-    await fse.writeFile(agentsMdPath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
-    const label = backlogPlatform === repo ? repo : `${backlogPlatform}→${repo}`
-    success(`AGENTS.md patched for platform workflow: ${label}`)
-  }
-}
-
-
+import fse from 'fs-extra'
+import path from 'path'
+import { fileURLToPath } from 'url'
+import { info, success, warn } from '../../utils/exec.js'
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const agentsContent = await fse.readJson(path.resolve(__dirname, '../../presets/agents-content.json'))
+
+// Steps are matched by their title text, not by exact heading string:
+// heading level (###/####) and step numbers have drifted before and silently
+// broke removal. Matching `^#{3,4} Step N, <title>` survives both.
+const HISTORY_STEP_TITLE = 'Archive project history'
+const CHAIN_STEP_TITLE = 'Chain make commands'
+
+const CHAIN_CONFIRM_LINE = '- ARCHITECTURE.md generated'
+
+function stepHeadingPattern(title) {
+  const escaped = title.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')
+  return new RegExp(`^#{2,4} Step \\d+, ${escaped}`)
+}
+
+// The step heading is kept and its body replaced with an explicit skip note.
+// Removing the block and renumbering the remaining steps (the old approach)
+// went stale against prose cross-references like "Skip steps 2, 3, and 4";
+// stable numbering plus a visible reason is unambiguous for the agent.
+export function skipStepBlock(content, title, note) {
+  const lines = content.split('\n')
+  const pattern = stepHeadingPattern(title)
+  const start = lines.findIndex(l => pattern.test(l.trim()))
+  if (start === -1) return { content, matched: false }
+
+  let end = lines.length
+  for (let i = start + 1; i < lines.length; i++) {
+    if (lines[i].trim() === '---') { end = i; break }
+  }
+
+  lines.splice(start + 1, end - start - 1, '', `> ${note}`, '')
+  return { content: lines.join('\n'), matched: true }
+}
+
+function removeConfirmLine(content, line) {
+  return content.split('\n').filter(l => l.trim() !== line.trim()).join('\n')
+}
+
+const PLATFORM_WORKFLOW_START = '<!-- PC-PLATFORM-WORKFLOW-START -->'
+const PLATFORM_WORKFLOW_END = '<!-- PC-PLATFORM-WORKFLOW-END -->'
+const PLATFORM_SKILLS_GUIDE_START = '<!-- PC-PLATFORM-SKILLS-GUIDE-START -->'
+const PLATFORM_SKILLS_GUIDE_END = '<!-- PC-PLATFORM-SKILLS-GUIDE-END -->'
+
+function platformContent(platform, key) {
+  const p = agentsContent.platform[platform] ?? agentsContent.platform.github
+  return p[key] ?? ''
+}
+
+function mixedPlatformContent(backlogPlatform, repoPlatform, key) {
+  if (backlogPlatform === repoPlatform) {
+    return platformContent(backlogPlatform, key)
+  }
+  const backlog = agentsContent.platform[backlogPlatform] ?? agentsContent.platform.github
+  const repo = agentsContent.platform[repoPlatform] ?? agentsContent.platform.github
+  const mixed = agentsContent.platform.mixed ?? {}
+  const mixedKey = `${backlogPlatform}-${repoPlatform}`
+  return mixed[mixedKey]?.[key] ?? mixed.default?.[key]?.replace('{backlog}', backlog[key] ?? '').replace('{repo}', repo[key] ?? '') ?? platformContent(repoPlatform, key)
+}
+
+function replaceBetween(content, start, end, replacement) {
+  if (!content.includes(start) || !content.includes(end)) return content
+  const pattern = new RegExp(`${start}[\\s\\S]*?${end}`)
+  // Replacer function so $&, $' and $` in injected content stay literal.
+  return content.replace(pattern, () => `${start}\n${replacement.trim()}\n${end}`)
+}
+
+export async function patchAgentsMd(ctx) {
+  const obInitSkillPath = path.join(process.cwd(), '.agents', 'skills', 'pc-repo-initialize', 'SKILL.md')
+  if (!await fse.pathExists(obInitSkillPath)) return
+
+  let content = await fse.readFile(obInitSkillPath, 'utf-8')
+  const patches = []
+
+  const skips = [
+    [ctx.hasOpenspec, HISTORY_STEP_TITLE, null,
+      'Skipped during onboarding: this project already had an openspec/ history. Do not archive again; continue with the next step.'],
+    [ctx.hasDesign || ctx.hasArchitecture, CHAIN_STEP_TITLE, CHAIN_CONFIRM_LINE,
+      'Skipped during onboarding: project files already exist. Run /make-architecture or /make-design individually to regenerate.'],
+  ]
+
+  for (const [enabled, title, confirmLine, note] of skips) {
+    if (!enabled) continue
+    const result = skipStepBlock(content, title, note)
+    if (!result.matched) {
+      warn(`pc-repo-initialize SKILL.md step "${title}" not found: template drift? Skipping this patch.`)
+      continue
+    }
+    content = result.content
+    if (confirmLine) content = removeConfirmLine(content, confirmLine)
+    patches.push(`Step "${title}" marked as skipped, file already exists`)
+  }
+
+  if (patches.length > 0) {
+    await fse.writeFile(obInitSkillPath, content, 'utf-8')
+    for (const msg of patches) info(msg)
+    success('pc-repo-initialize SKILL.md patched for existing project state')
+  }
+}
+
+export async function patchAgentGuidance(backlogPlatform, repoPlatform, cwd = process.cwd()) {
+  const agentsMdPath = path.join(cwd, 'AGENTS.md')
+  if (await fse.pathExists(agentsMdPath)) {
+    const repo = repoPlatform ?? backlogPlatform ?? 'github'
+    let content = await fse.readFile(agentsMdPath, 'utf-8')
+    content = replaceBetween(content, PLATFORM_WORKFLOW_START, PLATFORM_WORKFLOW_END, mixedPlatformContent(backlogPlatform, repo, 'workflow'))
+    content = replaceBetween(content, PLATFORM_SKILLS_GUIDE_START, PLATFORM_SKILLS_GUIDE_END, mixedPlatformContent(backlogPlatform, repo, 'skillsGuide'))
+    await fse.writeFile(agentsMdPath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
+    const label = backlogPlatform === repo ? repo : `${backlogPlatform}→${repo}`
+    success(`AGENTS.md patched for platform workflow: ${label}`)
+  }
+}
+
+

package/src/steps/copy/commands.js

@@ -1,91 +1,95 @@
-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'),
-}
-
-// Evidence comments target the backlog platform (where the issue/work-item lives).
-const _evidence = {
-  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/gh.md'), 'utf-8'),
-  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/az.md'), 'utf-8'),
-  jira: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/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, '\\$&')}`)
-  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)
-}
-
-export async function patchOpsEvidence(platform, cwd = process.cwd()) {
-  const backlogPlatform = typeof platform === 'object' ? (platform.backlogPlatform ?? 'github') : platform
-  const replacement = _evidence[backlogPlatform]
-  // browser/none have no CLI to post with: leave the markers empty (the skill skips commenting).
-  if (!replacement) return
-  patchFile('.agents/skills/pc-ops-evidence/SKILL.md', '<!-- PC-PLATFORM-EVIDENCE-START -->', '<!-- PC-PLATFORM-EVIDENCE-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'),
+}
+
+// Evidence comments target the backlog platform (where the issue/work-item lives).
+const _evidence = {
+  github: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/gh.md'), 'utf-8'),
+  azure: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/az.md'), 'utf-8'),
+  jira: await fse.readFile(path.resolve(__dirname, '../../fragments/ops-evidence/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)
+}
+
+export async function patchOpsEvidence(platform, cwd = process.cwd()) {
+  const backlogPlatform = typeof platform === 'object' ? (platform.backlogPlatform ?? 'github') : platform
+  const replacement = _evidence[backlogPlatform]
+  // browser/none have no CLI to post with: leave the markers empty (the skill skips commenting).
+  if (!replacement) return
+  patchFile('.agents/skills/pc-ops-evidence/SKILL.md', '<!-- PC-PLATFORM-EVIDENCE-START -->', '<!-- PC-PLATFORM-EVIDENCE-END -->', replacement, backlogPlatform, cwd)
+}
+

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

@@ -1,108 +1,109 @@
-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)}`)
+  // Replacer function so $&, $' and $` in injected content stay literal.
+  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 }
+}