@cocorograph/hub-agent 0.8.109 → 0.8.110

package/package.json

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

package/src/hub-bundle.mjs

@@ -330,22 +330,48 @@ export function hasInheritedOutputChild() {
   return inheritedOutputChildren > 0
 }
 
-function runOnce(argv, { logger } = {}) {
+export function runOnce(argv, { logger, redact = [] } = {}) {
+  const secrets = redact.filter((value) => value.length > 0).sort((a, b) => b.length - a.length)
+  const pattern = secrets.length
+    ? new RegExp(secrets.map(_escapeRegExp).join("|"), "g")
+    : null
+  const sanitize = (value) => pattern ? value.replace(pattern, "***") : value
+  const safeError = (err) => pattern ? new Error(sanitize(err.message)) : err
   return new Promise((resolve, reject) => {
     const [bin, ...args] = argv
-    inheritedOutputChildren++
+    const cmd = sanitize(argv.join(" "))
+    if (!pattern) inheritedOutputChildren++
     let released = false
     const release = () => {
       if (released) return
       released = true
-      inheritedOutputChildren--
+      if (!pattern) inheritedOutputChildren--
     }
     let child
     try {
-      child = spawn(bin, args, { stdio: "inherit" })
+      child = spawn(bin, args, { stdio: pattern ? ["inherit", "pipe", "pipe"] : "inherit" })
     } catch (err) {
       release()
-      throw err
+      throw safeError(err)
+    }
+    // close を待って全バイトを結合し、チャンク境界（UTF-8 の途中を含む）での漏出を防ぐ。
+    // 上限を超えたら収集済みも捨てて読み捨てに切り替える（途中で切ると資格の断片が残りうるため、
+    // 部分表示はしない）。pipe は close まで drain し続け、子を書き込み待ちで止めない。
+    const captured = { stdout: { chunks: [], bytes: 0, overflow: false }, stderr: { chunks: [], bytes: 0, overflow: false } }
+    if (pattern) {
+      for (const name of ["stdout", "stderr"]) {
+        const buf = captured[name]
+        child[name].on("data", (chunk) => {
+          if (buf.overflow) return
+          buf.bytes += chunk.length
+          if (buf.bytes > REDACTED_OUTPUT_MAX_BYTES) {
+            buf.overflow = true
+            buf.chunks = []
+            return
+          }
+          buf.chunks.push(chunk)
+        })
+      }
     }
     // spawn 失敗 (ENOENT 等) は exit が来ないことがある。error でも必ず settle させ、
     // bundle 同期を未完了のまま残さない (resolve/reject は最初の 1 回だけ有効)。
@@ -355,22 +381,54 @@ function runOnce(argv, { logger } = {}) {
     }
     child.once("error", (err) => {
       release()
-      reject(err)
-      safeLog("warn", { cmd: argv.join(" "), error: err.message }, "hub-bundle post_install failed")
+      reject(safeError(err))
+      safeLog("warn", { cmd, error: sanitize(err.message) }, "hub-bundle post_install failed")
     })
-    child.on("exit", (code) => {
+    child.on(pattern ? "close" : "exit", (code) => {
       release()
+      // 先に settle する。転送先の障害（ENOSPC / EPIPE 等）でコマンドの成否を変えない。
       if (code === 0) {
         resolve(code)
-        safeLog("info", { cmd: argv.join(" ") }, "hub-bundle post_install ok")
       } else {
-        reject(new Error(`post_install command failed (exit ${code}): ${argv.join(" ")}`))
-        safeLog("warn", { cmd: argv.join(" "), code }, "hub-bundle post_install failed")
+        reject(new Error(`post_install command failed (exit ${code}): ${cmd}`))
+      }
+      if (pattern) {
+        for (const name of ["stdout", "stderr"]) {
+          const buf = captured[name]
+          const text = buf.overflow
+            ? `[hub-agent] 出力が ${REDACTED_OUTPUT_MAX_BYTES} バイトを超えたため省略しました\n`
+            : sanitize(Buffer.concat(buf.chunks).toString("utf8"))
+          if (text) forwardRedactedOutput(process[name], text)
+        }
+      }
+      if (code === 0) {
+        safeLog("info", { cmd }, "hub-bundle post_install ok")
+      } else {
+        safeLog("warn", { cmd, code }, "hub-bundle post_install failed")
       }
     })
   })
 }
 
+// 伏せ字済み出力の 1 ストリームあたりの収集上限。claude mcp add の通常出力は数百バイト。
+const REDACTED_OUTPUT_MAX_BYTES = 1024 * 1024
+
+// 伏せ字済み出力を親の stdout / stderr へ書く。書き込み失敗は診断出力の欠落として扱い、
+// 同期例外も非同期の error も外へ出さない。error listener はこの書き込みの間だけ付け、
+// 共有ストリームの他のエラーを恒久的に握り潰さない。
+function forwardRedactedOutput(stream, text) {
+  const onError = () => {}
+  stream.on("error", onError)
+  try {
+    stream.write(text, () => {
+      // callback 後に error が emit される実装があるため、1 tick 置いてから外す。
+      setImmediate(() => stream.removeListener("error", onError))
+    })
+  } catch {
+    stream.removeListener("error", onError)
+  }
+}
+
 /**
  * Bundle response の `mcp_servers` を Claude Code CLI に登録する。
  *
@@ -474,7 +532,7 @@ export async function applyMcpServers(
           // 未登録 → remove は失敗するが正常系。
         }
       }
-      await run(argv, { logger })
+      await run(argv, wantHeader ? { logger, redact: [agentToken] } : { logger })
       added.push(s.name)
     } catch (err) {
       // ヘッダ不要の外部 MCP は既存なら "already exists" で失敗する。これは「既に正しく