@nofendian17/opencode-relay-plugin 2.0.5 → 2.0.6

package/README.md

@@ -65,7 +65,7 @@ Local file (auto-discovered):
 | Option | Env fallback | Description |
 | --- | --- | --- |
 | `url: string \| string[]` | `RELAY_URL` (comma-separated) | Relay endpoint(s), `http(s)` only. Invalid entries are skipped with a warning log; if none remain valid the plugin disables itself. |
-| `debug: boolean` | `RELAY_DEBUG=1` | Verbose per-request logging. When on, log lines are also appended to `/tmp/opencode-relay.log` (plugin `console.log` is not forwarded to opencode logs). Override the path with `RELAY_DEBUG_LOG` (used by tests for isolation). |
+| `debug: boolean` | `RELAY_DEBUG=1` | Verbose per-request logging. **Ini opsi plugin** (`plugins[].options.debug`), **bukan** top-level `debug` di `opencode.jsonc` — top-level `debug` tidak mengaktifkan log plugin ini. Baris penting (`init`, `disabled`, `fallback`) selalu ditulis ke file log; baris verbose per-request hanya saat `debug` on. Override path dengan `RELAY_DEBUG_LOG` (dipakai tests untuk isolasi). |
 
 ## Relay protocol (unchanged from v1)
 
@@ -89,10 +89,11 @@ access logs as secret-bearing.
 
 ## Verify
 
-1. Set `debug: true` + `url` to a local echo server, run a prompt, confirm
+1. Set plugin option `debug: true` + `url` ke local echo server, run prompt, confirm
    `x-relay-target` / `x-relay-path` headers arrive.
 2. Stop the relay → request still succeeds via direct fallback (check
-   `[relay] relay fallback to direct` in `/tmp/opencode-relay.log`).
+   `[relay] relay fallback to direct` in the log file — path is printed in the
+   `[relay] init` line as `logFile`).
 3. Unset `url`/`RELAY_URL` → `[relay] disabled` log, traffic goes direct.
 4. Confirm scope: a request to `https://opencode.ai/inference/...` must show
    `via relay` in the log. If the log only shows
@@ -100,8 +101,11 @@ access logs as secret-bearing.
    `https://opencode.ai/...` URL and zero `via relay` lines, the request never
    entered the pool — check the origin match in `index.ts`.
 
-Note: the debug file is the shared `/tmp/opencode-relay.log`; concurrent
-opencode processes append to the same file.
+Note: the default file is `os.tmpdir()/opencode-relay.log` — on macOS that is
+`$TMPDIR/opencode-relay.log` (e.g. `/var/folders/.../T/`), NOT `/tmp/`.
+Check the `logFile` field in the `[relay] init` line for the exact path.
+Concurrent opencode processes append to the same file. Plugin `console.log`
+is not forwarded to opencode logs, so always check the file, not the TUI.
 
 ## V1 vs V2 notes
 

package/index.ts

@@ -1,8 +1,26 @@
 import { Plugin } from "@opencode/plugin"
-import { appendFile } from "node:fs/promises"
+import { appendFileSync } from "node:fs"
 import { tmpdir } from "node:os"
 import { join } from "node:path"
 
+// Bun runtime: node:fs/promises appendFile fails silently. Use sync API.
+function safeAppendFile(path: string, data: string): void {
+  try {
+    appendFileSync(path, data)
+  } catch {
+    // Fallback: try Bun.write if available
+    try {
+      // @ts-ignore - Bun global may not exist in Node
+      if (typeof Bun !== "undefined") {
+        // @ts-ignore
+        Bun.write(path, data, { mode: "append" })
+      }
+    } catch {
+      // All failed - ignore
+    }
+  }
+}
+
 interface RelayOptions {
   url?: string | string[]
   debug?: boolean
@@ -198,15 +216,28 @@ function createRelayFetch(
   }
 }
 
+function parseDebug(value: unknown): boolean | undefined {
+  if (value === undefined) return undefined
+  if (typeof value === "boolean") return value
+  if (typeof value === "number") return value !== 0
+  if (typeof value === "string") {
+    const v = value.trim().toLowerCase()
+    if (["1", "true", "yes", "on", "debug"].includes(v)) return true
+    if (["0", "false", "no", "off", ""].includes(v)) return false
+    return true
+  }
+  return Boolean(value)
+}
+
 function parseOptions(options: Record<string, unknown> | undefined): {
   relays: string[]
   debug: boolean
 } {
   const url = options?.["url"] as string | string[] | undefined
-  const debugOpt = options?.["debug"] as boolean | undefined
+  const envDebug = process.env.RELAY_DEBUG
   return {
     relays: normalizeRelays(url),
-    debug: debugOpt ?? process.env.RELAY_DEBUG === "1",
+    debug: parseDebug(options?.["debug"]) ?? parseDebug(envDebug) ?? false,
   }
 }
 
@@ -219,29 +250,26 @@ export default Plugin.define({
     const debug = parsed.debug
     const prefix = "[relay]"
     // console output from v2 plugins is not forwarded to opencode logs,
-    // so when debug is on, mirror everything to a file as well.
+    // so mirror important events to a file unconditionally and verbose
+    // per-request lines only when debug is on.
+    // NOTE: this is the plugin option `plugins[].options.debug`
+    // (or RELAY_DEBUG=1), NOT the top-level opencode `debug` flag.
     // RELAY_DEBUG_LOG overrides the path (tests use an isolated file so a
     // live opencode process appending to the default never interferes).
-    // Async fire-and-forget: logging never breaks requests.
-    const debugLogFile = process.env.RELAY_DEBUG_LOG ?? DEBUG_LOG
-    let warnedLogFile = false
-    const fileLog: Logger | undefined = debug
-      ? (message, extra) => {
-          const line =
-            `${new Date().toISOString()} ${prefix} ${message}` +
-            (extra ? ` ${JSON.stringify(extra)}` : "") +
-            "\n"
-          appendFile(debugLogFile, line).catch((e: unknown) => {
-            if (!warnedLogFile) {
-              warnedLogFile = true
-              console.log(prefix, "debug log write failed", debugLogFile, String(e))
-            }
-          })
-        }
-      : undefined
+    // Default path is os.tmpdir() — on macOS that is NOT /tmp but
+    // $TMPDIR (e.g. /var/folders/.../T/opencode-relay.log).
+    // Logging never breaks requests.
+    const logFile = process.env.RELAY_DEBUG_LOG ?? DEBUG_LOG
+    const fileLog: Logger = (message, extra) => {
+      const line =
+        `${new Date().toISOString()} ${prefix} ${message}` +
+        (extra ? ` ${JSON.stringify(extra)}` : "") +
+        "\n"
+      safeAppendFile(logFile, line)
+    }
     const emit: Logger = (message, extra) => {
       console.log(prefix, message, extra ? JSON.stringify(extra) : "")
-      fileLog?.(message, extra)
+      fileLog(message, extra)
     }
     const log: Logger = (message, extra) => {
       if (debug) emit(message, extra)
@@ -262,14 +290,14 @@ export default Plugin.define({
       }
     }
 
-    emit("init", { debug, relayCount: relays.length })
+    emit("init", { debug, relayCount: relays.length, logFile })
 
     if (relays.length === 0) {
       emit("disabled: no relays configured (set plugin option `url` or RELAY_URL)", {})
       return
     }
 
-    log("relay pool loaded", { relays })
+    log("relay pool loaded", { relays, logFile })
 
     const rawFetch = globalThis.fetch
     const g = globalThis as Record<string, unknown>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nofendian17/opencode-relay-plugin",
-  "version": "2.0.5",
+  "version": "2.0.6",
   "description": "Route OpenCode v2 Zen (opencode.ai) traffic through relay endpoint(s) with round-robin, failover, and direct fallback. Other providers are left untouched.",
   "type": "module",
   "main": "./index.ts",
@@ -27,7 +27,7 @@
   },
   "devDependencies": {
     "@opencode/plugin": "^2.0.28",
-    "@types/node": "^26.6.4",
+    "@types/node": "^26.6.5",
     "typescript": "^5.9.3"
   }
 }