claude-recall 0.9.1 → 0.9.2

package/dist/cli/claude-recall-cli.js

@@ -565,73 +565,82 @@ async function main() {
             }
         }
     }
-    // Install hooks and skills to current project
-    function installHooksAndSkills(force = false) {
+    // Install skills and clean up old hooks (v0.9.0+ uses Skills, not hooks)
+    function installSkillsAndCleanupHooks(force = false) {
         const cwd = process.cwd();
         const projectName = path.basename(cwd);
-        console.log('\n📦 Installing Claude Recall hooks and skills...\n');
+        console.log('\n📦 Claude Recall v0.9.x Setup\n');
         console.log(`📍 Project: ${projectName}`);
         console.log(`📍 Directory: ${cwd}\n`);
         // Find the package directory (where claude-recall is installed)
-        // When run via npx, __dirname points to dist/cli, so we go up to find .claude
         const packageDir = path.resolve(__dirname, '../..');
-        const packageHooksDir = path.join(packageDir, '.claude/hooks');
         const packageSkillsDir = path.join(packageDir, '.claude/skills');
         const claudeDir = path.join(cwd, '.claude');
         const hooksDir = path.join(claudeDir, 'hooks');
-        // Create .claude/hooks directory
-        if (!fs.existsSync(hooksDir)) {
-            fs.mkdirSync(hooksDir, { recursive: true });
-        }
-        // Copy single enforcement hook
-        const hookSource = path.join(packageHooksDir, 'memory_enforcer.py');
-        const hookDest = path.join(hooksDir, 'memory_enforcer.py');
-        if (fs.existsSync(hookSource)) {
-            fs.copyFileSync(hookSource, hookDest);
-            fs.chmodSync(hookDest, 0o755);
-            console.log('✅ Installed memory_enforcer.py to .claude/hooks/');
-        }
-        else {
-            console.log(`⚠️  Hook not found at: ${packageHooksDir}`);
-        }
-        // Create or update .claude/settings.json with hook configuration
         const settingsPath = path.join(claudeDir, 'settings.json');
+        // === CLEANUP: Remove old hooks (v0.9.0+ doesn't use hooks) ===
+        if (fs.existsSync(hooksDir)) {
+            const oldHooks = [
+                'memory_enforcer.py',
+                'pre_tool_search_enforcer.py',
+                'mcp_tool_tracker.py',
+                'pubnub_pre_tool_hook.py',
+                'pubnub_prompt_hook.py',
+                'user_prompt_capture.py',
+                'user_prompt_reminder.py'
+            ];
+            let removedCount = 0;
+            for (const hook of oldHooks) {
+                const hookPath = path.join(hooksDir, hook);
+                if (fs.existsSync(hookPath)) {
+                    fs.unlinkSync(hookPath);
+                    removedCount++;
+                }
+            }
+            // Remove hooks directory if empty
+            try {
+                const remaining = fs.readdirSync(hooksDir);
+                if (remaining.length === 0) {
+                    fs.rmdirSync(hooksDir);
+                }
+            }
+            catch (e) {
+                // Ignore
+            }
+            if (removedCount > 0) {
+                console.log(`🧹 Removed ${removedCount} old hook file(s) from .claude/hooks/`);
+            }
+        }
+        // === CLEANUP: Clear hook configuration from settings.json ===
         let settings = {};
         if (fs.existsSync(settingsPath)) {
             try {
                 settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
+                const hadHooks = settings.hooks && Object.keys(settings.hooks).length > 0;
+                // Clear hooks - v0.9.0+ uses Skills instead
+                settings.hooks = {};
+                settings.hooksVersion = '2.0.0';
+                fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
+                if (hadHooks) {
+                    console.log('🧹 Cleared old hook configuration from .claude/settings.json');
+                }
             }
             catch (e) {
-                settings = {};
-            }
-        }
-        const CURRENT_HOOKS_VERSION = '1.0.0';
-        const needsUpdate = force || !settings.hooks || settings.hooksVersion !== CURRENT_HOOKS_VERSION;
-        if (needsUpdate) {
-            settings.hooksVersion = CURRENT_HOOKS_VERSION;
-            settings.hooks = {
-                PreToolUse: [
-                    {
-                        matcher: "mcp__claude-recall__.*|Write|Edit|Bash|Task",
-                        hooks: [
-                            {
-                                type: "command",
-                                command: `python3 ${path.join(hooksDir, 'memory_enforcer.py')}`,
-                            }
-                        ]
-                    }
-                ]
-            };
-            fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
-            console.log('✅ Configured hook in .claude/settings.json');
-            if (force) {
-                console.log('   → Force flag: overwrote existing configuration');
+                // If settings.json is invalid, create fresh one
+                if (!fs.existsSync(claudeDir)) {
+                    fs.mkdirSync(claudeDir, { recursive: true });
+                }
+                fs.writeFileSync(settingsPath, JSON.stringify({ hooks: {}, hooksVersion: '2.0.0' }, null, 2));
             }
         }
         else {
-            console.log(`ℹ️  Hooks already at version ${CURRENT_HOOKS_VERSION}`);
+            // Create new settings.json without hooks
+            if (!fs.existsSync(claudeDir)) {
+                fs.mkdirSync(claudeDir, { recursive: true });
+            }
+            fs.writeFileSync(settingsPath, JSON.stringify({ hooks: {}, hooksVersion: '2.0.0' }, null, 2));
         }
-        // Copy skills directory
+        // === INSTALL: Copy skills directory ===
         if (fs.existsSync(packageSkillsDir)) {
             const skillsDir = path.join(claudeDir, 'skills');
             copyDirRecursive(packageSkillsDir, skillsDir);
@@ -640,18 +649,19 @@ async function main() {
         else {
             console.log(`⚠️  Skills not found at: ${packageSkillsDir}`);
         }
-        console.log('\n✅ Installation complete!\n');
+        console.log('\n✅ Setup complete!\n');
+        console.log('ℹ️  v0.9.0+ uses native Claude Skills instead of hooks.');
         console.log('Restart Claude Code to activate.\n');
     }
-    // Setup command - shows activation instructions or installs hooks/skills
+    // Setup command - shows activation instructions or installs skills
     program
         .command('setup')
-        .description('Show activation instructions or install hooks/skills')
-        .option('--install', 'Install hooks and skills to current project')
+        .description('Show activation instructions or install skills')
+        .option('--install', 'Install skills and clean up old hooks')
         .action((options) => {
         if (options.install) {
-            // Install hooks and skills to current project
-            installHooksAndSkills();
+            // Install skills and clean up old hooks
+            installSkillsAndCleanupHooks();
         }
         else {
             // Show activation instructions
@@ -676,13 +686,13 @@ async function main() {
         }
         process.exit(0);
     });
-    // Repair command - simple alias for setup --install
+    // Repair command - cleans up old hooks and installs skills
     program
         .command('repair')
-        .description('Repair broken or missing hooks and skills')
-        .option('--force', 'Force overwrite existing hook configuration')
+        .description('Clean up old hooks and install skills (v0.9.0+ migration)')
+        .option('--force', 'Force overwrite existing configuration')
         .action((options) => {
-        installHooksAndSkills(options.force || false);
+        installSkillsAndCleanupHooks(options.force || false);
         process.exit(0);
     });
     // Check hooks function

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-recall",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Persistent memory for Claude Code with native Skills integration, automatic capture, failure learning, and project scoping via MCP server",
   "main": "dist/index.js",
   "bin": {

package/scripts/postinstall.js

@@ -4,7 +4,7 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 
-console.log('\n🚀 Setting up Claude Recall...\n');
+console.log('\n🚀 Setting up Claude Recall v0.9.x...\n');
 
 const { execSync } = require('child_process');
 
@@ -28,6 +28,22 @@ function copyDirRecursive(src, dest) {
   }
 }
 
+// Helper function to remove directory recursively
+function rmDirRecursive(dir) {
+  if (fs.existsSync(dir)) {
+    const entries = fs.readdirSync(dir, { withFileTypes: true });
+    for (const entry of entries) {
+      const fullPath = path.join(dir, entry.name);
+      if (entry.isDirectory()) {
+        rmDirRecursive(fullPath);
+      } else {
+        fs.unlinkSync(fullPath);
+      }
+    }
+    fs.rmdirSync(dir);
+  }
+}
+
 try {
   // Set up database location in user's home directory
   const dbDir = path.join(os.homedir(), '.claude-recall');
@@ -88,78 +104,90 @@ try {
     console.log('⚠️  Failed to register project (non-fatal):', error.message);
   }
 
-  // Install hook and skill to .claude/ directory
+  // Install skills and clean up old hooks (v0.9.0+ uses Skills, not hooks)
   try {
     const cwd = process.cwd();
     const projectName = path.basename(cwd);
-    const packageHooksDir = path.join(__dirname, '../.claude/hooks');
     const packageSkillsDir = path.join(__dirname, '../.claude/skills');
 
     if (projectName !== 'claude-recall' && !cwd.includes('node_modules/.pnpm') && !cwd.includes('node_modules/claude-recall')) {
       const claudeDir = path.join(cwd, '.claude');
       const hooksDir = path.join(claudeDir, 'hooks');
+      const settingsPath = path.join(claudeDir, 'settings.json');
 
-      // Create .claude/hooks directory
-      if (!fs.existsSync(hooksDir)) {
-        fs.mkdirSync(hooksDir, { recursive: true });
-      }
-
-      // Copy single enforcement hook
-      const hookSource = path.join(packageHooksDir, 'memory_enforcer.py');
-      const hookDest = path.join(hooksDir, 'memory_enforcer.py');
+      // === CLEANUP: Remove old hooks (v0.9.0+ doesn't use hooks) ===
+      if (fs.existsSync(hooksDir)) {
+        // Remove known hook files from previous versions
+        const oldHooks = [
+          'memory_enforcer.py',
+          'pre_tool_search_enforcer.py',
+          'mcp_tool_tracker.py',
+          'pubnub_pre_tool_hook.py',
+          'pubnub_prompt_hook.py',
+          'user_prompt_capture.py',
+          'user_prompt_reminder.py'
+        ];
+
+        let removedCount = 0;
+        for (const hook of oldHooks) {
+          const hookPath = path.join(hooksDir, hook);
+          if (fs.existsSync(hookPath)) {
+            fs.unlinkSync(hookPath);
+            removedCount++;
+          }
+        }
 
-      if (fs.existsSync(hookSource)) {
-        fs.copyFileSync(hookSource, hookDest);
-        fs.chmodSync(hookDest, 0o755);
-        console.log('✅ Installed memory_enforcer.py to .claude/hooks/');
-      }
+        // Remove hooks directory if empty
+        try {
+          const remaining = fs.readdirSync(hooksDir);
+          if (remaining.length === 0) {
+            fs.rmdirSync(hooksDir);
+          }
+        } catch (e) {
+          // Ignore
+        }
 
-      // Copy skills directory
-      if (fs.existsSync(packageSkillsDir)) {
-        const skillsDir = path.join(claudeDir, 'skills');
-        copyDirRecursive(packageSkillsDir, skillsDir);
-        console.log('✅ Installed SKILL.md to .claude/skills/');
+        if (removedCount > 0) {
+          console.log(`🧹 Removed ${removedCount} old hook file(s) from .claude/hooks/`);
+        }
       }
 
-      // Create .claude/settings.json with hook configuration
-      const settingsPath = path.join(claudeDir, 'settings.json');
-      let settings = {};
-
+      // === CLEANUP: Clear hook configuration from settings.json ===
       if (fs.existsSync(settingsPath)) {
         try {
-          settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
+          let settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
+          const hadHooks = settings.hooks && Object.keys(settings.hooks).length > 0;
+
+          // Clear hooks - v0.9.0+ uses Skills instead
+          settings.hooks = {};
+          settings.hooksVersion = '2.0.0'; // Bump version to indicate skills-based
+
+          fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
+
+          if (hadHooks) {
+            console.log('🧹 Cleared old hook configuration from .claude/settings.json');
+          }
         } catch (e) {
-          settings = {};
+          // If settings.json is invalid, create fresh one
+          fs.writeFileSync(settingsPath, JSON.stringify({ hooks: {}, hooksVersion: '2.0.0' }, null, 2));
         }
+      } else {
+        // Create new settings.json without hooks
+        if (!fs.existsSync(claudeDir)) {
+          fs.mkdirSync(claudeDir, { recursive: true });
+        }
+        fs.writeFileSync(settingsPath, JSON.stringify({ hooks: {}, hooksVersion: '2.0.0' }, null, 2));
       }
 
-      const CURRENT_HOOKS_VERSION = '1.0.0';
-      const needsUpdate = !settings.hooks || settings.hooksVersion !== CURRENT_HOOKS_VERSION;
-
-      if (needsUpdate) {
-        settings.hooksVersion = CURRENT_HOOKS_VERSION;
-        settings.hooks = {
-          PreToolUse: [
-            {
-              matcher: "mcp__claude-recall__.*|Write|Edit|Bash|Task",
-              hooks: [
-                {
-                  type: "command",
-                  command: `python3 ${path.join(hooksDir, 'memory_enforcer.py')}`
-                }
-              ]
-            }
-          ]
-        };
-
-        fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
-        console.log('✅ Configured hook in .claude/settings.json');
-      } else {
-        console.log(`ℹ️  Hooks already at version ${CURRENT_HOOKS_VERSION}`);
+      // === INSTALL: Copy skills directory ===
+      if (fs.existsSync(packageSkillsDir)) {
+        const skillsDir = path.join(claudeDir, 'skills');
+        copyDirRecursive(packageSkillsDir, skillsDir);
+        console.log('✅ Installed SKILL.md to .claude/skills/');
       }
     }
   } catch (error) {
-    console.log('⚠️  Failed to install hooks/skills (non-fatal):', error.message);
+    console.log('⚠️  Failed to install skills (non-fatal):', error.message);
   }
 
   console.log('\n✅ Installation complete!\n');
@@ -173,6 +201,7 @@ try {
   console.log('');
   console.log('━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━');
   console.log('');
+  console.log('ℹ️  v0.9.0+ uses native Claude Skills instead of hooks.');
   console.log('💡 Your memories persist across conversations and restarts.\n');
 
 } catch (error) {