@nofendian17/opencode-relay-plugin 2.0.5 → 2.0.7

package/README.md

@@ -16,7 +16,7 @@ gateways and are left untouched (no rewrite, no forced retries).
 V1 patched `provider.options.fetch` + `globalThis.fetch`. V2 providers no longer expose `fetch`,
 so this plugin uses the documented v2 interception points:
 
-- `ctx.session.hook("http.request")` — rewrite provider HTTP → `POST <relay>` with
+- `ctx.session.hook("http.request")` — rewrite provider HTTP → `<relay>` (original method/body/headers preserved) with
   `x-relay-target: <origin>` + `x-relay-path: <path+query>` (round-robin)
 - `ctx.session.hook("http.response")` — on relay `429 / 5xx`, try remaining relays, then direct
 - `ctx.session.hook("retry")` — relay network errors never reach `http.response`, so force a
@@ -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
@@ -32,15 +50,6 @@ function normalizeRelays(raw: string | string[] | undefined): string[] {
   return [...seen]
 }
 
-function createRoundRobin(count: number): () => number {
-  let idx = 0
-  return () => {
-    const cur = idx
-    idx = (idx + 1) % count
-    return cur
-  }
-}
-
 function buildRelayOrigins(relays: readonly string[]): ReadonlySet<string> {
   const origins = new Set<string>()
   for (const relay of relays) {
@@ -198,19 +207,33 @@ 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,
   }
 }
 
 const PATCH_MARKER = "__opencodeRelayZenPatched"
+const ROUND_ROBIN_KEY = "__opencodeRelayRoundRobinIdx"
 
 export default Plugin.define({
   id: "relay",
@@ -219,29 +242,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 +282,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>
@@ -279,7 +299,15 @@ export default Plugin.define({
     // Bind the pristine fetch, not a possibly already-patched one from a
     // prior setup without cleanup (avoids wrapping the patch recursively).
     const directFetch = originalForPatch.bind(globalThis)
-    const pickIndex = createRoundRobin(relays.length)
+    // Round-robin index is global (module/process-wide) so that setups created
+    // per session/conversation still rotate through the pool instead of every
+    // one starting at relay 0. Stored next to the patch marker.
+    if (g[ROUND_ROBIN_KEY] === undefined) g[ROUND_ROBIN_KEY] = 0
+    const pickIndex = () => {
+      const cur = (g[ROUND_ROBIN_KEY] as number) ?? 0
+      g[ROUND_ROBIN_KEY] = cur + 1
+      return cur % relays.length
+    }
     const relayOrigins = buildRelayOrigins(relays)
     const relayFetch = createRelayFetch(relays, pickIndex, directFetch, debug, log, emit)
 
@@ -288,6 +316,7 @@ export default Plugin.define({
     // else that calls fetch("https://opencode.ai/...") directly.
     if (!g[PATCH_MARKER]) {
       g[PATCH_MARKER] = originalForPatch
+      g[ROUND_ROBIN_KEY] = 0
       globalThis.fetch = (async (input: RequestInfo | URL, init?: RequestInit) => {
         const urlStr =
           typeof input === "string" ? input : input instanceof URL ? input.href : input.url
@@ -487,6 +516,7 @@ export default Plugin.define({
         // ignore
       }
       delete g[PATCH_MARKER]
+      delete g[ROUND_ROBIN_KEY]
     }
   },
 })

package/package.json

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