@cocorograph/hub-agent 0.8.104 → 0.8.106

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.104",
+  "version": "0.8.106",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの Claude / Codex チャットと tmux セッションを中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/claude-stream-bridge.mjs

@@ -966,8 +966,15 @@ export class ClaudeStreamSession {
     this.logger?.debug({ stream_id: this.stream_id, cwd: this.cwd, stderr: text }, "claude cli stderr")
   }
 
-  /** SDK options.canUseTool: tool 起動を許可するかを browser に問い合わせる。 */
-  async _canUseTool(toolName, input) {
+  /**
+   * SDK options.canUseTool: tool 起動を許可するかを browser に問い合わせる。
+   *
+   * SDK は第 3 引数 options.agentID に「サブエージェント内からの要求ならその ID」を渡す。
+   * これを載せないと、サブエージェントの承認要求がチーフ本人の要求と見分けられない
+   * (承認カードに依頼元が出ない)。チーフ本人の要求では agentID が無いので agent_id は省く。
+   * 「常に許可」の判定は browser 側でツール名単位のまま (依頼元で自動許可を止めない)。
+   */
+  async _canUseTool(toolName, input, options) {
     if (!this.onPermission) return { behavior: "allow", updatedInput: input }
     let displayState
     try {
@@ -976,15 +983,17 @@ export class ClaudeStreamSession {
       return { behavior: "deny", message: PERMISSION_MASK_DENIAL }
     }
     const request_id = randomUUID()
+    const agent_id = typeof options?.agentID === "string" && options.agentID ? options.agentID : undefined
     return await new Promise((resolve) => {
       this._permissionResolvers.set(request_id, {
         resolve,
         tool_name: toolName,
         input,
         displayState,
+        agent_id,
       })
       try {
-        this.onPermission({ tool_name: toolName, input: displayState ? copyPermissionInput(displayState.displayInput) : input, request_id })
+        this.onPermission({ tool_name: toolName, input: displayState ? copyPermissionInput(displayState.displayInput) : input, request_id, ...(agent_id ? { agent_id } : {}) })
       } catch (err) {
         this.logger?.warn(
           { stream_id: this.stream_id },
@@ -1108,7 +1117,7 @@ export class ClaudeStreamSession {
     if (!this.onPermission || this._permissionResolvers.size === 0) return
     for (const [request_id, r] of this._permissionResolvers) {
       try {
-        this.onPermission({ tool_name: r.tool_name, input: r.displayState ? copyPermissionInput(r.displayState.displayInput) : r.input, request_id })
+        this.onPermission({ tool_name: r.tool_name, input: r.displayState ? copyPermissionInput(r.displayState.displayInput) : r.input, request_id, ...(r.agent_id ? { agent_id: r.agent_id } : {}) })
       } catch (err) {
         this.logger?.warn(
           { stream_id: this.stream_id, request_id },
@@ -1504,7 +1513,7 @@ export class ClaudeStreamSession {
     const options = {
       cwd: this.cwd,
       env: siblingEnvFor(this.cwd),
-      canUseTool: (toolName, input) => this._canUseTool(toolName, input),
+      canUseTool: (toolName, input, options) => this._canUseTool(toolName, input, options),
       includePartialMessages: true,
       includeHookEvents: true,
       abortController: this._abortController,
@@ -2236,7 +2245,7 @@ export class ClaudeStreamSession {
     const options = {
       cwd: this.cwd,
       env: siblingEnvFor(this.cwd),
-      canUseTool: (toolName, input) => this._canUseTool(toolName, input),
+      canUseTool: (toolName, input, options) => this._canUseTool(toolName, input, options),
       includePartialMessages: true,
       includeHookEvents: true,
       abortController: this._residentAbortController,
@@ -3043,7 +3052,7 @@ export class ClaudeStreamBridge extends EventEmitter {
           this._liveBySession.set(key, session)
         }
       },
-      onPermission: ({ tool_name, input, request_id }) => {
+      onPermission: ({ tool_name, input, request_id, agent_id }) => {
         // 多端末共有: session_id を載せて backend が session group へ broadcast できるように
         // する (全端末に許可ダイアログを出し、先着 1 件の reply を採用する)。
         this.emit("permission", {
@@ -3053,6 +3062,7 @@ export class ClaudeStreamBridge extends EventEmitter {
           request_id,
           tool_name,
           input,
+          ...(agent_id ? { agent_id } : {}),
         })
       },
       onPermissionCancel: ({ request_ids, reason }) => {

package/src/codex-appserver-client.mjs

@@ -49,7 +49,9 @@ export class CodexAppServerClient extends EventEmitter {
     this._requestTimeoutMs = resolveRequestTimeoutMs(requestTimeoutMs)
     this.child = null
     this.closed = false
-    this._buf = ""
+    // 改行待ちの受信断片 (Buffer のまま保持する)。_onStdout の説明を参照。
+    this._pendingChunks = []
+    this._pendingNeedsRescan = false
     this._reqId = 0
     this._pending = new Map()
   }
@@ -143,24 +145,59 @@ export class CodexAppServerClient extends EventEmitter {
     })
   }
 
+  /**
+   * stdout の受信断片を改行区切りの JSON-RPC 行へ組み立てる。
+   *
+   * - 改行は**新しく届いた断片の中だけ**探す。以前は断片のたびに蓄積済み文字列全体を先頭から
+   *   探し直しており、thread/resume・thread/read の応答 (履歴全体を 1 行で返す。実測で最大 55MB)
+   *   では断片数 × 蓄積量の二乗になって、55MB の応答 1 本に約 4.4 秒かかっていた (まとめて
+   *   解析すれば約 60ms)。
+   * - 文字列化は**行が揃ってから 1 回だけ**行う。断片ごとに `toString("utf8")` すると、断片の
+   *   境目にまたがった日本語等のマルチバイト文字が U+FFFD (「�」) に化ける。改行 (0x0A) は
+   *   UTF-8 のマルチバイト列の途中には現れないので、改行位置で Buffer を切れば文字は割れない。
+   */
   _onStdout(chunk) {
-    this._buf += chunk.toString("utf8")
+    let buf = Buffer.isBuffer(chunk) ? chunk : Buffer.from(String(chunk), "utf8")
+    if (this._pendingNeedsRescan) {
+      // 前回 listener の例外で中断した。持ち越し分に完結した行が残っているので先頭から探し直す。
+      buf = Buffer.concat([...this._pendingChunks, buf])
+      this._pendingChunks = []
+      this._pendingNeedsRescan = false
+    }
+    let start = 0
     let idx
-    while ((idx = this._buf.indexOf("\n")) >= 0) {
-      const line = this._buf.slice(0, idx)
-      this._buf = this._buf.slice(idx + 1)
-      if (!line.trim()) continue
-      let msg
-      try {
-        msg = JSON.parse(line)
-      } catch (err) {
-        this.logger?.warn?.(
-          { err: err.message, line: line.slice(0, 500) },
-          "codex app-server: unparsable line (skipped)",
-        )
-        continue
+    let completed = false
+    // listener が例外を投げても、この断片の未処理分を失わない (旧実装と同じく次回へ持ち越す)。
+    try {
+      while ((idx = buf.indexOf(0x0a, start)) >= 0) {
+        const tail = buf.subarray(start, idx)
+        const line = (
+          this._pendingChunks.length
+            ? Buffer.concat([...this._pendingChunks, tail])
+            : tail
+        ).toString("utf8")
+        this._pendingChunks = []
+        start = idx + 1
+        if (!line.trim()) continue
+        let msg
+        try {
+          msg = JSON.parse(line)
+        } catch (err) {
+          this.logger?.warn?.(
+            { err: err.message, line: line.slice(0, 500) },
+            "codex app-server: unparsable line (skipped)",
+          )
+          continue
+        }
+        this._dispatch(msg)
+      }
+      completed = true
+    } finally {
+      if (start < buf.length) {
+        this._pendingChunks.push(buf.subarray(start))
+        // 正常終了なら残りに改行は無い。例外で抜けたときだけ残りに完結行があり得る。
+        if (!completed) this._pendingNeedsRescan = true
       }
-      this._dispatch(msg)
     }
   }
 

package/src/main.mjs

@@ -822,9 +822,10 @@ export function wireBridgeEvents(client, { claudeBridge, codexBridge }) {
       resolveChatContextPct(session, event)
       sendBridgeMessage(client, { type: "claude.event", stream_id, session_id, event })
     })
-    claudeBridge.on("permission", ({ stream_id, session_id, cwd, request_id, tool_name, input }) => {
+    claudeBridge.on("permission", ({ stream_id, session_id, cwd, request_id, tool_name, input, agent_id }) => {
       // 多端末共有: session_id を載せて backend が session group へ broadcast できるように
       // する (全端末に許可ダイアログ → 先着 1 件採用)。
+      // agent_id はサブエージェントからの要求のときだけ載る (承認カードで依頼元を示す)。
       sendBridgeMessage(client, {
         type: "claude.permission.request",
         stream_id,
@@ -832,6 +833,7 @@ export function wireBridgeEvents(client, { claudeBridge, codexBridge }) {
         request_id,
         tool_name,
         input,
+        ...(agent_id ? { agent_id } : {}),
       })
     })
     claudeBridge.on(