opencode-worker 0.1.8 → 0.1.10

package/README.md

@@ -69,10 +69,10 @@ Exit codes: `0` ok, `1` worker failed (timeout, nonzero exit, error event, model
 | `opencode` | `{bin, version}` |
 | `requested` | `{model, variant, agent, timeoutSec}` |
 | `actual` | provider/model/variant recorded in the isolated session DB (needs `sqlite3`; `null` if unavailable) |
-| `text` | worker's final text |
+| `text` | worker's final answer: the text after its last tool call, without narration like "I'll read the file." (all text, joined by blank lines, if nothing follows the last tool call). Safe to paste into the next job's prompt; workers never see each other's results otherwise |
 | `tools[]` | `{tool, status, input, exit, output, error}` per tool call; `exit` is the bash exit code (else `null`), `output` capped at 4000 chars |
 | `cost`, `tokens` | summed from `step_finish` events |
-| `diff` | `diff -ruN` of `<dir>` before vs after (excludes `.opencode-worker/`) |
+| `diff` | unified diff of `<dir>` before vs after. Excludes `.opencode-worker/` and tool caches (`__pycache__`, `*.pyc`, `.pytest_cache`, `.mypy_cache`, `.ruff_cache`, `.cache`); the worker also runs with `PYTHONDONTWRITEBYTECODE=1`, so test runs don't show up as changes |
 | `errors[]` | `timeout`, `cancelled`, `nonzero_exit`, `event_error`, `model_mismatch`, `diff_failed`, `unparsed_line` |
 | `artifacts` | raw `events.jsonl` and `stderr.txt` under `<dir>/.opencode-worker/runs/<id>/` |
 
@@ -117,6 +117,8 @@ env = { OPENCODE_WORKER_BIN = "/path/to/opencode" }
 default_tools_approval_mode = "approve"
 ```
 
+Codex does not read a project-level `.codex/config.toml`. To enable the worker only when needed, put the same block in a profile file `~/.codex/ocw.config.toml` and start Codex with `codex -p ocw`.
+
 OMP (`.omp/mcp.json` or `~/.omp/agent/mcp.json`): the JSON above with `"type": "stdio"`. Non-interactive: `omp -p --auto-approve "…"`.
 
 Job records are written to `$OPENCODE_WORKER_STATE_DIR` (default `$XDG_STATE_HOME/opencode-worker/jobs`, else `~/.local/state/opencode-worker/jobs`), so `worker_status`/`worker_result` work after a server or client restart.

package/lib/worker.mjs

@@ -60,6 +60,8 @@ function isolatedEnv(stateRoot, configPath) {
     OPENCODE_CONFIG_DIR: path.join(stateRoot, "xdg/config/opencode"),
     OPENCODE_DISABLE_PROJECT_CONFIG: "1",
     TERM: "dumb",
+    // Test runs must not show up as file changes in the diff.
+    PYTHONDONTWRITEBYTECODE: "1",
   };
   for (const k of ["LANG", "LC_ALL", "TMPDIR"]) if (process.env[k]) env[k] = process.env[k];
   for (const k of ["HOME", "XDG_CONFIG_HOME", "XDG_DATA_HOME", "XDG_CACHE_HOME", "XDG_STATE_HOME", "OPENCODE_CONFIG_DIR"])
@@ -95,27 +97,40 @@ function snapshot(dir, dest) {
   }
 }
 
+// Tool caches that commands write as a side effect; never meaningful in a worker diff.
+export const DIFF_EXCLUDES = [STATE_DIR, "__pycache__", "*.pyc", ".pytest_cache", ".mypy_cache", ".ruff_cache", ".cache"];
+
 function unifiedDiff(before, dir) {
-  const r = spawnSync("diff", ["-ruN", `--exclude=${STATE_DIR}`, "before", dir], {
+  const r = spawnSync("diff", ["-ruN", ...DIFF_EXCLUDES.map((p) => `--exclude=${p}`), "before", dir], {
     cwd: path.dirname(before),
     encoding: "utf8",
     maxBuffer: 32 * 1024 * 1024,
   });
   // diff exit: 0 same, 1 different, 2 trouble
   if (r.status === 2) return { diff: null, error: r.stderr.trim() };
-  return { diff: r.stdout.replaceAll(dir + "/", "after/"), error: null };
+  // Drop the per-file "diff -ruN --exclude=…" command lines; ---/+++ headers already name each file.
+  const body = r.stdout.split("\n").filter((l) => !l.startsWith("diff -ruN ")).join("\n");
+  return { diff: body.replaceAll(dir + "/", "after/"), error: null };
 }
 
+/**
+ * `text` is the worker's final answer: text emitted after its last tool call.
+ * Narration before tool calls ("I'll read the file.") is dropped so the answer can be
+ * handed to the next job verbatim. If nothing follows the last tool call, all text is kept.
+ */
 function parseEvents(stdout) {
   const out = { text: "", tools: [], cost: 0, tokens: { input: 0, output: 0, reasoning: 0, cacheRead: 0, cacheWrite: 0 }, errors: [], sessionID: null };
+  const allText = [];
+  let finalText = [];
   for (const line of stdout.split("\n")) {
     if (!line.trim()) continue;
     let ev;
     try { ev = JSON.parse(line); } catch { out.errors.push({ type: "unparsed_line", detail: line.slice(0, 300) }); continue; }
     out.sessionID ??= ev.sessionID ?? null;
     const p = ev.part ?? {};
-    if (ev.type === "text") out.text += p.text ?? "";
+    if (ev.type === "text" && p.text) { allText.push(p.text); finalText.push(p.text); }
     else if (ev.type === "tool_use") {
+      finalText = [];
       const s = p.state ?? {};
       const output = typeof s.output === "string" ? s.output : s.output == null ? null : JSON.stringify(s.output);
       out.tools.push({
@@ -137,6 +152,7 @@ function parseEvents(stdout) {
       out.tokens.cacheWrite += t.cache?.write ?? 0;
     } else if (ev.type === "error") out.errors.push({ type: "event_error", detail: ev.error ?? p });
   }
+  out.text = (finalText.length ? finalText : allText).join("\n\n").trim();
   return out;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-worker",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "description": "Run the official OpenCode CLI as an isolated, time-boxed worker. CLI + MCP stdio server.",
   "repository": {
     "type": "git",