@sean.holung/minicode 0.3.4 → 0.3.6

package/dist/src/tools/read-symbol.js

@@ -1,12 +1,14 @@
 import { readFile, stat } from "node:fs/promises";
 import { resolveWorkspacePath, validateFileReadSize, expectNonEmptyString, expectOptionalBoolean, } from "@minicode/agent-sdk";
 import { getSymbolDisplayName } from "../indexer/symbol-names.js";
+import { formatAmbiguousSymbolMatches, resolveSymbolInput, } from "../shared/symbol-resolution.js";
 const LEADING_CONTEXT_LINES = 3;
 export function createReadSymbolTool(config, projectIndex) {
     return {
         name: "read_symbol",
         description: "Read a specific function, class, or type definition by name. " +
             "Returns the symbol's source code, referenced types, callers, and callees. " +
+            "If a bare name matches multiple symbols, returns disambiguation candidates. " +
             "PREFER this over read_file for .ts/.tsx/.js/.jsx — use the code map to find symbol names.",
         inputSchema: {
             type: "object",
@@ -26,10 +28,14 @@ export function createReadSymbolTool(config, projectIndex) {
         execute: async (input) => {
             const name = expectNonEmptyString(input, "name");
             const includeBody = expectOptionalBoolean(input, "includeBody") ?? true;
-            const symbol = projectIndex.getSymbol(name);
-            if (!symbol) {
+            const resolution = resolveSymbolInput(projectIndex, name);
+            if (resolution.status === "missing") {
                 return `Symbol "${name}" not found in the project index. Try using search to find it, or use read_file to read the full file.`;
             }
+            if (resolution.status === "ambiguous") {
+                return formatAmbiguousSymbolMatches("read_symbol", name, resolution.matches);
+            }
+            const symbol = resolution.symbol;
             const filePath = resolveWorkspacePath(symbol.filePath, config.workspaceRoot);
             const fileStat = await stat(filePath);
             if (!fileStat.isFile()) {
@@ -75,7 +81,7 @@ export function createReadSymbolTool(config, projectIndex) {
                 if (calls.length > 0) {
                     parts.push("", "## Calls", "", calls.map((s) => `- ${s}`).join("\n"));
                 }
-                const cone = projectIndex.getDependencyCone(name, 1);
+                const cone = projectIndex.getDependencyCone(symbol.qualifiedName, 1);
                 const typeRefs = cone.filter((s) => s.qualifiedName !== symbol.qualifiedName &&
                     (s.kind === "interface" || s.kind === "type"));
                 if (typeRefs.length > 0) {

package/dist/src/tools/registry.js

@@ -14,6 +14,9 @@ export function createToolRegistry(config, projectIndex) {
         ? {
             afterWrite: (relPath, content) => projectIndex.reindexFile(relPath, content),
             afterEdit: (relPath, content) => projectIndex.reindexFile(relPath, content),
+            afterCommand: async () => {
+                await projectIndex.refreshFromWorkspace();
+            },
         }
         : undefined;
     const tools = [
@@ -22,7 +25,7 @@ export function createToolRegistry(config, projectIndex) {
         createEditFileTool(config, hooks ? { afterEdit: hooks.afterEdit } : undefined),
         createSearchTool(config),
         createListFilesTool(config),
-        createRunCommandTool(config),
+        createRunCommandTool(config, hooks ? { afterCommand: hooks.afterCommand } : undefined),
     ];
     if (projectIndex) {
         tools.splice(1, 0, createReadSymbolTool(config, projectIndex));

package/dist/src/tools/search-code-map.js

@@ -1,6 +1,21 @@
 import { expectNonEmptyString, expectOptionalNumber } from "@minicode/agent-sdk";
 import { getSymbolDisplayName, getSymbolLookupNames } from "../indexer/symbol-names.js";
 const DEFAULT_LIMIT = 30;
+function compareMatchedSymbols(pattern, a, b) {
+    const lowerPattern = pattern.toLowerCase();
+    const aDisplay = getSymbolDisplayName(a).toLowerCase();
+    const bDisplay = getSymbolDisplayName(b).toLowerCase();
+    const aExact = Number(aDisplay === lowerPattern || a.qualifiedName.toLowerCase() === lowerPattern || a.name.toLowerCase() === lowerPattern);
+    const bExact = Number(bDisplay === lowerPattern || b.qualifiedName.toLowerCase() === lowerPattern || b.name.toLowerCase() === lowerPattern);
+    if (aExact !== bExact) {
+        return bExact - aExact;
+    }
+    return Number(b.exported) - Number(a.exported) ||
+        aDisplay.localeCompare(bDisplay) ||
+        a.filePath.localeCompare(b.filePath) ||
+        a.startLine - b.startLine ||
+        a.qualifiedName.localeCompare(b.qualifiedName);
+}
 function matchesPattern(text, pattern) {
     const lowerText = text.toLowerCase();
     const lowerPattern = pattern.toLowerCase();
@@ -11,7 +26,7 @@ export function createSearchCodeMapTool(projectIndex) {
         name: "search_code_map",
         description: "Search the full project index for symbols by name or substring. " +
             "Use when the code map is truncated and you need to find a symbol not listed. " +
-            "Returns qualified names and file paths; use read_symbol with the result.",
+            "Returns disambiguated display names, qualified names, and file paths; use read_symbol with the result.",
         inputSchema: {
             type: "object",
             properties: {
@@ -53,9 +68,9 @@ export function createSearchCodeMapTool(projectIndex) {
                     return false;
                 }
                 return true;
-            });
+            }).sort((a, b) => compareMatchedSymbols(pattern, a, b));
             const shown = matches.slice(skip, skip + limit);
-            const lines = shown.map((s) => `- ${getSymbolDisplayName(s)} (${s.kind}) — ${s.filePath}`);
+            const lines = shown.map((s) => `- ${getSymbolDisplayName(s)} (${s.kind}) — ${s.filePath}:${s.startLine} — qualified: ${s.qualifiedName}`);
             const remaining = matches.length - skip - shown.length;
             const footer = remaining > 0
                 ? `\n... and ${remaining} more (use skip: ${skip + limit}, limit: ${limit} for next page)`