skillrepo 1.4.0 → 1.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillrepo",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "description": "Set up SkillRepo in any IDE — one command",
   "type": "module",
   "bin": {

package/src/lib/mergers/hooks-json.mjs

@@ -9,11 +9,12 @@
 
 import { unlinkSync } from "node:fs";
 import { readFileSafe, writeFileSafe } from "../fs-utils.mjs";
-import { claudeSettingsLocal, claudeSyncHook, claudePromptHook, claudeHooksDir } from "../paths.mjs";
+import { claudeSettingsLocal, claudeSyncHook, claudePromptHook, claudePreToolHook, claudeHooksDir } from "../paths.mjs";
 import { join } from "node:path";
 
 const DEFAULT_SYNC_COMMAND = "node .claude/hooks/skillrepo-sync.mjs";
 const DEFAULT_PROMPT_COMMAND = "node .claude/hooks/skillrepo-prompt-match.mjs";
+const DEFAULT_PRETOOL_COMMAND = "node .claude/hooks/skillrepo-pretool-match.mjs";
 
 /**
  * Extract a hook command from a hooks config group, falling back to a default.
@@ -50,11 +51,13 @@ function hasCommand(groups, command) {
  * @param {object} hooksConfig - The hooks config object from the server payload (settingsHooks.hooks)
  * @param {string} syncHookContent - The sync hook script content
  * @param {string} promptHookContent - The prompt-match hook script content
+ * @param {string} [preToolHookContent] - The pretool-match hook script content (optional for backward compat)
  * @returns {{ results: { path: string; action: string }[] }}
  */
-export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent) {
+export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent, preToolHookContent) {
   const syncCommand = extractCommand(hooksConfig, "SessionStart", DEFAULT_SYNC_COMMAND);
   const promptCommand = extractCommand(hooksConfig, "UserPromptSubmit", DEFAULT_PROMPT_COMMAND);
+  const preToolCommand = extractCommand(hooksConfig, "PreToolUse", DEFAULT_PRETOOL_COMMAND);
   const results = [];
 
   // Always write hook scripts (latest version from server)
@@ -72,6 +75,15 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
     action: promptExisted ? "updated" : "created",
   });
 
+  if (preToolHookContent) {
+    const preToolExisted = readFileSafe(claudePreToolHook()) !== null;
+    writeFileSafe(claudePreToolHook(), preToolHookContent);
+    results.push({
+      path: ".claude/hooks/skillrepo-pretool-match.mjs",
+      action: preToolExisted ? "updated" : "created",
+    });
+  }
+
   // Clean up stale .claude/hooks/hooks.json if it exists
   const staleHooksJson = join(claudeHooksDir(), "hooks.json");
   try {
@@ -118,6 +130,17 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
     changed = true;
   }
 
+  // Merge PreToolUse
+  if (preToolHookContent) {
+    if (!Array.isArray(config.hooks.PreToolUse)) config.hooks.PreToolUse = [];
+    if (!hasCommand(config.hooks.PreToolUse, preToolCommand)) {
+      config.hooks.PreToolUse.push({
+        hooks: [{ type: "command", command: preToolCommand }],
+      });
+      changed = true;
+    }
+  }
+
   if (existing === null) {
     writeFileSafe(filePath, JSON.stringify(config, null, 2) + "\n");
     results.push({ path: ".claude/settings.local.json", action: "created" });

package/src/lib/paths.mjs

@@ -15,8 +15,10 @@ export const claudeHooksDir = () => join(cwd(), ".claude", "hooks");
 export const claudeSettingsLocal = () => join(cwd(), ".claude", "settings.local.json");
 export const claudeSyncHook = () => join(cwd(), ".claude", "hooks", "skillrepo-sync.mjs");
 export const claudePromptHook = () => join(cwd(), ".claude", "hooks", "skillrepo-prompt-match.mjs");
+export const claudePreToolHook = () => join(cwd(), ".claude", "hooks", "skillrepo-pretool-match.mjs");
 export const claudeSkillrepoMd = () => join(cwd(), ".claude", "skillrepo.md");
 export const claudeSkillrepoIndex = () => join(cwd(), ".claude", "skillrepo-index.json");
+export const claudeSkillrepoConfig = () => join(cwd(), ".claude", "skillrepo-config.json");
 
 // Cursor
 export const cursorDir = () => join(cwd(), ".cursor");

package/src/lib/write-configs.mjs

@@ -4,7 +4,7 @@
  */
 
 import { readFileSafe, writeFileSafe } from "./fs-utils.mjs";
-import { claudeSkillrepoMd, claudeSkillrepoIndex, cursorSkillrepoMdc } from "./paths.mjs";
+import { claudeSkillrepoMd, claudeSkillrepoIndex, claudeSkillrepoConfig, cursorSkillrepoMdc } from "./paths.mjs";
 import { mergeClaudeMcpConfig } from "./mergers/claude-mcp.mjs";
 import { mergeCursorMcpConfig } from "./mergers/cursor-mcp.mjs";
 import { mergeWindsurfMcpConfig } from "./mergers/windsurf-mcp.mjs";
@@ -38,11 +38,19 @@ export function writeAllConfigs({ ides, mcpUrl, apiKey, payload }) {
     writeFileSafe(claudeSkillrepoIndex(), payload.claudeCode.skillIndex.content);
     results.push({ path: ".claude/skillrepo-index.json", action: indexExisted ? "updated" : "created" });
 
-    // Hooks (SessionStart sync + UserPromptSubmit prompt-match)
+    // Hook config (injection limits + companion map)
+    if (payload.claudeCode.skillrepoConfig?.content) {
+      const configExisted = readFileSafe(claudeSkillrepoConfig()) !== null;
+      writeFileSafe(claudeSkillrepoConfig(), payload.claudeCode.skillrepoConfig.content);
+      results.push({ path: ".claude/skillrepo-config.json", action: configExisted ? "updated" : "created" });
+    }
+
+    // Hooks (SessionStart sync + UserPromptSubmit prompt-match + PreToolUse pretool-match)
     const hookResults = mergeHooksConfig(
       payload.claudeCode.settingsHooks.hooks,
       payload.claudeCode.syncHook.content,
-      payload.claudeCode.promptHook.content
+      payload.claudeCode.promptHook.content,
+      payload.claudeCode.preToolHook?.content
     );
     results.push(...hookResults.results);
   }

package/src/test/mergers/hooks-json.test.mjs

@@ -20,10 +20,12 @@ afterEach(() => {
 
 const SYNC_CONTENT = "// sync hook";
 const PROMPT_CONTENT = "// prompt hook";
+const PRETOOL_CONTENT = "// pretool hook";
 
 const HOOKS_CONFIG = {
   SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: "node .claude/hooks/skillrepo-sync.mjs" }] }],
   UserPromptSubmit: [{ hooks: [{ type: "command", command: "node .claude/hooks/skillrepo-prompt-match.mjs" }] }],
+  PreToolUse: [{ hooks: [{ type: "command", command: "node .claude/hooks/skillrepo-pretool-match.mjs" }] }],
 };
 
 describe("Hooks settings.local.json merger", () => {
@@ -225,4 +227,47 @@ describe("Hooks settings.local.json merger", () => {
     const syncOnDisk = readFileSync(join(tempDir, ".claude/hooks/skillrepo-sync.mjs"), "utf-8");
     assert.equal(syncOnDisk, "// new sync content", "sync script should have new content (partial write)");
   });
+
+  it("writes pretool hook script when preToolHookContent is provided", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT, PRETOOL_CONTENT);
+
+    const preToolResult = results.find((r) => r.path.includes("pretool"));
+    assert.equal(preToolResult.action, "created");
+    assert.equal(readFileSync(join(tempDir, ".claude/hooks/skillrepo-pretool-match.mjs"), "utf-8"), PRETOOL_CONTENT);
+  });
+
+  it("merges PreToolUse hook into settings.local.json", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+    mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT, PRETOOL_CONTENT);
+
+    const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
+    assert.ok(Array.isArray(config.hooks.PreToolUse), "PreToolUse should be an array");
+    assert.equal(config.hooks.PreToolUse.length, 1);
+    assert.equal(config.hooks.PreToolUse[0].hooks[0].command, "node .claude/hooks/skillrepo-pretool-match.mjs");
+  });
+
+  it("skips pretool hook when preToolHookContent is not provided", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT);
+
+    const preToolResult = results.find((r) => r.path.includes("pretool"));
+    assert.equal(preToolResult, undefined, "no pretool result when content not provided");
+    assert.equal(existsSync(join(tempDir, ".claude/hooks/skillrepo-pretool-match.mjs")), false);
+  });
+
+  it("does not duplicate PreToolUse when already installed", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+
+    // First install
+    mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT, PRETOOL_CONTENT);
+    // Second install
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT, PRETOOL_CONTENT);
+
+    const settingsResult = results.find((r) => r.path.includes("settings.local.json"));
+    assert.equal(settingsResult.action, "skipped");
+
+    const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
+    assert.equal(config.hooks.PreToolUse.length, 1, "should not duplicate PreToolUse entry");
+  });
 });