claude-spotter 1.1.0 → 1.1.2

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## 1.1.2
+
+**v1.1.1 の code-review で発見した 2 件を修正**。Spotter 自身が監査役として指摘し、実装を補正する自己ドッグフーディング。
+
+### 変更点
+
+- **編集 [src/cli/install.mjs](src/cli/install.mjs)**: `refresh` 呼び出しを try/catch で包み、throw 直前に stderr で復旧経路 (`spotter db refresh`) を露出。§0 の fallback 禁止は守りつつ、「hook 登録済み + tool-db なし」状態に陥ったユーザーに次の一手を示す診断メッセージを追加
+- **編集 [src/cli/install.mjs](src/cli/install.mjs)**: `runInstall` に `refreshFn` パラメータを追加 (default: 実 refresh)。テストから mock を注入できるようにした
+- **編集 [test/install.test.mjs](test/install.test.mjs)**: 新規 2 件追加 — (1) 2 回目 install でも refresh が呼ばれる回帰ガード (v1.1.1 fix の直接検証)、(2) refresh 失敗時に stderr に復旧ヒントが出ることを確認
+- **編集 [docs/open-issues.md](docs/open-issues.md)**: P2 に「tool-db.json の並列書き込み race condition」を追記 (install と SessionStart bg refresh が同時に走ると last-writer-wins、実害観測なしなので放置)
+
+### 設計判断
+
+- **race condition は v1.1.2 で修正しない**: 実運用で install はユーザー対話的に 1 回叩く想定 = 並列発生頻度は極低、失われた差分は次 refresh で再投入されるので最終収束。lock 機構は over-engineering
+
+## 1.1.1
+
+**既 install プロジェクトで `spotter install` が refresh を skip してしまう bug の hot-fix**。v1.1.0 で追加した tool-db 自動構築が、hook 登録済みの場合に [install.mjs](src/cli/install.mjs) の早期 return に引っかかって走らない穴があった。これでは「既に install 済みのプロジェクトで tool-db.json が作られない」という v1.1.0 が解決すべき症状がそのまま残る。
+
+### 変更点
+
+- **編集 [src/cli/install.mjs](src/cli/install.mjs)**: hook 登録不要時の早期 return を削除、if/else 構造に組み直して **settings.json の差分有無に関わらず refresh が走る** ようにした。v1.1.0 升級後の既 install プロジェクトでも `spotter install` 再実行で tool-db.json が seed される。副次効果として `spotter install` 再実行が「hook 登録 + tool-db drift 補正」の標準オペレーションとして使える
+
 ## 1.1.0
 
 **`spotter install` が tool-db を自動構築 + SessionStart hook がバックグラウンド refresh**。install 直後から audit 対象が揃うようになり、以降の session でも MCP / スキル / サブエージェントの追加・削除が自動追従する。

package/package.json

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

package/src/cli/install.mjs

@@ -36,7 +36,7 @@ const HOOK_EVENTS = [
   { event: 'SessionEnd', sub: 'session-end', timeout: 3 },
 ];
 
-export async function runInstall({ target = 'project', autoYes = false, cwd = process.cwd(), skipRefresh = false } = {}) {
+export async function runInstall({ target = 'project', autoYes = false, cwd = process.cwd(), skipRefresh = false, refreshFn = refresh } = {}) {
   const settingsPath = target === 'user'
     ? join(homedir(), '.claude', 'settings.json')
     : join(cwd, '.claude', 'settings.json');
@@ -79,37 +79,48 @@ export async function runInstall({ target = 'project', autoYes = false, cwd = pr
 
   if (diff === null) {
     console.log('  hooks already registered — nothing to change');
-    return;
-  }
-
-  console.log('\n--- proposed .claude/settings.json changes ---');
-  console.log(diff);
-  console.log('---------------------------------------------\n');
-
-  if (!autoYes) {
-    const rl = createInterface({ input: process.stdin, output: process.stdout });
-    const answer = await rl.question('apply these changes? [y/N] ');
-    rl.close();
-    if (!/^y(es)?$/i.test(answer.trim())) {
-      console.log('aborted.');
-      return;
+  } else {
+    console.log('\n--- proposed .claude/settings.json changes ---');
+    console.log(diff);
+    console.log('---------------------------------------------\n');
+
+    if (!autoYes) {
+      const rl = createInterface({ input: process.stdin, output: process.stdout });
+      const answer = await rl.question('apply these changes? [y/N] ');
+      rl.close();
+      if (!/^y(es)?$/i.test(answer.trim())) {
+        console.log('aborted.');
+        return;
+      }
     }
-  }
 
-  await mkdir(dirname(settingsPath), { recursive: true });
-  await writeFile(settingsPath, JSON.stringify(updated, null, 2) + '\n', 'utf8');
-  console.log(`wrote ${settingsPath}`);
+    await mkdir(dirname(settingsPath), { recursive: true });
+    await writeFile(settingsPath, JSON.stringify(updated, null, 2) + '\n', 'utf8');
+    console.log(`wrote ${settingsPath}`);
+  }
 
   // Seed the tool-db so the first session has something to audit against.
+  // Runs regardless of whether settings.json changed — re-running `spotter install`
+  // on an already-installed project is the canonical way to refresh tool-db drift
+  // (skills added, MCP servers registered, etc.) alongside upgrades.
   // Skipped for user-mode (deprecated — no projectRoot) and when caller opts out
   // (tests set skipRefresh=true to avoid scanning the real user environment).
   if (target === 'project' && !skipRefresh) {
     console.log('\ndiscovering MCP servers, skills, and sub-agents...');
     const log = (msg) => process.stderr.write(`  ${msg}\n`);
-    const resolved = await refresh({ projectRoot: cwd, logFn: log });
-    console.log(`  ${resolved.size} tool(s) resolved`);
-    console.log(`  local DB:  ${localDbPath(cwd)}`);
-    console.log(`  global DB: ${globalDbPath()}`);
+    try {
+      const resolved = await refreshFn({ projectRoot: cwd, logFn: log });
+      console.log(`  ${resolved.size} tool(s) resolved`);
+      console.log(`  local DB:  ${localDbPath(cwd)}`);
+      console.log(`  global DB: ${globalDbPath()}`);
+    } catch (err) {
+      // §0: throw (fallback 禁止). But surface the recovery path so the user isn't
+      // left with "hooks registered, tool-db missing" and no clue what to run.
+      process.stderr.write(`\nspotter install: tool-db seeding failed.\n`);
+      process.stderr.write(`  hooks are registered but tool-db is not ready.\n`);
+      process.stderr.write(`  recover with: spotter db refresh\n`);
+      throw err;
+    }
   }
 
   console.log('\nnext steps:');