@sun-asterisk/impact-analyzer 1.0.1 → 1.0.2

package/modules/utils/method-call-graph.js

@@ -448,89 +448,64 @@ export class MethodCallGraph {
 
   /**
    * Get changed methods from git diff
-   * Detects both:
-   * 1. Methods that contain changed lines
-   * 2. Method calls that were added/removed
+   * Uses ts-morph AST to detect which specific methods contain changes
    */
   getChangedMethods(diff, filePath) {
-    const changedMethods = new Set();
-    const changedMethodCalls = new Set();
-    
     if (!diff || !filePath) return [];
 
     const sourceFile = this.project.getSourceFile(filePath);
     if (!sourceFile) return [];
 
-    const classes = sourceFile.getClasses();
-    if (classes.length === 0) return [];
-
-    // Parse diff to extract changed line numbers
+    // Extract changed line numbers from diff
     const changedLines = this.extractChangedLineNumbers(diff);
-    
     if (changedLines.length === 0) return [];
 
-    // 1. Find methods that contain the changed lines
+    const changedMethods = [];
+    
+    // Get all classes in the file
+    const classes = sourceFile.getClasses();
+    
     for (const classDecl of classes) {
       const className = classDecl.getName();
+      if (!className) continue;
+      
       const methods = classDecl.getMethods();
-
+      
       for (const method of methods) {
         const methodName = method.getName();
-        const startLine = method.getStartLineNumber();
-        const endLine = method.getEndLineNumber();
-
-        // Check if any changed line falls within this method's range
-        const hasChangedLine = changedLines.some(
-          lineNum => lineNum >= startLine && lineNum <= endLine
-        );
-
-        if (hasChangedLine) {
-          const fullName = `${className}.${methodName}`;
-          changedMethods.add(fullName);
-        }
-      }
-    }
-
-    // 2. Detect method calls that were modified in the diff
-    const diffLines = diff.split('\n');
-    
-    for (const line of diffLines) {
-      // Only look at added/removed lines
-      if (!line.startsWith('+') && !line.startsWith('-')) continue;
-      
-      const codeLine = line.substring(1).trim();
-      
-      // Pattern: object.methodName(...) or await object.methodName(...)
-      const methodCallPattern = /(?:await\s+)?(\w+)\.(\w+)\s*\(/g;
-      let match;
-      
-      while ((match = methodCallPattern.exec(codeLine)) !== null) {
-        const objectName = match[1];
-        const methodName = match[2];
         
-        // Try to resolve the type of the object
-        const resolvedType = this.resolveObjectType(objectName, sourceFile, classes);
-        
-        if (resolvedType) {
-          const fullName = `${resolvedType}.${methodName}`;
-          changedMethodCalls.add(fullName);
+        // Check if this specific method contains changes
+        if (this.methodContainsChanges(method, changedLines)) {
+          changedMethods.push({
+            className: className,
+            methodName: methodName,
+            file: filePath,
+            fullName: `${className}.${methodName}`
+          });
         }
       }
     }
-
-    // Combine both: methods containing changes + method calls that changed
-    const allChangedMethods = [...changedMethods, ...changedMethodCalls];
     
-    // Convert to objects with metadata
-    return [...new Set(allChangedMethods)].map(fullName => {
-      const parts = fullName.split('.');
-      return {
-        className: parts[0],
-        methodName: parts[1],
-        file: filePath,
-        fullName: fullName
-      };
-    });
+    return changedMethods;
+  }
+
+  /**
+   * Check if a method contains actual code changes using ts-morph AST
+   */
+  methodContainsChanges(method, changedLines) {
+    const body = method.getBody();
+    if (!body) return false;
+
+    // Get the method body block (excluding signature and decorators)
+    const bodyStartLine = body.getStartLineNumber();
+    const bodyEndLine = body.getEndLineNumber();
+
+    // Check if any changed line is within the method body
+    const hasChangesInBody = changedLines.some(
+      line => line >= bodyStartLine && line <= bodyEndLine
+    );
+
+    return hasChangesInBody;
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/impact-analyzer",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Automated impact analysis for TypeScript/JavaScript projects",
   "main": "index.js",
   "type": "module",