@aiready/testability 0.6.21 → 0.6.23

package/src/analyzer.ts

@@ -3,11 +3,12 @@ import {
   calculateTestabilityIndex,
   Severity,
   IssueType,
-  emitProgress,
+  runBatchAnalysis,
   getParser,
+  isTestFile,
+  detectTestFramework,
 } from '@aiready/core';
-import { readFileSync, existsSync } from 'fs';
-import { join } from 'path';
+import { readFileSync } from 'fs';
 import type {
   TestabilityOptions,
   TestabilityIssue,
@@ -89,62 +90,9 @@ async function analyzeFileTestability(filePath: string): Promise<FileAnalysis> {
   return result;
 }
 
-// ---------------------------------------------------------------------------
-// Test framework detection
-// ---------------------------------------------------------------------------
-
-function detectTestFramework(rootDir: string): boolean {
-  // Check common manifest files
-  const manifests = [
-    {
-      file: 'package.json',
-      deps: ['jest', 'vitest', 'mocha', 'mocha', 'jasmine', 'ava', 'tap'],
-    },
-    { file: 'requirements.txt', deps: ['pytest', 'unittest', 'nose'] },
-    { file: 'pyproject.toml', deps: ['pytest'] },
-    { file: 'pom.xml', deps: ['junit', 'testng'] },
-    { file: 'build.gradle', deps: ['junit', 'testng'] },
-    { file: 'go.mod', deps: ['testing'] }, // go testing is built-in
-  ];
-
-  for (const m of manifests) {
-    const p = join(rootDir, m.file);
-    if (existsSync(p)) {
-      if (m.file === 'go.mod') return true; // built-in
-      try {
-        const content = readFileSync(p, 'utf-8');
-        if (m.deps.some((d) => content.includes(d))) return true;
-      } catch {
-        // Ignore file read errors
-      }
-    }
-  }
-  return false;
-}
-
-// ---------------------------------------------------------------------------
 // Main analyzer
 // ---------------------------------------------------------------------------
 
-const TEST_PATTERNS = [
-  /\.(test|spec)\.(ts|tsx|js|jsx)$/,
-  /_test\.go$/,
-  /test_.*\.py$/,
-  /.*_test\.py$/,
-  /.*Test\.java$/,
-  /.*Tests\.cs$/,
-  /__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> {
@@ -177,29 +125,28 @@ export async function analyzeTestability(
     totalFunctions: number;
   }> = [];
 
-  let processed = 0;
-  for (const f of sourceFiles) {
-    processed++;
-    emitProgress(
-      processed,
-      sourceFiles.length,
-      'testability',
-      'analyzing files',
-      options.onProgress
-    );
-
-    const a = await analyzeFileTestability(f);
-    for (const key of Object.keys(aggregated) as Array<keyof FileAnalysis>) {
-      aggregated[key] += a[key];
-    }
-
-    // Collect file-level data
-    fileDetails.push({
+  await runBatchAnalysis(
+    sourceFiles,
+    'analyzing files',
+    'testability',
+    options.onProgress,
+    async (f: string) => ({
       filePath: f,
-      pureFunctions: a.pureFunctions,
-      totalFunctions: a.totalFunctions,
-    });
-  }
+      analysis: await analyzeFileTestability(f),
+    }),
+    (result: { filePath: string; analysis: FileAnalysis }) => {
+      const a = result.analysis;
+      for (const key of Object.keys(aggregated) as Array<keyof FileAnalysis>) {
+        aggregated[key] += a[key];
+      }
+      // Collect file-level data
+      fileDetails.push({
+        filePath: result.filePath,
+        pureFunctions: a.pureFunctions,
+        totalFunctions: a.totalFunctions,
+      });
+    }
+  );
 
   const hasTestFramework = detectTestFramework(options.rootDir);
 

package/src/cli.ts

@@ -88,28 +88,28 @@ EXAMPLES:
     } else {
       displayStandardConsoleReport({
         title: '🧪 Testability Analysis',
-        score: scoring.summary.score,
-        rating: scoring.summary.rating,
+        score: scoring.score,
+        rating: scoring.rating || report.summary.rating,
         dimensions: [
           {
             name: 'Test Coverage',
-            value: scoring.summary.dimensions.testCoverageRatio,
+            value: report.summary.dimensions.testCoverageRatio,
           },
           {
             name: 'Function Purity',
-            value: scoring.summary.dimensions.purityScore,
+            value: report.summary.dimensions.purityScore,
           },
           {
             name: 'Dependency Injection',
-            value: scoring.summary.dimensions.dependencyInjectionScore,
+            value: report.summary.dimensions.dependencyInjectionScore,
           },
           {
             name: 'Interface Focus',
-            value: scoring.summary.dimensions.interfaceFocusScore,
+            value: report.summary.dimensions.interfaceFocusScore,
           },
           {
             name: 'Observability',
-            value: scoring.summary.dimensions.observabilityScore,
+            value: report.summary.dimensions.observabilityScore,
           },
         ],
         stats: [
@@ -117,7 +117,7 @@ EXAMPLES:
           { label: 'Test Files', value: report.rawData.testFiles },
           {
             label: 'Coverage Ratio',
-            value: Math.round(scoring.summary.coverageRatio * 100) + '%',
+            value: Math.round(report.summary.coverageRatio * 100) + '%',
           },
         ],
         issues: report.issues,