claude-recall 0.8.6 → 0.8.8

package/.claude/hooks/pre_tool_search_enforcer.py

@@ -47,8 +47,8 @@ def should_require_search(tool_name: str, tool_input: Dict[str, Any]) -> bool:
 def check_recent_search(session_id: str) -> bool:
     """Check if memory search was performed recently in this session."""
     if not session_id:
-        # No session ID, be permissive
-        return True
+        # No session ID - strict mode: block execution
+        return False
 
     try:
         # Call claude-recall CLI to check recent tool usage
@@ -60,15 +60,15 @@ def check_recent_search(session_id: str) -> bool:
         )
 
         if result.returncode != 0:
-            # Command failed, be permissive (don't block)
-            return True
+            # Command failed - strict mode: block execution
+            return False
 
         # Check if mcp__claude-recall__search was called recently
         return 'mcp__claude-recall__search' in result.stdout
 
     except (subprocess.TimeoutExpired, FileNotFoundError, Exception):
-        # If check fails, be permissive (don't block)
-        return True
+        # Check failed - strict mode: block execution
+        return False
 
 
 def generate_search_query(tool_name: str, tool_input: Dict[str, Any]) -> str:

package/.claude/settings.json

@@ -4,6 +4,10 @@
       {
         "matcher": "Write|Edit",
         "hooks": [
+          {
+            "type": "command",
+            "command": "python3 .claude/hooks/pre_tool_search_enforcer.py"
+          },
           {
             "type": "command",
             "command": "python3 .claude/hooks/pubnub_pre_tool_hook.py"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-recall",
-  "version": "0.8.6",
+  "version": "0.8.8",
   "description": "Persistent memory for Claude Code with fire-and-forget PubNub architecture, automatic capture, failure learning, and project scoping via MCP server",
   "main": "dist/index.js",
   "bin": {

package/scripts/postinstall.js

@@ -6,50 +6,39 @@ const os = require('os');
 
 console.log('\n🚀 Setting up Claude Recall...\n');
 
-try {
-  // Configure MCP server in ~/.claude.json
-  const claudeConfigPath = path.join(os.homedir(), '.claude.json');
-  
-  // Read or create claude config
-  let config = { mcpServers: {} };
-  if (fs.existsSync(claudeConfigPath)) {
-    const configContent = fs.readFileSync(claudeConfigPath, 'utf8');
-    config = JSON.parse(configContent);
-    if (!config.mcpServers) {
-      config.mcpServers = {};
-    }
-  }
-
-  // Check if claude-recall is already configured
-  if (config.mcpServers['claude-recall']) {
-    console.log('⚠️  Claude Recall is already configured in ~/.claude.json');
-    console.log('   Updating configuration...');
-  }
+const { execSync } = require('child_process');
 
+try {
   // Set up database location in user's home directory
   const dbDir = path.join(os.homedir(), '.claude-recall');
-  
+
   // Create directory if it doesn't exist
   if (!fs.existsSync(dbDir)) {
     fs.mkdirSync(dbDir, { recursive: true });
     console.log(`📁 Created database directory: ${dbDir}`);
   }
 
-  // Add or update claude-recall configuration
-  // Remove env variables since we're hardcoding the path to ~/.claude-recall/claude-recall.db
-  config.mcpServers['claude-recall'] = {
-    type: 'stdio',
-    command: 'npx',
-    args: ['claude-recall', 'mcp', 'start']
-  };
-
-  // Write back the config with proper formatting
-  fs.writeFileSync(claudeConfigPath, JSON.stringify(config, null, 2));
+  // Register MCP server using official Claude CLI
+  try {
+    // Remove existing registration first (in case of update)
+    try {
+      execSync('claude mcp remove claude-recall', { stdio: 'ignore' });
+    } catch (e) {
+      // Ignore if not registered
+    }
 
-  console.log('✅ Successfully configured Claude Recall in ~/.claude.json');
+    // Register using official CLI
+    execSync('claude mcp add claude-recall -- npx claude-recall mcp start', {
+      stdio: 'inherit'
+    });
+    console.log('✅ Registered Claude Recall MCP server');
+  } catch (mcpError) {
+    console.log('⚠️  Could not auto-register MCP server.');
+    console.log('   Please run manually:');
+    console.log('   claude mcp add claude-recall -- npx claude-recall mcp start');
+  }
   
   // Update project CLAUDE.md with minimal instructions
-  const { execSync } = require('child_process');
   try {
     execSync('node ' + path.join(__dirname, 'postinstall-claude-md.js'), { stdio: 'inherit' });
   } catch (error) {
@@ -121,6 +110,7 @@ try {
       // Copy hook scripts from package
       const packageHooksDir = path.join(__dirname, '../.claude/hooks');
       const hookScripts = [
+        'pre_tool_search_enforcer.py',
         'pubnub_pre_tool_hook.py',
         'pubnub_prompt_hook.py'
       ];
@@ -152,8 +142,12 @@ try {
         settings.hooks = {
           PreToolUse: [
             {
-              matcher: "Write|Edit|Bash",
+              matcher: "Write|Edit",
               hooks: [
+                {
+                  type: "command",
+                  command: "python3 .claude/hooks/pre_tool_search_enforcer.py"
+                },
                 {
                   type: "command",
                   command: "python3 .claude/hooks/pubnub_pre_tool_hook.py"
@@ -174,9 +168,9 @@ try {
         };
 
         fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2));
-        console.log('✅ Configured PubNub hooks in .claude/settings.json');
-        console.log('   → PreToolUse: Publishes tool events to memory agent');
-        console.log('   → UserPromptSubmit: Publishes prompts for preference extraction');
+        console.log('✅ Configured hooks in .claude/settings.json');
+        console.log('   → PreToolUse: Enforces memory search before Write/Edit');
+        console.log('   → UserPromptSubmit: Captures prompts for preference extraction');
       }
     }
   } catch (error) {