ucn 3.1.0 → 3.1.2

package/cli/index.js

@@ -745,7 +745,7 @@ function runProjectCommand(rootDir, command, arg) {
 
         case 'context': {
             requireArg(arg, 'Usage: ucn . context <name>');
-            const ctx = index.context(arg, { includeMethods: flags.includeMethods });
+            const ctx = index.context(arg, { includeMethods: flags.includeMethods, file: flags.file });
             printOutput(ctx,
                 output.formatContextJson,
                 r => { printContext(r, { expand: flags.expand, root: index.root }); }
@@ -787,7 +787,7 @@ function runProjectCommand(rootDir, command, arg) {
 
         case 'about': {
             requireArg(arg, 'Usage: ucn . about <name>');
-            const aboutResult = index.about(arg, { withTypes: flags.withTypes });
+            const aboutResult = index.about(arg, { withTypes: flags.withTypes, file: flags.file });
             printOutput(aboutResult,
                 output.formatAboutJson,
                 r => output.formatAbout(r, { expand: flags.expand, root: index.root, depth: flags.depth })
@@ -797,7 +797,7 @@ function runProjectCommand(rootDir, command, arg) {
 
         case 'impact': {
             requireArg(arg, 'Usage: ucn . impact <name>');
-            const impactResult = index.impact(arg);
+            const impactResult = index.impact(arg, { file: flags.file });
             printOutput(impactResult, output.formatImpactJson, output.formatImpact);
             break;
         }
@@ -821,7 +821,7 @@ function runProjectCommand(rootDir, command, arg) {
         case 'trace': {
             requireArg(arg, 'Usage: ucn . trace <name>');
             const traceDepth = flags.depth ? parseInt(flags.depth) : 3;
-            const traceResult = index.trace(arg, { depth: traceDepth });
+            const traceResult = index.trace(arg, { depth: traceDepth, file: flags.file });
             printOutput(traceResult, output.formatTraceJson, output.formatTrace);
             break;
         }

package/core/imports.js

@@ -431,6 +431,12 @@ function resolveImport(importPath, fromFile, config = {}) {
             }
         }
 
+        // Check Go module imports
+        if (config.language === 'go') {
+            const resolved = resolveGoImport(importPath, fromFile, config.root);
+            if (resolved) return resolved;
+        }
+
         return null;  // External package
     }
 
@@ -439,6 +445,83 @@ function resolveImport(importPath, fromFile, config = {}) {
     return resolveFilePath(resolved, config.extensions || getExtensions(config.language));
 }
 
+// Cache for Go module paths
+const goModuleCache = new Map();
+
+/**
+ * Find and parse go.mod to get the module path
+ * @param {string} startDir - Directory to start searching from
+ * @returns {{modulePath: string, root: string}|null}
+ */
+function findGoModule(startDir) {
+    // Check cache first
+    if (goModuleCache.has(startDir)) {
+        return goModuleCache.get(startDir);
+    }
+
+    let dir = startDir;
+    while (dir !== path.dirname(dir)) {
+        const goModPath = path.join(dir, 'go.mod');
+        if (fs.existsSync(goModPath)) {
+            try {
+                const content = fs.readFileSync(goModPath, 'utf-8');
+                // Parse module line: module github.com/user/project
+                const match = content.match(/^module\s+(\S+)/m);
+                if (match) {
+                    const result = { modulePath: match[1], root: dir };
+                    goModuleCache.set(startDir, result);
+                    return result;
+                }
+            } catch (e) {
+                // Ignore read errors
+            }
+        }
+        dir = path.dirname(dir);
+    }
+
+    goModuleCache.set(startDir, null);
+    return null;
+}
+
+/**
+ * Resolve Go package import to local files
+ * @param {string} importPath - Go import path (e.g., "github.com/user/proj/pkg/util")
+ * @param {string} fromFile - File containing the import
+ * @param {string} projectRoot - Project root directory
+ * @returns {string|null} - Directory path containing the package, or null if external
+ */
+function resolveGoImport(importPath, fromFile, projectRoot) {
+    const goMod = findGoModule(path.dirname(fromFile));
+    if (!goMod) return null;
+
+    const { modulePath, root } = goMod;
+
+    // Check if the import is within this module
+    if (importPath.startsWith(modulePath)) {
+        // Convert module path to relative path
+        // e.g., "github.com/user/proj/pkg/util" -> "pkg/util"
+        const relativePath = importPath.slice(modulePath.length).replace(/^\//, '');
+        const pkgDir = path.join(root, relativePath);
+
+        // Go imports are directories, find a .go file in the directory
+        if (fs.existsSync(pkgDir) && fs.statSync(pkgDir).isDirectory()) {
+            // Return the first .go file in the directory (not _test.go)
+            try {
+                const files = fs.readdirSync(pkgDir);
+                for (const file of files) {
+                    if (file.endsWith('.go') && !file.endsWith('_test.go')) {
+                        return path.join(pkgDir, file);
+                    }
+                }
+            } catch (e) {
+                // Ignore read errors
+            }
+        }
+    }
+
+    return null;
+}
+
 /**
  * Try to resolve a path with various extensions
  */

package/core/project.js

@@ -706,11 +706,21 @@ class ProjectIndex {
      * Get context for a symbol (callers + callees)
      */
     context(name, options = {}) {
-        const definitions = this.symbols.get(name) || [];
+        let definitions = this.symbols.get(name) || [];
         if (definitions.length === 0) {
             return { function: name, file: null, callers: [], callees: [] };
         }
 
+        // Filter by file if specified
+        if (options.file) {
+            const filtered = definitions.filter(d =>
+                d.relativePath && d.relativePath.includes(options.file)
+            );
+            if (filtered.length > 0) {
+                definitions = filtered;
+            }
+        }
+
         // Prefer class/struct/interface definitions over functions/methods/constructors
         // This ensures context('ClassName') finds the class, not a constructor with same name
         const typeOrder = ['class', 'struct', 'interface', 'type', 'impl'];
@@ -974,6 +984,10 @@ class ProjectIndex {
             const calls = this.getCachedCalls(def.file);
             if (!calls) return [];
 
+            // Get file language for smart method call handling
+            const fileEntry = this.files.get(def.file);
+            const language = fileEntry?.language;
+
             const callees = new Map();  // name -> count
 
             for (const call of calls) {
@@ -982,8 +996,12 @@ class ProjectIndex {
                 if (call.enclosingFunction.name !== def.name) continue;
                 if (call.enclosingFunction.startLine !== def.startLine) continue;
 
-                // Skip method calls unless explicitly requested
-                if (call.isMethod && !options.includeMethods) continue;
+                // Smart method call handling:
+                // - Go: include all method calls (Go doesn't use this/self/cls)
+                // - Other languages: skip method calls unless explicitly requested
+                if (call.isMethod) {
+                    if (language !== 'go' && !options.includeMethods) continue;
+                }
 
                 // Skip keywords and built-ins
                 if (this.isKeyword(call.name)) continue;
@@ -2102,11 +2120,21 @@ class ProjectIndex {
         const maxDepth = Math.max(0, rawDepth);
         const direction = options.direction || 'down';  // 'down' = callees, 'up' = callers, 'both'
 
-        const definitions = this.symbols.get(name);
+        let definitions = this.symbols.get(name);
         if (!definitions || definitions.length === 0) {
             return null;
         }
 
+        // Filter by file if specified
+        if (options.file) {
+            const filtered = definitions.filter(d =>
+                d.relativePath && d.relativePath.includes(options.file)
+            );
+            if (filtered.length > 0) {
+                definitions = filtered;
+            }
+        }
+
         const def = definitions[0];
         const visited = new Set();
 
@@ -2184,11 +2212,21 @@ class ProjectIndex {
      * @returns {object} Impact analysis
      */
     impact(name, options = {}) {
-        const definitions = this.symbols.get(name);
+        let definitions = this.symbols.get(name);
         if (!definitions || definitions.length === 0) {
             return null;
         }
 
+        // Filter by file if specified
+        if (options.file) {
+            const filtered = definitions.filter(d =>
+                d.relativePath && d.relativePath.includes(options.file)
+            );
+            if (filtered.length > 0) {
+                definitions = filtered;
+            }
+        }
+
         const def = definitions[0];
         const usages = this.usages(name, { codeOnly: true });
         const calls = usages.filter(u => u.usageType === 'call' && !u.isDefinition);
@@ -2899,10 +2937,10 @@ class ProjectIndex {
         const maxCallees = options.maxCallees || 5;
 
         // Find symbol definition(s)
-        const definitions = this.find(name, { exact: true });
+        const definitions = this.find(name, { exact: true, file: options.file });
         if (definitions.length === 0) {
             // Try fuzzy match
-            const fuzzy = this.find(name);
+            const fuzzy = this.find(name, { file: options.file });
             if (fuzzy.length === 0) {
                 return null;
             }

package/languages/go.js

@@ -567,8 +567,9 @@ function findUsagesInCode(code, name, parser) {
     const usages = [];
 
     traverseTree(tree.rootNode, (node) => {
-        // Only look for identifiers with the matching name
-        if (node.type !== 'identifier' || node.text !== name) {
+        // Look for both identifier and field_identifier (method names in selector expressions)
+        const isIdentifier = node.type === 'identifier' || node.type === 'field_identifier';
+        if (!isIdentifier || node.text !== name) {
             return true;
         }
 
@@ -622,9 +623,8 @@ function findUsagesInCode(code, name, parser) {
             else if (parent.type === 'parameter_declaration') {
                 usageType = 'definition';
             }
-            // Method call: selector_expression followed by call
-            else if (parent.type === 'selector_expression' &&
-                     parent.childForFieldName('field') === node) {
+            // Method call: selector_expression followed by call (field_identifier case)
+            else if (parent.type === 'selector_expression') {
                 const grandparent = parent.parent;
                 if (grandparent && grandparent.type === 'call_expression') {
                     usageType = 'call';

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "ucn",
-  "version": "3.1.0",
-  "description": "Code navigation built by AI, for AI. Reduces context usage by 90%+ when working with large codebases.",
+  "version": "3.1.2",
+  "description": "Code navigation built by AI, for AI. Reduces context usage when working with large codebases.",
   "main": "index.js",
   "bin": {
     "ucn": "./cli/index.js"

package/test/parser.test.js

@@ -4297,6 +4297,168 @@ class Product {
             fs.rmSync(tmpDir, { recursive: true, force: true });
         }
     });
+
+    it('should resolve Go module imports for exporters', () => {
+        const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ucn-go-import-'));
+        try {
+            // Create go.mod file
+            fs.writeFileSync(path.join(tmpDir, 'go.mod'), `module example.com/myproject
+
+go 1.21
+`);
+
+            // Create package structure
+            fs.mkdirSync(path.join(tmpDir, 'pkg', 'config'), { recursive: true });
+            fs.writeFileSync(path.join(tmpDir, 'pkg', 'config', 'config.go'), `package config
+
+type Config struct {
+    Name string
+}
+
+func NewConfig() *Config {
+    return &Config{}
+}
+`);
+
+            fs.writeFileSync(path.join(tmpDir, 'main.go'), `package main
+
+import "example.com/myproject/pkg/config"
+
+func main() {
+    cfg := config.NewConfig()
+    _ = cfg
+}
+`);
+
+            const index = new ProjectIndex(tmpDir);
+            index.build('**/*.go', { quiet: true });
+
+            // Get exporters for the config package
+            const exportersResult = index.exporters(path.join(tmpDir, 'pkg', 'config', 'config.go'));
+            assert.ok(exportersResult.length > 0, 'Should find files that import the config package');
+
+            // main.go should be in the list
+            const mainFile = exportersResult.find(e => e.file.includes('main.go'));
+            assert.ok(mainFile, 'main.go should import the config package');
+        } finally {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    });
+
+    it('should detect Go method calls in usages (field_identifier)', () => {
+        const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ucn-go-method-'));
+        try {
+            fs.writeFileSync(path.join(tmpDir, 'go.mod'), `module example.com/test
+go 1.21
+`);
+
+            fs.writeFileSync(path.join(tmpDir, 'service.go'), `package main
+
+type Service struct{}
+
+func (s *Service) CollectAll() error {
+    return nil
+}
+
+func main() {
+    svc := &Service{}
+    svc.CollectAll()
+}
+`);
+
+            const index = new ProjectIndex(tmpDir);
+            index.build('**/*.go', { quiet: true });
+
+            // usages should find the method call
+            const usages = index.usages('CollectAll', { codeOnly: true });
+            const calls = usages.filter(u => u.usageType === 'call' && !u.isDefinition);
+            assert.ok(calls.length >= 1, 'Should find at least 1 call to CollectAll');
+        } finally {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    });
+
+    it('should find callees for Go receiver methods', () => {
+        const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ucn-go-callees-'));
+        try {
+            fs.writeFileSync(path.join(tmpDir, 'go.mod'), `module example.com/test
+go 1.21
+`);
+
+            fs.writeFileSync(path.join(tmpDir, 'client.go'), `package main
+
+type Client struct{}
+
+func (c *Client) GetPods() []string {
+    return nil
+}
+
+func (c *Client) GetNodes() []string {
+    return nil
+}
+
+func (c *Client) CollectAll() {
+    c.GetPods()
+    c.GetNodes()
+}
+`);
+
+            const index = new ProjectIndex(tmpDir);
+            index.build('**/*.go', { quiet: true });
+
+            // context should find callees (Go method calls)
+            const ctx = index.context('CollectAll');
+            assert.ok(ctx.callees, 'Should have callees');
+            assert.ok(ctx.callees.length >= 2, 'Should find at least 2 callees (GetPods, GetNodes)');
+
+            const calleeNames = ctx.callees.map(c => c.name);
+            assert.ok(calleeNames.includes('GetPods'), 'GetPods should be a callee');
+            assert.ok(calleeNames.includes('GetNodes'), 'GetNodes should be a callee');
+        } finally {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    });
+
+    it('should filter by --file for Go methods with same name', () => {
+        const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'ucn-go-file-filter-'));
+        try {
+            fs.writeFileSync(path.join(tmpDir, 'go.mod'), `module example.com/test
+go 1.21
+`);
+
+            fs.writeFileSync(path.join(tmpDir, 'service_a.go'), `package main
+
+type ServiceA struct{}
+
+func (s *ServiceA) Process() error {
+    return nil
+}
+`);
+
+            fs.writeFileSync(path.join(tmpDir, 'service_b.go'), `package main
+
+type ServiceB struct{}
+
+func (s *ServiceB) Process() error {
+    return nil
+}
+`);
+
+            const index = new ProjectIndex(tmpDir);
+            index.build('**/*.go', { quiet: true });
+
+            // Without file filter, should find both
+            const allDefs = index.find('Process');
+            assert.strictEqual(allDefs.length, 2, 'Should find 2 definitions of Process');
+
+            // With file filter, should find only one
+            const filteredDefs = index.find('Process', { file: 'service_a.go' });
+            assert.strictEqual(filteredDefs.length, 1, 'Should find 1 definition with file filter');
+            assert.ok(filteredDefs[0].relativePath.includes('service_a.go'), 'Should be from service_a.go');
+        } finally {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    });
 });
 
 console.log('UCN v3 Test Suite');