claude-spotter 1.1.0 → 1.1.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 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.1",
   "description": "Audit agent running alongside Claude Code that catches missed tool calls — 気づく役と実行する役の分離",
   "type": "module",
   "bin": {

package/src/cli/install.mjs

@@ -79,28 +79,30 @@ 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) {