@sun-asterisk/impact-analyzer 1.0.2 → 1.0.4

package/README.md

@@ -163,9 +163,9 @@ jobs:
   analyze:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 0  # Fetch full history
 
       - uses: actions/setup-node@v3
         with:
@@ -176,7 +176,6 @@ jobs:
           npx @sun-asterisk/impact-analyzer \
             --input=src \
             --base=origin/${{ github.base_ref }} \
-            --head=${{ github.sha }} \
             --output=impact-report.md \
             --json=impact-report.json
 
@@ -200,11 +199,12 @@ jobs:
 impact-analysis:
   stage: test
   image: node:18
+  variables:
+    GIT_DEPTH: 0  # Fetch full history
   script:
     - npx @sun-asterisk/impact-analyzer
         --input=src
         --base=origin/$CI_MERGE_REQUEST_TARGET_BRANCH_NAME
-        --head=$CI_COMMIT_SHA
         --output=impact-report.md
         --json=impact-report.json
   artifacts:

package/config/default-config.js

@@ -15,7 +15,7 @@ export function loadConfig(cli) {
 
     // Git references
     baseRef: cli.getArg('base', 'origin/main'),
-    headRef: cli.getArg('head', 'HEAD'),
+    headRef: cli.getArg('head', ''),  // Empty means current working directory
 
     // Analysis options
     maxDepth: parseInt(cli.getArg('max-depth', '3')),
@@ -38,9 +38,11 @@ function validateConfig(config) {
     throw new Error('Source directory (--input) is required');
   }
 
-  if (!config.baseRef || !config.headRef) {
-    throw new Error('Git references (--base and --head) are required');
+  if (!config.baseRef) {
+    throw new Error('Base git reference (--base) is required');
   }
+  
+  // headRef is optional, defaults to working directory comparison
 }
 
 export const DEFAULT_CONFIG = {

package/modules/report-generator.js

@@ -35,9 +35,14 @@ export class ReportGenerator {
     // Summary
     console.log('📊 SUMMARY:');
     console.log(`   Files Changed: ${changes.changedFiles.length}`);
-    console.log(`   Symbols Modified: ${changes.changedSymbols.length}`);
     console.log(`   Impact Score: ${impact.impactScore}`);
     console.log(`   Severity: ${this.getSeverityEmoji(impact.severity)} ${impact.severity.toUpperCase()}`);
+    if (impact.affectedEndpoints.length > 0) {
+      console.log(`   Affected Endpoints: ${impact.affectedEndpoints.length}`);
+    }
+    if (impact.databaseImpact.length > 0) {
+      console.log(`   Database Tables: ${impact.databaseImpact.length}`);
+    }
     console.log('');
 
     // Endpoints
@@ -106,7 +111,8 @@ export class ReportGenerator {
       timestamp: new Date().toISOString(),
       summary: {
         filesChanged: changes.changedFiles.length,
-        symbolsModified: changes.changedSymbols.length,
+        affectedEndpoints: impact.affectedEndpoints.length,
+        databaseTables: impact.databaseImpact.length,
         impactScore: impact.impactScore,
         severity: impact.severity,
       },
@@ -124,7 +130,8 @@ export class ReportGenerator {
 | Metric | Value |
 |--------|-------|
 | Files Changed | ${changes.changedFiles.length} |
-| Symbols Modified | ${changes.changedSymbols.length} |
+| Affected Endpoints | ${impact.affectedEndpoints.length} |
+| Database Tables | ${impact.databaseImpact.length} |
 | Impact Score | **${impact.impactScore}** |
 | Severity | ${this.getSeverityEmoji(impact.severity)} **${impact.severity.toUpperCase()}** |
 

package/modules/utils/git-utils.js

@@ -11,7 +11,7 @@ export class GitUtils {
   /**
    * Get list of changed files between two refs
    * @param {string} baseRef - Base git reference
-   * @param {string} headRef - Head git reference
+   * @param {string} headRef - Head git reference (empty = working directory)
    * @param {string} workDir - Working directory (optional, defaults to cwd)
    */
   static getChangedFiles(baseRef, headRef, workDir = null) {
@@ -23,10 +23,14 @@ export class GitUtils {
         throw new Error(`Directory does not exist: ${cwd}`);
       }
 
-      const diffCommand = `git diff --name-status ${baseRef}...${headRef}`;
+      // If headRef is empty, compare baseRef with working directory
+      const diffCommand = headRef 
+        ? `git diff --name-status ${baseRef}...${headRef}`
+        : `git diff --name-status ${baseRef}`;
+        
       const diffOutput = execSync(diffCommand, {
         encoding: 'utf-8',
-        cwd: cwd  // FIXED: Execute git command in the source directory
+        cwd: cwd
       });
 
       const changedFiles = [];
@@ -48,7 +52,7 @@ export class GitUtils {
       console.error('Error getting changed files:', error.message);
       console.error('  Working directory:', workDir || process.cwd());
       console.error('  Base ref:', baseRef);
-      console.error('  Head ref:', headRef);
+      console.error('  Head ref:', headRef || '(working directory)');
       return [];
     }
   }
@@ -77,7 +81,7 @@ export class GitUtils {
    * Get diff between two versions of a file
    * @param {string} filePath - File path relative to git root (e.g., 'src/modules/file.ts')
    * @param {string} baseRef - Base git reference
-   * @param {string} headRef - Head git reference  
+   * @param {string} headRef - Head git reference (empty = working directory)
    * @param {string} workDir - Working directory to run git from (optional)
    */
   static getFileDiff(filePath, baseRef, headRef, workDir = null) {
@@ -93,9 +97,10 @@ export class GitUtils {
         return '';
       }
 
-      // filePath is already relative to git root (from git diff --name-status)
-      // Execute git diff from git root
-      const diffCommand = `git diff ${baseRef}...${headRef} -- ${filePath}`;
+      // If headRef is empty, compare with working directory
+      const diffCommand = headRef
+        ? `git diff ${baseRef}...${headRef} -- ${filePath}`
+        : `git diff ${baseRef} -- ${filePath}`;
       
       const result = execSync(diffCommand, {
         encoding: 'utf-8',
@@ -108,7 +113,7 @@ export class GitUtils {
       console.error('Error getting file diff:', error.message);
       console.error('  File:', filePath);
       console.error('  Base:', baseRef);
-      console.error('  Head:', headRef);
+      console.error('  Head:', headRef || '(working directory)');
       console.error('  Working dir:', workDir);
       return '';
     }

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

@@ -704,6 +704,7 @@ export class MethodCallGraph {
             affectedBy: changedMethod,
             callChain: callChain,
             layers: this.getCallChainLayers(callChain),
+            impactLevel: this.calculateImpactLevel(callChain),
           });
         }
       }
@@ -721,6 +722,7 @@ export class MethodCallGraph {
           affectedBy: changedMethod,
           callChain: [changedMethod],
           layers: [startLayer],
+          impactLevel: 'high',  // Direct endpoint change is always high impact
         });
       }
 
@@ -793,6 +795,7 @@ export class MethodCallGraph {
             layers: [this.getMethodLayer(changedMethod), 'Command', this.getMethodLayer(endpointMethod)],
             viaCommand: commandName,
             endpointMethod: endpointMethod,
+            impactLevel: this.calculateImpactLevel([changedMethod, `Command: '${commandName}'`, endpointMethod]),
           });
         }
       }
@@ -924,4 +927,21 @@ export class MethodCallGraph {
         .reduce((sum, callers) => sum + callers.length, 0),
     };
   }
+
+  /**
+   * Calculate impact level based on call chain length
+   * Shorter chain = higher impact (closer to endpoint)
+   */
+  calculateImpactLevel(callChain) {
+    const chainLength = callChain.length;
+    
+    // Direct endpoint change or very short chain
+    if (chainLength <= 1) return 'high';
+    
+    // Short chain (2-3 hops)
+    if (chainLength <= 3) return 'medium';
+    
+    // Longer chain
+    return 'low';
+  }
 }

package/package.json

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