@cocorograph/hub-agent 0.8.27 → 0.8.29

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -3000,6 +3000,29 @@ export class ClaudeStreamBridge extends EventEmitter {
     )
   }
 
+  /**
+   * cwd と session_id で一意に確定したライブセッションの claude.event 経路へ内部イベントを流す。
+   * sibling-inject の delegate-event 専用で、本文 FIFO や sendMessage は経由しない。
+   */
+  emitToStream(cwd, sessionId, event) {
+    if (!cwd || !sessionId) return { status: "not_found" }
+    const matches = this._liveCandidatesForCwd(cwd, true).filter(
+      ({ session }) => session.sessionId === sessionId,
+    )
+    if (matches.length === 0) return { status: "not_found" }
+    if (matches.length > 1) {
+      return {
+        status: "ambiguous",
+        candidates: matches.map(({ session, attached }) =>
+          this._describeCandidate(session, attached),
+        ),
+      }
+    }
+    const { session, attached } = matches[0]
+    session._emit(event)
+    return { status: "delivered", session: this._describeCandidate(session, attached) }
+  }
+
   /**
    * cwd 内の live セッション候補を `{session, attached}` の配列で返す共通スキャン。
    * this.sessions (attached=browser 接続中) と _liveBySession (attached=false=soft-detached の warm)

package/src/sibling-inject.mjs

@@ -59,7 +59,7 @@
  */
 
 import { watch } from "node:fs"
-import { mkdir, readFile, writeFile, rename, unlink, readdir } from "node:fs/promises"
+import { chmod, lstat, mkdir, readFile, writeFile, rename, unlink, readdir } from "node:fs/promises"
 import path from "node:path"
 
 import { verifySiblingTokenAny, siblingTokenCwd } from "./sibling-token.mjs"
@@ -268,9 +268,46 @@ export class SiblingInjectBridge {
     return { claudeBridge: this.claudeBridge, codexBridge: this.codexBridge }
   }
 
+  async _lstatDir() {
+    return lstat(this.dir)
+  }
+
+  /** IPC ディレクトリを所有者限定にし、安全性を確認できなければ例外で閉じる。 */
+  async _prepareDir() {
+    try {
+      await this._lstatDir()
+    } catch (err) {
+      if (err?.code !== "ENOENT") throw err
+      await mkdir(this.dir, { recursive: true, mode: 0o700 })
+    }
+
+    const info = await this._lstatDir()
+    if (info.isSymbolicLink()) {
+      throw new Error("IPC directory must not be a symbolic link")
+    }
+    if (!info.isDirectory()) {
+      throw new Error("IPC path is not a directory")
+    }
+    const uid = process.getuid?.()
+    if (uid == null || info.uid !== uid) {
+      throw new Error("IPC directory is not owned by the current user")
+    }
+    if ((info.mode & 0o777) !== 0o700) {
+      await chmod(this.dir, 0o700)
+    }
+  }
+
   /** 監視ディレクトリを作成し、既存要求を sweep してから fs.watch を張る。 */
   async start() {
-    await mkdir(this.dir, { recursive: true })
+    try {
+      await this._prepareDir()
+    } catch (err) {
+      this.logger?.warn(
+        { err: err?.message, dir: this.dir },
+        "sibling-inject IPC directory is unsafe, IPC disabled",
+      )
+      return
+    }
     await this._sweep()
     try {
       this._watcher = watch(this.dir, (_event, filename) => {
@@ -332,7 +369,21 @@ export class SiblingInjectBridge {
     //    (T06271 レビュー major①): 旧 hub-agent は `!text` で invalid_request を返すため、
     //    未知フィールドを無視して sendToCwd 推測注入する事故が **単一リクエストで原理的に
     //    起きない** (probe→注入の TOCTOU を排除)。
-    const { op, cwd, target_cli_kind, text, message, token, target_session_id, source } = body || {}
+    const {
+      op,
+      cwd,
+      target_cli_kind,
+      text,
+      message,
+      token,
+      target_session_id,
+      source,
+      delegation_id,
+      phase,
+      agent,
+      description,
+      timestamp,
+    } = body || {}
     // 連携カード種別 (Phase 2, 2026-07-23): 注入を受け手画面で専用カード表示させるためのタグ。
     // 既知値 ('peer'|'orchestrator') のみ通し、未知値/未指定は undefined に落とす (未知の source を
     // frontend へ流して分岐を汚さない fail-closed)。bridge が user echo にこれを付ける。
@@ -340,9 +391,22 @@ export class SiblingInjectBridge {
     const isListAll = op === "list-all"
     const isList = op === "list"
     const isExplicit = op === "inject"
+    const isDelegateEvent = op === "delegate-event"
     let result
     const badKind = target_cli_kind !== "claude" && target_cli_kind !== "codex"
-    const badBody = isList
+    const delegatePhases = new Set(["started", "completed", "failed", "stopped"])
+    const badDelegateBody =
+      typeof delegation_id !== "string" ||
+      !delegation_id ||
+      !delegatePhases.has(phase) ||
+      (phase === "started" && (typeof agent !== "string" || !agent)) ||
+      (agent !== undefined && typeof agent !== "string") ||
+      (description !== undefined && typeof description !== "string") ||
+      (timestamp !== undefined &&
+        (typeof timestamp !== "string" || Number.isNaN(Date.parse(timestamp))))
+    const badBody = isDelegateEvent
+      ? badDelegateBody
+      : isList
       ? false
       : isExplicit
         ? !target_session_id || !message // 明示は id と message 必須 (空指定 fail-open を防ぐ)
@@ -361,7 +425,7 @@ export class SiblingInjectBridge {
         // 判定は live-sessions.mjs に一本化する (tmux.list_sessions の warm と同じ関数)。
         result = { ok: true, live_sessions: describeLiveSessionsUnion(this._bridges()) }
       }
-    } else if (!cwd || badKind || badBody) {
+    } else if (!cwd || (!isDelegateEvent && badKind) || badBody) {
       result = { ok: false, error: "invalid_request" }
     } else if (!verifySiblingTokenAny(token)) {
       // 2026-07-23 統一再設計 1-2: 認可を「ターゲット cwd 所有」→「いずれかの cwd の有効トークン
@@ -384,7 +448,34 @@ export class SiblingInjectBridge {
         }
       }
       const bridge = target_cli_kind === "codex" ? this.codexBridge : this.claudeBridge
-      if (isList) {
+      if (isDelegateEvent) {
+        // 委譲カードは別 Claude 会話へ誤配送すると誤解を生むため、cwd 内の 1 本へ推測配送しない。
+        // target_session_id が無い場合も、指定先が一意に見つからない場合も同じ fail-closed 契約にする。
+        if (!target_session_id) {
+          result = { ok: false, error: AMBIGUOUS_TARGET_ERROR }
+        } else {
+          const event = {
+            type: "system",
+            subtype: "codex_delegate",
+            delegation_id,
+            phase,
+            ...(agent !== undefined ? { agent } : {}),
+            ...(description !== undefined ? { description } : {}),
+            timestamp: timestamp || new Date().toISOString(),
+          }
+          const r = this.claudeBridge?.emitToStream
+            ? this.claudeBridge.emitToStream(cwd, target_session_id, event)
+            : { status: "not_found" }
+          result =
+            r.status === "delivered"
+              ? { ok: true, delivered_to: r.session }
+              : {
+                  ok: false,
+                  error: AMBIGUOUS_TARGET_ERROR,
+                  ...(r.candidates ? { candidates: r.candidates } : {}),
+                }
+        }
+      } else if (isList) {
         // 宛先候補の列挙 (cockpit_orch.py --list 用)。この cwd の同種セッションを返す。
         // ⚠️ tmux.list_sessions の `warm` と同じ関数を通す (二重定義にすると「グレー表示
         //    なのに宛先一覧に出ない」食い違いが再発する)。
@@ -586,7 +677,7 @@ export class SiblingInjectBridge {
     const fp = path.join(this.dir, `${request_id}.result.json`)
     const tmp = `${fp}.tmp`
     try {
-      await writeFile(tmp, JSON.stringify(result))
+      await writeFile(tmp, JSON.stringify(result), { mode: 0o600 })
       await rename(tmp, fp)
     } catch (err) {
       this.logger?.warn({ err: err?.message, request_id }, "sibling-inject result write failed")