@cocorograph/hub-agent 0.8.62 → 0.8.63

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.8.62",
+  "version": "0.8.63",
   "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 チェーンは維持したまま、現ターン
@@ -849,6 +858,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
@@ -1421,6 +1431,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 +1623,7 @@ export class ClaudeStreamSession {
       // jsonl を掴んでいない) のこの地点で、closed 済みならオンディスク jsonl をマスクする。
       // reap 経路は上の close() 内で既にマスク済み (冪等ガードで二重実行されない)。
       if (this._closed) this._maskJsonlAtRest()
+      finishRunLoop()
     }
   }
 
@@ -2083,6 +2095,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 +2311,7 @@ export class ClaudeStreamSession {
       // マスク (2026-06-23): 常駐 query 終了後 (= CLI が jsonl を掴んでいない) のこの地点で、
       // closed 済みならオンディスク jsonl をマスクする。reap 経路は close() 内で実行済み。
       if (this._closed) this._maskJsonlAtRest()
+      finishRunLoop()
     }
   }
 
@@ -2473,6 +2487,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 +2596,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 +2708,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/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 の