@plainconceptsplatform/agent-harness 3.1.1 → 3.3.0

package/cli/utils/legacy-check.js

@@ -1,30 +1,30 @@
-import fse from 'fs-extra'
-import path from 'node:path'
-import { CONFIG_FILE, LEGACY_CONFIG_FILES, OPENCODE_DIR } from './paths.js'
-
-/**
- * Detect a repo onboarded by opencode-onboard v1.x.
- *
- * v2 renamed both the state files and the `ob-` skill prefix, and it does not
- * migrate: see the rename plan. Running against a v1 repo would otherwise fail
- * silently in two places, because `patchFile` returns without logging when its
- * markers are absent and the skill sync only removes stale directories under
- * `forceOverwrite && !updateMode`. The result is a half-patched harness that
- * still reports success, so refuse the run instead.
- */
-export async function findLegacyInstall(cwd = process.cwd()) {
-  const opencodeDir = path.join(cwd, OPENCODE_DIR)
-  if (await fse.pathExists(path.join(opencodeDir, CONFIG_FILE))) return null
-
-  const found = []
-  for (const file of LEGACY_CONFIG_FILES) {
-    if (await fse.pathExists(path.join(opencodeDir, file))) found.push(`${OPENCODE_DIR}/${file}`)
-  }
-
-  const skillsDir = path.join(cwd, '.agents', 'skills')
-  const legacySkills = (await fse.readdir(skillsDir).catch(() => []))
-    .filter(entry => entry.startsWith('ob-'))
-
-  if (found.length === 0 && legacySkills.length === 0) return null
-  return { files: found, skills: legacySkills }
-}
+import fse from 'fs-extra'
+import path from 'node:path'
+import { CONFIG_FILE, LEGACY_CONFIG_FILES, OPENCODE_DIR } from './paths.js'
+
+/**
+ * Detect a repo onboarded by opencode-onboard v1.x.
+ *
+ * v2 renamed both the state files and the `ob-` skill prefix, and it does not
+ * migrate: see the rename plan. Running against a v1 repo would otherwise fail
+ * silently in two places, because `patchFile` returns without logging when its
+ * markers are absent and the skill sync only removes stale directories under
+ * `forceOverwrite && !updateMode`. The result is a half-patched harness that
+ * still reports success, so refuse the run instead.
+ */
+export async function findLegacyInstall(cwd = process.cwd()) {
+  const opencodeDir = path.join(cwd, OPENCODE_DIR)
+  if (await fse.pathExists(path.join(opencodeDir, CONFIG_FILE))) return null
+
+  const found = []
+  for (const file of LEGACY_CONFIG_FILES) {
+    if (await fse.pathExists(path.join(opencodeDir, file))) found.push(`${OPENCODE_DIR}/${file}`)
+  }
+
+  const skillsDir = path.join(cwd, '.agents', 'skills')
+  const legacySkills = (await fse.readdir(skillsDir).catch(() => []))
+    .filter(entry => entry.startsWith('ob-'))
+
+  if (found.length === 0 && legacySkills.length === 0) return null
+  return { files: found, skills: legacySkills }
+}

package/cli/utils/models-cache.js

@@ -1,58 +1,58 @@
-import fse from 'fs-extra'
-import path from 'path'
-import { parseModels } from './models-pricing.js'
-import { modelsCacheDir } from './paths.js'
-
-const CACHE_DIR = modelsCacheDir();
-const CACHE_FILE = path.join(CACHE_DIR, 'models-cache.json');
-const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 7 days
-const MODELS_URL = 'https://models.dev/api.json';
-
-async function loadCache() {
-  try {
-    if (!await fse.pathExists(CACHE_FILE)) return null;
-    const cache = await fse.readJson(CACHE_FILE);
-    if (!cache.timestamp || !cache.models) return null;
-    const age = Date.now() - cache.timestamp;
-    if (age > CACHE_TTL_MS) return null; // expired
-    return cache.models;
-  } catch {
-    return null;
-  }
-}
-
-async function saveCache(models) {
-  try {
-    await fse.ensureDir(CACHE_DIR);
-    await fse.writeJson(CACHE_FILE, { timestamp: Date.now(), models });
-  } catch {
-    // cache write failure is non-fatal
-  }
-}
-
-export async function fetchModels() {
-  // 1. Try cache first (fresh)
-  const cached = await loadCache();
-  if (cached) return { models: cached, source: 'cache' };
-
-  // 2. Try network
-  try {
-    const res = await fetch(MODELS_URL, { signal: AbortSignal.timeout(8000) });
-    if (!res.ok) throw new Error(`HTTP ${res.status}`);
-    const data = await res.json();
-    const models = parseModels(data);
-    await saveCache(models);
-    return { models, source: 'network' };
-  } catch {
-    // 3. Network failed, fall back to stale cache if available
-    try {
-      if (await fse.pathExists(CACHE_FILE)) {
-        const cache = await fse.readJson(CACHE_FILE);
-        if (cache.models?.length) return { models: cache.models, source: 'stale-cache' };
-      }
-    } catch {
-      // ignore
-    }
-    return { models: null, source: 'unavailable' };
-  }
-}
+import fse from 'fs-extra'
+import path from 'path'
+import { parseModels } from './models-pricing.js'
+import { modelsCacheDir } from './paths.js'
+
+const CACHE_DIR = modelsCacheDir();
+const CACHE_FILE = path.join(CACHE_DIR, 'models-cache.json');
+const CACHE_TTL_MS = 7 * 24 * 60 * 60 * 1000; // 7 days
+const MODELS_URL = 'https://models.dev/api.json';
+
+async function loadCache() {
+  try {
+    if (!await fse.pathExists(CACHE_FILE)) return null;
+    const cache = await fse.readJson(CACHE_FILE);
+    if (!cache.timestamp || !cache.models) return null;
+    const age = Date.now() - cache.timestamp;
+    if (age > CACHE_TTL_MS) return null; // expired
+    return cache.models;
+  } catch {
+    return null;
+  }
+}
+
+async function saveCache(models) {
+  try {
+    await fse.ensureDir(CACHE_DIR);
+    await fse.writeJson(CACHE_FILE, { timestamp: Date.now(), models });
+  } catch {
+    // cache write failure is non-fatal
+  }
+}
+
+export async function fetchModels() {
+  // 1. Try cache first (fresh)
+  const cached = await loadCache();
+  if (cached) return { models: cached, source: 'cache' };
+
+  // 2. Try network
+  try {
+    const res = await fetch(MODELS_URL, { signal: AbortSignal.timeout(8000) });
+    if (!res.ok) throw new Error(`HTTP ${res.status}`);
+    const data = await res.json();
+    const models = parseModels(data);
+    await saveCache(models);
+    return { models, source: 'network' };
+  } catch {
+    // 3. Network failed, fall back to stale cache if available
+    try {
+      if (await fse.pathExists(CACHE_FILE)) {
+        const cache = await fse.readJson(CACHE_FILE);
+        if (cache.models?.length) return { models: cache.models, source: 'stale-cache' };
+      }
+    } catch {
+      // ignore
+    }
+    return { models: null, source: 'unavailable' };
+  }
+}

package/cli/utils/models-pricing.js

@@ -1,42 +1,42 @@
-// Providers considered "canonical" for reference pricing, in priority order.
-// When a model's own provider has no cost (e.g. github-copilot shows $0),
-// we look up the same model name in these providers and attach canonicalCost.
-export const CANONICAL_PROVIDERS = ['anthropic', 'openai', 'google', 'mistral', 'meta', 'cohere'];
-
-export function parseModels(data) {
-  // Build name → canonical cost lookup from authoritative providers first
-  const canonicalCostByName = new Map();
-  for (const providerId of CANONICAL_PROVIDERS) {
-    const provider = data[providerId];
-    if (!provider?.models) continue;
-    for (const model of Object.values(provider.models)) {
-      if (!model.tool_call) continue;
-      const name = model.name;
-      if (name && model.cost?.input !== undefined && !canonicalCostByName.has(name)) {
-        canonicalCostByName.set(name, model.cost.input);
-      }
-    }
-  }
-
-  const models = [];
-  for (const [providerId, provider] of Object.entries(data)) {
-    if (!provider.models) continue;
-    for (const [modelId, model] of Object.entries(provider.models)) {
-      if (!model.tool_call) continue;
-      const name = model.name || modelId;
-      const cost = model.cost?.input;
-      const canonicalCost = canonicalCostByName.get(name);
-      models.push({
-        id: `${providerId}/${modelId}`,
-        name,
-        cost,
-        // canonicalCost: cost from the authoritative provider for this model name.
-        // Defined when cost !== canonicalCost (different provider, reseller, or $0 subscription).
-        canonicalCost: canonicalCost !== undefined && canonicalCost !== cost ? canonicalCost : undefined,
-        context: model.limit?.context,
-      });
-    }
-  }
-  models.sort((a, b) => (a.cost ?? Infinity) - (b.cost ?? Infinity));
-  return models;
-}
+// Providers considered "canonical" for reference pricing, in priority order.
+// When a model's own provider has no cost (e.g. github-copilot shows $0),
+// we look up the same model name in these providers and attach canonicalCost.
+export const CANONICAL_PROVIDERS = ['anthropic', 'openai', 'google', 'mistral', 'meta', 'cohere'];
+
+export function parseModels(data) {
+  // Build name → canonical cost lookup from authoritative providers first
+  const canonicalCostByName = new Map();
+  for (const providerId of CANONICAL_PROVIDERS) {
+    const provider = data[providerId];
+    if (!provider?.models) continue;
+    for (const model of Object.values(provider.models)) {
+      if (!model.tool_call) continue;
+      const name = model.name;
+      if (name && model.cost?.input !== undefined && !canonicalCostByName.has(name)) {
+        canonicalCostByName.set(name, model.cost.input);
+      }
+    }
+  }
+
+  const models = [];
+  for (const [providerId, provider] of Object.entries(data)) {
+    if (!provider.models) continue;
+    for (const [modelId, model] of Object.entries(provider.models)) {
+      if (!model.tool_call) continue;
+      const name = model.name || modelId;
+      const cost = model.cost?.input;
+      const canonicalCost = canonicalCostByName.get(name);
+      models.push({
+        id: `${providerId}/${modelId}`,
+        name,
+        cost,
+        // canonicalCost: cost from the authoritative provider for this model name.
+        // Defined when cost !== canonicalCost (different provider, reseller, or $0 subscription).
+        canonicalCost: canonicalCost !== undefined && canonicalCost !== cost ? canonicalCost : undefined,
+        context: model.limit?.context,
+      });
+    }
+  }
+  models.sort((a, b) => (a.cost ?? Infinity) - (b.cost ?? Infinity));
+  return models;
+}

package/cli/utils/process.js

@@ -1,3 +1,3 @@
-export function exit(code = 0) {
-  process.exit(code)
-}
+export function exit(code = 0) {
+  process.exit(code)
+}

package/cli/utils/terminal.js

@@ -1,6 +1,6 @@
-export const MARKERS = {
-  EMPTY: "  ",
-  OK_PREFIX: "✓ ",
-  WARN_PREFIX: "⚠ ",
-  ERROR_PREFIX: "✗ ",
-};
+export const MARKERS = {
+  EMPTY: "  ",
+  OK_PREFIX: "✓ ",
+  WARN_PREFIX: "⚠ ",
+  ERROR_PREFIX: "✗ ",
+};

package/cli/utils/update-manifest.js

@@ -1,75 +1,75 @@
-import crypto from 'node:crypto'
-import fse from 'fs-extra'
-import path from 'node:path'
-import { MANIFEST_FILE, OPENCODE_DIR } from './paths.js'
-
-const MANIFEST_RELATIVE_PATH = path.join(OPENCODE_DIR, MANIFEST_FILE)
-
-function normalizeRelativePath(relativePath) {
-  return relativePath.split(path.sep).join('/')
-}
-
-function hashContent(content) {
-  return crypto.createHash('sha256').update(content).digest('hex')
-}
-
-export async function hashFile(filePath) {
-  return hashContent(await fse.readFile(filePath))
-}
-
-// A file counts as untouched when it differs from what we shipped only in ways
-// the harness itself caused. Two of those, and both made the raw-bytes
-// comparison call an untouched file modified — permanently, because the flag is
-// re-derived on every update:
-//
-//   1. Marker pairs. The patchers rewrite them after the copy, so a file with
-//      any PC-* marker never matches its own source again.
-//   2. Line endings. A file that reaches a consumer's tree through git arrives
-//      CRLF on Windows while the shipped source is LF.
-//
-// This is the general form of the bug that left five repos on a stale
-// browser-automation skill through repeated updates.
-const MARKER_PAIR = /(<!-- PC-[A-Z0-9-]+-START -->)[\s\S]*?(<!-- PC-[A-Z0-9-]+-END -->)/g
-
-function comparableContent(buffer) {
-  return buffer.toString('utf-8').replace(/\r\n/g, '\n').replace(MARKER_PAIR, '$1$2')
-}
-
-export async function hashComparableFile(filePath) {
-  return hashContent(comparableContent(await fse.readFile(filePath)))
-}
-
-export async function readUpdateManifest(cwd = process.cwd()) {
-  const manifestPath = path.join(cwd, MANIFEST_RELATIVE_PATH)
-  const manifest = await fse.readJson(manifestPath).catch(() => ({ version: 1, files: {} }))
-  if (!manifest.files || typeof manifest.files !== 'object') manifest.files = {}
-  return manifest
-}
-
-export async function writeUpdateManifest(manifest, cwd = process.cwd()) {
-  const manifestPath = path.join(cwd, MANIFEST_RELATIVE_PATH)
-  await fse.ensureDir(path.dirname(manifestPath))
-  await fse.writeJson(manifestPath, manifest, { spaces: 2 })
-}
-
-export async function canUpdateManagedFile(relativePath, cwd, manifest) {
-  const normalizedPath = normalizeRelativePath(relativePath)
-  const destinationPath = path.join(cwd, relativePath)
-  if (!await fse.pathExists(destinationPath)) return true
-  const previousHash = manifest.files?.[normalizedPath]
-  if (!previousHash) return false
-  // Raw is accepted too: manifests written before this comparison existed hold
-  // raw source hashes, and re-recording them all would need an update to run
-  // first, which is the thing being unblocked.
-  return previousHash === await hashComparableFile(destinationPath)
-    || previousHash === await hashFile(destinationPath)
-}
-
-export async function recordManagedFile(manifest, relativePath, sourcePath) {
-  const files = manifest.files
-  files[normalizeRelativePath(relativePath)] = await hashComparableFile(sourcePath)
-}
-
-export function manifestPath() {
-  return MANIFEST_RELATIVE_PATH
-}
+import crypto from 'node:crypto'
+import fse from 'fs-extra'
+import path from 'node:path'
+import { MANIFEST_FILE, OPENCODE_DIR } from './paths.js'
+
+const MANIFEST_RELATIVE_PATH = path.join(OPENCODE_DIR, MANIFEST_FILE)
+
+function normalizeRelativePath(relativePath) {
+  return relativePath.split(path.sep).join('/')
+}
+
+function hashContent(content) {
+  return crypto.createHash('sha256').update(content).digest('hex')
+}
+
+export async function hashFile(filePath) {
+  return hashContent(await fse.readFile(filePath))
+}
+
+// A file counts as untouched when it differs from what we shipped only in ways
+// the harness itself caused. Two of those, and both made the raw-bytes
+// comparison call an untouched file modified — permanently, because the flag is
+// re-derived on every update:
+//
+//   1. Marker pairs. The patchers rewrite them after the copy, so a file with
+//      any PC-* marker never matches its own source again.
+//   2. Line endings. A file that reaches a consumer's tree through git arrives
+//      CRLF on Windows while the shipped source is LF.
+//
+// This is the general form of the bug that left five repos on a stale
+// browser-automation skill through repeated updates.
+const MARKER_PAIR = /(<!-- PC-[A-Z0-9-]+-START -->)[\s\S]*?(<!-- PC-[A-Z0-9-]+-END -->)/g
+
+function comparableContent(buffer) {
+  return buffer.toString('utf-8').replace(/\r\n/g, '\n').replace(MARKER_PAIR, '$1$2')
+}
+
+export async function hashComparableFile(filePath) {
+  return hashContent(comparableContent(await fse.readFile(filePath)))
+}
+
+export async function readUpdateManifest(cwd = process.cwd()) {
+  const manifestPath = path.join(cwd, MANIFEST_RELATIVE_PATH)
+  const manifest = await fse.readJson(manifestPath).catch(() => ({ version: 1, files: {} }))
+  if (!manifest.files || typeof manifest.files !== 'object') manifest.files = {}
+  return manifest
+}
+
+export async function writeUpdateManifest(manifest, cwd = process.cwd()) {
+  const manifestPath = path.join(cwd, MANIFEST_RELATIVE_PATH)
+  await fse.ensureDir(path.dirname(manifestPath))
+  await fse.writeJson(manifestPath, manifest, { spaces: 2 })
+}
+
+export async function canUpdateManagedFile(relativePath, cwd, manifest) {
+  const normalizedPath = normalizeRelativePath(relativePath)
+  const destinationPath = path.join(cwd, relativePath)
+  if (!await fse.pathExists(destinationPath)) return true
+  const previousHash = manifest.files?.[normalizedPath]
+  if (!previousHash) return false
+  // Raw is accepted too: manifests written before this comparison existed hold
+  // raw source hashes, and re-recording them all would need an update to run
+  // first, which is the thing being unblocked.
+  return previousHash === await hashComparableFile(destinationPath)
+    || previousHash === await hashFile(destinationPath)
+}
+
+export async function recordManagedFile(manifest, relativePath, sourcePath) {
+  const files = manifest.files
+  files[normalizeRelativePath(relativePath)] = await hashComparableFile(sourcePath)
+}
+
+export function manifestPath() {
+  return MANIFEST_RELATIVE_PATH
+}

package/harness/.opencode/commands/ops-backlog.md

@@ -1,10 +1,10 @@
----
-description: Create an issue in the backlog platform (GitHub, Azure DevOps, Jira) from a description.
----
-
-Backlog platform is set in `.opencode/harness.json` → `platform.backlog`. The platform-specific content below is injected by the CLI during onboarding.
-
-Input: `$ARGUMENTS` (the issue title/description)
-
-<!-- PC-PLATFORM-BACKLOG-START -->
-<!-- PC-PLATFORM-BACKLOG-END -->
+---
+description: Create an issue in the backlog platform (GitHub, Azure DevOps, Jira) from a description.
+---
+
+Backlog platform is set in `.opencode/harness.json` → `platform.backlog`. The platform-specific content below is injected by the CLI during onboarding.
+
+Input: `$ARGUMENTS` (the issue title/description)
+
+<!-- PC-PLATFORM-BACKLOG-START -->
+<!-- PC-PLATFORM-BACKLOG-END -->

package/harness/.opencode/commands/ops-review.md

@@ -1,8 +1,8 @@
----
-description: Read and triage PR review feedback. Reports what needs fixing, does not implement fixes.
----
-
-Repo platform is set in `.opencode/harness.json` → `platform.repo`. The platform-specific content below is injected by the CLI during onboarding.
-
-<!-- PC-PLATFORM-REVIEW-START -->
-<!-- PC-PLATFORM-REVIEW-END -->
+---
+description: Read and triage PR review feedback. Reports what needs fixing, does not implement fixes.
+---
+
+Repo platform is set in `.opencode/harness.json` → `platform.repo`. The platform-specific content below is injected by the CLI during onboarding.
+
+<!-- PC-PLATFORM-REVIEW-START -->
+<!-- PC-PLATFORM-REVIEW-END -->

package/harness/.opencode/commands/ops-ship.md

@@ -1,9 +1,9 @@
----
-description: Create a pull request for the current feature branch with screenshots if UI changed.
----
-
-Load the `pc-ops-ship` skill and execute it.
-
-Input:
-
-$ARGUMENTS
+---
+description: Create a pull request for the current feature branch with screenshots if UI changed.
+---
+
+Load the `pc-ops-ship` skill and execute it.
+
+Input:
+
+$ARGUMENTS

package/harness/.opencode/commands/plan-archive.md

@@ -1,5 +1,5 @@
----
-description: Archive the oldest merged unarchived OpenSpec change and update documentation.
----
-
-Load the `pc-plan-archive` skill and execute it in **interactive mode**. No input required.
+---
+description: Archive the oldest merged unarchived OpenSpec change and update documentation.
+---
+
+Load the `pc-plan-archive` skill and execute it in **interactive mode**. No input required.

package/harness/.opencode/commands/plan-explore.md

@@ -1,9 +1,9 @@
----
-description: Think through an idea, investigate a problem, or clarify requirements before creating a change.
----
-
-Load the `pc-plan-explore` skill and execute it in **interactive mode**.
-
-Input:
-
-$ARGUMENTS
+---
+description: Think through an idea, investigate a problem, or clarify requirements before creating a change.
+---
+
+Load the `pc-plan-explore` skill and execute it in **interactive mode**.
+
+Input:
+
+$ARGUMENTS

package/harness/.opencode/commands/plan-propose.md

@@ -1,9 +1,9 @@
----
-description: Parse a work item or idea and propose a change plan with enriched task assignments.
----
-
-Load the `pc-plan-propose` skill and execute it in **interactive mode**.
-
-Input:
-
-$ARGUMENTS
+---
+description: Parse a work item or idea and propose a change plan with enriched task assignments.
+---
+
+Load the `pc-plan-propose` skill and execute it in **interactive mode**.
+
+Input:
+
+$ARGUMENTS

package/harness/.opencode/commands/repo-help.md

@@ -1,5 +1,5 @@
----
-description: Show all available commands and when to use each one.
----
-
-Load the `pc-repo-help` skill and display its reference to the user exactly as written. Do not summarize.
+---
+description: Show all available commands and when to use each one.
+---
+
+Load the `pc-repo-help` skill and display its reference to the user exactly as written. Do not summarize.

package/harness/.opencode/package.json

@@ -1,5 +1,5 @@
-{
-  "dependencies": {
-    "@opencode/plugin": "2.0.22"
-  }
-}
+{
+  "dependencies": {
+    "@opencode/plugin": "2.0.22"
+  }
+}