grepmax 0.26.27 → 0.26.28

package/dist/commands/impact.js

@@ -45,6 +45,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.impact = void 0;
 const path = __importStar(require("node:path"));
 const commander_1 = require("commander");
+const graph_handler_1 = require("../lib/daemon/graph-handler");
 const impact_1 = require("../lib/graph/impact");
 const impact_rollup_1 = require("../lib/graph/impact-rollup");
 const test_hits_1 = require("../lib/graph/test-hits");
@@ -54,6 +55,7 @@ const exit_1 = require("../lib/utils/exit");
 const project_registry_1 = require("../lib/utils/project-registry");
 const project_root_1 = require("../lib/utils/project-root");
 const stale_hint_1 = require("../lib/utils/stale-hint");
+const store_access_1 = require("../lib/utils/store-access");
 exports.impact = new commander_1.Command("impact")
     .description("Analyze change impact: dependents and affected tests")
     .argument("<target>", "Symbol name or file path")
@@ -72,7 +74,8 @@ exports.impact = new commander_1.Command("impact")
     // commander maps --no-tests → opts.tests === false (defaults true).
     const includeTests = opts.tests !== false;
     const top = Math.min(Math.max(Number.parseInt(opts.top || "10", 10) || 10, 1), 100);
-    let vectorDb = null;
+    // Opened only if the daemon is unreachable; withStoreRead decides.
+    const local = { db: null };
     try {
         const root = (0, project_registry_1.resolveRootOrExit)(opts.root);
         if (root === null)
@@ -80,9 +83,31 @@ exports.impact = new commander_1.Command("impact")
         const projectRoot = (_a = (0, project_root_1.findProjectRoot)(root)) !== null && _a !== void 0 ? _a : root;
         (0, stale_hint_1.maybeWarnStaleChunker)(projectRoot, { agent: opts.agent });
         (0, stale_hint_1.maybeWarnStaleEmbedding)(projectRoot, { agent: opts.agent });
-        const paths = (0, project_root_1.ensureProjectPaths)(projectRoot);
-        vectorDb = new vector_db_1.VectorDB(paths.lancedbDir);
-        const { symbols, resolvedAsFile, symbolFamilies } = yield (0, impact_1.resolveTargetSymbols)(target, vectorDb, projectRoot);
+        const localDb = () => {
+            var _a;
+            (_a = local.db) !== null && _a !== void 0 ? _a : (local.db = new vector_db_1.VectorDB((0, project_root_1.ensureProjectPaths)(projectRoot).lancedbDir));
+            return local.db;
+        };
+        const { resolveScope } = yield Promise.resolve().then(() => __importStar(require("../lib/utils/scope-filter")));
+        const scope = resolveScope({
+            projectRoot,
+            in: opts.in,
+            exclude: opts.exclude,
+        });
+        const { symbols, resolvedAsFile, symbolFamilies } = yield (0, graph_handler_1.callGraphVerb)("graph.resolve", {
+            projectRoot,
+            scope,
+            payload: { target },
+            render: (resp) => {
+                var _a, _b;
+                return ({
+                    symbols: (_a = resp.symbols) !== null && _a !== void 0 ? _a : [],
+                    resolvedAsFile: resp.resolvedAsFile === true,
+                    symbolFamilies: (_b = resp.symbolFamilies) !== null && _b !== void 0 ? _b : null,
+                });
+            },
+            inProcess: () => (0, graph_handler_1.runGraphResolve)(localDb(), { target, projectRoot }),
+        });
         if (symbols.length === 0) {
             console.log(resolvedAsFile
                 ? `No symbols found in file: ${target}`
@@ -96,13 +121,7 @@ exports.impact = new commander_1.Command("impact")
                 ? target
                 : path.resolve(projectRoot, target)
             : undefined;
-        const excludePaths = targetPath ? new Set([targetPath]) : undefined;
-        const { resolveScope } = yield Promise.resolve().then(() => __importStar(require("../lib/utils/scope-filter")));
-        const scope = resolveScope({
-            projectRoot,
-            in: opts.in,
-            exclude: opts.exclude,
-        });
+        const excludePaths = targetPath ? [targetPath] : undefined;
         // Treat --in as an exclude-everything-else when set: any prefix that
         // isn't the --in scope becomes effectively excluded. Today findDependents
         // always queries within projectRoot; passing scope.pathPrefix when --in
@@ -112,15 +131,46 @@ exports.impact = new commander_1.Command("impact")
             : projectRoot;
         const useRollup = !opts.flat && ((resolvedAsFile && !opts.agent) || opts.rollup === true);
         const rollupLimit = Math.min(Math.max(top * 10, 100), 500);
+        const families = (0, graph_handler_1.decodeSymbolFamilies)(symbolFamilies);
         // Run dependents and tests in parallel. --no-tests skips the test
         // traversal entirely so the affected-tests section is omitted (not just
         // empty) below.
         const [dependents, tests] = yield Promise.all([
-            useRollup
-                ? (0, impact_1.findDependentsDetailed)(symbols, vectorDb, queryRoot, excludePaths, rollupLimit, scope.excludePrefixes, symbolFamilies)
-                : (0, impact_1.findDependents)(symbols, vectorDb, queryRoot, excludePaths, undefined, scope.excludePrefixes, symbolFamilies),
+            (0, graph_handler_1.callGraphVerb)("graph.dependents", {
+                projectRoot,
+                scope,
+                payload: {
+                    symbols,
+                    detailed: useRollup,
+                    excludePaths,
+                    limit: useRollup ? rollupLimit : undefined,
+                    families: symbolFamilies,
+                },
+                render: (resp) => { var _a; return (_a = resp.dependents) !== null && _a !== void 0 ? _a : []; },
+                inProcess: () => (0, graph_handler_1.runGraphDependents)(localDb(), {
+                    symbols,
+                    queryRoot,
+                    detailed: useRollup,
+                    excludePaths,
+                    limit: useRollup ? rollupLimit : undefined,
+                    excludePrefixes: scope.excludePrefixes,
+                    families,
+                }),
+            }),
             includeTests
-                ? (0, impact_1.findTests)(symbols, vectorDb, queryRoot, depth, scope.excludePrefixes, symbolFamilies)
+                ? (0, graph_handler_1.callGraphVerb)("graph.tests", {
+                    projectRoot,
+                    scope,
+                    payload: { symbols, depth, families: symbolFamilies },
+                    render: (resp) => { var _a; return (_a = resp.hits) !== null && _a !== void 0 ? _a : []; },
+                    inProcess: () => (0, graph_handler_1.runGraphTests)(localDb(), {
+                        symbols,
+                        queryRoot,
+                        depth,
+                        excludePrefixes: scope.excludePrefixes,
+                        families,
+                    }),
+                })
                 : Promise.resolve([]),
         ]);
         if (useRollup) {
@@ -188,14 +238,18 @@ exports.impact = new commander_1.Command("impact")
         }
     }
     catch (error) {
-        const msg = error instanceof Error ? error.message : "Unknown error";
-        console.error("Impact analysis failed:", msg);
-        process.exitCode = 1;
+        // A sandbox refusal is already one actionable line; the "Impact analysis
+        // failed:" prefix would bury the settings key that fixes it.
+        if (!(0, store_access_1.reportStoreAccessRefusal)(error)) {
+            const msg = error instanceof Error ? error.message : "Unknown error";
+            console.error("Impact analysis failed:", msg);
+            process.exitCode = 1;
+        }
     }
     finally {
-        if (vectorDb) {
+        if (local.db) {
             try {
-                yield vectorDb.close();
+                yield local.db.close();
             }
             catch (_b) { }
         }

package/dist/commands/log.js

@@ -46,12 +46,12 @@ exports.log = void 0;
 const fs = __importStar(require("node:fs"));
 const path = __importStar(require("node:path"));
 const commander_1 = require("commander");
-const vector_db_1 = require("../lib/store/vector-db");
+const rows_handler_1 = require("../lib/daemon/rows-handler");
 const exit_1 = require("../lib/utils/exit");
-const filter_builder_1 = require("../lib/utils/filter-builder");
 const git_1 = require("../lib/utils/git");
 const project_registry_1 = require("../lib/utils/project-registry");
 const project_root_1 = require("../lib/utils/project-root");
+const store_access_1 = require("../lib/utils/store-access");
 const useColors = process.stdout.isTTY && !process.env.NO_COLOR;
 const style = {
     bold: (s) => (useColors ? `\x1b[1m${s}\x1b[22m` : s),
@@ -63,24 +63,30 @@ function relativize(p, projectRoot) {
     const prefix = projectRoot.endsWith("/") ? projectRoot : `${projectRoot}/`;
     return p.startsWith(prefix) ? p.slice(prefix.length) : p;
 }
-function resolveSymbolPaths(vectorDb, symbol, projectRoot, inOpt, excludeOpt) {
+/**
+ * The one store read `log` does: symbol → defining files. Everything else —
+ * cwd-relative path resolution and `git log` itself — stays in this process,
+ * because both are tied to where the user is standing.
+ */
+function resolveSymbolPaths(symbol, projectRoot, lancedbDir, inOpt, excludeOpt) {
     return __awaiter(this, void 0, void 0, function* () {
-        const { resolveScope, buildScopeWhere } = yield Promise.resolve().then(() => __importStar(require("../lib/utils/scope-filter")));
+        const { resolveScope } = yield Promise.resolve().then(() => __importStar(require("../lib/utils/scope-filter")));
         const scope = resolveScope({
             projectRoot,
             in: inOpt,
             exclude: excludeOpt,
         });
-        const where = buildScopeWhere(scope, `array_contains(defined_symbols, '${(0, filter_builder_1.escapeSqlString)(symbol)}')`);
-        const table = yield vectorDb.ensureTable();
-        const rows = yield table
-            .query()
-            .select(["path"])
-            .where(where)
-            .limit(50)
-            .toArray();
+        const [rows] = yield (0, rows_handler_1.readRows)({
+            name: "log",
+            projectRoot,
+            lancedbDir,
+            scope,
+            select: ["path"],
+            matches: [{ kind: "definedSymbol", symbol }],
+            limit: 50,
+        });
         const paths = new Set();
-        for (const row of rows) {
+        for (const row of rows !== null && rows !== void 0 ? rows : []) {
             const p = String(row.path || "");
             if (p)
                 paths.add(p);
@@ -141,7 +147,6 @@ exports.log = new commander_1.Command("log")
     if (root === null)
         return;
     const projectRoot = (_a = (0, project_root_1.findProjectRoot)(root)) !== null && _a !== void 0 ? _a : root;
-    let vectorDb = null;
     try {
         // 1. Try arg as path (relative to projectRoot, then cwd).
         const candidates = [
@@ -180,8 +185,7 @@ exports.log = new commander_1.Command("log")
         }
         // 2. Try arg as symbol via index lookup.
         const paths = (0, project_root_1.ensureProjectPaths)(projectRoot);
-        vectorDb = new vector_db_1.VectorDB(paths.lancedbDir);
-        const symbolPaths = yield resolveSymbolPaths(vectorDb, arg, projectRoot, opts.in, opts.exclude);
+        const symbolPaths = yield resolveSymbolPaths(arg, projectRoot, paths.lancedbDir, opts.in, opts.exclude);
         if (symbolPaths.length === 0) {
             console.error(`gmax log: no file or symbol matched '${arg}'`);
             if (!opts.agent) {
@@ -217,17 +221,13 @@ exports.log = new commander_1.Command("log")
         }
     }
     catch (error) {
-        const msg = error instanceof Error ? error.message : "Unknown error";
-        console.error("Log failed:", msg);
-        process.exitCode = 1;
+        if (!(0, store_access_1.reportStoreAccessRefusal)(error)) {
+            const msg = error instanceof Error ? error.message : "Unknown error";
+            console.error("Log failed:", msg);
+            process.exitCode = 1;
+        }
     }
     finally {
-        if (vectorDb) {
-            try {
-                yield vectorDb.close();
-            }
-            catch (_b) { }
-        }
         yield (0, exit_1.gracefulExit)();
     }
 }));