@khalilgharbaoui/opencode-claude-code-plugin 0.34.1 → 0.35.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@khalilgharbaoui/opencode-claude-code-plugin",
-  "version": "0.34.1",
+  "version": "0.35.1",
   "description": "Claude Code CLI provider plugin for opencode",
   "author": "Khalil Gharbaoui",
   "type": "module",
@@ -21,11 +21,11 @@
     "build": "tsup",
     "dev": "tsup --watch",
     "typecheck": "tsc --noEmit",
-    "test": "OPENCODE_CLAUDE_CODE_LOG_FILE=0 tsx --test test-bridge.ts test-broker.ts test-proxy-mcp.ts test-proxy-task.ts test-auto-continue.ts test-has-new-user-content.ts test-get-claude-user-message.ts test-logger.ts test-cli-args.ts test-permission-presets.ts test-session-manager.ts test-compaction-model.ts test-tool-mapping.ts test-cwd-resolution.ts test-todo-ledger.ts test-session-affinity.ts test-config-models.ts test-ask-user-question.ts test-claude-session-wrapper.ts test-spawn-env.ts test-respawn.ts test-startup-diagnostics.ts test-subagent-hint.ts test-exit-plan-mode-question.ts test-compress-tool.ts test-agent-models.ts test-side-question.ts test-btw-command.ts test-effort-sessions.ts test-tool-block-index.ts test-context-usage.ts test-skill-bridge.ts test-turn-stats.ts test-cli-events.ts test-cli-events-stream.ts test-result-fallback.ts test-doctor.ts test-configure-skill.ts test-unattended-replay.ts test-process-lifecycle.ts test-account-failover.ts test-host-tools.ts test-v2-entrypoint.ts test-v2-client.ts test-tmp-dir.ts test-cleanup-stale.ts test-account-wrapper.ts test-runtime-status-sessions.ts test-index-hooks.ts test-silent-turn.ts test-mcp-tool-result-name.ts test-model-fallback.ts test-do-generate.ts test-interactive-usage.ts test-background-subagents.ts test-interactive-result.ts test-cli-probe-cache.ts test-mcp-late-connect.ts test-prologue-abort.ts"
+    "test": "OPENCODE_CLAUDE_CODE_LOG_FILE=0 tsx --test test-bridge.ts test-broker.ts test-proxy-mcp.ts test-proxy-task.ts test-auto-continue.ts test-has-new-user-content.ts test-get-claude-user-message.ts test-logger.ts test-cli-args.ts test-permission-presets.ts test-session-manager.ts test-compaction-model.ts test-tool-mapping.ts test-cwd-resolution.ts test-todo-ledger.ts test-session-affinity.ts test-config-models.ts test-ask-user-question.ts test-claude-session-wrapper.ts test-spawn-env.ts test-respawn.ts test-startup-diagnostics.ts test-subagent-hint.ts test-exit-plan-mode-question.ts test-compress-tool.ts test-agent-models.ts test-side-question.ts test-btw-command.ts test-effort-sessions.ts test-tool-block-index.ts test-context-usage.ts test-skill-bridge.ts test-turn-stats.ts test-cli-events.ts test-cli-events-stream.ts test-result-fallback.ts test-doctor.ts test-configure-skill.ts test-unattended-replay.ts test-process-lifecycle.ts test-account-failover.ts test-host-tools.ts test-v2-entrypoint.ts test-v2-client.ts test-tmp-dir.ts test-cleanup-stale.ts test-account-wrapper.ts test-runtime-status-sessions.ts test-index-hooks.ts test-silent-turn.ts test-mcp-tool-result-name.ts test-model-fallback.ts test-do-generate.ts test-interactive-usage.ts test-background-subagents.ts test-interactive-result.ts test-cli-probe-cache.ts test-mcp-late-connect.ts test-prologue-abort.ts test-diagnostic-bundle.ts test-ids.ts",
+    "generate:log-messages": "tsx scripts/generate-log-messages.ts"
   },
   "dependencies": {
     "@ai-sdk/provider": "^3.0.8",
-    "@ai-sdk/provider-utils": "^3.0.8",
     "jsonc-parser": "3.3.1"
   },
   "devDependencies": {

package/skills/claude-code-plugin/SKILL.md

@@ -679,6 +679,8 @@ configured/env directory). INFO is enough for startup diagnostics. Add
 `"level": "debug"` only if needed for lower-level events; `mode: "debug"` alone does
 not do that. Capture a bounded, redacted excerpt, then disable temporary logging and
 restart. Logs rotate above 5 MB to `plugin.log.1`, which can also contain private data.
+Never hand either file to anyone: for anything shared, use `/claude-code-doctor bundle`,
+which is redacted by allowlist (see "Filing an issue" below).
 
 ### Upgrade the plugin
 
@@ -814,6 +816,16 @@ warned about content that did not load. Read it whenever bridged skills are miss
 `opencode-skills@...` row is the skill bridge itself, which is a plugin bug to report,
 not something to fix in the user's config.
 
+A **Hooks Claude Code ran that failed** section appears only when one did. These are the
+user's own Claude Code hooks (`hook_response` with `outcome: "error"` or a non-zero
+`exit_code`), not opencode's. A failing `SessionStart` hook is otherwise invisible: the
+CLI drops its contribution and the turn succeeds, so the context it was meant to add is
+missing from every turn on that process. Read it whenever a hook's effect is absent. The
+first failure is also a WARN in the terminal. Only the hook's `stderr` is shown, capped
+at 200 characters, because its stdout is spliced into the model's context. The plugin
+never passes `--include-hook-events`, so only `SessionStart` (and `Setup`) hooks are
+reported at all; `cancelled` is not a failure, since an abort produces it.
+
 A **Background subagents** section is always printed: whether `background` was offered
 to Claude and `task_status` / `task_cancel` registered, the opencode major, what decided
 it, and the background tasks this process collected or cancelled. It reads
@@ -832,6 +844,39 @@ runs the user's `SessionStart` hooks and takes a few seconds; say that when sugg
 it. Do not propose `--bare` to skip the hooks: it never reads OAuth, so it reports
 nothing about a subscription.
 
+### Filing an issue: /claude-code-doctor bundle
+
+`/claude-code-doctor bundle` is what to tell a user to paste into a GitHub issue. It
+returns the normal report plus this process's recent `NOTICE`/`WARN`/`ERROR` lines from
+`plugin.log`, redacted. Unlike `usage` it starts no process, so it stays instant.
+**Never ask a user to attach `plugin.log` itself**: it has no redaction guarantee and
+can hold whole system prompts, spawn argv with `--settings` JSON, bridged MCP config
+paths and session ids. The same applies to the rotated `plugin.log.1`.
+A bundle still keeps folder paths below home (as `~/...`) and the `accounts` names, so
+tell the user to read it before pasting it publicly.
+
+The redaction is an allowlist, not a filter. Per line it keeps the timestamp, the level,
+the message **only** when it is one of the message literals extracted from the plugin's
+own source (`src/log-messages.ts`, generated by `npm run generate:log-messages`), and
+only data fields whose key is on an explicit allowlist and whose value is then the kind
+that entry declares: versions, counts, booleans, enums, durations, exit codes, model /
+tool / server names, paths and the loopback proxy URL. The allowlist applies at every
+nesting depth. Everything else, every unknown key included, becomes
+`[redacted, N chars]`, which keeps the shape without the content. Session ids become a
+per-bundle salted hash, so lines correlate inside one paste and nowhere else, and the
+home directory becomes `~` across the whole report including the table.
+
+Never in a bundle: prompt or reply text, system prompts, tool inputs or outputs, file
+contents, environment values, bearer tokens, the proxy `authToken`, API keys,
+`Authorization` headers, MCP server env or headers, URL credentials or query strings,
+or the raw spawn argv (option names are kept, every value is replaced).
+
+Capped at 120 lines and 24,000 bytes, newest first, with a line saying how many were
+left out. With file logging off the section says so, tells the user to relaunch under
+`OPENCODE_CLAUDE_CODE_LOG_FILE=1` or set `logging.file`, and the report is still
+returned. A message a maintainer cannot read in a bundle means that warning's text is
+built at runtime; its data fields still carry the diagnosis.
+
 Claude Code stream events the plugin now surfaces without debug logging: a rate-limit
 rejection, a context compaction the CLI did on its own, a `result` subtype other than
 `success` (which now finishes the turn as an error, not a clean stop), and a failed
@@ -865,6 +910,7 @@ Prefer the doctor: it needs no logging change and no restart.
 | No `claude-code` provider or model in the picker at all | The plugin never loaded, or it loaded and the CLI was not usable | Check for a `plugin ready` line first: absent means not loaded (wrong `plugin`/`plugins` key, a 2.x local install not pointing at `dist/`, or no full relaunch), present with `claudeCli.version: not detected` means the binary did not answer `--version`, which also disables every version-gated flag |
 | `Model unavailable` for a model id the user typed | The provider id is not what they assumed | Use the id the ready block's `providers` field lists. With no `accounts` configured on opencode 2 the id is `claude-code`, so `claude-code-default/<model>` fails while the plugin is healthy (measured on opencode 2.0.16, 2026-09-27). Declaring `accounts` is what creates `claude-code-default`; on 1.x with accounts the ids are `claude-code-default` / `claude-code-<name>` and never a bare `claude-code` |
 | `Tool result name changed`, turn aborts, on opencode 2 | Before 0.28.1 a CLI-executed tool's result reached opencode under a different name than its call, and 2.0.16 aborts the turn on the mismatch, breaking every Claude-side MCP server call | Upgrade to 0.28.1+ **and fully relaunch every opencode window**; plugin code is read once at process start, so upgrading the package under a running window changes nothing |
+| A `SessionStart` hook the user configured has no visible effect | It exited non-zero and Claude Code discarded its contribution; the turn still succeeded | Read the **Hooks Claude Code ran that failed** section of `/claude-code-doctor` for the exit code and the hook's stderr. It is their Claude Code settings to fix, not the plugin's |
 | `plugin ready` missing from the log | Logging is off (the default), or the plugin genuinely did not load | Confirm `OPENCODE_CLAUDE_CODE_LOG_FILE=1` and a relaunch before concluding anything. `/claude-code-doctor` answers the same questions with no logging change |
 | "Failed to authenticate: OAuth session expired", one account, turns failing in milliseconds | That account's CLI login lapsed | `claude auth status` for it, then log in again with the command the `▌ **claude account:**` note prints (`CLAUDE_CONFIG_DIR=<that account's dir> claude auth login`). Restart opencode after: a switch taken from the failover form lasts until restart. Login is a user action, never a diagnostic probe |
 | A tool call reported as rejected although it ran | Two fixed causes: opencode 1.18.32 aborts the provider signal of every step ending in tool calls, read as an operator stop (0.26.1); and a call waiting on an unanswered permission prompt was rejected at the flat 10-minute deadline, after which the late approval cancelled Claude's next call (0.26.2) | Upgrade to 0.26.2+ and relaunch. Do NOT raise `proxyToolTimeoutMs` for this: a deadline now waits while opencode reports the session busy |