pumuki-ast-hooks 5.4.8 → 5.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.4.8",
+  "version": "5.5.1",
   "description": "Enterprise-grade AST Intelligence System with multi-platform support (iOS, Android, Backend, Frontend) and Feature-First + DDD + Clean Architecture enforcement. Includes dynamic violations API for intelligent querying.",
   "main": "index.js",
   "bin": {

package/scripts/hooks-system/infrastructure/mcp/ast-intelligence-automation.js

@@ -18,8 +18,6 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
-const crypto = require('crypto');
-const os = require('os');
 const env = require('../../config/env');
 
 // Removed global requires for performance (Lazy Loading)
@@ -56,165 +54,8 @@ function resolveRepoRoot() {
 
 const REPO_ROOT = resolveRepoRoot();
 
-// Create unique lock per project using hash of REPO_ROOT
-// This allows multiple projects to run MCP simultaneously
-const repoHash = crypto.createHash('md5').update(REPO_ROOT).digest('hex').substring(0, 8);
-const MCP_LOCK_DIR = path.join(os.tmpdir(), `mcp-ast-intelligence-${repoHash}.lock`);
-const MCP_LOCK_PID = path.join(MCP_LOCK_DIR, 'pid');
-
-let MCP_IS_PRIMARY = true;
-
-function logMcpError(context, error) {
-    const msg = error instanceof Error ? error.message : String(error);
-    process.stderr.write(`[MCP][ERROR] ${context}: ${msg}\n`);
-}
-
-function logMcpDebug(message) {
-    if (env.getBool('DEBUG', false)) {
-        process.stderr.write(`[MCP][DEBUG] ${message}\n`);
-    }
-}
-
-function isPidRunning(pid) {
-    if (!pid || !Number.isFinite(pid) || pid <= 0) return false;
-    try {
-        process.kill(pid, 0);
-        return true;
-    } catch (error) {
-        logMcpDebug(`isPidRunning(${pid}) = false: ${error.code || error.message}`);
-        return false;
-    }
-}
-
-function safeReadPid(filePath) {
-    try {
-        if (!fs.existsSync(filePath)) return null;
-        const raw = String(fs.readFileSync(filePath, 'utf8') || '').trim();
-        const pid = Number(raw);
-        if (!Number.isFinite(pid) || pid <= 0) return null;
-        return pid;
-    } catch (error) {
-        logMcpError('safeReadPid', error);
-        return null;
-    }
-}
-
-function removeLockDir() {
-    try {
-        if (fs.existsSync(MCP_LOCK_PID)) {
-            fs.unlinkSync(MCP_LOCK_PID);
-            logMcpDebug('Removed lock PID file');
-        }
-    } catch (error) {
-        logMcpError('removeLockDir (pid file)', error);
-    }
-    try {
-        if (fs.existsSync(MCP_LOCK_DIR)) {
-            fs.rmdirSync(MCP_LOCK_DIR);
-            logMcpDebug('Removed lock directory');
-        }
-    } catch (error) {
-        logMcpError('removeLockDir (directory)', error);
-    }
-}
-
-function cleanupAndExit(code = 0) {
-    const myPid = process.pid;
-    const lockPid = safeReadPid(MCP_LOCK_PID);
-
-    if (lockPid === myPid) {
-        logMcpDebug(`Cleaning up lock (my pid=${myPid})`);
-        removeLockDir();
-    } else {
-        logMcpDebug(`Not cleaning lock (lockPid=${lockPid}, myPid=${myPid})`);
-    }
-
-    process.exit(code);
-}
-
-function installStdioExitHandlers() {
-    const handleStdioTermination = (source) => (error) => {
-        if (error) {
-            const code = String(error.code || '').toUpperCase();
-            if (code === 'EPIPE' || code === 'ERR_STREAM_DESTROYED' || code === 'ECONNRESET') {
-                logMcpDebug(`STDIO ${source} closed (${code}), exiting cleanly`);
-                cleanupAndExit(0);
-                return;
-            }
-            logMcpError(`STDIO ${source} error`, error);
-        } else {
-            logMcpDebug(`STDIO ${source} ended, exiting cleanly`);
-        }
-        cleanupAndExit(0);
-    };
-
-    try {
-        process.stdin.on('end', handleStdioTermination('stdin'));
-        process.stdin.on('close', handleStdioTermination('stdin'));
-        process.stdin.on('error', handleStdioTermination('stdin'));
-    } catch (error) {
-        logMcpError('installStdioExitHandlers (stdin)', error);
-    }
-
-    try {
-        process.stdout.on('error', handleStdioTermination('stdout'));
-        process.stderr.on('error', handleStdioTermination('stderr'));
-    } catch (error) {
-        logMcpError('installStdioExitHandlers (stdout/stderr)', error);
-    }
-}
-
-function acquireSingletonLock() {
-    // No need to create .audit_tmp since lock is in /tmp/
-
-    try {
-        fs.mkdirSync(MCP_LOCK_DIR);
-    } catch (error) {
-        const existingPid = safeReadPid(MCP_LOCK_PID);
-
-        if (existingPid && isPidRunning(existingPid)) {
-            process.stderr.write(`[MCP] Another instance is already running (pid ${existingPid}). Exiting.\n`);
-            process.exit(0);
-        }
-
-        logMcpDebug(`Lock exists but PID ${existingPid || 'unknown'} is not running, cleaning up`);
-        removeLockDir();
-
-        try {
-            fs.mkdirSync(MCP_LOCK_DIR);
-        } catch (retryError) {
-            logMcpError('acquireSingletonLock (retry mkdir)', retryError);
-            process.stderr.write(`[MCP] Failed to acquire lock after cleanup. Exiting.\n`);
-            process.exit(1);
-        }
-    }
-
-    try {
-        fs.writeFileSync(MCP_LOCK_PID, String(process.pid), { encoding: 'utf8' });
-        logMcpDebug(`Lock acquired, PID ${process.pid} written`);
-    } catch (error) {
-        logMcpError('acquireSingletonLock (write pid)', error);
-    }
-
-    process.on('exit', () => {
-        const lockPid = safeReadPid(MCP_LOCK_PID);
-        if (lockPid === process.pid) {
-            removeLockDir();
-        }
-    });
-
-    process.on('SIGINT', () => cleanupAndExit(0));
-    process.on('SIGTERM', () => cleanupAndExit(0));
-    process.on('SIGHUP', () => cleanupAndExit(0));
-
-    return { acquired: true, pid: process.pid };
-}
-
-const singleton = acquireSingletonLock();
-if (!singleton.acquired) {
-    process.exit(0);
-}
-installStdioExitHandlers();
+// NO singleton lock - Windsurf manages process lifecycle
+// Each project gets its own independent MCP process
 
 // Lazy-loaded CompositionRoot - only initialized when first needed
 let _compositionRoot = null;
@@ -1126,10 +967,14 @@ async function handleMcpMessage(message) {
 // Start protocol handler
 protocolHandler.start(handleMcpMessage);
 
+// Log MCP ready
+process.stderr.write(`[MCP] Server ready for ${REPO_ROOT}\n`);
+
 /**
  * Polling loop for background notifications and automations
+ * IMPORTANT: Delayed start to avoid blocking MCP initialization handshake
  */
-if (MCP_IS_PRIMARY) {
+setTimeout(() => {
     setInterval(async () => {
         try {
             const now = Date.now();
@@ -1186,10 +1031,8 @@ if (MCP_IS_PRIMARY) {
             if (process.env.DEBUG) console.error('[MCP] Polling loop error:', error);
         }
     }, 30000);
-}
 
-// AUTO-COMMIT: Only for project code changes (no node_modules, no library)
-if (MCP_IS_PRIMARY) {
+    // AUTO-COMMIT: Only for project code changes (no node_modules, no library)
     setInterval(async () => {
         if (!AUTO_COMMIT_ENABLED) {
             return;
@@ -1292,4 +1135,4 @@ if (MCP_IS_PRIMARY) {
             if (process.env.DEBUG) console.error('[MCP] Auto-commit error:', error);
         }
     }, AUTO_COMMIT_INTERVAL);
-}
+}, 2000); // Delay 2 seconds to allow MCP handshake to complete first