claude-spotter 1.4.8 → 1.4.9

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 1.4.9
+
+**Codex hooks feature 名の現行 CLI 追従**。現行 Codex CLI の `codex features list` は
+hook 機能を `hooks stable true` と表示するが、Spotter の `codex-hook diagnostics` は旧名
+`codex_hooks` だけを見ていたため、hooks 登録済みでも `availability:"unavailable"` と誤判定していた。
+
+### 変更点
+
+- **編集 [src/cli/codex-hook-cmd.mjs](src/cli/codex-hook-cmd.mjs)**:
+  `codexHookDiagnostics` が現行 `hooks` と旧 `codex_hooks` の両方を enabled evidence として扱う。
+  `installCodexHooks` は現行 CLI に合わせて `[features].hooks = true` を書く。既に旧
+  `codex_hooks = true` がある環境では削除せず、`hooks = true` を追加して現行 CLI で確実に有効化する。
+- **編集 [test/codex-hook-cmd.test.mjs](test/codex-hook-cmd.test.mjs)**:
+  現行 `hooks stable true` と旧 `codex_hooks stable true` の diagnostics 回帰テスト、
+  旧 feature key が残る config への install 回帰テストを追加。
+- **編集 docs**:
+  `open-issues.md` の Codex pending / hook event path 記述を v1.4.8 以降の
+  host-neutral `.spotter/pending/` / `.spotter/hook-events.jsonl` に追従。
+
+### ユーザー側で必要な手順
+
+1. `npm install -g claude-spotter@1.4.9`
+2. Codex hooks を使う環境では、各 Spotter install 済み project で `spotter install` を再実行
+   (`~/.codex/config.toml` に `[features].hooks = true` を確実に反映するため)
+
+### 検証
+
+- `node --test` 322 tests / 321 pass / 1 skip 緑
+- `spotter codex-hook diagnostics --project /home/kite/projects/Spotter` が
+  `availability:"available"` / evidence=`hooks stable true` を返すことを実機確認
+
 ## 1.4.8
 
 **Hook 挙動 parity (Codex → Claude) 移植**。Codex 側で確定していた 3 つの hook 挙動 — Stop

package/README.ja.md

@@ -45,6 +45,7 @@ spotter install
 
 `v0.3.0` 以降は**プロジェクト単位の明示的 install** を採用しています (v0.2 までの `postinstall` 自動登録はデーモン増殖の主因だったため撤回)。各プロジェクトの `.claude/settings.json` に hook を登録し、そのプロジェクトでの Claude Code セッションのみで有効になります。
 Codex CLI が使える環境では、同じ `spotter install` が user-level の Codex native hooks も登録します。実際に動くプロジェクトは `spotter install` が作る `.spotter/marker.json` で制限されるため、無関係な Codex セッションでは Spotter は起動しません。
+Codex 側では現行の `[features].hooks = true` を有効化し、互換のため旧 `codex_hooks` diagnostics output も認識します。
 
 Spotter を upgrade した後、release note で hook 設定変更が案内されている場合は、各 install 済みプロジェクトで `spotter install` を再実行してください。global package update でコード経路は変わりますが、既存 `.claude/settings.json` の timeout 値は自動では書き換わりません。
 

package/README.md

@@ -45,6 +45,7 @@ spotter install
 
 Since `v0.3.0`, Spotter requires **explicit per-project install** (the earlier `postinstall` auto-registration was the leading cause of orphan daemons). `spotter install` writes hooks into the project's `.claude/settings.json`; the audit is then active only in Claude Code sessions for that project.
 When the Codex CLI is available, the same `spotter install` also registers user-level Codex native hooks. Project activation still depends on the same per-project `.spotter/marker.json`, so unrelated Codex sessions do not trigger Spotter.
+For Codex, install enables the current `[features].hooks = true` flag and still recognizes older `codex_hooks` diagnostics output for compatibility.
 
 After upgrading Spotter, re-run `spotter install` in each installed project when release notes mention hook setting changes. The global package update changes the code path, but existing `.claude/settings.json` timeout values are not rewritten automatically.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-spotter",
-  "version": "1.4.8",
+  "version": "1.4.9",
   "description": "Audit agent running alongside Claude Code that catches missed tool calls — 気づく役と実行する役の分離",
   "type": "module",
   "bin": {

package/src/cli/codex-hook-cmd.mjs

@@ -35,6 +35,7 @@ const CODEX_HOOK_TIMEOUT_SEC = 60;
 const DEFAULT_CODEX_HOOK_AUDITOR_TIMEOUT_MS = 20_000;
 const SHORT_PROMPT_MAX_CHARS = 10;
 const DEFAULT_CODEX_STOP_SHORT_FINAL_MAX_CHARS = 120;
+const CODEX_HOOK_FEATURE_NAMES = ['hooks', 'codex_hooks'];
 
 const CODEX_HOOK_USAGE = `spotter codex-hook — Codex native hook adapter
 
@@ -364,7 +365,8 @@ export async function codexHookDiagnostics({ codexHome = defaultCodexHome(), pro
   const features = spawnSyncFn('codex', ['features', 'list'], { encoding: 'utf8', maxBuffer: 1024 * 1024 });
   const featureOutput = [features.stdout, features.stderr].filter(Boolean).join('\n');
   const hooks = await loadJson(join(codexHome, 'hooks.json'));
-  const codexHooksFeature = /^codex_hooks\s+\S+\s+true\b/m.test(featureOutput) ? 'enabled' : 'not-enabled';
+  const evidence = featureOutput.split('\n').find((line) => isEnabledCodexHookFeatureLine(line)) ?? null;
+  const codexHooksFeature = evidence ? 'enabled' : 'not-enabled';
   const installed = {
     sessionStart: hookState(hooks, 'SessionStart', 'codex-hook session-start'),
     userPromptSubmit: hookState(hooks, 'UserPromptSubmit', 'codex-hook user-prompt-submit'),
@@ -382,7 +384,7 @@ export async function codexHookDiagnostics({ codexHome = defaultCodexHome(), pro
     codexHome,
     hooksPath: join(codexHome, 'hooks.json'),
     installedHooks: installed,
-    evidence: featureOutput.split('\n').find((line) => line.trim().startsWith('codex_hooks')) ?? null,
+    evidence,
     runtime: runtimeProjectRoot
       ? await summarizeCodexHookEvents({ projectRoot: runtimeProjectRoot })
       : null,
@@ -653,7 +655,7 @@ function isSpotterCodexHook(hook) {
 async function ensureCodexHooksFeature(configPath) {
   let raw = '';
   if (existsSync(configPath)) raw = await readFile(configPath, 'utf8');
-  if (/^\s*codex_hooks\s*=\s*true\s*$/m.test(raw)) return 'already-enabled';
+  if (/^\s*hooks\s*=\s*true\s*$/m.test(raw)) return 'already-enabled';
   const next = enableCodexHooksFeature(raw);
   await mkdir(dirname(configPath), { recursive: true });
   await writeFile(configPath, next, 'utf8');
@@ -663,7 +665,7 @@ async function ensureCodexHooksFeature(configPath) {
 function enableCodexHooksFeature(raw) {
   const text = raw.trimEnd();
   if (!/^\s*\[features\]\s*$/m.test(text)) {
-    return `${text}${text ? '\n\n' : ''}[features]\ncodex_hooks = true\n`;
+    return `${text}${text ? '\n\n' : ''}[features]\nhooks = true\n`;
   }
   const lines = text.split('\n');
   let inFeatures = false;
@@ -678,15 +680,23 @@ function enableCodexHooksFeature(raw) {
       inFeatures = /^\s*\[features\]\s*$/.test(line);
       continue;
     }
-    if (inFeatures && /^\s*codex_hooks\s*=/.test(line)) {
-      lines[index] = 'codex_hooks = true';
+    if (inFeatures && /^\s*hooks\s*=/.test(line)) {
+      lines[index] = 'hooks = true';
       return `${lines.join('\n')}\n`;
     }
   }
-  lines.splice(insertAt, 0, 'codex_hooks = true');
+  lines.splice(insertAt, 0, 'hooks = true');
   return `${lines.join('\n')}\n`;
 }
 
+function isEnabledCodexHookFeatureLine(line) {
+  const trimmed = String(line ?? '').trim();
+  return CODEX_HOOK_FEATURE_NAMES.some((name) => {
+    const escaped = name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+    return new RegExp(`^${escaped}\\s+\\S+\\s+true\\b`).test(trimmed);
+  });
+}
+
 function hookState(settings, event, commandFragment) {
   const groups = settings?.hooks?.[event];
   if (!Array.isArray(groups)) return 'not-installed';