grepmax 0.8.1 → 0.8.2

package/dist/commands/add.js

@@ -155,9 +155,12 @@ Examples:
         }
         // Start watcher
         const launched = (0, watcher_launcher_1.launchWatcher)(projectRoot);
-        if (launched) {
+        if (launched.ok) {
             console.log(`Watcher started (PID: ${launched.pid})`);
         }
+        else if (launched.reason === "spawn-failed") {
+            console.warn(`[add] ${launched.message}`);
+        }
     }
     catch (error) {
         const message = error instanceof Error ? error.message : "Unknown error";

package/dist/commands/droid.js

@@ -130,7 +130,10 @@ function installPlugin() {
         const startScript = `
 const { spawn } = require("child_process");
 const fs = require("fs");
-const out = fs.openSync("/tmp/gmax.log", "a");
+const path = require("path");
+const logDir = path.join(require("os").homedir(), ".gmax", "logs");
+fs.mkdirSync(logDir, { recursive: true });
+const out = fs.openSync(path.join(logDir, "gmax.log"), "a");
 const child = spawn("gmax", ["serve"], { detached: true, stdio: ["ignore", out, out] });
 child.unref();
 `;

package/dist/commands/index.js

@@ -169,9 +169,12 @@ Examples:
             // Restart the watcher if we stopped one
             if (restartWatcher) {
                 const launched = (0, watcher_launcher_1.launchWatcher)(restartWatcher.projectRoot);
-                if (launched) {
+                if (launched.ok) {
                     console.log(`Restarted watcher for ${path.basename(restartWatcher.projectRoot)} (PID: ${launched.pid})`);
                 }
+                else if (launched.reason === "spawn-failed") {
+                    console.warn(`[index] ${launched.message}`);
+                }
             }
         }
     }

package/dist/commands/mcp.js

@@ -77,6 +77,7 @@ const format_helpers_1 = require("../lib/utils/format-helpers");
 const import_extractor_1 = require("../lib/utils/import-extractor");
 const project_registry_1 = require("../lib/utils/project-registry");
 const project_root_1 = require("../lib/utils/project-root");
+const watcher_launcher_1 = require("../lib/utils/watcher-launcher");
 const watcher_store_1 = require("../lib/utils/watcher-store");
 // ---------------------------------------------------------------------------
 // Tool definitions
@@ -352,17 +353,10 @@ exports.mcp = new commander_1.Command("mcp")
     }
     // --- Background watcher ---
     function ensureWatcher() {
-        // Only start watcher for registered projects
-        if (!(0, project_registry_1.getProject)(projectRoot))
-            return;
-        if ((0, watcher_store_1.getWatcherCoveringPath)(projectRoot))
-            return;
-        const child = (0, node_child_process_1.spawn)("gmax", ["watch", "-b", "--path", projectRoot], {
-            detached: true,
-            stdio: "ignore",
-        });
-        child.unref();
-        console.log(`[MCP] Started background watcher for ${projectRoot}`);
+        const result = (0, watcher_launcher_1.launchWatcher)(projectRoot);
+        if (result.ok && !result.reused) {
+            console.log(`[MCP] Started background watcher for ${projectRoot} (PID: ${result.pid})`);
+        }
     }
     // --- Tool handlers ---
     function handleSemanticSearch(args_1) {

package/dist/commands/remove.js

@@ -49,6 +49,7 @@ const commander_1 = require("commander");
 const meta_cache_1 = require("../lib/store/meta-cache");
 const vector_db_1 = require("../lib/store/vector-db");
 const exit_1 = require("../lib/utils/exit");
+const process_1 = require("../lib/utils/process");
 const project_marker_1 = require("../lib/utils/project-marker");
 const project_registry_1 = require("../lib/utils/project-registry");
 const project_root_1 = require("../lib/utils/project-root");
@@ -102,15 +103,7 @@ Examples:
         const watcher = (0, watcher_store_1.getWatcherForProject)(projectRoot);
         if (watcher) {
             console.log(`Stopping watcher (PID: ${watcher.pid})...`);
-            try {
-                process.kill(watcher.pid, "SIGTERM");
-            }
-            catch (_b) { }
-            for (let i = 0; i < 50; i++) {
-                if (!(0, watcher_store_1.isProcessRunning)(watcher.pid))
-                    break;
-                yield new Promise((r) => setTimeout(r, 100));
-            }
+            yield (0, process_1.killProcess)(watcher.pid);
             (0, watcher_store_1.unregisterWatcher)(watcher.pid);
         }
         // Delete vectors from LanceDB
@@ -139,13 +132,13 @@ Examples:
             try {
                 metaCache.close();
             }
-            catch (_c) { }
+            catch (_b) { }
         }
         if (vectorDb) {
             try {
                 yield vectorDb.close();
             }
-            catch (_d) { }
+            catch (_c) { }
         }
         yield (0, exit_1.gracefulExit)();
     }

package/dist/commands/search.js

@@ -529,7 +529,10 @@ Examples:
         // Ensure a watcher is running for live reindexing
         if (!process.env.VITEST && !((_d = process.env.NODE_ENV) === null || _d === void 0 ? void 0 : _d.includes("test"))) {
             const { launchWatcher } = yield Promise.resolve().then(() => __importStar(require("../lib/utils/watcher-launcher")));
-            launchWatcher(projectRoot);
+            const launched = launchWatcher(projectRoot);
+            if (!launched.ok && launched.reason === "spawn-failed") {
+                console.warn(`[search] ${launched.message}`);
+            }
         }
         const searcher = new searcher_1.Searcher(vectorDb);
         // Use --root or fall back to project root

package/dist/commands/serve.js

@@ -59,6 +59,7 @@ const setup_helpers_1 = require("../lib/setup/setup-helpers");
 const meta_cache_1 = require("../lib/store/meta-cache");
 const vector_db_1 = require("../lib/store/vector-db");
 const exit_1 = require("../lib/utils/exit");
+const log_rotate_1 = require("../lib/utils/log-rotate");
 const project_root_1 = require("../lib/utils/project-root");
 const server_registry_1 = require("../lib/utils/server-registry");
 function isMlxServerUp() {
@@ -84,8 +85,7 @@ function startMlxServer(mlxModel) {
     const serverDir = candidates.find((d) => fs.existsSync(path.join(d, "server.py")));
     if (!serverDir)
         return null;
-    const logPath = "/tmp/mlx-embed-server.log";
-    const out = fs.openSync(logPath, "a");
+    const out = (0, log_rotate_1.openRotatedLog)(path.join(config_1.PATHS.logsDir, "mlx-embed-server.log"));
     const env = Object.assign({}, process.env);
     if (mlxModel) {
         env.MLX_EMBED_MODEL = mlxModel;
@@ -121,14 +121,12 @@ exports.serve = new commander_1.Command("serve")
         const args = process.argv
             .slice(2)
             .filter((arg) => arg !== "-b" && arg !== "--background");
-        const logDir = path.join(config_1.PATHS.globalRoot, "logs");
-        fs.mkdirSync(logDir, { recursive: true });
         const safeName = path
             .basename(projectRoot)
             .replace(/[^a-zA-Z0-9._-]/g, "_");
-        const logFile = path.join(logDir, `server-${safeName}.log`);
-        const out = fs.openSync(logFile, "a");
-        const err = fs.openSync(logFile, "a");
+        const logFile = path.join(config_1.PATHS.logsDir, `server-${safeName}.log`);
+        const out = (0, log_rotate_1.openRotatedLog)(logFile);
+        const err = (0, log_rotate_1.openRotatedLog)(logFile);
         const child = (0, node_child_process_1.spawn)(process.argv[0], [process.argv[1], ...args], {
             detached: true,
             stdio: ["ignore", out, err],

package/dist/commands/watch.js

@@ -44,7 +44,6 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.watch = void 0;
 const node_child_process_1 = require("node:child_process");
-const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
 const commander_1 = require("commander");
 const config_1 = require("../config");
@@ -55,12 +54,13 @@ const watcher_1 = require("../lib/index/watcher");
 const meta_cache_1 = require("../lib/store/meta-cache");
 const vector_db_1 = require("../lib/store/vector-db");
 const exit_1 = require("../lib/utils/exit");
+const log_rotate_1 = require("../lib/utils/log-rotate");
+const process_1 = require("../lib/utils/process");
 const project_registry_1 = require("../lib/utils/project-registry");
 const project_root_1 = require("../lib/utils/project-root");
 const watcher_store_1 = require("../lib/utils/watcher-store");
 const IDLE_TIMEOUT_MS = 30 * 60 * 1000; // 30 minutes
 const IDLE_CHECK_INTERVAL_MS = 60 * 1000; // check every minute
-const MAX_LOG_BYTES = 5 * 1024 * 1024; // 5 MB — rotate log when exceeded
 exports.watch = new commander_1.Command("watch")
     .description("Start background file watcher for live reindexing")
     .option("-b, --background", "Run watcher in background and exit")
@@ -83,20 +83,9 @@ exports.watch = new commander_1.Command("watch")
         const args = process.argv
             .slice(2)
             .filter((arg) => arg !== "-b" && arg !== "--background");
-        const logDir = path.join(config_1.PATHS.globalRoot, "logs");
-        fs.mkdirSync(logDir, { recursive: true });
         const safeName = projectName.replace(/[^a-zA-Z0-9._-]/g, "_");
-        const logFile = path.join(logDir, `watch-${safeName}.log`);
-        // Rotate log if it exceeds MAX_LOG_BYTES
-        try {
-            const logStat = fs.statSync(logFile);
-            if (logStat.size > MAX_LOG_BYTES) {
-                const prev = `${logFile}.prev`;
-                fs.renameSync(logFile, prev);
-            }
-        }
-        catch (_c) { }
-        const out = fs.openSync(logFile, "a");
+        const logFile = path.join(config_1.PATHS.logsDir, `watch-${safeName}.log`);
+        const out = (0, log_rotate_1.openRotatedLog)(logFile);
         const child = (0, node_child_process_1.spawn)(process.argv[0], [process.argv[1], ...args], {
             detached: true,
             stdio: ["ignore", out, out],
@@ -228,11 +217,11 @@ exports.watch
     if (options.all) {
         const watchers = (0, watcher_store_1.listWatchers)();
         for (const w of watchers) {
-            try {
-                process.kill(w.pid, "SIGTERM");
-                (0, watcher_store_1.unregisterWatcher)(w.pid);
+            const killed = yield (0, process_1.killProcess)(w.pid);
+            (0, watcher_store_1.unregisterWatcher)(w.pid);
+            if (!killed) {
+                console.warn(`Warning: PID ${w.pid} did not exit after SIGKILL`);
             }
-            catch (_b) { }
         }
         console.log(`Stopped ${watchers.length} watcher(s).`);
         yield (0, exit_1.gracefulExit)();
@@ -245,14 +234,13 @@ exports.watch
         yield (0, exit_1.gracefulExit)();
         return;
     }
-    try {
-        process.kill(watcher.pid, "SIGTERM");
-        (0, watcher_store_1.unregisterWatcher)(watcher.pid);
+    const killed = yield (0, process_1.killProcess)(watcher.pid);
+    (0, watcher_store_1.unregisterWatcher)(watcher.pid);
+    if (killed) {
         console.log(`Stopped watcher (PID: ${watcher.pid})`);
     }
-    catch (_c) {
-        console.log("Watcher process not found.");
-        (0, watcher_store_1.unregisterWatcher)(watcher.pid);
+    else {
+        console.warn(`Warning: watcher PID ${watcher.pid} did not exit`);
     }
     yield (0, exit_1.gracefulExit)();
 }));

package/dist/config.js

@@ -93,6 +93,7 @@ exports.PATHS = {
     globalRoot: GLOBAL_ROOT,
     models: path.join(GLOBAL_ROOT, "models"),
     grammars: path.join(GLOBAL_ROOT, "grammars"),
+    logsDir: path.join(GLOBAL_ROOT, "logs"),
     // Centralized index storage — one database for all indexed directories
     lancedbDir: path.join(GLOBAL_ROOT, "lancedb"),
     cacheDir: path.join(GLOBAL_ROOT, "cache"),

package/dist/lib/index/watcher.js

@@ -136,9 +136,11 @@ function startWatcher(opts) {
                 const vectors = [];
                 const metaUpdates = new Map();
                 const metaDeletes = [];
+                const attempted = new Set();
                 for (const [absPath, event] of batch) {
                     if (batchAc.signal.aborted)
                         break;
+                    attempted.add(absPath);
                     if (event === "unlink") {
                         deletes.push(absPath);
                         metaDeletes.push(absPath);
@@ -200,6 +202,12 @@ function startWatcher(opts) {
                         }
                     }
                 }
+                // Requeue files that weren't attempted (aborted or pool unhealthy)
+                for (const [absPath, event] of batch) {
+                    if (!attempted.has(absPath) && !pending.has(absPath)) {
+                        pending.set(absPath, event);
+                    }
+                }
                 // Flush to VectorDB: insert first, then delete old (preserving new)
                 const newIds = vectors.map((v) => v.id);
                 if (vectors.length > 0) {

package/dist/lib/utils/log-rotate.js

@@ -0,0 +1,55 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.openRotatedLog = openRotatedLog;
+const fs = __importStar(require("node:fs"));
+const path = __importStar(require("node:path"));
+const MAX_LOG_BYTES = 5 * 1024 * 1024; // 5 MB
+/**
+ * Open a log file with rotation. Creates parent directories if needed.
+ * Rotates {name}.log -> {name}.log.prev when size exceeds maxBytes.
+ * Returns an fd suitable for stdio redirection.
+ */
+function openRotatedLog(logPath, maxBytes = MAX_LOG_BYTES) {
+    fs.mkdirSync(path.dirname(logPath), { recursive: true });
+    try {
+        const stat = fs.statSync(logPath);
+        if (stat.size > maxBytes) {
+            fs.renameSync(logPath, `${logPath}.prev`);
+        }
+    }
+    catch (_a) { }
+    return fs.openSync(logPath, "a");
+}

package/dist/lib/utils/process.js

@@ -0,0 +1,45 @@
+"use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.killProcess = killProcess;
+const watcher_store_1 = require("./watcher-store");
+/**
+ * Send SIGTERM, wait up to 3s, then SIGKILL if still alive.
+ * Returns true if process is confirmed dead.
+ */
+function killProcess(pid) {
+    return __awaiter(this, void 0, void 0, function* () {
+        try {
+            process.kill(pid, "SIGTERM");
+        }
+        catch (_a) {
+            return false;
+        }
+        // Poll up to 3s for graceful exit
+        for (let i = 0; i < 30; i++) {
+            if (!(0, watcher_store_1.isProcessRunning)(pid))
+                return true;
+            yield new Promise((r) => setTimeout(r, 100));
+        }
+        // Force kill
+        try {
+            process.kill(pid, "SIGKILL");
+        }
+        catch (_b) { }
+        // Give SIGKILL a moment
+        for (let i = 0; i < 10; i++) {
+            if (!(0, watcher_store_1.isProcessRunning)(pid))
+                return true;
+            yield new Promise((r) => setTimeout(r, 100));
+        }
+        return !(0, watcher_store_1.isProcessRunning)(pid);
+    });
+}

package/dist/lib/utils/watcher-launcher.js

@@ -8,39 +8,41 @@ exports.launchWatcher = launchWatcher;
 const node_child_process_1 = require("node:child_process");
 const project_registry_1 = require("./project-registry");
 const watcher_store_1 = require("./watcher-store");
-/**
- * Launch a background watcher for a project.
- *
- * Returns { pid } on success, null if:
- * - Project is not registered
- * - Watcher is already running
- * - Spawn fails
- */
 function launchWatcher(projectRoot) {
     var _a;
     // 1. Project must be registered
     const project = (0, project_registry_1.getProject)(projectRoot);
     if (!project) {
-        return null;
+        return {
+            ok: false,
+            reason: "not-registered",
+            message: `Project not registered. Run: gmax add ${projectRoot}`,
+        };
     }
     // 2. Check if watcher already running
     const existing = (_a = (0, watcher_store_1.getWatcherForProject)(projectRoot)) !== null && _a !== void 0 ? _a : (0, watcher_store_1.getWatcherCoveringPath)(projectRoot);
     if (existing && (0, watcher_store_1.isProcessRunning)(existing.pid)) {
-        return { pid: existing.pid };
+        return { ok: true, pid: existing.pid, reused: true };
     }
     // 3. Spawn
     try {
         const child = (0, node_child_process_1.spawn)(process.argv[0], [process.argv[1], "watch", "--path", projectRoot, "-b"], { detached: true, stdio: "ignore" });
         child.unref();
         if (child.pid) {
-            return { pid: child.pid };
+            return { ok: true, pid: child.pid, reused: false };
         }
-        console.error(`[watcher-launcher] Spawn returned no PID for ${projectRoot}`);
-        return null;
+        return {
+            ok: false,
+            reason: "spawn-failed",
+            message: `Spawn returned no PID for ${projectRoot}`,
+        };
     }
     catch (err) {
         const msg = err instanceof Error ? err.message : String(err);
-        console.error(`[watcher-launcher] Failed to start watcher for ${projectRoot}: ${msg}`);
-        return null;
+        return {
+            ok: false,
+            reason: "spawn-failed",
+            message: `Failed to start watcher for ${projectRoot}: ${msg}`,
+        };
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grepmax",
-  "version": "0.8.1",
+  "version": "0.8.2",
   "author": "Robert Owens <robowens@me.com>",
   "homepage": "https://github.com/reowens/grepmax",
   "bugs": {

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

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