specmem-hardwicksoftware 3.7.4 → 3.7.6

package/dist/mcp/embeddingServerManager.js

@@ -1518,9 +1518,14 @@ export class EmbeddingServerManager extends EventEmitter {
             // Environment variables are null-separated
             const envVars = environ.split('\0');
             for (const envVar of envVars) {
+                // Check both env var names - MCP manager uses SPECMEM_EMBEDDING_SOCKET,
+                // but specmem-init spawns with SPECMEM_SOCKET_PATH
                 if (envVar.startsWith('SPECMEM_EMBEDDING_SOCKET=')) {
                     return envVar.split('=')[1];
                 }
+                if (envVar.startsWith('SPECMEM_SOCKET_PATH=')) {
+                    return envVar.split('=')[1];
+                }
             }
             return null;
         }

package/dist/utils/processHealthCheck.js

@@ -70,7 +70,31 @@ export function checkProcessHealth(config) {
         if (commandLine) {
             const isEmbedding = commandLine.includes('frankenstein-embeddings.py') ||
                 (expectedProcessName && commandLine.includes(expectedProcessName));
-            const isCorrectProject = !projectPath || commandLine.includes(projectPath);
+            // Check project path in command line first, then fall back to env vars
+            // CRITICAL FIX: The embedding server receives project path via SPECMEM_PROJECT_PATH
+            // and socket path via SPECMEM_SOCKET_PATH env vars, NOT command line args.
+            // Without this check, every embedding server appears to be "wrong project" and gets killed.
+            let isCorrectProject = !projectPath || commandLine.includes(projectPath);
+            if (!isCorrectProject && projectPath && isEmbedding) {
+                // Check process environment for SPECMEM_PROJECT_PATH
+                const processEnvProjectPath = getProcessEnvVar(pid, 'SPECMEM_PROJECT_PATH');
+                if (processEnvProjectPath) {
+                    isCorrectProject = processEnvProjectPath === projectPath || processEnvProjectPath.startsWith(projectPath);
+                    if (isCorrectProject) {
+                        logger.debug({ pid, processEnvProjectPath, expectedProject: projectPath },
+                            '[ProcessHealthCheck] Project path matched via SPECMEM_PROJECT_PATH env var');
+                    }
+                }
+                // Also check SPECMEM_SOCKET_PATH which contains the project path
+                if (!isCorrectProject) {
+                    const processSocketPath = getProcessEnvVar(pid, 'SPECMEM_SOCKET_PATH');
+                    if (processSocketPath && processSocketPath.includes(projectPath)) {
+                        isCorrectProject = true;
+                        logger.debug({ pid, processSocketPath, expectedProject: projectPath },
+                            '[ProcessHealthCheck] Project path matched via SPECMEM_SOCKET_PATH env var');
+                    }
+                }
+            }
             isEmbeddingServer = isEmbedding && isCorrectProject;
             if (isEmbedding && !isCorrectProject) {
                 logger.warn({
@@ -144,6 +168,30 @@ export function checkProcessHealth(config) {
 // ============================================================================
 // LOW-LEVEL UTILITIES
 // ============================================================================
+/**
+ * Read a specific environment variable from a running process via /proc
+ * Returns the value or null if not found/readable
+ */
+function getProcessEnvVar(pid, varName) {
+    try {
+        const environPath = `/proc/${pid}/environ`;
+        if (!existsSync(environPath)) {
+            return null;
+        }
+        const environ = readFileSync(environPath, 'utf8');
+        const prefix = varName + '=';
+        const envVars = environ.split('\0');
+        for (const envVar of envVars) {
+            if (envVar.startsWith(prefix)) {
+                return envVar.slice(prefix.length);
+            }
+        }
+        return null;
+    }
+    catch {
+        return null;
+    }
+}
 /**
  * Read PID file with timestamp
  * Format: PID:TIMESTAMP

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specmem-hardwicksoftware",
-  "version": "3.7.4",
+  "version": "3.7.6",
   "type": "module",
   "description": "Persistent memory system for coding sessions - semantic search with pgvector, token compression, team coordination, file watching. Needs root: installs system-wide hooks, manages docker/PostgreSQL, writes global configs, handles screen sessions. justcalljon.pro",
   "main": "dist/index.js",

package/scripts/specmem-init.cjs

@@ -3129,6 +3129,14 @@ async function indexCodebase(projectPath, ui, embeddingResult) {
       // Use Python directly - built-in thread limiting via torch.set_num_threads()
       initLog('[EMBED] Starting frankenstein-embeddings.py directly');
       const pythonPath = getPythonPath();
+
+      // Redirect stdout/stderr to log file instead of piping
+      // CRITICAL: Piped stdio keeps a reference to the child process even after unref(),
+      // which can cause SIGTERM/SIGPIPE when the parent's event loop is under heavy load
+      // (e.g. during batch embedding). Using 'ignore' or file descriptors prevents this.
+      const embedLogPath = path.join(projectPath, 'specmem', 'sockets', 'embedding-autostart.log');
+      const embedLogFd = fs.openSync(embedLogPath, 'a');
+
       const embeddingProcess = spawn(pythonPath, [embeddingScript], {
         cwd: path.dirname(embeddingScript),
         env: {
@@ -3137,7 +3145,7 @@ async function indexCodebase(projectPath, ui, embeddingResult) {
           SPECMEM_PROJECT_PATH: projectPath
         },
         detached: true,
-        stdio: ['ignore', 'pipe', 'pipe']
+        stdio: ['ignore', embedLogFd, embedLogFd]
       });
 
       // error handler BEFORE unref - prevents silent spawn failures
@@ -3145,13 +3153,8 @@ async function indexCodebase(projectPath, ui, embeddingResult) {
         ui.setSubStatus('Embedding spawn error: ' + err.message);
       });
 
-      // CRITICAL: Consume stdout/stderr so Python startup banner doesn't leak to terminal
-      embeddingProcess.stdout.on('data', (chunk) => {
-        initLog('[EMBED-STDOUT] ' + chunk.toString().trim());
-      });
-      embeddingProcess.stderr.on('data', (chunk) => {
-        initLog('[EMBED-STDERR] ' + chunk.toString().trim());
-      });
+      // Close the fd in the parent so only the child holds it
+      fs.closeSync(embedLogFd);
 
       embeddingProcess.unref();