@aiready/testability 0.4.4 → 0.4.9

package/.turbo/turbo-build.log

@@ -1,6 +1,6 @@
 
 
-> @aiready/testability@0.4.3 build /Users/pengcao/projects/aiready/packages/testability
+> @aiready/testability@0.4.8 build /Users/pengcao/projects/aiready/packages/testability
 > tsup src/index.ts src/cli.ts --format cjs,esm --dts
 
 CLI Building entry: src/cli.ts, src/index.ts
@@ -9,15 +9,15 @@
 CLI Target: es2020
 CJS Build start
 ESM Build start
-ESM dist/index.mjs          1.28 KB
+CJS dist/index.js 13.60 KB
+CJS dist/cli.js   18.49 KB
+CJS ⚡️ Build success in 144ms
+ESM dist/chunk-LJLAJRZR.mjs 11.12 KB
 ESM dist/cli.mjs            5.75 KB
-ESM dist/chunk-2QLLQPTL.mjs 11.08 KB
-ESM ⚡️ Build success in 120ms
-CJS dist/index.js 13.56 KB
-CJS dist/cli.js   18.44 KB
-CJS ⚡️ Build success in 120ms
+ESM dist/index.mjs          1.28 KB
+ESM ⚡️ Build success in 145ms
 DTS Build start
-DTS ⚡️ Build success in 4473ms
+DTS ⚡️ Build success in 3157ms
 DTS dist/cli.d.ts    20.00 B
 DTS dist/index.d.ts  2.78 KB
 DTS dist/cli.d.mts   20.00 B

package/.turbo/turbo-test.log

@@ -1,16 +1,16 @@
 
 
-> @aiready/testability@0.4.3 test /Users/pengcao/projects/aiready/packages/testability
+> @aiready/testability@0.4.8 test /Users/pengcao/projects/aiready/packages/testability
 > vitest run
 
 [?25l
  RUN  v4.0.18 /Users/pengcao/projects/aiready/packages/testability
 
- ✓ src/__tests__/analyzer.test.ts (3 tests) 90ms
+ ✓ src/__tests__/analyzer.test.ts (3 tests) 33ms
 
  Test Files  1 passed (1)
       Tests  3 passed (3)
-   Start at  11:12:30
-   Duration  2.22s (transform 598ms, setup 0ms, import 1.53s, tests 90ms, environment 0ms)
+   Start at  13:29:33
+   Duration  802ms (transform 198ms, setup 0ms, import 526ms, tests 33ms, environment 0ms)
 
 [?25h

package/dist/chunk-LJLAJRZR.mjs

@@ -0,0 +1,342 @@
+// src/analyzer.ts
+import {
+  scanFiles,
+  calculateTestabilityIndex,
+  Severity,
+  IssueType,
+  emitProgress
+} from "@aiready/core";
+import { readFileSync, existsSync } from "fs";
+import { join } from "path";
+import { parse } from "@typescript-eslint/typescript-estree";
+function countMethodsInInterface(node) {
+  if (node.type === "TSInterfaceDeclaration") {
+    return node.body.body.filter(
+      (m) => m.type === "TSMethodSignature" || m.type === "TSPropertySignature"
+    ).length;
+  }
+  if (node.type === "TSTypeAliasDeclaration" && node.typeAnnotation.type === "TSTypeLiteral") {
+    return node.typeAnnotation.members.length;
+  }
+  return 0;
+}
+function hasDependencyInjection(node) {
+  for (const member of node.body.body) {
+    if (member.type === "MethodDefinition" && member.key.type === "Identifier" && member.key.name === "constructor") {
+      const fn = member.value;
+      if (fn.params && fn.params.length > 0) {
+        const typedParams = fn.params.filter((p) => {
+          const param = p;
+          return param.typeAnnotation != null || param.parameter?.typeAnnotation != null;
+        });
+        if (typedParams.length > 0) return true;
+      }
+    }
+  }
+  return false;
+}
+function isPureFunction(fn) {
+  let hasReturn = false;
+  let hasSideEffect = false;
+  function walk(node) {
+    if (node.type === "ReturnStatement" && node.argument) hasReturn = true;
+    if (node.type === "AssignmentExpression" && node.left.type === "MemberExpression")
+      hasSideEffect = true;
+    if (node.type === "CallExpression" && node.callee.type === "MemberExpression" && node.callee.object.type === "Identifier" && ["console", "process", "window", "document", "fs"].includes(
+      node.callee.object.name
+    ))
+      hasSideEffect = true;
+    for (const key of Object.keys(node)) {
+      if (key === "parent") continue;
+      const child = node[key];
+      if (child && typeof child === "object") {
+        if (Array.isArray(child)) {
+          child.forEach((c) => c?.type && walk(c));
+        } else if (child.type) {
+          walk(child);
+        }
+      }
+    }
+  }
+  if (fn.body?.type === "BlockStatement") {
+    fn.body.body.forEach((s) => walk(s));
+  } else if (fn.body) {
+    hasReturn = true;
+  }
+  return hasReturn && !hasSideEffect;
+}
+function hasExternalStateMutation(fn) {
+  let found = false;
+  function walk(node) {
+    if (found) return;
+    if (node.type === "AssignmentExpression" && node.left.type === "MemberExpression")
+      found = true;
+    for (const key of Object.keys(node)) {
+      if (key === "parent") continue;
+      const child = node[key];
+      if (child && typeof child === "object") {
+        if (Array.isArray(child)) child.forEach((c) => c?.type && walk(c));
+        else if (child.type) walk(child);
+      }
+    }
+  }
+  if (fn.body?.type === "BlockStatement") fn.body.body.forEach((s) => walk(s));
+  return found;
+}
+function analyzeFileTestability(filePath) {
+  const result = {
+    pureFunctions: 0,
+    totalFunctions: 0,
+    injectionPatterns: 0,
+    totalClasses: 0,
+    bloatedInterfaces: 0,
+    totalInterfaces: 0,
+    externalStateMutations: 0
+  };
+  let code;
+  try {
+    code = readFileSync(filePath, "utf-8");
+  } catch {
+    return result;
+  }
+  let ast;
+  try {
+    ast = parse(code, {
+      jsx: filePath.endsWith(".tsx") || filePath.endsWith(".jsx"),
+      range: false,
+      loc: false
+    });
+  } catch {
+    return result;
+  }
+  function visit(node) {
+    if (node.type === "FunctionDeclaration" || node.type === "FunctionExpression" || node.type === "ArrowFunctionExpression") {
+      result.totalFunctions++;
+      if (isPureFunction(node)) result.pureFunctions++;
+      if (hasExternalStateMutation(node)) result.externalStateMutations++;
+    }
+    if (node.type === "ClassDeclaration" || node.type === "ClassExpression") {
+      result.totalClasses++;
+      if (hasDependencyInjection(node)) result.injectionPatterns++;
+    }
+    if (node.type === "TSInterfaceDeclaration" || node.type === "TSTypeAliasDeclaration") {
+      result.totalInterfaces++;
+      const methodCount = countMethodsInInterface(node);
+      if (methodCount > 10) result.bloatedInterfaces++;
+    }
+    for (const key of Object.keys(node)) {
+      if (key === "parent") continue;
+      const child = node[key];
+      if (child && typeof child === "object") {
+        if (Array.isArray(child)) child.forEach((c) => c?.type && visit(c));
+        else if (child.type) visit(child);
+      }
+    }
+  }
+  ast.body.forEach(visit);
+  return result;
+}
+function detectTestFramework(rootDir) {
+  const pkgPath = join(rootDir, "package.json");
+  if (!existsSync(pkgPath)) return false;
+  try {
+    const pkg = JSON.parse(readFileSync(pkgPath, "utf-8"));
+    const allDeps = {
+      ...pkg.dependencies ?? {},
+      ...pkg.devDependencies ?? {}
+    };
+    const testFrameworks = [
+      "jest",
+      "vitest",
+      "mocha",
+      "jasmine",
+      "ava",
+      "tap",
+      "pytest",
+      "unittest"
+    ];
+    return testFrameworks.some((fw) => allDeps[fw]);
+  } catch {
+    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 = await scanFiles({
+    ...options,
+    include: options.include || ["**/*.{ts,tsx,js,jsx}"],
+    includeTests: true
+  });
+  const sourceFiles = allFiles.filter(
+    (f) => !isTestFile(f, options.testPatterns)
+  );
+  const testFiles = allFiles.filter((f) => isTestFile(f, options.testPatterns));
+  const aggregated = {
+    pureFunctions: 0,
+    totalFunctions: 0,
+    injectionPatterns: 0,
+    totalClasses: 0,
+    bloatedInterfaces: 0,
+    totalInterfaces: 0,
+    externalStateMutations: 0
+  };
+  let processed = 0;
+  for (const f of sourceFiles) {
+    processed++;
+    emitProgress(
+      processed,
+      sourceFiles.length,
+      "testability",
+      "analyzing files",
+      options.onProgress
+    );
+    const a = analyzeFileTestability(f);
+    for (const key of Object.keys(aggregated)) {
+      aggregated[key] += a[key];
+    }
+  }
+  const hasTestFramework = detectTestFramework(options.rootDir);
+  const indexResult = calculateTestabilityIndex({
+    testFiles: testFiles.length,
+    sourceFiles: sourceFiles.length,
+    pureFunctions: aggregated.pureFunctions,
+    totalFunctions: Math.max(1, aggregated.totalFunctions),
+    injectionPatterns: aggregated.injectionPatterns,
+    totalClasses: Math.max(1, aggregated.totalClasses),
+    bloatedInterfaces: aggregated.bloatedInterfaces,
+    totalInterfaces: Math.max(1, aggregated.totalInterfaces),
+    externalStateMutations: aggregated.externalStateMutations,
+    hasTestFramework
+  });
+  const issues = [];
+  const minCoverage = options.minCoverageRatio ?? 0.3;
+  const actualRatio = sourceFiles.length > 0 ? testFiles.length / sourceFiles.length : 0;
+  if (!hasTestFramework) {
+    issues.push({
+      type: IssueType.LowTestability,
+      dimension: "framework",
+      severity: Severity.Critical,
+      message: "No testing framework detected in package.json \u2014 AI changes cannot be verified at all.",
+      location: { file: options.rootDir, line: 0 },
+      suggestion: "Add Jest, Vitest, or another testing framework as a devDependency."
+    });
+  }
+  if (actualRatio < minCoverage) {
+    const needed = Math.ceil(sourceFiles.length * minCoverage) - testFiles.length;
+    issues.push({
+      type: IssueType.LowTestability,
+      dimension: "test-coverage",
+      severity: actualRatio === 0 ? Severity.Critical : Severity.Major,
+      message: `Test ratio is ${Math.round(actualRatio * 100)}% (${testFiles.length} test files for ${sourceFiles.length} source files). Need at least ${Math.round(minCoverage * 100)}%.`,
+      location: { file: options.rootDir, line: 0 },
+      suggestion: `Add ~${needed} test file(s) to reach the ${Math.round(minCoverage * 100)}% minimum for safe AI assistance.`
+    });
+  }
+  if (indexResult.dimensions.purityScore < 50) {
+    issues.push({
+      type: IssueType.LowTestability,
+      dimension: "purity",
+      severity: Severity.Major,
+      message: `Only ${indexResult.dimensions.purityScore}% of functions are pure \u2014 side-effectful functions require complex test setup.`,
+      location: { file: options.rootDir, line: 0 },
+      suggestion: "Extract pure transformation logic from I/O and mutation code."
+    });
+  }
+  if (indexResult.dimensions.observabilityScore < 50) {
+    issues.push({
+      type: IssueType.LowTestability,
+      dimension: "observability",
+      severity: Severity.Major,
+      message: `Many functions mutate external state directly \u2014 outputs are invisible to unit tests.`,
+      location: { file: options.rootDir, line: 0 },
+      suggestion: "Prefer returning values over mutating shared state."
+    });
+  }
+  return {
+    summary: {
+      sourceFiles: sourceFiles.length,
+      testFiles: testFiles.length,
+      coverageRatio: Math.round(actualRatio * 100) / 100,
+      score: indexResult.score,
+      rating: indexResult.rating,
+      aiChangeSafetyRating: indexResult.aiChangeSafetyRating,
+      dimensions: indexResult.dimensions
+    },
+    issues,
+    rawData: {
+      sourceFiles: sourceFiles.length,
+      testFiles: testFiles.length,
+      ...aggregated,
+      hasTestFramework
+    },
+    recommendations: indexResult.recommendations
+  };
+}
+
+// src/scoring.ts
+import { ToolName } from "@aiready/core";
+function calculateTestabilityScore(report) {
+  const { summary, rawData, recommendations } = report;
+  const factors = [
+    {
+      name: "Test Coverage",
+      impact: Math.round(summary.dimensions.testCoverageRatio - 50),
+      description: `${rawData.testFiles} test files / ${rawData.sourceFiles} source files (${Math.round(summary.coverageRatio * 100)}%)`
+    },
+    {
+      name: "Function Purity",
+      impact: Math.round(summary.dimensions.purityScore - 50),
+      description: `${rawData.pureFunctions}/${rawData.totalFunctions} functions are pure`
+    },
+    {
+      name: "Dependency Injection",
+      impact: Math.round(summary.dimensions.dependencyInjectionScore - 50),
+      description: `${rawData.injectionPatterns}/${rawData.totalClasses} classes use DI`
+    },
+    {
+      name: "Interface Focus",
+      impact: Math.round(summary.dimensions.interfaceFocusScore - 50),
+      description: `${rawData.bloatedInterfaces} interfaces have >10 methods`
+    },
+    {
+      name: "Observability",
+      impact: Math.round(summary.dimensions.observabilityScore - 50),
+      description: `${rawData.externalStateMutations} functions mutate external state`
+    }
+  ];
+  const recs = recommendations.map(
+    (action) => ({
+      action,
+      estimatedImpact: summary.aiChangeSafetyRating === "blind-risk" ? 15 : 8,
+      priority: summary.aiChangeSafetyRating === "blind-risk" || summary.aiChangeSafetyRating === "high-risk" ? "high" : "medium"
+    })
+  );
+  return {
+    toolName: ToolName.TestabilityIndex,
+    score: summary.score,
+    rawMetrics: {
+      ...rawData,
+      rating: summary.rating,
+      aiChangeSafetyRating: summary.aiChangeSafetyRating,
+      coverageRatio: summary.coverageRatio
+    },
+    factors,
+    recommendations: recs
+  };
+}
+
+export {
+  analyzeTestability,
+  calculateTestabilityScore
+};

package/dist/cli.js

@@ -216,10 +216,12 @@ async function analyzeTestability(options) {
   let processed = 0;
   for (const f of sourceFiles) {
     processed++;
-    options.onProgress?.(
+    (0, import_core.emitProgress)(
       processed,
       sourceFiles.length,
-      `testability: analyzing files`
+      "testability",
+      "analyzing files",
+      options.onProgress
     );
     const a = analyzeFileTestability(f);
     for (const key of Object.keys(aggregated)) {

package/dist/cli.mjs

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

package/dist/index.js

@@ -220,10 +220,12 @@ async function analyzeTestability(options) {
   let processed = 0;
   for (const f of sourceFiles) {
     processed++;
-    options.onProgress?.(
+    (0, import_core.emitProgress)(
       processed,
       sourceFiles.length,
-      `testability: analyzing files`
+      "testability",
+      "analyzing files",
+      options.onProgress
     );
     const a = analyzeFileTestability(f);
     for (const key of Object.keys(aggregated)) {

package/dist/index.mjs

@@ -1,7 +1,7 @@
 import {
   analyzeTestability,
   calculateTestabilityScore
-} from "./chunk-2QLLQPTL.mjs";
+} from "./chunk-LJLAJRZR.mjs";
 
 // src/index.ts
 import { ToolRegistry } from "@aiready/core";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/testability",
-  "version": "0.4.4",
+  "version": "0.4.9",
   "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.21.4"
+    "@aiready/core": "0.21.9"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -15,6 +15,7 @@ import {
   calculateTestabilityIndex,
   Severity,
   IssueType,
+  emitProgress,
 } from '@aiready/core';
 import { readFileSync, existsSync } from 'fs';
 import { join } from 'path';
@@ -306,10 +307,12 @@ export async function analyzeTestability(
   let processed = 0;
   for (const f of sourceFiles) {
     processed++;
-    options.onProgress?.(
+    emitProgress(
       processed,
       sourceFiles.length,
-      `testability: analyzing files`
+      'testability',
+      'analyzing files',
+      options.onProgress
     );
 
     const a = analyzeFileTestability(f);