@plainconceptsplatform/agent-harness 3.1.0 → 3.1.1

package/README.md

@@ -145,7 +145,7 @@ Commands that other commands (or agents) need to execute are thin wrappers aroun
 | `/make-architecture` | Generate or regenerate `ARCHITECTURE.md` from the codebase. |
 | `/make-design` | Generate or regenerate `DESIGN.md` from the design system. |
 | `/make-guardrails` | Generate a `pc-guardrails-project` skill from `ARCHITECTURE.md` and project config files. Extracts architecture boundaries, naming, code style, testing, and git workflow rules. Updates all `*-engineer.md` to load the skill. |
-| `/repo-verify` | Write the verification plan for the current branch's change: a journey through the new functionality as a fenced YAML block in `verification-plan.md`, stored with the change. The evidence workflow from `@plainconceptsplatform/workflows` replays it in headless Chromium when the pull request merges and archives one screenshot per waypoint on it. Backend-only changes are traced through to the frontend when their contract is consumed there; a `not-applicable` stub is written when no UI surface is reachable. Does not run checks or take screenshots. Runs automatically in `/plan-goal`. |
+| `/repo-verify` | Write the verification plan for the current branch's change: a journey through the new functionality as a fenced YAML block in `verification-plan.md`, stored with the change. The evidence workflow from `@plainconceptsplatform/workflows` replays it in headless Chromium when the pull request merges and archives one screenshot per waypoint in an `evidence/` folder beside the plan. Backend-only changes are traced through to the frontend when their contract is consumed there; a `not-applicable` stub is written when no UI surface is reachable. Does not run checks or take screenshots. Runs automatically in `/plan-goal`. |
 | `/make-user-model [user] <tier> <model>` | Set the model for a tier (`plan`, `build`, `fast`). Writes to `harness.json` (team) or `harness.user.json` (user override, gitignored) when `user` prefix is used. Restart to pick up changes: the `pc-subagent-tiers` plugin rebuilds tier agents at startup. Pass a model id or `current` for the active session model. |
 
 ---

package/cli/commands/update.js

@@ -1,121 +1,121 @@
-import chalk from 'chalk'
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import path from 'node:path'
-import { fileURLToPath } from 'url'
-import { readHarnessConfig } from './shared.js'
-import { copyContentStep } from '../steps/copy/index.js'
-import { writeModelsToConfigs } from '../steps/models/write.js'
-import { patchGuardrails } from '../steps/optimization/patch-guardrails.js'
-import { reconcileTools } from '../steps/optimization/detect.js'
-import { writeHarnessConfig } from '../steps/metadata/index.js'
-import { info } from '../utils/exec.js'
-import { exit } from '../utils/process.js'
-
-const SIMPLE_ENGLISH_ENTRY = {
-  source: 'AminBlg/SimpleEnglish',
-  sourceType: 'github',
-  skillPath: 'skills/simple-english/SKILL.md',
-}
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
-const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
-
-function resolvePlatform(value, fallback = 'github') {
-  return VALID_PLATFORMS.has(value) ? value : fallback
-}
-
-export async function runUpdate() {
-  const saved = await readHarnessConfig()
-  if (!saved?.platform) {
-    console.log(chalk.red('No agent-harness config found. Run the wizard first.'))
-    exit(1)
-    return
-  }
-
-  const backlogPlatform = resolvePlatform(saved.platform.backlog)
-  const repoPlatform = resolvePlatform(saved.platform.repo)
-
-  const ctx = {
-    hasDesign: !!saved.preexisting?.design,
-    hasArchitecture: !!saved.preexisting?.architecture,
-    hasOpenspec: !!saved.preexisting?.openspec,
-    sourceMode: saved.source?.mode ?? 'current',
-    sourceRoots: Array.isArray(saved.source?.roots) ? saved.source.roots : [],
-    maxConcurrentAgents: saved.agents?.maxConcurrent ?? 3,
-    installScope: 'local',
-    skipSkills: false,
-    updateMode: true,
-  }
-
-  console.log()
-  console.log(chalk.bold('Updating project from saved config'))
-  console.log(chalk.dim(`  backlog: ${backlogPlatform}  repo: ${repoPlatform}  agents: ${ctx.maxConcurrentAgents}`))
-  console.log()
-
-  await copyContentStep({ backlogPlatform, repoPlatform }, ctx)
-
-  if (saved.models) {
-    await writeModelsToConfigs({ buildModel: saved.models.build })
-  }
-
-  // Reconcile against what is actually installed before patching guardrails.
-  // The saved flags drift: a project can be running the codegraph and
-  // agentmemory MCP servers with harness.json still recording them as false,
-  // and patchGuardrails would then empty those marker sections and strip the
-  // guidance for tools the project genuinely uses.
-  const tools = reconcileTools(saved.tools ?? {})
-  const gained = Object.keys(tools).filter(k => tools[k] && !saved.tools?.[k])
-  if (gained.length > 0) {
-    info(`Detected installed tools missing from harness.json: ${gained.join(', ')}`)
-  }
-
-  await migrateCavemanSkill(tools)
-  await patchGuardrails({
-    rtk: !!tools.rtk,
-    simpleEnglish: !!(tools.simpleEnglish || tools.caveman),
-    codegraph: !!tools.codegraph,
-    memory: !!tools.memory,
-    humanizer: !!tools.humanizer,
-  })
-
-  await writeHarnessConfig({
-    ...ctx,
-    backlogPlatform,
-    repoPlatform,
-    maxConcurrentAgents: ctx.maxConcurrentAgents,
-    planModel: saved.models?.plan ?? null,
-    buildModel: saved.models?.build ?? null,
-    fastModel: saved.models?.fast ?? null,
-    optionalTools: tools,
-  })
-
-  console.log()
-  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
-  console.log(chalk.bold.green('  Update complete!'))
-  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
-  console.log()
-}
-
-async function migrateCavemanSkill(tools) {
-  if (!tools.caveman || tools.simpleEnglish) return
-
-  const lockPath = path.join(process.cwd(), 'skills-lock.json')
-  const lock = await fse.readJson(lockPath).catch(() => ({ version: 1, skills: {} }))
-  lock.skills ??= {}
-  delete lock.skills.caveman
-  lock.skills['simple-english'] = SIMPLE_ENGLISH_ENTRY
-  await fse.writeJson(lockPath, lock, { spaces: 2 })
-  await fse.remove(path.join(process.cwd(), '.agents', 'skills', 'caveman'))
-
-  const result = await execa('npx', ['skills', 'experimental_install', '--yes'], {
-    cwd: process.cwd(),
-    reject: false,
-    stdio: 'pipe',
-    timeout: 600000,
-  })
-  if (result.exitCode !== 0) {
-    console.log(chalk.yellow('Simple English was added to skills-lock.json but could not be installed. Run npx skills experimental_install --yes.'))
-  }
-}
+import chalk from 'chalk'
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import path from 'node:path'
+import { fileURLToPath } from 'url'
+import { readHarnessConfig } from './shared.js'
+import { copyContentStep } from '../steps/copy/index.js'
+import { writeModelsToConfigs } from '../steps/models/write.js'
+import { patchGuardrails } from '../steps/optimization/patch-guardrails.js'
+import { reconcileTools } from '../steps/optimization/detect.js'
+import { writeHarnessConfig } from '../steps/metadata/index.js'
+import { info } from '../utils/exec.js'
+import { exit } from '../utils/process.js'
+
+const SIMPLE_ENGLISH_ENTRY = {
+  source: 'AminBlg/SimpleEnglish',
+  sourceType: 'github',
+  skillPath: 'skills/simple-english/SKILL.md',
+}
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url))
+const platformsPreset = await fse.readJson(path.resolve(__dirname, '../presets/platforms.json'))
+const VALID_PLATFORMS = new Set(platformsPreset.map(p => p.value))
+
+function resolvePlatform(value, fallback = 'github') {
+  return VALID_PLATFORMS.has(value) ? value : fallback
+}
+
+export async function runUpdate() {
+  const saved = await readHarnessConfig()
+  if (!saved?.platform) {
+    console.log(chalk.red('No agent-harness config found. Run the wizard first.'))
+    exit(1)
+    return
+  }
+
+  const backlogPlatform = resolvePlatform(saved.platform.backlog)
+  const repoPlatform = resolvePlatform(saved.platform.repo)
+
+  const ctx = {
+    hasDesign: !!saved.preexisting?.design,
+    hasArchitecture: !!saved.preexisting?.architecture,
+    hasOpenspec: !!saved.preexisting?.openspec,
+    sourceMode: saved.source?.mode ?? 'current',
+    sourceRoots: Array.isArray(saved.source?.roots) ? saved.source.roots : [],
+    maxConcurrentAgents: saved.agents?.maxConcurrent ?? 3,
+    installScope: 'local',
+    skipSkills: false,
+    updateMode: true,
+  }
+
+  console.log()
+  console.log(chalk.bold('Updating project from saved config'))
+  console.log(chalk.dim(`  backlog: ${backlogPlatform}  repo: ${repoPlatform}  agents: ${ctx.maxConcurrentAgents}`))
+  console.log()
+
+  await copyContentStep({ backlogPlatform, repoPlatform }, ctx)
+
+  if (saved.models) {
+    await writeModelsToConfigs({ buildModel: saved.models.build, keepExisting: true })
+  }
+
+  // Reconcile against what is actually installed before patching guardrails.
+  // The saved flags drift: a project can be running the codegraph and
+  // agentmemory MCP servers with harness.json still recording them as false,
+  // and patchGuardrails would then empty those marker sections and strip the
+  // guidance for tools the project genuinely uses.
+  const tools = reconcileTools(saved.tools ?? {})
+  const gained = Object.keys(tools).filter(k => tools[k] && !saved.tools?.[k])
+  if (gained.length > 0) {
+    info(`Detected installed tools missing from harness.json: ${gained.join(', ')}`)
+  }
+
+  await migrateCavemanSkill(tools)
+  await patchGuardrails({
+    rtk: !!tools.rtk,
+    simpleEnglish: !!(tools.simpleEnglish || tools.caveman),
+    codegraph: !!tools.codegraph,
+    memory: !!tools.memory,
+    humanizer: !!tools.humanizer,
+  })
+
+  await writeHarnessConfig({
+    ...ctx,
+    backlogPlatform,
+    repoPlatform,
+    maxConcurrentAgents: ctx.maxConcurrentAgents,
+    planModel: saved.models?.plan ?? null,
+    buildModel: saved.models?.build ?? null,
+    fastModel: saved.models?.fast ?? null,
+    optionalTools: tools,
+  })
+
+  console.log()
+  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
+  console.log(chalk.bold.green('  Update complete!'))
+  console.log(chalk.bold.green('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━'))
+  console.log()
+}
+
+async function migrateCavemanSkill(tools) {
+  if (!tools.caveman || tools.simpleEnglish) return
+
+  const lockPath = path.join(process.cwd(), 'skills-lock.json')
+  const lock = await fse.readJson(lockPath).catch(() => ({ version: 1, skills: {} }))
+  lock.skills ??= {}
+  delete lock.skills.caveman
+  lock.skills['simple-english'] = SIMPLE_ENGLISH_ENTRY
+  await fse.writeJson(lockPath, lock, { spaces: 2 })
+  await fse.remove(path.join(process.cwd(), '.agents', 'skills', 'caveman'))
+
+  const result = await execa('npx', ['skills', 'experimental_install', '--yes'], {
+    cwd: process.cwd(),
+    reject: false,
+    stdio: 'pipe',
+    timeout: 600000,
+  })
+  if (result.exitCode !== 0) {
+    console.log(chalk.yellow('Simple English was added to skills-lock.json but could not be installed. Run npx skills experimental_install --yes.'))
+  }
+}

package/cli/steps/metadata/index.js

@@ -1,112 +1,115 @@
-import { execa } from 'execa'
-import fse from 'fs-extra'
-import path from 'path'
-import { createRequire } from 'node:module'
-import { header, success, warn } from '../../utils/exec.js'
-import { configPath } from '../../utils/paths.js'
-
-const require = createRequire(import.meta.url)
-const { version: harnessVersion } = require('../../../package.json')
-
-function clampConcurrency(n) {
-  const v = Number(n)
-  if (!Number.isFinite(v)) return 3
-  return Math.min(5, Math.max(1, Math.round(v)))
-}
-
-async function detectOpencodeVersion() {
-  try {
-    const result = await execa('opencode', ['--version'], { reject: false })
-    if (result.exitCode !== 0) return null
-    const output = (result.stdout || result.stderr || '').trim()
-    return output || null
-  } catch {
-    return null
-  }
-}
-
-export async function writeHarnessConfig(data) {
-  header('Step 10, Writing onboarding metadata')
-
-  const opencodeVersion = await detectOpencodeVersion()
-  const cwd = data.cwd ?? process.cwd()
-  const target = configPath(cwd)
-
-  // Read the existing file so load-bearing config (models, maxConcurrent)
-  // is preserved across metadata refreshes when not explicitly provided.
-  const existing = await fse.readJson(target).catch(() => null)
-
-  const selectedModels = Object.fromEntries(
-    Object.entries({
-      plan: data.planModel,
-      build: data.buildModel,
-      fast: data.fastModel,
-    }).filter(([, value]) => value)
-  )
-  const models = Object.keys(selectedModels).length > 0
-    ? { ...existing?.models, ...selectedModels }
-    : existing?.models ?? null
-  const maxConcurrent = clampConcurrency(data.maxConcurrentAgents ?? existing?.agents?.maxConcurrent ?? 3)
-
-  const optionalTools = data.optionalTools ?? existing?.tools ?? {}
-  const { caveman: _caveman, ...existingTools } = existing?.tools ?? {}
-  const tools = {
-    rtk: !!(optionalTools.rtk?.optedIn ?? optionalTools.rtk),
-    quota: !!(optionalTools.quota?.optedIn ?? optionalTools.quota),
-    simpleEnglish: !!(
-      optionalTools.simpleEnglish?.optedIn ?? optionalTools.simpleEnglish ??
-      optionalTools.caveman?.optedIn ?? optionalTools.caveman
-    ),
-    codegraph: !!(optionalTools.codegraph?.optedIn ?? optionalTools.codegraph),
-    memory: !!(optionalTools.memory?.optedIn ?? optionalTools.memory),
-    // humanizer was missing here, so a selection never survived a metadata
-    // write and patchGuardrails always saw it as off, leaving its marker
-    // section empty however the project was actually set up.
-    humanizer: !!(optionalTools.humanizer?.optedIn ?? optionalTools.humanizer),
-  }
-
-  const payload = {
-    ...existing,
-    version: 2,
-    generatedAt: new Date().toISOString(),
-    harnessVersion,
-    opencodeVersion,
-
-    platform: {
-      ...existing?.platform,
-      backlog: data.backlogPlatform ?? 'none',
-      repo: data.repoPlatform ?? 'none',
-    },
-
-    ...(models ? { models } : {}),
-
-    agents: {
-      ...existing?.agents,
-      maxConcurrent,
-    },
-
-    source: {
-      ...existing?.source,
-      mode: data.sourceMode ?? 'current',
-      roots: data.sourceRoots ?? [],
-    },
-
-    tools: { ...existingTools, ...tools },
-
-    preexisting: {
-      ...existing?.preexisting,
-      design: !!data.hasDesign,
-      architecture: !!data.hasArchitecture,
-      openspec: !!data.hasOpenspec,
-    },
-  }
-
-  try {
-    await fse.ensureDir(path.dirname(target))
-    await fse.writeJson(target, payload, { spaces: 2 })
-    success('Wrote .opencode/harness.json')
-    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
-  } catch (err) {
-    warn(`Could not write onboarding metadata: ${err.message}`)
-  }
-}
+import { execa } from 'execa'
+import fse from 'fs-extra'
+import path from 'path'
+import { createRequire } from 'node:module'
+import { header, success, warn } from '../../utils/exec.js'
+import { configPath } from '../../utils/paths.js'
+
+const require = createRequire(import.meta.url)
+const { version: harnessVersion } = require('../../../package.json')
+
+function clampConcurrency(n) {
+  const v = Number(n)
+  if (!Number.isFinite(v)) return 3
+  return Math.min(5, Math.max(1, Math.round(v)))
+}
+
+async function detectOpencodeVersion() {
+  try {
+    const result = await execa('opencode', ['--version'], { reject: false })
+    if (result.exitCode !== 0) return null
+    // `opencode --version` prints "opencode v2.0.22" on some builds and "2.0.22" on others;
+    // record the bare version so the field does not churn between machines.
+    const output = (result.stdout || result.stderr || '').trim()
+    return output.match(/\d+\.\d+\.\d+[\w.-]*/)?.[0] ?? (output || null)
+  } catch {
+    return null
+  }
+}
+
+export async function writeHarnessConfig(data) {
+  header('Step 10, Writing onboarding metadata')
+
+  const opencodeVersion = await detectOpencodeVersion()
+  const cwd = data.cwd ?? process.cwd()
+  const target = configPath(cwd)
+
+  // Read the existing file so load-bearing config (models, maxConcurrent)
+  // is preserved across metadata refreshes when not explicitly provided.
+  const existing = await fse.readJson(target).catch(() => null)
+
+  const selectedModels = Object.fromEntries(
+    Object.entries({
+      plan: data.planModel,
+      build: data.buildModel,
+      fast: data.fastModel,
+    }).filter(([, value]) => value)
+  )
+  const models = Object.keys(selectedModels).length > 0
+    ? { ...existing?.models, ...selectedModels }
+    : existing?.models ?? null
+  const maxConcurrent = clampConcurrency(data.maxConcurrentAgents ?? existing?.agents?.maxConcurrent ?? 3)
+
+  const optionalTools = data.optionalTools ?? existing?.tools ?? {}
+  const { caveman: _caveman, ...existingTools } = existing?.tools ?? {}
+  const tools = {
+    rtk: !!(optionalTools.rtk?.optedIn ?? optionalTools.rtk),
+    quota: !!(optionalTools.quota?.optedIn ?? optionalTools.quota),
+    simpleEnglish: !!(
+      optionalTools.simpleEnglish?.optedIn ?? optionalTools.simpleEnglish ??
+      optionalTools.caveman?.optedIn ?? optionalTools.caveman
+    ),
+    codegraph: !!(optionalTools.codegraph?.optedIn ?? optionalTools.codegraph),
+    memory: !!(optionalTools.memory?.optedIn ?? optionalTools.memory),
+    // humanizer was missing here, so a selection never survived a metadata
+    // write and patchGuardrails always saw it as off, leaving its marker
+    // section empty however the project was actually set up.
+    humanizer: !!(optionalTools.humanizer?.optedIn ?? optionalTools.humanizer),
+  }
+
+  const payload = {
+    ...existing,
+    // Never lower a schema version a repository already records.
+    version: Math.max(2, Number(existing?.version) || 0),
+    generatedAt: new Date().toISOString(),
+    harnessVersion,
+    opencodeVersion,
+
+    platform: {
+      ...existing?.platform,
+      backlog: data.backlogPlatform ?? 'none',
+      repo: data.repoPlatform ?? 'none',
+    },
+
+    ...(models ? { models } : {}),
+
+    agents: {
+      ...existing?.agents,
+      maxConcurrent,
+    },
+
+    source: {
+      ...existing?.source,
+      mode: data.sourceMode ?? 'current',
+      roots: data.sourceRoots ?? [],
+    },
+
+    tools: { ...existingTools, ...tools },
+
+    preexisting: {
+      ...existing?.preexisting,
+      design: !!data.hasDesign,
+      architecture: !!data.hasArchitecture,
+      openspec: !!data.hasOpenspec,
+    },
+  }
+
+  try {
+    await fse.ensureDir(path.dirname(target))
+    await fse.writeJson(target, payload, { spaces: 2 })
+    success('Wrote .opencode/harness.json')
+    if (!opencodeVersion) warn('Could not detect opencode version, saved as null')
+  } catch (err) {
+    warn(`Could not write onboarding metadata: ${err.message}`)
+  }
+}

package/cli/steps/models/write.js

@@ -1,34 +1,38 @@
-import fse from 'fs-extra'
-import { applyEdits, modify, parse } from 'jsonc-parser'
-import path from 'path'
-import { success, warn } from '../../utils/exec.js'
-
-/**
- * Writes the selected build model as the default model in opencode.jsonc.
- * Per-tier models (plan/build/fast) are recorded in harness.json (models)
- * by the metadata step, then consumed by the agent-variant generator and /plan-apply.
- *
- * opencode tolerates JSONC (comments, trailing commas) in its config, so this
- * must not use readJson/writeJson: a strict parse crashes on valid configs and
- * a strict rewrite deletes user comments. jsonc-parser edits only the `model`
- * key and preserves the rest of the file byte-for-byte.
- */
-export async function writeModelsToConfigs({ buildModel, cwd = process.cwd() }) {
-  const opencodeJsonPath = path.join(cwd, 'opencode.jsonc')
-  if (!await fse.pathExists(opencodeJsonPath)) return
-
-  const text = await fse.readFile(opencodeJsonPath, 'utf-8')
-  const errors = []
-  parse(text, errors)
-  if (errors.length > 0) {
-    warn('opencode.jsonc could not be parsed: leaving it untouched. Set "model" manually.')
-    return
-  }
-
-  // `undefined` removes the key; a string value sets it.
-  const edits = modify(text, ['model'], buildModel || undefined, {
-    formattingOptions: { insertSpaces: true, tabSize: 2 },
-  })
-  await fse.writeFile(opencodeJsonPath, applyEdits(text, edits), 'utf-8')
-  if (buildModel) success(`default model -> ${buildModel} (written to opencode.jsonc)`)
-}
+import fse from 'fs-extra'
+import { applyEdits, modify, parse } from 'jsonc-parser'
+import path from 'path'
+import { success, warn } from '../../utils/exec.js'
+
+/**
+ * Writes the selected build model as the default model in opencode.jsonc.
+ * Per-tier models (plan/build/fast) are recorded in harness.json (models)
+ * by the metadata step, then consumed by the agent-variant generator and /plan-apply.
+ *
+ * opencode tolerates JSONC (comments, trailing commas) in its config, so this
+ * must not use readJson/writeJson: a strict parse crashes on valid configs and
+ * a strict rewrite deletes user comments. jsonc-parser edits only the `model`
+ * key and preserves the rest of the file byte-for-byte.
+ */
+export async function writeModelsToConfigs({ buildModel, keepExisting = false, cwd = process.cwd() }) {
+  const opencodeJsonPath = path.join(cwd, 'opencode.jsonc')
+  if (!await fse.pathExists(opencodeJsonPath)) return
+
+  const text = await fse.readFile(opencodeJsonPath, 'utf-8')
+  const errors = []
+  parse(text, errors)
+  if (errors.length > 0) {
+    warn('opencode.jsonc could not be parsed: leaving it untouched. Set "model" manually.')
+    return
+  }
+
+  // A model the repository already chose is its decision, not the wizard's: `update` used to
+  // reset a deliberately pinned default to the build tier in every consumer it ran in.
+  if (keepExisting && typeof parse(text)?.model === 'string') return
+
+  // `undefined` removes the key; a string value sets it.
+  const edits = modify(text, ['model'], buildModel || undefined, {
+    formattingOptions: { insertSpaces: true, tabSize: 2 },
+  })
+  await fse.writeFile(opencodeJsonPath, applyEdits(text, edits), 'utf-8')
+  if (buildModel) success(`default model -> ${buildModel} (written to opencode.jsonc)`)
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plainconceptsplatform/agent-harness",
-  "version": "3.1.0",
+  "version": "3.1.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",
@@ -42,7 +42,6 @@
   "engines": {
     "node": ">=18"
   },
-  "packageManager": "pnpm@11.10.0",
   "dependencies": {
     "@inquirer/prompts": "^7.0.0",
     "chalk": "^5.0.0",
@@ -51,17 +50,6 @@
     "jsonc-parser": "^3.3.1",
     "ora": "^8.0.0"
   },
-  "scripts": {
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
-    "lint:md": "markdownlint-cli2 \"**/*.md\"",
-    "lint:md:fix": "markdownlint-cli2 --fix \"**/*.md\"",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "prepublishOnly": "npm run lint && npm run lint:md && npm run test",
-    "release:dry": "pnpm publish --dry-run --no-git-checks --access public",
-    "release": "pnpm publish --no-git-checks --access public"
-  },
   "devDependencies": {
     "@eslint/js": "^9.0.0",
     "eslint": "^9.0.0",
@@ -71,5 +59,15 @@
   },
   "vitest": {
     "environment": "node"
+  },
+  "scripts": {
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix",
+    "lint:md": "markdownlint-cli2 \"**/*.md\"",
+    "lint:md:fix": "markdownlint-cli2 --fix \"**/*.md\"",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "release:dry": "pnpm publish --dry-run --no-git-checks --access public",
+    "release": "pnpm publish --no-git-checks --access public"
   }
-}
+}

package/skills/pc-repo-help/SKILL.md

@@ -58,7 +58,7 @@ Display the following reference to the user exactly as written. Do not summarize
 
 **`/make-guardrails`**: Generate a `pc-guardrails-project` skill from `ARCHITECTURE.md` and project config files. Extracts concrete rules (architecture boundaries, naming, code style, testing, git workflow) that all agents must follow. Updates every `*-engineer.md` to load the skill.
 
-**`/repo-verify`**: Write the verification plan for the current branch's change: a journey through the new functionality as a fenced YAML block in `verification-plan.md`, stored with the change. The evidence workflow from `@plainconceptsplatform/workflows` replays it in headless Chromium when the pull request merges and archives one screenshot per waypoint on it. Backend-only changes are traced through to the frontend when their contract is consumed there; a `not-applicable` stub is written when no UI surface is reachable. Does not run checks or take screenshots. Runs automatically in `/plan-goal`.
+**`/repo-verify`**: Write the verification plan for the current branch's change: a journey through the new functionality as a fenced YAML block in `verification-plan.md`, stored with the change. The evidence workflow from `@plainconceptsplatform/workflows` replays it in headless Chromium when the pull request merges and archives one screenshot per waypoint in an `evidence/` folder beside the plan. Backend-only changes are traced through to the frontend when their contract is consumed there; a `not-applicable` stub is written when no UI surface is reachable. Does not run checks or take screenshots. Runs automatically in `/plan-goal`.
 
 **`/make-user-model <tier> <model>`**: Set the model for a tier (`plan`, `build`, or `fast`). Writes to `.opencode/harness.json` (`models`). Use `user` prefix for a personal override: `/make-user-model user fast opencode/big-pickle`. Use a model id or `current` for the active session model. Restart opencode for the `pc-subagent-tiers` plugin to rebuild tier agents.
 

package/skills/pc-repo-verify/SKILL.md

@@ -8,11 +8,11 @@ license: MIT
 
 Write the verification plan for the current branch's change. The checks gate lives in `pc-plan-apply` step 10 (lint, typecheck, test, build). This skill produces the reproduction plan instead: a journey through the new functionality with one screenshot per waypoint.
 
-The plan is executed by a machine, not read by one. When the pull request merges, the evidence workflow (`evidence.yml` from `@plainconceptsplatform/workflows`) checks out the pull request head, starts the app, replays every step below in headless Chromium with no model in the loop, and archives one screenshot per waypoint on the pull request. A step it cannot follow is a screenshot of the wrong screen. Write every step so it can be followed literally from a freshly started app.
+The plan is executed by a machine, not read by one. When the pull request merges, the evidence workflow (`evidence.yml` from `@plainconceptsplatform/workflows`) checks out the pull request head, starts the app, replays every step below in headless Chromium with no model in the loop, and archives one screenshot per waypoint in an `evidence/` folder beside this plan on the default branch, shown in one comment on the pull request. A step it cannot follow is a screenshot of the wrong screen. Write every step so it can be followed literally from a freshly started app.
 
 This skill never launches a browser, never takes a screenshot, never starts a server. It only reads the change and writes a plan file.
 
-Work only on the current branch: do not switch branches, push, create pull requests, or contact external platforms. Write the plan into the change directory; it is carried into the archive by `pc-plan-archive`.
+Work only on the current branch: do not switch branches, push, create pull requests, or contact external platforms. Write the plan into the change directory; it is carried into the archive by `pc-plan-archive`. Never create or edit an `evidence/` folder: the workflow writes it beside the archived plan after the merge.
 
 ## Project environment