@cocorograph/hub-agent 0.5.7 → 0.5.8

package/package.json

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

package/src/main.mjs

@@ -319,6 +319,7 @@ async function dispatch(msg, ctx) {
         try {
           await createTmuxSession(name, cwd, {
             claudeCmd: typeof msg.claude_cmd === "string" ? msg.claude_cmd : undefined,
+            logger: ctx.logger,
           })
           ctx.client.send({
             type: "tmux.create_session.result",
@@ -363,6 +364,7 @@ async function dispatch(msg, ctx) {
           // worktree dir で tmux session を作成 (claude_cmd は createSession の default)
           await createTmuxSession(wtName, wtPath, {
             claudeCmd: typeof msg.claude_cmd === "string" ? msg.claude_cmd : undefined,
+            logger: ctx.logger,
           })
           ctx.client.send({
             type: "worktree.create.result",

package/src/tmux.mjs

@@ -33,6 +33,20 @@ function sanitizeTmuxName(s) {
   return s.replace(/[.:]/g, "-")
 }
 
+/**
+ * 先頭の `~` / `~/` を `os.homedir()` に展開する。
+ *
+ * tmux は `-c` に渡された cwd をシェル展開しないため、フロントから
+ * 受け取った `~/hub/projects/...` を絶対パスに変換する必要がある。
+ * 既に絶対パス / 相対パスなら noop。
+ */
+export function expandTilde(p) {
+  if (typeof p !== "string" || p.length === 0) return p
+  if (p === "~") return os.homedir()
+  if (p.startsWith("~/")) return path.join(os.homedir(), p.slice(2))
+  return p
+}
+
 /**
  * git branch 名から worktree dir 名 (= tmux session 名) を導出する。
  * 例: "feature/multi-area" -> "feature-multi-area"
@@ -70,7 +84,10 @@ async function branchExists(repoDir, branch) {
  * 移植元: D00000_cockpit/webapp/lib/worktrees.ts (createWorktree)
  */
 export async function createWorktreeDir(parentDir, branch) {
-  const repoDir = path.resolve(HUB_PROJECTS_BASE, parentDir)
+  // parentDir に `~/...` 形式が来ても受け付けられるよう先に展開する。
+  // 展開後が絶対パスなら path.resolve は HUB_PROJECTS_BASE を無視して
+  // そのまま使う (Node の path.resolve 仕様)。
+  const repoDir = path.resolve(HUB_PROJECTS_BASE, expandTilde(parentDir))
   if (repoDir !== HUB_PROJECTS_BASE && !repoDir.startsWith(HUB_PROJECTS_BASE + path.sep)) {
     throw new Error("dir outside projects base")
   }
@@ -302,6 +319,20 @@ export async function killManySessions(names, opts = {}) {
  * - opts.claudeCmd で send-keys 内容を上書き可 (空文字なら claude 自動起動しない)
  */
 export async function createSession(name, cwd, opts = {}) {
+  // tmux は `-c` をシェル展開しないため、ここで `~` を絶対パスに展開する。
+  // 展開後の cwd は同名引数で以降の検証 / tmux に渡す。
+  const resolvedCwd = expandTilde(cwd)
+  // cwd 存在チェック (旧 Cockpit 単体実装と同等の防御)
+  // 無いまま tmux に渡すと pane の current_path が HOME など別ディレクトリに
+  // fallback して、SessionStart hook が project を解決できなくなる。
+  try {
+    const st = await fs.stat(resolvedCwd)
+    if (!st.isDirectory()) throw new Error("cwd not a directory")
+  } catch (err) {
+    if (err?.code === "ENOENT") throw new Error(`cwd not found: ${resolvedCwd}`)
+    if (err?.message === "cwd not a directory") throw err
+    throw new Error(`cwd stat failed: ${err?.message || String(err)}`)
+  }
   // 既存チェック
   try {
     await execFileP(tmuxBin(opts), ["has-session", "-t", name])
@@ -311,7 +342,30 @@ export async function createSession(name, cwd, opts = {}) {
     if (msg === "duplicate session") throw err
     // has-session が非 0 = セッション無し
   }
-  await execFileP(tmuxBin(opts), ["new-session", "-d", "-s", name, "-c", cwd])
+  await execFileP(tmuxBin(opts), ["new-session", "-d", "-s", name, "-c", resolvedCwd])
+  // 起動直後 pane の current_path を確認し、想定 cwd と異なれば WARN ログを残す
+  // (Phase 3: 任意検証。tmux 側のクオート / 権限問題で fallback した場合を検知する)
+  if (opts.logger) {
+    try {
+      const { stdout } = await execFileP(tmuxBin(opts), [
+        "display-message",
+        "-p",
+        "-t",
+        `${name}:`,
+        "-F",
+        "#{pane_current_path}",
+      ])
+      const actual = (stdout || "").trim()
+      if (actual && actual !== resolvedCwd) {
+        opts.logger.warn(
+          { session: name, requested: resolvedCwd, actual },
+          "tmux pane cwd differs from requested cwd",
+        )
+      }
+    } catch {
+      // display-message 失敗は致命的でないので飲み込む
+    }
+  }
   const claudeCmd = opts.claudeCmd ?? DEFAULT_CLAUDE_CMD
   if (claudeCmd) {
     await execFileP(tmuxBin(opts), ["send-keys", "-t", name, claudeCmd, "Enter"])