@plainconceptsplatform/agent-harness 2.0.0 → 2.0.1

package/src/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/src/utils/paths.js

@@ -1,64 +1,64 @@
-import os from 'node:os'
-import path from 'node:path'
-
-/**
- * Every harness path that lands in a consumer repo, in one place.
- *
- * These filenames are a contract with every repo the harness has been
- * installed into: the CLI writes them, and the opencode plugins shipped in
- * src/content/.opencode/plugins/ read them back at runtime. Changing a name
- * here means changing it in that payload too, or the plugins go blind.
- *
- * They are deliberately named for the harness rather than for the CLI, so a
- * future rename of the package does not become a rename of the on-disk state.
- */
-export const OPENCODE_DIR = '.opencode'
-
-/** Project config written by the metadata step. */
-export const CONFIG_FILE = 'harness.json'
-
-/** Per-developer model override, gitignored, never committed. */
-export const USER_CONFIG_FILE = 'harness.user.json'
-
-/** Hashes of every managed file, so `update` can detect local edits. */
-export const MANIFEST_FILE = 'harness-managed.json'
-
-/** Live subagent wave state, owned by the pc-subagent-monitor plugin. */
-export const RUN_STATE_FILE = 'harness-run.json'
-
-/** Names `.opencode/.gitignore` must cover. Mirrored in content/.opencode/_gitignore. */
-export const IGNORED_ENTRIES = [
-  'node_modules',
-  RUN_STATE_FILE,
-  USER_CONFIG_FILE,
-  MANIFEST_FILE,
-  'source-roots.json',
-  '*-engineer.*.md',
-]
-
-export function configPath(cwd = process.cwd()) {
-  return path.join(cwd, OPENCODE_DIR, CONFIG_FILE)
-}
-
-export function userConfigPath(cwd = process.cwd()) {
-  return path.join(cwd, OPENCODE_DIR, USER_CONFIG_FILE)
-}
-
-export function manifestPath(cwd = process.cwd()) {
-  return path.join(cwd, OPENCODE_DIR, MANIFEST_FILE)
-}
-
-/** Cross-run models cache, keyed to the CLI rather than to any one repo. */
-export function modelsCacheDir() {
-  return path.join(os.homedir(), '.config', 'agent-harness')
-}
-
-/**
- * Filenames written by opencode-onboard v1.x. Only used to detect a v1 repo
- * and refuse to half-patch it: nothing reads their contents.
- */
-export const LEGACY_CONFIG_FILES = [
-  'opencode-onboard.json',
-  'opencode-onboard.user.json',
-  'opencode-onboard-managed.json',
-]
+import os from 'node:os'
+import path from 'node:path'
+
+/**
+ * Every harness path that lands in a consumer repo, in one place.
+ *
+ * These filenames are a contract with every repo the harness has been
+ * installed into: the CLI writes them, and the opencode plugins shipped in
+ * src/content/.opencode/plugins/ read them back at runtime. Changing a name
+ * here means changing it in that payload too, or the plugins go blind.
+ *
+ * They are deliberately named for the harness rather than for the CLI, so a
+ * future rename of the package does not become a rename of the on-disk state.
+ */
+export const OPENCODE_DIR = '.opencode'
+
+/** Project config written by the metadata step. */
+export const CONFIG_FILE = 'harness.json'
+
+/** Per-developer model override, gitignored, never committed. */
+export const USER_CONFIG_FILE = 'harness.user.json'
+
+/** Hashes of every managed file, so `update` can detect local edits. */
+export const MANIFEST_FILE = 'harness-managed.json'
+
+/** Live subagent wave state, owned by the pc-subagent-monitor plugin. */
+export const RUN_STATE_FILE = 'harness-run.json'
+
+/** Names `.opencode/.gitignore` must cover. Mirrored in content/.opencode/_gitignore. */
+export const IGNORED_ENTRIES = [
+  'node_modules',
+  RUN_STATE_FILE,
+  USER_CONFIG_FILE,
+  MANIFEST_FILE,
+  'source-roots.json',
+  '*-engineer.*.md',
+]
+
+export function configPath(cwd = process.cwd()) {
+  return path.join(cwd, OPENCODE_DIR, CONFIG_FILE)
+}
+
+export function userConfigPath(cwd = process.cwd()) {
+  return path.join(cwd, OPENCODE_DIR, USER_CONFIG_FILE)
+}
+
+export function manifestPath(cwd = process.cwd()) {
+  return path.join(cwd, OPENCODE_DIR, MANIFEST_FILE)
+}
+
+/** Cross-run models cache, keyed to the CLI rather than to any one repo. */
+export function modelsCacheDir() {
+  return path.join(os.homedir(), '.config', 'agent-harness')
+}
+
+/**
+ * Filenames written by opencode-onboard v1.x. Only used to detect a v1 repo
+ * and refuse to half-patch it: nothing reads their contents.
+ */
+export const LEGACY_CONFIG_FILES = [
+  'opencode-onboard.json',
+  'opencode-onboard.user.json',
+  'opencode-onboard-managed.json',
+]

package/src/utils/update-manifest.js

@@ -1,49 +1,49 @@
-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))
-}
-
-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
-  return previousHash === await hashFile(destinationPath)
-}
-
-export async function recordManagedFile(manifest, relativePath, sourcePath) {
-  const files = manifest.files
-  files[normalizeRelativePath(relativePath)] = await hashFile(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))
+}
+
+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
+  return previousHash === await hashFile(destinationPath)
+}
+
+export async function recordManagedFile(manifest, relativePath, sourcePath) {
+  const files = manifest.files
+  files[normalizeRelativePath(relativePath)] = await hashFile(sourcePath)
+}
+
+export function manifestPath() {
+  return MANIFEST_RELATIVE_PATH
+}

package/src/content/.opencode/plugins/pc-system-reminders.test.js

@@ -1,35 +0,0 @@
-import { describe, expect, it } from "vitest"
-import { PcSystemReminders } from "./pc-system-reminders.js"
-
-describe("PcSystemReminders", () => {
-  it("reminds an agent to load its abilities before work", async () => {
-    const plugin = await PcSystemReminders({ directory: process.cwd() })
-    const output = {
-      messages: [{
-        info: { role: "user", sessionID: "session-1", agent: "missing-agent" },
-        parts: [{ type: "text", text: "Implement the change" }],
-      }],
-    }
-
-    await plugin["chat.message"]({ sessionID: "session-1", agent: "missing-agent" })
-    await plugin["experimental.chat.messages.transform"]({}, output)
-
-    expect(output.messages[0].parts[0].text).toContain("pc-guardrails-generic")
-  })
-
-  it("stops reminding after the required guardrail loads", async () => {
-    const plugin = await PcSystemReminders({ directory: process.cwd() })
-    const output = {
-      messages: [{
-        info: { role: "user", sessionID: "session-2", agent: "missing-agent" },
-        parts: [{ type: "text", text: "Implement the change" }],
-      }],
-    }
-
-    await plugin["chat.message"]({ sessionID: "session-2", agent: "missing-agent" })
-    await plugin["tool.execute.after"]({ tool: "skill", sessionID: "session-2", args: { name: "pc-guardrails-generic" } })
-    await plugin["experimental.chat.messages.transform"]({}, output)
-
-    expect(output.messages[0].parts[0].text).not.toContain("system-reminder")
-  })
-})