@optave/codegraph 3.1.0 → 3.1.2

package/src/commands/flow.js

@@ -0,0 +1,70 @@
+import { flowData, listEntryPointsData } from '../flow.js';
+import { outputResult } from '../infrastructure/result-formatter.js';
+import { kindIcon } from '../queries.js';
+
+/**
+ * CLI formatter — text or JSON output.
+ */
+export function flow(name, dbPath, opts = {}) {
+  if (opts.list) {
+    const data = listEntryPointsData(dbPath, {
+      noTests: opts.noTests,
+      limit: opts.limit,
+      offset: opts.offset,
+    });
+    if (outputResult(data, 'entries', opts)) return;
+    if (data.count === 0) {
+      console.log('No entry points found. Run "codegraph build" first.');
+      return;
+    }
+    console.log(`\nEntry points (${data.count} total):\n`);
+    for (const [type, entries] of Object.entries(data.byType)) {
+      console.log(`  ${type} (${entries.length}):`);
+      for (const e of entries) {
+        console.log(`    [${kindIcon(e.kind)}] ${e.name}  ${e.file}:${e.line}`);
+      }
+      console.log();
+    }
+    return;
+  }
+
+  const data = flowData(name, dbPath, opts);
+  if (outputResult(data, 'steps', opts)) return;
+
+  if (!data.entry) {
+    console.log(`No matching entry point or function found for "${name}".`);
+    return;
+  }
+
+  const e = data.entry;
+  const typeTag = e.type !== 'exported' ? ` (${e.type})` : '';
+  console.log(`\nFlow from: [${kindIcon(e.kind)}] ${e.name}${typeTag}  ${e.file}:${e.line}`);
+  console.log(
+    `Depth: ${data.depth}  Reached: ${data.totalReached} nodes  Leaves: ${data.leaves.length}`,
+  );
+  if (data.truncated) {
+    console.log(`  (truncated at depth ${data.depth})`);
+  }
+  console.log();
+
+  if (data.steps.length === 0) {
+    console.log('  (leaf node — no callees)');
+    return;
+  }
+
+  for (const step of data.steps) {
+    console.log(`  depth ${step.depth}:`);
+    for (const n of step.nodes) {
+      const isLeaf = data.leaves.some((l) => l.name === n.name && l.file === n.file);
+      const leafTag = isLeaf ? ' [leaf]' : '';
+      console.log(`    [${kindIcon(n.kind)}] ${n.name}  ${n.file}:${n.line}${leafTag}`);
+    }
+  }
+
+  if (data.cycles.length > 0) {
+    console.log('\n  Cycles detected:');
+    for (const c of data.cycles) {
+      console.log(`    ${c.from} -> ${c.to} (at depth ${c.depth})`);
+    }
+  }
+}

package/src/commands/manifesto.js

@@ -0,0 +1,77 @@
+import { outputResult } from '../infrastructure/result-formatter.js';
+import { manifestoData } from '../manifesto.js';
+
+/**
+ * CLI formatter — prints manifesto results and exits with code 1 on failure.
+ */
+export function manifesto(customDbPath, opts = {}) {
+  const data = manifestoData(customDbPath, opts);
+
+  if (outputResult(data, 'violations', opts)) {
+    if (!data.passed) process.exit(1);
+    return;
+  }
+
+  console.log('\n# Manifesto Rules\n');
+
+  // Rules table
+  console.log(
+    `  ${'Rule'.padEnd(20)} ${'Level'.padEnd(10)} ${'Status'.padEnd(8)} ${'Warn'.padStart(6)} ${'Fail'.padStart(6)} ${'Violations'.padStart(11)}`,
+  );
+  console.log(
+    `  ${'─'.repeat(20)} ${'─'.repeat(10)} ${'─'.repeat(8)} ${'─'.repeat(6)} ${'─'.repeat(6)} ${'─'.repeat(11)}`,
+  );
+
+  for (const rule of data.rules) {
+    const warn = rule.thresholds.warn != null ? String(rule.thresholds.warn) : '—';
+    const fail = rule.thresholds.fail != null ? String(rule.thresholds.fail) : '—';
+    const statusIcon = rule.status === 'pass' ? 'pass' : rule.status === 'warn' ? 'WARN' : 'FAIL';
+    console.log(
+      `  ${rule.name.padEnd(20)} ${rule.level.padEnd(10)} ${statusIcon.padEnd(8)} ${warn.padStart(6)} ${fail.padStart(6)} ${String(rule.violationCount).padStart(11)}`,
+    );
+  }
+
+  // Summary
+  const s = data.summary;
+  console.log(
+    `\n  ${s.total} rules | ${s.passed} passed | ${s.warned} warned | ${s.failed} failed | ${s.violationCount} violations`,
+  );
+
+  // Violations detail
+  if (data.violations.length > 0) {
+    const failViolations = data.violations.filter((v) => v.level === 'fail');
+    const warnViolations = data.violations.filter((v) => v.level === 'warn');
+
+    if (failViolations.length > 0) {
+      console.log(`\n## Failures (${failViolations.length})\n`);
+      for (const v of failViolations.slice(0, 20)) {
+        const loc = v.line ? `${v.file}:${v.line}` : v.file;
+        console.log(
+          `  [FAIL] ${v.rule}: ${v.name} (${v.value}) at ${loc} — threshold ${v.threshold}`,
+        );
+      }
+      if (failViolations.length > 20) {
+        console.log(`  ... and ${failViolations.length - 20} more`);
+      }
+    }
+
+    if (warnViolations.length > 0) {
+      console.log(`\n## Warnings (${warnViolations.length})\n`);
+      for (const v of warnViolations.slice(0, 20)) {
+        const loc = v.line ? `${v.file}:${v.line}` : v.file;
+        console.log(
+          `  [WARN] ${v.rule}: ${v.name} (${v.value}) at ${loc} — threshold ${v.threshold}`,
+        );
+      }
+      if (warnViolations.length > 20) {
+        console.log(`  ... and ${warnViolations.length - 20} more`);
+      }
+    }
+  }
+
+  console.log();
+
+  if (!data.passed) {
+    process.exit(1);
+  }
+}

package/src/commands/owners.js

@@ -0,0 +1,52 @@
+import { outputResult } from '../infrastructure/result-formatter.js';
+import { ownersData } from '../owners.js';
+
+/**
+ * CLI display function for the `owners` command.
+ */
+export function owners(customDbPath, opts = {}) {
+  const data = ownersData(customDbPath, opts);
+  if (outputResult(data, null, opts)) return;
+
+  if (!data.codeownersFile) {
+    console.log('No CODEOWNERS file found.');
+    return;
+  }
+
+  console.log(`\nCODEOWNERS: ${data.codeownersFile}\n`);
+
+  const s = data.summary;
+  console.log(
+    `  Coverage: ${s.coveragePercent}% (${s.ownedFiles}/${s.totalFiles} files owned, ${s.ownerCount} owners)\n`,
+  );
+
+  if (s.byOwner.length > 0) {
+    console.log('  Owners:\n');
+    for (const o of s.byOwner) {
+      console.log(`    ${o.owner}  ${o.fileCount} files`);
+    }
+    console.log();
+  }
+
+  if (data.files.length > 0 && opts.owner) {
+    console.log(`  Files owned by ${opts.owner}:\n`);
+    for (const f of data.files) {
+      console.log(`    ${f.file}`);
+    }
+    console.log();
+  }
+
+  if (data.boundaries.length > 0) {
+    console.log(`  Cross-owner boundaries: ${data.boundaries.length} edges\n`);
+    const shown = data.boundaries.slice(0, 30);
+    for (const b of shown) {
+      const srcOwner = b.from.owners.join(', ') || '(unowned)';
+      const tgtOwner = b.to.owners.join(', ') || '(unowned)';
+      console.log(`    ${b.from.name} [${srcOwner}] -> ${b.to.name} [${tgtOwner}]`);
+    }
+    if (data.boundaries.length > 30) {
+      console.log(`    ... and ${data.boundaries.length - 30} more`);
+    }
+    console.log();
+  }
+}

package/src/commands/query.js

@@ -0,0 +1,21 @@
+/**
+ * Re-export all query CLI wrappers from queries-cli.js.
+ * This barrel file provides the standard src/commands/ import path.
+ */
+export {
+  children,
+  context,
+  diffImpact,
+  explain,
+  fileDeps,
+  fileExports,
+  fnDeps,
+  fnImpact,
+  impactAnalysis,
+  moduleMap,
+  queryName,
+  roles,
+  stats,
+  symbolPath,
+  where,
+} from '../queries-cli.js';

package/src/commands/sequence.js

@@ -0,0 +1,33 @@
+import { outputResult } from '../infrastructure/result-formatter.js';
+import { kindIcon } from '../queries.js';
+import { sequenceData, sequenceToMermaid } from '../sequence.js';
+
+/**
+ * CLI entry point — format sequence data as mermaid, JSON, or ndjson.
+ */
+export function sequence(name, dbPath, opts = {}) {
+  const data = sequenceData(name, dbPath, opts);
+
+  if (outputResult(data, 'messages', opts)) return;
+
+  // Default: mermaid format
+  if (!data.entry) {
+    console.log(`No matching function found for "${name}".`);
+    return;
+  }
+
+  const e = data.entry;
+  console.log(`\nSequence from: [${kindIcon(e.kind)}] ${e.name}  ${e.file}:${e.line}`);
+  console.log(`Participants: ${data.participants.length}  Messages: ${data.totalMessages}`);
+  if (data.truncated) {
+    console.log(`  (truncated at depth ${data.depth})`);
+  }
+  console.log();
+
+  if (data.messages.length === 0) {
+    console.log('  (leaf node — no callees)');
+    return;
+  }
+
+  console.log(sequenceToMermaid(data));
+}

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,9 +1,15 @@
 import path from 'node:path';
 import Graph from 'graphology';
 import louvain from 'graphology-communities-louvain';
-import { openReadonlyOrFail } from './db.js';
-import { paginateResult, printNdjson } from './paginate.js';
-import { isTestFile } from './queries.js';
+import {
+  getCallableNodes,
+  getCallEdges,
+  getFileNodesAll,
+  getImportEdges,
+  openReadonlyOrFail,
+} from './db.js';
+import { isTestFile } from './infrastructure/test-filter.js';
+import { paginateResult } from './paginate.js';
 
 // ─── Graph Construction ───────────────────────────────────────────────
 
@@ -21,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();
@@ -33,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);
@@ -45,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();
@@ -55,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);
@@ -96,12 +98,15 @@ function getDirectory(filePath) {
 export function communitiesData(customDbPath, opts = {}) {
   const db = openReadonlyOrFail(customDbPath);
   const resolution = opts.resolution ?? 1.0;
-
-  const graph = buildGraphologyGraph(db, {
-    functions: opts.functions,
-    noTests: opts.noTests,
-  });
-  db.close();
+  let graph;
+  try {
+    graph = buildGraphologyGraph(db, {
+      functions: opts.functions,
+      noTests: opts.noTests,
+    });
+  } finally {
+    db.close();
+  }
 
   // Handle empty or trivial graphs
   if (graph.order === 0 || graph.size === 0) {
@@ -228,82 +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 (opts.ndjson) {
-    printNdjson(data, 'communities');
-    return;
-  }
-  if (opts.json) {
-    console.log(JSON.stringify(data, null, 2));
-    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();
-}