@koriit/opencode-claude-bridge 0.1.7 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@koriit/opencode-claude-bridge",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "An OpenCode plugin that bridges enabled Claude Code plugins (commands, agents, skills, MCP, LSP) into OpenCode at runtime, namespaced so they never shadow your existing items.",
   "type": "module",
   "license": "MIT",

package/src/index.ts

@@ -5,7 +5,7 @@ import { injectCommandsAndAgents } from "./inject.js"
 import { injectSkills } from "./skill-inject.js"
 import { injectMcp } from "./mcp-inject.js"
 import { injectLsp } from "./lsp-inject.js"
-import { createLogger, type LoggingClient } from "./logger.js"
+import { createLogger } from "./logger.js"
 import { listClaudePlugins, selectEnabledPlugins } from "./selection.js"
 import { collectExistingSkillNames } from "./skill-scan.js"
 
@@ -51,7 +51,7 @@ export const server: Plugin = async (_input, options) => {
 
   return {
     config: async (cfg) => {
-      const logger = createLogger(bridge.strict, _input.client as unknown as LoggingClient)
+      const logger = createLogger(bridge.strict)
       try {
         // Replay parse-time validation warnings (strict-promotable).
         for (const w of warnings) logger.warn(w)

package/src/logger.ts

@@ -1,8 +1,3 @@
-/** Minimal duck-type for the OpenCode client — only the log() method we use. */
-export interface LoggingClient {
-  log(params: { service?: string; level?: "debug" | "info" | "warn" | "error"; message?: string }): unknown
-}
-
 /** Thrown when a soft warning is promoted to a hard error under `strict` mode. */
 export class BridgeError extends Error {
   override name = "BridgeError"
@@ -29,14 +24,16 @@ export interface Logger {
   hadWarnings(): boolean
 }
 
-
 /**
- * Fallback output logger used when no real OpenCode client is available (tests,
- * edge-case early errors). Writes to process.stderr in OpenCode's structured
- * format and only when --print-logs is in argv — matching OpenCode's own
- * log-visibility behaviour.
+ * Internal output logger. Writes to process.stderr in OpenCode's structured
+ * log format, gated on --print-logs to match OpenCode's own log-visibility
+ * behaviour.
+ *
+ * Note: client.log() (HTTP POST to /log) was tried but is unreliable during
+ * the config hook because it fires during server bootstrap before the /log
+ * endpoint is ready. Direct stderr write is the safe, synchronous alternative.
  */
-const fallbackLog = (() => {
+const log = (() => {
   const enabled = process.argv.includes("--print-logs")
   let last = Date.now()
 
@@ -56,43 +53,21 @@ const fallbackLog = (() => {
 })()
 
 /**
- * Create a logger bound to the resolved `strict` flag and the OpenCode client.
- *
- * When a client is provided, log entries are posted to the server via
- * `client.log()` — they flow through OpenCode's own log pipeline, appear in
- * the log file, and respect `--print-logs` automatically.
- *
- * When no client is provided (tests, early-startup errors) the fallback logger
- * writes to process.stderr in the same format, gated on `--print-logs`.
+ * Create a logger bound to the resolved `strict` flag. The hook itself is
+ * responsible for catching {@link BridgeError} in non-strict paths; in strict
+ * mode the error propagates so OpenCode surfaces a hard failure.
  */
-export function createLogger(strict: boolean, client?: LoggingClient): Logger {
+export function createLogger(strict: boolean): Logger {
   let warningCount = 0
-
-  function logInfo(msg: string): void {
-    if (typeof client?.log === "function") {
-      void client.log({ service: "opencode-claude-bridge", level: "info", message: msg })
-    } else {
-      fallbackLog.info(msg)
-    }
-  }
-
-  function logWarn(msg: string): void {
-    if (typeof client?.log === "function") {
-      void client.log({ service: "opencode-claude-bridge", level: "warn", message: msg })
-    } else {
-      fallbackLog.warn(msg)
-    }
-  }
-
   return {
     info(msg) {
-      logInfo(msg)
+      log.info(msg)
     },
     warn(msg, opts) {
       const fatalInStrict = opts?.fatalInStrict ?? true
       warningCount++
       if (strict && fatalInStrict) throw new BridgeError(msg)
-      logWarn(msg)
+      log.warn(msg)
     },
     hadWarnings() {
       return warningCount > 0