@khalilgharbaoui/opencode-claude-code-plugin 0.4.13 → 0.4.15

package/README.md

@@ -321,11 +321,28 @@ Set `permissionMode: "plan"` to forward `--permission-mode plan` to Claude. The
 
 ## Debug logging
 
+Two independent knobs:
+
 ```bash
+# Verbose logging to stderr (opencode surfaces stderr as UI warnings):
 DEBUG=opencode-claude-code opencode
+
+# Persistent file log (default: OFF — file is not created at all):
+OPENCODE_CLAUDE_CODE_LOG_FILE=1 opencode
+
+# Both:
+DEBUG=opencode-claude-code OPENCODE_CLAUDE_CODE_LOG_FILE=1 opencode
 ```
 
-Goes to stderr.
+When `OPENCODE_CLAUDE_CODE_LOG_FILE` is set to any truthy value (`1`,
+`true`, `yes`, `on`), the plugin writes NOTICE/WARN/ERROR (plus INFO and
+DEBUG when `DEBUG=opencode-claude-code` is also set) to
+`~/.local/share/opencode-claude-code/plugin.log` with 5MB rotation. Override
+the directory with `OPENCODE_CLAUDE_CODE_LOG_DIR=/custom/path`.
+
+Default is off so the plugin doesn't accrete a log file on every user's
+disk. Opt in when you need to inspect auto-continue decisions, broker
+state, or other plugin internals.
 
 ## Known limitations
 

package/dist/index.js

@@ -9,6 +9,13 @@ var DEBUG = process.env.DEBUG?.includes("opencode-claude-code") ?? false;
 var LOG_DIR = process.env.OPENCODE_CLAUDE_CODE_LOG_DIR ?? join(homedir(), ".local", "share", "opencode-claude-code");
 var LOG_FILE = join(LOG_DIR, "plugin.log");
 var MAX_LOG_BYTES = 5 * 1024 * 1024;
+function isTruthyEnv(v) {
+  if (v == null) return false;
+  const s = v.toLowerCase().trim();
+  if (s === "") return false;
+  return s !== "0" && s !== "false" && s !== "no" && s !== "off";
+}
+var LOG_FILE_ENABLED = isTruthyEnv(process.env.OPENCODE_CLAUDE_CODE_LOG_FILE);
 var fileLoggingDisabled = false;
 function rotateIfNeeded() {
   try {
@@ -20,6 +27,7 @@ function rotateIfNeeded() {
   }
 }
 function writeToFile(line) {
+  if (!LOG_FILE_ENABLED) return;
   if (fileLoggingDisabled) return;
   try {
     mkdirSync(dirname(LOG_FILE), { recursive: true });
@@ -1563,9 +1571,15 @@ function looksLikeBlocker(text) {
 }
 function looksLikeFinalAnswer(text) {
   const normalized = normalizeVisibleText(text).toLowerCase();
-  if (normalized.length < 30) return false;
   if (looksLikeQuestion(normalized) || looksLikeBlocker(normalized)) return false;
-  return /\b(done|completed|fixed|implemented|verified|published|released|sent|delivered|updated)\b/.test(normalized) || /\b(checks?|tests?) passed\b/.test(normalized) || /\b(summary|what changed|verification)\b/.test(normalized);
+  if (/\b(we'?re done|we are done|all done|all set)\b/.test(normalized)) {
+    return true;
+  }
+  if (normalized.length < 30) return false;
+  return /\b(done|completed|fixed|implemented|verified|published|released|sent|delivered|updated|shipped|deployed|merged|tagged|live|pinned)\b/.test(normalized) || // v0.4.15: also accept present-tense "tests pass" / "checks pass".
+  // Real fire 03:31 ended in "78/78 tests pass" — past-tense-only regex
+  // missed it.
+  /\b(checks?|tests?) (?:pass|passes|passed)\b/.test(normalized) || /\b(summary|what changed|verification)\b/.test(normalized);
 }
 function continuationSignature(snapshot) {
   const text = normalizeVisibleText(snapshot.text).slice(-500);