@chappibunny/repolens 1.10.0 → 1.12.0

package/src/docs/generate-doc-set.js

@@ -5,6 +5,9 @@ import { inferDataFlows } from "../analyzers/flow-inference.js";
 import { analyzeGraphQL } from "../analyzers/graphql-analyzer.js";
 import { analyzeTypeScript } from "../analyzers/typescript-analyzer.js";
 import { analyzeDependencyGraph } from "../analyzers/dependency-graph.js";
+import { analyzeJSDoc } from "../analyzers/jsdoc-analyzer.js";
+import { analyzeSecurityPatterns } from "../analyzers/security-patterns.js";
+import { analyzeComplexity, computeCodeHealth } from "../analyzers/complexity-analyzer.js";
 import { buildSnapshot, loadBaseline, saveBaseline, detectDrift } from "../analyzers/drift-detector.js";
 import { parseCodeowners, buildOwnershipMap } from "../analyzers/codeowners.js";
 import { getActiveDocuments } from "../ai/document-plan.js";
@@ -25,7 +28,9 @@ import {
   renderGraphQLSchema,
   renderTypeGraph,
   renderDependencyGraph,
-  renderArchitectureDrift as renderDriftReport
+  renderArchitectureDrift as renderDriftReport,
+  renderCodeHealth,
+  renderSecurityHotspots
 } from "../renderers/renderAnalysis.js";
 import { info, warn } from "../utils/logger.js";
 import path from "node:path";
@@ -46,9 +51,15 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
   let graphqlResult = { detected: false };
   let tsResult = { detected: false };
   let depGraph = { stats: {}, graph: {} };
+  let jsdocResult = { detected: false, exports: [], summary: null };
   try { graphqlResult = await analyzeGraphQL(scanFiles, repoRoot); } catch (e) { warn(`GraphQL analysis failed: ${e.message}`); }
   try { tsResult = await analyzeTypeScript(scanFiles, repoRoot); } catch (e) { warn(`TypeScript analysis failed: ${e.message}`); }
   try { depGraph = await analyzeDependencyGraph(scanFiles, repoRoot); } catch (e) { warn(`Dependency graph analysis failed: ${e.message}`); }
+  try { jsdocResult = await analyzeJSDoc(scanFiles, repoRoot); } catch (e) { warn(`JSDoc analysis failed: ${e.message}`); }
+  let securityResult = { detected: false, findings: [], bySeverity: { high: 0, medium: 0, low: 0 } };
+  try { securityResult = await analyzeSecurityPatterns(scanFiles, repoRoot); } catch (e) { warn(`Security pattern analysis failed: ${e.message}`); }
+  let complexityResult = { files: [], functions: [], filesAnalyzed: 0 };
+  try { complexityResult = await analyzeComplexity(scanFiles, repoRoot); } catch (e) { warn(`Complexity analysis failed: ${e.message}`); }
   
   // Architecture drift detection
   const outputDir = path.join(repoRoot, ".repolens");
@@ -73,6 +84,10 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
   const ownershipMap = codeowners.found
     ? buildOwnershipMap(scanResult.modules, scanFiles, codeowners.rules)
     : {};
+
+  // Compute unified code health scores (synthesizes complexity + coupling + docs + security)
+  let codeHealthResult = { modules: [], summary: "No health data", stats: { totalFiles: 0, avgScore: 100, avgComplexity: 0, gradeDistribution: { A: 0, B: 0, C: 0, D: 0, F: 0 } }, hotspots: [], topComplexFunctions: [] };
+  try { codeHealthResult = computeCodeHealth(complexityResult, depGraph, jsdocResult, securityResult); } catch (e) { warn(`Code health computation failed: ${e.message}`); }
   
   // Get active documents based on config
   const activeDocuments = getActiveDocuments(config);
@@ -86,8 +101,11 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
     flows,
     graphql: graphqlResult.detected ? graphqlResult : undefined,
     typescript: tsResult.detected ? tsResult : undefined,
+    jsdoc: jsdocResult.detected ? jsdocResult : undefined,
     dependencyGraph: depGraph.stats,
     drift: driftResult,
+    security: securityResult.detected ? securityResult : undefined,
+    codeHealth: complexityResult.files.length > 0 ? true : undefined,
     codeowners: codeowners.found ? { file: codeowners.file, ruleCount: codeowners.rules.length } : undefined,
     ownershipMap: Object.keys(ownershipMap).length > 0 ? ownershipMap : undefined,
   };
@@ -117,8 +135,11 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
         diffData,
         graphqlResult,
         tsResult,
+        jsdocResult,
         depGraph,
         driftResult,
+        securityResult,
+        codeHealthResult,
         ownershipMap,
         pluginManager,
       });
@@ -167,8 +188,11 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
           diffData,
           graphqlResult,
           tsResult,
+          jsdocResult,
           depGraph,
           driftResult,
+          securityResult,
+          codeHealthResult,
         });
 
         documents.push({
@@ -203,7 +227,7 @@ export async function generateDocumentSet(scanResult, config, diffData = null, p
 
 async function generateDocument(docPlan, context) {
   const { key } = docPlan;
-  const { scanResult, config, aiContext, moduleContext, flows, diffData, graphqlResult, tsResult, depGraph, driftResult, ownershipMap, pluginManager } = context;
+  const { scanResult, config, aiContext, moduleContext, flows, diffData, graphqlResult, tsResult, jsdocResult, depGraph, driftResult, securityResult, codeHealthResult, ownershipMap, pluginManager } = context;
   
   switch (key) {
     case "executive_summary":
@@ -227,8 +251,8 @@ async function generateDocument(docPlan, context) {
       return renderRouteMapOriginal(config, scanResult, aiContext);
       
     case "api_surface":
-      // Hybrid: deterministic skeleton + AI enhancement (for now, just deterministic)
-      return renderApiSurfaceOriginal(config, scanResult);
+      // Hybrid: deterministic skeleton + JSDoc documentation
+      return renderApiSurfaceOriginal(config, scanResult, jsdocResult);
       
     case "data_flows":
       return await generateDataFlows(flows, aiContext, { depGraph, scanResult, moduleContext }, config);
@@ -258,6 +282,12 @@ async function generateDocument(docPlan, context) {
     case "architecture_drift":
       return renderDriftReport(driftResult);
       
+    case "security_hotspots":
+      return renderSecurityHotspots(securityResult);
+      
+    case "code_health":
+      return renderCodeHealth(codeHealthResult);
+      
     default: {
       // Check if a plugin provides this renderer
       if (pluginManager) {