@cocorograph/hub-agent 0.8.62 → 0.8.64

package/package.json

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

package/src/claude-history.mjs

@@ -17,7 +17,7 @@
  */
 import { randomUUID } from "node:crypto"
 import { constants as fsConstants } from "node:fs"
-import { open, readFile, readdir, stat } from "node:fs/promises"
+import { open, readFile, readdir, rm, stat, unlink } from "node:fs/promises"
 import os from "node:os"
 import path from "node:path"
 
@@ -646,6 +646,48 @@ export async function renameSession({ cwd, session_id, title, projectsRoot, logg
   }
 }
 
+/** Claude セッションの JSONL と同名の補助ディレクトリを削除する。 */
+export async function deleteSession({
+  cwd,
+  session_id,
+  projectsRoot,
+  logger,
+  rmImpl = rm,
+  unlinkImpl = unlink,
+  statImpl = stat,
+}) {
+  if (typeof cwd !== "string" || cwd.length === 0) return { ok: false, error: "cwd required" }
+  if (!/^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$/.test(session_id)) {
+    return { ok: false, error: "invalid session_id" }
+  }
+
+  const filePath = jsonlPath({ cwd, session_id, projectsRoot })
+  const dirPath = filePath.slice(0, -".jsonl".length)
+  try {
+    const fileStat = await statImpl(filePath)
+    if (!fileStat.isFile()) return { ok: false, error: "session not found" }
+  } catch (err) {
+    const notFound = err?.code === "ENOENT" || err?.code === "EISDIR"
+    if (notFound) return { ok: false, error: "session not found" }
+    logger?.warn({ err: err?.message || String(err), code: err?.code, filePath }, "claude session delete stat failed")
+    return { ok: false, error: "delete failed" }
+  }
+
+  try {
+    await rmImpl(dirPath, { recursive: true, force: true })
+    await unlinkImpl(filePath)
+  } catch (err) {
+    logger?.warn({ err: err?.message || String(err), code: err?.code, filePath }, "claude session delete failed")
+    return { ok: false, error: "delete failed" }
+  }
+
+  hydrateCache.delete(filePath)
+  const parentDir = path.dirname(filePath)
+  const fileMap = lastTurnDirCache.get(parentDir)
+  fileMap?.delete(filePath)
+  return { ok: true }
+}
+
 /**
  * jsonl 1 ファイルを読んで UI 表示用イベント配列を返す。
  *

package/src/claude-stream-bridge.mjs

@@ -538,6 +538,15 @@ export class ClaudeStreamSession {
      *  InputQueue にメッセージが消えることがある。このフラグで「push 直前に本当にまだ
      *  生きているか」を再確認する。新しい query を (再)起動するたびに false へリセットする。 */
     this._residentQueryReturned = false
+    /** 削除時に CLI 終了を待つための、活動中 run-loop の完了 Promise 集合。
+     *  ⚠️ 単一の「現在の Promise」にしないこと (cross-review blocker, 2026-09-05): per-message の
+     *  finally は `_drainPending()` で次ターンを**同期的に**起動してから自分の完了処理へ進むため、
+     *  単一スロットだと旧ターンの完了が新ターンの Promise を誤って resolve し、CLI が走行中でも
+     *  waitForRunLoopExit() が即 resolve して jsonl を unlink できてしまう (resident の異常再起動も同型)。
+     *  run-loop ごとに自分の Promise を持ち、自分のものだけを完了させる。 */
+    this._activeRunLoops = new Set()
+    /** 削除対象では close 後の tmp→rename が JSONL を復活させないよう at-rest mask を止める。 */
+    this._suppressAtRestMask = false
 
     /** 改修3: per-message セッションで busy 中に届いた送信を退避する pending キュー。
      *  常駐 query 化 (改修2) とは別レイヤー。resume チェーンは維持したまま、現ターン
@@ -554,6 +563,9 @@ export class ClaudeStreamSession {
      *  メッセージが後着して実行される「ゴースト実行」を防ぐ)。到着で消費 (delete)。
      *  無限増加を避けるため上限で全クリアする (キャンセルは低頻度)。 */
     this._canceledInputIds = new Set()
+    /** 取消成功した input_id の直近リング。accepted は配達確認と二重実行防止の根拠なので外さず、
+     *  browser が「accepted ＋ キュー不在」のバブル昇格から取消済み id を除外する別軸にする。 */
+    this._cancelledInputIds = []
     /** 改修3: 直近 browser へ通知した queue 署名 (件数 + id 列)。変化時のみ emit する。
      *  空キューの署名 ("0:") で初期化し、空→空の冗長 emit を抑止する。 */
     this._lastEmittedQueueSig = "0:"
@@ -849,6 +861,7 @@ export class ClaudeStreamSession {
    *  ファイルを掴んでいない「終了後」にのみ呼ぶこと (ライブ追記中の書き換えは会話本体を壊す)。
    *  display は _emit で常時マスク済みのため、これは at-rest ファイルの掃除専用。 */
   _maskJsonlAtRest() {
+    if (this._suppressAtRestMask) return
     if (this._jsonlMasked || !MASK_ENABLED) return
     if (!this.sessionId || !this.cwd) return
     this._jsonlMasked = true
@@ -1303,7 +1316,10 @@ export class ClaudeStreamSession {
     // チップを固着させる。ack を返す条件 (accepted) と同じ場所で 1 回だけ記録する。
     const inputId =
       typeof opts?.input_id === "string" && opts.input_id ? opts.input_id : null
-    if (inputId && result?.accepted) this._recordAcceptedInput(inputId)
+    if (inputId && result?.accepted) {
+      this._recordAcceptedInput(inputId)
+      if (result.preCancelled) this._recordCancelledInput(inputId)
+    }
     return result
   }
 
@@ -1326,7 +1342,7 @@ export class ClaudeStreamSession {
         { stream_id: this.stream_id, input_id: inputId },
         "dropped pre-canceled input (cancel arrived before input)",
       )
-      return { accepted: true }
+      return { accepted: true, preCancelled: true }
     }
 
     // ultracode ワンショット (0.6.28): このターンのみ xhigh effort + 常時 dynamic-workflow
@@ -1421,6 +1437,7 @@ export class ClaudeStreamSession {
   /** per-message 1 ターンを実行する (resume チェーン)。busy 中に届いた送信は sendMessage
    *  が pending キューへ退避し、本メソッドの finally で drain する。 */
   async _runPerMessage(prompt, opts = {}) {
+    const finishRunLoop = this._markRunLoopStarted()
     this._busy = true
     this._abortController = new AbortController()
     let aborted = false
@@ -1612,6 +1629,7 @@ export class ClaudeStreamSession {
       // jsonl を掴んでいない) のこの地点で、closed 済みならオンディスク jsonl をマスクする。
       // reap 経路は上の close() 内で既にマスク済み (冪等ガードで二重実行されない)。
       if (this._closed) this._maskJsonlAtRest()
+      finishRunLoop()
     }
   }
 
@@ -1701,6 +1719,8 @@ export class ClaudeStreamSession {
    *  snapshot の `accepted` として browser へ載せ、ack 喪失時の自己回復に使う。 */
   _recordAcceptedInput(inputId) {
     if (typeof inputId !== "string" || !inputId) return
+    const cancelledAt = this._cancelledInputIds.indexOf(inputId)
+    if (cancelledAt !== -1) this._cancelledInputIds.splice(cancelledAt, 1)
     const at = this._acceptedInputIds.indexOf(inputId)
     if (at !== -1) this._acceptedInputIds.splice(at, 1)
     this._acceptedInputIds.push(inputId)
@@ -1712,6 +1732,19 @@ export class ClaudeStreamSession {
     }
   }
 
+  _recordCancelledInput(inputId) {
+    if (typeof inputId !== "string" || !inputId) return
+    const at = this._cancelledInputIds.indexOf(inputId)
+    if (at !== -1) this._cancelledInputIds.splice(at, 1)
+    this._cancelledInputIds.push(inputId)
+    if (this._cancelledInputIds.length > ACCEPTED_INPUT_ID_RING_MAX) {
+      this._cancelledInputIds.splice(
+        0,
+        this._cancelledInputIds.length - ACCEPTED_INPUT_ID_RING_MAX,
+      )
+    }
+  }
+
   /** キャンセル機能 (0.6.26): browser から id 指定で送信待ちメッセージを 1 件取り消す。
    *  実行開始済み (既に drain された) メッセージは pending に残っていないので no-op になる。
    *  削除に成功したら queue_state を再 emit して送信待ちチップを更新する。 */
@@ -1728,6 +1761,8 @@ export class ClaudeStreamSession {
     this._pendingMessages = this._pendingMessages.filter((m) => m.id !== id)
     const removed = before !== this._pendingMessages.length
     if (removed) {
+      this._canceledInputIds.delete(id)
+      this._recordCancelledInput(id)
       this.logger?.info(
         { stream_id: this.stream_id, id, queued: this._pendingMessages.length },
         "queued message canceled",
@@ -1843,6 +1878,7 @@ export class ClaudeStreamSession {
       // と判定して楽観チップを消し、「キューに無い ＋ ここにも無い ＝ 未配達」なら
       // 消さずに再送導線を出す (無言削除の回帰を作らないための 2 軸判定)。
       accepted: [...this._acceptedInputIds],
+      cancelled: [...this._cancelledInputIds],
     }
   }
 
@@ -2083,6 +2119,7 @@ export class ClaudeStreamSession {
    *  常駐では query stream が唯一のイベント源のため watcher は張らない (過去確定分は browser が
    *  history.request で hydrate 済み。resume で query stream が過去行を replay しない前提)。 */
   async _runResidentQuery() {
+    const finishRunLoop = this._markRunLoopStarted()
     // SDK-P0-4 (2026-07-13): 常駐にも AbortController を渡す (per-message には元からある)。
     // 正規の中断は interrupt() だが、control request が失敗/無応答のときに generator を畳む
     // 最終手段が無いと、close()/shutdown() で CLI 子プロセスが残留する。
@@ -2298,6 +2335,7 @@ export class ClaudeStreamSession {
       // マスク (2026-06-23): 常駐 query 終了後 (= CLI が jsonl を掴んでいない) のこの地点で、
       // closed 済みならオンディスク jsonl をマスクする。reap 経路は close() 内で実行済み。
       if (this._closed) this._maskJsonlAtRest()
+      finishRunLoop()
     }
   }
 
@@ -2473,6 +2511,34 @@ export class ClaudeStreamSession {
     return true
   }
 
+  /** run-loop 開始を登録し、**その run-loop 自身だけ**を完了させる関数を返す (冪等)。 */
+  _markRunLoopStarted() {
+    let resolve
+    const promise = new Promise((r) => {
+      resolve = r
+    })
+    this._activeRunLoops.add(promise)
+    return () => {
+      if (!this._activeRunLoops.delete(promise)) return
+      resolve()
+    }
+  }
+
+  /** 削除専用 close。通常 close の at-rest mask だけを抑止する。 */
+  closeForDeletion() {
+    this._suppressAtRestMask = true
+    this.close()
+  }
+
+  /** 呼び出し時点で活動中の run-loop が**すべて**終わるまで待つ。待っている間に新しい run-loop が
+   *  同期起動された (pending drain / resident 再起動) 場合はそれも待つ。close 済みなら
+   *  `_drainPending()` / 再起動は `_closed` ガードで始まらないため、連鎖は有限で終わる。 */
+  async waitForRunLoopExit() {
+    while (this._activeRunLoops.size > 0) {
+      await Promise.all([...this._activeRunLoops])
+    }
+  }
+
   /** セッション終了。新規ターンを止め、実行中なら中断し、watcher も停止する。 */
   close() {
     this._closed = true
@@ -2554,6 +2620,12 @@ export class ClaudeStreamBridge extends EventEmitter {
     /** session_id → 生存セッション。再接続 (再アタッチ) で同一セッションを引く索引。
      *  browser が切れてもここに残し、走行中ターンの継続 + 再接続でのライブ追従を可能にする。 */
     this._liveBySession = new Map()
+    /** key(cwd+session_id) → 削除停止を開始済みで run-loop 終了待ちのセッション集合。
+     *  索引から外した後も再試行が同じ CLI 群を待てるように保持し、各 CLI の終了時に外す。
+     *  同じ key で新しい CLI が起動→再削除→timeout した場合も全件残す (1 件だけだと 2 件目が追跡漏れ)。 */
+    this._deletionTombstones = new Map()
+    /** テストでは短縮可能。実運用では CLI reap を最大 5 秒待つ。 */
+    this._sessionDeletionWaitMs = 5_000
 
     /** MCP control 専用 query (agent-global)。遅延起動・アイドル破棄。session 非依存。 */
     this._mcpControlQuery = null
@@ -2660,6 +2732,68 @@ export class ClaudeStreamBridge extends EventEmitter {
     return stopped
   }
 
+  /** cwd と session_id が完全一致する live セッションだけを停止し、CLI 終了まで待つ。
+   *  戻り値は今回新たに停止した数 (tombstone の待ち直しは数えない)。timeout は throw。 */
+  async stopSessionForDeletion({ cwd, session_id }) {
+    const key = this._liveKeyFor(cwd, session_id)
+    const newVictims = new Set()
+    for (const s of [...this._liveBySession.values(), ...this.sessions.values()]) {
+      if (s.cwd === cwd && s.sessionId === session_id && !s._closed) newVictims.add(s)
+    }
+    const waitTargets = new Set(newVictims)
+    for (const s of this._deletionTombstones.get(key) ?? []) waitTargets.add(s)
+
+    for (const s of newVictims) {
+      this._dropSessionMappings(s)
+      s.closeForDeletion()
+      this._rememberDeletionTombstone(key, s)
+    }
+
+    let timer
+    try {
+      // 全対象を 1 本の Promise にまとめて待つ (逐次だと先頭の timeout で後続を確認しないまま抜ける)。
+      await Promise.race([
+        Promise.all([...waitTargets].map((s) => s.waitForRunLoopExit())),
+        new Promise((_, reject) => {
+          timer = setTimeout(() => reject(new Error("session deletion stop timeout")), this._sessionDeletionWaitMs)
+          timer.unref?.()
+        }),
+      ])
+    } finally {
+      if (timer) clearTimeout(timer)
+      // timeout で throw する経路でも exit は必ず出す: session は既に close 済みで索引からも
+      // 外れているため、ここで送らないと browser は「生きている stream」を抱えたままになる
+      // (削除自体は main が `delete failed` で止める。exit と失敗応答は独立)。
+      // tombstone からの待ち直しは初回に exit 済みなので出さない (1 セッション 1 回)。
+      for (const s of newVictims) {
+        this.emit("exit", {
+          stream_id: s.stream_id,
+          code: 0,
+          reason: "session-deleted",
+          session_id: s.sessionId,
+        })
+      }
+    }
+    return newVictims.size
+  }
+
+  /** 削除停止したセッションを key ごとの集合に残し、その run-loop が終わったときだけ外す。 */
+  _rememberDeletionTombstone(key, session) {
+    if (!key) return
+    let set = this._deletionTombstones.get(key)
+    if (!set) {
+      set = new Set()
+      this._deletionTombstones.set(key, set)
+    }
+    set.add(session)
+    session.waitForRunLoopExit().then(() => {
+      const cur = this._deletionTombstones.get(key)
+      if (!cur) return
+      cur.delete(session)
+      if (cur.size === 0) this._deletionTombstones.delete(key)
+    })
+  }
+
   /**
    * 新しい Claude セッションを開始する。
    *

package/src/codex-stream-bridge.mjs

@@ -973,6 +973,9 @@ export class CodexStreamBridge extends EventEmitter {
       queueStateSeq: 0,
       /** ack 喪失からの自己回復用、受理済み browser input_id の bounded ring。 */
       acceptedInputIds: [],
+      /** 取消成功 id は accepted（二重実行ガード）から外さず、browser が「accepted ＋キュー不在」
+       *  のバブル昇格から除外するための別軸として保持する。 */
+      cancelledInputIds: [],
       /** @type {{resolve: Function, timer: any} | null} round3 major#2: turn/start 後に turn/started
        *  (activeTurnId 確定) を待つ barrier の待機者。timeout 付きでハングを防ぐ。 */
       _turnActiveWaiter: null,
@@ -2062,6 +2065,9 @@ export class CodexStreamBridge extends EventEmitter {
 
   _recordAcceptedInput(session, inputId) {
     if (!session || typeof inputId !== "string" || !inputId) return
+    const cancelledIds = session.cancelledInputIds || (session.cancelledInputIds = [])
+    const cancelledAt = cancelledIds.indexOf(inputId)
+    if (cancelledAt !== -1) cancelledIds.splice(cancelledAt, 1)
     const ids = session.acceptedInputIds || (session.acceptedInputIds = [])
     const at = ids.indexOf(inputId)
     if (at !== -1) ids.splice(at, 1)
@@ -2071,6 +2077,17 @@ export class CodexStreamBridge extends EventEmitter {
     }
   }
 
+  _recordCancelledInput(session, inputId) {
+    if (!session || typeof inputId !== "string" || !inputId) return
+    const ids = session.cancelledInputIds || (session.cancelledInputIds = [])
+    const at = ids.indexOf(inputId)
+    if (at !== -1) ids.splice(at, 1)
+    ids.push(inputId)
+    if (ids.length > ACCEPTED_INPUT_ID_RING_MAX) {
+      ids.splice(0, ids.length - ACCEPTED_INPUT_ID_RING_MAX)
+    }
+  }
+
   /**
    * 重複ガードの台帳から id を取り消す (2026-08-05)。
    *
@@ -2103,6 +2120,7 @@ export class CodexStreamBridge extends EventEmitter {
       seq: session.queueStateSeq || 0,
       generation: session.seq,
       accepted: [...(session.acceptedInputIds || [])],
+      cancelled: [...(session.cancelledInputIds || [])],
     }
   }
 
@@ -2121,6 +2139,7 @@ export class CodexStreamBridge extends EventEmitter {
       return false
     }
     const [item] = session.inputQueue.splice(index, 1)
+    this._recordCancelledInput(session, item.inputId || item.deliveryId)
     if (!item.acceptedAsQueued) {
       const err = Object.assign(new Error("codex queued input cancelled"), {
         reason: "cancelled",

package/src/main.mjs

@@ -59,6 +59,7 @@ import { UploadManager } from "./claude-upload.mjs"
 import { requestSelfUninstall } from "./service-install.mjs"
 import {
   drainTaskNotificationEnvelopeDrops,
+  deleteSession,
   fetchSessionHistory,
   listSessions,
   renameSession,
@@ -5032,6 +5033,42 @@ export async function dispatch(msg, ctx) {
         })
         return
       }
+      case "claude.session.delete": {
+        const stream_id = msg.stream_id
+        const cwd = typeof msg.cwd === "string" ? msg.cwd : ""
+        const session_id = typeof msg.session_id === "string" ? msg.session_id : ""
+        const request_id = typeof msg.request_id === "string" ? msg.request_id : undefined
+        let profileId
+        let result
+        let stopped = 0
+        if (!cwd) {
+          result = { ok: false, error: "cwd required" }
+        } else if (!/^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$/.test(session_id)) {
+          result = { ok: false, error: "invalid session_id" }
+        } else {
+          const resolved = await resolveProjectsRoot(msg.profile_id)
+          profileId = resolved.profileId
+          try {
+            const active = await getActiveProfile()
+            stopped = ctx.claudeBridge && resolved.profileId === active.id
+              ? await ctx.claudeBridge.stopSessionForDeletion({ cwd, session_id })
+              : 0
+            result = await deleteSession({ cwd, session_id, projectsRoot: resolved.projectsRoot, logger: ctx.logger })
+          } catch (err) {
+            ctx.logger?.warn?.({ err: err?.message || String(err) }, "claude session delete stop failed")
+            result = { ok: false, error: "delete failed" }
+          }
+        }
+        ctx.client.send({
+          type: "claude.session.delete.response",
+          request_id, stream_id, cwd, session_id,
+          profile_id: profileId,
+          ok: result.ok === true,
+          stopped: result.ok ? stopped : undefined,
+          error: result.ok ? undefined : result.error,
+        })
+        return
+      }
       case "profile.list": {
         // Claude アカウント・プロファイル一覧と active を返す (Cockpit のアカウント切替UI用)。
         // 各プロファイルの「最後にログインしていたアカウント」(<configDir>/.claude.json の