@plainconceptsplatform/agent-harness 3.1.1 → 3.2.0

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, 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.'))
-  }
-}
+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/fragments/archive/az.md

@@ -1,97 +1,97 @@
-2. **Find the oldest change with a completed PR**
-
-   List unarchived changes (top-level only, excludes `archive/`):
-
-   ```bash
-   find "$REPO_ROOT/openspec/changes" -mindepth 1 -maxdepth 1 -type d -not -name 'archive' | sort
-   ```
-
-   If empty, report a blocker and stop.
-
-   List completed PRs:
-
-   ```bash
-   az repos pr list --repository {repo} --status completed --query "sort_by(@, &closedDate)[].{name:title,sourceRefName:sourceRefName,closedDate:closedDate,pullRequestId:pullRequestId}"
-   ```
-
-   Match each change to a completed PR using its ID and slug as search hints:
-   - No match → skip (record as blocked: `no merged PR found`).
-   - One match → eligible.
-   - Multiple matches → ask the user which PR belongs to that change.
-
-   If nothing is eligible, report a blocker and stop. Otherwise select the eligible change with the **oldest** PR `closedDate` as the candidate.
-
-3. **Confirm the candidate**
-
-   Show the candidate (ID, title, PR ID, merged date) and any blocked changes, then ask:
-
-   ```text
-   Oldest unarchived merged change found:
-     ID: {change-id}
-     Title: {title from resolved PR}
-     PR ID: {pullRequestId}
-     Merged: {closedDate}
-
-   Proceed with archiving? [yes/no]
-   ```
-
-   Stop if the user does not confirm.
-
-4. **Archive the change**
-
-   ```bash
-   git checkout -b archive/{change-id}
-   ```
-
-   Load `@openspec-archive-change` skill and follow it to archive the change.
-
-5. **Update docs**
-
-   Compare the archived change's specs against `ARCHITECTURE.md` and `DESIGN.md`. If updates are needed, show them and get user approval before applying.
-
-6. **Create the archive PR**
-
-   ```bash
-   # Stage specific paths only: a shared tree may hold another agent's or a
-   # person's edits, and -A commits them under this message.
-   git add openspec/changes/   # plus ARCHITECTURE.md / DESIGN.md if step 5 updated them
-   git commit -m "archive: {title} ({change-id})"
-   git push origin archive/{change-id}
-
-   az repos pr create \
-     --repository {repo} \
-     --source-branch refs/heads/archive/{change-id} \
-     --target-branch "refs/heads/$DEFAULT_BRANCH" \
-     --title "archive: {title} ({change-id})" \
-     --description "Archive SDD artifacts for {change-id} after merge of {sourceRefName}." \
-     --auto-complete
-   ```
-
-   If work was stashed in step 1, restore it after the PR is created unless the user opts out.
-
-7. **Report**
-
-   Display:
-
-   ```text
-   Archive complete
-
-     Change ID: {change-id}
-     Title: {title}
-     Original PR: {original-pr-link}
-     Archive PR: {archive-pr-link}
-
-     Documentation updates:
-     - ARCHITECTURE.md: {count} changes applied
-     - DESIGN.md: {count} changes applied
-   ```
-
-## Rules
-
-- All OpenSpec paths resolve from `git rev-parse --show-toplevel`. Never use `/openspec/...`.
-- Only process top-level directories in `$REPO_ROOT/openspec/changes/`; exclude `archive/`.
-- Use change ID and slug only as search hints; do not assume the source branch name.
-- The oldest eligible merged change is the only candidate: never ask the user which change to archive (but do ask which PR if multiple match one change).
-- Never proceed if the selected PR is not completed.
-- Never use browser tools or direct web requests for Azure DevOps. Use `az` CLI only.
-- Never invent or guess PR, branch, or merge metadata.
+2. **Find the oldest change with a completed PR**
+
+   List unarchived changes (top-level only, excludes `archive/`):
+
+   ```bash
+   find "$REPO_ROOT/openspec/changes" -mindepth 1 -maxdepth 1 -type d -not -name 'archive' | sort
+   ```
+
+   If empty, report a blocker and stop.
+
+   List completed PRs:
+
+   ```bash
+   az repos pr list --repository {repo} --status completed --query "sort_by(@, &closedDate)[].{name:title,sourceRefName:sourceRefName,closedDate:closedDate,pullRequestId:pullRequestId}"
+   ```
+
+   Match each change to a completed PR using its ID and slug as search hints:
+   - No match → skip (record as blocked: `no merged PR found`).
+   - One match → eligible.
+   - Multiple matches → ask the user which PR belongs to that change.
+
+   If nothing is eligible, report a blocker and stop. Otherwise select the eligible change with the **oldest** PR `closedDate` as the candidate.
+
+3. **Confirm the candidate**
+
+   Show the candidate (ID, title, PR ID, merged date) and any blocked changes, then ask:
+
+   ```text
+   Oldest unarchived merged change found:
+     ID: {change-id}
+     Title: {title from resolved PR}
+     PR ID: {pullRequestId}
+     Merged: {closedDate}
+
+   Proceed with archiving? [yes/no]
+   ```
+
+   Stop if the user does not confirm.
+
+4. **Archive the change**
+
+   ```bash
+   git checkout -b archive/{change-id}
+   ```
+
+   Load `@openspec-archive-change` skill and follow it to archive the change.
+
+5. **Update docs**
+
+   Compare the archived change's specs against `ARCHITECTURE.md` and `DESIGN.md`. If updates are needed, show them and get user approval before applying.
+
+6. **Create the archive PR**
+
+   ```bash
+   # Stage specific paths only: a shared tree may hold another agent's or a
+   # person's edits, and -A commits them under this message.
+   git add openspec/changes/   # plus ARCHITECTURE.md / DESIGN.md if step 5 updated them
+   git commit -m "archive: {title} ({change-id})"
+   git push origin archive/{change-id}
+
+   az repos pr create \
+     --repository {repo} \
+     --source-branch refs/heads/archive/{change-id} \
+     --target-branch "refs/heads/$DEFAULT_BRANCH" \
+     --title "archive: {title} ({change-id})" \
+     --description "Archive SDD artifacts for {change-id} after merge of {sourceRefName}." \
+     --auto-complete
+   ```
+
+   If work was stashed in step 1, restore it after the PR is created unless the user opts out.
+
+7. **Report**
+
+   Display:
+
+   ```text
+   Archive complete
+
+     Change ID: {change-id}
+     Title: {title}
+     Original PR: {original-pr-link}
+     Archive PR: {archive-pr-link}
+
+     Documentation updates:
+     - ARCHITECTURE.md: {count} changes applied
+     - DESIGN.md: {count} changes applied
+   ```
+
+## Rules
+
+- All OpenSpec paths resolve from `git rev-parse --show-toplevel`. Never use `/openspec/...`.
+- Only process top-level directories in `$REPO_ROOT/openspec/changes/`; exclude `archive/`.
+- Use change ID and slug only as search hints; do not assume the source branch name.
+- The oldest eligible merged change is the only candidate: never ask the user which change to archive (but do ask which PR if multiple match one change).
+- Never proceed if the selected PR is not completed.
+- Never use browser tools or direct web requests for Azure DevOps. Use `az` CLI only.
+- Never invent or guess PR, branch, or merge metadata.

package/cli/fragments/archive/gh.md

@@ -1,96 +1,96 @@
-2. **Find the oldest change with a completed PR**
-
-   List unarchived changes (top-level only, excludes `archive/`):
-
-   ```bash
-   find "$REPO_ROOT/openspec/changes" -mindepth 1 -maxdepth 1 -type d -not -name 'archive' | sort
-   ```
-
-   If empty, report a blocker and stop.
-
-   List completed PRs:
-
-   ```bash
-   gh pr list --repo {owner}/{repo} --state merged --json title,headRefName,mergedAt,number --jq 'sort_by(.mergedAt) | .[] | {name: .title, sourceRefName: .headRefName, mergedAt: .mergedAt, pullRequestId: .number}'
-   ```
-
-   Match each change to a completed PR using its ID and slug as search hints:
-   - No match → skip (record as blocked: `no merged PR found`).
-   - One match → eligible.
-   - Multiple matches → ask the user which PR belongs to that change.
-
-   If nothing is eligible, report a blocker and stop. Otherwise select the eligible change with the **oldest** PR `mergedAt` as the candidate.
-
-3. **Confirm the candidate**
-
-   Show the candidate (ID, title, PR ID, merged date) and any blocked changes, then ask:
-
-   ```text
-   Oldest unarchived merged change found:
-     ID: {change-id}
-     Title: {title from resolved PR}
-     PR ID: {pullRequestId}
-     Merged: {mergedAt}
-
-   Proceed with archiving? [yes/no]
-   ```
-
-   Stop if the user does not confirm.
-
-4. **Archive the change**
-
-   ```bash
-   git checkout -b archive/{change-id}
-   ```
-
-   Load `@openspec-archive-change` skill and follow it to archive the change.
-
-5. **Update docs**
-
-   Compare the archived change's specs against `ARCHITECTURE.md` and `DESIGN.md`. If updates are needed, show them and get user approval before applying.
-
-6. **Create the archive PR**
-
-   ```bash
-   # Stage specific paths only: a shared tree may hold another agent's or a
-   # person's edits, and -A commits them under this message.
-   git add openspec/changes/   # plus ARCHITECTURE.md / DESIGN.md if step 5 updated them
-   git commit -m "archive: {title} ({change-id})"
-   git push origin archive/{change-id}
-
-   gh pr create \
-      --repo {owner}/{repo} \
-      --base "$DEFAULT_BRANCH" \
-      --head archive/{change-id} \
-      --title "archive: {title} ({change-id})" \
-      --body "Archive SDD artifacts for {change-id} after merge."
-   ```
-
-   If work was stashed in step 1, restore it after the PR is created unless the user opts out.
-
-7. **Report**
-
-   Display:
-
-   ```text
-   Archive complete
-
-     Change ID: {change-id}
-     Title: {title}
-     Original PR: {original-pr-link}
-     Archive PR: {archive-pr-link}
-
-     Documentation updates:
-     - ARCHITECTURE.md: {count} changes applied
-     - DESIGN.md: {count} changes applied
-   ```
-
-## Rules
-
-- All OpenSpec paths resolve from `git rev-parse --show-toplevel`. Never use `/openspec/...`.
-- Only process top-level directories in `$REPO_ROOT/openspec/changes/`; exclude `archive/`.
-- Use change ID and slug only as search hints; do not assume the source branch name.
-- The oldest eligible merged change is the only candidate: never ask the user which change to archive (but do ask which PR if multiple match one change).
-- Never proceed if the selected PR is not completed.
-- Never use browser tools or direct web requests for GitHub. Use `gh` CLI only.
-- Never invent or guess PR, branch, or merge metadata.
+2. **Find the oldest change with a completed PR**
+
+   List unarchived changes (top-level only, excludes `archive/`):
+
+   ```bash
+   find "$REPO_ROOT/openspec/changes" -mindepth 1 -maxdepth 1 -type d -not -name 'archive' | sort
+   ```
+
+   If empty, report a blocker and stop.
+
+   List completed PRs:
+
+   ```bash
+   gh pr list --repo {owner}/{repo} --state merged --json title,headRefName,mergedAt,number --jq 'sort_by(.mergedAt) | .[] | {name: .title, sourceRefName: .headRefName, mergedAt: .mergedAt, pullRequestId: .number}'
+   ```
+
+   Match each change to a completed PR using its ID and slug as search hints:
+   - No match → skip (record as blocked: `no merged PR found`).
+   - One match → eligible.
+   - Multiple matches → ask the user which PR belongs to that change.
+
+   If nothing is eligible, report a blocker and stop. Otherwise select the eligible change with the **oldest** PR `mergedAt` as the candidate.
+
+3. **Confirm the candidate**
+
+   Show the candidate (ID, title, PR ID, merged date) and any blocked changes, then ask:
+
+   ```text
+   Oldest unarchived merged change found:
+     ID: {change-id}
+     Title: {title from resolved PR}
+     PR ID: {pullRequestId}
+     Merged: {mergedAt}
+
+   Proceed with archiving? [yes/no]
+   ```
+
+   Stop if the user does not confirm.
+
+4. **Archive the change**
+
+   ```bash
+   git checkout -b archive/{change-id}
+   ```
+
+   Load `@openspec-archive-change` skill and follow it to archive the change.
+
+5. **Update docs**
+
+   Compare the archived change's specs against `ARCHITECTURE.md` and `DESIGN.md`. If updates are needed, show them and get user approval before applying.
+
+6. **Create the archive PR**
+
+   ```bash
+   # Stage specific paths only: a shared tree may hold another agent's or a
+   # person's edits, and -A commits them under this message.
+   git add openspec/changes/   # plus ARCHITECTURE.md / DESIGN.md if step 5 updated them
+   git commit -m "archive: {title} ({change-id})"
+   git push origin archive/{change-id}
+
+   gh pr create \
+      --repo {owner}/{repo} \
+      --base "$DEFAULT_BRANCH" \
+      --head archive/{change-id} \
+      --title "archive: {title} ({change-id})" \
+      --body "Archive SDD artifacts for {change-id} after merge."
+   ```
+
+   If work was stashed in step 1, restore it after the PR is created unless the user opts out.
+
+7. **Report**
+
+   Display:
+
+   ```text
+   Archive complete
+
+     Change ID: {change-id}
+     Title: {title}
+     Original PR: {original-pr-link}
+     Archive PR: {archive-pr-link}
+
+     Documentation updates:
+     - ARCHITECTURE.md: {count} changes applied
+     - DESIGN.md: {count} changes applied
+   ```
+
+## Rules
+
+- All OpenSpec paths resolve from `git rev-parse --show-toplevel`. Never use `/openspec/...`.
+- Only process top-level directories in `$REPO_ROOT/openspec/changes/`; exclude `archive/`.
+- Use change ID and slug only as search hints; do not assume the source branch name.
+- The oldest eligible merged change is the only candidate: never ask the user which change to archive (but do ask which PR if multiple match one change).
+- Never proceed if the selected PR is not completed.
+- Never use browser tools or direct web requests for GitHub. Use `gh` CLI only.
+- Never invent or guess PR, branch, or merge metadata.