grepmax 0.7.23 → 0.7.25

package/dist/commands/mcp.js

@@ -463,54 +463,37 @@ exports.mcp = new commander_1.Command("mcp")
                 _indexing = false;
                 _indexProgress = "";
                 _indexChildPid = null;
-                // Re-check if index now exists
-                try {
-                    const db = getVectorDb();
-                    if (yield db.hasRowsForPath(projectRoot)) {
-                        _indexReady = true;
-                        console.log("[MCP] Background indexing complete.");
-                        return;
-                    }
-                }
-                catch (_b) { }
             }
-            try {
-                const db = getVectorDb();
-                const hasIndex = yield db.hasRowsForPath(projectRoot);
-                if (!hasIndex) {
-                    if (_indexing)
-                        return; // Already indexing in background
-                    _indexing = true;
-                    _indexProgress = "starting...";
-                    console.log("[MCP] No index found, spawning background index...");
-                    // Spawn gmax index as a detached child process — doesn't hold
-                    // the lock inside this MCP process, so CLI `gmax index` won't conflict.
-                    const child = (0, node_child_process_1.spawn)(process.argv[0], [process.argv[1], "index", "--path", projectRoot], { detached: true, stdio: "ignore" });
-                    _indexChildPid = (_a = child.pid) !== null && _a !== void 0 ? _a : null;
-                    child.unref();
-                    _indexProgress = `PID ${_indexChildPid}`;
-                    console.log(`[MCP] Background index started (PID: ${_indexChildPid})`);
-                    child.on("exit", (code) => {
-                        _indexing = false;
-                        _indexProgress = "";
-                        _indexChildPid = null;
-                        if (code === 0) {
-                            _indexReady = true;
-                            console.log("[MCP] Background indexing complete.");
-                        }
-                        else {
-                            console.error(`[MCP] Background indexing failed (exit code: ${code})`);
-                        }
-                    });
+            // Check project registry — more reliable than querying the DB.
+            // Avoids false negatives from lock contention and cascade re-indexing.
+            const projects = (0, project_registry_1.listProjects)();
+            const isRegistered = projects.some((p) => p.root === projectRoot);
+            if (isRegistered) {
+                _indexReady = true;
+                return;
+            }
+            // Truly first-time: no registry entry at all
+            if (_indexing)
+                return;
+            _indexing = true;
+            _indexProgress = "starting...";
+            console.log("[MCP] First-time index for this project...");
+            const child = (0, node_child_process_1.spawn)(process.argv[0], [process.argv[1], "index", "--path", projectRoot], { detached: true, stdio: "ignore" });
+            _indexChildPid = (_a = child.pid) !== null && _a !== void 0 ? _a : null;
+            child.unref();
+            _indexProgress = `PID ${_indexChildPid}`;
+            child.on("exit", (code) => {
+                _indexing = false;
+                _indexProgress = "";
+                _indexChildPid = null;
+                if (code === 0) {
+                    _indexReady = true;
+                    console.log("[MCP] First-time indexing complete.");
                 }
                 else {
-                    console.log("[MCP] Index exists, ready.");
-                    _indexReady = true;
+                    console.error(`[MCP] Indexing failed (exit code: ${code})`);
                 }
-            }
-            catch (e) {
-                console.error("[MCP] Index sync failed:", e);
-            }
+            });
         });
     }
     // --- Background watcher ---
@@ -575,7 +558,6 @@ exports.mcp = new commander_1.Command("mcp")
             if (!query)
                 return err("Missing required parameter: query");
             const limit = Math.min(Math.max(Number(args.limit) || 3, 1), 50);
-            yield ensureIndexReady();
             ensureWatcher();
             if (_indexing) {
                 return ok(`Indexing in progress (${_indexProgress}). Results may be incomplete or empty — try again shortly.`);

package/dist/commands/search.js

@@ -357,7 +357,7 @@ Examples:
   gmax "handler" --name "handle.*" --exclude tests/
 `)
     .action((pattern, exec_path, _options, cmd) => __awaiter(void 0, void 0, void 0, function* () {
-    var _a, _b, _c, _d;
+    var _a, _b, _c, _d, _e;
     const options = cmd.optsWithGlobals();
     const root = process.cwd();
     const minScore = Number.isFinite(Number.parseFloat(options.minScore))
@@ -497,10 +497,23 @@ Examples:
                 throw e;
             }
         }
+        // Ensure a watcher is running for live reindexing
+        if (!process.env.VITEST && !((_c = process.env.NODE_ENV) === null || _c === void 0 ? void 0 : _c.includes("test"))) {
+            try {
+                const { execFileSync } = yield Promise.resolve().then(() => __importStar(require("node:child_process")));
+                execFileSync("gmax", ["watch", "-b", "--path", projectRoot], {
+                    timeout: 5000,
+                    stdio: "ignore",
+                });
+            }
+            catch (_f) {
+                // Watcher may already be running — ignore
+            }
+        }
         const searcher = new searcher_1.Searcher(vectorDb);
         // Use --root or fall back to project root
         const effectiveRoot = options.root
-            ? (_c = (0, project_root_1.findProjectRoot)(path.resolve(options.root))) !== null && _c !== void 0 ? _c : path.resolve(options.root)
+            ? (_d = (0, project_root_1.findProjectRoot)(path.resolve(options.root))) !== null && _d !== void 0 ? _d : path.resolve(options.root)
             : projectRoot;
         const searchPathPrefix = exec_path
             ? path.resolve(exec_path)
@@ -519,7 +532,7 @@ Examples:
         if (options.role)
             searchFilters.role = options.role;
         const searchResult = yield searcher.search(pattern, parseInt(options.m, 10), { rerank: true }, Object.keys(searchFilters).length > 0 ? searchFilters : undefined, pathFilter);
-        if ((_d = searchResult.warnings) === null || _d === void 0 ? void 0 : _d.length) {
+        if ((_e = searchResult.warnings) === null || _e === void 0 ? void 0 : _e.length) {
             for (const w of searchResult.warnings) {
                 console.warn(`Warning: ${w}`);
             }
@@ -536,7 +549,7 @@ Examples:
                     return defs.some((d) => regex.test(d));
                 });
             }
-            catch (_e) {
+            catch (_g) {
                 // Invalid regex — skip
             }
         }
@@ -621,7 +634,7 @@ Examples:
                     console.log(lines.join("\n"));
                 }
             }
-            catch (_f) {
+            catch (_h) {
                 // Trace failed — skip silently
             }
         }

package/dist/lib/index/watcher.js

@@ -48,6 +48,7 @@ const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
 const chokidar_1 = require("chokidar");
 const filter_builder_1 = require("../utils/filter-builder");
+const config_1 = require("../../config");
 const file_utils_1 = require("../utils/file-utils");
 const logger_1 = require("../utils/logger");
 const lock_1 = require("../utils/lock");
@@ -135,8 +136,13 @@ function startWatcher(opts) {
                         const stats = yield fs.promises.stat(absPath);
                         if (!(0, file_utils_1.isIndexableFile)(absPath, stats.size))
                             continue;
-                        // Check if content actually changed via hash
+                        // Quick mtime/size check — skip worker pool if unchanged
                         const cached = metaCache.get(absPath);
+                        if (cached &&
+                            cached.mtimeMs === stats.mtimeMs &&
+                            cached.size === stats.size) {
+                            continue;
+                        }
                         const result = yield pool.processFile({
                             path: absPath,
                             absolutePath: absPath,
@@ -285,8 +291,12 @@ function startWatcher(opts) {
     const onFileEvent = (event, absPath) => {
         if (closed)
             return;
-        if (event !== "unlink" && !(0, file_utils_1.isIndexableFile)(absPath))
-            return;
+        if (event !== "unlink") {
+            const ext = path.extname(absPath).toLowerCase();
+            const bn = path.basename(absPath).toLowerCase();
+            if (!config_1.INDEXABLE_EXTENSIONS.has(ext) && !config_1.INDEXABLE_EXTENSIONS.has(bn))
+                return;
+        }
         pending.set(absPath, event);
         scheduleBatch();
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grepmax",
-  "version": "0.7.23",
+  "version": "0.7.25",
   "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.7.23",
+  "version": "0.7.25",
   "description": "Semantic code search for Claude Code. Automatically indexes your project and provides intelligent search capabilities.",
   "author": {
     "name": "Robert Owens",

package/plugins/grepmax/skills/grepmax/SKILL.md

@@ -130,11 +130,31 @@ The watcher auto-starts when the MCP server connects — it detects file changes
 
 If search results include a warning like "Full-text search unavailable", results may be less precise. This resolves automatically — the index retries FTS every 5 minutes.
 
+## CLI vs MCP — when to use which
+
+**Prefer CLI (`Bash(gmax ...)`) for repeated searches.** The CLI is ~2x more token-efficient because MCP tool schemas add ~800 tokens of overhead per call. Every CLI flag maps to an MCP param:
+
+```
+Bash(gmax "auth handler" --role ORCHESTRATION --lang ts --plain -m 3)
+```
+
+is equivalent to `semantic_search` with `role: "ORCHESTRATION", language: "ts", limit: 3` — but costs half the tokens.
+
+**CLI commands for all MCP tools:**
+- `gmax "query" --plain` → `semantic_search`
+- `gmax trace <symbol> -d 2` → `trace_calls` with depth
+- `gmax skeleton <target> --json` → `code_skeleton`
+- `gmax project` → `summarize_project`
+- `gmax related <file>` → `related_files`
+- `gmax recent` → `recent_changes`
+
+**Use MCP tools when:** first exploring (tool descriptions guide usage), or when you need pointer mode output (more structured than CLI).
+
 ## Tips
 
 - **Be specific.** "auth" returns noise. "where does the server validate JWT tokens from the Authorization header" returns exactly what you need. Aim for 5+ words.
-- **ORCH results contain the logic** — prioritize over DEF/IMPL results.
+- **Use `--plain` for CLI searches** — agent-friendly output without ANSI codes.
+- **ORCH results contain the logic** — use `--role ORCHESTRATION` to filter noise.
 - **Summaries tell you what the code does** without reading it. Use them to decide what to `Read`.
-- **Use `root` for cross-project search** — absolute path to another indexed directory.
-- **Use `max_per_file`** when results cluster in one file but you need diversity.
+- **Use `--symbol` on CLI** to get search results + call graph in one shot.
 - **Don't search for exact strings** — use grep/Grep for that. gmax finds concepts, not literals.