sticky-note-cli 3.1.11 → 3.1.12

package/bin/cli.js

@@ -95,9 +95,8 @@ function resolveHookPaths(_obj, _rootDir) {
   return;
 }
 
-// Detects hook commands in an existing settings.json that point at an absolute
-// path (likely written by an older sticky-note-cli on another developer's
-// machine). Returns the list of stale command strings, or [] if all clean.
+// Detects hook commands in an existing settings.json that are broken or stale.
+// Returns the list of stale command strings, or [] if all clean.
 function findStaleHookPaths(settings) {
   const stale = [];
   const hooks = (settings && settings.hooks) || {};
@@ -107,9 +106,6 @@ function findStaleHookPaths(settings) {
         const cmd = h && h.command;
         if (typeof cmd !== "string") continue;
         // Absolute paths (Unix /Users, /home, /root or Windows C:/, D:/ etc.)
-        // inside the hook command are the smoking gun. Match anywhere in the
-        // string — the path may be preceded by other tokens (env vars, shell
-        // prefixes) and may contain spaces.
         if (
           /["'][A-Za-z]:[\\/]/.test(cmd) ||
           /["']\/(?:Users|home|root|tmp)\//.test(cmd) ||
@@ -117,12 +113,9 @@ function findStaleHookPaths(settings) {
         ) {
           stale.push(cmd);
         }
-        // Old relative-path format: node ".claude/hooks/foo.js"
-        // This fails when Claude Code launches from a subdirectory of the
-        // project root, because the path resolves relative to the launch cwd,
-        // not relative to where .claude/settings.json lives. The fix is to
-        // use `npx sticky-note run-hook foo` which walks up to find the hooks.
-        if (/node\s+["']\.claude[\\/]hooks[\\/][^"']+\.js["']/.test(cmd)) {
+        // Broken `npx sticky-note run-hook X` — npm has no package named "sticky-note"
+        // (the package is "sticky-note-cli"). Written by earlier 3.x releases.
+        if (/^npx\s+sticky-note\s+run-hook\s+\S+/.test(cmd)) {
           stale.push(cmd);
         }
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sticky-note-cli",
-  "version": "3.1.11",
+  "version": "3.1.12",
   "description": "Human-to-human handoff for AI coding assistants. Git-backed shared memory for Claude Code and Copilot CLI.",
   "license": "MIT",
   "bin": {

package/templates/hooks/sticky-utils.js

@@ -157,13 +157,13 @@ function selfHealHookPaths(settingsPath) {
     try { settings = JSON.parse(raw); } catch (_) { return result; }
     if (!settings || !settings.hooks) return result;
 
-    // Two stale patterns:
+    // Stale patterns that need rewriting to the canonical template format:
     // 1. Absolute paths (written by older sticky-note-cli or copied from another machine)
-    // 2. Old relative form `node ".claude/hooks/foo.js"` — fails when Claude Code
-    //    launches from a subdirectory, because the path resolves relative to the
-    //    launch cwd, not the directory containing .claude/settings.json.
+    // 2. `npx sticky-note run-hook X` — broken: npm has no package named "sticky-note"
+    //    (the package is "sticky-note-cli"). Written by earlier 3.x releases via the
+    //    now-removed selfHeal rewrite. Canonical form is `node ".claude/hooks/X.js"`.
     const STALE_ABSOLUTE = /["'][A-Za-z]:[\\/]|["']\/(?:Users|home|root|tmp)\/|\$\(git rev-parse/;
-    const STALE_RELATIVE = /^node\s+["']\.claude[\\/]hooks[\\/][^"']+\.js["']\s*$/;
+    const STALE_NPX = /^npx\s+sticky-note\s+run-hook\s+(\S+)/;
     // Match the script basename in either quoted or bare form.
     const BASENAME = /([A-Za-z0-9_.-]+)\.(?:js|sh|cjs|mjs)(?:["'`\s]|$)/;
 
@@ -174,12 +174,17 @@ function selfHealHookPaths(settingsPath) {
         for (const h of (entry && entry.hooks) || []) {
           const cmd = h && h.command;
           if (typeof cmd !== "string") continue;
-          const isStale = STALE_ABSOLUTE.test(cmd) || STALE_RELATIVE.test(cmd);
-          if (!isStale) continue;
-          const m = cmd.match(BASENAME);
-          if (!m) continue;
-          const basename = m[1];
-          const newCmd = `npx sticky-note run-hook ${basename}`;
+          let basename = null;
+          // Check for broken `npx sticky-note run-hook X` pattern first
+          const npxMatch = cmd.match(STALE_NPX);
+          if (npxMatch) {
+            basename = npxMatch[1];
+          } else if (STALE_ABSOLUTE.test(cmd)) {
+            const m = cmd.match(BASENAME);
+            if (m) basename = m[1];
+          }
+          if (!basename) continue;
+          const newCmd = `node ".claude/hooks/${basename}.js"`;
           result.push({ event, oldCommand: cmd, newCommand: newCmd });
           h.command = newCmd;
           changed = true;
@@ -192,9 +197,9 @@ function selfHealHookPaths(settingsPath) {
     fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + "\n", "utf-8");
     try {
       const summary =
-        `[STICKY-NOTE] ⚠️  Detected ${result.length} stale hook path(s) in ` +
+        `[STICKY-NOTE] ⚠️  Detected ${result.length} stale hook command(s) in ` +
         `${path.relative(process.cwd(), settingsPath) || settingsPath} — rewrote to ` +
-        `"npx sticky-note run-hook <name>" so hooks work from any subdirectory. ` +
+        `"node \\".claude/hooks/<name>.js\\"" (canonical form). ` +
         `Commit the change to share the fix with your team.`;
       process.stderr.write(summary + "\n");
       for (const { event, oldCommand, newCommand } of result.slice(0, 5)) {