@plainconceptsplatform/agent-harness 3.8.1 → 3.8.2

package/cli/steps/copy/agents.js

@@ -2,7 +2,7 @@ import fse from 'fs-extra'
 import path from 'path'
 import { fileURLToPath } from 'url'
 import { info, success, warn } from '../../utils/exec.js'
-import { ContractError, loadContract } from '../../../harness/.opencode/scripts/platform-contract.mjs'
+import { ContractError, isTemplateSource, loadContract } from '../../../harness/.opencode/scripts/platform-contract.mjs'
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const agentsContent = await fse.readJson(path.resolve(__dirname, '../../presets/agents-content.json'))
@@ -74,7 +74,7 @@ function replaceBetween(content, start, end, replacement) {
   return content.replace(pattern, () => (body ? `${start}\n${body}\n${end}` : `${start}\n${end}`))
 }
 
-export function reuseBlock(contract) {
+export function reuseBlock(contract, templateSource = false) {
   const { docs = [], packages = [], names = [] } = contract?.reuse ?? {}
   if (!docs.length && !packages.length && !names.length) return ''
   const code = list => list.map(item => `\`${item}\``).join(', ')
@@ -86,13 +86,14 @@ export function reuseBlock(contract) {
     ...(docs.length ? [`- Never design something new before reading ${code(docs)}.`] : []),
     ...(packages.length ? [`- Never re-implement an export of ${code(packages)}; import it.`] : []),
     ...(names.length ? [`- Never write your own ${code(names)}; reuse the existing one.`] : []),
-    `- Never edit a path listed under \`owned\`. Ask for the change at ${contract.source.changes}, or add a dated exception to \`${contract.exceptions}\`.`,
+    ...(templateSource ? [] : [`- Never edit a path listed under \`owned\`. Ask for the change at ${contract.source.changes}, or add a dated exception to \`${contract.exceptions}\`.`]),
   ].join('\n')
 }
 
 function contractBlock(cwd) {
   try {
-    return reuseBlock(loadContract(cwd))
+    const contract = loadContract(cwd)
+    return reuseBlock(contract, isTemplateSource(cwd, contract))
   } catch (err) {
     if (!(err instanceof ContractError)) throw err
     warn(`${err.message} The "Reuse before you build" block in AGENTS.md was left as it was.`)

package/harness/.opencode/plugins/pc-system-reminders.js

@@ -18,7 +18,7 @@ import { execFile } from "node:child_process"
 import fs from "node:fs/promises"
 import path from "node:path"
 import { promisify } from "node:util"
-import { ContractError, describeOwner, isOwned, loadContract } from "../scripts/platform-contract.mjs"
+import { ContractError, describeOwner, isOwned, isTemplateSource, loadContract } from "../scripts/platform-contract.mjs"
 
 const GUARDRAILS_SKILL = "pc-guardrails-generic"
 const TIER_SUFFIX = /\.(?:build|fast|plan)$/
@@ -210,7 +210,7 @@ async function checkPlatform(event, root) {
     return deny(event, error.message, "Fix that file before editing anything else: until it reads, nothing can tell which files the template owns.")
   }
   const owned = contract ? event.resources.filter(target => isOwnedTarget(target, root, contract)) : []
-  if (!owned.length || await syncInProgress(root, contract.sync.marker)) return
+  if (!owned.length || isTemplateSource(root, contract) || await syncInProgress(root, contract.sync.marker)) return
   const { owner, fix } = describeOwner(contract)
   return deny(event, `\`${owned[0]}\` is ${owner}.`, fix)
 }

package/harness/.opencode/scripts/check-diff.mjs

@@ -3,7 +3,7 @@ import fs from 'node:fs'
 import path from 'node:path'
 import { pathToFileURL } from 'node:url'
 import { commentBlocks, countLines, isGenerated, kindOf, lineKinds, slopFindings, testStats } from './code-rules.mjs'
-import { ContractError, describeOwner, isOwned, loadContract, nameKey, reusableNames } from './platform-contract.mjs'
+import { ContractError, describeOwner, isOwned, isTemplateSource, loadContract, nameKey, reusableNames } from './platform-contract.mjs'
 
 export const DEFAULT_LIMITS = {
   commentRatio: 0.1,
@@ -213,6 +213,7 @@ export function trackedFiles(root, baseline) {
 
 // A diff that changes the sync marker is a template sync, the one change allowed to rewrite owned files.
 export function platformViolations(root, base, paths, contract) {
+  if (isTemplateSource(root, contract)) return []
   const touched = new Set(untrackedFiles(root))
   for (const { status, from, to } of changes(root, base)) {
     touched.add(to)

package/harness/.opencode/scripts/platform-contract.mjs

@@ -1,3 +1,4 @@
+import { execFileSync } from 'node:child_process'
 import fs from 'node:fs'
 import path from 'node:path'
 
@@ -268,6 +269,32 @@ export function loadContract(root) {
   return { ...contract, rules: toRules(contract.owned), exceptionList: Array.isArray(entries) ? entries.filter(isMapping) : [] }
 }
 
+export function normalizeRepoUrl(url) {
+  return String(url ?? '')
+    .trim()
+    .replace(/^git\+/, '')
+    .replace(/^gh:/, 'https://github.com/')
+    .replace(/^ssh:\/\/(?:[^@/]+@)?([^/:]+)(?::\d+)?\//, 'https://$1/')
+    .replace(/^git@([^:]+):/, 'https://$1/')
+    .replace(/^(https?:\/\/)[^@/]+@/, '$1')
+    .replace(/^http:\/\//, 'https://')
+    .replace(/\/+$/, '')
+    .replace(/\.git$/, '')
+    .toLowerCase()
+}
+
+// The template repository carries its own contract, and its owned paths are its own to change.
+export function isTemplateSource(root, contract) {
+  if (!contract?.source.url) return false
+  let origin
+  try {
+    origin = execFileSync('git', ['config', '--get', 'remote.origin.url'], { cwd: root, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] })
+  } catch {
+    return false
+  }
+  return normalizeRepoUrl(origin) !== '' && normalizeRepoUrl(origin) === normalizeRepoUrl(contract.source.url)
+}
+
 const localDate = (now = new Date()) =>
   `${now.getFullYear()}-${String(now.getMonth() + 1).padStart(2, '0')}-${String(now.getDate()).padStart(2, '0')}`
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plainconceptsplatform/agent-harness",
-  "version": "3.8.1",
+  "version": "3.8.2",
   "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",