@cocorograph/hub-agent 0.7.11 → 0.7.13

package/package.json

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

package/src/claude-md-sync.mjs

@@ -29,6 +29,7 @@ const DEFAULT_HUB_API = "https://api.hub.cocorograph.com"
 const TOKEN_PATH = path.join(os.homedir(), ".claude", ".hub_token.json")
 const META_DIR = ".cockpit-sync"
 const META_FILE = "claude-md.json"
+const BASE_FILE = "CLAUDE.md.base"
 const BAK_KEEP = 3 // 通常 .bak と衝突 .bak それぞれ直近 3 件まで保持
 
 async function readJsonOrNull(p) {
@@ -160,6 +161,23 @@ async function writeMeta(targetDir, meta) {
   )
 }
 
+/**
+ * 同期成功時の本文を `.cockpit-sync/CLAUDE.md.base` に保存する。
+ *
+ * session-end スキル (対話中の Claude) が CLAUDE.md を Hub と 3-way merge する際の
+ * "base" 入力として使う。前回 sync 直後のローカル == Hub の本文を保持しておくことで、
+ * 「ローカル変更」と「Hub 変更」の差分を別々に抽出してから意味マージできる。
+ *
+ * meta(claude-md.json) は hash と updated_at しか持たないため、本文そのものが要る
+ * 3-way merge の用途ではこのファイルが必要。
+ */
+async function writeBaseBody(targetDir, body) {
+  if (body == null) return
+  const metaDir = path.join(targetDir, META_DIR)
+  await fs.mkdir(metaDir, { recursive: true })
+  await fs.writeFile(path.join(metaDir, BASE_FILE), body, "utf-8")
+}
+
 /**
  * 既存ローカル CLAUDE.md を .cockpit-sync/CLAUDE.md.<prefix>.<ts>.bak に退避する。
  * 同 prefix の .bak は新しい順に BAK_KEEP 件保持し、古いものから FIFO 削除する。
@@ -289,6 +307,8 @@ export async function syncClaudeMdWithHub({
       lastSyncedHash: hubHash || sha256(hubBody),
       lastSyncedAt: nowIso(),
     })
+    // 3-way merge 用 base: pull 直後はローカル == Hub なので Hub 本文を base に。
+    await writeBaseBody(targetDir, hubBody)
     logger?.info?.({ dirName }, "syncClaudeMdWithHub: pull")
     return { action: "pull" }
   }
@@ -315,6 +335,9 @@ export async function syncClaudeMdWithHub({
       lastSyncedHash: localCurrentHash,
       lastSyncedAt: nowIso(),
     })
+    // 3-way merge 用 base: push 直後は Hub の正本がローカルと一致するので、
+    // 次回の base は今 push した本文 (= localBody)。
+    await writeBaseBody(targetDir, localBody || "")
     logger?.info?.({ dirName }, "syncClaudeMdWithHub: push")
     return { action: "push" }
   }
@@ -339,6 +362,8 @@ export async function syncClaudeMdWithHub({
     lastSyncedHash: localCurrentHash,
     lastSyncedAt: nowIso(),
   })
+  // 3-way merge 用 base: conflict push 後の Hub 正本は端末側本文 (= localBody)。
+  await writeBaseBody(targetDir, localBody || "")
   logger?.warn?.(
     { dirName },
     "syncClaudeMdWithHub: conflict (terminal wins); hub body saved to .bak.conflict",
@@ -349,13 +374,15 @@ export async function syncClaudeMdWithHub({
 // メタ操作・ハッシュ計算は ensureClaudeMd 側 (Hub pull 初回配信) でも使うため
 // named export する。同じファイルパス・同じスキーマで管理することで、初回配信と
 // 以降の sync で食い違いを起こさない。
-export { sha256, readMeta, writeMeta, nowIso }
+export { sha256, readMeta, writeMeta, writeBaseBody, nowIso }
 
 // 内部テスト用エクスポート
 export const _internals = {
   sha256,
   readMeta,
   writeMeta,
+  writeBaseBody,
   backupLocalClaudeMd,
   BAK_KEEP,
+  BASE_FILE,
 }

package/src/claude-md.mjs

@@ -22,6 +22,7 @@ import path from "node:path"
 import {
   sha256,
   writeMeta,
+  writeBaseBody,
   nowIso,
 } from "./claude-md-sync.mjs"
 
@@ -379,6 +380,8 @@ export async function ensureClaudeMd({
         lastSyncedHash: hub.claude_md_hash || sha256(hubBody),
         lastSyncedAt: nowIso(),
       })
+      // 初回 pull 直後は base = Hub 本文。以降の session-end 3-way merge の入力。
+      await writeBaseBody(targetDir, hubBody)
       logger?.info?.(
         { dirName, claudeMdPath },
         "wrote CLAUDE.md from Hub project claude_md (initial sync)",

package/src/main.mjs

@@ -3022,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({
@@ -3035,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)