homegraph 1.4.1 → 1.5.0

package/dist/mcp/tools.js

@@ -40,6 +40,7 @@ const VIEWTREE_STRUCTURE_VIAS = new Set([
     'builder',
     'builder-param',
 ]);
+const sqlite_adapter_1 = require("../db/sqlite-adapter");
 /**
  * An expected, recoverable "homegraph can't serve this" condition — most
  * importantly a project with no index. The dispatch catch converts these to
@@ -713,10 +714,6 @@ exports.tools = [
                     description: 'Whether to include full code diffs per commit (default: true).',
                     default: true,
                 },
-                dbPath: {
-                    type: 'string',
-                    description: 'Explicit path to the Commit4Spec database. Overrides repoPath-based resolution.',
-                },
             },
             required: ['query'],
         },
@@ -739,10 +736,6 @@ exports.tools = [
                     type: 'string',
                     description: 'Path to the repository root. Defaults to the current working directory.',
                 },
-                dbPath: {
-                    type: 'string',
-                    description: 'Explicit path to the Commit4Spec database. Overrides repoPath-based resolution.',
-                },
             },
             required: ['filePath'],
         },
@@ -781,10 +774,6 @@ exports.tools = [
                     description: 'Maximum number of matching Specs to return (default: 10).',
                     default: 10,
                 },
-                dbPath: {
-                    type: 'string',
-                    description: 'Explicit path to the Commit4Spec database. Overrides repoPath-based resolution.',
-                },
             },
             required: ['symbol'],
         },
@@ -4545,7 +4534,7 @@ class ToolHandler {
                 // homegraph_node's findSymbolMatches.) Qualified tokens keep findAllSymbols.
                 const isQual = /[.\/]|::/.test(t);
                 const raw = isQual ? this.findAllSymbols(cg, t).nodes : cg.getNodesByName(t);
-                const cands = raw
+                let cands = raw
                     .filter((n) => SEED_KINDS.has(n.kind) && !isTestPath(n.filePath))
                     .sort((a, b) => {
                     // Prefer callables over types when both share a name, then body size.
@@ -4555,6 +4544,28 @@ class ToolHandler {
                         return bc - ac;
                     return (bodyLines(b) > 1 ? 1 : 0) - (bodyLines(a) > 1 ? 1 : 0) || bodyLines(b) - bodyLines(a);
                 });
+                // Field-name seeding fallback (#1196): a camelCase token that names NO
+                // definition of its own is usually an object-literal key / API field —
+                // seed its camel-infix callable definers instead.
+                if (cands.length === 0 && !isQual && /[a-z][A-Z]/.test(t)) {
+                    const lcToken = t.toLowerCase();
+                    cands = cg
+                        .getNodesByNameSubstring(t, {
+                        kinds: ['function', 'method', 'component'],
+                        limit: 60,
+                    })
+                        .filter((n) => CALLABLE.has(n.kind) && !isTestPath(n.filePath))
+                        .filter((n) => {
+                        const idx = n.name.toLowerCase().indexOf(lcToken);
+                        if (idx < 0)
+                            return false;
+                        if (idx === 0)
+                            return n.name.length > t.length;
+                        return /[A-Z]/.test(n.name.charAt(idx));
+                    })
+                        .sort((a, b) => a.name.length - b.name.length)
+                        .slice(0, 3);
+                }
                 // A specific name (<=3 defs) injects all its defs. An overloaded name
                 // (`validate` = 10, `request` = 44) would flood the subgraph, so inject
                 // only: the overloads whose file/class the query ALSO names (the agent
@@ -6248,20 +6259,28 @@ class ToolHandler {
             lines.push(`> ⚠ ${(0, worktree_1.worktreeMismatchWarning)(mismatch).replace(/\n/g, '\n> ')}`, '');
         }
         lines.push(`**Files indexed:** ${stats.fileCount}`, `**Total nodes:** ${stats.nodeCount}`, `**Total edges:** ${stats.edgeCount}`, `**Database size:** ${(stats.dbSizeBytes / 1024 / 1024).toFixed(2)} MB`);
-        // Surface the active SQLite backend (node:sqlite, Node's built-in real
-        // SQLite — full WAL + FTS5, no native build).
-        lines.push(`**Backend:** node:sqlite (Node built-in) — full WAL + FTS5`);
+        // Surface the active SQLite backend: node:sqlite → better-sqlite3 → wasm.
+        const backend = cg.getBackend();
+        if (backend === 'node-sqlite') {
+            lines.push(`**Backend:** node-sqlite (built-in)`);
+        }
+        else if (backend === 'native') {
+            lines.push(`**Backend:** native (better-sqlite3)`);
+        }
+        else {
+            lines.push(`**Backend:** ⚠ wasm (no WAL backend available) — ` +
+                `5-10x slower than WAL. Fix: ${sqlite_adapter_1.WASM_FALLBACK_FIX_RECIPE}`);
+        }
         // Effective journal mode. 'wal' ⇒ concurrent reads never block on a writer;
-        // anything else ⇒ they can ("database is locked"). node:sqlite supports WAL
-        // everywhere, so a non-wal mode means the filesystem can't (network/
-        // virtualized mounts, WSL2 /mnt). See issue #238.
+        // anything else ⇒ they can ("database is locked"). node:sqlite / native
+        // support WAL; wasm remaps to DELETE.
         const journalMode = cg.getJournalMode();
         if (journalMode === 'wal') {
             lines.push(`**Journal mode:** wal (concurrent reads safe)`);
         }
         else {
             lines.push(`**Journal mode:** ⚠ ${journalMode || 'unknown'} — WAL not active, so reads ` +
-                `can block on a concurrent write (WAL appears unsupported on this filesystem)`);
+                `can block on a concurrent write`);
         }
         lines.push('', '**Nodes by Kind:**');
         for (const [kind, count] of Object.entries(stats.nodesByKind)) {
@@ -6477,7 +6496,6 @@ class ToolHandler {
         if (typeof query !== 'string')
             return query;
         const repoPath = args.repoPath;
-        const explicitDbPath = args.dbPath;
         const topKRaw = Number(args.topK);
         const topK = Math.max(1, Math.min(isNaN(topKRaw) ? 5 : topKRaw, 50));
         const includeFragments = args.includeFragments !== false;
@@ -6487,7 +6505,7 @@ class ToolHandler {
         const { searchAndGetContext } = require('../spec/graph/queries');
         const { truncateCodeDiff, truncateSubtitles, computeBudgetProfile, } = require('../spec/utils');
         // Resolve the database path.
-        const dbPath = resolveDbPath(repoPath || process.cwd(), explicitDbPath);
+        const dbPath = resolveDbPath(repoPath || process.cwd());
         // Open the database.
         let db;
         try {
@@ -6587,13 +6605,12 @@ class ToolHandler {
         if (typeof filePath !== 'string')
             return filePath;
         const repoPath = args.repoPath;
-        const explicitDbPath = args.dbPath;
         // Lazily require spec modules so the MCP startup path stays lean.
         const { resolveDbPath } = require('../spec/utils');
         const { createDatabase } = require('../db/sqlite-adapter');
         const { findSpecsByFilePath } = require('../spec/graph/queries');
         // Resolve the database path.
-        const dbPath = resolveDbPath(repoPath || process.cwd(), explicitDbPath);
+        const dbPath = resolveDbPath(repoPath || process.cwd());
         // Open the database.
         let db;
         try {
@@ -6654,7 +6671,6 @@ class ToolHandler {
         const file = fileRaw;
         const line = typeof args.line === 'number' ? args.line : undefined;
         const repoPath = args.repoPath;
-        const explicitDbPath = args.dbPath;
         const topKRaw = Number(args.topK);
         const topK = Math.max(1, Math.min(isNaN(topKRaw) ? 10 : topKRaw, 50));
         // Lazily require all needed modules
@@ -6709,7 +6725,7 @@ class ToolHandler {
                 }, null, 2));
             }
             // Step 2: Resolve the Spec database path
-            const dbPath = resolveDbPath(repoPath || process.cwd(), explicitDbPath);
+            const dbPath = resolveDbPath(repoPath || process.cwd());
             // Step 3: Open the Spec database
             let db;
             try {