@vinaes/succ 1.4.0 → 1.5.42

package/hooks/succ-stop-reflection.cjs

@@ -8,77 +8,51 @@
 
 const fs = require('fs');
 const path = require('path');
+const adapter = require('./core/adapter.cjs');
+const { getDaemonPort } = require('./core/daemon-boot.cjs');
 
-let input = '';
-process.stdin.setEncoding('utf8');
-process.stdin.on('readable', () => {
-  let chunk;
-  while ((chunk = process.stdin.read()) !== null) {
-    input += chunk;
+adapter.runHook('stop-reflection', async ({ hookInput, succDir }) => {
+  const tmpDir = path.join(succDir, '.tmp');
+  if (!fs.existsSync(tmpDir)) {
+    fs.mkdirSync(tmpDir, { recursive: true });
   }
-});
-
-process.stdin.on('end', async () => {
-  try {
-    const hookInput = JSON.parse(input);
-    let projectDir = hookInput.cwd || process.cwd();
 
-    // Convert /c/... to C:/... on Windows if needed
-    if (process.platform === 'win32' && /^\/[a-z]\//.test(projectDir)) {
-      projectDir = projectDir[1].toUpperCase() + ':' + projectDir.slice(2);
-    }
+  // Get session_id — fall back to hookInput.session_id when transcript_path is absent
+  // (session-start registers a synthetic session_id in that case)
+  const transcriptPath = hookInput.transcript_path || '';
+  const sessionId = transcriptPath
+    ? path.basename(transcriptPath, '.jsonl')
+    : hookInput.session_id || null;
 
-    // Skip if succ is not initialized in this project
-    if (!fs.existsSync(path.join(projectDir, '.succ'))) {
-      process.exit(0);
-    }
+  const daemonPort = getDaemonPort(succDir);
 
-    const tmpDir = path.join(projectDir, '.succ', '.tmp');
-    if (!fs.existsSync(tmpDir)) {
-      fs.mkdirSync(tmpDir, { recursive: true });
-    }
+  // Check if this is a service session (e.g., reflection subagent)
+  const isServiceSession = process.env.SUCC_SERVICE_SESSION === '1';
 
-    // Get session_id from transcript_path (unique per session)
-    const transcriptPath = hookInput.transcript_path || '';
-    const sessionId = transcriptPath ? path.basename(transcriptPath, '.jsonl') : null;
+  // Notify daemon of stop activity (awaited)
+  const notifyDaemon = async () => {
+    if (!daemonPort || !sessionId) return;
 
-    // Read daemon port
-    let daemonPort = null;
     try {
-      const portFile = path.join(tmpDir, 'daemon.port');
-      if (fs.existsSync(portFile)) {
-        daemonPort = parseInt(fs.readFileSync(portFile, 'utf8').trim(), 10);
-      }
-    } catch {}
-
-    // Check if this is a service session (e.g., reflection subagent)
-    const isServiceSession = process.env.SUCC_SERVICE_SESSION === '1';
-
-    // Notify daemon of stop activity (awaited)
-    const notifyDaemon = async () => {
-      if (!daemonPort || !sessionId) return;
-
-      try {
-        await fetch(`http://127.0.0.1:${daemonPort}/api/session/activity`, {
-          method: 'POST',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({
-            session_id: sessionId,
-            type: 'stop',
-            transcript_path: transcriptPath,
-            is_service: isServiceSession,
-          }),
-          signal: AbortSignal.timeout(2000),
-        });
-      } catch {}
-    };
+      await fetch(`http://127.0.0.1:${daemonPort}/api/session/activity`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          session_id: sessionId,
+          type: 'stop',
+          transcript_path: transcriptPath,
+          is_service: isServiceSession,
+        }),
+        signal: AbortSignal.timeout(2000),
+      });
+    } catch {
+      // intentionally empty
+    }
+  };
 
-    // Wait for daemon notification before exit
-    await notifyDaemon();
-    process.exit(0);
-  } catch (err) {
-    process.exit(0);
-  }
+  // Wait for daemon notification before exit
+  await notifyDaemon();
+  process.exit(0);
 });
 
 // Safety timeout

package/hooks/succ-user-prompt.cjs

@@ -12,209 +12,166 @@
 
 const path = require('path');
 const fs = require('fs');
+const adapter = require('./core/adapter.cjs');
+const { getDaemonPort } = require('./core/daemon-boot.cjs');
+const { log: _log } = require('./core/log.cjs');
+const { loadMergedConfig } = require('./core/config.cjs');
 
-// Logging helper - writes to .succ/.tmp/hooks.log
-function log(succDir, message) {
-  try {
-    const tmpDir = path.join(succDir, '.tmp');
-    if (!fs.existsSync(tmpDir)) {
-      fs.mkdirSync(tmpDir, { recursive: true });
-    }
-    const logFile = path.join(tmpDir, 'hooks.log');
-    const timestamp = new Date().toISOString();
-    fs.appendFileSync(logFile, `[${timestamp}] [user-prompt] ${message}\n`);
-  } catch {
-    // Logging failed, not critical
-  }
-}
-
-let input = '';
-process.stdin.setEncoding('utf8');
-process.stdin.on('readable', () => {
-  let chunk;
-  while ((chunk = process.stdin.read()) !== null) {
-    input += chunk;
-  }
-});
+adapter.runHook('user-prompt', async ({ agent, hookInput, projectDir, succDir }) => {
+  const log = (msg) => _log(succDir, 'user-prompt', msg);
 
-process.stdin.on('end', async () => {
-  try {
-    const hookInput = JSON.parse(input);
-    let projectDir = hookInput.cwd || process.cwd();
+  const tmpDir = path.join(succDir, '.tmp');
+  if (!fs.existsSync(tmpDir)) {
+    fs.mkdirSync(tmpDir, { recursive: true });
+  }
 
-    // Windows path fix
-    if (process.platform === 'win32' && /^\/[a-z]\//.test(projectDir)) {
-      projectDir = projectDir[1].toUpperCase() + ':' + projectDir.slice(2);
-    }
+  // Get session_id from transcript_path (unique per session)
+  const transcriptPath = hookInput.transcript_path || '';
+  const sessionId = transcriptPath ? path.basename(transcriptPath, '.jsonl') : null;
 
-    // Skip if succ is not initialized in this project
-    if (!fs.existsSync(path.join(projectDir, '.succ'))) {
-      process.exit(0);
-    }
+  const daemonPort = getDaemonPort(succDir);
 
-    const tmpDir = path.join(projectDir, '.succ', '.tmp');
+  // Check if this is a service session (e.g., reflection subagent)
+  const isServiceSession = process.env.SUCC_SERVICE_SESSION === '1';
 
-    // Get session_id from transcript_path (unique per session)
-    const transcriptPath = hookInput.transcript_path || '';
-    const sessionId = transcriptPath ? path.basename(transcriptPath, '.jsonl') : null;
+  // Collect all context parts — output a single JSON at the end to avoid broken hook parsing
+  const contextParts = [];
 
-    // Read daemon port
-    let daemonPort = null;
+  // Check for compact-pending fallback
+  // This handles the case where SessionStart hook output was lost after /compact
+  // See: https://github.com/anthropics/claude-code/issues/15174
+  const compactPendingFile = path.join(tmpDir, 'compact-pending');
+  if (fs.existsSync(compactPendingFile) && !isServiceSession) {
     try {
-      const portFile = path.join(tmpDir, 'daemon.port');
-      if (fs.existsSync(portFile)) {
-        daemonPort = parseInt(fs.readFileSync(portFile, 'utf8').trim(), 10);
+      const pendingContext = fs.readFileSync(compactPendingFile, 'utf8');
+      // Delete the file before outputting to avoid duplicate injection
+      fs.unlinkSync(compactPendingFile);
+
+      if (pendingContext.trim()) {
+        log(`Injecting compact-pending fallback context (${pendingContext.length} chars)`);
+        contextParts.push(
+          `<compact-fallback reason="SessionStart output may have been lost">\n${pendingContext}\n</compact-fallback>`
+        );
       }
-    } catch {}
+    } catch (err) {
+      log(`Error processing compact-pending: ${err.message || err}`);
+    }
+  }
 
-    // Check if this is a service session (e.g., reflection subagent)
-    const isServiceSession = process.env.SUCC_SERVICE_SESSION === '1';
+  // Notify daemon of user activity
+  let daemonAlive = false;
+  if (daemonPort && sessionId) {
+    try {
+      const res = await fetch(`http://127.0.0.1:${daemonPort}/api/session/activity`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          session_id: sessionId,
+          type: 'user_prompt',
+          transcript_path: transcriptPath,
+          is_service: isServiceSession,
+        }),
+        signal: AbortSignal.timeout(2000),
+      });
+      if (res.ok) daemonAlive = true;
+    } catch {
+      // intentionally empty — daemon not reachable
+    }
+  }
 
-    const succDir = path.join(projectDir, '.succ');
+  // --- Skill Suggestions ---
+  // LLM-powered skill suggestions based on user prompt.
+  // Uses extractKeywords → BM25 search → LLM ranking.
+  // Respects cooldown to avoid suggesting on every prompt.
+  if (daemonAlive && !isServiceSession) {
+    const prompt = hookInput.prompt || hookInput.message || '';
+
+    // Load config to check if auto_suggest is enabled
+    const config = loadMergedConfig(projectDir);
+    const skillsConfig = config.skills || null;
+
+    // Defaults
+    const autoSuggest = skillsConfig?.auto_suggest || {};
+    const enabled = autoSuggest.enabled === true; // default: false (matches skills.ts)
+    const onUserPrompt = autoSuggest.on_user_prompt !== false; // default: true
+    const minPromptLength = autoSuggest.min_prompt_length || 20;
+    const cooldownPrompts = autoSuggest.cooldown_prompts || 3;
+
+    if (enabled && onUserPrompt && prompt.length >= minPromptLength) {
+      // Check cooldown (track prompts since last suggestion per session)
+      const cooldownFile = path.join(tmpDir, `skill-cooldown-${sessionId}`);
+      let promptsSinceLastSuggestion = 0;
 
-    // Check for compact-pending fallback
-    // This handles the case where SessionStart hook output was lost after /compact
-    // See: https://github.com/anthropics/claude-code/issues/15174
-    const compactPendingFile = path.join(tmpDir, 'compact-pending');
-    if (fs.existsSync(compactPendingFile) && !isServiceSession) {
       try {
-        const pendingContext = fs.readFileSync(compactPendingFile, 'utf8');
-        // Delete the file before outputting to avoid duplicate injection
-        fs.unlinkSync(compactPendingFile);
-
-        if (pendingContext.trim()) {
-          log(succDir, `Injecting compact-pending fallback context (${pendingContext.length} chars)`);
-
-          // Output as additionalContext - this will be injected into Claude's context
-          const output = {
-            hookSpecificOutput: {
-              hookEventName: 'UserPromptSubmit',
-              additionalContext: `<compact-fallback reason="SessionStart output may have been lost">\n${pendingContext}\n</compact-fallback>`
-            }
-          };
-          console.log(JSON.stringify(output));
+        if (fs.existsSync(cooldownFile)) {
+          promptsSinceLastSuggestion =
+            parseInt(fs.readFileSync(cooldownFile, 'utf8').trim(), 10) || 0;
         }
-      } catch (err) {
-        log(succDir, `Error processing compact-pending: ${err.message || err}`);
+      } catch {
+        // intentionally empty
       }
-    }
 
-    // Notify daemon of user activity
-    if (daemonPort && sessionId) {
-      try {
-        await fetch(`http://127.0.0.1:${daemonPort}/api/session/activity`, {
-          method: 'POST',
-          headers: { 'Content-Type': 'application/json' },
-          body: JSON.stringify({
-            session_id: sessionId,
-            type: 'user_prompt',
-            transcript_path: transcriptPath,
-            is_service: isServiceSession,
-          }),
-          signal: AbortSignal.timeout(2000),
-        });
-      } catch {}
-    }
-
-    // --- Skill Suggestions ---
-    // LLM-powered skill suggestions based on user prompt.
-    // Uses extractKeywords → BM25 search → LLM ranking.
-    // Respects cooldown to avoid suggesting on every prompt.
-    if (daemonPort && !isServiceSession) {
-      const prompt = hookInput.prompt || hookInput.message || '';
-
-      // Load config to check if auto_suggest is enabled
-      let skillsConfig = null;
-      const configPaths = [
-        path.join(succDir, 'config.json'),
-        path.join(require('os').homedir(), '.succ', 'config.json'),
-      ];
-      for (const configPath of configPaths) {
-        if (fs.existsSync(configPath)) {
-          try {
-            const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-            if (config.skills) {
-              skillsConfig = config.skills;
-              break;
+      // Only suggest if cooldown has passed
+      if (promptsSinceLastSuggestion >= cooldownPrompts || promptsSinceLastSuggestion === 0) {
+        try {
+          const response = await fetch(`http://127.0.0.1:${daemonPort}/api/skills/suggest`, {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({ prompt }),
+            signal: AbortSignal.timeout(15000), // 15s timeout for 2 LLM calls (extraction + ranking)
+          });
+
+          if (response.ok) {
+            const data = await response.json();
+            const suggestions = data.skills || [];
+
+            if (suggestions.length > 0) {
+              // Reset cooldown counter
+              fs.writeFileSync(cooldownFile, '0', 'utf8');
+
+              // Format suggestions
+              const lines = suggestions.map((s) => {
+                return `- **${s.name}**: ${s.reason} (confidence: ${(s.confidence * 100).toFixed(0)}%)`;
+              });
+
+              const suggestionContext = `<skill-suggestions hint="Use /${suggestions[0].name} to invoke">\n${lines.join('\n')}\n</skill-suggestions>`;
+
+              log(
+                `Suggesting ${suggestions.length} skill(s): ${suggestions.map((s) => s.name).join(', ')}`
+              );
+
+              contextParts.push(suggestionContext);
+            } else {
+              // No suggestions, increment cooldown
+              fs.writeFileSync(cooldownFile, String(promptsSinceLastSuggestion + 1), 'utf8');
             }
-          } catch {
-            // Ignore parse errors
           }
+        } catch (err) {
+          log(
+            `Skill suggestion error: ${err.message || err}${err.cause ? ' | cause: ' + (err.cause.message || err.cause) : ''} | port=${daemonPort}`
+          );
         }
-      }
-
-      // Defaults
-      const autoSuggest = skillsConfig?.auto_suggest || {};
-      const enabled = autoSuggest.enabled !== false; // default: true
-      const onUserPrompt = autoSuggest.on_user_prompt !== false; // default: true
-      const minPromptLength = autoSuggest.min_prompt_length || 20;
-      const cooldownPrompts = autoSuggest.cooldown_prompts || 3;
-
-      if (enabled && onUserPrompt && prompt.length >= minPromptLength) {
-        // Check cooldown (track prompts since last suggestion per session)
-        const cooldownFile = path.join(tmpDir, `skill-cooldown-${sessionId}`);
-        let promptsSinceLastSuggestion = 0;
-
+      } else {
+        // Cooldown not passed, increment counter
         try {
-          if (fs.existsSync(cooldownFile)) {
-            promptsSinceLastSuggestion = parseInt(fs.readFileSync(cooldownFile, 'utf8').trim(), 10) || 0;
-          }
-        } catch {}
-
-        // Only suggest if cooldown has passed
-        if (promptsSinceLastSuggestion >= cooldownPrompts || promptsSinceLastSuggestion === 0) {
-          try {
-            const response = await fetch(`http://127.0.0.1:${daemonPort}/api/skills/suggest`, {
-              method: 'POST',
-              headers: { 'Content-Type': 'application/json' },
-              body: JSON.stringify({ prompt }),
-              signal: AbortSignal.timeout(15000), // 15s timeout for 2 LLM calls (extraction + ranking)
-            });
-
-            if (response.ok) {
-              const data = await response.json();
-              const suggestions = data.suggestions || [];
-
-              if (suggestions.length > 0) {
-                // Reset cooldown counter
-                fs.writeFileSync(cooldownFile, '0', 'utf8');
-
-                // Format suggestions
-                const lines = suggestions.map((s) => {
-                  return `- **${s.name}**: ${s.reason} (confidence: ${(s.confidence * 100).toFixed(0)}%)`;
-                });
-
-                const suggestionContext = `<skill-suggestions hint="Use /${suggestions[0].name} to invoke">\n${lines.join('\n')}\n</skill-suggestions>`;
-
-                log(succDir, `Suggesting ${suggestions.length} skill(s): ${suggestions.map(s => s.name).join(', ')}`);
-
-                // Output as additionalContext
-                const output = {
-                  hookSpecificOutput: {
-                    hookEventName: 'UserPromptSubmit',
-                    additionalContext: suggestionContext
-                  }
-                };
-                console.log(JSON.stringify(output));
-              } else {
-                // No suggestions, increment cooldown
-                fs.writeFileSync(cooldownFile, String(promptsSinceLastSuggestion + 1), 'utf8');
-              }
-            }
-          } catch (err) {
-            log(succDir, `Skill suggestion error: ${err.message || err}`);
-          }
-        } else {
-          // Cooldown not passed, increment counter
-          try {
-            fs.writeFileSync(cooldownFile, String(promptsSinceLastSuggestion + 1), 'utf8');
-          } catch {}
+          fs.writeFileSync(cooldownFile, String(promptsSinceLastSuggestion + 1), 'utf8');
+        } catch {
+          // intentionally empty
         }
       }
     }
+  }
 
-    process.exit(0);
-  } catch (err) {
-    process.exit(0);
+  // Emit a single hook payload with all collected context
+  if (contextParts.length > 0) {
+    const { json, exitCode } = adapter.formatOutput(agent, 'UserPromptSubmit', {
+      additionalContext: contextParts.join('\n\n'),
+    });
+    if (json && Object.keys(json).length > 0) {
+      console.log(JSON.stringify(json));
+    }
+    if (exitCode) process.exit(exitCode);
   }
+
+  process.exit(0);
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vinaes/succ",
-  "version": "1.4.0",
+  "version": "1.5.42",
   "description": "Semantic Understanding for Code Contexts — persistent memory for AI coding assistants (Claude Code, Cursor, Windsurf, Continue.dev)",
   "type": "module",
   "publishConfig": {
@@ -27,8 +27,9 @@
     "succ-mcp": "./dist/mcp-server.js"
   },
   "scripts": {
-    "build": "prettier --check src/ && eslint src/ && tsc",
-    "build:compile": "tsc",
+    "clean:dist": "node --input-type=module -e \"import { rmSync } from 'node:fs'; rmSync('dist', { recursive: true, force: true });\"",
+    "build": "npm run clean:dist && prettier --check src/ && eslint src/ && tsc",
+    "build:compile": "npm run clean:dist && tsc",
     "dev": "tsx src/cli.ts",
     "test": "vitest",
     "test:storage": "vitest --run src/lib/storage/",
@@ -78,12 +79,21 @@
     "commander": "^14.0.3",
     "cross-spawn": "^7.0.6",
     "glob": "^13.0.3",
-    "gray-matter": "^4.0.3",
+    "graphology": "^0.26.0",
+    "graphology-communities-louvain": "^2.0.2",
+    "graphology-library": "^0.8.0",
+    "graphology-metrics": "^2.1.0",
+    "graphology-shortest-path": "^2.0.2",
     "inquirer": "^13.2.2",
     "is-installed-globally": "^1.0.0",
+    "minimatch": "^10.2.4",
     "onnxruntime-node": "1.24.1",
     "ora": "^9.3.0",
+    "p-limit": "^7.3.0",
+    "parse-diff": "^0.11.1",
+    "smol-toml": "^1.6.0",
     "sqlite-vec": "0.1.7-alpha.2",
+    "vscode-languageserver-protocol": "^3.17.5",
     "web-tree-sitter": "^0.26.5",
     "ws": "^8.19.0",
     "zod": "^3.25.76"
@@ -93,16 +103,17 @@
     "pg": "^8.13.0"
   },
   "devDependencies": {
+    "@eslint/js": "^10.0.1",
     "@types/better-sqlite3": "^7.6.0",
-    "@types/inquirer": "^9.0.9",
     "@types/node": "^25.2.3",
     "@types/pg": "^8.11.0",
     "@types/ws": "^8.18.1",
     "@typescript-eslint/eslint-plugin": "^8.54.0",
     "@typescript-eslint/parser": "^8.54.0",
-    "eslint": "^9.39.2",
+    "eslint": "^10.0.1",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-prettier": "^5.5.5",
+    "globals": "^17.3.0",
     "prettier": "^3.8.1",
     "tsx": "^4.0.0",
     "typescript": "^5.0.0",