grepmax 0.21.2 → 0.21.3

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
       "name": "grepmax",
       "source": "./plugins/grepmax",
       "description": "Semantic code search for Claude Code. Automatically indexes your project and provides intelligent search capabilities.",
-      "version": "0.21.2",
+      "version": "0.21.3",
       "author": {
         "name": "Robert Owens",
         "email": "robowens@me.com"

package/dist/config.js

@@ -84,7 +84,7 @@ exports.CONFIG = {
     // reindex to take effect. Must equal the latest entry's `v` in
     // CHUNKER_VERSION_HISTORY below — see that list for per-version severity and
     // the user-facing note rendered by `gmax doctor` and the staleness hint.
-    CHUNKER_VERSION: 3,
+    CHUNKER_VERSION: 4,
 };
 /**
  * Per-version record of what changed in the chunker and how much it matters to
@@ -105,6 +105,11 @@ exports.CHUNKER_VERSION_HISTORY = [
         severity: "additive",
         note: "type-position edges; dead/trace miss type-only callers until reindex.",
     },
+    {
+        v: 4,
+        severity: "additive",
+        note: "member-call edges recorded separately (obj.method()); substrate for receiver-aware call resolution, no query effect until reindex.",
+    },
 ];
 /**
  * Describe the gap between an index's stamped chunker version and the current

package/dist/lib/graph/graph-builder.js

@@ -13,6 +13,7 @@ exports.GraphBuilder = void 0;
 const languages_1 = require("../core/languages");
 const filter_builder_1 = require("../utils/filter-builder");
 const query_timeout_1 = require("../utils/query-timeout");
+const callsites_1 = require("./callsites");
 const graph_traversal_1 = require("./graph-traversal");
 class GraphBuilder {
     constructor(db, pathPrefix, excludePrefixes) {
@@ -130,9 +131,14 @@ class GraphBuilder {
             const callers = yield this.getCallers(symbol, anchorFamily);
             // 3. Get Callees — resolve each to a GraphNode with file:line
             const calleeNames = center ? center.calls.slice(0, 15) : [];
+            const centerFile = center ? center.file : "";
             const calleeNodes = [];
             for (const name of calleeNames) {
                 const esc = (0, filter_builder_1.escapeSqlString)(name);
+                // Pull a few candidates (was .limit(1)) so a callee name defined in more
+                // than one file can prefer the center's OWN file instead of an arbitrary
+                // same-named definition in an unrelated module — the cheapest correct
+                // disambiguation, and strictly better than the old first-row guess.
                 const rows = yield table
                     .query()
                     .where(this.scopeWhere(`array_contains(defined_symbols, '${esc}')`))
@@ -145,12 +151,21 @@ class GraphBuilder {
                     "parent_symbol",
                     "complexity",
                 ])
-                    .limit(1)
+                    .limit(25)
                     .toArray();
                 if (rows.length > 0) {
-                    calleeNodes.push(this.mapRowToNode(rows[0], name, "center"));
+                    // Prefer-self-file: a callee the center also defines locally is almost
+                    // certainly that local definition. Falls back to the first row when no
+                    // candidate is in the center's file (unchanged behaviour then).
+                    const selfRow = rows.find((r) => { var _a; return String((_a = r.path) !== null && _a !== void 0 ? _a : "") === centerFile; });
+                    calleeNodes.push(this.mapRowToNode((selfRow !== null && selfRow !== void 0 ? selfRow : rows[0]), name, "center"));
                 }
-                else {
+                else if (!(0, callsites_1.isBuiltinCallee)(name)) {
+                    // Unresolved + a known builtin (.map/.get/forEach) → suppress the
+                    // phantom callee edge instead of emitting a "(not indexed)" stub.
+                    // Mirrors the display-layer guard (peek.ts: `c.file || !isBuiltinCallee`):
+                    // a project symbol that shadows a builtin name still resolves above and
+                    // is kept, so only genuine builtins are dropped.
                     calleeNodes.push({
                         symbol: name,
                         file: "",
@@ -274,6 +289,11 @@ class GraphBuilder {
             const out = [];
             for (const h of hits) {
                 const loc = yield this.resolveLocation(h.symbol, anchorFamily);
+                // Drop unresolved builtins (.map/.get/forEach) reached via callee edges —
+                // same resolution-aware rule as buildGraph/peek. A neighbor that resolves
+                // to an indexed definition is kept even if it shadows a builtin name.
+                if (!loc && (0, callsites_1.isBuiltinCallee)(h.symbol))
+                    continue;
                 out.push(Object.assign(Object.assign({}, h), { file: (_a = loc === null || loc === void 0 ? void 0 : loc.file) !== null && _a !== void 0 ? _a : "", line: (_b = loc === null || loc === void 0 ? void 0 : loc.line) !== null && _b !== void 0 ? _b : 0 }));
             }
             return out;

package/dist/lib/index/chunker.js

@@ -571,6 +571,15 @@ class TreeSitterChunker {
                             typeReferencedSymbols.push(n);
                         }
                     };
+                    // Member-call names (`obj.method()`), recorded additively alongside
+                    // referencedSymbols (see Chunk.memberReferencedSymbols). Deduped, like
+                    // type refs — a future resolver queries membership, not occurrence count.
+                    const memberReferencedSymbols = [];
+                    const addMemberRef = (n) => {
+                        if (n && !memberReferencedSymbols.includes(n)) {
+                            memberReferencedSymbols.push(n);
+                        }
+                    };
                     // Leaf identifier node types across grammars (a bare name with no
                     // named children — `ErrorCodes`, not `a.ErrorCodes`).
                     const LEAF_ID_TYPES = new Set([
@@ -677,24 +686,33 @@ class TreeSitterChunker {
                                 : null;
                             if (func) {
                                 let funcName = func.text;
+                                let isMemberCall = false;
                                 // Handle member access (obj.method) to extract just 'method'
                                 if (func.type === "member_expression") {
                                     // JS/TS: object.property
                                     const prop = func.childForFieldName
                                         ? func.childForFieldName("property")
                                         : null;
-                                    if (prop)
+                                    if (prop) {
                                         funcName = prop.text;
+                                        isMemberCall = true;
+                                    }
                                 }
                                 else if (func.type === "attribute") {
                                     // Python: object.attribute
                                     const attr = func.childForFieldName
                                         ? func.childForFieldName("attribute")
                                         : null;
-                                    if (attr)
+                                    if (attr) {
                                         funcName = attr.text;
+                                        isMemberCall = true;
+                                    }
                                 }
                                 referencedSymbols.push(funcName);
+                                // Additive: member calls ALSO go here. referencedSymbols is left
+                                // untouched (no recall loss); this just tags which were members.
+                                if (isMemberCall)
+                                    addMemberRef(funcName);
                             }
                             else {
                                 // Swift/Kotlin: call_expression has no "function" field;
@@ -702,15 +720,20 @@ class TreeSitterChunker {
                                 const firstChild = ((_a = n.namedChildren) !== null && _a !== void 0 ? _a : [])[0];
                                 if (firstChild) {
                                     let funcName = firstChild.text;
+                                    let isMemberCall = false;
                                     if (firstChild.type === "navigation_expression") {
                                         const suffix = ((_b = firstChild.namedChildren) !== null && _b !== void 0 ? _b : []).find((c) => c.type === "navigation_suffix");
                                         const methodId = suffix
                                             ? ((_c = suffix.namedChildren) !== null && _c !== void 0 ? _c : []).find((c) => c.type === "simple_identifier")
                                             : null;
-                                        if (methodId)
+                                        if (methodId) {
                                             funcName = methodId.text;
+                                            isMemberCall = true;
+                                        }
                                     }
                                     referencedSymbols.push(funcName);
+                                    if (isMemberCall)
+                                        addMemberRef(funcName);
                                 }
                             }
                         }
@@ -905,6 +928,7 @@ class TreeSitterChunker {
                         definedSymbols,
                         referencedSymbols,
                         typeReferencedSymbols,
+                        memberReferencedSymbols,
                         role,
                         parentSymbol: stack.length > 1
                             ? stack[stack.length - 1].replace(/^(Class|Method|Function|Interface|Type): /, "")
@@ -990,6 +1014,9 @@ class TreeSitterChunker {
         if (sub.typeReferencedSymbols) {
             sub.typeReferencedSymbols = sub.typeReferencedSymbols.filter(occurs);
         }
+        if (sub.memberReferencedSymbols) {
+            sub.memberReferencedSymbols = sub.memberReferencedSymbols.filter(occurs);
+        }
         return sub;
     }
     splitIfTooBig(chunk) {

package/dist/lib/store/vector-db.js

@@ -271,6 +271,7 @@ class VectorDB {
             defined_symbols: [],
             referenced_symbols: [],
             type_referenced_symbols: [],
+            member_referenced_symbols: [],
             imports: [],
             exports: [],
             role: "",
@@ -340,6 +341,7 @@ class VectorDB {
             new apache_arrow_1.Field("defined_symbols", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
             new apache_arrow_1.Field("referenced_symbols", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
             new apache_arrow_1.Field("type_referenced_symbols", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
+            new apache_arrow_1.Field("member_referenced_symbols", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
             new apache_arrow_1.Field("imports", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
             new apache_arrow_1.Field("exports", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true),
             new apache_arrow_1.Field("role", new apache_arrow_1.Utf8(), true),
@@ -360,16 +362,27 @@ class VectorDB {
         return __awaiter(this, void 0, void 0, function* () {
             const schema = yield table.schema();
             const fields = new Set(schema.fields.map((f) => f.name));
-            if (fields.has("type_referenced_symbols"))
-                return;
-            try {
-                yield table.addColumns(new apache_arrow_1.Field("type_referenced_symbols", new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true));
-                (0, logger_1.log)("db", "Added type_referenced_symbols column to existing table");
-            }
-            catch (err) {
-                // Lost a race with another writer that already added it, or a transient
-                // commit conflict — the next ensureTable() re-checks and no-ops.
-                (0, logger_1.debug)("vectordb", `evolveSchema skipped: ${err.message}`);
+            // Additive list columns that older tables may predate. Each is checked and
+            // added INDEPENDENTLY: a single early-return on the first existing column
+            // (as this used to do) would permanently strand every table that already has
+            // `type_referenced_symbols` — i.e. all of them — without ever gaining a newer
+            // column like `member_referenced_symbols`.
+            const additiveListColumns = [
+                "type_referenced_symbols",
+                "member_referenced_symbols",
+            ];
+            for (const col of additiveListColumns) {
+                if (fields.has(col))
+                    continue;
+                try {
+                    yield table.addColumns(new apache_arrow_1.Field(col, new apache_arrow_1.List(new apache_arrow_1.Field("item", new apache_arrow_1.Utf8(), true)), true));
+                    (0, logger_1.log)("db", `Added ${col} column to existing table`);
+                }
+                catch (err) {
+                    // Lost a race with another writer that already added it, or a transient
+                    // commit conflict — the next ensureTable() re-checks and no-ops.
+                    (0, logger_1.debug)("vectordb", `evolveSchema ${col} skipped: ${err.message}`);
+                }
             }
         });
     }
@@ -395,7 +408,7 @@ class VectorDB {
     }
     insertBatch(records) {
         return __awaiter(this, void 0, void 0, function* () {
-            var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q;
+            var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o, _p, _q, _r;
             if (!records.length)
                 return;
             this.ensureDiskOk();
@@ -473,12 +486,14 @@ class VectorDB {
                 rec.defined_symbols = (_g = rec.defined_symbols) !== null && _g !== void 0 ? _g : [];
                 rec.referenced_symbols = (_h = rec.referenced_symbols) !== null && _h !== void 0 ? _h : [];
                 rec.type_referenced_symbols = (_j = rec.type_referenced_symbols) !== null && _j !== void 0 ? _j : [];
-                rec.imports = (_k = rec.imports) !== null && _k !== void 0 ? _k : [];
-                rec.exports = (_l = rec.exports) !== null && _l !== void 0 ? _l : [];
-                rec.role = (_m = rec.role) !== null && _m !== void 0 ? _m : "";
-                rec.parent_symbol = (_o = rec.parent_symbol) !== null && _o !== void 0 ? _o : "";
-                rec.file_skeleton = (_p = rec.file_skeleton) !== null && _p !== void 0 ? _p : "";
-                rec.summary = (_q = rec.summary) !== null && _q !== void 0 ? _q : null;
+                rec.member_referenced_symbols =
+                    (_k = rec.member_referenced_symbols) !== null && _k !== void 0 ? _k : [];
+                rec.imports = (_l = rec.imports) !== null && _l !== void 0 ? _l : [];
+                rec.exports = (_m = rec.exports) !== null && _m !== void 0 ? _m : [];
+                rec.role = (_o = rec.role) !== null && _o !== void 0 ? _o : "";
+                rec.parent_symbol = (_p = rec.parent_symbol) !== null && _p !== void 0 ? _p : "";
+                rec.file_skeleton = (_q = rec.file_skeleton) !== null && _q !== void 0 ? _q : "";
+                rec.summary = (_r = rec.summary) !== null && _r !== void 0 ? _r : null;
             }
             try {
                 yield this.withWriteGate(() => table.add(records));

package/dist/lib/workers/orchestrator.js

@@ -227,6 +227,7 @@ class WorkerOrchestrator {
                 defined_symbols: chunk.definedSymbols,
                 referenced_symbols: chunk.referencedSymbols,
                 type_referenced_symbols: chunk.typeReferencedSymbols,
+                member_referenced_symbols: chunk.memberReferencedSymbols,
                 role: chunk.role,
                 parent_symbol: chunk.parentSymbol,
                 file_skeleton: chunk.isAnchor ? skeleton : undefined,

package/package.json

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