feat: type analysis modules, parser WASM maps, extract canStripTypes (#554 follow-up)

src/db/index.js

@@ -56,6 +56,8 @@ export {
   getFileNodesAll,
   getFunctionNodeId,
   getImportEdges,
+  getLineCountForNode,
+  getMaxEndLineForFile,
   getNodeId,
   hasCfgTables,
   hasCoChanges,

src/db/repository/index.ts

@@ -42,6 +42,8 @@ export {
   findNodesForTriage,
   findNodesWithFanIn,
   getFunctionNodeId,
+  getLineCountForNode,
+  getMaxEndLineForFile,
   getNodeId,
   iterateFunctionNodes,
   listFunctionNodes,

src/db/repository/nodes.ts

@@ -275,3 +275,37 @@ export function findNodeByQualifiedName(
     'SELECT * FROM nodes WHERE qualified_name = ? ORDER BY file, line',
   ).all(qualifiedName);
 }
+
+// ─── Metric helpers ──────────────────────────────────────────────────────
+
+const _getLineCountForNodeStmt = new WeakMap();
+
+/**
+ * Get line_count from node_metrics for a given node.
+ * @param {object} db
+ * @param {number} nodeId
+ * @returns {{ line_count: number } | undefined}
+ */
+export function getLineCountForNode(db: BetterSqlite3Database, nodeId: number) {
+  return cachedStmt(
+    _getLineCountForNodeStmt,
+    db,
+    'SELECT line_count FROM node_metrics WHERE node_id = ?',
+  ).get(nodeId);
+}
+
+const _getMaxEndLineForFileStmt = new WeakMap();
+
+/**
+ * Get the maximum end_line across all nodes in a file.
+ * @param {object} db
+ * @param {string} file
+ * @returns {{ max_end: number | null } | undefined}
+ */
+export function getMaxEndLineForFile(db: BetterSqlite3Database, file: string) {
+  return cachedStmt(
+    _getMaxEndLineForFileStmt,
+    db,
+    'SELECT MAX(end_line) as max_end FROM nodes WHERE file = ?',
+  ).get(file);
+}

src/domain/analysis/brief.ts

@@ -9,6 +9,7 @@ import {
 } from '../../db/index.js';
 import { loadConfig } from '../../infrastructure/config.js';
 import { isTestFile } from '../../infrastructure/test-filter.js';
+import type { BetterSqlite3Database, ImportEdgeRow, NodeRow, RelatedNodeRow } from '../../types.js';
 
 /** Symbol kinds meaningful for a file brief — excludes parameters, properties, constants. */
 const BRIEF_KINDS = new Set([
@@ -47,15 +48,19 @@ function computeRiskTier(
  * BFS to count transitive callers for a single node.
  * Lightweight variant — only counts, does not collect details.
  */
-// biome-ignore lint/suspicious/noExplicitAny: db handle from better-sqlite3
-function countTransitiveCallers(db: any, startId: number, noTests: boolean, maxDepth = 5): number {
+function countTransitiveCallers(
+  db: BetterSqlite3Database,
+  startId: number,
+  noTests: boolean,
+  maxDepth = 5,
+): number {
   const visited = new Set([startId]);
   let frontier = [startId];
 
   for (let d = 1; d <= maxDepth; d++) {
     const nextFrontier: number[] = [];
     for (const fid of frontier) {
-      const callers = findDistinctCallers(db, fid);
+      const callers = findDistinctCallers(db, fid) as RelatedNodeRow[];
       for (const c of callers) {
         if (!visited.has(c.id) && (!noTests || !isTestFile(c.file))) {
           visited.add(c.id);
@@ -75,8 +80,7 @@ function countTransitiveCallers(db: any, startId: number, noTests: boolean, maxD
  * Depth-bounded to match countTransitiveCallers and keep hook latency predictable.
  */
 function countTransitiveImporters(
-  // biome-ignore lint/suspicious/noExplicitAny: db handle from better-sqlite3
-  db: any,
+  db: BetterSqlite3Database,
   fileNodeIds: number[],
   noTests: boolean,
   maxDepth = 5,
@@ -87,7 +91,7 @@ function countTransitiveImporters(
   for (let d = 1; d <= maxDepth; d++) {
     const nextFrontier: number[] = [];
     for (const current of frontier) {
-      const dependents = findImportDependents(db, current);
+      const dependents = findImportDependents(db, current) as RelatedNodeRow[];
       for (const dep of dependents) {
         if (!visited.has(dep.id) && (!noTests || !isTestFile(dep.file))) {
           visited.add(dep.id);
@@ -108,10 +112,10 @@ function countTransitiveImporters(
  */
 export function briefData(
   file: string,
-  customDbPath: string | undefined,
-  // biome-ignore lint/suspicious/noExplicitAny: config shape not yet typed
+  customDbPath: string,
+  // biome-ignore lint/suspicious/noExplicitAny: config shape is dynamic
   opts: { noTests?: boolean; config?: any } = {},
-): object {
+) {
   const db = openReadonlyOrFail(customDbPath);
   try {
     const noTests = opts.noTests || false;
@@ -120,26 +124,28 @@ export function briefData(
     const importerDepth = config.analysis?.briefImporterDepth ?? 5;
     const highRiskCallers = config.analysis?.briefHighRiskCallers ?? 10;
     const mediumRiskCallers = config.analysis?.briefMediumRiskCallers ?? 3;
-    const fileNodes = findFileNodes(db, `%${file}%`);
+    const fileNodes = findFileNodes(db, `%${file}%`) as NodeRow[];
     if (fileNodes.length === 0) {
       return { file, results: [] };
     }
 
     const results = fileNodes.map((fn) => {
       // Direct importers
-      let importedBy = findImportSources(db, fn.id);
+      let importedBy = findImportSources(db, fn.id) as ImportEdgeRow[];
       if (noTests) importedBy = importedBy.filter((i) => !isTestFile(i.file));
       const directImporters = [...new Set(importedBy.map((i) => i.file))];
 
       // Transitive importer count
       const totalImporterCount = countTransitiveImporters(db, [fn.id], noTests, importerDepth);
 
       // Direct imports
-      let importsTo = findImportTargets(db, fn.id);
+      let importsTo = findImportTargets(db, fn.id) as ImportEdgeRow[];
       if (noTests) importsTo = importsTo.filter((i) => !isTestFile(i.file));
 
       // Symbol definitions with roles and caller counts
-      const defs = findNodesByFile(db, fn.file).filter((d) => BRIEF_KINDS.has(d.kind));
+      const defs = (findNodesByFile(db, fn.file) as NodeRow[]).filter((d) =>
+        BRIEF_KINDS.has(d.kind),
+      );
       const symbols = defs.map((d) => {
         const callerCount = countTransitiveCallers(db, d.id, noTests, callerDepth);
         return {