lark-remote 0.1.12 → 0.2.0

package/dist/runner/opencode/acp/translator.js

@@ -2,148 +2,40 @@
  * OpencodeAcpTranslator: translates opencode ACP session/update notifications
  * and session/request_permission server requests into AgentEvents.
  *
- * Channel choice (event contract in common/acp/protocol-types.ts header):
- * opencode's wire `content.text` IS an incremental delta
- * (opencode acp/event.ts:231-258 handlePartDelta sends `props.delta`
- * verbatim), so text/thinking chunks go straight onto the `assistant`
- * incremental channel — NO accumulation in the translator, NO turn_diff
- * snapshot. (Contrast kimi: same field is also a delta but kimi's contract
- * accumulates into turn_diff snapshots; both end with exactly one
- * concatenation in run-state, enforced by each side's seam test.)
+ * Extends BaseAcpTranslator (shared envelope dispatch, prompt-response
+ * stopReason mapping, tool_call normalization, usage occupancy). opencode's
+ * divergence from kimi is the content channel: wire `content.text` IS an
+ * incremental delta (opencode acp/event.ts:231-258 handlePartDelta sends
+ * `props.delta` verbatim), so text/thinking chunks go straight onto the
+ * `assistant` incremental channel — NO accumulation, NO turn_diff snapshot.
+ * (Contrast kimi: same field is also a delta but kimi's contract accumulates
+ * into turn_diff snapshots; both end with exactly one concatenation in
+ * run-state, enforced by each side's seam test.)
  *
  * Event mapping (source: opencode acp/service.ts + event.ts + tool.ts,
  * dev@1c965451b5):
  *   agent_message_chunk   → assistant/text delta (content.text passthrough)
  *   agent_thought_chunk   → assistant/thinking delta (content.text passthrough)
- *   tool_call             → assistant/tool_use (rawInput object passthrough;
- *                           missing → {} per contract, string → JSON.parse)
- *   tool_call_update      → user/tool_result (status:'failed' → is_error:true;
- *                           rawOutput is an OBJECT: {output} / {error})
- *   usage_update          → live context occupancy {used, size} + cost →
- *                           {total_tokens, context_limit, total_cost_usd}
- *   available_commands_update / plan / session_info_update /
- *   current_mode_update / config_option_update / user_message_chunk → discard
+ *   tool_call / tool_call_update / usage_update / plan / control-plane noise →
+ *                         base class (BaseAcpTranslator; opencode overrides
+ *                         tool_result content extraction + session cost)
  *   prompt stopReason:'end_turn'   → result success
  *   prompt stopReason:'cancelled'  → result interrupted (独立终态)
  *   prompt other stopReason/error  → result error
  *   session/request_permission     → approval_requested (optionId echo)
  */
-import { NotificationMethod, ServerRequestMethod, SessionEventType, } from '../../common/acp/protocol-types.js';
+import { deriveAcpAvailableDecisions, truncateWithEllipsis, } from '../../common/acp/protocol-helpers.js';
 import { getLogger } from '../../../logger/index.js';
+import { BaseAcpTranslator } from '../../common/acp/base-translator.js';
 // =============================================================================
 // Translator
 // =============================================================================
-export class OpencodeAcpTranslator {
-    /** Pending tool calls keyed by toolCallId (for matching tool_call_update). */
-    pendingToolCalls = new Map();
-    /** Current operation kind (turn vs compact). */
-    operationKind = 'turn';
-    /** Live usage snapshot from usage_update events (context occupancy only). */
-    liveUsage = {};
-    /** Session cost from usage_update cost.amount (USD). */
-    liveCostUsd;
-    /** Whether any usage_update has been seen this turn (no update → no usage). */
-    hasLiveUsage = false;
-    /** Current turn id, set by produceTurnStarted. */
-    currentTurnId = '';
-    /**
-     * Handle a notification from the ACP server and return translated events.
-     */
-    handleNotification(method, params) {
-        if (method !== NotificationMethod.SESSION_UPDATE) {
-            return [];
-        }
-        // Envelope: {sessionId, update: {sessionUpdate: ...}} (same as kimi).
-        const notif = params;
-        const update = notif.update;
-        switch (update.sessionUpdate) {
-            case SessionEventType.AGENT_MESSAGE_CHUNK:
-                return this.handleAgentMessageChunk(update);
-            case SessionEventType.AGENT_THOUGHT_CHUNK:
-                return this.handleAgentThoughtChunk(update);
-            case SessionEventType.TOOL_CALL:
-                return this.handleToolCall(update);
-            case SessionEventType.TOOL_CALL_UPDATE:
-                return this.handleToolCallUpdate(update);
-            case SessionEventType.USAGE_UPDATE:
-                return this.handleUsageUpdate(update);
-            case SessionEventType.PLAN:
-                getLogger().debug(`[opencode-acp-translator] discarding plan event`);
-                return [];
-            default:
-                // Control-plane noise (command list, mode/config echoes, user replay)
-                // — not content.
-                return [];
-        }
-    }
-    /**
-     * Handle a server request (reverse RPC from the ACP server).
-     */
-    handleServerRequest(id, method, params) {
-        if (method === ServerRequestMethod.REQUEST_PERMISSION) {
-            return this.handleRequestPermission(id, params);
-        }
-        return [];
-    }
-    /**
-     * Produce a turn_started event. Called by the runner when session/prompt
-     * is sent (no wire notification for this — self-produced).
-     */
-    produceTurnStarted(sessionId, turnId) {
-        this.currentTurnId = turnId;
-        return {
-            type: 'turn_started',
-            threadId: sessionId,
-            turnId,
-            operationKind: this.operationKind === 'compact' ? 'compaction' : 'turn',
-            timestamp: new Date().toISOString(),
-        };
-    }
-    /**
-     * Translate a prompt response into a result event.
-     * stopReason: end_turn → success; cancelled → interrupted; anything else
-     * (max_tokens / refusal / …) → error.
-     */
-    handlePromptResponse(sessionId, result) {
-        const subtype = result.stopReason === 'end_turn'
-            ? 'success'
-            : result.stopReason === 'cancelled'
-                ? 'interrupted'
-                : 'error';
-        return {
-            type: 'result',
-            subtype,
-            session_id: sessionId,
-            ...(this.hasLiveUsage ? { usage: this.liveUsageSnapshot() } : {}),
-            ...(this.liveCostUsd !== undefined ? { total_cost_usd: this.liveCostUsd } : {}),
-            ...(subtype === 'error'
-                ? { errorMessage: `Prompt ended with stopReason: ${result.stopReason}` }
-                : {}),
-        };
-    }
-    /**
-     * Produce an error result event (for JSON-RPC errors during prompt).
-     */
-    produceErrorResult(sessionId, errorMessage) {
-        return {
-            type: 'result',
-            subtype: 'error',
-            session_id: sessionId,
-            errorMessage,
-            ...(this.hasLiveUsage ? { usage: this.liveUsageSnapshot() } : {}),
-            ...(this.liveCostUsd !== undefined ? { total_cost_usd: this.liveCostUsd } : {}),
-        };
-    }
-    /**
-     * Set the operation kind (turn vs compact).
-     */
-    setOperationKind(kind) {
-        this.operationKind = kind;
-    }
+export class OpencodeAcpTranslator extends BaseAcpTranslator {
+    logTag = 'opencode-acp-translator';
     // =========================================================================
-    // Private notification handlers
+    // Content channel: wire deltas → assistant incremental channel
     // =========================================================================
-    handleAgentMessageChunk(event) {
+    handleAgentMessageChunk(event, _sessionId) {
         // opencode wire delta → assistant incremental channel directly
         // (event.ts handlePartDelta: content.text = props.delta). The reducer
         // appends assistant text — no accumulation here, no turn_diff.
@@ -157,7 +49,7 @@ export class OpencodeAcpTranslator {
             },
         ];
     }
-    handleAgentThoughtChunk(event) {
+    handleAgentThoughtChunk(event, _sessionId) {
         return [
             {
                 type: 'assistant',
@@ -168,78 +60,47 @@ export class OpencodeAcpTranslator {
             },
         ];
     }
-    handleToolCall(event) {
-        // opencode pendingToolCall always carries rawInput as an object
-        // (tool.ts:124-138), but the event contract requires defensive
-        // normalization: missing → {} (lazy-create), string → JSON.parse.
-        let input = event.rawInput ?? {};
-        if (typeof event.rawInput === 'string') {
-            try {
-                input = JSON.parse(event.rawInput);
-            }
-            catch {
-                // Keep raw string as-is
-            }
+    // =========================================================================
+    // opencode-specific hooks
+    // =========================================================================
+    /**
+     * opencode rawOutput is an OBJECT (tool.ts): completed →
+     * {output, metadata?}; failed → {error, metadata}; in_progress →
+     * content[] with shell output snapshot.
+     */
+    extractToolResultContent(event) {
+        const rawOutput = event.rawOutput;
+        if (rawOutput && typeof rawOutput === 'object') {
+            const rec = rawOutput;
+            if (typeof rec.output === 'string')
+                return rec.output;
+            if (typeof rec.error === 'string')
+                return rec.error;
         }
-        this.pendingToolCalls.set(event.toolCallId, {
-            id: event.toolCallId,
-            name: event.title,
-            input,
-        });
-        return [
-            {
-                type: 'assistant',
-                message: {
-                    content: [
-                        {
-                            type: 'tool_use',
-                            id: event.toolCallId,
-                            name: event.title,
-                            input,
-                        },
-                    ],
-                },
-                timestamp: new Date().toISOString(),
-            },
-        ];
-    }
-    handleToolCallUpdate(event) {
-        const isError = event.status === 'failed';
-        const content = extractToolResultContent(event);
-        return [
-            {
-                type: 'user',
-                message: {
-                    content: [
-                        {
-                            type: 'tool_result',
-                            tool_use_id: event.toolCallId,
-                            content,
-                            is_error: isError,
-                        },
-                    ],
-                },
-                timestamp: new Date().toISOString(),
-            },
-        ];
+        if (typeof rawOutput === 'string')
+            return rawOutput;
+        const content = event.content;
+        if (Array.isArray(content)) {
+            const texts = content
+                .map((item) => {
+                const c = item?.content;
+                return c?.type === 'text' && typeof c.text === 'string' ? c.text : '';
+            })
+                .filter((t) => t.length > 0);
+            if (texts.length > 0)
+                return texts.join('\n');
+        }
+        return '';
     }
-    handleUsageUpdate(event) {
-        // opencode usage_update: {used, size, cost:{amount, currency:'USD'}}
-        // (service.ts:653-663) — context occupancy + session cost, no
-        // input/output split.
-        this.liveUsage = {
-            total_tokens: event.used,
-            context_limit: event.size,
-        };
+    /** opencode usage_update also carries session cost: {cost:{amount}} (USD). */
+    onUsageSample(event) {
         const cost = event.cost;
         if (typeof cost?.amount === 'number') {
             this.liveCostUsd = cost.amount;
         }
-        this.hasLiveUsage = true;
-        return [];
     }
     // =========================================================================
-    // Private approval handlers
+    // Approval
     // =========================================================================
     handleRequestPermission(requestId, params) {
         // opencode always sends a toolCall (permission.ts:62-70 — built via
@@ -254,10 +115,10 @@ export class OpencodeAcpTranslator {
             requestId,
             kind: 'command',
             command: params.toolCall.title ?? undefined,
-            reason: rawInput ? truncate(stringify(rawInput), 200) : undefined,
+            reason: rawInput ? truncateWithEllipsis(stringify(rawInput), 200) : undefined,
             // §P4: 从服务端 options kind 派生——带 allow_always（或 approve_always）
             // 才提供「本会话总是允许」（acceptForSession）。
-            availableDecisions: this.deriveAvailableDecisions(params),
+            availableDecisions: deriveAcpAvailableDecisions(params.options ?? []),
         };
         return [
             {
@@ -272,66 +133,10 @@ export class OpencodeAcpTranslator {
             },
         ];
     }
-    /**
-     * Derive the approval decision list from the server's offered options.
-     * accept/decline/cancel are universal; acceptForSession requires an
-     * always-class option (opencode `allow_always`, kimi `approve_always`).
-     */
-    deriveAvailableDecisions(params) {
-        const decisions = ['accept', 'decline', 'cancel'];
-        const hasAlwaysOption = (params.options ?? []).some((opt) => opt.kind === 'allow_always' || opt.kind === 'approve_always');
-        if (hasAlwaysOption) {
-            decisions.push('acceptForSession');
-        }
-        return decisions;
-    }
-    // =========================================================================
-    // Private helpers
-    // =========================================================================
-    /**
-     * ResultEvent.usage nominally requires input_tokens/output_tokens, but the
-     * ACP wire has no such split — the live snapshot is context occupancy only.
-     * Cast is deliberate: consumers must tolerate undefined input/output.
-     */
-    liveUsageSnapshot() {
-        return { ...this.liveUsage };
-    }
 }
 // =============================================================================
 // Helpers
 // =============================================================================
-/**
- * Extract a display string from a tool_call_update. opencode shapes
- * (tool.ts): completed → rawOutput {output, metadata?}; failed → rawOutput
- * {error, metadata}; in_progress → content[] with shell output snapshot.
- */
-function extractToolResultContent(event) {
-    const rawOutput = event.rawOutput;
-    if (rawOutput && typeof rawOutput === 'object') {
-        const rec = rawOutput;
-        if (typeof rec.output === 'string')
-            return rec.output;
-        if (typeof rec.error === 'string')
-            return rec.error;
-    }
-    if (typeof rawOutput === 'string')
-        return rawOutput;
-    const content = event.content;
-    if (Array.isArray(content)) {
-        const texts = content
-            .map((item) => {
-            const c = item?.content;
-            return c?.type === 'text' && typeof c.text === 'string' ? c.text : '';
-        })
-            .filter((t) => t.length > 0);
-        if (texts.length > 0)
-            return texts.join('\n');
-    }
-    return '';
-}
-function truncate(text, maxLen) {
-    return text.length <= maxLen ? text : text.slice(0, maxLen) + '…';
-}
 function stringify(value) {
     if (typeof value === 'string')
         return value;

package/dist/runner/pi/rpc/index.d.ts

@@ -1,4 +1,2 @@
 export { PiRpcRunner } from './runner.js';
 export type { PiRpcRunnerOptions } from './runner.js';
-export { PiRpcClient } from './client.js';
-export { PiRpcTranslator } from './translator.js';

package/dist/runner/pi/rpc/index.js

@@ -1,4 +1,2 @@
 export { PiRpcRunner } from './runner.js';
-export { PiRpcClient } from './client.js';
-export { PiRpcTranslator } from './translator.js';
 //# sourceMappingURL=index.js.map

package/dist/runner/pi/rpc/translator.d.ts

@@ -30,8 +30,11 @@ export declare class PiRpcTranslator {
     private accOutput;
     private accCacheRead;
     private accCacheCreation;
+    private accReasoning;
     private lastTotalTokens;
     private hasUsage;
+    /** toolcall_start bookkeeping：占位不发事件，end 时补发完整 tool_use（C3.4）。 */
+    private pendingToolCall;
     /** Provider failure (stopReason="error") on the final assistant message. */
     private lastAssistantStopReason;
     private lastAssistantErrorMessage;

package/dist/runner/pi/rpc/translator.js

@@ -21,8 +21,11 @@ export class PiRpcTranslator {
     accOutput = 0;
     accCacheRead = 0;
     accCacheCreation = 0;
+    accReasoning = 0;
     lastTotalTokens;
     hasUsage = false;
+    /** toolcall_start bookkeeping：占位不发事件，end 时补发完整 tool_use（C3.4）。 */
+    pendingToolCall = false;
     /** Provider failure (stopReason="error") on the final assistant message. */
     lastAssistantStopReason;
     lastAssistantErrorMessage;
@@ -41,6 +44,7 @@ export class PiRpcTranslator {
             this.accOutput === 0 &&
             this.accCacheRead === 0 &&
             this.accCacheCreation === 0 &&
+            this.accReasoning === 0 &&
             this.lastTotalTokens === undefined) {
             return undefined;
         }
@@ -49,6 +53,8 @@ export class PiRpcTranslator {
             output_tokens: this.accOutput,
             cache_read_tokens: this.accCacheRead,
             cache_creation_tokens: this.accCacheCreation,
+            // 推理 token（pi usage.reasoning 累加）：> 0 才携带，缺省不展示。
+            ...(this.accReasoning > 0 ? { reasoning_tokens: this.accReasoning } : {}),
             ...(this.lastTotalTokens != null ? { total_tokens: this.lastTotalTokens } : {}),
         };
     }
@@ -92,9 +98,36 @@ export class PiRpcTranslator {
                 }
                 return [];
             }
+            case 'compaction_start': {
+                // 信息保真 C3.4：压缩过程用户可见（info 通知），不再静默。
+                const c = evt;
+                return [
+                    {
+                        type: 'notice',
+                        level: 'info',
+                        code: 'compaction',
+                        text: `上下文压缩中${c.reason ? `（${c.reason}）` : ''}`,
+                        timestamp: new Date().toISOString(),
+                    },
+                ];
+            }
+            case 'compaction_end': {
+                // 压缩中止（aborted）是 warn：用户需要知道压缩未完成。
+                const c = evt;
+                return [
+                    {
+                        type: 'notice',
+                        level: c.aborted ? 'warn' : 'info',
+                        code: 'compaction',
+                        text: c.aborted ? `上下文压缩未完成：${c.errorMessage ?? '已中止'}` : '上下文压缩完成',
+                        timestamp: new Date().toISOString(),
+                    },
+                ];
+            }
             default:
-                // turn_start/turn_end, agent_start/agent_end, compaction_start/end,
-                // tool_execution_*, and unknown types are ignored.
+                // turn_start/turn_end, agent_start/agent_end, tool_execution_*, and
+                // unknown types are ignored. (compaction_start/end 自 C3.4 起映射为
+                // notice 事件，不再静默。)
                 return [];
         }
     }
@@ -154,6 +187,9 @@ export class PiRpcTranslator {
     handleMessageEnd(evt) {
         const m = evt.message;
         if (m.role === 'assistant') {
+            // pendingToolCall 复位：消息边界后 start/end 配对作废（防御 start 后
+            // 永无 end 的异常流，避免跨消息串扰到下一条 assistant 消息）。
+            this.pendingToolCall = false;
             this.lastAssistantStopReason = m.stopReason;
             this.lastAssistantErrorMessage = m.stopReason === 'error' ? m.errorMessage : undefined;
             if (m.usage)
@@ -193,6 +229,7 @@ export class PiRpcTranslator {
         this.accOutput += u.output ?? 0;
         this.accCacheRead += u.cacheRead ?? 0;
         this.accCacheCreation += u.cacheWrite ?? 0;
+        this.accReasoning += u.reasoning ?? 0;
         if (u.totalTokens != null)
             this.lastTotalTokens = u.totalTokens;
     }
@@ -234,19 +271,20 @@ export class PiRpcTranslator {
                 }
                 break;
             case 'toolcall_start':
-                this.content.push({ type: 'tool_use', id: '', name: '', input: {} });
-                this.currentContentIndex++;
+                // 信息保真 C3.4：不再 push 空壳 tool_use（空 id/name 会被渲染成
+                // 无意义工具块）。只保留 bookkeeping，toolcall_end 补发完整事件。
+                this.pendingToolCall = true;
                 break;
             case 'toolcall_end':
-                if (this.currentContentIndex >= 0 &&
-                    this.content[this.currentContentIndex]?.type === 'tool_use' &&
-                    evt.toolCall) {
-                    this.content[this.currentContentIndex] = {
+                if (this.pendingToolCall && evt.toolCall) {
+                    this.content.push({
                         type: 'tool_use',
                         id: evt.toolCall.id,
                         name: evt.toolCall.name,
                         input: evt.toolCall.arguments,
-                    };
+                    });
+                    this.currentContentIndex = this.content.length - 1;
+                    this.pendingToolCall = false;
                 }
                 break;
             default:

package/dist/runner/probe.d.ts

@@ -1,24 +1,21 @@
 /**
  * Agent availability probe — detect whether each CLI binary is installed
- * by checking whether it resolves on PATH via `which`.
+ * via a pure-Node PATH lookup (platform seam `resolveExecutable`).
  *
- * Why `which` over `<binary> --help`:
+ * Why a PATH lookup over `<binary> --help`:
  * - `--help` starts a full Node.js process per agent (2–8 s on
  *   resource-constrained devices like Raspberry Pi). Under startup load
  *   (auto-resume spawns, CPU contention) probes time out and installed
  *   agents are falsely reported as unavailable.
- * - `which` is a lightweight PATH lookup (<10 ms), immune to CPU contention.
+ * - A PATH lookup is lightweight (<10 ms), immune to CPU contention.
  * - "Installed but broken" cases surface at run time via SpawningRunner's
  *   ENOENT/error handling (src/runner/common/spawning-runner.ts), which
  *   shows a friendly error card instead of a startup-time mislabel.
  *
- * Why `which` over `command -v`:
- * - `which` is a standalone executable (e.g. /usr/bin/which on macOS and
- *   Debian-based Linux), so no `shell: true` and no Node DEP0190 warning.
- * - Caveat: `which` is not POSIX-guaranteed. macOS, Raspberry Pi OS and
- *   other Debian-based distributions ship it by default; if Windows or
- *   minimal Alpine containers become supported, prefer a pure-Node
- *   PATH + X_OK lookup instead.
+ * Why the platform seam over spawning `which`:
+ * - `which` is not POSIX-guaranteed and does not exist on Windows; the seam
+ *   does PATH + X_OK on posix (same semantics) and PATH × PATHEXT on win32,
+ *   with no child process at all.
  */
 import type { AgentKind } from './types.js';
 /**
@@ -29,7 +26,7 @@ export declare function probeAgentAvailability(kind: AgentKind): Promise<boolean
 /**
  * Probe all agents concurrently.
  *
- * Concurrent (not sequential) because `which` is a lightweight PATH lookup
+ * Concurrent (not sequential) because the lookup is a lightweight PATH scan
  * (<10 ms) with no CPU contention risk — unlike the previous `--help`
  * probes, which needed to run one-at-a-time to avoid starving each other
  * on resource-constrained devices.

package/dist/runner/probe.js

@@ -1,26 +1,23 @@
 /**
  * Agent availability probe — detect whether each CLI binary is installed
- * by checking whether it resolves on PATH via `which`.
+ * via a pure-Node PATH lookup (platform seam `resolveExecutable`).
  *
- * Why `which` over `<binary> --help`:
+ * Why a PATH lookup over `<binary> --help`:
  * - `--help` starts a full Node.js process per agent (2–8 s on
  *   resource-constrained devices like Raspberry Pi). Under startup load
  *   (auto-resume spawns, CPU contention) probes time out and installed
  *   agents are falsely reported as unavailable.
- * - `which` is a lightweight PATH lookup (<10 ms), immune to CPU contention.
+ * - A PATH lookup is lightweight (<10 ms), immune to CPU contention.
  * - "Installed but broken" cases surface at run time via SpawningRunner's
  *   ENOENT/error handling (src/runner/common/spawning-runner.ts), which
  *   shows a friendly error card instead of a startup-time mislabel.
  *
- * Why `which` over `command -v`:
- * - `which` is a standalone executable (e.g. /usr/bin/which on macOS and
- *   Debian-based Linux), so no `shell: true` and no Node DEP0190 warning.
- * - Caveat: `which` is not POSIX-guaranteed. macOS, Raspberry Pi OS and
- *   other Debian-based distributions ship it by default; if Windows or
- *   minimal Alpine containers become supported, prefer a pure-Node
- *   PATH + X_OK lookup instead.
+ * Why the platform seam over spawning `which`:
+ * - `which` is not POSIX-guaranteed and does not exist on Windows; the seam
+ *   does PATH + X_OK on posix (same semantics) and PATH × PATHEXT on win32,
+ *   with no child process at all.
  */
-import { spawn } from 'child_process';
+import { resolveExecutable } from '../platform/command.js';
 /** Map from AgentKind to its CLI binary name. */
 const BINARY_MAP = {
     claude: 'claude',
@@ -28,24 +25,23 @@ const BINARY_MAP = {
     opencode: 'opencode',
     pi: 'pi',
     kimi: 'kimi',
-    // DSH is an HTTP-only agent (no CLI). `which ''` fails → reported unavailable,
-    // which is semantically correct for the /config availability display.
+    // DSH is an HTTP-only agent (no CLI) → reported unavailable, which is
+    // semantically correct for the /config availability display.
     dsh: '',
 };
 const CACHE_TTL_MS = 5 * 60_000;
 /** Module-level cache, survives across calls within the process. */
 const cache = new Map();
 /**
- * Probe a single agent's availability by spawning `which <binary>`.
- * Exit code 0 = binary on PATH; error / non-zero = unavailable.
+ * Probe a single agent's availability (pure PATH lookup; dsh has no CLI
+ * binary and is always unavailable here).
  */
 async function probeOne(kind) {
     const binary = BINARY_MAP[kind];
-    return new Promise((resolve) => {
-        const proc = spawn('which', [binary], { stdio: 'ignore' });
-        proc.on('error', () => resolve(false));
-        proc.on('exit', (code) => resolve(code === 0));
-    });
+    if (!binary)
+        return false;
+    // resolveExecutable 纯 Node PATH 查找（win32 PATH×PATHEXT），命中即可用。
+    return resolveExecutable(binary) !== null;
 }
 /**
  * Probe a single agent, using cache when fresh.
@@ -62,7 +58,7 @@ export async function probeAgentAvailability(kind) {
 /**
  * Probe all agents concurrently.
  *
- * Concurrent (not sequential) because `which` is a lightweight PATH lookup
+ * Concurrent (not sequential) because the lookup is a lightweight PATH scan
  * (<10 ms) with no CPU contention risk — unlike the previous `--help`
  * probes, which needed to run one-at-a-time to avoid starving each other
  * on resource-constrained devices.

package/dist/runner/registry.d.ts

@@ -4,10 +4,10 @@ import type { AgentKind, Runner } from './index.js';
  * factory per available agent at startup; `Bridge.getRunner(workspace)` looks
  * up `config.defaultAgent` here to pick the concrete runner.
  *
- * Singleton caching: the factory MAY cache its instance internally. Most
- * current spawn-type agents (claude, opencode, pi, kimi) are spawn-per-message; codex is
- * workspace-lifetime (app-server); dsh is HTTP+WS direct connection (no local
- * subprocess). Each factory returns a runner per call.
+ * Singleton caching: the factory MAY cache its instance internally. All
+ * local-subprocess agents (claude/codex/opencode/pi/kimi) are
+ * workspace-lifetime (long-lived session); dsh is HTTP+WS direct connection
+ * (no local subprocess). Each factory returns a runner per call.
  *
  * Dynamic config reload: factory can read latest config via registry's
  * configContainer, enabling runtime config changes to take effect.

package/dist/runner/registry.js

@@ -8,10 +8,10 @@ let globalRegistry;
  * factory per available agent at startup; `Bridge.getRunner(workspace)` looks
  * up `config.defaultAgent` here to pick the concrete runner.
  *
- * Singleton caching: the factory MAY cache its instance internally. Most
- * current spawn-type agents (claude, opencode, pi, kimi) are spawn-per-message; codex is
- * workspace-lifetime (app-server); dsh is HTTP+WS direct connection (no local
- * subprocess). Each factory returns a runner per call.
+ * Singleton caching: the factory MAY cache its instance internally. All
+ * local-subprocess agents (claude/codex/opencode/pi/kimi) are
+ * workspace-lifetime (long-lived session); dsh is HTTP+WS direct connection
+ * (no local subprocess). Each factory returns a runner per call.
  *
  * Dynamic config reload: factory can read latest config via registry's
  * configContainer, enabling runtime config changes to take effect.