@aiready/testability 0.1.6 → 0.1.8

package/dist/cli.js

@@ -27,63 +27,10 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 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_core = require("@aiready/core");
-var SRC_EXTENSIONS = /* @__PURE__ */ new Set([".ts", ".tsx", ".js", ".jsx"]);
-var DEFAULT_EXCLUDES = [
-  "node_modules",
-  "dist",
-  ".git",
-  "coverage",
-  ".turbo",
-  "build"
-];
-var TEST_PATTERNS = [
-  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
-  /__tests__\//,
-  /\/tests?\//,
-  /\/e2e\//,
-  /\/fixtures\//
-];
-function isTestFile(filePath, extra) {
-  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
-  if (extra) return extra.some((p) => filePath.includes(p));
-  return false;
-}
-function isSourceFile(filePath) {
-  return SRC_EXTENSIONS.has((0, import_path.extname)(filePath));
-}
-function collectFiles(dir, options, depth = 0) {
-  if (depth > (options.maxDepth ?? 20)) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...options.exclude ?? []];
-  const files = [];
-  let entries;
-  try {
-    entries = (0, import_fs.readdirSync)(dir);
-  } catch {
-    return 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() && isSourceFile(full)) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
 function countMethodsInInterface(node) {
   if (node.type === "TSInterfaceDeclaration") {
     return node.body.body.filter(
@@ -235,8 +182,24 @@ function detectTestFramework(rootDir) {
     return false;
   }
 }
+var TEST_PATTERNS = [
+  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
+  /__tests__\//,
+  /\/tests?\//,
+  /\/e2e\//,
+  /\/fixtures\//
+];
+function isTestFile(filePath, extra) {
+  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
+  if (extra) return extra.some((p) => filePath.includes(p));
+  return false;
+}
 async function analyzeTestability(options) {
-  const allFiles = collectFiles(options.rootDir, options);
+  const allFiles = await (0, import_core.scanFiles)({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx}"],
+    includeTests: true
+  });
   const sourceFiles = allFiles.filter(
     (f) => !isTestFile(f, options.testPatterns)
   );
@@ -250,7 +213,14 @@ async function analyzeTestability(options) {
     totalInterfaces: 0,
     externalStateMutations: 0
   };
+  let processed = 0;
   for (const f of sourceFiles) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      sourceFiles.length,
+      `testability: analyzing files`
+    );
     const a = analyzeFileTestability(f);
     for (const key of Object.keys(aggregated)) {
       aggregated[key] += a[key];

package/dist/cli.mjs

@@ -2,7 +2,7 @@
 import {
   analyzeTestability,
   calculateTestabilityScore
-} from "./chunk-YLYLRZRS.mjs";
+} from "./chunk-DDNB7FI4.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.d.mts

@@ -13,6 +13,8 @@ interface TestabilityOptions {
     include?: string[];
     /** File glob patterns to exclude */
     exclude?: string[];
+    /** Progress callback */
+    onProgress?: (processed: number, total: number, message: string) => void;
 }
 interface TestabilityIssue extends Issue {
     type: 'low-testability';

package/dist/index.d.ts

@@ -13,6 +13,8 @@ interface TestabilityOptions {
     include?: string[];
     /** File glob patterns to exclude */
     exclude?: string[];
+    /** Progress callback */
+    onProgress?: (processed: number, total: number, message: string) => void;
 }
 interface TestabilityIssue extends Issue {
     type: 'low-testability';

package/dist/index.js

@@ -26,63 +26,10 @@ __export(index_exports, {
 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_core = require("@aiready/core");
-var SRC_EXTENSIONS = /* @__PURE__ */ new Set([".ts", ".tsx", ".js", ".jsx"]);
-var DEFAULT_EXCLUDES = [
-  "node_modules",
-  "dist",
-  ".git",
-  "coverage",
-  ".turbo",
-  "build"
-];
-var TEST_PATTERNS = [
-  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
-  /__tests__\//,
-  /\/tests?\//,
-  /\/e2e\//,
-  /\/fixtures\//
-];
-function isTestFile(filePath, extra) {
-  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
-  if (extra) return extra.some((p) => filePath.includes(p));
-  return false;
-}
-function isSourceFile(filePath) {
-  return SRC_EXTENSIONS.has((0, import_path.extname)(filePath));
-}
-function collectFiles(dir, options, depth = 0) {
-  if (depth > (options.maxDepth ?? 20)) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...options.exclude ?? []];
-  const files = [];
-  let entries;
-  try {
-    entries = (0, import_fs.readdirSync)(dir);
-  } catch {
-    return 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() && isSourceFile(full)) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
 function countMethodsInInterface(node) {
   if (node.type === "TSInterfaceDeclaration") {
     return node.body.body.filter(
@@ -234,8 +181,24 @@ function detectTestFramework(rootDir) {
     return false;
   }
 }
+var TEST_PATTERNS = [
+  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
+  /__tests__\//,
+  /\/tests?\//,
+  /\/e2e\//,
+  /\/fixtures\//
+];
+function isTestFile(filePath, extra) {
+  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
+  if (extra) return extra.some((p) => filePath.includes(p));
+  return false;
+}
 async function analyzeTestability(options) {
-  const allFiles = collectFiles(options.rootDir, options);
+  const allFiles = await (0, import_core.scanFiles)({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx}"],
+    includeTests: true
+  });
   const sourceFiles = allFiles.filter(
     (f) => !isTestFile(f, options.testPatterns)
   );
@@ -249,7 +212,14 @@ async function analyzeTestability(options) {
     totalInterfaces: 0,
     externalStateMutations: 0
   };
+  let processed = 0;
   for (const f of sourceFiles) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      sourceFiles.length,
+      `testability: analyzing files`
+    );
     const a = analyzeFileTestability(f);
     for (const key of Object.keys(aggregated)) {
       aggregated[key] += a[key];

package/dist/index.mjs

@@ -1,7 +1,7 @@
 import {
   analyzeTestability,
   calculateTestabilityScore
-} from "./chunk-YLYLRZRS.mjs";
+} from "./chunk-DDNB7FI4.mjs";
 export {
   analyzeTestability,
   calculateTestabilityScore

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/testability",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Measures how safely and verifiably AI-generated changes can be made to your codebase",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -40,7 +40,7 @@
     "chalk": "^5.3.0",
     "commander": "^14.0.0",
     "glob": "^13.0.0",
-    "@aiready/core": "0.9.33"
+    "@aiready/core": "0.9.35"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -10,82 +10,17 @@
  * 5. Observability (return values vs. external state mutations)
  */
 
-import { readdirSync, statSync, readFileSync, existsSync } from 'fs';
-import { join, extname, basename } from 'path';
+import { scanFiles, calculateTestabilityIndex } from '@aiready/core';
+import { readFileSync, existsSync } from 'fs';
+import { join } from 'path';
 import { parse } from '@typescript-eslint/typescript-estree';
 import type { TSESTree } from '@typescript-eslint/types';
-import { calculateTestabilityIndex } from '@aiready/core';
 import type {
   TestabilityOptions,
   TestabilityIssue,
   TestabilityReport,
 } from './types';
 
-// ---------------------------------------------------------------------------
-// File classification
-// ---------------------------------------------------------------------------
-
-const SRC_EXTENSIONS = new Set(['.ts', '.tsx', '.js', '.jsx']);
-const DEFAULT_EXCLUDES = [
-  'node_modules',
-  'dist',
-  '.git',
-  'coverage',
-  '.turbo',
-  'build',
-];
-const TEST_PATTERNS = [
-  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
-  /__tests__\//,
-  /\/tests?\//,
-  /\/e2e\//,
-  /\/fixtures\//,
-];
-
-function isTestFile(filePath: string, extra?: string[]): boolean {
-  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
-  if (extra) return extra.some((p) => filePath.includes(p));
-  return false;
-}
-
-function isSourceFile(filePath: string): boolean {
-  return SRC_EXTENSIONS.has(extname(filePath));
-}
-
-function collectFiles(
-  dir: string,
-  options: TestabilityOptions,
-  depth = 0
-): string[] {
-  if (depth > (options.maxDepth ?? 20)) return [];
-  const excludes = [...DEFAULT_EXCLUDES, ...(options.exclude ?? [])];
-  const files: string[] = [];
-  let entries: string[];
-  try {
-    entries = readdirSync(dir);
-  } catch {
-    return 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() && isSourceFile(full)) {
-      if (!options.include || options.include.some((p) => full.includes(p))) {
-        files.push(full);
-      }
-    }
-  }
-  return files;
-}
-
 // ---------------------------------------------------------------------------
 // Per-file analysis
 // ---------------------------------------------------------------------------
@@ -324,10 +259,29 @@ function detectTestFramework(rootDir: string): boolean {
 // Main analyzer
 // ---------------------------------------------------------------------------
 
+const TEST_PATTERNS = [
+  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
+  /__tests__\//,
+  /\/tests?\//,
+  /\/e2e\//,
+  /\/fixtures\//,
+];
+
+function isTestFile(filePath: string, extra?: string[]): boolean {
+  if (TEST_PATTERNS.some((p) => p.test(filePath))) return true;
+  if (extra) return extra.some((p) => filePath.includes(p));
+  return false;
+}
+
 export async function analyzeTestability(
   options: TestabilityOptions
 ): Promise<TestabilityReport> {
-  const allFiles = collectFiles(options.rootDir, options);
+  // Use core scanFiles which respects .gitignore recursively
+  const allFiles = await scanFiles({
+    ...options,
+    include: options.include || ['**/*.{ts,tsx,js,jsx}'],
+    includeTests: true,
+  });
 
   const sourceFiles = allFiles.filter(
     (f) => !isTestFile(f, options.testPatterns)
@@ -344,7 +298,15 @@ export async function analyzeTestability(
     externalStateMutations: 0,
   };
 
+  let processed = 0;
   for (const f of sourceFiles) {
+    processed++;
+    options.onProgress?.(
+      processed,
+      sourceFiles.length,
+      `testability: analyzing files`
+    );
+
     const a = analyzeFileTestability(f);
     for (const key of Object.keys(aggregated) as Array<keyof FileAnalysis>) {
       aggregated[key] += a[key];

package/src/scoring.ts

@@ -1,4 +1,3 @@
-import { calculateTestabilityIndex } from '@aiready/core';
 import type { ToolScoringOutput } from '@aiready/core';
 import type { TestabilityReport } from './types';
 

package/src/types.ts

@@ -1,4 +1,4 @@
-import type { ScanOptions, Issue } from '@aiready/core';
+import type { Issue } from '@aiready/core';
 
 export interface TestabilityOptions {
   /** Root directory to scan */
@@ -13,6 +13,8 @@ export interface TestabilityOptions {
   include?: string[];
   /** File glob patterns to exclude */
   exclude?: string[];
+  /** Progress callback */
+  onProgress?: (processed: number, total: number, message: string) => void;
 }
 
 export interface TestabilityIssue extends Issue {