@plainconceptsplatform/agent-harness 3.4.1 → 3.6.0

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

@@ -1,10 +1,10 @@
----
-description: Read and triage PR review feedback. Reports what needs fixing, does not implement fixes.
-agent: planner
-subagent: false
----
-
-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.
+agent: planner
+subagent: false
+---
+
+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,7 +1,7 @@
----
-description: Archive the oldest merged unarchived OpenSpec change and update documentation.
-agent: planner
-subagent: false
----
-
-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.
+agent: planner
+subagent: false
+---
+
+Load the `pc-plan-archive` skill and execute it in **interactive mode**. No input required.

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

@@ -1,11 +1,11 @@
----
-description: Think through an idea, investigate a problem, or clarify requirements before creating a change.
-agent: planner
-subagent: false
----
-
-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.
+agent: planner
+subagent: false
+---
+
+Load the `pc-plan-explore` skill and execute it in **interactive mode**.
+
+Input:
+
+$ARGUMENTS

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

@@ -1,11 +1,11 @@
----
-description: Parse a work item or idea and propose a change plan with enriched task assignments.
-agent: planner
-subagent: false
----
-
-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.
+agent: planner
+subagent: false
+---
+
+Load the `pc-plan-propose` skill and execute it in **interactive mode**.
+
+Input:
+
+$ARGUMENTS

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

@@ -1,7 +1,7 @@
----
-description: "Quick plan: analyze the codebase and create a task checklist using the Todo pane. No files, no OpenSpec."
-agent: planner
-subagent: false
----
-
-Load the `pc-plan-quick` skill.
+---
+description: "Quick plan: analyze the codebase and create a task checklist using the Todo pane. No files, no OpenSpec."
+agent: planner
+subagent: false
+---
+
+Load the `pc-plan-quick` skill.

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

@@ -1,11 +1,11 @@
----
-description: Write a detailed, repo-aware user story from a feature idea or need.
-agent: planner
-subagent: false
----
-
-Load the `pc-plan-story` skill.
-
-Input:
-
-$ARGUMENTS
+---
+description: Write a detailed, repo-aware user story from a feature idea or need.
+agent: planner
+subagent: false
+---
+
+Load the `pc-plan-story` skill.
+
+Input:
+
+$ARGUMENTS

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

@@ -1,7 +1,7 @@
----
-description: Audit every configured source root against the fullstack engineer's guardrails and abilities. Read-only.
-agent: planner
-subagent: false
----
-
-Load the `pc-repo-audit` skill.
+---
+description: Audit every configured source root against the fullstack engineer's guardrails and abilities. Read-only.
+agent: planner
+subagent: false
+---
+
+Load the `pc-repo-audit` skill.

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"
+  }
+}

package/harness/.opencode/plugins/pc-commit-coauthor.js

@@ -0,0 +1,41 @@
+// pc-commit-coauthor: every commit an OpenCode agent makes names OpenCode as a
+// co-author, so agent work stays identifiable in history and in the metrics
+// built from it.
+//
+// The rule is enforced, not requested. `tool.execute.before` rewrites the shell
+// tool's command before it runs (and before the permission check sees it):
+// each `git commit` gains a `--trailer`, which git folds into the message
+// however it was written (`-m`, `-F`, an editor, `--amend`). It fails open: a
+// bug here leaves the command as the agent wrote it.
+
+import { Plugin } from "@opencode/plugin"
+
+export const TRAILER = "Co-Authored-By: opencode <noreply@opencode.ai>"
+
+const SHELL_TOOL = "shell"
+
+// `git commit` at the start of the command or of any segment, after git's own
+// `-C <path>` / `-c <key=value>` options. `commit-tree` and `commit-graph` are
+// other commands and do not match.
+const GIT_COMMIT = /(^|[;&|\n({`])(\s*git(?:\s+-[Cc]\s+(?:"[^"]*"|'[^']*'|[^\s"']+)+)*\s+commit)(?![\w-])/g
+
+export function withTrailer(command) {
+  if (typeof command !== "string" || command.includes("noreply@opencode.ai")) return command
+  return command.replace(GIT_COMMIT, (_, before, commit) => `${before}${commit} --trailer "${TRAILER}"`)
+}
+
+export default Plugin.define({
+  id: "pc.commit-coauthor",
+  async setup(ctx) {
+    await ctx.tool.hook("execute.before", event => {
+      try {
+        if (event.tool !== SHELL_TOOL) return
+        const command = event.input?.command
+        const next = withTrailer(command)
+        if (next !== command) event.input = { ...event.input, command: next }
+      } catch (error) {
+        console.error(`[harness] commit co-author hook failed open: ${error?.message}`)
+      }
+    })
+  },
+})

package/harness/.opencode/plugins/pc-subagent-monitor.js

@@ -1,184 +1,184 @@
-// pc-subagent-monitor: holds a spawn wave to agents.maxConcurrent and keeps a
-// live record of child sessions in .opencode/harness-run.json.
-//
-// OpenCode 2 denies through the permission system, not by throwing from a tool
-// hook, so the cap is a `subagent` permission rule: an `evaluate` hook counts
-// the children of the asking session and denies the launch over the limit.
-//
-// Child sessions are observed from the event stream. V2 does not document the
-// event type strings (the payload schema is an opaque `string`), so the handler
-// is deliberately tolerant and, with PC_HARNESS_DEBUG set, prints every event
-// type once so the names can be pinned from a real run. Counting via the event
-// stream is the iteration-one mechanism; `ctx.session` in the plugin context is
-// not documented to expose `list`, so it is not used here.
-
-import { Plugin } from "@opencode/plugin"
-import fs from "node:fs/promises"
-import path from "node:path"
-
-const PENDING_TTL_MS = 15_000
-
-export default Plugin.define({
-  id: "pc.subagent-monitor",
-  async setup(ctx) {
-    const root = ctx.location.directory || process.cwd()
-    const statePath = path.join(root, ".opencode", "harness-run.json")
-    const state = { updatedAt: null, agents: {} }
-    const pending = []
-    const seenTypes = new Set()
-    let capCache = null
-
-    try {
-      const prev = JSON.parse(await fs.readFile(statePath, "utf-8"))
-      if (prev?.agents && typeof prev.agents === "object") {
-        for (const [id, entry] of Object.entries(prev.agents)) {
-          if (entry?.status === "running") entry.stale = true
-          state.agents[id] = entry
-        }
-      }
-    } catch {
-      // fresh start
-    }
-
-    let persistScheduled = false
-    function persist() {
-      if (persistScheduled) return
-      persistScheduled = true
-      queueMicrotask(async () => {
-        persistScheduled = false
-        state.updatedAt = new Date().toISOString()
-        try {
-          await fs.mkdir(path.dirname(statePath), { recursive: true })
-          const tmpPath = `${statePath}.tmp`
-          await fs.writeFile(tmpPath, JSON.stringify(state, null, 2), "utf-8")
-          await fs.rename(tmpPath, statePath)
-        } catch {
-          // best-effort
-        }
-      })
-    }
-
-    function pruneStale() {
-      for (const [id, entry] of Object.entries(state.agents)) {
-        if (entry?.stale) delete state.agents[id]
-      }
-    }
-
-    async function maxConcurrent() {
-      if (capCache) return capCache
-      let configured = 3
-      try {
-        const raw = await fs.readFile(path.join(root, ".opencode", "harness.json"), "utf-8")
-        const value = JSON.parse(raw)?.agents?.maxConcurrent
-        if (Number.isFinite(value)) configured = value
-      } catch {
-        // no config yet; the CLI's own default is 3
-      }
-      capCache = Math.min(5, Math.max(1, Math.trunc(configured)))
-      return capCache
-    }
-
-    function runningFor(parentID) {
-      return Object.values(state.agents)
-        .filter(entry => entry?.status === "running" && !entry.stale && entry.parentID === parentID)
-        .length
-    }
-
-    function livePending(parentID) {
-      const now = Date.now()
-      for (let i = pending.length - 1; i >= 0; i--) {
-        if (now - pending[i].at > PENDING_TTL_MS) pending.splice(i, 1)
-      }
-      return pending.filter(entry => entry.parentID === parentID).length
-    }
-
-    function releasePending(parentID) {
-      const index = pending.findIndex(entry => entry.parentID === parentID)
-      if (index !== -1) pending.splice(index, 1)
-    }
-
-    // The cap was prose in pc-plan-apply ("you enforce the cap"), which asked the
-    // lead to count its own parallel calls. Six disjoint groups and a cap of
-    // three is exactly the moment a model stops counting.
-    await ctx.permission.hook("evaluate", async event => {
-      if (event.action !== "subagent") return
-      let cap
-      let live
-      try {
-        pruneStale()
-        cap = await maxConcurrent()
-        live = runningFor(event.sessionID) + livePending(event.sessionID)
-        if (live < cap) {
-          pending.push({ parentID: event.sessionID, at: Date.now() })
-          return
-        }
-      } catch (error) {
-        console.error(`[harness] concurrency check failed open: ${error?.message}`)
-        return
-      }
-      deny(event, live, cap)
-    })
-
-    function deny(event, live, cap) {
-      event.effect = "deny"
-      event.message =
-        `[harness] ${live} subagents are already in flight and agents.maxConcurrent is ${cap}.\n` +
-        "Collect a running worker before spawning another. A denied spawn is not a failed task: re-issue it in the next wave."
-    }
-
-    function extractSession(event) {
-      const data = event?.data ?? event?.properties ?? event ?? {}
-      const info = data?.info ?? data?.session ?? data ?? {}
-      const id = info.id ?? info.sessionID ?? data.sessionID
-      const parentID = info.parentID ?? info.parentId ?? data.parentID
-      const status = info.status ?? data.status
-      return { id, parentID, agent: info.agent ?? null, title: info.title ?? null, status }
-    }
-
-    const controller = new AbortController()
-    void (async () => {
-      try {
-        for await (const event of ctx.event.subscribe({ signal: controller.signal })) {
-          try {
-            const type = String(event?.type ?? "")
-            if (process.env.PC_HARNESS_DEBUG && type && !seenTypes.has(type)) {
-              seenTypes.add(type)
-              console.error(`[pc-subagent-monitor] event: ${type}`)
-            }
-            const { id, parentID, agent, title, status } = extractSession(event)
-            if (!id || !parentID) continue
-
-            if (status === "running" || /created|started/.test(type)) {
-              pruneStale()
-              releasePending(parentID)
-              state.agents[id] = {
-                agent,
-                parentID,
-                title,
-                status: "running",
-                startedAt: new Date().toISOString(),
-                endedAt: null,
-              }
-              await persist()
-              continue
-            }
-
-            const entry = state.agents[id]
-            if (!entry) continue
-            if (status === "idle" || /idle|completed|succeeded|failed/.test(type)) {
-              entry.status = "done"
-              entry.endedAt = new Date().toISOString()
-              await persist()
-            }
-          } catch {
-            // monitoring must never disrupt the run
-          }
-        }
-      } catch {
-        // stream aborted on unload
-      }
-    })()
-
-    return () => controller.abort()
-  },
-})
+// pc-subagent-monitor: holds a spawn wave to agents.maxConcurrent and keeps a
+// live record of child sessions in .opencode/harness-run.json.
+//
+// OpenCode 2 denies through the permission system, not by throwing from a tool
+// hook, so the cap is a `subagent` permission rule: an `evaluate` hook counts
+// the children of the asking session and denies the launch over the limit.
+//
+// Child sessions are observed from the event stream. V2 does not document the
+// event type strings (the payload schema is an opaque `string`), so the handler
+// is deliberately tolerant and, with PC_HARNESS_DEBUG set, prints every event
+// type once so the names can be pinned from a real run. Counting via the event
+// stream is the iteration-one mechanism; `ctx.session` in the plugin context is
+// not documented to expose `list`, so it is not used here.
+
+import { Plugin } from "@opencode/plugin"
+import fs from "node:fs/promises"
+import path from "node:path"
+
+const PENDING_TTL_MS = 15_000
+
+export default Plugin.define({
+  id: "pc.subagent-monitor",
+  async setup(ctx) {
+    const root = ctx.location.directory || process.cwd()
+    const statePath = path.join(root, ".opencode", "harness-run.json")
+    const state = { updatedAt: null, agents: {} }
+    const pending = []
+    const seenTypes = new Set()
+    let capCache = null
+
+    try {
+      const prev = JSON.parse(await fs.readFile(statePath, "utf-8"))
+      if (prev?.agents && typeof prev.agents === "object") {
+        for (const [id, entry] of Object.entries(prev.agents)) {
+          if (entry?.status === "running") entry.stale = true
+          state.agents[id] = entry
+        }
+      }
+    } catch {
+      // fresh start
+    }
+
+    let persistScheduled = false
+    function persist() {
+      if (persistScheduled) return
+      persistScheduled = true
+      queueMicrotask(async () => {
+        persistScheduled = false
+        state.updatedAt = new Date().toISOString()
+        try {
+          await fs.mkdir(path.dirname(statePath), { recursive: true })
+          const tmpPath = `${statePath}.tmp`
+          await fs.writeFile(tmpPath, JSON.stringify(state, null, 2), "utf-8")
+          await fs.rename(tmpPath, statePath)
+        } catch {
+          // best-effort
+        }
+      })
+    }
+
+    function pruneStale() {
+      for (const [id, entry] of Object.entries(state.agents)) {
+        if (entry?.stale) delete state.agents[id]
+      }
+    }
+
+    async function maxConcurrent() {
+      if (capCache) return capCache
+      let configured = 3
+      try {
+        const raw = await fs.readFile(path.join(root, ".opencode", "harness.json"), "utf-8")
+        const value = JSON.parse(raw)?.agents?.maxConcurrent
+        if (Number.isFinite(value)) configured = value
+      } catch {
+        // no config yet; the CLI's own default is 3
+      }
+      capCache = Math.min(5, Math.max(1, Math.trunc(configured)))
+      return capCache
+    }
+
+    function runningFor(parentID) {
+      return Object.values(state.agents)
+        .filter(entry => entry?.status === "running" && !entry.stale && entry.parentID === parentID)
+        .length
+    }
+
+    function livePending(parentID) {
+      const now = Date.now()
+      for (let i = pending.length - 1; i >= 0; i--) {
+        if (now - pending[i].at > PENDING_TTL_MS) pending.splice(i, 1)
+      }
+      return pending.filter(entry => entry.parentID === parentID).length
+    }
+
+    function releasePending(parentID) {
+      const index = pending.findIndex(entry => entry.parentID === parentID)
+      if (index !== -1) pending.splice(index, 1)
+    }
+
+    // The cap was prose in pc-plan-apply ("you enforce the cap"), which asked the
+    // lead to count its own parallel calls. Six disjoint groups and a cap of
+    // three is exactly the moment a model stops counting.
+    await ctx.permission.hook("evaluate", async event => {
+      if (event.action !== "subagent") return
+      let cap
+      let live
+      try {
+        pruneStale()
+        cap = await maxConcurrent()
+        live = runningFor(event.sessionID) + livePending(event.sessionID)
+        if (live < cap) {
+          pending.push({ parentID: event.sessionID, at: Date.now() })
+          return
+        }
+      } catch (error) {
+        console.error(`[harness] concurrency check failed open: ${error?.message}`)
+        return
+      }
+      deny(event, live, cap)
+    })
+
+    function deny(event, live, cap) {
+      event.effect = "deny"
+      event.message =
+        `[harness] ${live} subagents are already in flight and agents.maxConcurrent is ${cap}.\n` +
+        "Collect a running worker before spawning another. A denied spawn is not a failed task: re-issue it in the next wave."
+    }
+
+    function extractSession(event) {
+      const data = event?.data ?? event?.properties ?? event ?? {}
+      const info = data?.info ?? data?.session ?? data ?? {}
+      const id = info.id ?? info.sessionID ?? data.sessionID
+      const parentID = info.parentID ?? info.parentId ?? data.parentID
+      const status = info.status ?? data.status
+      return { id, parentID, agent: info.agent ?? null, title: info.title ?? null, status }
+    }
+
+    const controller = new AbortController()
+    void (async () => {
+      try {
+        for await (const event of ctx.event.subscribe({ signal: controller.signal })) {
+          try {
+            const type = String(event?.type ?? "")
+            if (process.env.PC_HARNESS_DEBUG && type && !seenTypes.has(type)) {
+              seenTypes.add(type)
+              console.error(`[pc-subagent-monitor] event: ${type}`)
+            }
+            const { id, parentID, agent, title, status } = extractSession(event)
+            if (!id || !parentID) continue
+
+            if (status === "running" || /created|started/.test(type)) {
+              pruneStale()
+              releasePending(parentID)
+              state.agents[id] = {
+                agent,
+                parentID,
+                title,
+                status: "running",
+                startedAt: new Date().toISOString(),
+                endedAt: null,
+              }
+              await persist()
+              continue
+            }
+
+            const entry = state.agents[id]
+            if (!entry) continue
+            if (status === "idle" || /idle|completed|succeeded|failed/.test(type)) {
+              entry.status = "done"
+              entry.endedAt = new Date().toISOString()
+              await persist()
+            }
+          } catch {
+            // monitoring must never disrupt the run
+          }
+        }
+      } catch {
+        // stream aborted on unload
+      }
+    })()
+
+    return () => controller.abort()
+  },
+})