ucn 3.8.12 → 3.8.14

package/mcp/server.js

@@ -33,7 +33,7 @@ try {
 const { ProjectIndex } = require('../core/project');
 const { findProjectRoot } = require('../core/discovery');
 const output = require('../core/output');
-const { getMcpCommandEnum, normalizeParams } = require('../core/registry');
+const { getMcpCommandEnum, normalizeParams, BROAD_COMMANDS: BROAD_CANONICAL, toMcpName } = require('../core/registry');
 const { execute } = require('../core/execute');
 const { ExpandCache } = require('../core/expand-cache');
 
@@ -47,6 +47,7 @@ const expandCacheInstance = new ExpandCache();
 
 function getIndex(projectDir, options) {
     const maxFiles = options && options.maxFiles;
+    const followSymlinks = options && options.followSymlinks;
     const absDir = path.resolve(projectDir);
     if (!fs.existsSync(absDir) || !fs.statSync(absDir).isDirectory()) {
         throw new Error(`Project directory not found: ${absDir}`);
@@ -67,6 +68,7 @@ function getIndex(projectDir, options) {
     const index = new ProjectIndex(root);
     const buildOpts = { quiet: true, forceRebuild: false };
     if (maxFiles) buildOpts.maxFiles = maxFiles;
+    if (followSymlinks === false) buildOpts.followSymlinks = false;
     const loaded = index.loadCache();
     if (loaded && !maxFiles && !index.isCacheStale()) {
         // Disk cache is fresh (skip when maxFiles is set — cached index may have different file count)
@@ -118,8 +120,8 @@ const DEFAULT_OUTPUT_CHARS = 10000;  // ~2.5K tokens — targeted commands (abou
 const BROAD_OUTPUT_CHARS = 3000;     // ~750 tokens — broad commands where truncated listings are useless
 const MAX_OUTPUT_CHARS = 100000;     // hard ceiling even with max_chars override
 
-// Broad commands: output is project-wide, truncation means you need a filter, not more text
-const BROAD_COMMANDS = new Set(['toc', 'entrypoints', 'diff_impact', 'affected_tests', 'deadcode', 'usages']);
+// Broad commands (derived from registry): output is project-wide, truncation means you need a filter
+const BROAD_COMMANDS = new Set([...BROAD_CANONICAL].map(toMcpName));
 
 function toolResult(text, command, maxChars) {
     if (!text) return { content: [{ type: 'text', text: '(no output)' }] };
@@ -300,7 +302,8 @@ server.registerTool(
             decorator: z.string().optional().describe('Filter by decorator/annotation (structural search). E.g. "Route", "Test".'),
             exported: z.boolean().optional().describe('Only exported/public symbols (structural search).'),
             unused: z.boolean().optional().describe('Only symbols with zero callers (structural search).'),
-            framework: z.string().optional().describe('Filter entrypoints by framework (e.g. "express", "spring", "flask"). Comma-separated for multiple.')
+            framework: z.string().optional().describe('Filter entrypoints by framework (e.g. "express", "spring", "flask"). Comma-separated for multiple.'),
+            follow_symlinks: z.boolean().optional().describe('Follow symlinks during file discovery (default: true)')
 
         })
     },
@@ -340,30 +343,36 @@ server.registerTool(
 
             case 'context': {
                 index = getIndex(project_dir, ep);
-                const { ok, result: ctx, error } = execute(index, 'context', ep);
+                const { ok, result: ctx, error, note } = execute(index, 'context', ep);
                 if (!ok) return tr(error); // context uses soft error (not toolError)
                 const { text, expandable } = output.formatContext(ctx, {
                     expandHint: 'Use expand command with item number to see code for any item.',
                     showConfidence: ep.showConfidence !== false,
                 });
                 expandCacheInstance.save(index.root, ep.name, ep.file, expandable);
-                return tr(text);
+                let ctxText = text;
+                if (note) ctxText += '\n\n' + note;
+                return tr(ctxText);
             }
 
             case 'impact': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'impact', ep);
+                const { ok, result, error, note } = execute(index, 'impact', ep);
                 if (!ok) return tr(error); // soft error
-                return tr(output.formatImpact(result));
+                let impactText = output.formatImpact(result);
+                if (note) impactText += '\n\n' + note;
+                return tr(impactText);
             }
 
             case 'blast': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'blast', ep);
+                const { ok, result, error, note } = execute(index, 'blast', ep);
                 if (!ok) return tr(error); // soft error
-                return tr(output.formatBlast(result, {
+                let blastText = output.formatBlast(result, {
                     allHint: 'Set depth to expand all children.',
-                }));
+                });
+                if (note) blastText += '\n\n' + note;
+                return tr(blastText);
             }
 
             case 'smart': {
@@ -375,21 +384,25 @@ server.registerTool(
 
             case 'trace': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'trace', ep);
+                const { ok, result, error, note } = execute(index, 'trace', ep);
                 if (!ok) return tr(error); // soft error
-                return tr(output.formatTrace(result, {
+                let traceText = output.formatTrace(result, {
                     allHint: 'Set depth to expand all children.',
                     methodsHint: 'Note: obj.method() calls excluded. Use include_methods=true to include them.'
-                }));
+                });
+                if (note) traceText += '\n\n' + note;
+                return tr(traceText);
             }
 
             case 'reverse_trace': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'reverseTrace', ep);
+                const { ok, result, error, note } = execute(index, 'reverseTrace', ep);
                 if (!ok) return tr(error);
-                return tr(output.formatReverseTrace(result, {
+                let rtText = output.formatReverseTrace(result, {
                     allHint: 'Set depth to expand all children.',
-                }));
+                });
+                if (note) rtText += '\n\n' + note;
+                return tr(rtText);
             }
 
             case 'example': {
@@ -402,13 +415,15 @@ server.registerTool(
 
             case 'related': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'related', ep);
+                const { ok, result, error, note } = execute(index, 'related', ep);
                 if (!ok) return tr(error);
                 if (!result) return tr(`Symbol "${ep.name}" not found.`);
-                return tr(output.formatRelated(result, {
+                let relText = output.formatRelated(result, {
                     all: ep.all || false, top: ep.top,
                     allHint: 'Repeat with all=true to show all.'
-                }));
+                });
+                if (note) relText += '\n\n' + note;
+                return tr(relText);
             }
 
             // ── Finding Code ────────────────────────────────────────────
@@ -461,9 +476,11 @@ server.registerTool(
 
             case 'affected_tests': {
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'affectedTests', ep);
+                const { ok, result, error, note } = execute(index, 'affectedTests', ep);
                 if (!ok) return tr(error);
-                return tr(output.formatAffectedTests(result, { all: ep.all }));
+                let atText = output.formatAffectedTests(result, { all: ep.all });
+                if (note) atText += '\n\n' + note;
+                return tr(atText);
             }
 
             case 'deadcode': {
@@ -590,15 +607,15 @@ server.registerTool(
                 const err = requireName(ep.name);
                 if (err) return err;
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'fn', ep);
+                const { ok, result, error, note } = execute(index, 'fn', ep);
                 if (!ok) return tr(error); // soft error
                 // MCP path security: validate all result files are within project root
                 for (const entry of result.entries) {
                     const check = resolveAndValidatePath(index, entry.match.relativePath || path.relative(index.root, entry.match.file));
                     if (typeof check !== 'string') return check;
                 }
-                const notes = result.notes.length ? result.notes.map(n => 'Note: ' + n).join('\n') + '\n\n' : '';
-                return tr(notes + output.formatFnResult(result));
+                const fnText = (note ? note + '\n\n' : '') + output.formatFnResult(result);
+                return tr(fnText);
             }
 
             case 'class': {
@@ -608,15 +625,15 @@ server.registerTool(
                     return toolError(`Invalid max_lines: ${ep.maxLines}. Must be a positive integer.`);
                 }
                 index = getIndex(project_dir, ep);
-                const { ok, result, error } = execute(index, 'class', ep);
+                const { ok, result, error, note } = execute(index, 'class', ep);
                 if (!ok) return tr(error);  // soft error (class not found)
                 // MCP path security: validate all result files are within project root
                 for (const entry of result.entries) {
                     const check = resolveAndValidatePath(index, entry.match.relativePath || path.relative(index.root, entry.match.file));
                     if (typeof check !== 'string') return check;
                 }
-                const notes = result.notes.length ? result.notes.map(n => 'Note: ' + n).join('\n') + '\n\n' : '';
-                return tr(notes + output.formatClassResult(result));
+                const classText = (note ? note + '\n\n' : '') + output.formatClassResult(result);
+                return tr(classText);
             }
 
             case 'lines': {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucn",
-  "version": "3.8.12",
+  "version": "3.8.14",
   "mcpName": "io.github.mleoca/ucn",
   "description": "Code intelligence toolkit for AI agents — extract functions, trace call chains, find callers, detect dead code without reading entire files. Works as MCP server, CLI, or agent skill. Supports JS/TS, Python, Go, Rust, Java.",
   "main": "index.js",
@@ -9,8 +9,9 @@
     "ucn-mcp": "mcp/server.js"
   },
   "scripts": {
-    "test": "node --test test/parser-unit.test.js test/integration.test.js test/cache.test.js test/formatter.test.js test/interactive.test.js test/feature.test.js test/regression-js.test.js test/regression-py.test.js test/regression-go.test.js test/regression-java.test.js test/regression-rust.test.js test/regression-cross.test.js test/accuracy.test.js test/systematic-test.js test/mcp-edge-cases.js test/parity-test.js",
-    "benchmark:agent": "node test/agent-understanding-benchmark.js"
+    "test": "node --test test/parser-unit.test.js test/integration.test.js test/cache.test.js test/formatter.test.js test/interactive.test.js test/feature.test.js test/regression-js.test.js test/regression-py.test.js test/regression-go.test.js test/regression-java.test.js test/regression-rust.test.js test/regression-cross.test.js test/regression-mcp.test.js test/regression-parser.test.js test/regression-commands.test.js test/regression-fixes.test.js test/regression-bugfixes.test.js test/cross-language.test.js test/accuracy.test.js test/command-coverage.test.js test/systematic-test.js test/mcp-edge-cases.js test/parity-test.js",
+    "benchmark:agent": "node test/agent-understanding-benchmark.js",
+    "lint": "eslint core/ cli/ mcp/ languages/"
   },
   "keywords": [
     "mcp",
@@ -60,5 +61,9 @@
   "optionalDependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
     "zod": "^3.25.0"
+  },
+  "devDependencies": {
+    "@eslint/js": "^10.0.1",
+    "eslint": "^10.0.3"
   }
 }