@optave/codegraph 3.1.5 → 3.3.0

package/src/mcp/tools/implementations.js

@@ -0,0 +1,14 @@
+import { effectiveLimit, effectiveOffset } from '../middleware.js';
+
+export const name = 'implementations';
+
+export async function handler(args, ctx) {
+  const { implementationsData } = await ctx.getQueries();
+  return implementationsData(args.name, ctx.dbPath, {
+    file: args.file,
+    kind: args.kind,
+    noTests: args.no_tests,
+    limit: effectiveLimit(args, name),
+    offset: effectiveOffset(args),
+  });
+}

package/src/mcp/tools/index.js

@@ -6,6 +6,7 @@ import * as astQuery from './ast-query.js';
 import * as audit from './audit.js';
 import * as batchQuery from './batch-query.js';
 import * as branchCompare from './branch-compare.js';
+import * as brief from './brief.js';
 import * as cfg from './cfg.js';
 import * as check from './check.js';
 import * as coChanges from './co-changes.js';
@@ -22,6 +23,8 @@ import * as fileExports from './file-exports.js';
 import * as findCycles from './find-cycles.js';
 import * as fnImpact from './fn-impact.js';
 import * as impactAnalysis from './impact-analysis.js';
+import * as implementations from './implementations.js';
+import * as interfaces from './interfaces.js';
 import * as listFunctions from './list-functions.js';
 import * as listRepos from './list-repos.js';
 import * as moduleMap from './module-map.js';
@@ -67,5 +70,8 @@ export const TOOL_HANDLERS = new Map([
   [dataflow.name, dataflow],
   [check.name, check],
   [astQuery.name, astQuery],
+  [brief.name, brief],
+  [implementations.name, implementations],
+  [interfaces.name, interfaces],
   [listRepos.name, listRepos],
 ]);

package/src/mcp/tools/interfaces.js

@@ -0,0 +1,14 @@
+import { effectiveLimit, effectiveOffset } from '../middleware.js';
+
+export const name = 'interfaces';
+
+export async function handler(args, ctx) {
+  const { interfacesData } = await ctx.getQueries();
+  return interfacesData(args.name, ctx.dbPath, {
+    file: args.file,
+    kind: args.kind,
+    noTests: args.no_tests,
+    limit: effectiveLimit(args, name),
+    offset: effectiveOffset(args),
+  });
+}

package/src/presentation/brief.js

@@ -0,0 +1,51 @@
+import { briefData } from '../domain/analysis/brief.js';
+import { outputResult } from './result-formatter.js';
+
+/**
+ * Format a compact brief for hook context injection.
+ * Single-block, token-efficient output.
+ *
+ * Example:
+ *   src/domain/graph/builder.js [HIGH RISK]
+ *   Symbols: buildGraph [core, 12 callers], collectFiles [leaf, 2 callers]
+ *   Imports: src/db/index.js, src/domain/parser.js
+ *   Imported by: src/cli/commands/build.js (+8 transitive)
+ */
+export function brief(file, customDbPath, opts = {}) {
+  const data = briefData(file, customDbPath, opts);
+  if (outputResult(data, 'results', opts)) return;
+
+  if (data.results.length === 0) {
+    console.log(`No file matching "${file}" in graph`);
+    return;
+  }
+
+  for (const r of data.results) {
+    console.log(`${r.file} [${r.risk.toUpperCase()} RISK]`);
+
+    // Symbols line
+    if (r.symbols.length > 0) {
+      const parts = r.symbols.map((s) => {
+        const tags = [];
+        if (s.role) tags.push(s.role);
+        tags.push(`${s.callerCount} caller${s.callerCount !== 1 ? 's' : ''}`);
+        return `${s.name} [${tags.join(', ')}]`;
+      });
+      console.log(`  Symbols: ${parts.join(', ')}`);
+    }
+
+    // Imports line
+    if (r.imports.length > 0) {
+      console.log(`  Imports: ${r.imports.join(', ')}`);
+    }
+
+    // Imported by line with transitive count
+    if (r.importedBy.length > 0) {
+      const transitive = r.totalImporterCount - r.importedBy.length;
+      const suffix = transitive > 0 ? ` (+${transitive} transitive)` : '';
+      console.log(`  Imported by: ${r.importedBy.join(', ')}${suffix}`);
+    } else if (r.totalImporterCount > 0) {
+      console.log(`  Imported by: ${r.totalImporterCount} transitive importers`);
+    }
+  }
+}

package/src/presentation/queries-cli/exports.js

@@ -1,19 +1,7 @@
 import { exportsData, kindIcon } from '../../domain/queries.js';
 import { outputResult } from '../../infrastructure/result-formatter.js';
 
-export function fileExports(file, customDbPath, opts = {}) {
-  const data = exportsData(file, customDbPath, opts);
-  if (outputResult(data, 'results', opts)) return;
-
-  if (data.results.length === 0) {
-    if (opts.unused) {
-      console.log(`No unused exports found for "${file}".`);
-    } else {
-      console.log(`No exported symbols found for "${file}". Run "codegraph build" first.`);
-    }
-    return;
-  }
-
+function printExportHeader(data, opts) {
   if (opts.unused) {
     console.log(
       `\n# ${data.file} — ${data.totalUnused} unused export${data.totalUnused !== 1 ? 's' : ''} (of ${data.totalExported} exported)\n`,
@@ -24,8 +12,10 @@ export function fileExports(file, customDbPath, opts = {}) {
       `\n# ${data.file} — ${data.totalExported} exported${unusedNote}, ${data.totalInternal} internal\n`,
     );
   }
+}
 
-  for (const sym of data.results) {
+function printExportSymbols(results) {
+  for (const sym of results) {
     const icon = kindIcon(sym.kind);
     const sig = sym.signature?.params ? `(${sym.signature.params})` : '';
     const role = sym.role ? ` [${sym.role}]` : '';
@@ -38,9 +28,76 @@ export function fileExports(file, customDbPath, opts = {}) {
       }
     }
   }
+}
+
+function printReexportedSymbols(reexportedSymbols) {
+  // Group by origin file
+  const byOrigin = new Map();
+  for (const sym of reexportedSymbols) {
+    if (!byOrigin.has(sym.originFile)) byOrigin.set(sym.originFile, []);
+    byOrigin.get(sym.originFile).push(sym);
+  }
+
+  for (const [originFile, syms] of byOrigin) {
+    console.log(`\n  from ${originFile}:`);
+    for (const sym of syms) {
+      const icon = kindIcon(sym.kind);
+      const sig = sym.signature?.params ? `(${sym.signature.params})` : '';
+      const role = sym.role ? ` [${sym.role}]` : '';
+      console.log(`    ${icon} ${sym.name}${sig}${role} :${sym.line}`);
+      if (sym.consumers.length === 0) {
+        console.log('      (no consumers)');
+      } else {
+        for (const c of sym.consumers) {
+          console.log(`      <- ${c.name} (${c.file}:${c.line})`);
+        }
+      }
+    }
+  }
+}
+
+export function fileExports(file, customDbPath, opts = {}) {
+  const data = exportsData(file, customDbPath, opts);
+  if (outputResult(data, 'results', opts)) return;
+
+  const hasReexported = data.reexportedSymbols && data.reexportedSymbols.length > 0;
+
+  if (data.results.length === 0 && !hasReexported) {
+    if (opts.unused) {
+      console.log(`No unused exports found for "${file}".`);
+    } else {
+      console.log(`No exported symbols found for "${file}". Run "codegraph build" first.`);
+    }
+    return;
+  }
+
+  if (data.results.length > 0) {
+    printExportHeader(data, opts);
+    printExportSymbols(data.results);
+  }
+
+  if (hasReexported) {
+    const totalReexported = opts.unused
+      ? (data.totalReexportedUnused ?? data.reexportedSymbols.length)
+      : (data.totalReexported ?? data.reexportedSymbols.length);
+    if (data.results.length === 0) {
+      if (opts.unused) {
+        console.log(
+          `\n# ${data.file} — barrel file (${totalReexported} unused re-exported symbol${totalReexported !== 1 ? 's' : ''} from sub-modules)\n`,
+        );
+      } else {
+        console.log(
+          `\n# ${data.file} — barrel file (${totalReexported} re-exported symbol${totalReexported !== 1 ? 's' : ''} from sub-modules)\n`,
+        );
+      }
+    } else {
+      console.log(`\n  Re-exported symbols (${totalReexported} from sub-modules):`);
+    }
+    printReexportedSymbols(data.reexportedSymbols);
+  }
 
   if (data.reexports.length > 0) {
-    console.log(`\n  Re-exports: ${data.reexports.map((r) => r.file).join(', ')}`);
+    console.log(`\n  Re-exported by: ${data.reexports.map((r) => r.file).join(', ')}`);
   }
   console.log();
 }

package/src/presentation/queries-cli/impact.js

@@ -132,6 +132,56 @@ export function fnImpact(name, customDbPath, opts = {}) {
   }
 }
 
+function printDiffFunctions(data) {
+  console.log(`\ndiff-impact: ${data.changedFiles} files changed\n`);
+  console.log(`  ${data.affectedFunctions.length} functions changed:\n`);
+  for (const fn of data.affectedFunctions) {
+    console.log(`  ${kindIcon(fn.kind)} ${fn.name} -- ${fn.file}:${fn.line}`);
+    if (fn.transitiveCallers > 0) console.log(`    ^ ${fn.transitiveCallers} transitive callers`);
+  }
+}
+
+function printDiffCoupled(data) {
+  if (!data.historicallyCoupled?.length) return;
+  console.log('\n  Historically coupled (not in static graph):\n');
+  for (const c of data.historicallyCoupled) {
+    const pct = `${(c.jaccard * 100).toFixed(0)}%`;
+    console.log(
+      `    ${c.file}  <- coupled with ${c.coupledWith} (${pct}, ${c.commitCount} commits)`,
+    );
+  }
+}
+
+function printDiffOwnership(data) {
+  if (!data.ownership) return;
+  console.log(`\n  Affected owners: ${data.ownership.affectedOwners.join(', ')}`);
+  console.log(`  Suggested reviewers: ${data.ownership.suggestedReviewers.join(', ')}`);
+}
+
+function printDiffBoundaries(data) {
+  if (!data.boundaryViolations?.length) return;
+  console.log(`\n  Boundary violations (${data.boundaryViolationCount}):\n`);
+  for (const v of data.boundaryViolations) {
+    console.log(`    [${v.name}] ${v.file} -> ${v.targetFile}`);
+    if (v.message) console.log(`      ${v.message}`);
+  }
+}
+
+function printDiffSummary(summary) {
+  if (!summary) return;
+  let line = `\n  Summary: ${summary.functionsChanged} functions changed -> ${summary.callersAffected} callers affected across ${summary.filesAffected} files`;
+  if (summary.historicallyCoupledCount > 0) {
+    line += `, ${summary.historicallyCoupledCount} historically coupled`;
+  }
+  if (summary.ownersAffected > 0) {
+    line += `, ${summary.ownersAffected} owners affected`;
+  }
+  if (summary.boundaryViolationCount > 0) {
+    line += `, ${summary.boundaryViolationCount} boundary violations`;
+  }
+  console.log(`${line}\n`);
+}
+
 export function diffImpact(customDbPath, opts = {}) {
   if (opts.format === 'mermaid') {
     console.log(diffImpactMermaid(customDbPath, opts));
@@ -156,43 +206,9 @@ export function diffImpact(customDbPath, opts = {}) {
     return;
   }
 
-  console.log(`\ndiff-impact: ${data.changedFiles} files changed\n`);
-  console.log(`  ${data.affectedFunctions.length} functions changed:\n`);
-  for (const fn of data.affectedFunctions) {
-    console.log(`  ${kindIcon(fn.kind)} ${fn.name} -- ${fn.file}:${fn.line}`);
-    if (fn.transitiveCallers > 0) console.log(`    ^ ${fn.transitiveCallers} transitive callers`);
-  }
-  if (data.historicallyCoupled && data.historicallyCoupled.length > 0) {
-    console.log('\n  Historically coupled (not in static graph):\n');
-    for (const c of data.historicallyCoupled) {
-      const pct = `${(c.jaccard * 100).toFixed(0)}%`;
-      console.log(
-        `    ${c.file}  <- coupled with ${c.coupledWith} (${pct}, ${c.commitCount} commits)`,
-      );
-    }
-  }
-  if (data.ownership) {
-    console.log(`\n  Affected owners: ${data.ownership.affectedOwners.join(', ')}`);
-    console.log(`  Suggested reviewers: ${data.ownership.suggestedReviewers.join(', ')}`);
-  }
-  if (data.boundaryViolations && data.boundaryViolations.length > 0) {
-    console.log(`\n  Boundary violations (${data.boundaryViolationCount}):\n`);
-    for (const v of data.boundaryViolations) {
-      console.log(`    [${v.name}] ${v.file} -> ${v.targetFile}`);
-      if (v.message) console.log(`      ${v.message}`);
-    }
-  }
-  if (data.summary) {
-    let summaryLine = `\n  Summary: ${data.summary.functionsChanged} functions changed -> ${data.summary.callersAffected} callers affected across ${data.summary.filesAffected} files`;
-    if (data.summary.historicallyCoupledCount > 0) {
-      summaryLine += `, ${data.summary.historicallyCoupledCount} historically coupled`;
-    }
-    if (data.summary.ownersAffected > 0) {
-      summaryLine += `, ${data.summary.ownersAffected} owners affected`;
-    }
-    if (data.summary.boundaryViolationCount > 0) {
-      summaryLine += `, ${data.summary.boundaryViolationCount} boundary violations`;
-    }
-    console.log(`${summaryLine}\n`);
-  }
+  printDiffFunctions(data);
+  printDiffCoupled(data);
+  printDiffOwnership(data);
+  printDiffBoundaries(data);
+  printDiffSummary(data.summary);
 }

package/src/presentation/queries-cli/index.js

@@ -1,5 +1,13 @@
 export { fileExports } from './exports.js';
 export { diffImpact, fileDeps, fnDeps, fnImpact, impactAnalysis } from './impact.js';
-export { children, context, explain, queryName, where } from './inspect.js';
+export {
+  children,
+  context,
+  explain,
+  implementations,
+  interfaces,
+  queryName,
+  where,
+} from './inspect.js';
 export { moduleMap, roles, stats } from './overview.js';
 export { symbolPath } from './path.js';