@cocorograph/hub-agent 0.7.10 → 0.7.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocorograph/hub-agent",
-  "version": "0.7.10",
+  "version": "0.7.12",
   "description": "Hub Hosted Cockpit のローカル常駐 agent。Hub と outbound WSS で接続し、ローカルの tmux/pty を中継する。",
   "type": "module",
   "license": "UNLICENSED",

package/src/extract-paragraph.mjs

@@ -0,0 +1,94 @@
+/**
+ * claude TUI (Claude Code 2.1.x) のペイン出力から、
+ * 「直近のアシスタント説明テキスト段落」を保守的に抽出する。
+ *
+ * 目的: PreToolUse フック発火時点で jsonl はまだ前ターン止まりのため、フックの
+ * transcript ベース context_text 抽出は前ターンの古い説明を掴む「取り違え」を起こす。
+ * 一方、claude TUI のペインには「いま回答中の assistant 本文」が既に描画されている
+ * ため、tmux capture-pane の結果からこれを抜き出して permission.request に同梱する
+ * のが回答前に説明文をユーザーへ届ける唯一の経路。
+ *
+ * 出力フォーマット観察 (2026-06-24 実機 claude 2.1.x):
+ *   ⏺ <本文テキスト>            ← アシスタント説明 (これを抽出したい)
+ *   ⏺ Bash(<cmd>)               ← ツール呼び出し (除外したい)
+ *   ⏺ Read(...)                 ← ツール呼び出し
+ *     ⎿  <結果>                 ← ツール結果
+ *
+ * 抽出戦略: ペイン末尾から後ろ向き走査で「⏺ で始まり、続く語がツール名形式
+ * (CamelCase + `(`) でない最初の行」を探し、そこから下方向に連続する「説明文の段落」
+ * を取る。少しでも自信が無いケースは null 返却して呼び出し側で「説明なし」へ縮退。
+ *
+ * 安全方針: 抽出失敗 = null (壊さない)。誤抽出より null の方が UX 上安全。
+ */
+
+const ASSIST_PREFIX = "⏺"
+
+const _MAX_CHARS = 2000
+
+const _MIN_CHARS = 6
+
+const _TOOL_CALL_RE = /^[A-Z][A-Za-z0-9_]*\s*\(/
+
+const _BOX_LINE_RE = /^[─-▟]/
+
+const _STATUS_LINE_RE = /^[✨✹●⚫◯]\s/
+
+const _ELLIPSIS_TAIL_RE = /\n[ \t]*…\s*\+\d+\s+lines[^\n]*$/
+
+/**
+ * @param {string|undefined|null} paneText capturePane の出力 (ANSI 除去済み)
+ * @returns {string|null} 直近のアシスタント説明 (失敗時 null)
+ */
+export function extractLastAssistantText(paneText) {
+  if (typeof paneText !== "string" || !paneText) return null
+  const lines = paneText.replace(/\r/g, "").split("\n")
+
+  let startIdx = -1
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const line = lines[i]
+    if (!line.startsWith(ASSIST_PREFIX + " ")) continue
+    const rest = line.slice(ASSIST_PREFIX.length + 1).trimStart()
+    if (_TOOL_CALL_RE.test(rest)) continue
+    startIdx = i
+    break
+  }
+  if (startIdx < 0) return null
+
+  const parts = []
+  let consecBlank = 0
+  for (let i = startIdx; i < lines.length; i++) {
+    const line = lines[i]
+    if (i > startIdx) {
+      if (line.startsWith(ASSIST_PREFIX + " ")) break
+      if (_BOX_LINE_RE.test(line)) break
+      if (_STATUS_LINE_RE.test(line.trim())) break
+    }
+    if (!line.trim()) {
+      consecBlank++
+      if (consecBlank >= 2) break
+      parts.push(line)
+      continue
+    }
+    consecBlank = 0
+    parts.push(line)
+  }
+
+  let text = parts.join("\n").trimEnd()
+  if (text.startsWith(ASSIST_PREFIX)) {
+    text = text.slice(ASSIST_PREFIX.length).trimStart()
+  }
+  text = text.replace(_ELLIPSIS_TAIL_RE, "").trim()
+  if (text.length < _MIN_CHARS) return null
+
+  if (text.length > _MAX_CHARS) {
+    text = "…" + text.slice(-_MAX_CHARS)
+  }
+  return text
+}
+
+export const __test = {
+  _MAX_CHARS,
+  _MIN_CHARS,
+  _TOOL_CALL_RE,
+  _BOX_LINE_RE,
+}

package/src/main.mjs

@@ -20,6 +20,7 @@ import path from "node:path"
 import pino from "pino"
 
 import { detectPlatform, readConfig, writeConfig } from "./config.mjs"
+import { extractLastAssistantText } from "./extract-paragraph.mjs"
 import { loadPlugins, runHookBroadcast, runHookChain } from "./plugin-loader.mjs"
 import { WsClient } from "./ws-client.mjs"
 import { PtyBridge } from "./pty-bridge.mjs"
@@ -670,7 +671,7 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
   const tuiPermissionBridge = new TuiPermissionBridge({ logger })
   tuiPermissionBridge.on(
     "permission",
-    ({ request_id, session_id, cwd, tool_name, input, context_text }) => {
+    async ({ request_id, session_id, cwd, tool_name, input, context_text }) => {
       if (cwd) {
         try {
           recordChatActivity(cwd, { inputPending: true })
@@ -678,6 +679,33 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
           /* ignore */
         }
       }
+      // ── 直前アシスタント説明の抽出 (T05xxx 根治) ────────────────────────────
+      // PreToolUse フック発火時点で jsonl はまだ前ターン止まり (フックがブロック
+      // している間 claude は jsonl を commit しない) のため、フック側の transcript
+      // 抽出は前ターンの古い説明を掴む「取り違え」を起こす。
+      // 代わりに hub-agent がここで tmux capture-pane を打ち、TUI のペイン上に
+      // 既に描画されている「現ターンの assistant 本文段落 (⏺ プレフィックス)」を
+      // 抜き出して context_text に注入する。フックが既に context_text を
+      // 持っている場合 (旧経路) はそれを尊重し、空 / null のときだけ上書きする。
+      // 抽出失敗時は null フォールバック (= 説明なし) で安全縮退。
+      let liveContext = null
+      if (!context_text && cwd) {
+        try {
+          const sessions = await listTmuxSessions({ logger })
+          const match = Array.isArray(sessions)
+            ? sessions.find((s) => s && s.cwd === cwd)
+            : null
+          if (match && match.name) {
+            const pane = await capturePane(match.name, { noCache: true })
+            liveContext = extractLastAssistantText(pane)
+          }
+        } catch (err) {
+          logger?.debug?.(
+            { err: err?.message, cwd },
+            "capture-pane based context_text extraction failed; falling back to null",
+          )
+        }
+      }
       client.send({
         type: "claude.permission.request",
         stream_id: null,
@@ -686,8 +714,10 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
         request_id,
         tool_name,
         input,
-        // 質問/承認カードの直前アシスタント説明 (フック由来。browser がカード上部に表示)。
-        context_text: context_text ?? null,
+        // 質問/承認カードの直前アシスタント説明 (browser がカード上部に表示)。
+        // 優先順位: (1) フックが既に同梱した context_text、(2) hub-agent が
+        // tmux capture-pane で抽出した liveContext、(3) null。
+        context_text: context_text || liveContext || null,
       })
     },
   )
@@ -2992,9 +3022,12 @@ async function dispatch(msg, ctx) {
         return
       }
       case "worktree.remove": {
-        // body: { request_id, name }
+        // body: { request_id, name, force? }
         // cockpit (PR 1719) のサイドバー削除ボタンから呼ばれる。
-        // git worktree remove --force を実行し、走行中 session があれば事前に kill する。
+        // 2 段階削除:
+        //   - force=false (デフォルト): 未コミット/未 push を検出して dirty なら削除せず
+        //     `dirty: true, changes, unpushed_commits` を返す。フロントが「破棄して削除」確認を出す。
+        //   - force=true: 検査をスキップして `git worktree remove --force` で物理削除。
         const name = (msg.name || "").trim()
         if (!name) {
           ctx.client.send({
@@ -3005,14 +3038,18 @@ async function dispatch(msg, ctx) {
           })
           return
         }
+        const force = msg.force === true
         try {
-          const { name: removedName, wt_path: wtPath } = await removeWorktreeDir(name)
+          const result = await removeWorktreeDir(name, { force })
           ctx.client.send({
             type: "worktree.remove.result",
             request_id: msg.request_id,
             ok: true,
-            name: removedName,
-            wt_path: wtPath,
+            name: result.name,
+            wt_path: result.wt_path,
+            dirty: result.dirty === true,
+            changes: result.changes || [],
+            unpushed_commits: result.unpushed_commits || 0,
           })
         } catch (err) {
           ctx.client.send({

package/src/tmux.mjs

@@ -232,13 +232,66 @@ export async function listWorktreeStubs(liveSessionNames) {
   return stubs
 }
 
+/**
+ * worktree 内の未コミット変更と未 push コミットを検出する（破棄前確認用）。
+ *
+ * - `git status --porcelain` で未コミット変更（追跡/未追跡/変更/削除）を列挙
+ * - `git rev-list --count @{u}..HEAD` で upstream に未 push のコミット数を取得
+ *
+ * upstream 未設定の worktree（新規ブランチで未 push）も「unpushed あり扱い」にして
+ * 誤って削除されないようにする（rev-list が失敗しても unpushed_commits=1 で警告）。
+ *
+ * @param {string} wtPath  worktree の絶対パス
+ * @returns {Promise<{dirty: boolean, changes: string[], unpushed_commits: number}>}
+ */
+async function inspectWorktreeDirty(wtPath) {
+  /** @type {string[]} */
+  let changes = []
+  let unpushed = 0
+  try {
+    const { stdout } = await execFileP("git", ["-C", wtPath, "status", "--porcelain"])
+    changes = String(stdout || "")
+      .split("\n")
+      .map((line) => line.replace(/\r$/, ""))
+      .filter((line) => line.length > 0)
+  } catch {
+    // git status 自体が失敗するケースは滅多に無いが、保守的に「変更あり扱い」にせず空配列で続行
+    changes = []
+  }
+  try {
+    const { stdout } = await execFileP("git", [
+      "-C",
+      wtPath,
+      "rev-list",
+      "--count",
+      "@{u}..HEAD",
+    ])
+    unpushed = parseInt(String(stdout || "0").trim(), 10) || 0
+  } catch {
+    // upstream 未設定 (新規ブランチで未 push) は rev-list が失敗する → 警告のため 1 扱い
+    unpushed = 1
+  }
+  return {
+    dirty: changes.length > 0 || unpushed > 0,
+    changes,
+    unpushed_commits: unpushed,
+  }
+}
+
 /**
  * `git worktree remove --force <wtPath>` を実行する。事前に live tmux session が
  * あれば kill する。cockpit `worktree.remove` ハンドラから呼ばれる。
  *
  * - name から path を解決するには buildWorktreeIndex で fs を走査する (path 検証も兼ねる)
  * - HUB_PROJECTS_BASE 配下に居ない path は拒否 (path traversal 防止)
- * - 未コミットの変更等で git remove が失敗した場合はそのエラーを上位に投げる
+ * - **未コミット変更 / 未 push コミットがあれば、`opts.force=true` でない限り
+ *   削除を中止して構造化応答 `{ ok: true, dirty: true, ... }` を返す**（throw しない、
+ *   呼び出し元が「破棄して削除」確認ダイアログを出して再呼び出ししやすい設計）。
+ * - 未コミットの変更等で git remove 自体が失敗した場合はそのエラーを上位に投げる
+ *
+ * @param {string} name
+ * @param {{ force?: boolean, tmuxBin?: string }} [opts]
+ * @returns {Promise<{ name: string, wt_path: string, dirty?: boolean, changes?: string[], unpushed_commits?: number }>}
  */
 export async function removeWorktree(name, opts = {}) {
   const sanitized = sanitizeTmuxName(String(name || "").trim())
@@ -251,6 +304,22 @@ export async function removeWorktree(name, opts = {}) {
   if (!resolved.startsWith(projectsBase + path.sep)) {
     throw new Error("worktree path outside projects base")
   }
+
+  // force=false (デフォルト): 未コミット / 未 push を検出して dirty なら削除中止
+  if (!opts.force) {
+    const inspect = await inspectWorktreeDirty(resolved)
+    if (inspect.dirty) {
+      // セッションは kill せず（誤操作からの復帰を妨げない）、構造化応答で UI に判断を返す
+      return {
+        name: sanitized,
+        wt_path: resolved,
+        dirty: true,
+        changes: inspect.changes,
+        unpushed_commits: inspect.unpushed_commits,
+      }
+    }
+  }
+
   // 走行中の session があれば kill
   try {
     await execFileP(tmuxBin(opts), ["kill-session", "-t", sanitized])
@@ -271,7 +340,7 @@ export async function removeWorktree(name, opts = {}) {
     ? resolved.slice(0, -wtSegment.length - 1)
     : path.resolve(resolved, "..", "..", "..")
   await execFileP("git", ["-C", parentRepo, "worktree", "remove", "--force", resolved])
-  return { name: sanitized, wt_path: resolved }
+  return { name: sanitized, wt_path: resolved, dirty: false }
 }
 
 /**

package/src/tool-result-mask.mjs

@@ -38,6 +38,18 @@ export const SECRET_PLACEHOLDER = "[REDACTED]"
 export const SKILL_PROMPT_PLACEHOLDER =
   "[get_skill_prompt の本文はマスクされています（必要なら再取得できます）]"
 
+/**
+ * デプロイ接続情報の置換マーカー。
+ *
+ * hub_helper.deploy_creds や mcp__hub_staff__get_deploy_target が返す情報は、
+ * 「秘匿値そのものは返さない」設計（参照パス・has_* フラグのみ）になっているが、
+ * host/port/user/deploy_method 等のメタが画面・jsonl に出ると別案件サーバーへの
+ * 誤デプロイの原因になる（人間オペレーターの目視ミスを誘発）。
+ * 一律にプレースホルダで潰し、AI には「Hub から再取得できる」ことだけ知らせる方針。
+ */
+export const DEPLOY_CRED_PLACEHOLDER =
+  "[デプロイ接続情報はマスクされています（必要なら hub_helper.py deploy_creds で再取得できます）]"
+
 /**
  * 既知の秘匿情報パターン (誤検知を抑えるため、形が明確なトークンに限定する)。
  * 各要素は global フラグ付き RegExp。replace で SECRET_PLACEHOLDER に置換する。
@@ -103,11 +115,33 @@ export function isSkillPromptTool(name) {
   )
 }
 
+/**
+ * tool 名がデプロイ接続情報を返す MCP ツールか判定する。素名と MCP 名前空間
+ * 付き (`mcp__hub_staff__get_deploy_target` / `mcp__hub_staff__deploy_creds` 等)
+ * の両方を拾う。
+ * @param {unknown} name
+ * @returns {boolean}
+ */
+export function isDeployCredentialTool(name) {
+  if (typeof name !== "string") return false
+  return (
+    name === "get_deploy_target" ||
+    name === "deploy_creds" ||
+    name.endsWith("__get_deploy_target") ||
+    name.endsWith("__deploy_creds")
+  )
+}
+
 /** tool_result の content (string | block[]) をプレースホルダ 1 本へ潰す。 */
 function skillPlaceholderContent() {
   return [{ type: "text", text: SKILL_PROMPT_PLACEHOLDER }]
 }
 
+/** tool_result の content をデプロイ接続情報プレースホルダ 1 本へ潰す。 */
+function deployCredPlaceholderContent() {
+  return [{ type: "text", text: DEPLOY_CRED_PLACEHOLDER }]
+}
+
 /** content blocks 配列の text を redactSecrets で処理した新配列を返す (非破壊)。 */
 function redactBlocks(blocks) {
   if (!Array.isArray(blocks)) return blocks
@@ -166,6 +200,10 @@ export function maskMessageObject(obj, toolNames) {
             changed = true
             return { ...b, content: skillPlaceholderContent() }
           }
+          if (isDeployCredentialTool(name)) {
+            changed = true
+            return { ...b, content: deployCredPlaceholderContent() }
+          }
           // 非スキル tool_result: 中身 (string | block[]) の秘匿情報を伏せる。
           if (typeof b.content === "string") {
             const masked = redactSecrets(b.content)