@cocorograph/hub-agent 0.7.11 → 0.7.12

package/package.json

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

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)