@evandu/super-pi 0.1.39 → 0.1.41

package/agents/super-pi-task/index.ts

@@ -5,6 +5,7 @@ import {
   executeTask,
   readTaskConfig,
   type TaskParams,
+  type TaskToolResult,
 } from "./core.ts";
 import {
   WORKFLOW_DESCRIPTION,
@@ -110,7 +111,34 @@ const WorkflowParameters = Type.Object({
 type ExtensionApi = { registerTool(tool: ExtensionToolDefinition): void };
 
 const extension = (pi: ExtensionApi): void => {
-  pi.registerTool({ name: "question", label: "Question", description: "Ask the user a blocking single or multiple choice question with an optional extra note.", parameters: QuestionParameters, execute: async (_id, raw) => { const endpoint = process.env.SUPER_PI_QUESTION_ENDPOINT; const conversationId = process.env.SUPER_PI_TASK_CONVERSATION; if (!endpoint || !conversationId) return { error: "Interactive question is unavailable" }; const p = raw as { question: string; options: string[]; multiple?: boolean; extraNote?: boolean }; const response = await fetch(endpoint, { method: "POST", headers: { "content-type": "application/json" }, body: JSON.stringify({ conversationId, ...p }) }); return await response.json(); } });
+  // THE QUESTION TOOL. It blocks on an HTTP request until the person answers, and the answer comes back
+  // as the tool result, which is what keeps the run alive across the exchange - the model does not decide
+  // to continue, it cannot: the tool has not returned.
+  //
+  // IT RETURNS THE SHAPE EVERY OTHER TOOL HERE RETURNS, AND THAT IS NOT COSMETIC. `TaskToolResult` is
+  // `{content: [{type:"text", text}], details}` - `todo`, `task` and `workflow` all build it through the
+  // local `text()` helper. This tool used to `return` the parsed JSON object bare, which Pi does not read
+  // as a tool result at all: the model received an EMPTY result and said so - 「我无法复述你的答案，
+  // question 工具调用后返回了空结果」. The answer was in the store the whole time; it simply never arrived.
+  //
+  // A REFUSED QUESTION THROWS, and that is the whole of the dismissal semantics: a failed call is what
+  // stops the run. OpenCode is the reference and is explicit - rejecting a question fails the deferred
+  // and the tool dies (`packages/core/src/question.ts:128-141`). Nothing is invented to fill the gap, not
+  // `options[0]` and not an empty selection, which would look to the model exactly like a real choice.
+  pi.registerTool({ name: "question", label: "Question", description: "Ask the user a blocking single or multiple choice question with an optional extra note.", parameters: QuestionParameters, execute: async (_id, raw): Promise<TaskToolResult> => {
+    const endpoint = process.env.SUPER_PI_QUESTION_ENDPOINT;
+    const conversationId = process.env.SUPER_PI_TASK_CONVERSATION;
+    const text = (body: string, isError?: boolean): TaskToolResult => ({ content: [{ type: "text", text: body }], details: undefined, ...(isError ? { isError: true } : {}) });
+    if (!endpoint || !conversationId) return text("This run cannot ask the user anything: the question endpoint is not available.", true);
+    const p = raw as { question: string; options: string[]; multiple?: boolean; extraNote?: boolean };
+    const response = await fetch(endpoint, { method: "POST", headers: { "content-type": "application/json" }, body: JSON.stringify({ conversationId, ...p }) });
+    const payload = await response.json() as { error?: string; answers?: { id: string; selected: string[]; note?: string }[]; summary?: string };
+    if (!response.ok || payload?.error) return text(payload?.error ?? `The question could not be delivered (HTTP ${response.status}).`, true);
+    // READABLE, NOT JUST PRESENT. The bare `{answers:[{id, selected}]}` carries a UUID and an array, and
+    // a model handed that has to guess what it means - which it did, describing a real answer as a
+    // cancellation. The text says what was asked and what was chosen, in words.
+    return text(JSON.stringify({ answers: (payload.answers ?? []).map((answer) => ({ question: p.question, selected: answer.selected, ...(answer.note ? { note: answer.note } : {}) })), ...(payload.summary ? { summary: payload.summary } : {}) }));
+  } });
   // Read ONCE at load, not per call: the environment cannot change under a running Pi, and `prepareLoadout`
   // is synchronous - it cannot await a fetch for a roster - so the roster has to already be here.
   const config = readTaskConfig(process.env);