@optave/codegraph 3.1.0 → 3.1.2

package/src/manifesto.js

@@ -3,7 +3,7 @@ import { loadConfig } from './config.js';
 import { findCycles } from './cycles.js';
 import { openReadonlyOrFail } from './db.js';
 import { debug } from './logger.js';
-import { paginateResult, printNdjson } from './paginate.js';
+import { paginateResult } from './paginate.js';
 
 // ─── Rule Definitions ─────────────────────────────────────────────────
 
@@ -427,84 +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 (opts.ndjson) {
-    printNdjson(data, 'violations');
-    if (!data.passed) process.exit(1);
-    return;
-  }
-  if (opts.json) {
-    console.log(JSON.stringify(data, null, 2));
-    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/mcp.js

@@ -836,26 +836,27 @@ export async function startMCPServer(customDbPath, options = {}) {
     process.exit(1);
   }
 
-  // Lazy import query functions to avoid circular deps at module load
-  const {
-    impactAnalysisData,
-    moduleMapData,
-    fileDepsData,
-    exportsData,
-    fnDepsData,
-    fnImpactData,
-    pathData,
-    contextData,
-    childrenData,
-    explainData,
-    whereData,
-    diffImpactData,
-    listFunctionsData,
-    rolesData,
-  } = await import('./queries.js');
+  // Connect transport FIRST so the server can receive the client's
+  // `initialize` request while heavy modules (queries, better-sqlite3)
+  // are still loading.  These are lazy-loaded on the first tool call
+  // and cached for subsequent calls.
+  let _queries;
+  let _Database;
 
-  const require = createRequire(import.meta.url);
-  const Database = require('better-sqlite3');
+  async function getQueries() {
+    if (!_queries) {
+      _queries = await import('./queries.js');
+    }
+    return _queries;
+  }
+
+  function getDatabase() {
+    if (!_Database) {
+      const require = createRequire(import.meta.url);
+      _Database = require('better-sqlite3');
+    }
+    return _Database;
+  }
 
   const server = new Server(
     { name: 'codegraph', version: '1.0.0' },
@@ -868,8 +869,24 @@ export async function startMCPServer(customDbPath, options = {}) {
 
   server.setRequestHandler(CallToolRequestSchema, async (request) => {
     const { name, arguments: args } = request.params;
-
     try {
+      const {
+        impactAnalysisData,
+        moduleMapData,
+        fileDepsData,
+        exportsData,
+        fnDepsData,
+        fnImpactData,
+        pathData,
+        contextData,
+        childrenData,
+        explainData,
+        whereData,
+        diffImpactData,
+        listFunctionsData,
+        rolesData,
+      } = await getQueries();
+      const Database = getDatabase();
       if (!multiRepo && args.repo) {
         throw new Error(
           'Multi-repo access is disabled. Restart with `codegraph mcp --multi-repo` to access other repositories.',

package/src/owners.js

@@ -1,7 +1,7 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { findDbPath, openReadonlyOrFail } from './db.js';
-import { isTestFile } from './queries.js';
+import { isTestFile } from './infrastructure/test-filter.js';
 
 // ─── CODEOWNERS Parsing ──────────────────────────────────────────────
 
@@ -163,197 +163,142 @@ export function ownersForFiles(filePaths, repoRoot) {
  */
 export function ownersData(customDbPath, opts = {}) {
   const db = openReadonlyOrFail(customDbPath);
-  const dbPath = findDbPath(customDbPath);
-  const repoRoot = path.resolve(path.dirname(dbPath), '..');
-
-  const parsed = parseCodeowners(repoRoot);
-  if (!parsed) {
-    db.close();
-    return {
-      codeownersFile: null,
-      files: [],
-      symbols: [],
-      boundaries: [],
-      summary: {
-        totalFiles: 0,
-        ownedFiles: 0,
-        unownedFiles: 0,
-        coveragePercent: 0,
-        ownerCount: 0,
-        byOwner: [],
-      },
-    };
-  }
-
-  // Get all distinct files from nodes
-  let allFiles = db
-    .prepare('SELECT DISTINCT file FROM nodes')
-    .all()
-    .map((r) => r.file);
+  try {
+    const dbPath = findDbPath(customDbPath);
+    const repoRoot = path.resolve(path.dirname(dbPath), '..');
+
+    const parsed = parseCodeowners(repoRoot);
+    if (!parsed) {
+      return {
+        codeownersFile: null,
+        files: [],
+        symbols: [],
+        boundaries: [],
+        summary: {
+          totalFiles: 0,
+          ownedFiles: 0,
+          unownedFiles: 0,
+          coveragePercent: 0,
+          ownerCount: 0,
+          byOwner: [],
+        },
+      };
+    }
 
-  if (opts.noTests) allFiles = allFiles.filter((f) => !isTestFile(f));
-  if (opts.file) {
-    const filter = opts.file;
-    allFiles = allFiles.filter((f) => f.includes(filter));
-  }
+    // Get all distinct files from nodes
+    let allFiles = db
+      .prepare('SELECT DISTINCT file FROM nodes')
+      .all()
+      .map((r) => r.file);
 
-  // Map files to owners
-  const fileOwners = allFiles.map((file) => ({
-    file,
-    owners: matchOwners(file, parsed.rules),
-  }));
-
-  // Build owner-to-files index
-  const ownerIndex = new Map();
-  let ownedCount = 0;
-  for (const fo of fileOwners) {
-    if (fo.owners.length > 0) ownedCount++;
-    for (const o of fo.owners) {
-      if (!ownerIndex.has(o)) ownerIndex.set(o, []);
-      ownerIndex.get(o).push(fo.file);
+    if (opts.noTests) allFiles = allFiles.filter((f) => !isTestFile(f));
+    if (opts.file) {
+      const filter = opts.file;
+      allFiles = allFiles.filter((f) => f.includes(filter));
     }
-  }
 
-  // Filter files if --owner specified
-  let filteredFiles = fileOwners;
-  if (opts.owner) {
-    filteredFiles = fileOwners.filter((fo) => fo.owners.includes(opts.owner));
-  }
-
-  // Get symbols for filtered files
-  const fileSet = new Set(filteredFiles.map((fo) => fo.file));
-  let symbols = db
-    .prepare('SELECT name, kind, file, line FROM nodes')
-    .all()
-    .filter((n) => fileSet.has(n.file));
-
-  if (opts.noTests) symbols = symbols.filter((s) => !isTestFile(s.file));
-  if (opts.kind) symbols = symbols.filter((s) => s.kind === opts.kind);
-
-  const symbolsWithOwners = symbols.map((s) => ({
-    ...s,
-    owners: matchOwners(s.file, parsed.rules),
-  }));
-
-  // Boundary analysis — cross-owner call edges
-  const boundaries = [];
-  if (opts.boundary) {
-    const edges = db
-      .prepare(
-        `SELECT e.id, e.kind AS edgeKind,
-                s.name AS srcName, s.kind AS srcKind, s.file AS srcFile, s.line AS srcLine,
-                t.name AS tgtName, t.kind AS tgtKind, t.file AS tgtFile, t.line AS tgtLine
-         FROM edges e
-         JOIN nodes s ON e.source_id = s.id
-         JOIN nodes t ON e.target_id = t.id
-         WHERE e.kind = 'calls'`,
-      )
-      .all();
-
-    for (const e of edges) {
-      if (opts.noTests && (isTestFile(e.srcFile) || isTestFile(e.tgtFile))) continue;
-      const srcOwners = matchOwners(e.srcFile, parsed.rules);
-      const tgtOwners = matchOwners(e.tgtFile, parsed.rules);
-      // Cross-boundary: different owner sets
-      const srcKey = srcOwners.sort().join(',');
-      const tgtKey = tgtOwners.sort().join(',');
-      if (srcKey !== tgtKey) {
-        boundaries.push({
-          from: {
-            name: e.srcName,
-            kind: e.srcKind,
-            file: e.srcFile,
-            line: e.srcLine,
-            owners: srcOwners,
-          },
-          to: {
-            name: e.tgtName,
-            kind: e.tgtKind,
-            file: e.tgtFile,
-            line: e.tgtLine,
-            owners: tgtOwners,
-          },
-          edgeKind: e.edgeKind,
-        });
+    // Map files to owners
+    const fileOwners = allFiles.map((file) => ({
+      file,
+      owners: matchOwners(file, parsed.rules),
+    }));
+
+    // Build owner-to-files index
+    const ownerIndex = new Map();
+    let ownedCount = 0;
+    for (const fo of fileOwners) {
+      if (fo.owners.length > 0) ownedCount++;
+      for (const o of fo.owners) {
+        if (!ownerIndex.has(o)) ownerIndex.set(o, []);
+        ownerIndex.get(o).push(fo.file);
       }
     }
-  }
 
-  // Summary
-  const byOwner = [...ownerIndex.entries()]
-    .map(([owner, files]) => ({ owner, fileCount: files.length }))
-    .sort((a, b) => b.fileCount - a.fileCount);
-
-  db.close();
-  return {
-    codeownersFile: parsed.path,
-    files: filteredFiles,
-    symbols: symbolsWithOwners,
-    boundaries,
-    summary: {
-      totalFiles: allFiles.length,
-      ownedFiles: ownedCount,
-      unownedFiles: allFiles.length - ownedCount,
-      coveragePercent: allFiles.length > 0 ? Math.round((ownedCount / allFiles.length) * 100) : 0,
-      ownerCount: ownerIndex.size,
-      byOwner,
-    },
-  };
-}
-
-// ─── 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 (opts.json) {
-    console.log(JSON.stringify(data, null, 2));
-    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`);
+    // Filter files if --owner specified
+    let filteredFiles = fileOwners;
+    if (opts.owner) {
+      filteredFiles = fileOwners.filter((fo) => fo.owners.includes(opts.owner));
     }
-    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}`);
+    // Get symbols for filtered files
+    const fileSet = new Set(filteredFiles.map((fo) => fo.file));
+    let symbols = db
+      .prepare('SELECT name, kind, file, line FROM nodes')
+      .all()
+      .filter((n) => fileSet.has(n.file));
+
+    if (opts.noTests) symbols = symbols.filter((s) => !isTestFile(s.file));
+    if (opts.kind) symbols = symbols.filter((s) => s.kind === opts.kind);
+
+    const symbolsWithOwners = symbols.map((s) => ({
+      ...s,
+      owners: matchOwners(s.file, parsed.rules),
+    }));
+
+    // Boundary analysis — cross-owner call edges
+    const boundaries = [];
+    if (opts.boundary) {
+      const edges = db
+        .prepare(
+          `SELECT e.id, e.kind AS edgeKind,
+                  s.name AS srcName, s.kind AS srcKind, s.file AS srcFile, s.line AS srcLine,
+                  t.name AS tgtName, t.kind AS tgtKind, t.file AS tgtFile, t.line AS tgtLine
+           FROM edges e
+           JOIN nodes s ON e.source_id = s.id
+           JOIN nodes t ON e.target_id = t.id
+           WHERE e.kind = 'calls'`,
+        )
+        .all();
+
+      for (const e of edges) {
+        if (opts.noTests && (isTestFile(e.srcFile) || isTestFile(e.tgtFile))) continue;
+        const srcOwners = matchOwners(e.srcFile, parsed.rules);
+        const tgtOwners = matchOwners(e.tgtFile, parsed.rules);
+        // Cross-boundary: different owner sets
+        const srcKey = srcOwners.sort().join(',');
+        const tgtKey = tgtOwners.sort().join(',');
+        if (srcKey !== tgtKey) {
+          boundaries.push({
+            from: {
+              name: e.srcName,
+              kind: e.srcKind,
+              file: e.srcFile,
+              line: e.srcLine,
+              owners: srcOwners,
+            },
+            to: {
+              name: e.tgtName,
+              kind: e.tgtKind,
+              file: e.tgtFile,
+              line: e.tgtLine,
+              owners: tgtOwners,
+            },
+            edgeKind: e.edgeKind,
+          });
+        }
+      }
     }
-    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();
+    // Summary
+    const byOwner = [...ownerIndex.entries()]
+      .map(([owner, files]) => ({ owner, fileCount: files.length }))
+      .sort((a, b) => b.fileCount - a.fileCount);
+
+    return {
+      codeownersFile: parsed.path,
+      files: filteredFiles,
+      symbols: symbolsWithOwners,
+      boundaries,
+      summary: {
+        totalFiles: allFiles.length,
+        ownedFiles: ownedCount,
+        unownedFiles: allFiles.length - ownedCount,
+        coveragePercent: allFiles.length > 0 ? Math.round((ownedCount / allFiles.length) * 100) : 0,
+        ownerCount: ownerIndex.size,
+        byOwner,
+      },
+    };
+  } finally {
+    db.close();
   }
 }