@chappibunny/repolens 1.16.3 → 1.16.4

package/src/renderers/renderAnalysis.js

@@ -356,6 +356,8 @@ export function renderDependencyGraph(depResult) {
     lines.push(`| External packages | ${s.externalDeps} |`);
   }
   lines.push(`| Circular dependencies | ${s.cycles} |`);
+  if (s.typeOnlyCycles > 0) lines.push(`| Type-only cycles (not runtime, not counted) | ${s.typeOnlyCycles} |`);
+  if (s.typeOnlyEdges > 0) lines.push(`| Type-only imports (not import edges) | ${s.typeOnlyEdges} |`);
   lines.push(`| Orphan files (no imports or importers) | ${s.orphanFiles} |`);
   lines.push("");
 
@@ -395,9 +397,27 @@ export function renderDependencyGraph(depResult) {
   }
 
   if (Array.isArray(depResult.typeOnlyReports) && depResult.typeOnlyReports.length > 0) {
+    const reports = depResult.typeOnlyReports;
     lines.push("## Type-only cycles");
     lines.push("");
-    lines.push(`**${depResult.typeOnlyReports.length}** cycle${depResult.typeOnlyReports.length === 1 ? " exists" : "s exist"} only through \`import type\` / \`export type\`. They are not runtime dependencies. Priority: Low — type-only imports; verify.`);
+    lines.push(`**${reports.length}** cycle${reports.length === 1 ? " closes" : "s close"} only through type-only imports. The compiler removes those imports, so there is no runtime cycle. They are not counted as circular dependencies and do not lower the health score. Priority: Low — type-only imports; verify.`);
+    lines.push("");
+    lines.push("A type-only import is `import type`, `export type`, or an import list whose names are all marked `type` (*syntax*). In TypeScript files it is also an import whose names are interfaces or type aliases, or are used only in type positions (*inferred*). Inference is off under `verbatimModuleSyntax`, `preserveValueImports`, `importsNotUsedAsValues: \"preserve\"`, or `erasableSyntaxOnly`, which keep the import at runtime.");
+    lines.push("");
+    const limit = Math.min(reports.length, 20);
+    for (let i = 0; i < limit; i++) {
+      const report = reports[i];
+      const pathText = Array.isArray(report.path) ? report.path.join("` → `") : String(report.path);
+      lines.push(`${i + 1}. \`${pathText}\``);
+      const links = Array.isArray(report.typeOnlyLinks) ? report.typeOnlyLinks : [];
+      if (links.length > 0) {
+        const text = links.map((link) => `\`${link.from}\` → \`${link.to}\` (${link.reason === "inferred" ? "inferred" : "syntax"})`).join("; ");
+        lines.push(`   Type-only link${links.length === 1 ? "" : "s"}: ${text}`);
+      }
+    }
+    if (reports.length > 20) {
+      lines.push(`\n*...and ${reports.length - 20} more type-only cycles*`);
+    }
     lines.push("");
   }
 
@@ -434,7 +454,7 @@ export function renderDependencyGraph(depResult) {
 
   lines.push("---");
   lines.push("");
-  lines.push("*Generated by RepoLens import analysis. Dependency detection covers JavaScript and TypeScript (ES modules, dynamic import, CommonJS, re-exports; comments, strings, type-only imports, and Node.js built-ins are skipped), Python (`import` / `from ... import`, including relative imports; `TYPE_CHECKING` blocks are skipped), and Go (`go.mod` module paths, hubs grouped by package). Go `*_test.go` files are not part of the graph.*");
+  lines.push("*Generated by RepoLens import analysis. Dependency detection covers JavaScript and TypeScript (ES modules, dynamic import, CommonJS, re-exports; comments, strings, and Node.js built-ins are skipped; type-only imports, including TypeScript imports whose names are used only as types, are not import edges), Python (`import` / `from ... import`, including relative imports; `TYPE_CHECKING` blocks are skipped), and Go (`go.mod` module paths, hubs grouped by package). Go `*_test.go` files are not part of the graph.*");
   lines.push("");
 
   return lines.join("\n");