pi-opencode-cli-bridge 0.2.1 → 0.2.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-opencode-cli-bridge",
-  "version": "0.2.1",
+  "version": "0.2.4",
   "description": "Bridge the OpenCode CLI into Pi Coding Agent",
   "type": "module",
   "main": "./src/index.ts",

package/src/index.ts

@@ -1009,6 +1009,9 @@ export function parseToolCallResponse(
   text: string,
   allowedToolNames?: ReadonlySet<string>,
 ): ToolCallParseResult {
+  // A turn without tools (e.g. a compaction summary) cannot call any, so tool
+  // syntax it quotes is prose.
+  if (allowedToolNames?.size === 0) return { ok: true, calls: [] };
   const normalizedText = dsmlToMarkers(normalizeQuotedToolCallResponse(text));
   if (normalizedText === undefined) {
     return { ok: false, rejection: { reason: "incomplete_dsml" } };
@@ -1410,6 +1413,14 @@ async function createTempAgentDir(): Promise<string> {
   const pluginsDir = join(configDir, "plugins");
   await mkdir(agentsDir, { recursive: true });
   await mkdir(pluginsDir, { recursive: true });
+  // `opencode run` streams an OpenCode compaction summary as reply text, where
+  // its quoted markers break the turn. Pi owns compaction, so overflow must
+  // surface as an error instead.
+  await writeFile(
+    join(configDir, "opencode.json"),
+    JSON.stringify({ compaction: { auto: false } }),
+    "utf8",
+  );
   await writeFile(
     join(agentsDir, `${AGENT_ID}.md`),
     `---
@@ -1704,10 +1715,29 @@ export function streamOpenCode(
           toolCallRejectionMessage(toolCallResult, allowedToolNames),
         );
       }
-      const toolCalls = [...toolCallResult.calls, ...(nativeCalls ?? [])];
       // Unrecoverable marker text is stripped so it never leaks into the
       // visible chat response; remaining prose still displays normally.
       const displayText = toolCallResult.cleanedText ?? accumulatedText;
+      let textCalls = toolCallResult.calls;
+      // DeepSeek sometimes writes its only calls in the reasoning and replies
+      // with nothing, which would end the turn silently.
+      if (textCalls.length === 0 && !nativeCalls && !displayText.trim()) {
+        let reasoningResult = parseToolCallResponse(
+          accumulatedThinking,
+          allowedToolNames,
+        );
+        // Markers stripped with no call surviving were meant as calls too.
+        if (reasoningResult.ok && reasoningResult.calls.length === 0 && reasoningResult.cleanedText !== undefined) {
+          reasoningResult = { ok: false, rejection: { reason: "malformed_markers" } };
+        }
+        if (!reasoningResult.ok) {
+          throw new Error(
+            toolCallRejectionMessage(reasoningResult, allowedToolNames),
+          );
+        }
+        textCalls = reasoningResult.calls;
+      }
+      const toolCalls = [...textCalls, ...(nativeCalls ?? [])];
       if (toolCalls.length === 0 && displayText.trim()) {
         // Check if the model tried to use XML-style tool-use instead of
         // <pi_tool_call> markers — a common mistake with Claude-based models.