@aiready/change-amplification 0.1.6 → 0.1.8

package/.turbo/turbo-build.log

@@ -1,6 +1,6 @@
 
 
-> @aiready/change-amplification@0.1.6 build /Users/pengcao/projects/aiready/packages/change-amplification
+> @aiready/change-amplification@0.1.8 build /Users/pengcao/projects/aiready/packages/change-amplification
 > tsup src/index.ts src/cli.ts --format cjs,esm --dts
 
 CLI Building entry: src/cli.ts, src/index.ts
@@ -10,7 +10,7 @@
 CJS Build start
 ESM Build start
 
-[10:18:49 PM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
+[12:55:24 AM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
 
     package.json:33:6:
       33 │       "types": "./dist/index.d.ts"
@@ -31,7 +31,7 @@
 
 
 
-[10:18:49 PM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
+[12:55:24 AM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
 
     package.json:38:6:
       38 │       "types": "./dist/cli.d.ts"
@@ -51,8 +51,12 @@
 
 
 
+ESM dist/index.mjs          110.00 B
+ESM dist/cli.mjs            2.78 KB
+ESM dist/chunk-MARO4FT6.mjs 3.52 KB
+ESM ⚡️ Build success in 440ms
 
-[10:18:49 PM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
+[12:55:25 AM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
 
     package.json:33:6:
       33 │       "types": "./dist/index.d.ts"
@@ -73,7 +77,7 @@
 
 
 
-[10:18:49 PM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
+[12:55:25 AM]  WARN  ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
 
     package.json:38:6:
       38 │       "types": "./dist/cli.d.ts"
@@ -93,15 +97,11 @@
 
 
 
-CJS dist/cli.js   8.08 KB
-CJS dist/index.js 5.21 KB
-CJS ⚡️ Build success in 145ms
-ESM dist/cli.mjs            2.78 KB
-ESM dist/index.mjs          110.00 B
-ESM dist/chunk-A5ACPWWO.mjs 3.92 KB
-ESM ⚡️ Build success in 146ms
+CJS dist/index.js 4.79 KB
+CJS dist/cli.js   7.65 KB
+CJS ⚡️ Build success in 631ms
 DTS Build start
-DTS ⚡️ Build success in 4081ms
+DTS ⚡️ Build success in 10596ms
 DTS dist/cli.d.ts    152.00 B
 DTS dist/index.d.ts  999.00 B
 DTS dist/cli.d.mts   152.00 B

package/.turbo/turbo-lint.log

@@ -0,0 +1,5 @@
+
+
+> @aiready/change-amplification@0.1.6 lint /Users/pengcao/projects/aiready/packages/change-amplification
+> eslint src --ext .ts
+

package/.turbo/turbo-test.log

@@ -1,16 +1,16 @@
 
 
-> @aiready/change-amplification@0.1.6 test /Users/pengcao/projects/aiready/packages/change-amplification
+> @aiready/change-amplification@0.1.8 test /Users/pengcao/projects/aiready/packages/change-amplification
 > vitest run
 
 [?25l
  RUN  v4.0.18 /Users/pengcao/projects/aiready/packages/change-amplification
 
- ✓ src/__tests__/dummy.test.ts (1 test) 2ms
+ ✓ src/__tests__/dummy.test.ts (1 test) 13ms
 
  Test Files  1 passed (1)
       Tests  1 passed (1)
-   Start at  22:19:16
-   Duration  285ms (transform 44ms, setup 0ms, import 97ms, tests 2ms, environment 0ms)
+   Start at  00:56:24
+   Duration  1.16s (transform 306ms, setup 0ms, import 474ms, tests 13ms, environment 0ms)
 
 [?25h

package/dist/chunk-MARO4FT6.mjs

@@ -0,0 +1,112 @@
+var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
+  get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
+}) : x)(function(x) {
+  if (typeof require !== "undefined") return require.apply(this, arguments);
+  throw Error('Dynamic require of "' + x + '" is not supported');
+});
+
+// src/analyzer.ts
+import * as fs from "fs";
+import * as path from "path";
+import {
+  scanFiles,
+  calculateChangeAmplification,
+  getParser
+} from "@aiready/core";
+async function analyzeChangeAmplification(options) {
+  const rootDir = path.resolve(options.rootDir || ".");
+  const files = await scanFiles({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx,py,go}"]
+  });
+  const dependencyGraph = /* @__PURE__ */ new Map();
+  const reverseGraph = /* @__PURE__ */ new Map();
+  for (const file of files) {
+    dependencyGraph.set(file, []);
+    reverseGraph.set(file, []);
+  }
+  let processed = 0;
+  for (const file of files) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      files.length,
+      `change-amplification: analyzing files`
+    );
+    try {
+      const parser = getParser(file);
+      if (!parser) continue;
+      const content = fs.readFileSync(file, "utf8");
+      const parseResult = parser.parse(content, file);
+      const dependencies = parseResult.imports.map((i) => i.source);
+      for (const dep of dependencies) {
+        const depDir = path.dirname(file);
+        const resolvedPath = files.find((f) => {
+          if (dep.startsWith(".")) {
+            return f.startsWith(path.resolve(depDir, dep));
+          } else {
+            return f.includes(dep);
+          }
+        });
+        if (resolvedPath) {
+          dependencyGraph.get(file)?.push(resolvedPath);
+          reverseGraph.get(resolvedPath)?.push(file);
+        }
+      }
+    } catch (err) {
+      void err;
+    }
+  }
+  const fileMetrics = files.map((file) => {
+    const fanOut = dependencyGraph.get(file)?.length || 0;
+    const fanIn = reverseGraph.get(file)?.length || 0;
+    return { file, fanOut, fanIn };
+  });
+  const riskResult = calculateChangeAmplification({ files: fileMetrics });
+  const results = [];
+  for (const hotspot of riskResult.hotspots) {
+    const issues = [];
+    if (hotspot.amplificationFactor > 20) {
+      issues.push({
+        type: "change-amplification",
+        severity: hotspot.amplificationFactor > 40 ? "critical" : "major",
+        message: `High change amplification detected (Factor: ${hotspot.amplificationFactor}). Changes here cascade heavily.`,
+        location: { file: hotspot.file, line: 1 },
+        suggestion: `Reduce coupling. Fan-out is ${hotspot.fanOut}, Fan-in is ${hotspot.fanIn}.`
+      });
+    }
+    if (hotspot.amplificationFactor > 5) {
+      results.push({
+        fileName: hotspot.file,
+        issues,
+        metrics: {
+          aiSignalClarityScore: 100 - hotspot.amplificationFactor
+          // Just a rough score
+        }
+      });
+    }
+  }
+  return {
+    summary: {
+      totalFiles: files.length,
+      totalIssues: results.reduce((sum, r) => sum + r.issues.length, 0),
+      criticalIssues: results.reduce(
+        (sum, r) => sum + r.issues.filter((i) => i.severity === "critical").length,
+        0
+      ),
+      majorIssues: results.reduce(
+        (sum, r) => sum + r.issues.filter((i) => i.severity === "major").length,
+        0
+      ),
+      score: riskResult.score,
+      rating: riskResult.rating,
+      recommendations: riskResult.recommendations
+    },
+    results
+  };
+}
+
+export {
+  __require,
+  analyzeChangeAmplification
+};

package/dist/chunk-V5J3BP4N.mjs

@@ -0,0 +1,117 @@
+var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
+  get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
+}) : x)(function(x) {
+  if (typeof require !== "undefined") return require.apply(this, arguments);
+  throw Error('Dynamic require of "' + x + '" is not supported');
+});
+
+// src/analyzer.ts
+import * as fs from "fs";
+import * as path from "path";
+import { globSync } from "glob";
+import { calculateChangeAmplification } from "@aiready/core";
+import { getParser } from "@aiready/core";
+function collectFiles(dir, options) {
+  const includePatterns = options.include && options.include.length > 0 ? options.include : ["**/*.{ts,tsx,js,jsx,py,go}"];
+  const excludePatterns = options.exclude && options.exclude.length > 0 ? options.exclude : ["**/node_modules/**", "**/dist/**", "**/.git/**"];
+  let matchedFiles = [];
+  for (const pattern of includePatterns) {
+    const files = globSync(pattern, {
+      cwd: dir,
+      ignore: excludePatterns,
+      absolute: true
+    });
+    matchedFiles = matchedFiles.concat(files);
+  }
+  return [...new Set(matchedFiles)];
+}
+async function analyzeChangeAmplification(options) {
+  const rootDir = path.resolve(options.rootDir || ".");
+  const files = collectFiles(rootDir, options);
+  const dependencyGraph = /* @__PURE__ */ new Map();
+  const reverseGraph = /* @__PURE__ */ new Map();
+  for (const file of files) {
+    dependencyGraph.set(file, []);
+    reverseGraph.set(file, []);
+  }
+  let processed = 0;
+  for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `change-amplification: analyzing ${file.substring(rootDir.length + 1)}`);
+    try {
+      const parser = getParser(file);
+      if (!parser) continue;
+      const content = fs.readFileSync(file, "utf8");
+      const parseResult = parser.parse(content, file);
+      const dependencies = parseResult.imports.map((i) => i.source);
+      for (const dep of dependencies) {
+        const depDir = path.dirname(file);
+        const resolvedPath = files.find((f) => {
+          if (dep.startsWith(".")) {
+            return f.startsWith(path.resolve(depDir, dep));
+          } else {
+            return f.includes(dep);
+          }
+        });
+        if (resolvedPath) {
+          dependencyGraph.get(file)?.push(resolvedPath);
+          reverseGraph.get(resolvedPath)?.push(file);
+        }
+      }
+    } catch (err) {
+      void err;
+    }
+  }
+  const fileMetrics = files.map((file) => {
+    const fanOut = dependencyGraph.get(file)?.length || 0;
+    const fanIn = reverseGraph.get(file)?.length || 0;
+    return { file, fanOut, fanIn };
+  });
+  const riskResult = calculateChangeAmplification({ files: fileMetrics });
+  const results = [];
+  for (const hotspot of riskResult.hotspots) {
+    const issues = [];
+    if (hotspot.amplificationFactor > 20) {
+      issues.push({
+        type: "change-amplification",
+        severity: hotspot.amplificationFactor > 40 ? "critical" : "major",
+        message: `High change amplification detected (Factor: ${hotspot.amplificationFactor}). Changes here cascade heavily.`,
+        location: { file: hotspot.file, line: 1 },
+        suggestion: `Reduce coupling. Fan-out is ${hotspot.fanOut}, Fan-in is ${hotspot.fanIn}.`
+      });
+    }
+    if (hotspot.amplificationFactor > 5) {
+      results.push({
+        fileName: hotspot.file,
+        issues,
+        metrics: {
+          aiSignalClarityScore: 100 - hotspot.amplificationFactor
+          // Just a rough score
+        }
+      });
+    }
+  }
+  return {
+    summary: {
+      totalFiles: files.length,
+      totalIssues: results.reduce((sum, r) => sum + r.issues.length, 0),
+      criticalIssues: results.reduce(
+        (sum, r) => sum + r.issues.filter((i) => i.severity === "critical").length,
+        0
+      ),
+      majorIssues: results.reduce(
+        (sum, r) => sum + r.issues.filter((i) => i.severity === "major").length,
+        0
+      ),
+      score: riskResult.score,
+      rating: riskResult.rating,
+      recommendations: riskResult.recommendations
+    },
+    results
+  };
+}
+
+export {
+  __require,
+  analyzeChangeAmplification
+};

package/dist/cli.js

@@ -42,35 +42,29 @@ var fs2 = __toESM(require("fs"));
 // src/analyzer.ts
 var fs = __toESM(require("fs"));
 var path = __toESM(require("path"));
-var import_glob = require("glob");
 var import_core = require("@aiready/core");
-var import_core2 = require("@aiready/core");
-function collectFiles(dir, options) {
-  const includePatterns = options.include && options.include.length > 0 ? options.include : ["**/*.{ts,tsx,js,jsx,py,go}"];
-  const excludePatterns = options.exclude && options.exclude.length > 0 ? options.exclude : ["**/node_modules/**", "**/dist/**", "**/.git/**"];
-  let matchedFiles = [];
-  for (const pattern of includePatterns) {
-    const files = (0, import_glob.globSync)(pattern, {
-      cwd: dir,
-      ignore: excludePatterns,
-      absolute: true
-    });
-    matchedFiles = matchedFiles.concat(files);
-  }
-  return [...new Set(matchedFiles)];
-}
 async function analyzeChangeAmplification(options) {
   const rootDir = path.resolve(options.rootDir || ".");
-  const files = collectFiles(rootDir, options);
+  const files = await (0, import_core.scanFiles)({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx,py,go}"]
+  });
   const dependencyGraph = /* @__PURE__ */ new Map();
   const reverseGraph = /* @__PURE__ */ new Map();
   for (const file of files) {
     dependencyGraph.set(file, []);
     reverseGraph.set(file, []);
   }
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      files.length,
+      `change-amplification: analyzing files`
+    );
     try {
-      const parser = (0, import_core2.getParser)(file);
+      const parser = (0, import_core.getParser)(file);
       if (!parser) continue;
       const content = fs.readFileSync(file, "utf8");
       const parseResult = parser.parse(content, file);

package/dist/cli.mjs

@@ -2,7 +2,7 @@
 import {
   __require,
   analyzeChangeAmplification
-} from "./chunk-A5ACPWWO.mjs";
+} from "./chunk-MARO4FT6.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.js

@@ -37,35 +37,29 @@ module.exports = __toCommonJS(index_exports);
 // src/analyzer.ts
 var fs = __toESM(require("fs"));
 var path = __toESM(require("path"));
-var import_glob = require("glob");
 var import_core = require("@aiready/core");
-var import_core2 = require("@aiready/core");
-function collectFiles(dir, options) {
-  const includePatterns = options.include && options.include.length > 0 ? options.include : ["**/*.{ts,tsx,js,jsx,py,go}"];
-  const excludePatterns = options.exclude && options.exclude.length > 0 ? options.exclude : ["**/node_modules/**", "**/dist/**", "**/.git/**"];
-  let matchedFiles = [];
-  for (const pattern of includePatterns) {
-    const files = (0, import_glob.globSync)(pattern, {
-      cwd: dir,
-      ignore: excludePatterns,
-      absolute: true
-    });
-    matchedFiles = matchedFiles.concat(files);
-  }
-  return [...new Set(matchedFiles)];
-}
 async function analyzeChangeAmplification(options) {
   const rootDir = path.resolve(options.rootDir || ".");
-  const files = collectFiles(rootDir, options);
+  const files = await (0, import_core.scanFiles)({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx,py,go}"]
+  });
   const dependencyGraph = /* @__PURE__ */ new Map();
   const reverseGraph = /* @__PURE__ */ new Map();
   for (const file of files) {
     dependencyGraph.set(file, []);
     reverseGraph.set(file, []);
   }
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      files.length,
+      `change-amplification: analyzing files`
+    );
     try {
-      const parser = (0, import_core2.getParser)(file);
+      const parser = (0, import_core.getParser)(file);
       if (!parser) continue;
       const content = fs.readFileSync(file, "utf8");
       const parseResult = parser.parse(content, file);

package/dist/index.mjs

@@ -1,6 +1,6 @@
 import {
   analyzeChangeAmplification
-} from "./chunk-A5ACPWWO.mjs";
+} from "./chunk-MARO4FT6.mjs";
 export {
   analyzeChangeAmplification
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/change-amplification",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "AI-Readiness: Change Amplification Detection",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -10,7 +10,7 @@
     "commander": "^14.0.0",
     "glob": "^13.0.0",
     "chalk": "^5.3.0",
-    "@aiready/core": "0.9.33"
+    "@aiready/core": "0.9.35"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -1,45 +1,26 @@
 import * as fs from 'fs';
 import * as path from 'path';
-import { globSync } from 'glob';
-import { calculateChangeAmplification } from '@aiready/core';
+import {
+  scanFiles,
+  calculateChangeAmplification,
+  getParser,
+} from '@aiready/core';
 import type {
   ChangeAmplificationOptions,
   ChangeAmplificationReport,
   FileChangeAmplificationResult,
   ChangeAmplificationIssue,
 } from './types';
-import { getParser } from '@aiready/core';
-
-function collectFiles(
-  dir: string,
-  options: ChangeAmplificationOptions
-): string[] {
-  const includePatterns =
-    options.include && options.include.length > 0
-      ? options.include
-      : ['**/*.{ts,tsx,js,jsx,py,go}'];
-  const excludePatterns =
-    options.exclude && options.exclude.length > 0
-      ? options.exclude
-      : ['**/node_modules/**', '**/dist/**', '**/.git/**'];
-
-  let matchedFiles: string[] = [];
-  for (const pattern of includePatterns) {
-    const files = globSync(pattern, {
-      cwd: dir,
-      ignore: excludePatterns,
-      absolute: true,
-    });
-    matchedFiles = matchedFiles.concat(files);
-  }
-  return [...new Set(matchedFiles)];
-}
 
 export async function analyzeChangeAmplification(
   options: ChangeAmplificationOptions
 ): Promise<ChangeAmplificationReport> {
   const rootDir = path.resolve(options.rootDir || '.');
-  const files = collectFiles(rootDir, options);
+  // Use core scanFiles which respects .gitignore recursively
+  const files = await scanFiles({
+    ...options,
+    include: options.include || ['**/*.{ts,tsx,js,jsx,py,go}'],
+  });
 
   // Compute graph metrics: fanIn and fanOut
   const dependencyGraph = new Map<string, string[]>(); // key: file, value: imported files
@@ -52,7 +33,15 @@ export async function analyzeChangeAmplification(
   }
 
   // Parse files to build dependency graph
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      files.length,
+      `change-amplification: analyzing files`
+    );
+
     try {
       const parser = getParser(file);
       if (!parser) continue;