@cocorograph/hub-agent 0.7.87 → 0.7.88

package/package.json

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

package/src/claude-stream-bridge.mjs

@@ -764,13 +764,17 @@ class ClaudeStreamSession {
    *    常駐 query が異常終了して resume 再起動する際 (_runResidentQuery) に最新値で
    *    再 spawn させるため、および per-message セッションが次ターンの options に反映する
    *    ため。
-   *  - 起動済みの常駐 query があれば SDK の制御メソッド (setModel / setPermissionMode /
-   *    setMaxThinkingTokens) を呼び、プロセス再起動なしで次ターンから即反映する
+   *  - model / permissionMode は起動済み常駐 query の SDK 制御メソッド (setModel /
+   *    setPermissionMode) を呼び、プロセス再起動なしで次ターンから即反映する
    *    (公式 streaming input mode のランタイム制御。stdin に control_request を流す)。
+   *  - maxThinkingTokens(思考 budget) / effort はランタイム setter が無い (SDK の
+   *    setMaxThinkingTokens は @deprecated、setThinking/setEffort は未提供) ため、
+   *    保持値の更新のみ行い restart-on-change で反映する (T06119)。per-message は次
+   *    ターンの options 構築で即反映、常駐 query は次 spawn (resume 再起動) で反映。
    *
    *  値が undefined のキーは「変更なし」として無視する (バッジ未送出時に既存値を消さない)。
    *  model に空文字/null が来たら setModel(undefined) でデフォルトへ戻す。
-   *  maxThinkingTokens に 0/null が来たら setMaxThinkingTokens(null) でオフにする。 */
+   *  maxThinkingTokens に 0/null が来たら次 spawn 時に budget オフ (thinking 未指定) となる。 */
   applyRuntimeOptions({ model, permissionMode, maxThinkingTokens, effort } = {}) {
     const q = this._residentQuery
     // モデル
@@ -805,23 +809,18 @@ class ClaudeStreamSession {
         }
       }
     }
-    // 拡張思考予算
+    // 拡張思考予算 (非 effort モデルの固定 budget)。SDK の
+    // setMaxThinkingTokens は @deprecated で、公式後継 `thinking` オプションは
+    // 構築時専用 (ランタイム setThinking 相当が無い)。そこで effort と同じ
+    // restart-on-change に統一し、保持値の更新のみ行う。per-message は次ターンの
+    // options 構築時に _applyThinkingOptions が this.maxThinkingTokens を読むため
+    // 即反映。常駐 query は次 spawn (resume 再起動) 時に反映される。
     if (maxThinkingTokens !== undefined) {
       const next =
         typeof maxThinkingTokens === "number" && maxThinkingTokens > 0
           ? maxThinkingTokens
           : null
-      if (next !== this.maxThinkingTokens) {
-        this.maxThinkingTokens = next
-        if (q && typeof q.setMaxThinkingTokens === "function") {
-          q.setMaxThinkingTokens(next).catch((err) =>
-            this.logger?.warn(
-              { err: err?.message, stream_id: this.stream_id },
-              "setMaxThinkingTokens failed",
-            ),
-          )
-        }
-      }
+      this.maxThinkingTokens = next
     }
     // effort (Opus 4.6+)。SDK には setEffort 相当のランタイム制御メソッドが無いため、
     // 保持値の更新のみ行う。per-message セッションは次ターンの options 構築時に
@@ -858,9 +857,12 @@ class ClaudeStreamSession {
       // budget 方式 (maxThinkingTokens) は使わない (Opus 4.7+ で非対応)。
       options.thinking = { type: "adaptive" }
       if (this.effort) options.effort = this.effort
-    } else if (this.maxThinkingTokens != null) {
-      // 非 effort モデル (Sonnet / Haiku 等) は従来の budget 方式を維持。
-      options.maxThinkingTokens = this.maxThinkingTokens
+    } else if (this.maxThinkingTokens != null && this.maxThinkingTokens > 0) {
+      // 非 effort モデル (Sonnet / Haiku 等) は固定 budget 方式を維持するが、
+      // SDK 公式後継の `thinking:{type:'enabled',budgetTokens}` で渡す。旧
+      // `options.maxThinkingTokens`(@deprecated) は使わない。0/null は何も
+      // 設定せず SDK 既定に委ねる (従来と同じ挙動)。
+      options.thinking = { type: "enabled", budgetTokens: this.maxThinkingTokens }
     }
   }
 
@@ -1845,6 +1847,8 @@ export class ClaudeStreamBridge extends EventEmitter {
     this._mcpControlIdleTimer = null
     /** 起動中の Promise (同時アクセスの二重起動防止)。 */
     this._mcpControlStarting = null
+    /** listSupportedCommands() の短命キャッシュ {at:ms, commands:SlashCommand[]} (T06117)。 */
+    this._supportedCommandsCache = null
   }
 
   /** _liveBySession の複合キー (T06271): cwd を含めることで、別 workspace の同一 session_id を
@@ -2213,8 +2217,24 @@ export class ClaudeStreamBridge extends EventEmitter {
       // control メソッド (mcpServerStatus 等) は別チャネル経由なので drain と独立に動く。
       ;(async () => {
         try {
-          for await (const _msg of q) {
-            void _msg
+          for await (const msg of q) {
+            // T06117: supportedCommands() は init スナップショットで、以降の動的
+            // 変更 (サブディレクトリでのスキル発見 / plugin reload 等) を反映しない。
+            // 権威的な最新一覧は commands_changed push でしか来ないため、この control
+            // query の stream で拾ってキャッシュを更新する (TTL 再取得では stale 固定)。
+            // ⚠️ 同一性ガード (codex-review v3): teardown → 新 query 起動後に、畳んだ
+            // 旧 query のバッファ済み / 遅延 commands_changed が到着すると、新 query 由来の
+            // 最新キャッシュを旧一覧で上書きしてしまう。この drain が現アクティブ query の
+            // ものか (this._mcpControlQuery === q) を確認してから更新する (finally と同型)。
+            if (
+              this._mcpControlQuery === q &&
+              msg?.type === "system" &&
+              msg?.subtype === "commands_changed"
+            ) {
+              this._supportedCommandsCache = {
+                commands: Array.isArray(msg.commands) ? msg.commands : [],
+              }
+            }
           }
         } catch (err) {
           this.logger?.warn(
@@ -2225,6 +2245,9 @@ export class ClaudeStreamBridge extends EventEmitter {
           if (this._mcpControlQuery === q) {
             this._mcpControlQuery = null
             this._mcpControlInput = null
+            // control query が死んだらキャッシュも破棄する。次回アクセスで新しい
+            // query から取り直す (別 query の init スナップショットに更新される)。
+            this._supportedCommandsCache = null
           }
         }
       })()
@@ -2262,6 +2285,12 @@ export class ClaudeStreamBridge extends EventEmitter {
     const q = this._mcpControlQuery
     this._mcpControlInput = null
     this._mcpControlQuery = null
+    // T06117 (codex-review): teardown は _mcpControlQuery を先に null 化するため、
+    // 後追いで走る drain finally の同一性ガード (this._mcpControlQuery === q) が外れ、
+    // finally 側のキャッシュ破棄が発火しない。ここで直接破棄しないと、idle reap /
+    // reload / shutdown 後も古いキャッシュが残り続け、listSupportedCommands が新しい
+    // query を起動せず stale を返し続ける (ライフタイムキャッシュの永続化バグ)。
+    this._supportedCommandsCache = null
     try {
       input?.close()
     } catch {
@@ -2324,6 +2353,56 @@ export class ClaudeStreamBridge extends EventEmitter {
     }
   }
 
+  /**
+   * SDK `supportedCommands()` を control-only query 経由で引き、そのセッションで実際に
+   * ロードされている slash command / skill の権威一覧 (SlashCommand[]) を返す (T06117)。
+   *
+   * hub-agent 既定 cwd の control query を使うため、返るのは **組込コマンド + グローバル
+   * skill (~/.claude/skills) + plugin / dynamic workflow** で、FS 走査では取りこぼす
+   * プラグイン提供・無効化・動的発見スキルも SDK が正しく反映する。project-local skill
+   * (~/hub/projects/<name>/.claude/skills) は control query の cwd 外なので含まれず、
+   * 呼び出し側 (skills.mjs) が FS 走査で補完する。
+   *
+   * キャッシュは control query の **ライフタイム単位**。初回に init スナップショットを
+   * 積み、以降は control query stream の commands_changed push で更新、query 死亡
+   * (idle reap 等) で破棄する (_ensureMcpControlQuery の drain / finally)。
+   * ⚠️ supportedCommands() の再取得は init スナップショットを返すだけで動的更新
+   * (commands_changed) を取り込めない (SDK 仕様) ため、TTL 再取得はしない
+   * (取り込んだ最新一覧を stale スナップショットで上書きしてしまう)。失敗時は null を
+   * 返し、呼び出し側は FS 走査へフォールバックする (SDK 未実装 / 一時エラーでも
+   * SkillBar が空にならない)。
+   *
+   * @returns {Promise<Array<{name:string,description:string,argumentHint?:string,aliases?:string[]}>|null>}
+   */
+  async listSupportedCommands() {
+    // ライフタイムキャッシュ: 存在すれば (init スナップショット + commands_changed で
+    // 反映済みの最新) をそのまま返す。再取得しない (上記 ⚠️)。
+    if (this._supportedCommandsCache) return this._supportedCommandsCache.commands
+    try {
+      const q = await this._ensureMcpControlQuery()
+      this._armMcpControlIdle()
+      if (typeof q.supportedCommands !== "function") {
+        this.logger?.warn({}, "SDK に supportedCommands() が無い (FS 走査へフォールバック)")
+        return null
+      }
+      const commands = await q.supportedCommands()
+      const arr = Array.isArray(commands) ? commands : []
+      // commands_changed が先に届いてキャッシュを埋めていたら、その最新を尊重する
+      // (init スナップショットで上書きしない)。
+      if (!this._supportedCommandsCache) {
+        this._supportedCommandsCache = { commands: arr }
+      }
+      this.logger?.info({ count: this._supportedCommandsCache.commands.length }, "supported commands loaded from SDK")
+      return this._supportedCommandsCache.commands
+    } catch (err) {
+      this.logger?.warn(
+        { err: err?.message },
+        "listSupportedCommands failed (skills は FS 走査へフォールバック)",
+      )
+      return null
+    }
+  }
+
   /**
    * アカウント全体の 5h/7d rate-limit 使用率 + プラン種別を返す (usage.mjs
    * recordChatUsageSnapshot に渡す想定)。mcpStatus と同じ control-only query を共有する

package/src/main.mjs

@@ -5471,9 +5471,18 @@ async function dispatch(msg, ctx) {
           // Cockpit Codex 両対応 (Stream A3, 2026-07-02): browser が cli_kind を
           // 添えれば Codex 向け skills (~/.codex/skills/) を返す。未指定は
           // 従来どおり claude 向け。
+          const skillsCliKind = msg.cli_kind === "codex" ? "codex" : "claude"
+          // T06117: claude は SDK supportedCommands() を権威に一覧を作る (組込 +
+          // グローバル skill + plugin/dynamic を FS 走査の取りこぼしなく反映)。
+          // 取れなければ null で listSkills が FS 走査へフォールバックする。
+          let sdkCommands = null
+          if (skillsCliKind === "claude" && ctx.claudeBridge?.listSupportedCommands) {
+            sdkCommands = await ctx.claudeBridge.listSupportedCommands()
+          }
           const skills = await listSkills({
             sessionName: msg.session_name,
-            cliKind: msg.cli_kind === "codex" ? "codex" : "claude",
+            cliKind: skillsCliKind,
+            sdkCommands,
           })
           ctx.client.send({
             type: "skills.response",

package/src/skills.mjs

@@ -28,7 +28,12 @@ function skillsDirFor(cliKind) {
   return cliKind === "codex" ? CODEX_GLOBAL_SKILLS_DIR : GLOBAL_SKILLS_DIR
 }
 
-// Claude Code 組み込みスラッシュコマンド (ファイルから取得できないのでハードコード)
+// Claude Code 組み込みスラッシュコマンド。
+// ⚠️ T06117: 通常経路では **SDK supportedCommands() を権威** に組込コマンドを得るため
+// このハードコードは使わない。ここに残すのは「SDK が引けない (未実装 SDK / 一時エラー /
+// 起動直後で control query 未確立) ときの cold-start フォールバック専用」。SDK 実返却の
+// 組込名 (usage / context / reload-skills 等) とは一部ズレるが、フォールバックとして
+// SkillBar を空にしないための安全網。正確な一覧は SDK 側が権威。
 const BUILT_IN_COMMANDS = [
   { name: "clear", description: "会話履歴をクリア (短期メモリのリセット)", source: "builtin" },
   { name: "compact", description: "会話を要約して圧縮 (context 節約)", source: "builtin" },
@@ -145,6 +150,20 @@ async function loadCommandsDir(commandsDir, source) {
  */
 export async function listSkills(opts = {}) {
   const cliKind = opts.cliKind === "codex" ? "codex" : "claude"
+  const sessionName = opts.sessionName
+  const projectsRoot = opts.projectsRoot || path.join(os.homedir(), "hub", "projects")
+
+  // SDK 権威経路 (T06117): claude かつ呼び出し側が supportedCommands() の結果
+  // (opts.sdkCommands) を渡してきたら、組込 + グローバル skill + plugin / dynamic
+  // workflow を **SDK 一覧** から構築する。FS 走査では取りこぼすプラグイン提供・
+  // 無効化・動的発見スキルも SDK が正しく反映する。label_ja は SDK が持たないため
+  // FS の frontmatter から name でオーバーレイし、control query の cwd 外にある
+  // project-local skill は FS 走査で補完する。
+  if (cliKind === "claude" && Array.isArray(opts.sdkCommands)) {
+    return await listSkillsFromSdk(opts.sdkCommands, sessionName, projectsRoot)
+  }
+
+  // フォールバック経路: SDK 未取得 / Codex は従来どおり FS 走査 + 組込ハードコード。
   const out = []
   out.push(...(cliKind === "codex" ? CODEX_BUILT_IN_COMMANDS : BUILT_IN_COMMANDS))
   out.push(...(await loadSkillsDir(skillsDirFor(cliKind), "skill")))
@@ -152,8 +171,6 @@ export async function listSkills(opts = {}) {
     out.push(...(await loadCommandsDir(GLOBAL_COMMANDS_DIR, "command")))
   }
 
-  const sessionName = opts.sessionName
-  const projectsRoot = opts.projectsRoot || path.join(os.homedir(), "hub", "projects")
   if (sessionName && /^D\d{5}_/.test(sessionName)) {
     const projectDirName = cliKind === "codex" ? ".codex" : ".claude"
     const projectDir = path.join(projectsRoot, sessionName, projectDirName)
@@ -173,3 +190,77 @@ export async function listSkills(opts = {}) {
   deduped.sort((a, b) => a.name.localeCompare(b.name))
   return deduped
 }
+
+/** SDK supportedCommands() の description 末尾に付くソース注記
+ *  (` (user)` / ` (dynamic workflow)` / ` (resumable with /resume)` 等) を除去する。
+ *  SDK は末尾の括弧にソースを埋めるため、tooltip 表示用に落とす。 */
+export function stripCommandSourceSuffix(desc) {
+  if (!desc) return ""
+  return desc.replace(/\s*\([^()]*\)\s*$/, "").trim()
+}
+
+/** description 末尾のソース注記から CockpitSkill.source を導出する。
+ *  `(user)` → skill / `(project…)` → project-skill / それ以外 (組込・plugin・
+ *  dynamic workflow・注記なし) → builtin。claude は常に `/` 起動なので source は主に
+ *  frontend の「builtin 日本語ラベル辞書引き」に使われる (skill は label_ja を持つ)。 */
+export function sourceFromCommandSuffix(desc) {
+  const m = (desc || "").match(/\(([^()]*)\)\s*$/)
+  const tag = m ? m[1].toLowerCase() : ""
+  if (tag === "user") return "skill"
+  if (tag.startsWith("project")) return "project-skill"
+  return "builtin"
+}
+
+/**
+ * SDK supportedCommands() の結果 (SlashCommand[]) を CockpitSkill[] へ変換する。
+ * - ソース注記を strip / source を導出
+ * - FS の frontmatter (label_ja) を name でオーバーレイ
+ * - control query の cwd 外にある project-local skill/command を FS 走査で補完
+ * - 同名 dedup / 名前順 sort
+ */
+async function listSkillsFromSdk(sdkCommands, sessionName, projectsRoot) {
+  const entries = []
+  const seen = new Set()
+  for (const c of sdkCommands) {
+    if (!c || !c.name || seen.has(c.name)) continue
+    seen.add(c.name)
+    entries.push({
+      name: c.name,
+      description: stripCommandSourceSuffix(c.description),
+      source: sourceFromCommandSuffix(c.description),
+    })
+  }
+
+  // label_ja オーバーレイ用の FS 走査 (global) + project-local の補完材料。
+  const globalLabelSources = [
+    ...(await loadSkillsDir(GLOBAL_SKILLS_DIR, "skill")),
+    ...(await loadCommandsDir(GLOBAL_COMMANDS_DIR, "command")),
+  ]
+  let projectEntries = []
+  if (sessionName && /^D\d{5}_/.test(sessionName)) {
+    const projectDir = path.join(projectsRoot, sessionName, ".claude")
+    projectEntries = [
+      ...(await loadSkillsDir(path.join(projectDir, "skills"), "project-skill")),
+      ...(await loadCommandsDir(path.join(projectDir, "commands"), "project-command")),
+    ]
+  }
+
+  const labelMap = new Map()
+  for (const s of [...globalLabelSources, ...projectEntries]) {
+    if (s.label) labelMap.set(s.name, s.label)
+  }
+  for (const e of entries) {
+    const label = labelMap.get(e.name)
+    if (label) e.label = label
+  }
+
+  // project-local は control query の cwd 外なので SDK 一覧に無い名前だけ追加する。
+  for (const p of projectEntries) {
+    if (seen.has(p.name)) continue
+    seen.add(p.name)
+    entries.push(p)
+  }
+
+  entries.sort((a, b) => a.name.localeCompare(b.name))
+  return entries
+}