@cocorograph/hub-agent 0.8.25 → 0.8.26

package/package.json

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

package/src/claude-history-watch.mjs

@@ -8,15 +8,18 @@
  * 設計:
  * - 対象ファイルのバイトオフセットを記録し、増分だけ読んで行単位でパース
  * - fs.watch(file) の change イベントで増分読み取り (取りこぼし対策に軽いポーリング併用)
- * - DISPLAY_TYPES (user/assistant/system/result) の行だけ onEvent に渡す
+ * - historyEventFromLine が表示対象と判定した行だけ onEvent に渡す
+ *   (DISPLAY_TYPES の 4 種 + queue-operation から復元したタスク終端イベント)
  * - 各行の uuid を含めて渡す (frontend 側で重複排除に使う)
  * - ファイル不在時は出現を待つ (ディレクトリ監視はせず、ポーリングで存在チェック)
  */
 import { watch as fsWatch } from "node:fs"
 import { open, realpath, stat } from "node:fs/promises"
 
-// 履歴 hydrate と live watch で jsonl→SDK message 整形を共通化 (フィールド取りこぼし防止)。
-import { DISPLAY_TYPES, normalizeHistoryEvent } from "./claude-history.mjs"
+// 履歴 hydrate と live watch で「どの行を拾うか + jsonl→SDK message 整形」を共通化
+// (フィールド取りこぼし・片方だけ直すドリフトの防止)。queue-operation からの終端イベント
+// 合成もこの関数の中に閉じているので、live watch も自動的に同じ挙動になる。
+import { historyEventFromLine } from "./claude-history.mjs"
 
 // アダプティブ・ポーリング: 最後の追記から FAST_WINDOW_MS 以内は fast、以降は idle 周期。
 // ~/.claude が iCloud Drive への symlink のため fs.watch (FSEvents) が遅延/不発しやすく、
@@ -98,8 +101,8 @@ export function watchSessionFile({ filePath, onEvent, fromEnd = true, logger })
           } catch {
             continue
           }
-          if (!obj || !DISPLAY_TYPES.has(obj.type)) continue
-          const event = normalizeHistoryEvent(obj)
+          const event = historyEventFromLine(obj)
+          if (!event) continue
           try {
             onEvent(event)
           } catch (err) {

package/src/claude-history.mjs

@@ -10,6 +10,8 @@
  * - cwd エンコード規則: `/`, `.`, `_` などの非英数字を `-` 置換 (Claude CLI と同じ)
  * - 1 行 = 1 JSON。`type` が 'user' / 'assistant' / 'system' / 'result' のものだけ抽出
  *   (内部 type 'attachment' / 'last-prompt' / 'permission-mode' 等は UI 表示対象外)
+ *   例外として `queue-operation` の未配送タスク通知だけは終端イベントへ合成する
+ *   (synthesizeTaskNotificationEvent 参照。行そのものを表示するわけではない)
  * - 上限 MAX_HISTORY_LINES (デフォルト 500) で末尾から切る
  * - ファイル不在 (新規セッション) なら空配列を返す (エラーにしない)
  */
@@ -210,6 +212,149 @@ export function normalizeHistoryEvent(obj) {
   return event
 }
 
+const TASK_NOTIFICATION_MARKER = "<task-notification>"
+const TASK_NOTIFICATION_CLOSE = "</task-notification>"
+// ヘッダ領域の終わり。ここから先は自由文 (報告全文) なので、鍵と状態の抽出対象にしない。
+const TASK_NOTIFICATION_BODY_TAGS = ["<result>", "<summary>", "<note>"]
+// マスクを素通りする経路なので、長さでも歯止めを掛ける (実データの tool_use_id は 30〜40 字)。
+const MAX_TOOL_USE_ID_LEN = 200
+const MAX_STATUS_LEN = 64
+
+/**
+ * 「通知の体裁を持つのに合成しなかった」件数。**内容は一切持たない**。
+ *
+ * ⚠️ これは観測窓であって判定ではない。封筒の検査を厳しくすると、Claude CLI が将来ラッパー・
+ * 空白・タグ順を変えたときに**正当な通知まで落ちて幽霊カードが静かに戻る**。その「静かな戻り」に
+ * 気づけるよう、件数だけを数える (内容をログへ出すと、まさに伏せ字を回避したい本文が漏れる)。
+ */
+let taskNotificationEnvelopeDrops = 0
+
+/** 封筒の検査で合成しなかった**累計**件数を返す (テスト用)。 */
+export function getTaskNotificationEnvelopeDrops() {
+  return taskNotificationEnvelopeDrops
+}
+
+/** 前回 drain 以降に増えた分を返して 0 に戻す (ログへ出す側が使う)。 */
+let taskNotificationEnvelopeDropsReported = 0
+export function drainTaskNotificationEnvelopeDrops() {
+  const delta = taskNotificationEnvelopeDrops - taskNotificationEnvelopeDropsReported
+  taskNotificationEnvelopeDropsReported = taskNotificationEnvelopeDrops
+  return delta
+}
+
+/** `<tag>…</tag>` の **最初の** 中身を返す。見つからなければ ""。 */
+function firstTagValue(text, tag) {
+  const open = `<${tag}>`
+  const start = text.indexOf(open)
+  if (start < 0) return ""
+  const from = start + open.length
+  const end = text.indexOf(`</${tag}>`, from)
+  if (end < 0) return ""
+  return text.slice(from, end).trim()
+}
+
+/**
+ * `queue-operation` 行に埋まっている「タスク終端の宣言」を、画面が扱える最小イベントへ
+ * 合成する (2026-08-05)。
+ *
+ * ## なぜ必要か
+ * サブエージェント / バックグラウンドタスクの完了通知は、CLI がまず **キューへ積む**
+ * (`type:"queue-operation", operation:"enqueue"`) 形で jsonl に書かれ、その後キューから
+ * 取り出された時に初めて `type:"user"` の行として配送される。ところが配送前に
+ * セッションが閉じる・`remove` されるなどで **user 行が書かれないまま終わる**ケースが多く
+ * (2026-08-05 に手元の全 jsonl 2,136 本を走査した実測で、終端宣言 788 件 (tool_use_id
+ * ユニーク) のうち 332 件 = 42% が未配送。内訳 completed 285 / failed 29 / killed 18)、
+ * 画面側の台帳はそのカードを
+ * 「実行中」のまま抱え続ける (= リロードで残る幽霊カード)。
+ *
+ * enqueue 行の `content` には `<status>` と `<tool-use-id>` が入っており、これは
+ * **CLI 自身が書いた終端宣言**なので、時間で推測せずに終端を復元できる。
+ *
+ * ## ⚠️ 載せてよいのは「鍵」と「状態」と「時刻」だけ
+ * `<result>` にはサブエージェントの報告全文が入る (実データで確認済み)。しかしマスク処理
+ * `maskMessageObject` (src/tool-result-mask.mjs:165-168) は `const message = obj.message;
+ * if (!message …) return obj` で **即 return** するため、`message` を持たないこの合成イベントは
+ * **マスクを素通りする**。よって `<result>` / `<summary>` / `<note>` / `<output-file>` の本文は
+ * 絶対に載せない (載せると秘匿情報がマスクされないまま browser へ流れる)。`<task-id>` も
+ * 画面側が使っていないので載せない。
+ *
+ * 画面側 (Hub 本体 frontend/lib/cockpit/agentRequest.ts の applyTaskNotificationEvent) は
+ * `type:"system" / subtype:"task_notification" / tool_use_id / status` を終端として処理し、
+ * `requireRegistered: true` なので未知の tool_use_id で幻のカードが増えることはない。
+ * `timestamp` は猶予表示の起点になるので必ず載せる。
+ *
+ * @param {Record<string, unknown>} obj jsonl 1 行をパースした生 object
+ * @returns {Record<string, unknown>|null} 合成イベント。対象外なら null
+ */
+export function synthesizeTaskNotificationEvent(obj) {
+  if (!obj || obj.type !== "queue-operation" || obj.operation !== "enqueue") return null
+  const content = obj.content
+  if (typeof content !== "string") return null
+  const trimmed = content.trim()
+  // ⚠️ **「通知を含む」ではなく「通知そのものである」ことを要求する** (2026-08-05 のクロス
+  // レビュー minor)。含むだけで拾うと、**利用者が通知の文面を本文へ貼っただけの入力**を終端
+  // 宣言として扱ってしまう。実測でも、このレビューのやり取り自体がキューに積まれ、状態として
+  // 日本語の文章を、鍵として 215 字の文字列を拾っていた (実データ 1,692 行中 2 件)。
+  // 実データでは正規の通知 1,648 件すべてが `<task-notification>` で始まり、閉じタグを持つ。
+  if (!trimmed.startsWith(TASK_NOTIFICATION_MARKER)) {
+    if (trimmed.includes(TASK_NOTIFICATION_MARKER)) taskNotificationEnvelopeDrops += 1
+    return null
+  }
+  // ⚠️ 閉じタグは **最後の出現**を採る。最初の出現にすると、`<result>` の本文が
+  // `</task-notification>` を引用しているだけで封筒が途中で切れ、正当な通知を落とす。
+  const closeAt = trimmed.lastIndexOf(TASK_NOTIFICATION_CLOSE)
+  if (closeAt < TASK_NOTIFICATION_MARKER.length) {
+    taskNotificationEnvelopeDrops += 1
+    return null
+  }
+  // 閉じタグの後ろに本文が続くものは通知そのものではない (貼り付けの typical な形)。
+  if (trimmed.slice(closeAt + TASK_NOTIFICATION_CLOSE.length).trim()) {
+    taskNotificationEnvelopeDrops += 1
+    return null
+  }
+  const block = trimmed.slice(TASK_NOTIFICATION_MARKER.length, closeAt)
+  // 鍵と状態は **ヘッダ領域** (自由文の本文タグより前) からだけ採る。本文に紛れたタグや、
+  // 正規の `<status>` が欠けたときに本文側のタグを拾い上げることを防ぐ。
+  let headerEnd = block.length
+  for (const tag of TASK_NOTIFICATION_BODY_TAGS) {
+    const at = block.indexOf(tag)
+    if (at >= 0 && at < headerEnd) headerEnd = at
+  }
+  const header = block.slice(0, headerEnd)
+  const tool_use_id = firstTagValue(header, "tool-use-id")
+  const status = firstTagValue(header, "status")
+  // 鍵か状態が欠けた通知は画面側でも無視されるので、流すだけ無駄 → 合成しない。
+  if (!tool_use_id || !status) {
+    taskNotificationEnvelopeDrops += 1
+    return null
+  }
+  // マスクを素通りする経路なので、長さでも歯止めを掛ける (異常に長い値は封筒が壊れたサイン)。
+  if (tool_use_id.length > MAX_TOOL_USE_ID_LEN || status.length > MAX_STATUS_LEN) {
+    taskNotificationEnvelopeDrops += 1
+    return null
+  }
+  const event = { type: "system", subtype: "task_notification", tool_use_id, status }
+  if (obj.timestamp !== undefined) event.timestamp = obj.timestamp
+  return event
+}
+
+/**
+ * jsonl 1 行 (パース済み) を UI 表示用イベントに変換する。表示対象外なら null。
+ *
+ * ⚠️ **認識と合成はこの 1 関数に閉じる**。履歴 hydrate (readSessionHistory) と live watch
+ * (claude-history-watch.mjs) の両方がここを通ること。過去、行のフィルタ / 整形が 2 箇所に
+ * 分散して片方だけ更新されるドリフトが起きているため (normalizeHistoryEvent の注記参照)。
+ *
+ * @param {Record<string, unknown>} obj
+ * @returns {Record<string, unknown>|null}
+ */
+export function historyEventFromLine(obj) {
+  if (!obj || typeof obj !== "object") return null
+  // 1 行から出すイベントは最大 1 個 (画面側の終端は先勝ちなので重複は無害だが、増やさない)。
+  if (DISPLAY_TYPES.has(obj.type)) return normalizeHistoryEvent(obj)
+  return synthesizeTaskNotificationEvent(obj)
+}
+
 /**
  * cwd 文字列を Claude Code の project dir 名に変換する。
  * 例: `/Users/kaz/hub/projects/D00585_partition-lab`
@@ -297,10 +442,10 @@ export async function readSessionHistory(filePath, { maxLines = MAX_HISTORY_LINE
     } catch {
       continue
     }
-    if (!obj || typeof obj !== "object") continue
-    if (!DISPLAY_TYPES.has(obj.type)) continue
-    // SDK message と同じ shape にする (余分な meta は落とす)。整形は共通関数に集約。
-    let event = normalizeHistoryEvent(obj)
+    // SDK message と同じ shape にする (余分な meta は落とす)。認識・整形・合成は
+    // historyEventFromLine 1 箇所に集約 (live watch も同じ関数を通る)。
+    let event = historyEventFromLine(obj)
+    if (!event) continue
     // 秘匿情報 / 非公開スキルプロンプト本文を hydrate 時にも伏せる (古いセッション再表示対策)。
     if (MASK_ENABLED) event = maskMessageObject(event, toolNames)
     events.push(event)

package/src/main.mjs

@@ -54,6 +54,7 @@ import { enforceTotalLiveBound } from "./warm-bounds-total.mjs"
 import { UploadManager } from "./claude-upload.mjs"
 import { requestSelfUninstall } from "./service-install.mjs"
 import {
+  drainTaskNotificationEnvelopeDrops,
   fetchSessionHistory,
   listSessions,
 } from "./claude-history.mjs"
@@ -4401,6 +4402,18 @@ async function dispatch(msg, ctx) {
           projectsRoot,
           logger: ctx.logger,
         })
+        // ⚠️ **観測窓を必ずどこかから読む**。タスク終端通知の封筒検査は、Claude CLI が
+        // ラッパー・タグ順・空白を変えると正当な通知まで落とし、**幽霊カードが静かに戻る**。
+        // 誰も読まないカウンタは「対策した気になるだけの飾り」になる (バンドル自己更新の
+        // 通知で発火率 0 の検査を残していた前例あり)。件数だけを出す (中身は伏せ字を
+        // 素通りするので絶対に出さない)。
+        const envelopeDrops = drainTaskNotificationEnvelopeDrops()
+        if (envelopeDrops > 0) {
+          ctx.logger?.info?.(
+            { drops: envelopeDrops },
+            "history: task-notification の封筒検査で合成しなかった行があります (CLI の形式変更なら通知の復元が効かなくなります)",
+          )
+        }
         ctx.client.send({
           type: "claude.history.response",
           stream_id,