@optave/codegraph 3.9.1 → 3.9.2

package/src/features/audit.ts

@@ -7,6 +7,7 @@ import { loadConfig } from '../infrastructure/config.js';
 import { debug } from '../infrastructure/logger.js';
 import { isTestFile } from '../infrastructure/test-filter.js';
 import { toErrorMessage } from '../shared/errors.js';
+import { toSymbolRef } from '../shared/normalize.js';
 import type { BetterSqlite3Database, CodegraphConfig } from '../types.js';
 import { RULE_DEFS } from './manifesto.js';
 
@@ -317,7 +318,7 @@ function enrichSymbol(
          WHERE e.source_id = ? AND e.kind = 'calls'`,
         )
         .all(nodeId) as SymbolRef[]
-    ).map((c) => ({ name: c.name, kind: c.kind, file: c.file, line: c.line }));
+    ).map(toSymbolRef);
 
     callers = (
       db
@@ -327,7 +328,7 @@ function enrichSymbol(
          WHERE e.target_id = ? AND e.kind = 'calls'`,
         )
         .all(nodeId) as SymbolRef[]
-    ).map((c) => ({ name: c.name, kind: c.kind, file: c.file, line: c.line }));
+    ).map(toSymbolRef);
     if (noTests) callers = callers.filter((c) => !isTestFile(c.file));
 
     const testCallerRows = db

package/src/features/boundaries.ts

@@ -1,5 +1,5 @@
-import { debug } from '../infrastructure/logger.js';
 import { isTestFile } from '../infrastructure/test-filter.js';
+import { BoundaryError } from '../shared/errors.js';
 import type { BetterSqlite3Database } from '../types.js';
 
 // ─── Glob-to-Regex ───────────────────────────────────────────────────
@@ -269,8 +269,7 @@ export function evaluateBoundaries(
 
   const { valid, errors } = validateBoundaryConfig(boundaryConfig);
   if (!valid) {
-    debug(`boundary config validation failed: ${errors.join('; ')}`);
-    return { violations: [], violationCount: 0 };
+    throw new BoundaryError(`Invalid boundary configuration: ${errors.join('; ')}`);
   }
 
   const modules = resolveModules(boundaryConfig);
@@ -297,8 +296,7 @@ export function evaluateBoundaries(
       )
       .all() as Array<{ source: string; target: string }>;
   } catch (err) {
-    debug(`boundary edge query failed: ${(err as Error).message}`);
-    return { violations: [], violationCount: 0 };
+    throw new BoundaryError('Boundary evaluation failed', { cause: err as Error });
   }
 
   if (opts.noTests) {

package/src/features/branch-compare.ts

@@ -9,6 +9,7 @@ import { debug } from '../infrastructure/logger.js';
 import { getNative, isNativeAvailable } from '../infrastructure/native.js';
 import { isTestFile } from '../infrastructure/test-filter.js';
 import { toErrorMessage } from '../shared/errors.js';
+import { toSymbolRef } from '../shared/normalize.js';
 import type { EngineMode, NativeDatabase } from '../types.js';
 
 // ─── Git Helpers ────────────────────────────────────────────────────────
@@ -255,9 +256,7 @@ function loadCallersFromDb(
             if (!visited.has(c.id) && (!noTests || !isTestFile(c.file))) {
               visited.add(c.id);
               nextFrontier.push(c.id);
-              allCallers.add(
-                JSON.stringify({ name: c.name, kind: c.kind, file: c.file, line: c.line }),
-              );
+              allCallers.add(JSON.stringify(toSymbolRef(c)));
             }
           }
         }

package/src/features/flow.ts

@@ -8,6 +8,7 @@
 import { openReadonlyOrFail } from '../db/index.js';
 import { CORE_SYMBOL_KINDS, findMatchingNodes } from '../domain/queries.js';
 import { isTestFile } from '../infrastructure/test-filter.js';
+import { toSymbolRef } from '../shared/normalize.js';
 import { paginateResult } from '../shared/paginate.js';
 import { FRAMEWORK_ENTRY_PREFIXES } from './structure.js';
 
@@ -175,7 +176,7 @@ function bfsCallees(
 
         visited.add(c.id);
         nextFrontier.push(c.id);
-        const nodeInfo: NodeInfo = { name: c.name, kind: c.kind, file: c.file, line: c.line };
+        const nodeInfo: NodeInfo = toSymbolRef(c);
         levelNodes.push(nodeInfo);
         nodeDepths.set(c.id, d);
         idToNode.set(c.id, nodeInfo);

package/src/features/manifesto.ts

@@ -5,6 +5,7 @@ import { loadConfig } from '../infrastructure/config.js';
 import { debug } from '../infrastructure/logger.js';
 import { paginateResult } from '../shared/paginate.js';
 import type { BetterSqlite3Database, CodegraphConfig, ThresholdRule } from '../types.js';
+import type { BoundaryViolation } from './boundaries.js';
 import { evaluateBoundaries } from './boundaries.js';
 
 // ─── Rule Definitions ─────────────────────────────────────────────────
@@ -416,7 +417,20 @@ function evaluateBoundaryRules(
     return;
   }
 
-  const result = evaluateBoundaries(db, boundaryConfig, { noTests: opts.noTests || false });
+  let result: { violations: BoundaryViolation[]; violationCount: number };
+  try {
+    result = evaluateBoundaries(db, boundaryConfig, { noTests: opts.noTests || false });
+  } catch (e: unknown) {
+    debug(`boundary check failed: ${(e as Error).message}`);
+    ruleResults.push({
+      name: 'boundaries',
+      level: 'graph',
+      status: 'warn',
+      thresholds: effectiveThresholds,
+      violationCount: 0,
+    });
+    return;
+  }
   const hasBoundaryViolations = result.violationCount > 0;
 
   if (!hasBoundaryViolations) {

package/src/infrastructure/config.ts

@@ -23,6 +23,7 @@ export const DEFAULTS = {
     incremental: true,
     dbPath: '.codegraph/graph.db',
     driftThreshold: 0.2,
+    smallFilesThreshold: 5,
   },
   query: {
     defaultDepth: 3,

package/src/presentation/batch.ts

@@ -36,6 +36,7 @@ export function batchQuery(
   const { command: defaultCommand = 'where', ...rest } = opts;
   const isMulti =
     targets.length > 0 &&
+    targets[0] !== null &&
     typeof targets[0] === 'object' &&
     !!(targets[0] as MultiBatchTarget).command;
 

package/src/presentation/structure.ts

@@ -75,7 +75,7 @@ export function formatHotspots(data: HotspotsResult): string {
 
 interface ModuleBoundaryEntry {
   directory: string;
-  cohesion: number;
+  cohesion: number | null;
   fileCount: number;
   symbolCount: number;
   fanIn: number;
@@ -95,7 +95,7 @@ export function formatModuleBoundaries(data: ModuleBoundariesResult): string {
   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)`,
+      `  ${m.directory}/  cohesion=${m.cohesion !== null ? m.cohesion.toFixed(2) : 'n/a'}  (${m.fileCount} files, ${m.symbolCount} symbols)`,
     );
     lines.push(`    Incoming: ${m.fanIn} edges    Outgoing: ${m.fanOut} edges`);
     if (m.files.length > 0) {

package/src/shared/normalize.ts

@@ -19,6 +19,16 @@ export function getFileHash(db: DbHandle, file: string): string | null {
   return row ? row.hash : null;
 }
 
+/** Pick the 4-field symbol reference from any row that carries at least {name, kind, file, line}. */
+export function toSymbolRef(row: { name: string; kind: string; file: string; line: number }): {
+  name: string;
+  kind: string;
+  file: string;
+  line: number;
+} {
+  return { name: row.name, kind: row.kind, file: row.file, line: row.line };
+}
+
 export function kindIcon(kind: string): string {
   switch (kind) {
     case 'function':

package/src/types.ts

@@ -1085,6 +1085,7 @@ export interface CodegraphConfig {
     incremental: boolean;
     dbPath: string;
     driftThreshold: number;
+    smallFilesThreshold: number;
   };
 
   query: {