ucn 3.7.19 → 3.7.21

package/core/output.js

@@ -8,6 +8,17 @@
 const fs = require('fs');
 const path = require('path');
 
+const FILE_ERROR_MESSAGES = {
+    'file-not-found': 'File not found in project',
+    'file-ambiguous': 'Ambiguous file match'
+};
+
+function formatFileError(errorObj, fallbackPath) {
+    const msg = FILE_ERROR_MESSAGES[errorObj.error] || errorObj.error;
+    const file = errorObj.filePath || fallbackPath || '';
+    return `Error: ${msg}: ${file}`;
+}
+
 /**
  * Normalize parameters for display
  * Collapses multiline params to single line
@@ -381,7 +392,7 @@ function formatSearchJson(results, term) {
  * Format imports as JSON
  */
 function formatImportsJson(imports, filePath) {
-    if (imports?.error === 'file-not-found') return JSON.stringify({ found: false, error: 'File not found', file: imports.filePath }, null, 2);
+    if (imports?.error) return JSON.stringify({ found: false, error: imports.error, file: imports.filePath || filePath }, null, 2);
     return JSON.stringify({
         file: filePath,
         importCount: imports.length,
@@ -406,7 +417,7 @@ function formatStatsJson(stats) {
  * Format dependency graph as JSON
  */
 function formatGraphJson(graph) {
-    if (graph?.error === 'file-not-found') return JSON.stringify({ found: false, error: 'File not found', file: graph.filePath }, null, 2);
+    if (graph?.error) return JSON.stringify({ found: false, error: graph.error, file: graph.filePath }, null, 2);
     return JSON.stringify({
         file: graph.file,
         depth: graph.depth,
@@ -456,7 +467,7 @@ function formatSmartJson(result) {
  * Format imports command output - text
  */
 function formatImports(imports, filePath) {
-    if (imports?.error === 'file-not-found') return `Error: File not found in project: ${imports.filePath}`;
+    if (imports?.error) return formatFileError(imports, filePath);
     const lines = [`Imports in ${filePath}:\n`];
 
     const internal = imports.filter(i => !i.isExternal && !i.isDynamic);
@@ -505,7 +516,7 @@ function formatImports(imports, filePath) {
  * Format exporters command output - text
  */
 function formatExporters(exporters, filePath) {
-    if (exporters?.error === 'file-not-found') return `Error: File not found in project: ${exporters.filePath}`;
+    if (exporters?.error) return formatFileError(exporters, filePath);
     const lines = [`Files that import ${filePath}:\n`];
 
     if (exporters.length === 0) {
@@ -641,7 +652,7 @@ function formatDisambiguation(matches, name, command) {
  * Format exporters as JSON
  */
 function formatExportersJson(exporters, filePath) {
-    if (exporters?.error === 'file-not-found') return JSON.stringify({ found: false, error: 'File not found', file: exporters.filePath }, null, 2);
+    if (exporters?.error) return JSON.stringify({ found: false, error: exporters.error, file: exporters.filePath || filePath }, null, 2);
     return JSON.stringify({
         file: filePath,
         importerCount: exporters.length,
@@ -1815,7 +1826,7 @@ function formatGraph(graph, options = {}) {
     if (typeof options === 'boolean') {
         options = { showAll: options };
     }
-    if (graph?.error === 'file-not-found') return `Error: File not found in project: ${graph.filePath}`;
+    if (graph?.error) return formatFileError(graph);
     if (graph.nodes.length === 0) {
         const file = options.file || graph.root || '';
         return file ? `File not found: ${file}` : 'File not found.';
@@ -1985,7 +1996,7 @@ function formatSearch(results, term) {
  * Format file-exports command output
  */
 function formatFileExports(exports, filePath) {
-    if (exports?.error === 'file-not-found') return `Error: File not found in project: ${exports.filePath}`;
+    if (exports?.error) return formatFileError(exports, filePath);
     if (exports.length === 0) return `No exports found in ${filePath}`;
 
     const lines = [];

package/core/parser.js

@@ -67,6 +67,9 @@ const { detectLanguage, getParser, getLanguageModule, isSupported, PARSE_OPTIONS
  * @returns {ParseResult}
  */
 function parse(code, language) {
+    if (!language) {
+        throw new Error('Language parameter is required');
+    }
     // Detect language if file path provided
     if (language.includes('.') || language.includes('/')) {
         language = detectLanguage(language);

package/core/project.js

@@ -364,6 +364,9 @@ class ProjectIndex {
             const seenModules = new Set();
 
             for (const importModule of fileEntry.imports) {
+                // Skip null modules (e.g., dynamic include! macros in Rust)
+                if (!importModule) continue;
+
                 // Deduplicate: same module imported multiple times in one file
                 // (e.g., lazy imports inside different functions)
                 if (seenModules.has(importModule)) continue;
@@ -1511,6 +1514,7 @@ class ProjectIndex {
             }
 
             const tree = safeParse(parser, content);
+            if (!tree) return false;
 
             // Find all occurrences of name in the line
             const nameRegex = new RegExp('(?<![a-zA-Z0-9_$])' + escapeRegExp(name) + '(?![a-zA-Z0-9_$])', 'g');
@@ -2117,6 +2121,7 @@ class ProjectIndex {
             }
 
             const tree = safeParse(parser, content);
+            if (!tree) return false;
             const tokenType = getTokenTypeAtPosition(tree.rootNode, lineNum, column);
             return tokenType === 'comment' || tokenType === 'string';
         } catch (e) {
@@ -5006,6 +5011,7 @@ class ProjectIndex {
             const parser = getParser(language);
             const content = this._readFile(filePath);
             const tree = safeParse(parser, content);
+            if (!tree) return result;
 
             const row = lineNum - 1;
             const node = tree.rootNode.descendantForPosition({ row, column: 0 });

package/languages/go.js

@@ -448,6 +448,7 @@ function findCallsInCode(code, parser) {
         if (node.type === 'short_var_declaration' || node.type === 'var_declaration') {
             // Check if RHS contains a func_literal
             const hasFunc = (n) => {
+                if (!n) return false;
                 if (n.type === 'func_literal') return true;
                 for (let i = 0; i < n.childCount; i++) {
                     if (hasFunc(n.child(i))) return true;
@@ -518,7 +519,7 @@ function findCallsInCode(code, parser) {
         onLeave: (node) => {
             if (isFunctionNode(node)) {
                 const leaving = functionStack.pop();
-                closureScopes.delete(leaving.startLine);
+                if (leaving) closureScopes.delete(leaving.startLine);
             }
         }
     });

package/languages/javascript.js

@@ -1453,7 +1453,9 @@ function findImportsInCode(code, parser) {
                         }
                     }
 
-                    imports.push({ module: modulePath, names, type: 'require', line, dynamic });
+                    if (modulePath) {
+                        imports.push({ module: modulePath, names, type: 'require', line, dynamic });
+                    }
                 }
             }
 
@@ -1466,8 +1468,8 @@ function findImportsInCode(code, parser) {
                     if (firstArg && firstArg.type === 'string') {
                         const modulePath = firstArg.text.slice(1, -1);
                         imports.push({ module: modulePath, names: [], type: 'dynamic', line, dynamic: false });
-                    } else {
-                        imports.push({ module: firstArg ? firstArg.text : null, names: [], type: 'dynamic', line, dynamic: true });
+                    } else if (firstArg) {
+                        imports.push({ module: firstArg.text, names: [], type: 'dynamic', line, dynamic: true });
                     }
                 }
             }

package/languages/rust.js

@@ -845,16 +845,19 @@ function findImportsInCode(code, parser) {
         if (node.type === 'macro_invocation') {
             const nameNode = node.childForFieldName('macro');
             if (nameNode && /^include(_str|_bytes)?$/.test(nameNode.text)) {
-                const argsNode = node.childForFieldName('argument_list');
+                const argsNode = node.namedChildren.find(c => c.type === 'token_tree');
                 const arg = argsNode?.namedChild(0);
                 const dynamic = !arg || arg.type !== 'string_literal';
-                imports.push({
-                    module: arg ? arg.text.replace(/^["']|["']$/g, '') : null,
-                    names: [],
-                    type: 'include',
-                    dynamic,
-                    line: node.startPosition.row + 1
-                });
+                const modulePath = arg ? arg.text.replace(/^["']|["']$/g, '') : null;
+                if (modulePath) {
+                    imports.push({
+                        module: modulePath,
+                        names: [],
+                        type: 'include',
+                        dynamic,
+                        line: node.startPosition.row + 1
+                    });
+                }
             }
         }
         return true;

package/mcp/server.js

@@ -108,6 +108,7 @@ const server = new McpServer({
 const MAX_OUTPUT_CHARS = 100000; // ~100KB, safe for all MCP clients
 
 function toolResult(text) {
+    if (!text) return { content: [{ type: 'text', text: '(no output)' }] };
     if (text.length > MAX_OUTPUT_CHARS) {
         const truncated = text.substring(0, MAX_OUTPUT_CHARS);
         // Cut at last newline to avoid breaking mid-line

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucn",
-  "version": "3.7.19",
+  "version": "3.7.21",
   "mcpName": "io.github.mleoca/ucn",
   "description": "Universal Code Navigator — AST-based call graph analysis for AI agents. Find callers, trace impact, detect dead code across JS/TS, Python, Go, Rust, Java, and HTML. CLI, MCP server, and agent skill.",
   "main": "index.js",