@optave/codegraph 3.4.0 → 3.4.1

package/src/features/complexity.ts

@@ -12,15 +12,10 @@ import {
 } from '../ast-analysis/shared.js';
 import { walkWithVisitors } from '../ast-analysis/visitor.js';
 import { createComplexityVisitor } from '../ast-analysis/visitors/complexity-visitor.js';
-import { getFunctionNodeId, openReadonlyOrFail } from '../db/index.js';
-import { buildFileConditionSQL } from '../db/query-builder.js';
-import { loadConfig } from '../infrastructure/config.js';
+import { getFunctionNodeId } from '../db/index.js';
 import { debug, info } from '../infrastructure/logger.js';
-import { isTestFile } from '../infrastructure/test-filter.js';
-import { paginateResult } from '../shared/paginate.js';
 import type {
   BetterSqlite3Database,
-  CodegraphConfig,
   ComplexityRules,
   HalsteadDerivedMetrics,
   HalsteadRules,
@@ -305,8 +300,7 @@ export function computeAllMetrics(
     nestingNodeTypes: nestingNodes,
   });
 
-  // biome-ignore lint/complexity/useLiteralKeys: noPropertyAccessFromIndexSignature requires bracket notation
-  const rawResult = results['complexity'] as {
+  const rawResult = results.complexity as {
     cognitive: number;
     cyclomatic: number;
     maxNesting: number;
@@ -388,7 +382,6 @@ function getTreeForFile(
   rootDir: string,
   parsers: unknown,
   extToLang: Map<string, string> | null,
-  // biome-ignore lint/suspicious/noExplicitAny: dynamic import from parser.js
   getParser: (parsers: any, absPath: string) => any,
 ): { tree: { rootNode: TreeSitterNode }; langId: string } | null {
   let tree = symbols._tree;
@@ -556,359 +549,7 @@ export async function buildComplexityMetrics(
   }
 }
 
-// ─── Query-Time Functions ─────────────────────────────────────────────────
-
-interface ComplexityRow {
-  name: string;
-  kind: string;
-  file: string;
-  line: number;
-  end_line: number | null;
-  cognitive: number;
-  cyclomatic: number;
-  max_nesting: number;
-  loc: number;
-  sloc: number;
-  maintainability_index: number;
-  halstead_volume: number;
-  halstead_difficulty: number;
-  halstead_effort: number;
-  halstead_bugs: number;
-}
-
-export function complexityData(
-  customDbPath?: string,
-  opts: {
-    target?: string;
-    limit?: number;
-    sort?: string;
-    aboveThreshold?: boolean;
-    file?: string;
-    kind?: string;
-    noTests?: boolean;
-    config?: CodegraphConfig;
-    offset?: number;
-  } = {},
-): Record<string, unknown> {
-  const db = openReadonlyOrFail(customDbPath);
-  try {
-    const sort = opts.sort || 'cognitive';
-    const noTests = opts.noTests || false;
-    const aboveThreshold = opts.aboveThreshold || false;
-    const target = opts.target || null;
-    const fileFilter = opts.file || null;
-    const kindFilter = opts.kind || null;
-
-    // Load thresholds from config
-    const config = opts.config || loadConfig(process.cwd());
-    // biome-ignore lint/suspicious/noExplicitAny: thresholds come from config with dynamic keys
-    const thresholds: any = config.manifesto?.rules || {
-      cognitive: { warn: 15, fail: null },
-      cyclomatic: { warn: 10, fail: null },
-      maxNesting: { warn: 4, fail: null },
-      maintainabilityIndex: { warn: 20, fail: null },
-    };
-
-    // Build query
-    let where = "WHERE n.kind IN ('function','method')";
-    const params: unknown[] = [];
-
-    if (noTests) {
-      where += ` AND n.file NOT LIKE '%.test.%'
-       AND n.file NOT LIKE '%.spec.%'
-       AND n.file NOT LIKE '%__test__%'
-       AND n.file NOT LIKE '%__tests__%'
-       AND n.file NOT LIKE '%.stories.%'`;
-    }
-    if (target) {
-      where += ' AND n.name LIKE ?';
-      params.push(`%${target}%`);
-    }
-    {
-      const fc = buildFileConditionSQL(fileFilter as string, 'n.file');
-      where += fc.sql;
-      params.push(...fc.params);
-    }
-    if (kindFilter) {
-      where += ' AND n.kind = ?';
-      params.push(kindFilter);
-    }
-
-    const isValidThreshold = (v: unknown): v is number =>
-      typeof v === 'number' && Number.isFinite(v);
-
-    let having = '';
-    if (aboveThreshold) {
-      const conditions: string[] = [];
-      if (isValidThreshold(thresholds.cognitive?.warn)) {
-        conditions.push(`fc.cognitive >= ${thresholds.cognitive.warn}`);
-      }
-      if (isValidThreshold(thresholds.cyclomatic?.warn)) {
-        conditions.push(`fc.cyclomatic >= ${thresholds.cyclomatic.warn}`);
-      }
-      if (isValidThreshold(thresholds.maxNesting?.warn)) {
-        conditions.push(`fc.max_nesting >= ${thresholds.maxNesting.warn}`);
-      }
-      if (isValidThreshold(thresholds.maintainabilityIndex?.warn)) {
-        conditions.push(
-          `fc.maintainability_index > 0 AND fc.maintainability_index <= ${thresholds.maintainabilityIndex.warn}`,
-        );
-      }
-      if (conditions.length > 0) {
-        having = `AND (${conditions.join(' OR ')})`;
-      }
-    }
-
-    const orderMap: Record<string, string> = {
-      cognitive: 'fc.cognitive DESC',
-      cyclomatic: 'fc.cyclomatic DESC',
-      nesting: 'fc.max_nesting DESC',
-      mi: 'fc.maintainability_index ASC',
-      volume: 'fc.halstead_volume DESC',
-      effort: 'fc.halstead_effort DESC',
-      bugs: 'fc.halstead_bugs DESC',
-      loc: 'fc.loc DESC',
-    };
-    const orderBy = orderMap[sort] || 'fc.cognitive DESC';
-
-    let rows: ComplexityRow[];
-    try {
-      rows = db
-        .prepare<ComplexityRow>(
-          `SELECT n.name, n.kind, n.file, n.line, n.end_line,
-                fc.cognitive, fc.cyclomatic, fc.max_nesting,
-                fc.loc, fc.sloc, fc.maintainability_index,
-                fc.halstead_volume, fc.halstead_difficulty, fc.halstead_effort, fc.halstead_bugs
-         FROM function_complexity fc
-         JOIN nodes n ON fc.node_id = n.id
-         ${where} ${having}
-         ORDER BY ${orderBy}`,
-        )
-        .all(...params);
-    } catch (e: unknown) {
-      debug(`complexity query failed (table may not exist): ${(e as Error).message}`);
-      // Check if graph has nodes even though complexity table is missing/empty
-      let hasGraph = false;
-      try {
-        hasGraph = (db.prepare<{ c: number }>('SELECT COUNT(*) as c FROM nodes').get()?.c ?? 0) > 0;
-      } catch (e2: unknown) {
-        debug(`nodes table check failed: ${(e2 as Error).message}`);
-      }
-      return { functions: [], summary: null, thresholds, hasGraph };
-    }
-
-    // Post-filter test files if needed (belt-and-suspenders for isTestFile)
-    const filtered = noTests ? rows.filter((r) => !isTestFile(r.file)) : rows;
-
-    const functions = filtered.map((r) => {
-      const exceeds: string[] = [];
-      if (
-        isValidThreshold(thresholds.cognitive?.warn) &&
-        r.cognitive >= (thresholds.cognitive?.warn ?? 0)
-      )
-        exceeds.push('cognitive');
-      if (
-        isValidThreshold(thresholds.cyclomatic?.warn) &&
-        r.cyclomatic >= (thresholds.cyclomatic?.warn ?? 0)
-      )
-        exceeds.push('cyclomatic');
-      if (
-        isValidThreshold(thresholds.maxNesting?.warn) &&
-        r.max_nesting >= (thresholds.maxNesting?.warn ?? 0)
-      )
-        exceeds.push('maxNesting');
-      if (
-        isValidThreshold(thresholds.maintainabilityIndex?.warn) &&
-        r.maintainability_index > 0 &&
-        r.maintainability_index <= (thresholds.maintainabilityIndex?.warn ?? 0)
-      )
-        exceeds.push('maintainabilityIndex');
-
-      return {
-        name: r.name,
-        kind: r.kind,
-        file: r.file,
-        line: r.line,
-        endLine: r.end_line || null,
-        cognitive: r.cognitive,
-        cyclomatic: r.cyclomatic,
-        maxNesting: r.max_nesting,
-        loc: r.loc || 0,
-        sloc: r.sloc || 0,
-        maintainabilityIndex: r.maintainability_index || 0,
-        halstead: {
-          volume: r.halstead_volume || 0,
-          difficulty: r.halstead_difficulty || 0,
-          effort: r.halstead_effort || 0,
-          bugs: r.halstead_bugs || 0,
-        },
-        exceeds: exceeds.length > 0 ? exceeds : undefined,
-      };
-    });
-
-    // Summary stats
-    let summary: Record<string, unknown> | null = null;
-    try {
-      const allRows = db
-        .prepare<{
-          cognitive: number;
-          cyclomatic: number;
-          max_nesting: number;
-          maintainability_index: number;
-        }>(
-          `SELECT fc.cognitive, fc.cyclomatic, fc.max_nesting, fc.maintainability_index
-         FROM function_complexity fc JOIN nodes n ON fc.node_id = n.id
-         WHERE n.kind IN ('function','method')
-         ${noTests ? `AND n.file NOT LIKE '%.test.%' AND n.file NOT LIKE '%.spec.%' AND n.file NOT LIKE '%__test__%' AND n.file NOT LIKE '%__tests__%' AND n.file NOT LIKE '%.stories.%'` : ''}`,
-        )
-        .all();
-
-      if (allRows.length > 0) {
-        const miValues = allRows.map((r) => r.maintainability_index || 0);
-        summary = {
-          analyzed: allRows.length,
-          avgCognitive: +(allRows.reduce((s, r) => s + r.cognitive, 0) / allRows.length).toFixed(1),
-          avgCyclomatic: +(allRows.reduce((s, r) => s + r.cyclomatic, 0) / allRows.length).toFixed(
-            1,
-          ),
-          maxCognitive: Math.max(...allRows.map((r) => r.cognitive)),
-          maxCyclomatic: Math.max(...allRows.map((r) => r.cyclomatic)),
-          avgMI: +(miValues.reduce((s, v) => s + v, 0) / miValues.length).toFixed(1),
-          minMI: +Math.min(...miValues).toFixed(1),
-          aboveWarn: allRows.filter(
-            (r) =>
-              (isValidThreshold(thresholds.cognitive?.warn) &&
-                r.cognitive >= (thresholds.cognitive?.warn ?? 0)) ||
-              (isValidThreshold(thresholds.cyclomatic?.warn) &&
-                r.cyclomatic >= (thresholds.cyclomatic?.warn ?? 0)) ||
-              (isValidThreshold(thresholds.maxNesting?.warn) &&
-                r.max_nesting >= (thresholds.maxNesting?.warn ?? 0)) ||
-              (isValidThreshold(thresholds.maintainabilityIndex?.warn) &&
-                r.maintainability_index > 0 &&
-                r.maintainability_index <= (thresholds.maintainabilityIndex?.warn ?? 0)),
-          ).length,
-        };
-      }
-    } catch (e: unknown) {
-      debug(`complexity summary query failed: ${(e as Error).message}`);
-    }
-
-    // When summary is null (no complexity rows), check if graph has nodes
-    let hasGraph = false;
-    if (summary === null) {
-      try {
-        hasGraph = (db.prepare<{ c: number }>('SELECT COUNT(*) as c FROM nodes').get()?.c ?? 0) > 0;
-      } catch (e: unknown) {
-        debug(`nodes table check failed: ${(e as Error).message}`);
-      }
-    }
-
-    const base = { functions, summary, thresholds, hasGraph };
-    return paginateResult(base, 'functions', { limit: opts.limit, offset: opts.offset });
-  } finally {
-    db.close();
-  }
-}
-
-interface IterComplexityRow {
-  name: string;
-  kind: string;
-  file: string;
-  line: number;
-  end_line: number | null;
-  cognitive: number;
-  cyclomatic: number;
-  max_nesting: number;
-  loc: number;
-  sloc: number;
-}
-
-export function* iterComplexity(
-  customDbPath?: string,
-  opts: {
-    noTests?: boolean;
-    file?: string;
-    target?: string;
-    kind?: string;
-    sort?: string;
-  } = {},
-): Generator<{
-  name: string;
-  kind: string;
-  file: string;
-  line: number;
-  endLine: number | null;
-  cognitive: number;
-  cyclomatic: number;
-  maxNesting: number;
-  loc: number;
-  sloc: number;
-}> {
-  const db = openReadonlyOrFail(customDbPath);
-  try {
-    const noTests = opts.noTests || false;
-    const sort = opts.sort || 'cognitive';
-
-    let where = "WHERE n.kind IN ('function','method')";
-    const params: unknown[] = [];
-
-    if (noTests) {
-      where += ` AND n.file NOT LIKE '%.test.%'
-         AND n.file NOT LIKE '%.spec.%'
-         AND n.file NOT LIKE '%__test__%'
-         AND n.file NOT LIKE '%__tests__%'
-         AND n.file NOT LIKE '%.stories.%'`;
-    }
-    if (opts.target) {
-      where += ' AND n.name LIKE ?';
-      params.push(`%${opts.target}%`);
-    }
-    {
-      const fc = buildFileConditionSQL(opts.file as string, 'n.file');
-      where += fc.sql;
-      params.push(...fc.params);
-    }
-    if (opts.kind) {
-      where += ' AND n.kind = ?';
-      params.push(opts.kind);
-    }
-
-    const orderMap: Record<string, string> = {
-      cognitive: 'fc.cognitive DESC',
-      cyclomatic: 'fc.cyclomatic DESC',
-      nesting: 'fc.max_nesting DESC',
-      mi: 'fc.maintainability_index ASC',
-      volume: 'fc.halstead_volume DESC',
-      effort: 'fc.halstead_effort DESC',
-      bugs: 'fc.halstead_bugs DESC',
-      loc: 'fc.loc DESC',
-    };
-    const orderBy = orderMap[sort] || 'fc.cognitive DESC';
-
-    const stmt = db.prepare<IterComplexityRow>(
-      `SELECT n.name, n.kind, n.file, n.line, n.end_line,
-              fc.cognitive, fc.cyclomatic, fc.max_nesting, fc.loc, fc.sloc
-       FROM function_complexity fc
-       JOIN nodes n ON fc.node_id = n.id
-       ${where}
-       ORDER BY ${orderBy}`,
-    );
-    for (const r of stmt.iterate(...params)) {
-      if (noTests && isTestFile(r.file)) continue;
-      yield {
-        name: r.name,
-        kind: r.kind,
-        file: r.file,
-        line: r.line,
-        endLine: r.end_line || null,
-        cognitive: r.cognitive,
-        cyclomatic: r.cyclomatic,
-        maxNesting: r.max_nesting,
-        loc: r.loc || 0,
-        sloc: r.sloc || 0,
-      };
-    }
-  } finally {
-    db.close();
-  }
-}
+// ─── Query-Time Functions (re-exported from complexity-query.ts) ──────────
+// Split to separate query-time concerns (DB reads, filtering, pagination)
+// from compute-time concerns (AST traversal, metric algorithms).
+export { complexityData, iterComplexity } from './complexity-query.js';

package/src/features/dataflow.ts

@@ -65,7 +65,7 @@ export function extractDataflow(
     getFunctionName: () => null, // dataflow visitor handles its own name extraction
   });
 
-  return results['dataflow'] as DataflowResult;
+  return results.dataflow as DataflowResult;
 }
 
 // ── Build-Time Helpers ──────────────────────────────────────────────────────
@@ -79,7 +79,6 @@ interface FileSymbolsDataflow {
 
 async function initDataflowParsers(
   fileSymbols: Map<string, FileSymbolsDataflow>,
-  // biome-ignore lint/suspicious/noExplicitAny: dynamic import from parser.js
 ): Promise<{ parsers: unknown; getParserFn: ((parsers: any, absPath: string) => any) | null }> {
   let needsFallback = false;
 
@@ -94,7 +93,6 @@ async function initDataflowParsers(
   }
 
   let parsers: unknown = null;
-  // biome-ignore lint/suspicious/noExplicitAny: dynamic import from parser.js
   let getParserFn: ((parsers: any, absPath: string) => any) | null = null;
 
   if (needsFallback) {
@@ -113,7 +111,6 @@ function getDataflowForFile(
   rootDir: string,
   extToLang: Map<string, string>,
   parsers: unknown,
-  // biome-ignore lint/suspicious/noExplicitAny: dynamic import from parser.js
   getParserFn: ((parsers: any, absPath: string) => any) | null,
 ): DataflowResult | null {
   if (symbols.dataflow) return symbols.dataflow;
@@ -369,74 +366,55 @@ export function dataflowData(
     const results = nodes.map((node: NodeRow) => {
       const sym = normalizeSymbol(node, db, hc);
 
-      const flowsTo = flowsToOut.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          target: r.target_name,
-          kind: r.target_kind,
-          file: r.target_file,
-          line: r.line,
-          paramIndex: r.param_index,
-          expression: r.expression,
-          confidence: r.confidence,
-        }),
-      );
-
-      const flowsFrom = flowsToIn.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          source: r.source_name,
-          kind: r.source_kind,
-          file: r.source_file,
-          line: r.line,
-          paramIndex: r.param_index,
-          expression: r.expression,
-          confidence: r.confidence,
-        }),
-      );
-
-      const returnConsumers = returnsOut.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          consumer: r.target_name,
-          kind: r.target_kind,
-          file: r.target_file,
-          line: r.line,
-          expression: r.expression,
-        }),
-      );
-
-      const returnedBy = returnsIn.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          producer: r.source_name,
-          kind: r.source_kind,
-          file: r.source_file,
-          line: r.line,
-          expression: r.expression,
-        }),
-      );
-
-      const mutatesTargets = mutatesOut.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          target: r.target_name,
-          expression: r.expression,
-          line: r.line,
-        }),
-      );
-
-      const mutatedBy = mutatesIn.all(node.id).map(
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row
-        (r: any) => ({
-          source: r.source_name,
-          expression: r.expression,
-          line: r.line,
-        }),
-      );
+      const flowsTo = flowsToOut.all(node.id).map((r: any) => ({
+        target: r.target_name,
+        kind: r.target_kind,
+        file: r.target_file,
+        line: r.line,
+        paramIndex: r.param_index,
+        expression: r.expression,
+        confidence: r.confidence,
+      }));
+
+      const flowsFrom = flowsToIn.all(node.id).map((r: any) => ({
+        source: r.source_name,
+        kind: r.source_kind,
+        file: r.source_file,
+        line: r.line,
+        paramIndex: r.param_index,
+        expression: r.expression,
+        confidence: r.confidence,
+      }));
+
+      const returnConsumers = returnsOut.all(node.id).map((r: any) => ({
+        consumer: r.target_name,
+        kind: r.target_kind,
+        file: r.target_file,
+        line: r.line,
+        expression: r.expression,
+      }));
+
+      const returnedBy = returnsIn.all(node.id).map((r: any) => ({
+        producer: r.source_name,
+        kind: r.source_kind,
+        file: r.source_file,
+        line: r.line,
+        expression: r.expression,
+      }));
+
+      const mutatesTargets = mutatesOut.all(node.id).map((r: any) => ({
+        target: r.target_name,
+        expression: r.expression,
+        line: r.line,
+      }));
+
+      const mutatedBy = mutatesIn.all(node.id).map((r: any) => ({
+        source: r.source_name,
+        expression: r.expression,
+        line: r.line,
+      }));
 
       if (noTests) {
-        // biome-ignore lint/suspicious/noExplicitAny: raw DB row results
         const filter = (arr: any[]) => arr.filter((r: any) => !isTestFile(r.file));
         return {
           ...sym,

package/src/features/export.ts

@@ -1,5 +1,4 @@
 import path from 'node:path';
-import type BetterSqlite3 from 'better-sqlite3';
 import { isTestFile } from '../infrastructure/test-filter.js';
 import {
   renderFileLevelDOT,
@@ -12,7 +11,7 @@ import {
   renderFunctionLevelNeo4jCSV,
 } from '../presentation/export.js';
 import { paginateResult } from '../shared/paginate.js';
-import type { ExportNeo4jCSVResult, ExportOpts } from '../types.js';
+import type { BetterSqlite3Database, ExportNeo4jCSVResult, ExportOpts } from '../types.js';
 
 const DEFAULT_MIN_CONFIDENCE = 0.5;
 
@@ -73,7 +72,7 @@ interface FunctionLevelLoadOpts {
  * Load file-level edges from DB with filtering.
  */
 function loadFileLevelEdges(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   {
     noTests,
     minConfidence,
@@ -108,7 +107,7 @@ function loadFileLevelEdges(
  * Returns the maximal field set needed by any serializer.
  */
 function loadFunctionLevelEdges(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   { noTests, minConfidence, limit }: FunctionLevelLoadOpts,
 ): { edges: FunctionLevelEdge[]; totalEdges: number } {
   const minConf = minConfidence ?? DEFAULT_MIN_CONFIDENCE;
@@ -141,7 +140,7 @@ function loadFunctionLevelEdges(
  * Load directory groupings for file-level graphs.
  * Uses DB directory nodes if available, falls back to path.dirname().
  */
-function loadDirectoryGroups(db: BetterSqlite3.Database, allFiles: Set<string>): DirectoryGroup[] {
+function loadDirectoryGroups(db: BetterSqlite3Database, allFiles: Set<string>): DirectoryGroup[] {
   const hasDirectoryNodes =
     (db.prepare("SELECT COUNT(*) as c FROM nodes WHERE kind = 'directory'").get() as { c: number })
       .c > 0;
@@ -196,7 +195,7 @@ function loadDirectoryGroups(db: BetterSqlite3.Database, allFiles: Set<string>):
  * Load directory groupings for Mermaid file-level graphs (simplified — no cohesion, string arrays).
  */
 function loadMermaidDirectoryGroups(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   allFiles: Set<string>,
 ): MermaidDirectoryGroup[] {
   const hasDirectoryNodes =
@@ -239,10 +238,7 @@ function loadMermaidDirectoryGroups(
 /**
  * Load node roles for Mermaid function-level styling.
  */
-function loadNodeRoles(
-  db: BetterSqlite3.Database,
-  edges: FunctionLevelEdge[],
-): Map<string, string> {
+function loadNodeRoles(db: BetterSqlite3Database, edges: FunctionLevelEdge[]): Map<string, string> {
   const roles = new Map<string, string>();
   const seen = new Set<string>();
   for (const e of edges) {
@@ -267,7 +263,7 @@ function loadNodeRoles(
 /**
  * Export the dependency graph in DOT (Graphviz) format.
  */
-export function exportDOT(db: BetterSqlite3.Database, opts: ExportOpts = {}): string {
+export function exportDOT(db: BetterSqlite3Database, opts: ExportOpts = {}): string {
   const fileLevel = opts.fileLevel !== false;
   const noTests = opts.noTests || false;
   const minConfidence = opts.minConfidence;
@@ -292,7 +288,7 @@ export function exportDOT(db: BetterSqlite3.Database, opts: ExportOpts = {}): st
  * Export the dependency graph in Mermaid format.
  */
 export function exportMermaid(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   opts: ExportOpts & { direction?: string } = {},
 ): string {
   const fileLevel = opts.fileLevel !== false;
@@ -332,7 +328,7 @@ export function exportMermaid(
  * Export as JSON adjacency list.
  */
 export function exportJSON(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   opts: ExportOpts = {},
 ): { nodes: unknown[]; edges: unknown[] } {
   const noTests = opts.noTests || false;
@@ -366,7 +362,7 @@ export function exportJSON(
 /**
  * Export the dependency graph in GraphML (XML) format.
  */
-export function exportGraphML(db: BetterSqlite3.Database, opts: ExportOpts = {}): string {
+export function exportGraphML(db: BetterSqlite3Database, opts: ExportOpts = {}): string {
   const fileLevel = opts.fileLevel !== false;
   const noTests = opts.noTests || false;
   const minConfidence = opts.minConfidence;
@@ -385,7 +381,7 @@ export function exportGraphML(db: BetterSqlite3.Database, opts: ExportOpts = {})
  * Export the dependency graph in TinkerPop GraphSON v3 format.
  */
 export function exportGraphSON(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   opts: ExportOpts = {},
 ): { vertices: unknown[]; edges: unknown[] } {
   const noTests = opts.noTests || false;
@@ -459,7 +455,7 @@ export function exportGraphSON(
  * Returns { nodes: string, relationships: string }.
  */
 export function exportNeo4jCSV(
-  db: BetterSqlite3.Database,
+  db: BetterSqlite3Database,
   opts: ExportOpts = {},
 ): ExportNeo4jCSVResult {
   const fileLevel = opts.fileLevel !== false;

package/src/features/flow.ts

@@ -9,7 +9,6 @@ import { openReadonlyOrFail } from '../db/index.js';
 import { CORE_SYMBOL_KINDS, findMatchingNodes } from '../domain/queries.js';
 import { isTestFile } from '../infrastructure/test-filter.js';
 import { paginateResult } from '../shared/paginate.js';
-import type { BetterSqlite3Database } from '../types.js';
 import { FRAMEWORK_ENTRY_PREFIXES } from './structure.js';
 
 export function entryPointType(name: string): 'route' | 'event' | 'command' | 'exported' | null {

package/src/features/graph-enrichment.ts

@@ -356,9 +356,7 @@ function prepareFileLevelData(
     const color: string =
       cfg.colorBy === 'community' && community !== null
         ? COMMUNITY_COLORS[community % COMMUNITY_COLORS.length] || '#ccc'
-        : cfg.nodeColors?.['file'] ||
-          (DEFAULT_NODE_COLORS as Record<string, string>)['file'] ||
-          '#ccc';
+        : cfg.nodeColors?.file || (DEFAULT_NODE_COLORS as Record<string, string>).file || '#ccc';
 
     return {
       id,