@archora/core 1.3.0 → 2.0.0

package/src/report/buildFixPlan.ts

@@ -1,4 +1,5 @@
 import type { ModuleId, ModuleNode, Recommendation, ScanResult } from '../analyzer/types';
+import { buildDeadCodeReport, isReviewModule, unreachableRepair } from './buildDeadCodeReport';
 
 export interface FixPlanFinding {
   type:
@@ -223,33 +224,19 @@ function buildPriorityFindings(scan: ScanResult, generated: Set<ModuleId>): FixP
     });
   }
 
-  for (const module of scan.modules) {
-    if (!isReviewModule(module.id)) continue;
-    const metrics = scan.metrics[module.id];
-    const fanIn = metrics?.fanIn ?? 0;
-    const fanOut = metrics?.fanOut ?? 0;
-    if (
-      fanIn === 0 &&
-      fanOut === 0 &&
-      module.exports.length === 0 &&
-      module.kind !== 'entry' &&
-      module.kind !== 'test' &&
-      !module.isGenerated &&
-      !module.isInfra
-    ) {
-      const repair = unreachableRepair(module.id);
-      findings.push({
-        type: 'unreachable-from-entries',
-        id: `${module.id}:unreachable`,
-        title: 'Unreachable module candidate',
-        weight: Math.min(70, 35 + module.loc / 10),
-        targets: [module.id],
-        reason: 'No resolved imports connect this module to the analyzed dependency model.',
-        action: repair.action,
-        verify: repair.verify,
-        params: { loc: module.loc, ...repair.params },
-      });
-    }
+  for (const candidate of buildDeadCodeReport(scan).candidates) {
+    const repair = unreachableRepair(candidate.id);
+    findings.push({
+      type: 'unreachable-from-entries',
+      id: `${candidate.id}:unreachable`,
+      title: 'Unreachable module candidate',
+      weight: Math.min(70, 35 + candidate.loc / 10),
+      targets: [candidate.id],
+      reason: candidate.reason,
+      action: repair.action,
+      verify: repair.verify,
+      params: { loc: candidate.loc, ...repair.params },
+    });
   }
 
   for (const violation of scan.contractViolations.slice(0, 50)) {
@@ -318,52 +305,10 @@ function buildPriorityFindings(scan: ScanResult, generated: Set<ModuleId>): FixP
     .slice(0, 100);
 }
 
-function isReviewModule(id: ModuleId): boolean {
-  return !/(^|\/)(fixtures|test\/fixtures|__fixtures__|__tests__|__mocks__)(\/|$)|\.(test|spec)\./u.test(
-    id,
-  );
-}
-
 function isLikelyGeneratedPath(id: ModuleId): boolean {
   return /(^|\/)(generated|__generated__|openapi|swagger|graphql-codegen)(\/|$)/iu.test(id);
 }
 
-function unreachableRepair(id: ModuleId): {
-  action: string;
-  verify: string;
-  params: Record<string, unknown>;
-} {
-  if (isScriptEntryCandidate(id)) {
-    return {
-      action: `Treat ${id} as a script entry: add it to architecture entry configuration or exclude it from review scope; delete only after confirming no package script or CI job calls it.`,
-      verify:
-        'Run archora report . --format fix-plan after entry/exclude config and confirm this script is no longer a priority finding.',
-      params: { entryCandidate: 'script' },
-    };
-  }
-
-  return {
-    action:
-      'Check whether this file is dead code, dynamically loaded outside analyzer reach, or should be declared as an entry point.',
-    verify:
-      'Re-scan after deletion, ignore, or entry-point configuration and confirm the candidate is gone.',
-    params: {},
-  };
-}
-
-function isScriptEntryCandidate(id: ModuleId): boolean {
-  const normalized = id.replace(/\\/gu, '/');
-  let scriptsIndex = 0;
-  if (!normalized.startsWith('scripts/')) {
-    const nestedIndex = normalized.indexOf('/scripts/');
-    if (nestedIndex < 0) return false;
-    scriptsIndex = nestedIndex + 1;
-  }
-  const file = normalized.slice(scriptsIndex + 'scripts/'.length);
-  if (!file) return false;
-  return file.endsWith('.js') || file.endsWith('.cjs') || file.endsWith('.mjs');
-}
-
 function barrelCycleRepair(
   scan: ScanResult,
   modules: readonly ModuleId[],

package/src/views/__tests__/analyzerViews.test.ts

@@ -168,12 +168,18 @@ describe('analyzer view helpers', () => {
         changedModules: [],
         newCycles: current.cycles,
         resolvedCycles: [],
+        newLayerViolations: [],
+        resolvedLayerViolations: [],
+        newContractViolations: [],
+        resolvedContractViolations: [],
         summary: {
           addedModules: 0,
           removedModules: 0,
           changedModules: 0,
           newCycles: current.cycles.length,
           resolvedCycles: 0,
+          newLayerViolations: 0,
+          newContractViolations: 0,
         },
       },
     });

package/src/views/analyzerViews.ts

@@ -10,6 +10,7 @@ import type {
 import { detectLayer } from '../analyzer/layers';
 import { diffScans } from '../diff';
 import type { ScanDiff } from '../diff/types';
+import { isReviewModule } from '../report/buildDeadCodeReport';
 
 export type MatrixGrouping = 'area' | 'layer' | 'folder' | 'package';
 
@@ -1188,12 +1189,6 @@ function isSourceFile(part: string): boolean {
   return /\.[cm]?[jt]sx?$/u.test(part) || /\.(vue|svelte)$/u.test(part);
 }
 
-function isReviewModule(id: ModuleId): boolean {
-  return !/(^|\/)(fixtures|test\/fixtures|__fixtures__|__tests__|__mocks__)(\/|$)|\.(test|spec)\./u.test(
-    id,
-  );
-}
-
 function stripExtension(part: string): string {
   return part.replace(/\.[^.]+$/u, '');
 }