@optave/codegraph 3.1.1 → 3.1.2

package/src/index.js

@@ -8,13 +8,11 @@
 // AST node queries
 export { AST_NODE_KINDS, astQuery, astQueryData } from './ast.js';
 // Audit (composite report)
-export { audit, auditData } from './audit.js';
+export { auditData } from './audit.js';
 // Batch querying
 export {
   BATCH_COMMANDS,
-  batch,
   batchData,
-  batchQuery,
   multiBatchData,
   splitTargets,
 } from './batch.js';
@@ -29,13 +27,12 @@ export {
   buildCFGData,
   buildFunctionCFG,
   CFG_RULES,
-  cfg,
   cfgData,
   cfgToDOT,
   cfgToMermaid,
 } from './cfg.js';
 // Check (CI validation predicates)
-export { check, checkData } from './check.js';
+export { checkData } from './check.js';
 // Co-change analysis
 export {
   analyzeCoChanges,
@@ -45,12 +42,23 @@ export {
   computeCoChanges,
   scanGitHistory,
 } from './cochange.js';
+export { audit } from './commands/audit.js';
+export { batch, batchQuery } from './commands/batch.js';
+export { cfg } from './commands/cfg.js';
+export { check } from './commands/check.js';
+export { communities } from './commands/communities.js';
+export { complexity } from './commands/complexity.js';
+export { dataflow } from './commands/dataflow.js';
+export { manifesto } from './commands/manifesto.js';
+export { owners } from './commands/owners.js';
+export { sequence } from './commands/sequence.js';
+export { formatHotspots, formatModuleBoundaries, formatStructure } from './commands/structure.js';
+export { triage } from './commands/triage.js';
 // Community detection
-export { communities, communitiesData, communitySummaryForStats } from './communities.js';
+export { communitiesData, communitySummaryForStats } from './communities.js';
 // Complexity metrics
 export {
   COMPLEXITY_RULES,
-  complexity,
   complexityData,
   computeFunctionComplexity,
   computeHalsteadMetrics,
@@ -69,7 +77,6 @@ export { findCycles, formatCycles } from './cycles.js';
 // Dataflow analysis
 export {
   buildDataflowEdges,
-  dataflow,
   dataflowData,
   dataflowImpactData,
   dataflowPathData,
@@ -123,14 +130,18 @@ export {
 } from './export.js';
 // Execution flow tracing
 export { entryPointType, flowData, listEntryPointsData } from './flow.js';
+// Result formatting
+export { outputResult } from './infrastructure/result-formatter.js';
+// Test file detection
+export { isTestFile, TEST_PATTERN } from './infrastructure/test-filter.js';
 // Logger
 export { setVerbose } from './logger.js';
 // Manifesto rule engine
-export { manifesto, manifestoData, RULE_DEFS } from './manifesto.js';
+export { manifestoData, RULE_DEFS } from './manifesto.js';
 // Native engine
 export { isNativeAvailable } from './native.js';
 // Ownership (CODEOWNERS)
-export { matchOwners, owners, ownersData, ownersForFiles, parseCodeowners } from './owners.js';
+export { matchOwners, ownersData, ownersForFiles, parseCodeowners } from './owners.js';
 // Pagination utilities
 export { MCP_DEFAULTS, MCP_MAX_LIMIT, paginate, paginateResult, printNdjson } from './paginate.js';
 // Unified parser API
@@ -198,10 +209,8 @@ export {
   saveRegistry,
   unregisterRepo,
 } from './registry.js';
-// Result formatting
-export { outputResult } from './result-formatter.js';
 // Sequence diagram generation
-export { sequence, sequenceData, sequenceToMermaid } from './sequence.js';
+export { sequenceData, sequenceToMermaid } from './sequence.js';
 // Snapshot management
 export {
   snapshotDelete,
@@ -216,17 +225,12 @@ export {
   buildStructure,
   classifyNodeRoles,
   FRAMEWORK_ENTRY_PREFIXES,
-  formatHotspots,
-  formatModuleBoundaries,
-  formatStructure,
   hotspotsData,
   moduleBoundariesData,
   structureData,
 } from './structure.js';
-// Test file detection
-export { isTestFile, TEST_PATTERN } from './test-filter.js';
 // Triage — composite risk audit
-export { triage, triageData } from './triage.js';
+export { triageData } from './triage.js';
 // Interactive HTML viewer
 export { generatePlotHTML, loadPlotConfig } from './viewer.js';
 // Watch mode

package/src/{result-formatter.js → infrastructure/result-formatter.js}

@@ -1,4 +1,4 @@
-import { printNdjson } from './paginate.js';
+import { printNdjson } from '../paginate.js';
 
 /**
  * Shared JSON / NDJSON output dispatch for CLI wrappers.

package/src/kinds.js

@@ -33,6 +33,7 @@ export const ALL_SYMBOL_KINDS = CORE_SYMBOL_KINDS;
 export const CORE_EDGE_KINDS = [
   'imports',
   'imports-type',
+  'dynamic-imports',
   'reexports',
   'calls',
   'extends',

package/src/manifesto.js

@@ -4,7 +4,6 @@ import { findCycles } from './cycles.js';
 import { openReadonlyOrFail } from './db.js';
 import { debug } from './logger.js';
 import { paginateResult } from './paginate.js';
-import { outputResult } from './result-formatter.js';
 
 // ─── Rule Definitions ─────────────────────────────────────────────────
 
@@ -428,78 +427,3 @@ export function manifestoData(customDbPath, opts = {}) {
     db.close();
   }
 }
-
-/**
- * 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/owners.js

@@ -1,8 +1,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { findDbPath, openReadonlyOrFail } from './db.js';
-import { outputResult } from './result-formatter.js';
-import { isTestFile } from './test-filter.js';
+import { isTestFile } from './infrastructure/test-filter.js';
 
 // ─── CODEOWNERS Parsing ──────────────────────────────────────────────
 
@@ -303,57 +302,3 @@ export function ownersData(customDbPath, opts = {}) {
     db.close();
   }
 }
-
-// ─── CLI Display ─────────────────────────────────────────────────────
-
-/**
- * CLI display function for the `owners` command.
- * @param {string} [customDbPath]
- * @param {object} [opts]
- */
-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/queries-cli.js

@@ -7,6 +7,7 @@
  */
 
 import path from 'node:path';
+import { outputResult } from './infrastructure/result-formatter.js';
 import {
   childrenData,
   contextData,
@@ -26,7 +27,6 @@ import {
   statsData,
   whereData,
 } from './queries.js';
-import { outputResult } from './result-formatter.js';
 
 // ─── symbolPath ─────────────────────────────────────────────────────────