@plainconceptsplatform/agent-harness 3.2.0 → 3.3.1

package/cli/steps/optimization/memory.js

@@ -1,88 +1,91 @@
-import { execa } from 'execa'
-import { addSkillToLock } from './skills-lock.js'
-import fse from 'fs-extra'
-import path from 'node:path'
-import { applyEdits, modify, parse as parseJsonc } from 'jsonc-parser'
-import { error, info, loading, success, warn } from '../../utils/exec.js'
-
-/**
- * Configures the agentmemory MCP server in opencode.jsonc
- * and installs the agentmemory skills.
- *
- * agentmemory runs as a persistent local server on localhost:3111.
- * Requires Node.js 18+ (npm install -g @agentmemory/agentmemory).
- * MCP bridge: npx -y @agentmemory/mcp (proxies to the running server).
- * Skill source: https://github.com/rohitg00/agentmemory
- */
-export async function installMemory(options = {}) {
-  if (!options.skipHeader) info('Configuring agentmemory local memory server...')
-
-  // Install agentmemory globally so the `agentmemory` command is on PATH
-  loading('installing agentmemory (this may take a minute)...')
-  try {
-    const installResult = await execa('npm', ['install', '-g', '@agentmemory/agentmemory'], {
-      reject: false,
-      timeout: 300000,
-      stdio: 'pipe',
-    })
-    if (installResult.exitCode === 0) {
-      success('agentmemory installed')
-    } else {
-      warn('agentmemory install exited with non-zero code, MCP may fall back to npx resolution')
-    }
-  } catch (err) {
-    warn(`agentmemory install failed: ${err.message}`)
-  }
-
-  // Configure MCP server in opencode.jsonc
-  try {
-    const opencodePath = path.join(process.cwd(), 'opencode.jsonc')
-
-    let opencodeText = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
-    let opencode
-    if (await fse.pathExists(opencodePath)) {
-      const errors = []
-      opencodeText = await fse.readFile(opencodePath, 'utf-8')
-      opencode = parseJsonc(opencodeText, errors)
-      if (errors.length > 0 || !opencode || typeof opencode !== 'object' || Array.isArray(opencode)) {
-        throw new Error('opencode.jsonc could not be parsed')
-      }
-    } else {
-      opencode = { $schema: 'https://opencode.ai/config.json' }
-    }
-
-    if (!opencode.mcp?.['agentmemory']) {
-      const agentmemory = {
-        type: 'local',
-        command: ['npx', '-y', '@agentmemory/mcp'],
-        env: {
-          AGENTMEMORY_URL: 'http://localhost:3111',
-        },
-        enabled: true,
-        timeout: 120000,
-      }
-      opencodeText = applyEdits(opencodeText, modify(opencodeText, ['mcp', 'agentmemory'], agentmemory, {
-        formattingOptions: { insertSpaces: true, tabSize: 2 },
-      }))
-    }
-
-    await fse.writeFile(opencodePath, opencodeText, 'utf-8')
-    success('agentmemory MCP server configured in opencode.jsonc')
-  } catch (err) {
-    error(`Failed to configure agentmemory MCP: ${err.message}`)
-    return { optedIn: true, installed: false }
-  }
-
-  // Add agentmemory skills to skills-lock.json for batch install
-  await addSkillToLock('agentmemory-agents', {
-    source: 'rohitg00/agentmemory',
-    sourceType: 'github',
-    skillPath: 'plugin/skills/agentmemory-agents/SKILL.md',
-  })
-
-  info('Memory server: run `agentmemory` to start on localhost:3111')
-  info('Real-time viewer: http://localhost:3113')
-  info('Health check: curl http://localhost:3111/agentmemory/health')
-
-  return { optedIn: true, installed: true }
-}
+import { execa } from 'execa'
+import { addSkillToLock } from './skills-lock.js'
+import fse from 'fs-extra'
+import path from 'node:path'
+import { applyEdits, modify, parse as parseJsonc } from 'jsonc-parser'
+import { error, info, loading, success, warn } from '../../utils/exec.js'
+
+/**
+ * Configures the agentmemory MCP server in opencode.jsonc
+ * and installs the agentmemory skills.
+ *
+ * agentmemory runs as a persistent local server on localhost:3111.
+ * Requires Node.js 18+ (npm install -g @agentmemory/agentmemory).
+ * MCP bridge: npx -y @agentmemory/mcp (proxies to the running server).
+ * Skill source: https://github.com/rohitg00/agentmemory
+ */
+export async function installMemory(options = {}) {
+  if (!options.skipHeader) info('Configuring agentmemory local memory server...')
+
+  // Install agentmemory globally so the `agentmemory` command is on PATH
+  loading('installing agentmemory (this may take a minute)...')
+  try {
+    const installResult = await execa('npm', ['install', '-g', '@agentmemory/agentmemory'], {
+      reject: false,
+      timeout: 300000,
+      stdio: 'pipe',
+    })
+    if (installResult.exitCode === 0) {
+      success('agentmemory installed')
+    } else {
+      warn('agentmemory install exited with non-zero code, MCP may fall back to npx resolution')
+    }
+  } catch (err) {
+    warn(`agentmemory install failed: ${err.message}`)
+  }
+
+  // Configure MCP server in opencode.jsonc
+  try {
+    const opencodePath = path.join(process.cwd(), 'opencode.jsonc')
+
+    let opencodeText = JSON.stringify({ $schema: 'https://opencode.ai/config.json' }, null, 2)
+    let opencode
+    if (await fse.pathExists(opencodePath)) {
+      const errors = []
+      opencodeText = await fse.readFile(opencodePath, 'utf-8')
+      opencode = parseJsonc(opencodeText, errors)
+      if (errors.length > 0 || !opencode || typeof opencode !== 'object' || Array.isArray(opencode)) {
+        throw new Error('opencode.jsonc could not be parsed')
+      }
+    } else {
+      opencode = { $schema: 'https://opencode.ai/config.json' }
+    }
+
+    if (!opencode.mcp?.servers?.agentmemory) {
+      const agentmemory = {
+        type: 'local',
+        command: ['npx', '-y', '@agentmemory/mcp'],
+        environment: {
+          AGENTMEMORY_URL: 'http://localhost:3111',
+        },
+        disabled: false,
+        timeout: {
+          "catalog": 120000,
+          "execution": 120000
+        }
+      }
+      opencodeText = applyEdits(opencodeText, modify(opencodeText, ['mcp', 'servers', 'agentmemory'], agentmemory, {
+        formattingOptions: { insertSpaces: true, tabSize: 2 },
+      }))
+    }
+
+    await fse.writeFile(opencodePath, opencodeText, 'utf-8')
+    success('agentmemory MCP server configured in opencode.jsonc')
+  } catch (err) {
+    error(`Failed to configure agentmemory MCP: ${err.message}`)
+    return { optedIn: true, installed: false }
+  }
+
+  // Add agentmemory skills to skills-lock.json for batch install
+  await addSkillToLock('agentmemory-agents', {
+    source: 'rohitg00/agentmemory',
+    sourceType: 'github',
+    skillPath: 'plugin/skills/agentmemory-agents/SKILL.md',
+  })
+
+  info('Memory server: run `agentmemory` to start on localhost:3111')
+  info('Real-time viewer: http://localhost:3113')
+  info('Health check: curl http://localhost:3111/agentmemory/health')
+
+  return { optedIn: true, installed: true }
+}

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

@@ -1,111 +1,111 @@
-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: 'Agentmemory carries context across sessions and agents: before exploring an unfamiliar area, call `memory_smart_search` with the key concepts of the task (components, error text) and reuse what it returns; when the work completes, call `memory_save` for durable non-obvious learnings (architecture constraints, root causes and fixes, decisions and why). Never store secrets.',
-}
-
-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 -->',
-    )
-    // A third replace used to rewrite the engineer workflow's step 1 into "the
-    // plugin has already loaded the skills". That was never true: the plugin
-    // appends a reminder and never calls the skill tool. The shipped skill now
-    // carries the honest wording, and update refreshes this file, so the
-    // migration is gone rather than perpetuated.
-    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: 'Agentmemory carries context across sessions and agents: before exploring an unfamiliar area, call `memory_smart_search` with the key concepts of the task (components, error text) and reuse what it returns; when the work completes, call `memory_save` for durable non-obvious learnings (architecture constraints, root causes and fixes, decisions and why). Never store secrets.',
+}
+
+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 -->',
+    )
+    // A third replace used to rewrite the engineer workflow's step 1 into "the
+    // plugin has already loaded the skills". That was never true: the plugin
+    // appends a reminder and never calls the skill tool. The shipped skill now
+    // carries the honest wording, and update refreshes this file, so the
+    // migration is gone rather than perpetuated.
+    for (const [key, markerSuffix] of Object.entries(MARKER_SECTIONS)) {
+      let sectionContent = ''
+      if (selections[key]) {
+        const presetName = key === 'simpleEnglish' ? 'simple-english' : key
+        const presetPath = path.join(GUARDRAILS_FRAGMENT_DIR, `${presetName}.md`)
+        if (await fse.pathExists(presetPath)) {
+          sectionContent = (await fse.readFile(presetPath, 'utf-8')).trim()
+        }
+      }
+      content = replaceMarkerSection(
+        content,
+        `<!-- PC-GUARDRAILS-${markerSuffix}-START -->`,
+        `<!-- PC-GUARDRAILS-${markerSuffix}-END -->`,
+        sectionContent,
+      )
+      if (selections[key]) patchedCount++
+    }
+    await fse.writeFile(guardrailsPath, `${content.replace(/\s*$/, '')}\n`, 'utf-8')
+  } else {
+    info('pc-guardrails-generic not found, skipping guardrails patching')
+  }
+
+  const optionalPatched = await patchOptionalToolGuidance(destSkillsDir, selections)
+  success(`Guardrails patched: ${patchedCount} sections injected; ${optionalPatched} optional tool file(s) updated`)
+  return { patched: true, count: patchedCount, optionalPatched }
+}

package/cli/steps/optimization/simple-english.js

@@ -1,17 +1,17 @@
-import { addSkillToLock } from './skills-lock.js'
-import { header, success, info } from '../../utils/exec.js'
-
-const SKILL_ENTRY = {
-  source: 'AminBlg/SimpleEnglish',
-  sourceType: 'github',
-  skillPath: 'skills/simple-english/SKILL.md',
-}
-
-export async function installSimpleEnglish(options = {}) {
-  if (!options.skipHeader) header('Installing Simple English')
-
-  info('Adding Simple English to skills-lock.json for batch install')
-  await addSkillToLock('simple-english', SKILL_ENTRY)
-  success('Simple English queued for installation')
-  return { optedIn: true, installed: true }
-}
+import { addSkillToLock } from './skills-lock.js'
+import { header, success, info } from '../../utils/exec.js'
+
+const SKILL_ENTRY = {
+  source: 'AminBlg/SimpleEnglish',
+  sourceType: 'github',
+  skillPath: 'skills/simple-english/SKILL.md',
+}
+
+export async function installSimpleEnglish(options = {}) {
+  if (!options.skipHeader) header('Installing Simple English')
+
+  info('Adding Simple English to skills-lock.json for batch install')
+  await addSkillToLock('simple-english', SKILL_ENTRY)
+  success('Simple English queued for installation')
+  return { optedIn: true, installed: true }
+}

package/cli/steps/optimization/skills-lock.js

@@ -1,30 +1,30 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { info, warn } from '../../utils/exec.js'
-
-export async function addSkillToLock(skillName, entry) {
-  if (!entry) {
-    warn(`No entry provided for skills-lock.json: ${skillName}`)
-    return false
-  }
-
-  const destLock = path.join(process.cwd(), 'skills-lock.json')
-
-  let lock = { version: 1, skills: {} }
-  if (await fse.pathExists(destLock)) {
-    try {
-      lock = await fse.readJson(destLock)
-      if (!lock.skills) lock.skills = {}
-    } catch {
-      warn('Could not parse existing skills-lock.json, creating fresh')
-    }
-  }
-
-  if (!lock.skills[skillName]) {
-    lock.skills[skillName] = entry
-    await fse.writeJson(destLock, lock, { spaces: 2 })
-    info(`Added ${skillName} to skills-lock.json`)
-  }
-
-  return true
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { info, warn } from '../../utils/exec.js'
+
+export async function addSkillToLock(skillName, entry) {
+  if (!entry) {
+    warn(`No entry provided for skills-lock.json: ${skillName}`)
+    return false
+  }
+
+  const destLock = path.join(process.cwd(), 'skills-lock.json')
+
+  let lock = { version: 1, skills: {} }
+  if (await fse.pathExists(destLock)) {
+    try {
+      lock = await fse.readJson(destLock)
+      if (!lock.skills) lock.skills = {}
+    } catch {
+      warn('Could not parse existing skills-lock.json, creating fresh')
+    }
+  }
+
+  if (!lock.skills[skillName]) {
+    lock.skills[skillName] = entry
+    await fse.writeJson(destLock, lock, { spaces: 2 })
+    info(`Added ${skillName} to skills-lock.json`)
+  }
+
+  return true
+}