codebyplan 1.13.34 → 1.13.35

package/dist/cli.js

@@ -14,7 +14,7 @@ var VERSION, PACKAGE_NAME;
 var init_version = __esm({
   "src/lib/version.ts"() {
     "use strict";
-    VERSION = "1.13.34";
+    VERSION = "1.13.35";
     PACKAGE_NAME = "codebyplan";
   }
 });
@@ -1606,22 +1606,16 @@ function mergeEnabledPluginsIntoSettings(settings, plugins) {
   }
   return settings;
 }
-function stripOwnedHooksFromSettings(settings) {
+function collapseEmptyHookBlocks(settings) {
   if (!settings.hooks) {
-    return settings;
+    return;
   }
   for (const event of Object.keys(settings.hooks)) {
     const eventBlocks = settings.hooks[event];
     if (!eventBlocks) {
       continue;
     }
-    const survivingBlocks = [];
-    for (const block of eventBlocks) {
-      block.hooks = block.hooks.filter((e) => e._owner !== OWNER);
-      if (block.hooks.length > 0) {
-        survivingBlocks.push(block);
-      }
-    }
+    const survivingBlocks = eventBlocks.filter((b) => b.hooks.length > 0);
     if (survivingBlocks.length > 0) {
       settings.hooks[event] = survivingBlocks;
     } else {
@@ -1631,6 +1625,54 @@ function stripOwnedHooksFromSettings(settings) {
   if (Object.keys(settings.hooks).length === 0) {
     delete settings.hooks;
   }
+}
+function stripOwnedHooksFromSettings(settings) {
+  if (!settings.hooks) {
+    return settings;
+  }
+  for (const event of Object.keys(settings.hooks)) {
+    const eventBlocks = settings.hooks[event];
+    if (!eventBlocks) {
+      continue;
+    }
+    for (const block of eventBlocks) {
+      block.hooks = block.hooks.filter((e) => e._owner !== OWNER);
+    }
+  }
+  collapseEmptyHookBlocks(settings);
+  return settings;
+}
+function pruneMissingManagedHooks(settings, hooksJson, onPruned) {
+  if (!settings.hooks) {
+    return settings;
+  }
+  const liveIdsByEvent = /* @__PURE__ */ new Map();
+  for (const [event, matchers] of Object.entries(hooksJson.hooks)) {
+    const ids = /* @__PURE__ */ new Set();
+    for (const block of matchers) {
+      for (const cmd of block.hooks) {
+        ids.add(extractHookId(cmd.command));
+      }
+    }
+    liveIdsByEvent.set(event, ids);
+  }
+  for (const event of Object.keys(settings.hooks)) {
+    const eventBlocks = settings.hooks[event];
+    if (!eventBlocks) {
+      continue;
+    }
+    const liveIds = liveIdsByEvent.get(event) ?? /* @__PURE__ */ new Set();
+    for (const block of eventBlocks) {
+      block.hooks = block.hooks.filter((e) => {
+        const shouldPrune = e._owner === OWNER && e._hook_id !== void 0 && !liveIds.has(e._hook_id);
+        if (shouldPrune && onPruned) {
+          onPruned(e._hook_id);
+        }
+        return !shouldPrune;
+      });
+    }
+  }
+  collapseEmptyHookBlocks(settings);
   return settings;
 }
 var OWNER, PLACEHOLDER_RE, REPLACEMENT, SCALAR_BASE_KEYS, DEPRECATED_BASE_KEYS;
@@ -2558,9 +2600,9 @@ async function runLspFull(projectPath, opts = {}) {
     `  Detected LSP servers: ${servers.map((s) => s.plugin).join(", ")}
 `
   );
-  const settingsPath = join10(projectPath, ".claude", "settings.json");
+  const settingsLocalPath = join10(projectPath, ".claude", "settings.local.json");
   let settings = {};
-  const existingSettingsRaw = await readJsonFile(settingsPath);
+  const existingSettingsRaw = await readJsonFile(settingsLocalPath);
   if (existingSettingsRaw) {
     settings = existingSettingsRaw;
   }
@@ -2569,15 +2611,15 @@ async function runLspFull(projectPath, opts = {}) {
     servers.map((s) => s.plugin)
   );
   if (dryRun) {
-    console.log(`  [dry-run] would update ${settingsPath}`);
+    console.log(`  [dry-run] would update ${settingsLocalPath}`);
   } else {
     await mkdir4(join10(projectPath, ".claude"), { recursive: true });
     await writeFile6(
-      settingsPath,
+      settingsLocalPath,
       JSON.stringify(settings, null, 2) + "\n",
       "utf-8"
     );
-    console.log(`  Updated ${settingsPath}`);
+    console.log(`  Updated ${settingsLocalPath}`);
   }
   const lspJsonPath = join10(projectPath, ".codebyplan", "lsp.json");
   const lspJsonContent = {
@@ -2648,7 +2690,7 @@ async function runLspFull(projectPath, opts = {}) {
     `    Enabled plugins: ${pluginNames.length > 0 ? pluginNames.join(", ") : "none"}`
   );
   console.log(
-    `    (settings.json key format: <plugin>@claude-plugins-official)`
+    `    (settings.local.json key format: <plugin>@claude-plugins-official)`
   );
   if (autoInstalled.length > 0) {
     console.log(
@@ -2811,6 +2853,13 @@ async function runInstall(opts, deps = {}) {
         fs3.readFileSync(hooksJsonPath, "utf8")
       );
       mergeHooksIntoSettings(existingSettings, hooksJson);
+      pruneMissingManagedHooks(
+        existingSettings,
+        hooksJson,
+        opts.verbose ? (id) => console.log(
+          `${opts.dryRun ? "[dry-run] would prune" : "pruned"} removed managed hook: ${id}`
+        ) : void 0
+      );
     }
     if (!opts.dryRun) {
       fs3.mkdirSync(path4.dirname(settingsPath), { recursive: true });
@@ -8806,6 +8855,13 @@ async function runUpdate(opts, deps = {}) {
         fs7.readFileSync(hooksJsonPath, "utf8")
       );
       mergeHooksIntoSettings(existingSettings, hooksJson);
+      pruneMissingManagedHooks(
+        existingSettings,
+        hooksJson,
+        opts.verbose ? (id) => console.log(
+          `${opts.dryRun ? "[dry-run] would prune" : "pruned"} removed managed hook: ${id}`
+        ) : void 0
+      );
     }
     if (!opts.dryRun) {
       fs7.mkdirSync(path8.dirname(settingsPath), { recursive: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codebyplan",
-  "version": "1.13.34",
+  "version": "1.13.35",
   "description": "CLI for CodeByPlan — AI-powered development planning and tracking",
   "type": "module",
   "bin": {