@aiready/testability 0.2.0 → 0.2.3

package/.turbo/turbo-build.log

@@ -1,6 +1,6 @@
 
 
-> @aiready/testability@0.1.12 build /Users/pengcao/projects/aiready/packages/testability
+> @aiready/testability@0.2.2 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,16 +9,16 @@
 CLI Target: es2020
 CJS Build start
 ESM Build start
+CJS dist/index.js 12.28 KB
+CJS dist/cli.js   18.38 KB
+CJS ⚡️ Build success in 38ms
+ESM dist/chunk-7MM2QISW.mjs 11.02 KB
 ESM dist/index.mjs          152.00 B
-ESM dist/chunk-DDNB7FI4.mjs 10.93 KB
 ESM dist/cli.mjs            5.75 KB
-ESM ⚡️ Build success in 117ms
-CJS dist/index.js 12.12 KB
-CJS dist/cli.js   18.21 KB
-CJS ⚡️ Build success in 117ms
+ESM ⚡️ Build success in 38ms
 DTS Build start
-DTS ⚡️ Build success in 3196ms
+DTS ⚡️ Build success in 2233ms
 DTS dist/cli.d.ts    20.00 B
-DTS dist/index.d.ts  2.64 KB
+DTS dist/index.d.ts  2.66 KB
 DTS dist/cli.d.mts   20.00 B
-DTS dist/index.d.mts 2.64 KB
+DTS dist/index.d.mts 2.66 KB

package/.turbo/turbo-test.log

@@ -1,16 +1,16 @@
 
 
-> @aiready/testability@0.1.12 test /Users/pengcao/projects/aiready/packages/testability
+> @aiready/testability@0.2.2 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) 344ms
+ ✓ src/__tests__/analyzer.test.ts (3 tests) 217ms
 
  Test Files  1 passed (1)
       Tests  3 passed (3)
-   Start at  16:12:21
-   Duration  2.16s (transform 365ms, setup 0ms, import 1.64s, tests 344ms, environment 0ms)
+   Start at  20:01:55
+   Duration  2.14s (transform 162ms, setup 0ms, import 1.21s, tests 217ms, environment 0ms)
 
 [?25h

package/dist/chunk-7MM2QISW.mjs

@@ -0,0 +1,333 @@
+// src/analyzer.ts
+import { scanFiles, calculateTestabilityIndex, Severity, IssueType } 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++;
+    options.onProgress?.(
+      processed,
+      sourceFiles.length,
+      `testability: analyzing files`
+    );
+    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
+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: "testability",
+    score: summary.score,
+    rawMetrics: {
+      ...rawData,
+      rating: summary.rating,
+      aiChangeSafetyRating: summary.aiChangeSafetyRating,
+      coverageRatio: summary.coverageRatio
+    },
+    factors,
+    recommendations: recs
+  };
+}
+
+export {
+  analyzeTestability,
+  calculateTestabilityScore
+};

package/dist/chunk-QIOGSXKY.mjs

@@ -0,0 +1,333 @@
+// src/analyzer.ts
+import { scanFiles, calculateTestabilityIndex, Severity } 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++;
+    options.onProgress?.(
+      processed,
+      sourceFiles.length,
+      `testability: analyzing files`
+    );
+    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: "low-testability",
+      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: "low-testability",
+      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: "low-testability",
+      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: "low-testability",
+      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
+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: "testability",
+    score: summary.score,
+    rawMetrics: {
+      ...rawData,
+      rating: summary.rating,
+      aiChangeSafetyRating: summary.aiChangeSafetyRating,
+      coverageRatio: summary.coverageRatio
+    },
+    factors,
+    recommendations: recs
+  };
+}
+
+export {
+  analyzeTestability,
+  calculateTestabilityScore
+};

package/dist/cli.js

@@ -244,9 +244,9 @@ async function analyzeTestability(options) {
   const actualRatio = sourceFiles.length > 0 ? testFiles.length / sourceFiles.length : 0;
   if (!hasTestFramework) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "framework",
-      severity: "critical",
+      severity: import_core.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."
@@ -255,9 +255,9 @@ async function analyzeTestability(options) {
   if (actualRatio < minCoverage) {
     const needed = Math.ceil(sourceFiles.length * minCoverage) - testFiles.length;
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "test-coverage",
-      severity: actualRatio === 0 ? "critical" : "major",
+      severity: actualRatio === 0 ? import_core.Severity.Critical : import_core.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.`
@@ -265,9 +265,9 @@ async function analyzeTestability(options) {
   }
   if (indexResult.dimensions.purityScore < 50) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "purity",
-      severity: "major",
+      severity: import_core.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."
@@ -275,9 +275,9 @@ async function analyzeTestability(options) {
   }
   if (indexResult.dimensions.observabilityScore < 50) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "observability",
-      severity: "major",
+      severity: import_core.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."

package/dist/cli.mjs

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

package/dist/index.d.mts

@@ -1,4 +1,4 @@
-import { Issue, ToolScoringOutput } from '@aiready/core';
+import { Issue, IssueType, ToolScoringOutput } from '@aiready/core';
 
 interface TestabilityOptions {
     /** Root directory to scan */
@@ -17,7 +17,7 @@ interface TestabilityOptions {
     onProgress?: (processed: number, total: number, message: string) => void;
 }
 interface TestabilityIssue extends Issue {
-    type: 'low-testability';
+    type: IssueType.LowTestability;
     /** Category of testability barrier */
     dimension: 'test-coverage' | 'purity' | 'dependency-injection' | 'interface-focus' | 'observability' | 'framework';
 }

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { Issue, ToolScoringOutput } from '@aiready/core';
+import { Issue, IssueType, ToolScoringOutput } from '@aiready/core';
 
 interface TestabilityOptions {
     /** Root directory to scan */
@@ -17,7 +17,7 @@ interface TestabilityOptions {
     onProgress?: (processed: number, total: number, message: string) => void;
 }
 interface TestabilityIssue extends Issue {
-    type: 'low-testability';
+    type: IssueType.LowTestability;
     /** Category of testability barrier */
     dimension: 'test-coverage' | 'purity' | 'dependency-injection' | 'interface-focus' | 'observability' | 'framework';
 }

package/dist/index.js

@@ -243,9 +243,9 @@ async function analyzeTestability(options) {
   const actualRatio = sourceFiles.length > 0 ? testFiles.length / sourceFiles.length : 0;
   if (!hasTestFramework) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "framework",
-      severity: "critical",
+      severity: import_core.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."
@@ -254,9 +254,9 @@ async function analyzeTestability(options) {
   if (actualRatio < minCoverage) {
     const needed = Math.ceil(sourceFiles.length * minCoverage) - testFiles.length;
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "test-coverage",
-      severity: actualRatio === 0 ? "critical" : "major",
+      severity: actualRatio === 0 ? import_core.Severity.Critical : import_core.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.`
@@ -264,9 +264,9 @@ async function analyzeTestability(options) {
   }
   if (indexResult.dimensions.purityScore < 50) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "purity",
-      severity: "major",
+      severity: import_core.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."
@@ -274,9 +274,9 @@ async function analyzeTestability(options) {
   }
   if (indexResult.dimensions.observabilityScore < 50) {
     issues.push({
-      type: "low-testability",
+      type: import_core.IssueType.LowTestability,
       dimension: "observability",
-      severity: "major",
+      severity: import_core.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."

package/dist/index.mjs

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiready/testability",
-  "version": "0.2.0",
+  "version": "0.2.3",
   "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.19.0"
+    "@aiready/core": "0.19.3"
   },
   "devDependencies": {
     "@types/node": "^24.0.0",

package/src/analyzer.ts

@@ -10,7 +10,7 @@
  * 5. Observability (return values vs. external state mutations)
  */
 
-import { scanFiles, calculateTestabilityIndex } from '@aiready/core';
+import { scanFiles, calculateTestabilityIndex, Severity, IssueType } from '@aiready/core';
 import { readFileSync, existsSync } from 'fs';
 import { join } from 'path';
 import { parse } from '@typescript-eslint/typescript-estree';
@@ -336,9 +336,9 @@ export async function analyzeTestability(
 
   if (!hasTestFramework) {
     issues.push({
-      type: 'low-testability',
+      type: IssueType.LowTestability,
       dimension: 'framework',
-      severity: 'critical',
+      severity: Severity.Critical,
       message:
         'No testing framework detected in package.json — AI changes cannot be verified at all.',
       location: { file: options.rootDir, line: 0 },
@@ -351,9 +351,9 @@ export async function analyzeTestability(
     const needed =
       Math.ceil(sourceFiles.length * minCoverage) - testFiles.length;
     issues.push({
-      type: 'low-testability',
+      type: IssueType.LowTestability,
       dimension: 'test-coverage',
-      severity: actualRatio === 0 ? 'critical' : 'major',
+      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.`,
@@ -362,9 +362,9 @@ export async function analyzeTestability(
 
   if (indexResult.dimensions.purityScore < 50) {
     issues.push({
-      type: 'low-testability',
+      type: IssueType.LowTestability,
       dimension: 'purity',
-      severity: 'major',
+      severity: Severity.Major,
       message: `Only ${indexResult.dimensions.purityScore}% of functions are pure — side-effectful functions require complex test setup.`,
       location: { file: options.rootDir, line: 0 },
       suggestion:
@@ -374,9 +374,9 @@ export async function analyzeTestability(
 
   if (indexResult.dimensions.observabilityScore < 50) {
     issues.push({
-      type: 'low-testability',
+      type: IssueType.LowTestability,
       dimension: 'observability',
-      severity: 'major',
+      severity: Severity.Major,
       message: `Many functions mutate external state directly — outputs are invisible to unit tests.`,
       location: { file: options.rootDir, line: 0 },
       suggestion: 'Prefer returning values over mutating shared state.',

package/src/types.ts

@@ -1,4 +1,4 @@
-import type { Issue } from '@aiready/core';
+import type { Issue, IssueType } from '@aiready/core';
 
 export interface TestabilityOptions {
   /** Root directory to scan */
@@ -18,7 +18,7 @@ export interface TestabilityOptions {
 }
 
 export interface TestabilityIssue extends Issue {
-  type: 'low-testability';
+  type: IssueType.LowTestability;
   /** Category of testability barrier */
   dimension:
     | 'test-coverage'