@zhihand/mcp 0.17.0 → 0.17.2

package/dist/daemon/dispatcher.js

@@ -387,8 +387,9 @@ function dispatchGemini(prompt, startTime, log, model) {
 }
 // ── Codex Dispatch ─────────────────────────────────────────
 function dispatchCodex(prompt, startTime, model) {
-    // codex exec --full-auto --skip-git-repo-check --json [-m model] <prompt>
-    const args = ["exec", "--full-auto", "--skip-git-repo-check", "--json"];
+    // --dangerously-bypass-approvals-and-sandbox is required so MCP tool calls
+    // are not auto-cancelled in non-interactive mode (--full-auto cancels them)
+    const args = ["exec", "--dangerously-bypass-approvals-and-sandbox", "--skip-git-repo-check", "--json"];
     const codexModel = model ?? process.env.CLAUDE_CODEX_MODEL;
     if (codexModel) {
         args.push("-m", codexModel);
@@ -412,48 +413,16 @@ function dispatchClaude(prompt, startTime, model) {
     activeChild = child;
     return collectChildOutput(child, startTime);
 }
-// ── Codex JSONL Output Parser ──────────────────────────────
-/** Parse codex JSONL output and extract agent message text. */
-function parseCodexJsonl(raw) {
-    const lines = raw.split("\n").filter(Boolean);
-    const texts = [];
-    let hasError = false;
-    for (const line of lines) {
-        try {
-            const event = JSON.parse(line);
-            const type = String(event.type ?? "");
-            // Extract text from completed agent messages
-            if (type === "item.completed") {
-                const item = event.item;
-                if (item && typeof item.text === "string" && item.text.trim()) {
-                    texts.push(item.text.trim());
-                }
-            }
-            // Capture errors
-            if (type === "error") {
-                const msg = String(event.message ?? "");
-                if (msg)
-                    texts.push(`Error: ${msg}`);
-                hasError = true;
-            }
-            if (type === "turn.failed") {
-                hasError = true;
-            }
-        }
-        catch {
-            // Not valid JSON — skip (truncated line or stderr mixed in)
-        }
-    }
-    if (texts.length > 0) {
-        return { text: texts.join("\n\n"), success: !hasError };
-    }
-    return { text: raw.trim(), success: false };
-}
+/**
+ * Collect codex JSONL output with streaming line parsing.
+ * Processes each JSONL line as it arrives so we extract agent text
+ * without buffering large binary payloads (e.g. base64 screenshots).
+ */
 function collectCodexOutput(child, startTime) {
     return new Promise((resolve) => {
-        const chunks = [];
-        let totalBytes = 0;
-        let truncated = false;
+        const texts = [];
+        let hasError = false;
+        let lineBuffer = "";
         let settled = false;
         function settle(result) {
             if (settled)
@@ -461,36 +430,59 @@ function collectCodexOutput(child, startTime) {
             settled = true;
             resolve(result);
         }
-        const timer = setTimeout(() => { closeChild(child); }, CLI_TIMEOUT);
-        const collectOutput = (data) => {
-            if (truncated)
+        function processLine(line) {
+            if (!line.trim())
                 return;
-            totalBytes += data.length;
-            if (totalBytes > MAX_OUTPUT_BYTES) {
-                truncated = true;
-                chunks.push(data.subarray(0, MAX_OUTPUT_BYTES - (totalBytes - data.length)));
+            try {
+                const event = JSON.parse(line);
+                const type = String(event.type ?? "");
+                if (type === "item.completed") {
+                    const item = event.item;
+                    if (item && typeof item.text === "string" && item.text.trim()) {
+                        texts.push(item.text.trim());
+                    }
+                }
+                if (type === "error") {
+                    const msg = String(event.message ?? "");
+                    if (msg)
+                        texts.push(`Error: ${msg}`);
+                    hasError = true;
+                }
+                if (type === "turn.failed") {
+                    hasError = true;
+                }
             }
-            else {
-                chunks.push(data);
+            catch {
+                // Not valid JSON — skip
+            }
+        }
+        const timer = setTimeout(() => { closeChild(child); }, CLI_TIMEOUT);
+        const onData = (data) => {
+            lineBuffer += data.toString("utf8");
+            const lines = lineBuffer.split("\n");
+            // Keep the last (possibly incomplete) line in the buffer
+            lineBuffer = lines.pop() ?? "";
+            for (const line of lines) {
+                processLine(line);
             }
         };
-        child.stdout?.on("data", collectOutput);
-        child.stderr?.on("data", collectOutput);
+        child.stdout?.on("data", onData);
+        // stderr is not JSONL, just discard
+        child.stderr?.resume();
         child.on("close", (code) => {
             clearTimeout(timer);
             activeChild = null;
+            // Process any remaining data in the buffer
+            if (lineBuffer.trim())
+                processLine(lineBuffer);
             const durationMs = Date.now() - startTime;
-            const raw = Buffer.concat(chunks).toString("utf8").trim();
-            const parsed = parseCodexJsonl(raw);
-            let text = parsed.text;
-            if (truncated)
-                text += "\n\n[Output truncated at 100KB]";
+            let text = texts.join("\n\n");
             if (!text) {
                 text = code === 0
                     ? "Task completed (no output)."
                     : `CLI process exited with code ${code}.`;
             }
-            settle({ text, success: parsed.success && code === 0, durationMs });
+            settle({ text, success: !hasError && code === 0, durationMs });
         });
         child.on("error", (err) => {
             clearTimeout(timer);

package/dist/index.js

@@ -5,7 +5,7 @@ import { controlSchema, screenshotSchema, pairSchema } from "./tools/schemas.js"
 import { executeControl } from "./tools/control.js";
 import { handleScreenshot } from "./tools/screenshot.js";
 import { handlePair } from "./tools/pair.js";
-const PACKAGE_VERSION = "0.17.0";
+const PACKAGE_VERSION = "0.17.2";
 export function createServer(deviceName) {
     const server = new McpServer({
         name: "zhihand",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zhihand/mcp",
-  "version": "0.17.0",
+  "version": "0.17.2",
   "private": false,
   "type": "module",
   "description": "ZhiHand MCP Server — phone control tools for Claude Code, Codex, Gemini CLI, and OpenClaw",