claude-spotter 1.4.10 → 1.4.12

package/CHANGELOG.md

@@ -1,5 +1,47 @@
 # Changelog
 
+## 1.4.12
+
+**macOS/Homebrew install verification docs**。npm registry からの clean global install 後に
+`spotter install` と `spotter codex-hook install` が通ること、Codex hook が versioned
+Homebrew Cellar Node path ではなく安定 symlink を使うことを README に明記した。
+
+### 変更点
+
+- **編集 [README.md](README.md) / [README.ja.md](README.ja.md)**:
+  macOS Homebrew Node 環境では Codex hook command が `/opt/homebrew/bin/node` を使うこと、
+  release install smoke 手順 (`npm uninstall -g`, `npm install -g`, `spotter install -y`,
+  `spotter codex-hook install`) を追加。
+
+### 検証
+
+- npm registry からの実インストールで確認:
+  `npm uninstall -g claude-spotter`, `npm install -g claude-spotter`,
+  `spotter --version`, `spotter install -y`, `spotter codex-hook install` が exit 0。
+
+## 1.4.11
+
+**macOS Homebrew Node の versioned Cellar path を Codex hook に残さない修正**。
+`spotter codex-hook install` が `/opt/homebrew/Cellar/node/<version>/bin/node` を hook command に
+書くと、Homebrew の Node 更新後に Codex hook が古い Node を指したまま壊れるため、現在の
+Node 実体と一致する PATH 上の安定 symlink を優先するようにした。
+
+### 変更点
+
+- **編集 [src/cli/codex-hook-cmd.mjs](src/cli/codex-hook-cmd.mjs)**:
+  `resolveCodexHookNodePath()` を追加。PATH 上の `node` が `process.execPath` と同じ realpath を
+  指す場合は、その安定パスを hook command に使う。該当しない環境では従来通り
+  `process.execPath` に fallback するため、Windows / WSL2 の挙動は維持される。
+- 既存の Spotter-owned Codex hook が古い Cellar Node path を指している場合、再実行で現在の
+  stable Node path と npm global package path に書き換える。Caveat など他 tool の hook は保持する。
+
+### 検証
+
+- `node --test` 334 tests / 333 pass / 1 skip 緑。
+- npm registry からの実インストールで確認:
+  `npm uninstall -g claude-spotter`, `npm install -g claude-spotter`,
+  `spotter --version`, `spotter install -y`, `spotter codex-hook install` が exit 0。
+
 ## 1.4.10
 
 **Claude host primary auditor: Codex CLI 検出で自動採用、なければ Haiku の 2 段選択へ**。

package/README.ja.md

@@ -43,6 +43,11 @@ cd your-project
 spotter install
 ```
 
+macOS の Homebrew Node 環境では、Codex hook command の Node パスに現在の実体と一致する
+安定 symlink (`/opt/homebrew/bin/node`) を使います。
+`/opt/homebrew/Cellar/node/<version>/...` のような version 固定パスを書かないため、
+Homebrew で Node が更新されても Codex hook が古い Node パスに取り残されません。
+
 `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 も認識します。
@@ -53,6 +58,16 @@ Spotter を upgrade した後、release note で hook 設定変更が案内さ
 spotter uninstall        # このプロジェクトの hook 登録を解除
 ```
 
+リリース時の npm install smoke:
+
+```bash
+npm uninstall -g claude-spotter
+npm install -g claude-spotter
+spotter --version
+spotter install -y
+spotter codex-hook install
+```
+
 ## 動作要件
 
 - **Node.js 22.5 以上**

package/README.md

@@ -43,6 +43,11 @@ cd your-project
 spotter install
 ```
 
+On macOS with Homebrew Node, Codex hook commands use the stable
+`/opt/homebrew/bin/node` symlink when it resolves to the current Node binary,
+instead of a versioned `/opt/homebrew/Cellar/node/<version>/...` path. That keeps
+Codex hooks working across Homebrew Node upgrades.
+
 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.
@@ -53,6 +58,16 @@ After upgrading Spotter, re-run `spotter install` in each installed project when
 spotter uninstall        # remove hooks from this project
 ```
 
+Release install smoke:
+
+```bash
+npm uninstall -g claude-spotter
+npm install -g claude-spotter
+spotter --version
+spotter install -y
+spotter codex-hook install
+```
+
 ## Requirements
 
 - **Node.js 22.5+**

package/package.json

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

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

@@ -1,8 +1,8 @@
 import { spawnSync } from 'node:child_process';
-import { existsSync } from 'node:fs';
+import { existsSync, realpathSync } from 'node:fs';
 import { mkdir, readFile, writeFile } from 'node:fs/promises';
 import { homedir } from 'node:os';
-import { dirname, join, resolve } from 'node:path';
+import { delimiter, dirname, join, resolve } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { createAuditorBackend } from '../core/auditor-backend.mjs';
 import { codexLastAssistantMessage, readCodexUsedTools } from '../core/codex-transcript.mjs';
@@ -315,7 +315,7 @@ export async function runCodexHookDiagnosticsCommand({
   writeOutput(JSON.stringify(result, null, 2) + '\n');
 }
 
-export async function installCodexHooks({ codexHome = defaultCodexHome(), nodePath = process.execPath, spotterBin = SPOTTER_BIN } = {}) {
+export async function installCodexHooks({ codexHome = defaultCodexHome(), nodePath = resolveCodexHookNodePath(), spotterBin = SPOTTER_BIN } = {}) {
   const hooksPath = join(codexHome, 'hooks.json');
   const configPath = join(codexHome, 'config.toml');
   const current = await loadJson(hooksPath);
@@ -340,6 +340,41 @@ export async function installCodexHooks({ codexHome = defaultCodexHome(), nodePa
   };
 }
 
+function safeRealpath(path, realpath = realpathSync.native) {
+  try {
+    return realpath(path);
+  } catch {
+    return null;
+  }
+}
+
+export function resolveCodexHookNodePath({
+  env = process.env,
+  execPath = process.execPath,
+  platform = process.platform,
+  exists = existsSync,
+  realpath = realpathSync.native,
+} = {}) {
+  const execRealpath = safeRealpath(execPath, realpath);
+  const pathEnv = env.PATH || env.Path || '';
+  const names = platform === 'win32'
+    ? ['node.exe', 'node.cmd', 'node.bat', 'node']
+    : ['node'];
+
+  for (const dir of pathEnv.split(delimiter).filter(Boolean)) {
+    for (const name of names) {
+      const candidate = join(dir, name);
+      if (!exists(candidate)) continue;
+      const candidateRealpath = safeRealpath(candidate, realpath);
+      if (execRealpath && candidateRealpath && candidateRealpath === execRealpath) {
+        return candidate;
+      }
+    }
+  }
+
+  return execPath;
+}
+
 export async function uninstallCodexHooks({ codexHome = defaultCodexHome() } = {}) {
   const hooksPath = join(codexHome, 'hooks.json');
   const current = await loadJson(hooksPath);

package/src/core/codex-sidecar-runner.mjs

@@ -468,7 +468,7 @@ function execPortable(execFileFn, cmd, args, spawnOptions) {
     windowsHide: true,
     maxBuffer: 10 * 1024 * 1024,
   };
-  if (process.platform === 'win32') {
+  if (process.platform === 'win32' && cmd !== process.execPath) {
     return execFileFn('cmd.exe', ['/c', cmd, ...args], options);
   }
   return execFileFn(cmd, args, options);

package/src/version.mjs

@@ -1,2 +1,7 @@
-import pkg from '../package.json' with { type: 'json' };
+import { readFileSync } from 'node:fs';
+import { fileURLToPath } from 'node:url';
+import { dirname, resolve } from 'node:path';
+
+const pkgPath = resolve(dirname(fileURLToPath(import.meta.url)), '..', 'package.json');
+const pkg = JSON.parse(readFileSync(pkgPath, 'utf8'));
 export const version = pkg.version;