@cocorograph/hub-agent 0.6.69 → 0.6.71

package/package.json

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

package/src/chat-signals.mjs

@@ -28,7 +28,12 @@ const _byCwd = new Map()
 
 // チャット信号を「生きている」と見なす最大経過時間 (ms)。これを過ぎたら tmux
 // スクレイプ結果へフォールバックさせる (チャット終了/別モード移行の検知)。
-const CHAT_SIGNAL_STALE_MS = 15 * 60 * 1000
+// 90s: SDK の result/abort を取りこぼしても status=processing がこの時間で解除される
+// (RC-4 偽陽性 = 中断/異常終了で三点リーダーが固着する最長時間の短縮。旧 15 分)。正常な
+// 生成中は SDK の assistant イベントが頻繁に updatedAtMs/statusAt を前進させるため途中で
+// 切れない。abort/異常終了は claude-stream-bridge の finally が即 clearChatSignal するのが
+// 一次対策で、本 TTL はその取りこぼし時のバックストップ。
+const CHAT_SIGNAL_STALE_MS = 90 * 1000
 
 const VALID_STATUS = new Set(["processing", "waiting", "idle"])
 

package/src/claude-md.mjs

@@ -82,6 +82,18 @@ async function fetchRepositories({ hubUrl, accessToken, dirName, fetchImpl }) {
  *
  * リポジトリが 1 件もない場合は空文字列を返し、CLAUDE.md には何も追記しない。
  */
+/**
+ * repo_url が GitHub の `owner/repo` slug 形式かを判定する。
+ *
+ * `://` や `@` を含むフル URL（GitLab の `git@gitlab.com:...` や
+ * `https://gitlab.com/...` 等）は false を返す。これらは `gh` ではなく
+ * 素の `git clone` でチェックアウトする。
+ */
+function isGitHubSlug(repoUrl) {
+  if (/:\/\/|@/.test(repoUrl)) return false
+  return /^[A-Za-z0-9_.-]+\/[A-Za-z0-9_.-]+$/.test(repoUrl)
+}
+
 function renderRepositorySection(repositories, dirName = "") {
   if (!Array.isArray(repositories) || repositories.length === 0) return ""
 
@@ -113,16 +125,28 @@ function renderRepositorySection(repositories, dirName = "") {
     if (productionName) lines.push(`- **本番**: ${productionName}`)
     lines.push("")
 
+    const githubSlug = isGitHubSlug(repoUrl)
     lines.push("**作業手順（クローンレス）**:")
     lines.push("")
     lines.push("```bash")
     lines.push("TMPDIR=$(mktemp -d)")
-    lines.push(`gh repo clone ${repoUrl} "$TMPDIR" -- --depth 1 --filter=blob:none --no-checkout`)
-    lines.push(`cd "$TMPDIR" && git fetch origin ${branch} && git checkout -b feat/<your-change> origin/${branch}`)
-    lines.push("# ファイル編集 → 動作確認")
-    lines.push('git commit -am "<message>"')
-    lines.push("git push -u origin HEAD")
-    lines.push("gh pr create --base " + branch + " --fill")
+    if (githubSlug) {
+      // GitHub: gh CLI で clone・PR 作成
+      lines.push(`gh repo clone ${repoUrl} "$TMPDIR" -- --depth 1 --filter=blob:none --no-checkout`)
+      lines.push(`cd "$TMPDIR" && git fetch origin ${branch} && git checkout -b feat/<your-change> origin/${branch}`)
+      lines.push("# ファイル編集 → 動作確認")
+      lines.push('git commit -am "<message>"')
+      lines.push("git push -u origin HEAD")
+      lines.push(`gh pr create --base ${branch} --fill`)
+    } else {
+      // GitLab 等: 素の git clone。MR は push 後に Web/glab で作成
+      lines.push(`git clone --depth 1 --filter=blob:none --no-checkout ${repoUrl} "$TMPDIR"`)
+      lines.push(`cd "$TMPDIR" && git fetch origin ${branch} && git checkout -b feat/<your-change> origin/${branch}`)
+      lines.push("# ファイル編集 → 動作確認")
+      lines.push('git commit -am "<message>"')
+      lines.push("git push -u origin HEAD")
+      lines.push("# MR は GitLab の Web UI（push 時に表示される URL）か `glab mr create` で作成")
+    }
     lines.push("# 作業終了後")
     lines.push('cd / && rm -rf "$TMPDIR"')
     lines.push("```")
@@ -165,7 +189,7 @@ function renderRepositorySection(repositories, dirName = "") {
     "- **本番反映の前に必ず現状バックアップを取る**（DB ダンプ / 対象ディレクトリの退避）。",
     "- ステージングで動作確認してから本番へ。",
     "- 破壊的操作（`reset --hard` / `rm -rf` / DB 操作）の前に対象を確認する。",
-    "- Git 認証はローカルの `gh` CLI / SSH 鍵を使用します。",
+    "- Git 認証はローカルの `gh` CLI（GitHub）/ SSH 鍵（GitLab 等）を使用します。",
     "",
   )
   return lines.join("\n")

package/src/claude-stream-bridge.mjs

@@ -206,6 +206,7 @@ class ClaudeStreamSession {
     onExit,
     onError,
     onReap,
+    onTurnSettled,
   }) {
     this.stream_id = stream_id
     /** 多端末共有: このセッションを購読中の全端末 stream_id。CHAT_SHARED_ENABLED 時のみ
@@ -236,6 +237,10 @@ class ClaudeStreamSession {
     this.onError = onError
     /** ターン完走後に遅延クローズする際、manager にセッション撤去を依頼するコールバック */
     this.onReap = onReap
+    /** RC-4: result を届けずにターンが終わった (abort/異常終了) ときに呼ぶコールバック。
+     *  manager 経由で main.mjs がチャット信号の status=processing を waiting へ落とす
+     *  (三点リーダー/ステータスドットが TTL まで固着するのを防ぐ)。 */
+    this.onTurnSettled = onTurnSettled
 
     /** 次の query() で resume に使う session_id。各ターンの system/init で更新。 */
     this.sessionId = resumeSessionId || null
@@ -632,6 +637,9 @@ class ClaudeStreamSession {
     this._busy = true
     this._abortController = new AbortController()
     let aborted = false
+    // RC-4: このターンで SDK の result イベントが届いたか。届かずに終わった
+    // (abort / 異常終了) ときだけ onTurnSettled でチャット信号を waiting へ落とす。
+    let resultDelivered = false
 
     const options = {
       cwd: this.cwd,
@@ -677,8 +685,9 @@ class ClaudeStreamSession {
           this._ensureWatch()
         }
         // result イベントでも session_id が来ることがある (念のため拾う)
-        if (msg?.type === "result" && typeof msg.session_id === "string") {
-          this.sessionId = msg.session_id
+        if (msg?.type === "result") {
+          resultDelivered = true
+          if (typeof msg.session_id === "string") this.sessionId = msg.session_id
         }
         try {
           this.onEvent?.(msg)
@@ -726,6 +735,18 @@ class ClaudeStreamSession {
       if (aborted) {
         this.logger?.info({ stream_id: this.stream_id }, "claude turn aborted")
       }
+      // RC-4: result を届けずにターンが終わった (abort / 異常終了) 場合、チャット信号の
+      // status=processing が解除されず三点リーダー/ステータスドットが TTL まで固着する。
+      // ターン確定として onTurnSettled を呼び processing→waiting へ落とす。正常完了は
+      // result が既に waiting にしているので !resultDelivered のときだけ。_drainPending で
+      // 次ターン (processing) が始まる前に呼ぶこと (順序: ここ → reap/drain)。
+      if (!resultDelivered) {
+        try {
+          this.onTurnSettled?.()
+        } catch {
+          /* ignore */
+        }
+      }
       // graceful detach: browser が切れている間にターンが完走したら、ここで遅延
       // クローズする。manager 側で sessions Map から撤去 + exit を emit する。
       if (this._reapAfterTurn && !this._closed) {
@@ -1305,6 +1326,15 @@ export class ClaudeStreamBridge extends EventEmitter {
           session_id: session.sessionId,
         })
       },
+      onTurnSettled: () => {
+        // RC-4: result 不在でターンが終わった (abort/異常終了)。main.mjs が cwd 一致の
+        // チャット信号を processing→waiting へ落とす (固着解除)。
+        this.emit("turnsettled", {
+          stream_id: session.stream_id,
+          session_id: session.sessionId,
+          cwd: session.cwd,
+        })
+      },
     })
     this.sessions.set(stream_id, session)
     this.logger?.info(

package/src/main.mjs

@@ -489,6 +489,18 @@ export async function startDaemon({ version, ptyModule, claudeSdk } = {}) {
     claudeBridge.on("exit", ({ stream_id, code, reason, session_id }) => {
       client.send({ type: "claude.exit", stream_id, code, reason, session_id })
     })
+    claudeBridge.on("turnsettled", ({ cwd }) => {
+      // RC-4: result を届けずにターンが終わった (abort/異常終了)。チャット信号の
+      // status=processing を waiting に落として三点リーダー/ステータスドットの固着を防ぐ
+      // (正常完了は上の result ハンドラが既に waiting にしている)。
+      if (cwd) {
+        try {
+          recordChatActivity(cwd, { status: "waiting", inputPending: false })
+        } catch {
+          /* ignore */
+        }
+      }
+    })
     claudeBridge.on("error", ({ stream_id, session_id, error }) => {
       client.send({ type: "claude.error", stream_id, session_id, error })
     })
@@ -875,6 +887,16 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge }) {
   const lastTurnAtByName = new Map() // session_name → 最後に event ファイル化した turnAt
   let stopped = false
 
+  // RC-8: WS 再接続のたびに差分送信の基準 (lastByName) をクリアし、次 tick で全 session.state を
+  // 再 push する。差分送信のままだと、切断中に起きた status 遷移 (processing→stop 等) の push が
+  // 失われ、再接続後も frontend が古い processing に固着する (zombie WS の検知が遅れると最大数十秒)。
+  // client は (再)接続ごとに "open" を emit する。フロントの tmux.list_sessions poll でも回復するが、
+  // これで再接続直後の次 tick (≤intervalMs) に短縮する。初回接続時は空 Map の clear で no-op。
+  const onReopen = () => {
+    lastByName.clear()
+  }
+  client.on("open", onReopen)
+
   const tick = async () => {
     if (stopped) return
     try {
@@ -963,6 +985,7 @@ function startStateLoop({ client, plugins, logger, intervalMs, claudeBridge }) {
       stopped = true
       clearInterval(ti)
       clearTimeout(t0)
+      client.off?.("open", onReopen)
     },
   }
 }

package/src/state.mjs

@@ -120,8 +120,38 @@ function stripAnsi(s) {
     .replace(/\x1b[()][AB012]/g, "")
 }
 
+/**
+ * capturePane 結果の末尾フッター領域 (既定 8 行) を返す。claude TUI の作業スピナー /
+ * 中断フッターはペイン最下部に固定描画されるため、本文中に同じ語が出ても誤検出しない
+ * よう、作業スピナーの補助判定はこの領域に限定する。
+ */
+function footerRegion(text, lines = 8) {
+  if (!text) return ""
+  const rows = text.split("\n")
+  return rows.length <= lines ? text : rows.slice(-lines).join("\n")
+}
+
+/**
+ * 作業スピナーのフッター行を検出する。claude TUI の生成中フッターは
+ * "✻ Cogitating… (12s · ↑ 1.2k tokens · esc to interrupt)" のように
+ * 「経過秒」と「トークンカウンタ」が同一行に同居する。idle/waiting の権限バナーや
+ * 通常本文には出ないシグネチャなので、中断フッター文言 ("esc to interrupt") が
+ * 未描画 (ターン序盤) / locale 差 / ツール実行中などで取りこぼされる区間でも
+ * 生成中を拾える。誤検出を抑えるためフッター領域 + 同一行同居を必須にする。
+ */
+function detectWorkingSpinner(text) {
+  const footer = footerRegion(text)
+  for (const line of footer.split("\n")) {
+    if (/\btokens\b/i.test(line) && /(?:^|[\s(])\d+\s*s\b/.test(line)) return true
+  }
+  return false
+}
+
 export function detectStatusFromText(text) {
+  // 主シグナル: 中断フッター (従来どおり全体一致。回帰防止のため範囲を狭めない)。
   if (/esc to interrupt/i.test(text)) return "processing"
+  // 補助シグナル: 作業スピナーの「経過秒 + トークンカウンタ」(フッター領域・同一行限定)。
+  if (detectWorkingSpinner(text)) return "processing"
   if (/❯\s/.test(text) || /^>\s/m.test(text)) return "waiting"
   return "idle"
 }
@@ -276,8 +306,11 @@ export async function capturePane(sessionName, opts = {}) {
       "-p",
       "-t",
       sessionName,
+      // 末尾 50 行を取得 (旧 30 行)。copy-mode スクロールや背の高いペインで作業/中断
+      // フッターが取得窓外に出て生成中を取りこぼす偽陰性を減らす。status 判定の作業
+      // スピナー検出はこのうち末尾フッター領域 (footerRegion) に限定する。
       "-S",
-      "-30",
+      "-50",
       "-E",
       "-",
     ])