fix(mcp,doctor): improve desire path coverage, daemon auto-start, doctor fixes

- MCP: add is_error: true to 11 soft-fail null-handler return objects
  (cord 5, team 3, provider 3) so desire path hook catches them
- Doctor: fix stale hooks check (was using deprecated hooks.json key,
  now reads settings.json correctly)
- Doctor: add daemon health check with PID/uptime/service count
- Doctor: expand --fix to auto-apply hooks install and daemon start
- Hook: add daemon-auto-start.js — checks PID file on PostToolUse,
  spawns daemon if not running (throttled to once per 30min per session)
- Tests: update hook counts 10→11, optional 5→6

Author: StackMemory Bot (CLI)

scripts/install-claude-hooks-auto.js

@@ -117,6 +117,13 @@ try {
       commandPrefix: 'node',
       required: false,
     },
+    {
+      scriptName: 'daemon-auto-start.js',
+      eventType: 'PostToolUse',
+      timeout: 2,
+      commandPrefix: 'node',
+      required: false,
+    },
   ];
 
   const DEAD_HOOKS = ['sms-response-handler.js'];

src/cli/commands/setup.ts

@@ -22,7 +22,6 @@ import { execSync } from 'child_process';
 const CLAUDE_DIR = join(homedir(), '.claude');
 const CLAUDE_CONFIG_FILE = join(CLAUDE_DIR, 'config.json');
 const MCP_CONFIG_FILE = join(CLAUDE_DIR, 'stackmemory-mcp.json');
-const HOOKS_JSON = join(CLAUDE_DIR, 'hooks.json');
 
 interface DiagnosticResult {
   name: string;
@@ -264,39 +263,41 @@ export function createDoctorCommand(): Command {
         });
       }
 
-      // 4. Check Claude hooks
-      if (existsSync(HOOKS_JSON)) {
-        try {
-          const hooks = JSON.parse(readFileSync(HOOKS_JSON, 'utf8'));
-          const hasTraceHook = !!hooks['tool-use-approval'];
-          if (hasTraceHook) {
-            results.push({
-              name: 'Claude Hooks',
-              status: 'ok',
-              message: 'Tool tracing hook installed',
-            });
-          } else {
-            results.push({
-              name: 'Claude Hooks',
-              status: 'warn',
-              message: 'Hooks file exists but tracing not configured',
-              fix: 'Run: stackmemory hooks install',
-            });
+      // 4. Check Claude hooks (settings.json)
+      {
+        const settingsFile = join(homedir(), '.claude', 'settings.json');
+        let hookCount = 0;
+        if (existsSync(settingsFile)) {
+          try {
+            const settings = JSON.parse(readFileSync(settingsFile, 'utf8'));
+            if (settings.hooks) {
+              for (const groups of Object.values(settings.hooks) as Array<
+                Array<{ hooks: Array<{ command: string }> }>
+              >) {
+                for (const group of groups) {
+                  hookCount += group.hooks.length;
+                }
+              }
+            }
+          } catch {
+            // Cannot parse settings.json
           }
-        } catch {
+        }
+
+        if (hookCount > 0) {
+          results.push({
+            name: 'Claude Hooks',
+            status: 'ok',
+            message: `${hookCount} hooks registered in settings.json`,
+          });
+        } else {
           results.push({
             name: 'Claude Hooks',
             status: 'warn',
-            message: 'Could not read hooks.json',
+            message: 'No hooks registered in settings.json',
+            fix: 'Run: stackmemory hooks install',
           });
         }
-      } else {
-        results.push({
-          name: 'Claude Hooks',
-          status: 'warn',
-          message: 'Claude hooks not installed (optional)',
-          fix: 'Run: stackmemory hooks install',
-        });
       }
 
       // 5. Check MCP tool definitions
@@ -659,7 +660,44 @@ export function createDoctorCommand(): Command {
         }
       }
 
-      // 12. Check file permissions
+      // 12. Check daemon health
+      {
+        try {
+          const { readDaemonStatus } =
+            await import('../../daemon/daemon-config.js');
+          const status = readDaemonStatus();
+          if (status.running) {
+            const uptime = status.startedAt
+              ? Math.round((Date.now() - status.startedAt) / 1000 / 60)
+              : 0;
+            const svcCount = Object.values(status.services || {}).filter(
+              (s: { enabled?: boolean }) => s.enabled
+            ).length;
+            results.push({
+              name: 'Background Daemon',
+              status: 'ok',
+              message: `Running (PID: ${status.pid}, ${svcCount} services, ${uptime}min uptime)`,
+            });
+          } else {
+            results.push({
+              name: 'Background Daemon',
+              status: 'warn',
+              message:
+                'Daemon not running — context auto-save and maintenance disabled',
+              fix: 'Run: stackmemory daemon start',
+            });
+          }
+        } catch {
+          results.push({
+            name: 'Background Daemon',
+            status: 'warn',
+            message: 'Could not check daemon status',
+            fix: 'Run: stackmemory daemon start',
+          });
+        }
+      }
+
+      // 13. Check file permissions
       const homeStackmemory = join(homedir(), '.stackmemory');
       if (existsSync(homeStackmemory)) {
         try {
@@ -701,13 +739,23 @@ export function createDoctorCommand(): Command {
           console.log(chalk.cyan(`    Fix: ${result.fix}`));
 
           if (options.fix && result.status !== 'ok') {
-            // Auto-fix logic for specific issues
-            if (result.fix.includes('stackmemory setup-mcp')) {
-              console.log(chalk.gray('    Attempting auto-fix...'));
-              try {
-                execSync('stackmemory setup-mcp', { stdio: 'inherit' });
-              } catch {
-                console.log(chalk.red('    Auto-fix failed'));
+            const fixCmds = [
+              'stackmemory setup-mcp',
+              'stackmemory hooks install',
+              'stackmemory daemon start',
+            ];
+            for (const cmd of fixCmds) {
+              if (result.fix.includes(cmd)) {
+                console.log(chalk.gray(`    Attempting: ${cmd}...`));
+                try {
+                  execSync(cmd, {
+                    stdio: 'inherit',
+                    timeout: 15000,
+                  });
+                } catch {
+                  console.log(chalk.red('    Auto-fix failed'));
+                }
+                break;
               }
             }
           }

src/integrations/mcp/server.ts

@@ -1652,6 +1652,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Cord handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.cordHandlers.handleCordSpawn(args);
@@ -1664,6 +1665,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Cord handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.cordHandlers.handleCordFork(args);
@@ -1676,6 +1678,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Cord handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.cordHandlers.handleCordComplete(args);
@@ -1688,6 +1691,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Cord handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.cordHandlers.handleCordAsk(args);
@@ -1700,6 +1704,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Cord handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.cordHandlers.handleCordTree(args);
@@ -1713,6 +1718,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Team handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.teamHandlers.handleTeamContextGet(args);
@@ -1725,6 +1731,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Team handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.teamHandlers.handleTeamContextShare(args);
@@ -1737,6 +1744,7 @@ class LocalStackMemoryMCP {
                   content: [
                     { type: 'text', text: 'Team handlers not initialized.' },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.teamHandlers.handleTeamSearch(args);
@@ -1753,6 +1761,7 @@ class LocalStackMemoryMCP {
                       text: 'Multi-provider routing is disabled.',
                     },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.providerHandlers.handleDelegateToModel(
@@ -1770,6 +1779,7 @@ class LocalStackMemoryMCP {
                       text: 'Multi-provider routing is disabled.',
                     },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.providerHandlers.handleBatchSubmit(
@@ -1787,6 +1797,7 @@ class LocalStackMemoryMCP {
                       text: 'Multi-provider routing is disabled.',
                     },
                   ],
+                  is_error: true,
                 };
               } else {
                 result = await this.providerHandlers.handleBatchCheck(

src/utils/__tests__/hook-installer.test.ts

@@ -18,8 +18,8 @@ const HOOKS_DIR = path.join(os.homedir(), '.claude', 'hooks');
 
 describe('hook-installer', () => {
   describe('CANONICAL_HOOKS', () => {
-    it('defines all 10 hooks', () => {
-      expect(CANONICAL_HOOKS).toHaveLength(10);
+    it('defines all 11 hooks', () => {
+      expect(CANONICAL_HOOKS).toHaveLength(11);
       const names = CANONICAL_HOOKS.map((h) => h.scriptName);
       expect(names).toContain('session-rescue.sh');
       expect(names).toContain('stop-checkpoint.js');
@@ -31,19 +31,21 @@ describe('hook-installer', () => {
       expect(names).toContain('team-task-complete.js');
       expect(names).toContain('team-teammate-idle.js');
       expect(names).toContain('desire-path-trace.js');
+      expect(names).toContain('daemon-auto-start.js');
     });
 
     it('core hooks are required, optional hooks are not', () => {
       const required = CANONICAL_HOOKS.filter((h) => h.required);
       const optional = CANONICAL_HOOKS.filter((h) => !h.required);
       expect(required).toHaveLength(5);
-      expect(optional).toHaveLength(5);
+      expect(optional).toHaveLength(6);
       const optionalNames = optional.map((h) => h.scriptName);
       expect(optionalNames).toContain('theory-capture.js');
       expect(optionalNames).toContain('team-subagent-stop.js');
       expect(optionalNames).toContain('team-task-complete.js');
       expect(optionalNames).toContain('team-teammate-idle.js');
       expect(optionalNames).toContain('desire-path-trace.js');
+      expect(optionalNames).toContain('daemon-auto-start.js');
     });
 
     it('js hooks have node commandPrefix', () => {

src/utils/hook-installer.ts

@@ -98,6 +98,13 @@ export const CANONICAL_HOOKS: HookEntry[] = [
     commandPrefix: 'node',
     required: false,
   },
+  {
+    scriptName: 'daemon-auto-start.js',
+    eventType: 'PostToolUse',
+    timeout: 2,
+    commandPrefix: 'node',
+    required: false,
+  },
 ];
 
 /** Script names that should be removed from settings (dead/deprecated hooks) */