homegraph 1.5.1 → 1.5.2

package/dist/db/queries.js

@@ -255,20 +255,20 @@ class QueryBuilder {
      */
     insertNode(node) {
         if (!this.stmts.insertNode) {
-            this.stmts.insertNode = this.db.prepare(`
-        INSERT OR REPLACE INTO nodes (
-          id, kind, name, qualified_name, file_path, language,
-          start_line, end_line, start_column, end_column,
-          docstring, signature, visibility,
-          is_exported, is_async, is_static, is_abstract,
-          decorators, type_parameters, return_type, updated_at
-        ) VALUES (
-          @id, @kind, @name, @qualifiedName, @filePath, @language,
-          @startLine, @endLine, @startColumn, @endColumn,
-          @docstring, @signature, @visibility,
-          @isExported, @isAsync, @isStatic, @isAbstract,
-          @decorators, @typeParameters, @returnType, @updatedAt
-        )
+            this.stmts.insertNode = this.db.prepare(`
+        INSERT OR REPLACE INTO nodes (
+          id, kind, name, qualified_name, file_path, language,
+          start_line, end_line, start_column, end_column,
+          docstring, signature, visibility,
+          is_exported, is_async, is_static, is_abstract,
+          decorators, type_parameters, return_type, updated_at
+        ) VALUES (
+          @id, @kind, @name, @qualifiedName, @filePath, @language,
+          @startLine, @endLine, @startColumn, @endColumn,
+          @docstring, @signature, @visibility,
+          @isExported, @isAsync, @isStatic, @isAbstract,
+          @decorators, @typeParameters, @returnType, @updatedAt
+        )
       `);
         }
         // Validate required fields to prevent SQLite bind errors
@@ -364,29 +364,29 @@ class QueryBuilder {
      */
     updateNode(node) {
         if (!this.stmts.updateNode) {
-            this.stmts.updateNode = this.db.prepare(`
-        UPDATE nodes SET
-          kind = @kind,
-          name = @name,
-          qualified_name = @qualifiedName,
-          file_path = @filePath,
-          language = @language,
-          start_line = @startLine,
-          end_line = @endLine,
-          start_column = @startColumn,
-          end_column = @endColumn,
-          docstring = @docstring,
-          signature = @signature,
-          visibility = @visibility,
-          is_exported = @isExported,
-          is_async = @isAsync,
-          is_static = @isStatic,
-          is_abstract = @isAbstract,
-          decorators = @decorators,
-          type_parameters = @typeParameters,
-          return_type = @returnType,
-          updated_at = @updatedAt
-        WHERE id = @id
+            this.stmts.updateNode = this.db.prepare(`
+        UPDATE nodes SET
+          kind = @kind,
+          name = @name,
+          qualified_name = @qualifiedName,
+          file_path = @filePath,
+          language = @language,
+          start_line = @startLine,
+          end_line = @endLine,
+          start_column = @startColumn,
+          end_column = @endColumn,
+          docstring = @docstring,
+          signature = @signature,
+          visibility = @visibility,
+          is_exported = @isExported,
+          is_async = @isAsync,
+          is_static = @isStatic,
+          is_abstract = @isAbstract,
+          decorators = @decorators,
+          type_parameters = @typeParameters,
+          return_type = @returnType,
+          updated_at = @updatedAt
+        WHERE id = @id
       `);
         }
         // Invalidate cache before update
@@ -518,12 +518,12 @@ class QueryBuilder {
         const placeholders = variants.map(() => '?').join(', ');
         const whens = variants.map(() => 'WHEN ? THEN ?').join(' ');
         const rows = this.db
-            .prepare(`SELECT name, COUNT(DISTINCT CASE segment ${whens} END) AS matches
-         FROM name_segment_vocab
-         WHERE segment IN (${placeholders})
-         GROUP BY name
-         HAVING matches >= ?
-         ORDER BY matches DESC, length(name) ASC
+            .prepare(`SELECT name, COUNT(DISTINCT CASE segment ${whens} END) AS matches
+         FROM name_segment_vocab
+         WHERE segment IN (${placeholders})
+         GROUP BY name
+         HAVING matches >= ?
+         ORDER BY matches DESC, length(name) ASC
          LIMIT ?`)
             .all(...variants.flatMap((v) => [v.segment, v.word]), ...variants.map((v) => v.segment), minWords, limit);
         return rows;
@@ -535,7 +535,7 @@ class QueryBuilder {
             return new Map();
         const placeholders = segments.map(() => '?').join(', ');
         const rows = this.db
-            .prepare(`SELECT segment, COUNT(*) AS n FROM name_segment_vocab
+            .prepare(`SELECT segment, COUNT(*) AS n FROM name_segment_vocab
          WHERE segment IN (${placeholders}) GROUP BY segment`)
             .all(...segments);
         return new Map(rows.map((r) => [r.segment, r.n]));
@@ -695,15 +695,15 @@ class QueryBuilder {
             // protobuf stub) outranks the real `server/etcdserver/server.go`
             // (470 edges) by 4×, and the boost would push the agent toward
             // generated code.
-            this.stmts.getDominantFile = this.db.prepare(`
-        SELECT n.file_path AS file_path, COUNT(*) AS edge_count
-        FROM edges e
-        JOIN nodes n ON e.source = n.id
-        JOIN nodes m ON e.target = m.id
-        WHERE n.file_path = m.file_path
-        GROUP BY n.file_path
-        ORDER BY edge_count DESC
-        LIMIT 20
+            this.stmts.getDominantFile = this.db.prepare(`
+        SELECT n.file_path AS file_path, COUNT(*) AS edge_count
+        FROM edges e
+        JOIN nodes n ON e.source = n.id
+        JOIN nodes m ON e.target = m.id
+        WHERE n.file_path = m.file_path
+        GROUP BY n.file_path
+        ORDER BY edge_count DESC
+        LIMIT 20
       `);
         }
         const rows = this.stmts.getDominantFile.all();
@@ -730,13 +730,13 @@ class QueryBuilder {
      */
     getTopRouteFile() {
         if (!this.stmts.getTopRouteFile) {
-            this.stmts.getTopRouteFile = this.db.prepare(`
-        SELECT file_path, COUNT(*) AS cnt
-        FROM nodes
-        WHERE kind = 'route'
-        GROUP BY file_path
-        ORDER BY cnt DESC
-        LIMIT 20
+            this.stmts.getTopRouteFile = this.db.prepare(`
+        SELECT file_path, COUNT(*) AS cnt
+        FROM nodes
+        WHERE kind = 'route'
+        GROUP BY file_path
+        ORDER BY cnt DESC
+        LIMIT 20
       `);
         }
         const rows = this.stmts.getTopRouteFile.all();
@@ -767,21 +767,21 @@ class QueryBuilder {
             // Edge kind varies across framework resolvers: Spring/Rails/
             // Laravel/Drupal emit `references`, Express emits `calls`. Accept
             // both — the semantic is the same (route → its handler).
-            this.stmts.getRoutingManifest = this.db.prepare(`
-        SELECT
-          r.name AS url,
-          h.name AS handler,
-          h.file_path AS handler_file,
-          h.start_line AS handler_line,
-          h.kind AS handler_kind
-        FROM nodes r
-        JOIN edges e ON e.source = r.id
-        JOIN nodes h ON e.target = h.id
-        WHERE r.kind = 'route'
-          AND e.kind IN ('references', 'calls')
-          AND h.kind IN ('function', 'method', 'class')
-        ORDER BY r.file_path, r.start_line
-        LIMIT ?
+            this.stmts.getRoutingManifest = this.db.prepare(`
+        SELECT
+          r.name AS url,
+          h.name AS handler,
+          h.file_path AS handler_file,
+          h.start_line AS handler_line,
+          h.kind AS handler_kind
+        FROM nodes r
+        JOIN edges e ON e.source = r.id
+        JOIN nodes h ON e.target = h.id
+        WHERE r.kind = 'route'
+          AND e.kind IN ('references', 'calls')
+          AND h.kind IN ('function', 'method', 'class')
+        ORDER BY r.file_path, r.start_line
+        LIMIT ?
       `);
         }
         const rows = this.stmts.getRoutingManifest.all(limit);
@@ -1036,10 +1036,10 @@ class QueryBuilder {
         const seen = new Set(existing.map((r) => r.node.id));
         const merged = [...existing];
         const apiLimit = Math.min(20, Math.max(5, Math.floor(limit / 4)));
-        let sql = `
-      SELECT nodes.*, 0.9 as score
-      FROM ohos_api.nodes
-      WHERE (name LIKE ? OR qualified_name LIKE ?)
+        let sql = `
+      SELECT nodes.*, 0.9 as score
+      FROM ohos_api.nodes
+      WHERE (name LIKE ? OR qualified_name LIKE ?)
     `;
         const pattern = `%${text}%`;
         const params = [pattern, pattern];
@@ -1178,11 +1178,11 @@ class QueryBuilder {
         // Fetch 5x requested limit so post-hoc rescoring (kindBonus, pathRelevance,
         // nameMatchBonus) can promote results that BM25 alone undervalues.
         const ftsLimit = Math.max(limit * 5, 100);
-        let sql = `
-      SELECT nodes.*, bm25(nodes_fts, 0, 20, 5, 1, 2) as score
-      FROM nodes_fts
-      JOIN nodes ON nodes_fts.id = nodes.id
-      WHERE nodes_fts MATCH ?
+        let sql = `
+      SELECT nodes.*, bm25(nodes_fts, 0, 20, 5, 1, 2) as score
+      FROM nodes_fts
+      JOIN nodes ON nodes_fts.id = nodes.id
+      WHERE nodes_fts MATCH ?
     `;
         const params = [ftsQuery];
         if (kinds && kinds.length > 0) {
@@ -1213,21 +1213,21 @@ class QueryBuilder {
      */
     searchNodesLike(query, options) {
         const { kinds, languages, limit = 100, offset = 0 } = options;
-        let sql = `
-      SELECT nodes.*,
-        CASE
-          WHEN name = ? THEN 1.0
-          WHEN name LIKE ? THEN 0.9
-          WHEN name LIKE ? THEN 0.8
-          WHEN qualified_name LIKE ? THEN 0.7
-          ELSE 0.5
-        END as score
-      FROM nodes
-      WHERE (
-        name LIKE ? OR
-        qualified_name LIKE ? OR
-        name LIKE ?
-      )
+        let sql = `
+      SELECT nodes.*,
+        CASE
+          WHEN name = ? THEN 1.0
+          WHEN name LIKE ? THEN 0.9
+          WHEN name LIKE ? THEN 0.8
+          WHEN qualified_name LIKE ? THEN 0.7
+          ELSE 0.5
+        END as score
+      FROM nodes
+      WHERE (
+        name LIKE ? OR
+        qualified_name LIKE ? OR
+        name LIKE ?
+      )
     `;
         // Pattern variants for better matching
         const exactMatch = query;
@@ -1301,10 +1301,10 @@ class QueryBuilder {
         const allResults = [];
         const seenIds = new Set();
         for (const name of names) {
-            let sql = `
-        SELECT nodes.*, 1.0 as score
-        FROM nodes
-        WHERE name COLLATE NOCASE = ?
+            let sql = `
+        SELECT nodes.*, 1.0 as score
+        FROM nodes
+        WHERE name COLLATE NOCASE = ?
       `;
             const params = [name];
             if (kinds && kinds.length > 0) {
@@ -1335,10 +1335,10 @@ class QueryBuilder {
                 allResults.push(r);
             }
             if (this.hasOhosApiAttached()) {
-                let apiSql = `
-          SELECT nodes.*, 0.95 as score
-          FROM ohos_api.nodes
-          WHERE name COLLATE NOCASE = ?
+                let apiSql = `
+          SELECT nodes.*, 0.95 as score
+          FROM ohos_api.nodes
+          WHERE name COLLATE NOCASE = ?
         `;
                 const apiParams = [name];
                 if (kinds && kinds.length > 0) {
@@ -1374,10 +1374,10 @@ class QueryBuilder {
      */
     findNodesByNameSubstring(substring, options = {}) {
         const { kinds, languages, limit = 30, excludePrefix } = options;
-        let sql = `
-      SELECT nodes.*, 1.0 as score
-      FROM nodes
-      WHERE name LIKE ?
+        let sql = `
+      SELECT nodes.*, 1.0 as score
+      FROM nodes
+      WHERE name LIKE ?
     `;
         const params = [`%${substring}%`];
         // Exclude prefix matches (handled by FTS-based prefix search in Step 2b)
@@ -1409,9 +1409,9 @@ class QueryBuilder {
      */
     insertEdge(edge) {
         if (!this.stmts.insertEdge) {
-            this.stmts.insertEdge = this.db.prepare(`
-        INSERT OR IGNORE INTO edges (source, target, kind, metadata, line, col, provenance)
-        VALUES (@source, @target, @kind, @metadata, @line, @col, @provenance)
+            this.stmts.insertEdge = this.db.prepare(`
+        INSERT OR IGNORE INTO edges (source, target, kind, metadata, line, col, provenance)
+        VALUES (@source, @target, @kind, @metadata, @line, @col, @provenance)
       `);
         }
         this.stmts.insertEdge.run({
@@ -1538,12 +1538,12 @@ class QueryBuilder {
      * it. One indexed query (idx_nodes_file_path + idx_edges_target_kind).
      */
     getDependentFilePaths(filePath) {
-        const sql = `SELECT DISTINCT src.file_path AS fp
-      FROM edges e
-      JOIN nodes tgt ON tgt.id = e.target
-      JOIN nodes src ON src.id = e.source
-      WHERE tgt.file_path = ?
-        AND e.kind != 'contains'
+        const sql = `SELECT DISTINCT src.file_path AS fp
+      FROM edges e
+      JOIN nodes tgt ON tgt.id = e.target
+      JOIN nodes src ON src.id = e.source
+      WHERE tgt.file_path = ?
+        AND e.kind != 'contains'
         AND src.file_path != ?`;
         const rows = this.db.prepare(sql).all(filePath, filePath);
         return rows.map((r) => r.fp);
@@ -1555,12 +1555,12 @@ class QueryBuilder {
      * (all kinds except `contains`); same reason imports-only is insufficient.
      */
     getDependencyFilePaths(filePath) {
-        const sql = `SELECT DISTINCT tgt.file_path AS fp
-      FROM edges e
-      JOIN nodes src ON src.id = e.source
-      JOIN nodes tgt ON tgt.id = e.target
-      WHERE src.file_path = ?
-        AND e.kind != 'contains'
+        const sql = `SELECT DISTINCT tgt.file_path AS fp
+      FROM edges e
+      JOIN nodes src ON src.id = e.source
+      JOIN nodes tgt ON tgt.id = e.target
+      WHERE src.file_path = ?
+        AND e.kind != 'contains'
         AND tgt.file_path != ?`;
         const rows = this.db.prepare(sql).all(filePath, filePath);
         return rows.map((r) => r.fp);
@@ -1576,13 +1576,13 @@ class QueryBuilder {
      * {@link getDependentFilePaths}: all kinds except `contains`.
      */
     getCrossFileIncomingEdgesWithTarget(filePath) {
-        const sql = `SELECT e.*, tgt.name AS target_name, tgt.kind AS target_kind,
-        src.file_path AS source_file_path, src.language AS source_language
-      FROM edges e
-      JOIN nodes tgt ON tgt.id = e.target
-      JOIN nodes src ON src.id = e.source
-      WHERE tgt.file_path = ?
-        AND e.kind != 'contains'
+        const sql = `SELECT e.*, tgt.name AS target_name, tgt.kind AS target_kind,
+        src.file_path AS source_file_path, src.language AS source_language
+      FROM edges e
+      JOIN nodes tgt ON tgt.id = e.target
+      JOIN nodes src ON src.id = e.source
+      WHERE tgt.file_path = ?
+        AND e.kind != 'contains'
         AND src.file_path != ?`;
         const rows = this.db.prepare(sql).all(filePath, filePath);
         return rows.map(row => ({
@@ -1599,10 +1599,10 @@ class QueryBuilder {
      */
     deleteArkTSCrossFileCallEdges() {
         const result = this.db
-            .prepare(`DELETE FROM edges
-         WHERE kind = 'calls'
-           AND provenance = 'heuristic'
-           AND metadata LIKE '%"synthesizedBy":"arkanalyzer"%'
+            .prepare(`DELETE FROM edges
+         WHERE kind = 'calls'
+           AND provenance = 'heuristic'
+           AND metadata LIKE '%"synthesizedBy":"arkanalyzer"%'
            AND metadata LIKE '%"sourceFile"%'`)
             .run();
         return result.changes;
@@ -1615,17 +1615,17 @@ class QueryBuilder {
      */
     upsertFile(file) {
         if (!this.stmts.upsertFile) {
-            this.stmts.upsertFile = this.db.prepare(`
-        INSERT INTO files (path, content_hash, language, size, modified_at, indexed_at, node_count, errors)
-        VALUES (@path, @contentHash, @language, @size, @modifiedAt, @indexedAt, @nodeCount, @errors)
-        ON CONFLICT(path) DO UPDATE SET
-          content_hash = @contentHash,
-          language = @language,
-          size = @size,
-          modified_at = @modifiedAt,
-          indexed_at = @indexedAt,
-          node_count = @nodeCount,
-          errors = @errors
+            this.stmts.upsertFile = this.db.prepare(`
+        INSERT INTO files (path, content_hash, language, size, modified_at, indexed_at, node_count, errors)
+        VALUES (@path, @contentHash, @language, @size, @modifiedAt, @indexedAt, @nodeCount, @errors)
+        ON CONFLICT(path) DO UPDATE SET
+          content_hash = @contentHash,
+          language = @language,
+          size = @size,
+          modified_at = @modifiedAt,
+          indexed_at = @indexedAt,
+          node_count = @nodeCount,
+          errors = @errors
       `);
         }
         this.stmts.upsertFile.run({
@@ -1699,9 +1699,9 @@ class QueryBuilder {
      */
     insertUnresolvedRef(ref) {
         if (!this.stmts.insertUnresolved) {
-            this.stmts.insertUnresolved = this.db.prepare(`
-        INSERT INTO unresolved_refs (from_node_id, reference_name, reference_kind, line, col, candidates, file_path, language)
-        VALUES (@fromNodeId, @referenceName, @referenceKind, @line, @col, @candidates, @filePath, @language)
+            this.stmts.insertUnresolved = this.db.prepare(`
+        INSERT INTO unresolved_refs (from_node_id, reference_name, reference_kind, line, col, candidates, file_path, language)
+        VALUES (@fromNodeId, @referenceName, @referenceKind, @line, @col, @candidates, @filePath, @language)
       `);
         }
         this.stmts.insertUnresolved.run({
@@ -2085,11 +2085,11 @@ class QueryBuilder {
      */
     getStats() {
         // Single query for all three aggregate counts
-        const counts = this.db.prepare(`
-      SELECT
-        (SELECT COUNT(*) FROM nodes) AS node_count,
-        (SELECT COUNT(*) FROM edges) AS edge_count,
-        (SELECT COUNT(*) FROM files) AS file_count
+        const counts = this.db.prepare(`
+      SELECT
+        (SELECT COUNT(*) FROM nodes) AS node_count,
+        (SELECT COUNT(*) FROM edges) AS edge_count,
+        (SELECT COUNT(*) FROM files) AS file_count
     `).get();
         const nodesByKind = {};
         const nodeKindRows = this.db