@plainconceptsplatform/agent-harness 2.0.0 → 2.1.0

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

@@ -1,83 +1,85 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { success } from '../../utils/exec.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, mostly used by the user for architecture and planning. You are more complete but less accurate than specialized engineers, 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}`,
-    'mode: primary',
-    'color: warning',
-    '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'
+
+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',
+    'color: warning',
+    '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/src/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, patchOpsEvidence } from "./commands.js"
-import { installSkills } 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, "../../content")
-
-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, pc-ops-evidence),
-    // so they must run after installSkills has copied the skills into the project.
-    await patchArchiveCommand({ backlogPlatform, repoPlatform })
-    await patchOpsShip({ backlogPlatform, repoPlatform })
-    await patchOpsEvidence({ backlogPlatform, repoPlatform })
-    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, patchOpsEvidence } from "./commands.js"
+import { installSkills } 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, "../../content")
+
+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, pc-ops-evidence),
+    // so they must run after installSkills has copied the skills into the project.
+    await patchArchiveCommand({ backlogPlatform, repoPlatform })
+    await patchOpsShip({ backlogPlatform, repoPlatform })
+    await patchOpsEvidence({ backlogPlatform, repoPlatform })
+    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)
+  }
+}

package/src/steps/copy/opencode-json.js

@@ -1,129 +1,147 @@
-import fse from 'fs-extra'
-import { applyEdits, modify, parse } from 'jsonc-parser'
-import path from 'node:path'
-import { success } from '../../utils/exec.js'
-
-// The reminder plugin loads every agent ability before work, including
-// user-installed skills outside the pc-* and openspec-* namespaces.
-const SHARED_PERMISSIONS = [
-  ['question', 'allow'],
-  ['todowrite', 'allow'],
-]
-
-function applyModify(text, jsonPath, value) {
-  const edits = modify(text, jsonPath, value, {
-    formattingOptions: { insertSpaces: true, tabSize: 2 },
-  })
-  return applyEdits(text, edits)
-}
-
-export async function patchOpencodeJson(cwd = process.cwd()) {
-  const opencodePath = path.join(cwd, 'opencode.jsonc')
-
-  let text
-  if (await fse.pathExists(opencodePath)) {
-    text = await fse.readFile(opencodePath, 'utf-8')
-  } else {
-    text = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
-  }
-
-  const errors = []
-  const parsed = parse(text, errors)
-  if (errors.length > 0 || typeof parsed !== 'object') {
-    return { patched: false, reason: 'parse error' }
-  }
-
-  const needsAgentDisable = !(
-    parsed?.agent?.build?.disable === true &&
-    parsed?.agent?.plan?.disable === true
-  )
-  const needsSkillPermission = parsed?.permission?.skill !== 'allow'
-  const missingSharedPermissions = SHARED_PERMISSIONS.filter(
-    ([key, value]) => parsed?.permission?.[key] !== value,
-  )
-  // opencode only scans .opencode/skills/ by default, but we install to .agents/skills/
-  const needsSkillsPaths = !(
-    Array.isArray(parsed?.skills?.paths) &&
-    parsed.skills.paths.includes('.agents/skills')
-  )
-  const needsCompaction = !(
-    parsed?.compaction?.auto === true &&
-    parsed?.compaction?.prune === true
-  )
-
-  if (!needsAgentDisable && !needsSkillPermission && missingSharedPermissions.length === 0 && !needsSkillsPaths && !needsCompaction) {
-    return { patched: false }
-  }
-
-  // Apply edits sequentially so offsets stay correct
-  if (needsAgentDisable) {
-    text = applyModify(text, ['agent', 'build', 'disable'], true)
-    text = applyModify(text, ['agent', 'plan', 'disable'], true)
-  }
-  if (needsSkillPermission) text = applyModify(text, ['permission', 'skill'], 'allow')
-  for (const [key, value] of missingSharedPermissions) {
-    text = applyModify(text, ['permission', key], value)
-  }
-
-  // Ensure .agents/skills is in skills.paths so opencode discovers installed skills
-  if (needsSkillsPaths) {
-    const existingPaths = Array.isArray(parsed?.skills?.paths) ? parsed.skills.paths : []
-    if (!existingPaths.includes('.agents/skills')) {
-      const updatedPaths = [...existingPaths, '.agents/skills']
-      text = applyModify(text, ['skills', 'paths'], updatedPaths)
-    }
-  }
-
-  // Enable context pruning to reduce token costs in long agent sessions
-  if (needsCompaction) {
-    text = applyModify(text, ['compaction', 'auto'], true)
-    text = applyModify(text, ['compaction', 'prune'], true)
-    text = applyModify(text, ['compaction', 'reserved'], 10000)
-  }
-
-  await fse.writeFile(opencodePath, text, 'utf-8')
-  if (needsAgentDisable) {
-    success('Disabled built-in build/plan agents in opencode.jsonc')
-  }
-  if (needsSkillPermission) {
-    success('Allowed skill loading in opencode.jsonc')
-  }
-  if (missingSharedPermissions.length > 0) {
-    success('Allowed question/todowrite tools in opencode.jsonc')
-  }
-  if (needsSkillsPaths) {
-    success('Added .agents/skills to skills.paths in opencode.jsonc')
-  }
-  if (needsCompaction) {
-    success('Enabled context pruning (compaction.prune) in opencode.jsonc')
-  }
-
-  return { patched: true }
-}
-
-const OPENCODE_PACKAGE_DEPENDENCIES = {
-  '@opencode-ai/plugin': '1.18.19',
-  '@opentui/core': '0.5.6',
-  '@opentui/solid': '0.5.6',
-  'solid-js': '1.9.12',
-}
-
-export async function patchOpencodePackage(cwd = process.cwd()) {
-  const packagePath = path.join(cwd, '.opencode', 'package.json')
-  if (!await fse.pathExists(packagePath)) return { patched: false }
-
-  const packageJson = await fse.readJson(packagePath).catch(() => null)
-  if (!packageJson?.dependencies) return { patched: false }
-
-  let patched = false
-  for (const [name, version] of Object.entries(OPENCODE_PACKAGE_DEPENDENCIES)) {
-    if (!(name in packageJson.dependencies) || packageJson.dependencies[name] === version) continue
-    packageJson.dependencies[name] = version
-    patched = true
-  }
-  if (!patched) return { patched: false }
-
-  await fse.writeJson(packagePath, packageJson, { spaces: 2 })
-  success('Updated OpenCode plugin dependencies in .opencode/package.json')
-  return { patched: true }
-}
+import fse from 'fs-extra'
+import { applyEdits, modify, parse } from 'jsonc-parser'
+import path from 'node:path'
+import { success } from '../../utils/exec.js'
+
+// The reminder plugin loads every agent ability before work, including
+// user-installed skills outside the pc-* and openspec-* namespaces.
+const SHARED_PERMISSIONS = [
+  ['question', 'allow'],
+  ['todowrite', 'allow'],
+]
+
+function applyModify(text, jsonPath, value) {
+  const edits = modify(text, jsonPath, value, {
+    formattingOptions: { insertSpaces: true, tabSize: 2 },
+  })
+  return applyEdits(text, edits)
+}
+
+export async function patchOpencodeJson(cwd = process.cwd()) {
+  const opencodePath = path.join(cwd, 'opencode.jsonc')
+
+  let text
+  if (await fse.pathExists(opencodePath)) {
+    text = await fse.readFile(opencodePath, 'utf-8')
+  } else {
+    text = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
+  }
+
+  const errors = []
+  const parsed = parse(text, errors)
+  if (errors.length > 0 || typeof parsed !== 'object') {
+    return { patched: false, reason: 'parse error' }
+  }
+
+  // build and plan are overridden, not disabled. Earlier versions disabled them
+  // and pointed default_agent at fullstack-engineer; now they are the only two
+  // primaries and pc-subagent-tiers regenerates their agent files from
+  // fullstack-engineer.md. A repo patched by an earlier version still carries
+  // `disable: true`, which would hide both agents, so that flag is cleared here.
+  const hasStaleDisable = (
+    parsed?.agent?.build?.disable !== undefined ||
+    parsed?.agent?.plan?.disable !== undefined
+  )
+  const needsAgentOverride = hasStaleDisable || !(
+    parsed?.agent?.build?.mode === 'primary' &&
+    parsed?.agent?.plan?.mode === 'primary' &&
+    parsed?.agent?.plan?.permission?.edit === 'deny'
+  )
+  const needsSkillPermission = parsed?.permission?.skill !== 'allow'
+  const missingSharedPermissions = SHARED_PERMISSIONS.filter(
+    ([key, value]) => parsed?.permission?.[key] !== value,
+  )
+  // opencode only scans .opencode/skills/ by default, but we install to .agents/skills/
+  const needsSkillsPaths = !(
+    Array.isArray(parsed?.skills?.paths) &&
+    parsed.skills.paths.includes('.agents/skills')
+  )
+  const needsCompaction = !(
+    parsed?.compaction?.auto === true &&
+    parsed?.compaction?.prune === true
+  )
+
+  if (!needsAgentOverride && !needsSkillPermission && missingSharedPermissions.length === 0 && !needsSkillsPaths && !needsCompaction) {
+    return { patched: false }
+  }
+
+  // Apply edits sequentially so offsets stay correct
+  if (needsAgentOverride) {
+    // undefined removes the key, clearing the disable left by earlier versions.
+    if (parsed?.agent?.build?.disable !== undefined) {
+      text = applyModify(text, ['agent', 'build', 'disable'], undefined)
+    }
+    if (parsed?.agent?.plan?.disable !== undefined) {
+      text = applyModify(text, ['agent', 'plan', 'disable'], undefined)
+    }
+    text = applyModify(text, ['agent', 'build', 'mode'], 'primary')
+    text = applyModify(text, ['agent', 'plan', 'mode'], 'primary')
+    text = applyModify(text, ['agent', 'plan', 'permission', 'edit'], 'deny')
+  }
+  if (needsSkillPermission) text = applyModify(text, ['permission', 'skill'], 'allow')
+  for (const [key, value] of missingSharedPermissions) {
+    text = applyModify(text, ['permission', key], value)
+  }
+
+  // Ensure .agents/skills is in skills.paths so opencode discovers installed skills
+  if (needsSkillsPaths) {
+    const existingPaths = Array.isArray(parsed?.skills?.paths) ? parsed.skills.paths : []
+    if (!existingPaths.includes('.agents/skills')) {
+      const updatedPaths = [...existingPaths, '.agents/skills']
+      text = applyModify(text, ['skills', 'paths'], updatedPaths)
+    }
+  }
+
+  // Enable context pruning to reduce token costs in long agent sessions
+  if (needsCompaction) {
+    text = applyModify(text, ['compaction', 'auto'], true)
+    text = applyModify(text, ['compaction', 'prune'], true)
+    text = applyModify(text, ['compaction', 'reserved'], 10000)
+  }
+
+  await fse.writeFile(opencodePath, text, 'utf-8')
+  if (needsAgentOverride) {
+    success('Set build/plan as the primary agents in opencode.jsonc (plan is read-only)')
+  }
+  if (needsSkillPermission) {
+    success('Allowed skill loading in opencode.jsonc')
+  }
+  if (missingSharedPermissions.length > 0) {
+    success('Allowed question/todowrite tools in opencode.jsonc')
+  }
+  if (needsSkillsPaths) {
+    success('Added .agents/skills to skills.paths in opencode.jsonc')
+  }
+  if (needsCompaction) {
+    success('Enabled context pruning (compaction.prune) in opencode.jsonc')
+  }
+
+  return { patched: true }
+}
+
+const OPENCODE_PACKAGE_DEPENDENCIES = {
+  '@opencode-ai/plugin': '1.18.19',
+  '@opentui/core': '0.5.6',
+  '@opentui/solid': '0.5.6',
+  'solid-js': '1.9.12',
+}
+
+export async function patchOpencodePackage(cwd = process.cwd()) {
+  const packagePath = path.join(cwd, '.opencode', 'package.json')
+  if (!await fse.pathExists(packagePath)) return { patched: false }
+
+  const packageJson = await fse.readJson(packagePath).catch(() => null)
+  if (!packageJson?.dependencies) return { patched: false }
+
+  let patched = false
+  for (const [name, version] of Object.entries(OPENCODE_PACKAGE_DEPENDENCIES)) {
+    if (!(name in packageJson.dependencies) || packageJson.dependencies[name] === version) continue
+    packageJson.dependencies[name] = version
+    patched = true
+  }
+  if (!patched) return { patched: false }
+
+  await fse.writeJson(packagePath, packageJson, { spaces: 2 })
+  success('Updated OpenCode plugin dependencies in .opencode/package.json')
+  return { patched: true }
+}