@optave/codegraph 3.1.1 → 3.1.3

package/src/commands/structure.js

@@ -0,0 +1,64 @@
+import path from 'node:path';
+import { hotspotsData, moduleBoundariesData, structureData } from '../structure.js';
+
+export { structureData, hotspotsData, moduleBoundariesData };
+
+export function formatStructure(data) {
+  if (data.count === 0) return 'No directory structure found. Run "codegraph build" first.';
+
+  const lines = [`\nProject structure (${data.count} directories):\n`];
+  for (const d of data.directories) {
+    const cohStr = d.cohesion !== null ? ` cohesion=${d.cohesion.toFixed(2)}` : '';
+    const depth = d.directory.split('/').length - 1;
+    const indent = '  '.repeat(depth);
+    lines.push(
+      `${indent}${d.directory}/  (${d.fileCount} files, ${d.symbolCount} symbols, <-${d.fanIn} ->${d.fanOut}${cohStr})`,
+    );
+    for (const f of d.files) {
+      lines.push(
+        `${indent}  ${path.basename(f.file)}  ${f.lineCount}L ${f.symbolCount}sym <-${f.fanIn} ->${f.fanOut}`,
+      );
+    }
+  }
+  if (data.warning) {
+    lines.push('');
+    lines.push(`⚠ ${data.warning}`);
+  }
+  return lines.join('\n');
+}
+
+export function formatHotspots(data) {
+  if (data.hotspots.length === 0) return 'No hotspots found. Run "codegraph build" first.';
+
+  const lines = [`\nHotspots by ${data.metric} (${data.level}-level, top ${data.limit}):\n`];
+  let rank = 1;
+  for (const h of data.hotspots) {
+    const extra =
+      h.kind === 'directory'
+        ? `${h.fileCount} files, cohesion=${h.cohesion !== null ? h.cohesion.toFixed(2) : 'n/a'}`
+        : `${h.lineCount || 0}L, ${h.symbolCount || 0} symbols`;
+    lines.push(
+      `  ${String(rank++).padStart(2)}. ${h.name}  <-${h.fanIn || 0} ->${h.fanOut || 0}  (${extra})`,
+    );
+  }
+  return lines.join('\n');
+}
+
+export function formatModuleBoundaries(data) {
+  if (data.count === 0) return `No modules found with cohesion >= ${data.threshold}.`;
+
+  const lines = [`\nModule boundaries (cohesion >= ${data.threshold}, ${data.count} modules):\n`];
+  for (const m of data.modules) {
+    lines.push(
+      `  ${m.directory}/  cohesion=${m.cohesion.toFixed(2)}  (${m.fileCount} files, ${m.symbolCount} symbols)`,
+    );
+    lines.push(`    Incoming: ${m.fanIn} edges    Outgoing: ${m.fanOut} edges`);
+    if (m.files.length > 0) {
+      lines.push(
+        `    Files: ${m.files.slice(0, 5).join(', ')}${m.files.length > 5 ? ` ... +${m.files.length - 5}` : ''}`,
+      );
+    }
+    lines.push('');
+  }
+  return lines.join('\n');
+}

package/src/commands/triage.js

@@ -0,0 +1,49 @@
+import { outputResult } from '../infrastructure/result-formatter.js';
+import { triageData } from '../triage.js';
+
+/**
+ * Print triage results to console.
+ */
+export function triage(customDbPath, opts = {}) {
+  const data = triageData(customDbPath, opts);
+
+  if (outputResult(data, 'items', opts)) return;
+
+  if (data.items.length === 0) {
+    if (data.summary.total === 0) {
+      console.log('\nNo symbols found. Run "codegraph build" first.\n');
+    } else {
+      console.log('\nNo symbols match the given filters.\n');
+    }
+    return;
+  }
+
+  console.log('\n# Risk Audit Queue\n');
+
+  console.log(
+    `  ${'Symbol'.padEnd(35)} ${'File'.padEnd(28)} ${'Role'.padEnd(8)} ${'Score'.padStart(6)} ${'Fan-In'.padStart(7)} ${'Cog'.padStart(4)} ${'Churn'.padStart(6)} ${'MI'.padStart(5)}`,
+  );
+  console.log(
+    `  ${'─'.repeat(35)} ${'─'.repeat(28)} ${'─'.repeat(8)} ${'─'.repeat(6)} ${'─'.repeat(7)} ${'─'.repeat(4)} ${'─'.repeat(6)} ${'─'.repeat(5)}`,
+  );
+
+  for (const it of data.items) {
+    const name = it.name.length > 33 ? `${it.name.slice(0, 32)}…` : it.name;
+    const file = it.file.length > 26 ? `…${it.file.slice(-25)}` : it.file;
+    const role = (it.role || '-').padEnd(8);
+    const score = it.riskScore.toFixed(2).padStart(6);
+    const fanIn = String(it.fanIn).padStart(7);
+    const cog = String(it.cognitive).padStart(4);
+    const churn = String(it.churn).padStart(6);
+    const mi = it.maintainabilityIndex > 0 ? String(it.maintainabilityIndex).padStart(5) : '    -';
+    console.log(
+      `  ${name.padEnd(35)} ${file.padEnd(28)} ${role} ${score} ${fanIn} ${cog} ${churn} ${mi}`,
+    );
+  }
+
+  const s = data.summary;
+  console.log(
+    `\n  ${s.analyzed} symbols scored (of ${s.total} total) | avg: ${s.avgScore.toFixed(2)} | max: ${s.maxScore.toFixed(2)} | sort: ${opts.sort || 'risk'}`,
+  );
+  console.log();
+}

package/src/communities.js

@@ -1,10 +1,15 @@
 import path from 'node:path';
 import Graph from 'graphology';
 import louvain from 'graphology-communities-louvain';
-import { openReadonlyOrFail } from './db.js';
+import {
+  getCallableNodes,
+  getCallEdges,
+  getFileNodesAll,
+  getImportEdges,
+  openReadonlyOrFail,
+} from './db.js';
+import { isTestFile } from './infrastructure/test-filter.js';
 import { paginateResult } from './paginate.js';
-import { outputResult } from './result-formatter.js';
-import { isTestFile } from './test-filter.js';
 
 // ─── Graph Construction ───────────────────────────────────────────────
 
@@ -22,9 +27,7 @@ function buildGraphologyGraph(db, opts = {}) {
 
   if (opts.functions) {
     // Function-level: nodes = function/method/class symbols, edges = calls
-    let nodes = db
-      .prepare("SELECT id, name, kind, file FROM nodes WHERE kind IN ('function','method','class')")
-      .all();
+    let nodes = getCallableNodes(db);
     if (opts.noTests) nodes = nodes.filter((n) => !isTestFile(n.file));
 
     const nodeIds = new Set();
@@ -34,7 +37,7 @@ function buildGraphologyGraph(db, opts = {}) {
       nodeIds.add(n.id);
     }
 
-    const edges = db.prepare("SELECT source_id, target_id FROM edges WHERE kind = 'calls'").all();
+    const edges = getCallEdges(db);
     for (const e of edges) {
       if (!nodeIds.has(e.source_id) || !nodeIds.has(e.target_id)) continue;
       const src = String(e.source_id);
@@ -46,7 +49,7 @@ function buildGraphologyGraph(db, opts = {}) {
     }
   } else {
     // File-level: nodes = files, edges = imports + imports-type (deduplicated, cross-file)
-    let nodes = db.prepare("SELECT id, name, file FROM nodes WHERE kind = 'file'").all();
+    let nodes = getFileNodesAll(db);
     if (opts.noTests) nodes = nodes.filter((n) => !isTestFile(n.file));
 
     const nodeIds = new Set();
@@ -56,9 +59,7 @@ function buildGraphologyGraph(db, opts = {}) {
       nodeIds.add(n.id);
     }
 
-    const edges = db
-      .prepare("SELECT source_id, target_id FROM edges WHERE kind IN ('imports','imports-type')")
-      .all();
+    const edges = getImportEdges(db);
     for (const e of edges) {
       if (!nodeIds.has(e.source_id) || !nodeIds.has(e.target_id)) continue;
       const src = String(e.source_id);
@@ -232,75 +233,3 @@ export function communitySummaryForStats(customDbPath, opts = {}) {
   const data = communitiesData(customDbPath, { ...opts, drift: true });
   return data.summary;
 }
-
-// ─── CLI Display ──────────────────────────────────────────────────────
-
-/**
- * CLI entry point: run community detection and print results.
- *
- * @param {string} [customDbPath]
- * @param {object} [opts]
- */
-export function communities(customDbPath, opts = {}) {
-  const data = communitiesData(customDbPath, opts);
-
-  if (outputResult(data, 'communities', opts)) return;
-
-  if (data.summary.communityCount === 0) {
-    console.log(
-      '\nNo communities detected. The graph may be too small or disconnected.\n' +
-        'Run "codegraph build" first to populate the graph.\n',
-    );
-    return;
-  }
-
-  const mode = opts.functions ? 'Function' : 'File';
-  console.log(`\n# ${mode}-Level Communities\n`);
-  console.log(
-    `  ${data.summary.communityCount} communities | ${data.summary.nodeCount} nodes | modularity: ${data.summary.modularity} | drift: ${data.summary.driftScore}%\n`,
-  );
-
-  if (!opts.drift) {
-    for (const c of data.communities) {
-      const dirs = Object.entries(c.directories)
-        .sort((a, b) => b[1] - a[1])
-        .map(([d, n]) => `${d} (${n})`)
-        .join(', ');
-      console.log(`  Community ${c.id} (${c.size} members): ${dirs}`);
-      if (c.members) {
-        const shown = c.members.slice(0, 8);
-        for (const m of shown) {
-          const kind = m.kind ? ` [${m.kind}]` : '';
-          console.log(`    - ${m.name}${kind}  ${m.file}`);
-        }
-        if (c.members.length > 8) {
-          console.log(`    ... and ${c.members.length - 8} more`);
-        }
-      }
-    }
-  }
-
-  // Drift analysis
-  const d = data.drift;
-  if (d.splitCandidates.length > 0 || d.mergeCandidates.length > 0) {
-    console.log(`\n# Drift Analysis (score: ${data.summary.driftScore}%)\n`);
-
-    if (d.splitCandidates.length > 0) {
-      console.log('  Split candidates (directories spanning multiple communities):');
-      for (const s of d.splitCandidates.slice(0, 10)) {
-        console.log(`    - ${s.directory} → ${s.communityCount} communities`);
-      }
-    }
-
-    if (d.mergeCandidates.length > 0) {
-      console.log('  Merge candidates (communities spanning multiple directories):');
-      for (const m of d.mergeCandidates.slice(0, 10)) {
-        console.log(
-          `    - Community ${m.communityId} (${m.size} members) → ${m.directoryCount} dirs: ${m.directories.join(', ')}`,
-        );
-      }
-    }
-  }
-
-  console.log();
-}