skillrepo 1.6.0 → 1.6.2

package/package.json

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

package/src/lib/fs-utils.mjs

@@ -3,7 +3,7 @@
  * Creates directories as needed, handles errors cleanly.
  */
 
-import { readFileSync, writeFileSync, existsSync, mkdirSync, statSync } from "node:fs";
+import { readFileSync, writeFileSync, existsSync, mkdirSync, statSync, chmodSync } from "node:fs";
 import { dirname } from "node:path";
 
 /**
@@ -35,6 +35,15 @@ export function writeFileSafe(filePath, content) {
   writeFileSync(filePath, content, "utf-8");
 }
 
+/**
+ * Write a file and mark it executable (0o755).
+ * Used for the Cursor session hook which is invoked directly via shebang.
+ */
+export function writeExecutable(filePath, content) {
+  writeFileSafe(filePath, content);
+  chmodSync(filePath, 0o755);
+}
+
 /**
  * Check if a path exists (file or directory).
  */

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

@@ -12,40 +12,57 @@ import { readFileSafe, writeFileSafe } from "../fs-utils.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";
+// The three hook script paths (relative to project root).
+const HOOK_SCRIPTS = {
+  sync: ".claude/hooks/skillrepo-sync.mjs",
+  prompt: ".claude/hooks/skillrepo-prompt-match.mjs",
+  pretool: ".claude/hooks/skillrepo-pretool-match.mjs",
+};
 
 /**
- * Extract a hook command from a hooks config group, falling back to a default.
- * @param {object} hooksConfig - The hooks config object from the server payload
- * @param {string} eventName - e.g. "SessionStart" or "UserPromptSubmit"
- * @param {string} fallback - default command if extraction fails
- * @returns {string}
+ * Build the full command string for a hook script.
+ * Uses process.execPath (the absolute path to the current Node binary)
+ * so hooks work on nvm/fnm/mise systems where `node` is not on /bin/sh's PATH.
  */
-function extractCommand(hooksConfig, eventName, fallback) {
-  try {
-    return hooksConfig[eventName][0].hooks[0].command || fallback;
-  } catch {
-    return fallback;
-  }
+function buildCommand(scriptPath) {
+  return `"${process.execPath}" ${scriptPath}`;
 }
 
 /**
- * Check if a command is already installed in a hook group array.
- * @param {Array} groups
- * @param {string} command
- * @returns {boolean}
+ * Check if a SkillRepo hook for the given script path is already installed,
+ * regardless of how node was referenced (bare `node`, shebang, or full path).
  */
-function hasCommand(groups, command) {
+function hasSkillRepoHook(groups, scriptPath) {
   if (!Array.isArray(groups)) return false;
   return groups.some(
     (group) =>
       Array.isArray(group.hooks) &&
-      group.hooks.some((h) => h.command === command)
+      group.hooks.some((h) => h.command?.includes(scriptPath))
   );
 }
 
+/**
+ * Replace any existing SkillRepo hook command with the current node path.
+ * Handles all legacy formats: `node .claude/hooks/...`, `.claude/hooks/...`,
+ * or `/old/path/to/node .claude/hooks/...`.
+ * Returns true if any replacement was made.
+ */
+function updateExistingCommands(groups, scriptPath) {
+  if (!Array.isArray(groups)) return false;
+  const newCommand = buildCommand(scriptPath);
+  let updated = false;
+  for (const group of groups) {
+    if (!Array.isArray(group.hooks)) continue;
+    for (const hook of group.hooks) {
+      if (hook.command?.includes(scriptPath) && hook.command !== newCommand) {
+        hook.command = newCommand;
+        updated = true;
+      }
+    }
+  }
+  return updated;
+}
+
 /**
  * Merge SkillRepo hooks into .claude/settings.local.json and write hook scripts.
  * @param {object} hooksConfig - The hooks config object from the server payload (settingsHooks.hooks)
@@ -54,24 +71,24 @@ function hasCommand(groups, command) {
  * @param {string} [preToolHookContent] - The pretool-match hook script content (optional for backward compat)
  * @returns {{ results: { path: string; action: string }[] }}
  */
+// NOTE: The hooksConfig parameter is accepted for backward compatibility but
+// its command strings are not used. All commands are built locally using
+// process.execPath to ensure hooks work on nvm/fnm/mise systems.
 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)
+  // Always write hook scripts (latest version from server).
   const syncExisted = readFileSafe(claudeSyncHook()) !== null;
   writeFileSafe(claudeSyncHook(), syncHookContent);
   results.push({
-    path: ".claude/hooks/skillrepo-sync.mjs",
+    path: HOOK_SCRIPTS.sync,
     action: syncExisted ? "updated" : "created",
   });
 
   const promptExisted = readFileSafe(claudePromptHook()) !== null;
   writeFileSafe(claudePromptHook(), promptHookContent);
   results.push({
-    path: ".claude/hooks/skillrepo-prompt-match.mjs",
+    path: HOOK_SCRIPTS.prompt,
     action: promptExisted ? "updated" : "created",
   });
 
@@ -79,7 +96,7 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
     const preToolExisted = readFileSafe(claudePreToolHook()) !== null;
     writeFileSafe(claudePreToolHook(), preToolHookContent);
     results.push({
-      path: ".claude/hooks/skillrepo-pretool-match.mjs",
+      path: HOOK_SCRIPTS.pretool,
       action: preToolExisted ? "updated" : "created",
     });
   }
@@ -111,9 +128,28 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
   if (!config.hooks) config.hooks = {};
   let changed = false;
 
+  // Build commands using the absolute node path from this machine.
+  // This ensures hooks work on nvm/fnm/mise systems where `node` is
+  // not on /bin/sh's PATH.
+  const syncCommand = buildCommand(HOOK_SCRIPTS.sync);
+  const promptCommand = buildCommand(HOOK_SCRIPTS.prompt);
+  const preToolCommand = buildCommand(HOOK_SCRIPTS.pretool);
+
+  // Update existing SkillRepo hook commands to use the current node path
+  // (handles all legacy formats: bare `node`, shebang paths, old full paths)
+  for (const [event, scriptPath] of [
+    ["SessionStart", HOOK_SCRIPTS.sync],
+    ["UserPromptSubmit", HOOK_SCRIPTS.prompt],
+    ["PreToolUse", HOOK_SCRIPTS.pretool],
+  ]) {
+    if (Array.isArray(config.hooks[event]) && updateExistingCommands(config.hooks[event], scriptPath)) {
+      changed = true;
+    }
+  }
+
   // Merge SessionStart
   if (!Array.isArray(config.hooks.SessionStart)) config.hooks.SessionStart = [];
-  if (!hasCommand(config.hooks.SessionStart, syncCommand)) {
+  if (!hasSkillRepoHook(config.hooks.SessionStart, HOOK_SCRIPTS.sync)) {
     config.hooks.SessionStart.push({
       matcher: "startup|resume",
       hooks: [{ type: "command", command: syncCommand }],
@@ -123,7 +159,7 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
 
   // Merge UserPromptSubmit
   if (!Array.isArray(config.hooks.UserPromptSubmit)) config.hooks.UserPromptSubmit = [];
-  if (!hasCommand(config.hooks.UserPromptSubmit, promptCommand)) {
+  if (!hasSkillRepoHook(config.hooks.UserPromptSubmit, HOOK_SCRIPTS.prompt)) {
     config.hooks.UserPromptSubmit.push({
       hooks: [{ type: "command", command: promptCommand }],
     });
@@ -133,7 +169,7 @@ export function mergeHooksConfig(hooksConfig, syncHookContent, promptHookContent
   // Merge PreToolUse
   if (preToolHookContent) {
     if (!Array.isArray(config.hooks.PreToolUse)) config.hooks.PreToolUse = [];
-    if (!hasCommand(config.hooks.PreToolUse, preToolCommand)) {
+    if (!hasSkillRepoHook(config.hooks.PreToolUse, HOOK_SCRIPTS.pretool)) {
       config.hooks.PreToolUse.push({
         hooks: [{ type: "command", command: preToolCommand }],
       });

package/src/lib/write-configs.mjs

@@ -4,7 +4,7 @@
  */
 
 import { resolve } from "node:path";
-import { readFileSafe, writeFileSafe } from "./fs-utils.mjs";
+import { readFileSafe, writeFileSafe, writeExecutable } from "./fs-utils.mjs";
 import { claudeSkillrepoMd, claudeSkillrepoIndex, claudeSkillrepoConfig } from "./paths.mjs";
 
 /**
@@ -95,7 +95,7 @@ export function writeAllConfigs({ ides, mcpUrl, apiKey, payload }) {
     }
     if (payload.cursor?.sessionHook) {
       const existed = readFileSafe(payload.cursor.sessionHook.path) !== null;
-      writeFileSafe(payload.cursor.sessionHook.path, payload.cursor.sessionHook.content);
+      writeExecutable(payload.cursor.sessionHook.path, payload.cursor.sessionHook.content);
       results.push({ path: payload.cursor.sessionHook.path, action: existed ? "updated" : "created" });
     }
     if (payload.cursor?.skillIndex) {

package/src/test/e2e/payload-factory.mjs

@@ -473,12 +473,13 @@ import { tmpdir } from "os";
 
 const CONTENT_URL = "${contentUrl}";
 
-let hookConfig = { maxTotalBytes: 10000, reinjectionLineThreshold: 50 };
+let hookConfig = { maxTotalBytes: 10000, reinjectionLineThreshold: 50, maxSkillsPerPrompt: 3 };
 try {
   const cfg = JSON.parse(readFileSync(join(process.cwd(), ".claude", "skillrepo-config.json"), "utf-8"));
   if (cfg.hookInjection) hookConfig = { ...hookConfig, ...cfg.hookInjection };
 } catch {}
 const MAX_BYTES = hookConfig.maxTotalBytes;
+const MAX_SKILLS = hookConfig.maxSkillsPerPrompt;
 const REINJECT_THRESHOLD = hookConfig.reinjectionLineThreshold ?? 50;
 
 let dynamicSignals = [];
@@ -489,13 +490,8 @@ try {
   }
 } catch {}
 
-const FALLBACK_SIGNALS = [
-  { toolName: "skill__affaan-m__frontend-patterns", files: ["**/*.tsx", "**/*.jsx", "**/*.css"], project: [], tasks: [] },
-  { toolName: "skill__affaan-m__e2e-testing", files: ["**/*.test.*", "**/*.spec.*"], project: [], tasks: [] },
-  { toolName: "skill__atxpace__gh-issue-workflow", files: [], project: [], tasks: ["github issue", "pull request"] },
-  { toolName: "skill__affaan-m__api-routes", files: ["**/api/**/route.ts", "**/api/**/route.tsx"], project: [], tasks: [] },
-];
-const SIGNALS = dynamicSignals.length > 0 ? dynamicSignals : FALLBACK_SIGNALS;
+const SIGNALS = dynamicSignals;
+if (SIGNALS.length === 0) { process.stdout.write("{}"); process.exit(0); }
 
 // Minimal glob matcher — self-contained, no npm deps.
 function matchGlob(filePath, pattern) {
@@ -621,7 +617,7 @@ if (matchedSkills.length === 0) { process.stdout.write("{}"); process.exit(0); }
 
 const { path: statePath, state } = readState(sessionId);
 const currentLines = countTranscriptLines(transcriptPath);
-const candidates = matchedSkills.filter((s) => shouldReinject(s.toolName, state, currentLines));
+const candidates = matchedSkills.filter((s) => shouldReinject(s.toolName, state, currentLines)).slice(0, MAX_SKILLS);
 if (candidates.length === 0) { process.stdout.write("{}"); process.exit(0); }
 
 const contents = [];
@@ -675,21 +671,21 @@ function buildSettingsHooks() {
       {
         matcher: "startup|resume",
         hooks: [
-          { type: "command", command: "node .claude/hooks/skillrepo-sync.mjs" },
+          { type: "command", command: ".claude/hooks/skillrepo-sync.mjs" },
         ],
       },
     ],
     UserPromptSubmit: [
       {
         hooks: [
-          { type: "command", command: "node .claude/hooks/skillrepo-prompt-match.mjs" },
+          { type: "command", command: ".claude/hooks/skillrepo-prompt-match.mjs" },
         ],
       },
     ],
     PreToolUse: [
       {
         hooks: [
-          { type: "command", command: "node .claude/hooks/skillrepo-pretool-match.mjs" },
+          { type: "command", command: ".claude/hooks/skillrepo-pretool-match.mjs" },
         ],
       },
     ],
@@ -766,7 +762,7 @@ function buildCursorHooksJson() {
     hooks: [
       {
         event: "sessionStart",
-        command: "node .cursor/hooks/skillrepo-session.mjs",
+        command: ".cursor/hooks/skillrepo-session.mjs",
       },
     ],
   }, null, 2) + "\n";

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

@@ -22,10 +22,11 @@ const SYNC_CONTENT = "// sync hook";
 const PROMPT_CONTENT = "// prompt hook";
 const PRETOOL_CONTENT = "// pretool hook";
 
+// Server payload hook commands (bare script paths — the CLI builds full commands locally)
 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" }] }],
+  SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: ".claude/hooks/skillrepo-sync.mjs" }] }],
+  UserPromptSubmit: [{ hooks: [{ type: "command", command: ".claude/hooks/skillrepo-prompt-match.mjs" }] }],
+  PreToolUse: [{ hooks: [{ type: "command", command: ".claude/hooks/skillrepo-pretool-match.mjs" }] }],
 };
 
 describe("Hooks settings.local.json merger", () => {
@@ -80,14 +81,15 @@ describe("Hooks settings.local.json merger", () => {
     assert.equal(config.hooks.UserPromptSubmit.length, 1);
   });
 
-  it("skips merge when hooks are already installed", async () => {
+  it("skips merge when hooks are already installed with current node path", async () => {
     const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
 
-    // Pre-existing with both hooks already
+    // Pre-existing with hooks using the CURRENT node path (already up to date)
+    const nodeCmd = (script) => `"${process.execPath}" ${script}`;
     const existing = {
       hooks: {
-        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" }] }],
+        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: nodeCmd(".claude/hooks/skillrepo-sync.mjs") }] }],
+        UserPromptSubmit: [{ hooks: [{ type: "command", command: nodeCmd(".claude/hooks/skillrepo-prompt-match.mjs") }] }],
       },
     };
     mkdirSync(join(tempDir, ".claude"), { recursive: true });
@@ -105,7 +107,7 @@ describe("Hooks settings.local.json merger", () => {
     // Only SessionStart installed
     const existing = {
       hooks: {
-        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: "node .claude/hooks/skillrepo-sync.mjs" }] }],
+        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: ".claude/hooks/skillrepo-sync.mjs" }] }],
       },
     };
     mkdirSync(join(tempDir, ".claude"), { recursive: true });
@@ -119,21 +121,22 @@ describe("Hooks settings.local.json merger", () => {
     const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
     assert.equal(config.hooks.SessionStart.length, 1);
     assert.equal(config.hooks.UserPromptSubmit.length, 1);
-    assert.equal(config.hooks.UserPromptSubmit[0].hooks[0].command, "node .claude/hooks/skillrepo-prompt-match.mjs");
+    assert.ok(config.hooks.UserPromptSubmit[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-prompt-match.mjs"), "Command should end with script path");
   });
 
   it("always updates hook scripts even when settings merge is skipped", async () => {
     const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
 
-    // Install once with old scripts
+    // Install once with old scripts but current-format commands
+    const nodeCmd = (script) => `"${process.execPath}" ${script}`;
     mkdirSync(join(tempDir, ".claude/hooks"), { recursive: true });
     writeFileSync(join(tempDir, ".claude/hooks/skillrepo-sync.mjs"), "// old sync");
     writeFileSync(join(tempDir, ".claude/hooks/skillrepo-prompt-match.mjs"), "// old prompt");
     mkdirSync(join(tempDir, ".claude"), { recursive: true });
     const existing = {
       hooks: {
-        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" }] }],
+        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: nodeCmd(".claude/hooks/skillrepo-sync.mjs") }] }],
+        UserPromptSubmit: [{ hooks: [{ type: "command", command: nodeCmd(".claude/hooks/skillrepo-prompt-match.mjs") }] }],
       },
     };
     writeFileSync(join(tempDir, ".claude/settings.local.json"), JSON.stringify(existing, null, 2));
@@ -149,7 +152,7 @@ describe("Hooks settings.local.json merger", () => {
     assert.equal(promptResult.action, "updated");
     assert.equal(readFileSync(join(tempDir, ".claude/hooks/skillrepo-prompt-match.mjs"), "utf-8"), "// new prompt");
 
-    // Settings skipped
+    // Settings skipped (commands already use current node path)
     const settingsResult = results.find((r) => r.path.includes("settings.local.json"));
     assert.equal(settingsResult.action, "skipped");
   });
@@ -203,7 +206,7 @@ describe("Hooks settings.local.json merger", () => {
     const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
     assert.ok(Array.isArray(config.hooks.SessionStart), "SessionStart should be an array");
     assert.equal(config.hooks.SessionStart.length, 1);
-    assert.equal(config.hooks.SessionStart[0].hooks[0].command, "node .claude/hooks/skillrepo-sync.mjs");
+    assert.ok(config.hooks.SessionStart[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-sync.mjs"), "Command should end with script path");
   });
 
   it("documents partial write when JSON parse fails (hook scripts written before throw)", async () => {
@@ -244,7 +247,7 @@ describe("Hooks settings.local.json merger", () => {
     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");
+    assert.ok(config.hooks.PreToolUse[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-pretool-match.mjs"), "Command should end with script path");
   });
 
   it("skips pretool hook when preToolHookContent is not provided", async () => {
@@ -270,4 +273,88 @@ describe("Hooks settings.local.json merger", () => {
     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");
   });
+
+  it("migrates any non-current-node-path commands to use process.execPath", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+
+    // Pre-existing settings with bare `node` command format
+    const oldSettings = {
+      hooks: {
+        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" }]
+        }]
+      }
+    };
+    mkdirSync(join(tempDir, ".claude"), { recursive: true });
+    writeFileSync(join(tempDir, ".claude/settings.local.json"), JSON.stringify(oldSettings, null, 2));
+    mkdirSync(join(tempDir, ".claude/hooks"), { recursive: true });
+
+    // Run merger with NEW hook config (HOOKS_CONFIG uses .claude/hooks/... without "node" prefix)
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT, PRETOOL_CONTENT);
+
+    // Settings should show "merged" (not "created" or "skipped")
+    const settingsResult = results.find((r) => r.path.includes("settings.local.json"));
+    assert.equal(settingsResult.action, "merged");
+
+    // Read the result and verify: ONE entry per event, with NEW command format
+    const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
+    assert.equal(config.hooks.SessionStart.length, 1);
+    assert.ok(config.hooks.SessionStart[0].hooks[0].command.includes(process.execPath), "Command should contain full node path");
+    assert.ok(config.hooks.SessionStart[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-sync.mjs"), "Command should end with script path");
+    assert.equal(config.hooks.UserPromptSubmit.length, 1);
+    assert.ok(config.hooks.UserPromptSubmit[0].hooks[0].command.includes(process.execPath), "Command should contain full node path");
+    assert.ok(config.hooks.UserPromptSubmit[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-prompt-match.mjs"), "Command should end with script path");
+    assert.equal(config.hooks.PreToolUse.length, 1);
+    assert.ok(config.hooks.PreToolUse[0].hooks[0].command.includes(process.execPath), "Command should contain full node path");
+    assert.ok(config.hooks.PreToolUse[0].hooks[0].command.endsWith(".claude/hooks/skillrepo-pretool-match.mjs"), "Command should end with script path");
+  });
+
+  it("migrates bare shebang-style commands (v1.6.1 format)", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+
+    // v1.6.1 used bare script paths relying on shebang execution
+    const oldSettings = {
+      hooks: {
+        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: ".claude/hooks/skillrepo-sync.mjs" }] }],
+      }
+    };
+    mkdirSync(join(tempDir, ".claude/hooks"), { recursive: true });
+    writeFileSync(join(tempDir, ".claude/settings.local.json"), JSON.stringify(oldSettings, null, 2));
+
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT);
+
+    const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
+    assert.equal(config.hooks.SessionStart.length, 1, "Should not duplicate");
+    const cmd = config.hooks.SessionStart[0].hooks[0].command;
+    assert.ok(cmd.includes(process.execPath), "Should contain full node path");
+    assert.ok(cmd.endsWith(".claude/hooks/skillrepo-sync.mjs"), "Should end with script path");
+  });
+
+  it("migrates old full-path-to-node commands (nvm version change)", async () => {
+    const { mergeHooksConfig } = await import("../../lib/mergers/hooks-json.mjs");
+
+    // User ran init with an older nvm node version
+    const oldSettings = {
+      hooks: {
+        SessionStart: [{ matcher: "startup|resume", hooks: [{ type: "command", command: "/usr/local/bin/node .claude/hooks/skillrepo-sync.mjs" }] }],
+      }
+    };
+    mkdirSync(join(tempDir, ".claude/hooks"), { recursive: true });
+    writeFileSync(join(tempDir, ".claude/settings.local.json"), JSON.stringify(oldSettings, null, 2));
+
+    const { results } = mergeHooksConfig(HOOKS_CONFIG, SYNC_CONTENT, PROMPT_CONTENT);
+
+    const config = JSON.parse(readFileSync(join(tempDir, ".claude/settings.local.json"), "utf-8"));
+    assert.equal(config.hooks.SessionStart.length, 1, "Should not duplicate");
+    const cmd = config.hooks.SessionStart[0].hooks[0].command;
+    assert.ok(cmd.includes(process.execPath), "Should contain CURRENT node path");
+    assert.ok(!cmd.includes("/usr/local/bin/node"), "Should NOT contain old node path");
+  });
 });