@mindfoldhq/trellis 0.5.0-beta.5 → 0.5.0-beta.7

package/dist/migrations/manifests/0.5.0-beta.6.json

@@ -0,0 +1,9 @@
+{
+  "version": "0.5.0-beta.6",
+  "description": "Fix codex trellis-check agent: change from read-only to workspace-write, rewrite developer_instructions to match other platforms' self-fix behavior per workflow.md contract.",
+  "breaking": false,
+  "recommendMigrate": false,
+  "changelog": "**Bug Fixes:**\n- fix(codex): `trellis-check.toml` was shipped with `sandbox_mode = \"read-only\"` and 'Read-only Trellis reviewer' framing, which directly contradicted `workflow.md` (the check agent is supposed to auto-fix issues it finds — all other platforms' check agents have Write+Edit tools). Codex users running `trellis-check` could report findings but not fix them, forcing a manual roundtrip through the main agent. Changed to `sandbox_mode = \"workspace-write\"` and rewrote developer_instructions to instruct self-fix + re-run lint/type-check until green, matching the behavior contract of every other platform.",
+  "migrations": [],
+  "notes": "Pure template-content fix for codex users. No path migrations — `trellis update` will flag `.codex/agents/trellis-check.toml` as a template update and either auto-update (if unmodified) or prompt for confirm (if you customized it)."
+}

package/dist/migrations/manifests/0.5.0-beta.7.json

@@ -0,0 +1,9 @@
+{
+  "version": "0.5.0-beta.7",
+  "description": "Fix opencode plugin loader incompatibility with OpenCode 1.2.x. Plugins shipped `export default { id, server: async (...) => hooks }` object shape; OpenCode iterates `Object.entries(mod)` and calls each export as a function (`packages/opencode/src/plugin/index.ts:90 — for ([_, fn] of Object.entries(mod)) await fn(input)`), which crashes with `TypeError: fn is not a function`. Fixed by changing all 3 plugins to the current factory-function shape: `export default async (input) => hooks`.",
+  "breaking": false,
+  "recommendMigrate": false,
+  "changelog": "**Bug Fixes:**\n- fix(opencode): plugins now use the factory-function export shape (`export default async ({ directory, client }) => hooks`) that OpenCode 1.2.x requires. The previous `{ id, server }` object shape crashed OpenCode on startup with `TypeError: fn3 is not a function. (In 'fn3(input)', 'fn3' is an instance of Object)` at `src/plugin/index.ts:90:28`, because OpenCode 1.2.x iterates every module export with `Object.entries(mod)` and calls each one as a function (`for ([_, fn] of Object.entries(mod)) await fn(input)`). The `server:` field was never unwrapped by the runtime. Affects all 3 plugins: `inject-subagent-context.js`, `inject-workflow-state.js`, `session-start.js`.\n  Users on any trellis version that configured opencode — including 0.4.x stable — were affected as soon as they updated opencode to 1.2.x. Upgrade to this version to restore opencode startup.",
+  "migrations": [],
+  "notes": "Pure template-content fix for opencode users. No path migrations. `trellis update` auto-updates the 3 plugin files (unmodified copies) or prompts for confirm with diff (if you customized them)."
+}

package/dist/templates/codex/agents/trellis-check.toml

@@ -1,23 +1,38 @@
 name = "trellis-check"
-description = "Read-only Trellis reviewer focused on correctness, missing tests, and spec drift."
-sandbox_mode = "read-only"
+description = "Workspace-write Trellis reviewer that self-fixes spec drift, lint/type-check failures, and missing tests."
+sandbox_mode = "workspace-write"
 
 developer_instructions = """
 You are the Trellis reviewer agent.
 
+Your job is to review code changes against specs AND fix issues directly — not just report them. You have write access; use it.
+
 Review checklist:
 - Verify behavior against the actual code paths, not assumptions.
 - Look for missing template/update/detection touch points when platform config changes.
 - Check whether tests should be added or updated.
 - Check whether `.trellis/spec/` docs need sync after implementation.
+- Run lint and type-check; fix any failures.
 - Prefer concrete findings over speculative warnings.
 
+When you find an issue:
+1. Fix it directly using edit/write tools.
+2. Re-run lint and type-check until green.
+3. Record what you changed and why.
+
 Output format:
-## Findings
-- Severity: <high|medium|low>
+## Findings (fixed)
 - File: <path>
-- Issue: <what is wrong>
-- Recommendation: <specific fix>
+- Issue: <what was wrong>
+- Fix: <what you changed>
+
+## Findings (not fixed)
+Only list issues you could not self-fix (e.g. missing product decision, out-of-scope). Explain why.
+
+## Verification
+- Lint: pass/fail
+- TypeCheck: pass/fail
+- Tests: pass/fail (if applicable)
 
-If no issues are found, say so explicitly.
+If no issues are found, say so explicitly after verifying lint/type-check pass.
 """

package/dist/templates/opencode/plugins/inject-subagent-context.js

@@ -255,13 +255,16 @@ ${originalPrompt}
   return templates[agentType] || originalPrompt
 }
 
-export default {
-  id: "trellis.inject-subagent-context",
-  server: async ({ directory }) => {
-    const ctx = new TrellisContext(directory)
-    debugLog("inject", "Plugin loaded, directory:", directory)
-
-    return {
+// OpenCode plugin factory: `export default async (input) => hooks`.
+// OpenCode 1.2.x iterates every module export and invokes it as a function
+// (packages/opencode/src/plugin/index.ts — `for ([_, fn] of Object.entries(mod)) await fn(input)`);
+// the previous `{ id, server }` object shape failed with
+// `TypeError: fn is not a function` in 1.2.x.
+export default async ({ directory }) => {
+  const ctx = new TrellisContext(directory)
+  debugLog("inject", "Plugin loaded, directory:", directory)
+
+  return {
       "tool.execute.before": async (input, output) => {
         try {
           debugLog("inject", "tool.execute.before called, tool:", input?.tool)
@@ -337,5 +340,4 @@ export default {
         }
       }
     }
-  }
 }

package/dist/templates/opencode/plugins/inject-workflow-state.js

@@ -107,13 +107,12 @@ function buildBreadcrumb(id, status, templates) {
   return `<workflow-state>\n${header}\n${body}\n</workflow-state>`
 }
 
-export default {
-  id: "trellis.inject-workflow-state",
-  server: async ({ directory }) => {
-    const ctx = new TrellisContext(directory)
-    debugLog("workflow-state", "Plugin loaded, directory:", directory)
+// OpenCode 1.2.x expects plugins to be factory functions (see inject-subagent-context.js comment).
+export default async ({ directory }) => {
+  const ctx = new TrellisContext(directory)
+  debugLog("workflow-state", "Plugin loaded, directory:", directory)
 
-    return {
+  return {
       // chat.message fires on every user message. Inject breadcrumb in-place
       // so it persists in conversation history.
       "chat.message": async (input, output) => {
@@ -155,6 +154,5 @@ export default {
           )
         }
       },
-    }
-  },
+  }
 }

package/dist/templates/opencode/plugins/session-start.js

@@ -432,13 +432,12 @@ async function hasPersistedInjectedContext(client, directory, sessionID) {
   }
 }
 
-export default {
-  id: "trellis.session-start",
-  server: async ({ directory, client }) => {
-    const ctx = new TrellisContext(directory)
-    debugLog("session", "Plugin loaded, directory:", directory)
+// OpenCode 1.2.x expects plugins to be factory functions (see inject-subagent-context.js comment).
+export default async ({ directory, client }) => {
+  const ctx = new TrellisContext(directory)
+  debugLog("session", "Plugin loaded, directory:", directory)
 
-    return {
+  return {
       // Clear in-memory dedupe after compaction so context can be re-injected.
       event: ({ event }) => {
         try {
@@ -511,6 +510,5 @@ export default {
           debugLog("session", "Error in chat.message:", error.message, error.stack)
         }
       }
-    }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mindfoldhq/trellis",
-  "version": "0.5.0-beta.5",
+  "version": "0.5.0-beta.7",
   "description": "AI capabilities grow like ivy — Trellis provides the structure to guide them along a disciplined path",
   "type": "module",
   "main": "./dist/index.js",