grepmax 0.14.7 → 0.14.9

package/dist/lib/daemon/daemon.js

@@ -131,7 +131,10 @@ class Daemon {
                     stale: 120000,
                     onCompromised: () => {
                         console.error("[daemon] Lock compromised — another daemon took over. Shutting down.");
-                        this.shutdown();
+                        // Force exit after timeout — shutdown() is async and may not fully
+                        // clear event loop references, leaving zombie daemon processes.
+                        setTimeout(() => process.exit(1), 10000).unref();
+                        this.shutdown().finally(() => process.exit(0));
                     },
                 });
                 (0, logger_1.debug)("daemon", "lock acquired");
@@ -143,20 +146,77 @@ class Daemon {
                 }
                 throw err;
             }
-            // 2. Kill existing per-project watchers
+            // 2. Stale socket cleanup + start socket server EARLY.
+            // The socket must be listening before the PID file is written so that
+            // other daemons checking isDaemonRunning() never see a PID for a process
+            // that can't respond to pings. Without this, the slow initialization
+            // steps below (LanceDB, MLX, project watchers) create a window where
+            // new daemons kill this one as "unresponsive".
+            try {
+                fs.unlinkSync(config_1.PATHS.daemonSocket);
+            }
+            catch (_b) { }
+            this.server = net.createServer((conn) => {
+                (0, logger_1.debug)("daemon", "client connected");
+                let buf = "";
+                conn.on("data", (chunk) => {
+                    buf += chunk.toString();
+                    if (buf.length > 1000000) {
+                        conn.destroy();
+                        return;
+                    }
+                    const nl = buf.indexOf("\n");
+                    if (nl === -1)
+                        return;
+                    const line = buf.slice(0, nl);
+                    buf = buf.slice(nl + 1);
+                    let cmd;
+                    try {
+                        cmd = JSON.parse(line);
+                    }
+                    catch (_a) {
+                        conn.write(`${JSON.stringify({ ok: false, error: "invalid JSON" })}\n`);
+                        conn.end();
+                        return;
+                    }
+                    (0, ipc_handler_1.handleCommand)(this, cmd, conn).then((resp) => {
+                        // null means the handler is managing the connection (streaming)
+                        if (resp !== null) {
+                            conn.write(`${JSON.stringify(resp)}\n`);
+                            conn.end();
+                        }
+                    });
+                });
+                conn.on("error", () => { });
+            });
+            yield new Promise((resolve, reject) => {
+                this.server.on("error", (err) => {
+                    const code = err.code;
+                    if (code === "EADDRINUSE") {
+                        console.error("[daemon] Socket already in use");
+                        reject(err);
+                    }
+                    else if (code === "EOPNOTSUPP") {
+                        console.error("[daemon] Filesystem does not support Unix sockets");
+                        process.exitCode = 2;
+                        reject(err);
+                    }
+                    else {
+                        reject(err);
+                    }
+                });
+                this.server.listen(config_1.PATHS.daemonSocket, () => resolve());
+            });
+            // 3. Write PID file AFTER socket is listening — ensures any process that
+            // reads the PID can immediately ping this daemon and get a response.
+            fs.writeFileSync(config_1.PATHS.daemonPidFile, String(process.pid));
+            // 4. Kill existing per-project watchers
             const existing = (0, watcher_store_1.listWatchers)();
             for (const w of existing) {
                 console.log(`[daemon] Taking over from per-project watcher (PID: ${w.pid}, ${path.basename(w.projectRoot)})`);
                 yield (0, process_1.killProcess)(w.pid);
                 (0, watcher_store_1.unregisterWatcher)(w.pid);
             }
-            // 3. Write PID file (informational only — lock is the real guard)
-            fs.writeFileSync(config_1.PATHS.daemonPidFile, String(process.pid));
-            // 4. Stale socket cleanup
-            try {
-                fs.unlinkSync(config_1.PATHS.daemonSocket);
-            }
-            catch (_b) { }
             // 5. Open shared resources
             try {
                 fs.mkdirSync(config_1.PATHS.cacheDir, { recursive: true });
@@ -219,58 +279,6 @@ class Daemon {
                     this.shutdown();
                 }
             }, HEARTBEAT_INTERVAL_MS);
-            // 11. Socket server
-            this.server = net.createServer((conn) => {
-                (0, logger_1.debug)("daemon", "client connected");
-                let buf = "";
-                conn.on("data", (chunk) => {
-                    buf += chunk.toString();
-                    if (buf.length > 1000000) {
-                        conn.destroy();
-                        return;
-                    }
-                    const nl = buf.indexOf("\n");
-                    if (nl === -1)
-                        return;
-                    const line = buf.slice(0, nl);
-                    buf = buf.slice(nl + 1);
-                    let cmd;
-                    try {
-                        cmd = JSON.parse(line);
-                    }
-                    catch (_a) {
-                        conn.write(`${JSON.stringify({ ok: false, error: "invalid JSON" })}\n`);
-                        conn.end();
-                        return;
-                    }
-                    (0, ipc_handler_1.handleCommand)(this, cmd, conn).then((resp) => {
-                        // null means the handler is managing the connection (streaming)
-                        if (resp !== null) {
-                            conn.write(`${JSON.stringify(resp)}\n`);
-                            conn.end();
-                        }
-                    });
-                });
-                conn.on("error", () => { });
-            });
-            yield new Promise((resolve, reject) => {
-                this.server.on("error", (err) => {
-                    const code = err.code;
-                    if (code === "EADDRINUSE") {
-                        console.error("[daemon] Socket already in use");
-                        reject(err);
-                    }
-                    else if (code === "EOPNOTSUPP") {
-                        console.error("[daemon] Filesystem does not support Unix sockets");
-                        process.exitCode = 2;
-                        reject(err);
-                    }
-                    else {
-                        reject(err);
-                    }
-                });
-                this.server.listen(config_1.PATHS.daemonSocket, () => resolve());
-            });
             console.log(`[daemon] Started (PID: ${process.pid}, ${this.processors.size} projects)`);
         });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grepmax",
-  "version": "0.14.7",
+  "version": "0.14.9",
   "author": "Robert Owens <78518764+reowens@users.noreply.github.com>",
   "homepage": "https://github.com/reowens/grepmax",
   "bugs": {

package/plugins/grepmax/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "grepmax",
-  "version": "0.14.7",
+  "version": "0.14.9",
   "description": "Semantic code search for Claude Code. Automatically indexes your project and provides intelligent search capabilities.",
   "author": {
     "name": "Robert Owens",