@khalilgharbaoui/opencode-claude-code-plugin 0.4.22 → 0.5.0

package/README.md

@@ -313,6 +313,32 @@ Set `permissionMode: "plan"` to forward `--permission-mode plan` to Claude. The
 
 ---
 
+## AskUserQuestion
+
+opencode has no native structured ask-question executor to proxy through (unlike `Bash`/`Task`), so the plugin handles `AskUserQuestion` specially:
+
+1. **It renders the full question.** The tool's payload — every question, header, option label, and option description — is emitted as readable markdown into the assistant stream so the user actually sees the choices (same approach as `ExitPlanMode`).
+2. **It is never auto-allowed at the CLI gate.** Allowing it would let the headless Claude CLI resolve its own question (no TTY → fabricated/empty answer) and proceed on a guess. `controlRequestBehaviorForTool` hard-denies `AskUserQuestion` and returns a message telling the model to wait for the operator's answer — or, if the run is non-interactive, to proceed with the single most reasonable option and state its assumption rather than stall.
+
+This hard-deny sits **below** `controlRequestToolBehaviors` in precedence but **above** the global `controlRequestBehavior`. So:
+
+- The global `controlRequestBehavior: "allow"` does **not** override it (interactive setups stay correct by default).
+- An explicit per-tool entry **does**. For a fully unattended/automated deployment that prefers "guess and continue" over "stop and wait", restore the old auto-allow:
+
+  ```json
+  "provider": {
+    "claude-code": {
+      "options": {
+        "controlRequestToolBehaviors": { "AskUserQuestion": "allow" }
+      }
+    }
+  }
+  ```
+
+  With `"allow"`, the Claude CLI answers its own `AskUserQuestion` internally and the run never blocks — appropriate only when no operator is watching and forward progress matters more than a correct decision.
+
+---
+
 ## Compaction
 
 When you run `/compact` in opencode, the plugin handles it on a short-lived dedicated Claude CLI spawn instead of routing it through your main conversation process. Three reasons:
@@ -435,6 +461,7 @@ plugin internals.
 - No streaming of tool inputs as they're being constructed (Anthropic's `input_json_delta`); the plugin emits them once complete.
 - Raw chain-of-thought is not available. Claude 4 family models ship summarized thinking only. See [Extended thinking](#extended-thinking) for the full picture.
 - Recommended Claude Code CLI: **2.1.142+**. Older CLIs work for everything else but skip the `--thinking-display` flag, so Claude Opus 4.7 turns may render empty Thinking rows. If something breaks after a Claude Code update, the CLI version is the first thing to check.
+- **Subagent todos require explicit permission.** opencode's task tool gates `todowrite` per subagent: without a `permission: { todowrite: "allow" }` rule on the subagent definition, opencode injects `todowrite: false` into the tools dict and the plugin's synthetic `todowrite` emissions surface as `⚙ invalid todowrite` rows. The built-in `general` subagent denies `todowrite` by default; use a custom subagent for parallel work that needs todo visibility. Subagent todos render inline in the **subagent's** session view (navigate with the TUI's `session.child.next` / `session.parent` commands), not in the parent session's panel.
 
 ---
 
@@ -481,6 +508,16 @@ git push origin master --follow-tags
 
 The GitHub Actions workflow at `.github/workflows/publish.yml` runs `npm publish --access public` on tag push (requires `NPM_TOKEN` secret in the repo settings — use a classic automation token so 2FA isn't required at workflow time).
 
+## Star History
+
+<a href="https://www.star-history.com/?repos=khalilgharbaoui%2Fopencode-claude-code-plugin&type=date&legend=top-left">
+ <picture>
+   <source media="(prefers-color-scheme: dark)" srcset="https://api.star-history.com/chart?repos=khalilgharbaoui/opencode-claude-code-plugin&type=date&theme=dark&legend=top-left" />
+   <source media="(prefers-color-scheme: light)" srcset="https://api.star-history.com/chart?repos=khalilgharbaoui/opencode-claude-code-plugin&type=date&legend=top-left" />
+   <img alt="Star History Chart" src="https://api.star-history.com/chart?repos=khalilgharbaoui/opencode-claude-code-plugin&type=date&legend=top-left" />
+ </picture>
+</a>
+
 ## License
 
 MIT. See [LICENSE](./LICENSE).

package/dist/index.js

@@ -124,6 +124,91 @@ var log = {
   }
 };
 
+// src/todo-ledger.ts
+var ledgers = /* @__PURE__ */ new Map();
+var PENDING_CREATE_TTL_MS = 6e4;
+var TASK_CREATED_PATTERN = /Task\s*#?\s*(\d+)\s+created/i;
+var VALID_STATUSES = /* @__PURE__ */ new Set(["pending", "in_progress", "completed"]);
+function getOrCreate(sessionId) {
+  let ledger = ledgers.get(sessionId);
+  if (!ledger) {
+    ledger = { todos: /* @__PURE__ */ new Map(), pendingCreates: /* @__PURE__ */ new Map() };
+    ledgers.set(sessionId, ledger);
+  }
+  return ledger;
+}
+function prunePending(ledger) {
+  const cutoff = Date.now() - PENDING_CREATE_TTL_MS;
+  for (const [id, pending] of ledger.pendingCreates) {
+    if (pending.createdAt < cutoff) ledger.pendingCreates.delete(id);
+  }
+}
+function materialize(ledger) {
+  return Array.from(ledger.todos.values());
+}
+function resolveSubject(input) {
+  const subject = typeof input?.subject === "string" ? input.subject.trim() : "";
+  if (subject) return subject;
+  const description = typeof input?.description === "string" ? input.description.trim() : "";
+  if (description) return description;
+  return "(no subject)";
+}
+function applyTaskCreateToolUse(sessionId, toolUseId, input) {
+  if (!sessionId || !toolUseId) return;
+  const ledger = getOrCreate(sessionId);
+  prunePending(ledger);
+  ledger.pendingCreates.set(toolUseId, {
+    subject: resolveSubject(input),
+    createdAt: Date.now()
+  });
+}
+function applyTaskCreateToolResult(sessionId, toolUseId, resultText) {
+  if (!sessionId || !toolUseId) return null;
+  const ledger = ledgers.get(sessionId);
+  if (!ledger) return null;
+  const pending = ledger.pendingCreates.get(toolUseId);
+  if (!pending) return null;
+  ledger.pendingCreates.delete(toolUseId);
+  const match = typeof resultText === "string" ? resultText.match(TASK_CREATED_PATTERN) : null;
+  if (!match) {
+    log.debug("TaskCreate result did not match expected format", { sessionId, toolUseId, resultText });
+    return null;
+  }
+  const claudeId = match[1];
+  if (ledger.todos.has(claudeId)) {
+    log.debug("TaskCreate result for already-known claude id; overwriting", { sessionId, claudeId });
+  }
+  ledger.todos.set(claudeId, { id: claudeId, content: pending.subject, status: "pending" });
+  return materialize(ledger);
+}
+function applyTaskUpdate(sessionId, input) {
+  if (!sessionId) return null;
+  const taskId = typeof input?.taskId === "string" ? input.taskId : null;
+  if (!taskId) return null;
+  const ledger = ledgers.get(sessionId);
+  if (!ledger) return null;
+  const entry = ledger.todos.get(taskId);
+  if (!entry) {
+    log.debug("TaskUpdate for unknown task id", { sessionId, taskId });
+    return null;
+  }
+  if (input?.status === "deleted") {
+    ledger.todos.delete(taskId);
+    return materialize(ledger);
+  }
+  if (typeof input?.status === "string" && VALID_STATUSES.has(input.status)) {
+    entry.status = input.status;
+  }
+  if (typeof input?.subject === "string" && input.subject.trim().length > 0) {
+    entry.content = input.subject.trim();
+  }
+  return materialize(ledger);
+}
+function clearLedger(sessionId) {
+  if (!sessionId) return;
+  ledgers.delete(sessionId);
+}
+
 // src/tool-mapping.ts
 function mapToolInput(name, input) {
   if (!input) return input;
@@ -199,17 +284,42 @@ var CLAUDE_INTERNAL_TOOLS = /* @__PURE__ */ new Set([
   "ToolSearch",
   "Agent",
   "AskFollowupQuestion",
-  "TaskCreate",
-  "TaskUpdate",
   "TaskList",
   "TaskGet",
   "TaskStop"
 ]);
+function emitTodoWrite(todos) {
+  return {
+    name: "todowrite",
+    input: {
+      todos: todos.map((todo) => ({
+        id: todo.id,
+        content: todo.content,
+        status: todo.status,
+        priority: "medium"
+      }))
+    },
+    executed: false
+  };
+}
 function mapTool(name, input, opts) {
   if (CLAUDE_INTERNAL_TOOLS.has(name)) {
     log.debug("skipping Claude CLI internal tool", { name });
     return { name, input, executed: true, skip: true };
   }
+  if (name === "TaskCreate") {
+    if (opts?.sessionId && opts?.toolUseId) {
+      applyTaskCreateToolUse(opts.sessionId, opts.toolUseId, input);
+    }
+    return { name, input, executed: true, skip: true };
+  }
+  if (name === "TaskUpdate") {
+    if (opts?.sessionId) {
+      const list = applyTaskUpdate(opts.sessionId, input);
+      if (list !== null) return emitTodoWrite(list);
+    }
+    return { name, input, executed: true, skip: true };
+  }
   if (name === "EnterPlanMode") return { name: "plan_enter", input: {}, executed: false };
   if (name === "ExitPlanMode") return { name: "plan_exit", input, executed: false };
   if (name === "TodoWrite") {
@@ -1181,6 +1291,8 @@ function setClaudeSessionId(key, sessionId) {
   claudeSessions.set(key, sessionId);
 }
 function deleteClaudeSessionId(key) {
+  const claudeSessionId = claudeSessions.get(key);
+  if (claudeSessionId) clearLedger(claudeSessionId);
   claudeSessions.delete(key);
 }
 function spawnClaudeProcess(cliPath, cliArgs, cwd, sessionKey2, proxyServer, mcpHash, systemPromptFile) {
@@ -1885,6 +1997,46 @@ var AUTO_CONTINUE_PROMPT = "Continue the task from where you stopped. Do not sum
 function normalizeVisibleText(text) {
   return text.replace(/\s+/g, " ").trim();
 }
+function isAskUserQuestionTool(name) {
+  if (!name) return false;
+  const n = name.toLowerCase();
+  return n === "askuserquestion" || n === "ask_user_question";
+}
+function formatAskUserQuestion(input) {
+  const anyInput = input;
+  const questions = Array.isArray(anyInput?.questions) ? anyInput.questions : [];
+  if (questions.length === 0) {
+    const single = anyInput?.question ?? anyInput?.text;
+    const q = typeof single === "string" && single.trim() ? single.trim() : "Question?";
+    return `
+
+**${q}**
+
+_Reply with your answer to continue._
+
+`;
+  }
+  const out = ["\n\n"];
+  const multiQ = questions.length > 1;
+  questions.forEach((q, i) => {
+    const text = typeof q?.question === "string" && q.question.trim() || typeof q?.text === "string" && q.text.trim() || "Question?";
+    const header = typeof q?.header === "string" && q.header.trim() ? q.header.trim() : "";
+    out.push(`**${multiQ ? `${i + 1}. ` : ""}${text}**`);
+    if (header) out.push(` _(${header})_`);
+    out.push("\n\n");
+    const options = Array.isArray(q?.options) ? q.options : [];
+    options.forEach((opt, j) => {
+      const label = typeof opt?.label === "string" && opt.label.trim() || typeof opt === "string" && opt.trim() || `Option ${j + 1}`;
+      const desc = typeof opt?.description === "string" && opt.description.trim() ? ` \u2014 ${opt.description.trim()}` : "";
+      out.push(`${j + 1}. **${label}**${desc}
+`);
+    });
+    out.push(
+      q?.multiSelect === true ? "\n_Select one or more \u2014 reply with the numbers or labels._\n\n" : "\n_Reply with your choice (the number or label)._\n\n"
+    );
+  });
+  return out.join("");
+}
 function looksLikeQuestion(text) {
   const normalized = normalizeVisibleText(text).toLowerCase();
   if (!normalized) return false;
@@ -2213,6 +2365,7 @@ var ClaudeCodeLanguageModel = class {
         }
       }
     }
+    if (isAskUserQuestionTool(toolName)) return "deny";
     return this.config.controlRequestBehavior ?? "allow";
   }
   writeControlResponse(proc, requestId, response) {
@@ -2256,9 +2409,10 @@ var ClaudeCodeLanguageModel = class {
           toolName
         });
       } else {
+        const denyMessage = isAskUserQuestionTool(toolName) ? "Your question and its options have already been presented to the operator in full. Prefer to stop here and wait for their answer in the next message \u2014 do not silently guess. But if this is an automated or otherwise non-interactive run where no operator will reply, do not stall: proceed with the single most reasonable option and state, in one line, the assumption you made so it can be corrected later." : this.config.controlRequestDenyMessage ?? `Denied by opencode-claude-code policy for tool ${toolName}`;
         this.writeControlResponse(proc, requestId, {
           behavior: "deny",
-          message: this.config.controlRequestDenyMessage ?? `Denied by opencode-claude-code policy for tool ${toolName}`,
+          message: denyMessage,
           toolUseID: request.tool_use_id
         });
         log.info("control request auto-denied", {
@@ -2576,14 +2730,9 @@ var ClaudeCodeLanguageModel = class {
                 thinkingText += block.thinking;
               }
               if (block.type === "tool_use" && block.id && block.name) {
-                if (block.name === "AskUserQuestion" || block.name === "ask_user_question") {
+                if (isAskUserQuestionTool(block.name)) {
                   const parsedInput = block.input ?? {};
-                  const question = parsedInput?.question || "Question?";
-                  responseText += `
-
-_Asking: ${question}_
-
-`;
+                  responseText += formatAskUserQuestion(parsedInput);
                   continue;
                 }
                 if (block.name === "ExitPlanMode") {
@@ -2703,7 +2852,11 @@ ${plan}
         input: mappedInput,
         executed,
         skip
-      } = mapTool(tc.name, tc.args, { webSearch: this.config.webSearch });
+      } = mapTool(tc.name, tc.args, {
+        webSearch: this.config.webSearch,
+        sessionId: getClaudeSessionId(sk),
+        toolUseId: tc.id
+      });
       if (skip) continue;
       content.push({
         type: "tool-call",
@@ -3146,7 +3299,11 @@ ${plan}
                     const { name: mappedName, skip, executed } = mapTool(
                       block.name,
                       void 0,
-                      { webSearch: self.config.webSearch }
+                      {
+                        webSearch: self.config.webSearch,
+                        sessionId: getClaudeSessionId(sk),
+                        toolUseId: block.id
+                      }
                     );
                     if (!skip) {
                       controller.enqueue({
@@ -3236,22 +3393,12 @@ ${plan}
                     parsedInput = JSON.parse(tc.inputJson || "{}");
                   } catch {
                   }
-                  if (tc.name === "AskUserQuestion" || tc.name === "ask_user_question") {
-                    let question = "Question?";
-                    if (parsedInput?.questions && Array.isArray(parsedInput.questions) && parsedInput.questions.length > 0) {
-                      question = parsedInput.questions[0].question || parsedInput.questions[0].text || "Question?";
-                    } else {
-                      question = parsedInput?.question || parsedInput?.text || "Question?";
-                    }
+                  if (isAskUserQuestionTool(tc.name)) {
                     const askId = startTextBlock();
                     controller.enqueue({
                       type: "text-delta",
                       id: askId,
-                      delta: `
-
-_Asking: ${question}_
-
-`
+                      delta: formatAskUserQuestion(parsedInput)
                     });
                     endTextBlock();
                   } else if (tc.name === "ExitPlanMode") {
@@ -3280,7 +3427,11 @@ ${plan}
                       input: mappedInput,
                       executed,
                       skip
-                    } = mapTool(tc.name, parsedInput, { webSearch: self.config.webSearch });
+                    } = mapTool(tc.name, parsedInput, {
+                      webSearch: self.config.webSearch,
+                      sessionId: getClaudeSessionId(sk),
+                      toolUseId: tc.id
+                    });
                     if (!skip) {
                       toolCallsById.set(tc.id, {
                         id: tc.id,
@@ -3401,23 +3552,12 @@ ${plan}
                       name: block.name,
                       input: parsedInput
                     });
-                    if (block.name === "AskUserQuestion" || block.name === "ask_user_question") {
-                      let question = "Question?";
-                      if (parsedInput?.questions && Array.isArray(parsedInput.questions) && parsedInput.questions.length > 0) {
-                        const q = parsedInput.questions[0];
-                        question = q.question || q.text || "Question?";
-                      } else {
-                        question = parsedInput?.question || parsedInput?.text || "Question?";
-                      }
+                    if (isAskUserQuestionTool(block.name)) {
                       const askId = startTextBlock();
                       controller.enqueue({
                         type: "text-delta",
                         id: askId,
-                        delta: `
-
-_Asking: ${question}_
-
-`
+                        delta: formatAskUserQuestion(parsedInput)
                       });
                       endTextBlock();
                     } else if (block.name === "ExitPlanMode") {
@@ -3446,7 +3586,11 @@ ${plan}
                         input: mappedInput,
                         executed,
                         skip
-                      } = mapTool(block.name, parsedInput, { webSearch: self.config.webSearch });
+                      } = mapTool(block.name, parsedInput, {
+                        webSearch: self.config.webSearch,
+                        sessionId: getClaudeSessionId(sk),
+                        toolUseId: block.id
+                      });
                       if (!skip) {
                         if (!executed) skipResultForIds.add(block.id);
                         controller.enqueue({
@@ -3487,16 +3631,48 @@ ${plan}
                       });
                       continue;
                     }
+                    let resultText = "";
+                    if (typeof block.content === "string") {
+                      resultText = block.content;
+                    } else if (Array.isArray(block.content)) {
+                      resultText = block.content.filter(
+                        (c) => c.type === "text" && typeof c.text === "string"
+                      ).map((c) => c.text).join("\n");
+                    }
+                    const claudeSessionId = getClaudeSessionId(sk);
+                    if (claudeSessionId) {
+                      const list = applyTaskCreateToolResult(
+                        claudeSessionId,
+                        block.tool_use_id,
+                        resultText
+                      );
+                      if (list) {
+                        const synthId = `todowrite_${block.tool_use_id}`;
+                        controller.enqueue({
+                          type: "tool-input-start",
+                          id: synthId,
+                          toolName: "todowrite",
+                          providerExecuted: false
+                        });
+                        controller.enqueue({
+                          type: "tool-call",
+                          toolCallId: synthId,
+                          toolName: "todowrite",
+                          input: JSON.stringify({
+                            todos: list.map((t) => ({
+                              id: t.id,
+                              content: t.content,
+                              status: t.status,
+                              priority: "medium"
+                            }))
+                          }),
+                          providerExecuted: false
+                        });
+                        noteToolActivity();
+                      }
+                    }
                     const toolCall = toolCallsById.get(block.tool_use_id);
                     if (toolCall) {
-                      let resultText = "";
-                      if (typeof block.content === "string") {
-                        resultText = block.content;
-                      } else if (Array.isArray(block.content)) {
-                        resultText = block.content.filter(
-                          (c) => c.type === "text" && typeof c.text === "string"
-                        ).map((c) => c.text).join("\n");
-                      }
                       controller.enqueue({
                         type: "tool-result",
                         toolCallId: block.tool_use_id,