@aiready/doc-drift 0.1.6 → 0.1.8

package/.turbo/turbo-build.log

@@ -1,6 +1,6 @@
 
 
-> @aiready/doc-drift@0.1.6 build /Users/pengcao/projects/aiready/packages/doc-drift
+> @aiready/doc-drift@0.1.8 build /Users/pengcao/projects/aiready/packages/doc-drift
 > 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: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"
@@ -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: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"
@@ -51,15 +51,15 @@
 
 
 
-ESM dist/cli.mjs            1.39 KB
-ESM dist/chunk-BBGJNBVI.mjs 5.95 KB
+CJS dist/cli.js   7.58 KB
+CJS dist/index.js 5.55 KB
+CJS ⚡️ Build success in 511ms
+ESM dist/chunk-5EFFNN6L.mjs 4.85 KB
 ESM dist/index.mjs          88.00 B
-ESM ⚡️ Build success in 49ms
-CJS dist/cli.js   8.76 KB
-CJS dist/index.js 6.73 KB
-CJS ⚡️ Build success in 51ms
+ESM dist/cli.mjs            1.39 KB
+ESM ⚡️ Build success in 495ms
 DTS Build start
-DTS ⚡️ Build success in 4363ms
+DTS ⚡️ Build success in 10279ms
 DTS dist/cli.d.ts    108.00 B
 DTS dist/index.d.ts  950.00 B
 DTS dist/cli.d.mts   108.00 B

package/.turbo/turbo-lint.log

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

package/.turbo/turbo-test.log

@@ -1,16 +1,17 @@
 
 
-> @aiready/doc-drift@0.1.6 test /Users/pengcao/projects/aiready/packages/doc-drift
+> @aiready/doc-drift@0.1.8 test /Users/pengcao/projects/aiready/packages/doc-drift
 > vitest run
 
 [?25l
  RUN  v4.0.18 /Users/pengcao/projects/aiready/packages/doc-drift
 
- ✓ src/__tests__/analyzer.test.ts (1 test) 19ms
+ ✓ src/__tests__/analyzer.test.ts (1 test) 364ms
+     ✓ detects missing param documentation and uncommented complexity  361ms
 
  Test Files  1 passed (1)
       Tests  1 passed (1)
-   Start at  22:19:16
-   Duration  1.48s (transform 208ms, setup 0ms, import 1.21s, tests 19ms, environment 0ms)
+   Start at  00:56:24
+   Duration  5.43s (transform 957ms, setup 0ms, import 3.81s, tests 364ms, environment 0ms)
 
 [?25h

package/dist/chunk-5EFFNN6L.mjs

@@ -0,0 +1,145 @@
+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 {
+  scanFiles,
+  calculateDocDrift,
+  getFileCommitTimestamps,
+  getLineRangeLastModifiedCached
+} from "@aiready/core";
+import { readFileSync } from "fs";
+import { parse } from "@typescript-eslint/typescript-estree";
+async function analyzeDocDrift(options) {
+  const rootDir = options.rootDir;
+  const files = await scanFiles(options);
+  const issues = [];
+  const results = [];
+  const staleMonths = options.staleMonths ?? 6;
+  const staleSeconds = staleMonths * 30 * 24 * 60 * 60;
+  let uncommentedExports = 0;
+  let totalExports = 0;
+  let outdatedComments = 0;
+  let undocumentedComplexity = 0;
+  const now = Math.floor(Date.now() / 1e3);
+  let processed = 0;
+  for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `doc-drift: analyzing files`);
+    let code;
+    try {
+      code = readFileSync(file, "utf-8");
+    } catch {
+      continue;
+    }
+    let ast;
+    try {
+      ast = parse(code, {
+        jsx: file.endsWith(".tsx") || file.endsWith(".jsx"),
+        loc: true,
+        comment: true
+      });
+    } catch {
+      continue;
+    }
+    const comments = ast.comments || [];
+    let fileLineStamps;
+    for (const node of ast.body) {
+      if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
+        const decl = node.declaration;
+        if (!decl) continue;
+        if (decl.type === "FunctionDeclaration" || decl.type === "ClassDeclaration" || decl.type === "VariableDeclaration") {
+          totalExports++;
+          const nodeLine = node.loc.start.line;
+          const jsdocs = comments.filter(
+            (c) => c.type === "Block" && c.value.startsWith("*") && c.loc.end.line === nodeLine - 1
+          );
+          if (jsdocs.length === 0) {
+            uncommentedExports++;
+            if (decl.type === "FunctionDeclaration" && decl.body?.loc) {
+              const lines = decl.body.loc.end.line - decl.body.loc.start.line;
+              if (lines > 20) undocumentedComplexity++;
+            }
+          } else {
+            const jsdoc = jsdocs[0];
+            const jsdocText = jsdoc.value;
+            if (decl.type === "FunctionDeclaration") {
+              const params = decl.params.map((p) => p.name || p.left && p.left.name).filter(Boolean);
+              const paramTags = Array.from(
+                jsdocText.matchAll(/@param\s+(?:\{[^}]+\}\s+)?([a-zA-Z0-9_]+)/g)
+              ).map((m) => m[1]);
+              const missingParams = params.filter(
+                (p) => !paramTags.includes(p)
+              );
+              if (missingParams.length > 0) {
+                outdatedComments++;
+                issues.push({
+                  type: "doc-drift",
+                  severity: "major",
+                  message: `JSDoc @param mismatch: function has parameters (${missingParams.join(", ")}) not documented in JSDoc.`,
+                  location: { file, line: nodeLine }
+                });
+                continue;
+              }
+            }
+            if (!fileLineStamps) {
+              fileLineStamps = getFileCommitTimestamps(file);
+            }
+            const commentModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
+              jsdoc.loc.start.line,
+              jsdoc.loc.end.line
+            );
+            const bodyModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
+              decl.loc.start.line,
+              decl.loc.end.line
+            );
+            if (commentModified > 0 && bodyModified > 0) {
+              if (now - commentModified > staleSeconds && bodyModified - commentModified > staleSeconds / 2) {
+                outdatedComments++;
+                issues.push({
+                  type: "doc-drift",
+                  severity: "minor",
+                  message: `JSDoc is significantly older than the function body implementation. Code may have drifted.`,
+                  location: { file, line: jsdoc.loc.start.line }
+                });
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+  const riskResult = calculateDocDrift({
+    uncommentedExports,
+    totalExports,
+    outdatedComments,
+    undocumentedComplexity
+  });
+  return {
+    summary: {
+      filesAnalyzed: files.length,
+      functionsAnalyzed: totalExports,
+      score: riskResult.score,
+      rating: riskResult.rating
+    },
+    issues,
+    rawData: {
+      uncommentedExports,
+      totalExports,
+      outdatedComments,
+      undocumentedComplexity
+    },
+    recommendations: riskResult.recommendations
+  };
+}
+
+export {
+  __require,
+  analyzeDocDrift
+};

package/dist/chunk-VLBPAYS3.mjs

@@ -0,0 +1,209 @@
+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 { calculateDocDrift } from "@aiready/core";
+import { readdirSync, statSync, readFileSync } from "fs";
+import { join, extname } from "path";
+import { parse } from "@typescript-eslint/typescript-estree";
+import { execSync } from "child_process";
+var SRC_EXTENSIONS = /* @__PURE__ */ new Set([".ts", ".tsx", ".js", ".jsx"]);
+var DEFAULT_EXCLUDES = [
+  "node_modules",
+  "dist",
+  ".git",
+  "coverage",
+  ".turbo",
+  "build"
+];
+function collectFiles(dir, options, depth = 0) {
+  if (depth > 20) return [];
+  const excludes = [...DEFAULT_EXCLUDES, ...options.exclude ?? []];
+  let entries;
+  try {
+    entries = readdirSync(dir);
+  } catch {
+    return [];
+  }
+  const files = [];
+  for (const entry of entries) {
+    if (excludes.some((ex) => entry === ex || entry.includes(ex))) continue;
+    const full = join(dir, entry);
+    let stat;
+    try {
+      stat = statSync(full);
+    } catch {
+      continue;
+    }
+    if (stat.isDirectory()) {
+      files.push(...collectFiles(full, options, depth + 1));
+    } else if (stat.isFile() && SRC_EXTENSIONS.has(extname(full))) {
+      if (!options.include || options.include.some((p) => full.includes(p))) {
+        files.push(full);
+      }
+    }
+  }
+  return files;
+}
+function getFileCommitTimestamps(file) {
+  const lineStamps = {};
+  try {
+    const output = execSync(`git blame -t "${file}"`, {
+      encoding: "utf-8",
+      stdio: ["ignore", "pipe", "ignore"]
+    });
+    const lines = output.split("\n");
+    for (const line of lines) {
+      if (!line) continue;
+      const match = line.match(/^\S+\s+\(.*?(\d{10,})\s+[-+]\d+\s+(\d+)\)/);
+      if (match) {
+        const ts = parseInt(match[1], 10);
+        const ln = parseInt(match[2], 10);
+        lineStamps[ln] = ts;
+      }
+    }
+  } catch {
+  }
+  return lineStamps;
+}
+function getLineRangeLastModifiedCached(lineStamps, startLine, endLine) {
+  let latest = 0;
+  for (let i = startLine; i <= endLine; i++) {
+    if (lineStamps[i] && lineStamps[i] > latest) {
+      latest = lineStamps[i];
+    }
+  }
+  return latest;
+}
+async function analyzeDocDrift(options) {
+  const rootDir = options.rootDir;
+  const files = collectFiles(rootDir, options);
+  const staleMonths = options.staleMonths ?? 6;
+  const staleSeconds = staleMonths * 30 * 24 * 60 * 60;
+  let uncommentedExports = 0;
+  let totalExports = 0;
+  let outdatedComments = 0;
+  let undocumentedComplexity = 0;
+  const issues = [];
+  const now = Math.floor(Date.now() / 1e3);
+  let processed = 0;
+  for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `doc-drift: analyzing ${file.substring(rootDir.length + 1)}`);
+    let code;
+    try {
+      code = readFileSync(file, "utf-8");
+    } catch {
+      continue;
+    }
+    let ast;
+    try {
+      ast = parse(code, {
+        jsx: file.endsWith(".tsx") || file.endsWith(".jsx"),
+        loc: true,
+        comment: true
+      });
+    } catch {
+      continue;
+    }
+    const comments = ast.comments || [];
+    let fileLineStamps;
+    for (const node of ast.body) {
+      if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
+        const decl = node.declaration;
+        if (!decl) continue;
+        if (decl.type === "FunctionDeclaration" || decl.type === "ClassDeclaration" || decl.type === "VariableDeclaration") {
+          totalExports++;
+          const nodeLine = node.loc.start.line;
+          const jsdocs = comments.filter(
+            (c) => c.type === "Block" && c.value.startsWith("*") && c.loc.end.line === nodeLine - 1
+          );
+          if (jsdocs.length === 0) {
+            uncommentedExports++;
+            if (decl.type === "FunctionDeclaration" && decl.body?.loc) {
+              const lines = decl.body.loc.end.line - decl.body.loc.start.line;
+              if (lines > 20) undocumentedComplexity++;
+            }
+          } else {
+            const jsdoc = jsdocs[0];
+            const jsdocText = jsdoc.value;
+            if (decl.type === "FunctionDeclaration") {
+              const params = decl.params.map((p) => p.name || p.left && p.left.name).filter(Boolean);
+              const paramTags = Array.from(
+                jsdocText.matchAll(/@param\s+(?:\{[^}]+\}\s+)?([a-zA-Z0-9_]+)/g)
+              ).map((m) => m[1]);
+              const missingParams = params.filter(
+                (p) => !paramTags.includes(p)
+              );
+              if (missingParams.length > 0) {
+                outdatedComments++;
+                issues.push({
+                  type: "doc-drift",
+                  severity: "major",
+                  message: `JSDoc @param mismatch: function has parameters (${missingParams.join(", ")}) not documented in JSDoc.`,
+                  location: { file, line: nodeLine }
+                });
+                continue;
+              }
+            }
+            if (!fileLineStamps) {
+              fileLineStamps = getFileCommitTimestamps(file);
+            }
+            const commentModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
+              jsdoc.loc.start.line,
+              jsdoc.loc.end.line
+            );
+            const bodyModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
+              decl.loc.start.line,
+              decl.loc.end.line
+            );
+            if (commentModified > 0 && bodyModified > 0) {
+              if (now - commentModified > staleSeconds && bodyModified - commentModified > staleSeconds / 2) {
+                outdatedComments++;
+                issues.push({
+                  type: "doc-drift",
+                  severity: "minor",
+                  message: `JSDoc is significantly older than the function body implementation. Code may have drifted.`,
+                  location: { file, line: jsdoc.loc.start.line }
+                });
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+  const riskResult = calculateDocDrift({
+    uncommentedExports,
+    totalExports,
+    outdatedComments,
+    undocumentedComplexity
+  });
+  return {
+    summary: {
+      filesAnalyzed: files.length,
+      functionsAnalyzed: totalExports,
+      score: riskResult.score,
+      rating: riskResult.rating
+    },
+    issues,
+    rawData: {
+      uncommentedExports,
+      totalExports,
+      outdatedComments,
+      undocumentedComplexity
+    },
+    recommendations: riskResult.recommendations
+  };
+}
+
+export {
+  __require,
+  analyzeDocDrift
+};

package/dist/cli.js

@@ -38,76 +38,23 @@ var import_commander = require("commander");
 // src/analyzer.ts
 var import_core = require("@aiready/core");
 var import_fs = require("fs");
-var import_path = require("path");
 var import_typescript_estree = require("@typescript-eslint/typescript-estree");
-var import_child_process = require("child_process");
-var SRC_EXTENSIONS = /* @__PURE__ */ new Set([".ts", ".tsx", ".js", ".jsx"]);
-var DEFAULT_EXCLUDES = [
-  "node_modules",
-  "dist",
-  ".git",
-  "coverage",
-  ".turbo",
-  "build"
-];
-function collectFiles(dir, options, depth = 0) {
-  if (depth > 20) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...options.exclude ?? []];
-  let entries;
-  try {
-    entries = (0, import_fs.readdirSync)(dir);
-  } catch {
-    return [];
-  }
-  const files = [];
-  for (const entry of entries) {
-    if (excludes.some((ex) => entry === ex || entry.includes(ex))) continue;
-    const full = (0, import_path.join)(dir, entry);
-    let stat;
-    try {
-      stat = (0, import_fs.statSync)(full);
-    } catch {
-      continue;
-    }
-    if (stat.isDirectory()) {
-      files.push(...collectFiles(full, options, depth + 1));
-    } else if (stat.isFile() && SRC_EXTENSIONS.has((0, import_path.extname)(full))) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
-function getLineRangeLastModified(file, startLine, endLine) {
-  try {
-    const output = (0, import_child_process.execSync)(
-      `git log -1 --format=%ct -L ${startLine},${endLine}:"${file}"`,
-      {
-        encoding: "utf-8",
-        stdio: ["ignore", "pipe", "ignore"]
-      }
-    );
-    const match = output.trim().split("\n")[0];
-    if (match && !isNaN(parseInt(match, 10))) {
-      return parseInt(match, 10);
-    }
-  } catch {
-  }
-  return 0;
-}
 async function analyzeDocDrift(options) {
   const rootDir = options.rootDir;
-  const files = collectFiles(rootDir, options);
+  const files = await (0, import_core.scanFiles)(options);
+  const issues = [];
+  const results = [];
   const staleMonths = options.staleMonths ?? 6;
   const staleSeconds = staleMonths * 30 * 24 * 60 * 60;
   let uncommentedExports = 0;
   let totalExports = 0;
   let outdatedComments = 0;
   let undocumentedComplexity = 0;
-  const issues = [];
   const now = Math.floor(Date.now() / 1e3);
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `doc-drift: analyzing files`);
     let code;
     try {
       code = (0, import_fs.readFileSync)(file, "utf-8");
@@ -125,6 +72,7 @@ async function analyzeDocDrift(options) {
       continue;
     }
     const comments = ast.comments || [];
+    let fileLineStamps;
     for (const node of ast.body) {
       if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
         const decl = node.declaration;
@@ -163,13 +111,16 @@ async function analyzeDocDrift(options) {
                 continue;
               }
             }
-            const commentModified = getLineRangeLastModified(
-              file,
+            if (!fileLineStamps) {
+              fileLineStamps = (0, import_core.getFileCommitTimestamps)(file);
+            }
+            const commentModified = (0, import_core.getLineRangeLastModifiedCached)(
+              fileLineStamps,
               jsdoc.loc.start.line,
               jsdoc.loc.end.line
             );
-            const bodyModified = getLineRangeLastModified(
-              file,
+            const bodyModified = (0, import_core.getLineRangeLastModifiedCached)(
+              fileLineStamps,
               decl.loc.start.line,
               decl.loc.end.line
             );

package/dist/cli.mjs

@@ -1,7 +1,7 @@
 import {
   __require,
   analyzeDocDrift
-} from "./chunk-BBGJNBVI.mjs";
+} from "./chunk-5EFFNN6L.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.js

@@ -27,76 +27,23 @@ module.exports = __toCommonJS(index_exports);
 // src/analyzer.ts
 var import_core = require("@aiready/core");
 var import_fs = require("fs");
-var import_path = require("path");
 var import_typescript_estree = require("@typescript-eslint/typescript-estree");
-var import_child_process = require("child_process");
-var SRC_EXTENSIONS = /* @__PURE__ */ new Set([".ts", ".tsx", ".js", ".jsx"]);
-var DEFAULT_EXCLUDES = [
-  "node_modules",
-  "dist",
-  ".git",
-  "coverage",
-  ".turbo",
-  "build"
-];
-function collectFiles(dir, options, depth = 0) {
-  if (depth > 20) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...options.exclude ?? []];
-  let entries;
-  try {
-    entries = (0, import_fs.readdirSync)(dir);
-  } catch {
-    return [];
-  }
-  const files = [];
-  for (const entry of entries) {
-    if (excludes.some((ex) => entry === ex || entry.includes(ex))) continue;
-    const full = (0, import_path.join)(dir, entry);
-    let stat;
-    try {
-      stat = (0, import_fs.statSync)(full);
-    } catch {
-      continue;
-    }
-    if (stat.isDirectory()) {
-      files.push(...collectFiles(full, options, depth + 1));
-    } else if (stat.isFile() && SRC_EXTENSIONS.has((0, import_path.extname)(full))) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
-function getLineRangeLastModified(file, startLine, endLine) {
-  try {
-    const output = (0, import_child_process.execSync)(
-      `git log -1 --format=%ct -L ${startLine},${endLine}:"${file}"`,
-      {
-        encoding: "utf-8",
-        stdio: ["ignore", "pipe", "ignore"]
-      }
-    );
-    const match = output.trim().split("\n")[0];
-    if (match && !isNaN(parseInt(match, 10))) {
-      return parseInt(match, 10);
-    }
-  } catch {
-  }
-  return 0;
-}
 async function analyzeDocDrift(options) {
   const rootDir = options.rootDir;
-  const files = collectFiles(rootDir, options);
+  const files = await (0, import_core.scanFiles)(options);
+  const issues = [];
+  const results = [];
   const staleMonths = options.staleMonths ?? 6;
   const staleSeconds = staleMonths * 30 * 24 * 60 * 60;
   let uncommentedExports = 0;
   let totalExports = 0;
   let outdatedComments = 0;
   let undocumentedComplexity = 0;
-  const issues = [];
   const now = Math.floor(Date.now() / 1e3);
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `doc-drift: analyzing files`);
     let code;
     try {
       code = (0, import_fs.readFileSync)(file, "utf-8");
@@ -114,6 +61,7 @@ async function analyzeDocDrift(options) {
       continue;
     }
     const comments = ast.comments || [];
+    let fileLineStamps;
     for (const node of ast.body) {
       if (node.type === "ExportNamedDeclaration" || node.type === "ExportDefaultDeclaration") {
         const decl = node.declaration;
@@ -152,13 +100,16 @@ async function analyzeDocDrift(options) {
                 continue;
               }
             }
-            const commentModified = getLineRangeLastModified(
-              file,
+            if (!fileLineStamps) {
+              fileLineStamps = (0, import_core.getFileCommitTimestamps)(file);
+            }
+            const commentModified = (0, import_core.getLineRangeLastModifiedCached)(
+              fileLineStamps,
               jsdoc.loc.start.line,
               jsdoc.loc.end.line
             );
-            const bodyModified = getLineRangeLastModified(
-              file,
+            const bodyModified = (0, import_core.getLineRangeLastModifiedCached)(
+              fileLineStamps,
               decl.loc.start.line,
               decl.loc.end.line
             );

package/dist/index.mjs

@@ -1,6 +1,6 @@
 import {
   analyzeDocDrift
-} from "./chunk-BBGJNBVI.mjs";
+} from "./chunk-5EFFNN6L.mjs";
 export {
   analyzeDocDrift
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/doc-drift",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "AI-Readiness: Documentation Drift Detection",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -10,7 +10,7 @@
     "commander": "^14.0.0",
     "glob": "^13.0.0",
     "picocolors": "^1.0.0",
-    "@aiready/core": "0.9.33"
+    "@aiready/core": "0.9.35"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -1,85 +1,23 @@
-import { calculateDocDrift } from '@aiready/core';
+import {
+  scanFiles,
+  readFileContent,
+  calculateDocDrift,
+  getFileCommitTimestamps,
+  getLineRangeLastModifiedCached,
+} from '@aiready/core';
 import type { DocDriftOptions, DocDriftReport, DocDriftIssue } from './types';
-import { readdirSync, statSync, readFileSync } from 'fs';
-import { join, extname } from 'path';
+import { readFileSync } from 'fs';
 import { parse } from '@typescript-eslint/typescript-estree';
 import type { TSESTree } from '@typescript-eslint/types';
-import { execSync } from 'child_process';
-
-const SRC_EXTENSIONS = new Set(['.ts', '.tsx', '.js', '.jsx']);
-const DEFAULT_EXCLUDES = [
-  'node_modules',
-  'dist',
-  '.git',
-  'coverage',
-  '.turbo',
-  'build',
-];
-
-function collectFiles(
-  dir: string,
-  options: DocDriftOptions,
-  depth = 0
-): string[] {
-  if (depth > 20) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...(options.exclude ?? [])];
-  let entries: string[];
-  try {
-    entries = readdirSync(dir);
-  } catch {
-    return [];
-  }
-
-  const files: string[] = [];
-  for (const entry of entries) {
-    if (excludes.some((ex) => entry === ex || entry.includes(ex))) continue;
-    const full = join(dir, entry);
-    let stat;
-    try {
-      stat = statSync(full);
-    } catch {
-      continue;
-    }
-    if (stat.isDirectory()) {
-      files.push(...collectFiles(full, options, depth + 1));
-    } else if (stat.isFile() && SRC_EXTENSIONS.has(extname(full))) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
-
-function getLineRangeLastModified(
-  file: string,
-  startLine: number,
-  endLine: number
-): number {
-  try {
-    // format %ct is committer date, UNIX timestamp
-    const output = execSync(
-      `git log -1 --format=%ct -L ${startLine},${endLine}:"${file}"`,
-      {
-        encoding: 'utf-8',
-        stdio: ['ignore', 'pipe', 'ignore'],
-      }
-    );
-    const match = output.trim().split('\n')[0];
-    if (match && !isNaN(parseInt(match, 10))) {
-      return parseInt(match, 10);
-    }
-  } catch {
-    // Ignore errors (file untracked, new file, etc)
-  }
-  return 0; // Unknown or not committed
-}
 
 export async function analyzeDocDrift(
   options: DocDriftOptions
 ): Promise<DocDriftReport> {
   const rootDir = options.rootDir;
-  const files = collectFiles(rootDir, options);
+  // Use core scanFiles which respects .gitignore recursively
+  const files = await scanFiles(options);
+  const issues: DocDriftIssue[] = [];
+  const results: DocDriftIssue[] = [];
   const staleMonths = options.staleMonths ?? 6;
   const staleSeconds = staleMonths * 30 * 24 * 60 * 60;
 
@@ -88,10 +26,13 @@ export async function analyzeDocDrift(
   let outdatedComments = 0;
   let undocumentedComplexity = 0;
 
-  const issues: DocDriftIssue[] = [];
   const now = Math.floor(Date.now() / 1000);
 
+  let processed = 0;
   for (const file of files) {
+    processed++;
+    options.onProgress?.(processed, files.length, `doc-drift: analyzing files`);
+
     let code: string;
     try {
       code = readFileSync(file, 'utf-8');
@@ -111,6 +52,7 @@ export async function analyzeDocDrift(
     }
 
     const comments = ast.comments || [];
+    let fileLineStamps: Record<number, number> | undefined;
 
     for (const node of ast.body) {
       if (
@@ -174,13 +116,16 @@ export async function analyzeDocDrift(
             }
 
             // Timestamp comparison
-            const commentModified = getLineRangeLastModified(
-              file,
+            if (!fileLineStamps) {
+              fileLineStamps = getFileCommitTimestamps(file);
+            }
+            const commentModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
               jsdoc.loc.start.line,
               jsdoc.loc.end.line
             );
-            const bodyModified = getLineRangeLastModified(
-              file,
+            const bodyModified = getLineRangeLastModifiedCached(
+              fileLineStamps,
               decl.loc.start.line,
               decl.loc.end.line
             );