@arcbridge/core 0.1.2 → 0.1.3

package/dist/index.d.ts

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import Database from 'better-sqlite3';
+import { DatabaseSync } from 'node:sqlite';
 
 declare const ServiceSchema: z.ZodObject<{
     name: z.ZodString;
@@ -720,13 +720,24 @@ declare const AgentRoleSchema: z.ZodObject<{
 }>;
 type AgentRole = z.infer<typeof AgentRoleSchema>;
 
-declare function openDatabase(dbPath: string): Database.Database;
-declare function openMemoryDatabase(): Database.Database;
+/** Re-export DatabaseSync as Database for use across the codebase. */
+type Database = DatabaseSync;
+declare function suppressSqliteWarning(): void;
+declare function openDatabase(dbPath: string): Database;
+declare function openMemoryDatabase(): Database;
+/**
+ * Wrap a synchronous function in a SQLite transaction (BEGIN/COMMIT/ROLLBACK).
+ * Supports nesting via SAVEPOINTs — inner calls create savepoints
+ * instead of starting a new transaction.
+ *
+ * fn must be synchronous — passing an async function will throw.
+ */
+declare function transaction<T>(db: Database, fn: () => T): T;
 
 declare const CURRENT_SCHEMA_VERSION = 2;
-declare function initializeSchema(db: Database.Database): void;
+declare function initializeSchema(db: Database): void;
 
-declare function migrate(db: Database.Database): void;
+declare function migrate(db: Database): void;
 
 interface InitProjectInput {
     name: string;
@@ -751,14 +762,14 @@ declare function generatePlan(targetDir: string, input: InitProjectInput): void;
 declare function generateAgentRoles(targetDir: string, template?: string): AgentRole[];
 
 interface GenerateDatabaseResult {
-    db: Database.Database;
+    db: Database;
     warnings: string[];
 }
 /**
  * Re-read arc42 files from disk and update the database.
  * Uses INSERT OR REPLACE to pick up changes made since initial generation.
  */
-declare function refreshFromDocs(db: Database.Database, targetDir: string): string[];
+declare function refreshFromDocs(db: Database, targetDir: string): string[];
 declare function generateDatabase(targetDir: string, input: InitProjectInput): GenerateDatabaseResult;
 
 interface IndexerOptions {
@@ -818,7 +829,7 @@ declare function discoverDotnetServices(projectRoot: string): DotnetProjectInfo[
  * Scan the project root for package dependency manifests (package.json, .csproj)
  * and write discovered dependencies to the package_dependencies table.
  */
-declare function indexPackageDependencies(db: Database.Database, projectRoot: string, service?: string): number;
+declare function indexPackageDependencies(db: Database, projectRoot: string, service?: string): number;
 
 type ProjectLanguage = "typescript" | "csharp" | "auto";
 /**
@@ -831,7 +842,7 @@ declare function detectProjectLanguage(projectRoot: string): "typescript" | "csh
  * Index a project, auto-detecting the language unless explicitly specified.
  * Dispatches to the TypeScript or .NET indexer accordingly.
  */
-declare function indexProject(db: Database.Database, options: IndexerOptions): Promise<IndexResult>;
+declare function indexProject(db: Database, options: IndexerOptions): Promise<IndexResult>;
 
 type DriftKind = "undocumented_module" | "missing_module" | "dependency_violation" | "unlinked_test" | "stale_adr" | "new_dependency";
 type DriftSeverity = "info" | "warning" | "error";
@@ -853,12 +864,12 @@ interface DriftOptions {
  * Compares building block code_paths against actual indexed files,
  * checks cross-block dependencies, and validates ADR references.
  */
-declare function detectDrift(db: Database.Database, options?: DriftOptions): DriftEntry[];
+declare function detectDrift(db: Database, options?: DriftOptions): DriftEntry[];
 /**
  * Write drift entries to the drift_log table.
  * Clears existing unresolved entries and inserts fresh ones.
  */
-declare function writeDriftLog(db: Database.Database, entries: DriftEntry[]): void;
+declare function writeDriftLog(db: Database, entries: DriftEntry[]): void;
 
 interface TaskInferenceResult {
     taskId: string;
@@ -870,11 +881,11 @@ interface TaskInferenceResult {
  * Infer task status from code state.
  * Checks if building block code exists, acceptance criteria symbols are present, etc.
  */
-declare function inferTaskStatuses(db: Database.Database, phaseId: string): TaskInferenceResult[];
+declare function inferTaskStatuses(db: Database, phaseId: string): TaskInferenceResult[];
 /**
  * Apply inferred statuses to the database and optionally write back to YAML.
  */
-declare function applyInferences(db: Database.Database, inferences: TaskInferenceResult[], projectRoot: string): void;
+declare function applyInferences(db: Database, inferences: TaskInferenceResult[], projectRoot: string): void;
 
 /**
  * Update a task's status in the YAML task file.
@@ -998,10 +1009,10 @@ interface MetricsResult {
 }
 type ExportFormat = "json" | "csv" | "markdown";
 
-declare function insertActivity(db: Database.Database, params: InsertActivityParams): number;
-declare function getSessionTotals(db: Database.Database, since?: string, model?: string): SessionTotals;
-declare function queryMetrics(db: Database.Database, params: QueryMetricsParams): MetricsResult;
-declare function exportMetrics(db: Database.Database, projectRoot: string, format: ExportFormat, params: Omit<QueryMetricsParams, "groupBy" | "limit">, maxRows?: number): string;
+declare function insertActivity(db: Database, params: InsertActivityParams): number;
+declare function getSessionTotals(db: Database, since?: string, model?: string): SessionTotals;
+declare function queryMetrics(db: Database, params: QueryMetricsParams): MetricsResult;
+declare function exportMetrics(db: Database, projectRoot: string, format: ExportFormat, params: Omit<QueryMetricsParams, "groupBy" | "limit">, maxRows?: number): string;
 
 interface ChangedFile {
     status: "added" | "modified" | "deleted" | "renamed";
@@ -1020,7 +1031,7 @@ interface GitRef {
  * - "last-phase" → commit stored in arcbridge_meta under "phase_sync_commit", or HEAD~5 fallback
  * - anything else → treated as a literal git ref (branch, tag, SHA)
  */
-declare function resolveRef(projectRoot: string, since: string, db?: Database.Database): GitRef;
+declare function resolveRef(projectRoot: string, since: string, db?: Database): GitRef;
 /**
  * Get list of changed files between a ref and HEAD.
  */
@@ -1036,7 +1047,7 @@ declare function getHeadSha(projectRoot: string): string | null;
 /**
  * Store the current sync point in arcbridge_meta.
  */
-declare function setSyncCommit(db: Database.Database, key: "last_sync_commit" | "phase_sync_commit", sha: string): void;
+declare function setSyncCommit(db: Database, key: "last_sync_commit" | "phase_sync_commit", sha: string): void;
 
 type TestOutcome = "passed" | "failed" | "missing" | "error";
 interface ScenarioTestResult {
@@ -1062,7 +1073,7 @@ interface VerifyResult {
  *
  * Optionally filter by specific scenario IDs.
  */
-declare function verifyScenarios(db: Database.Database, projectRoot: string, options: {
+declare function verifyScenarios(db: Database, projectRoot: string, options: {
     testCommand: string;
     timeoutMs: number;
     scenarioIds?: string[];
@@ -1097,4 +1108,4 @@ declare function loadConfig(projectRoot: string): {
     error: string | null;
 };
 
-export { type ActivityRow, type AdrFrontmatter, AdrFrontmatterSchema, type AgentRole, AgentRoleSchema, type AggregatedRow, type ArcBridgeConfig, ArcBridgeConfigSchema, type BuildingBlock, BuildingBlockSchema, type BuildingBlocksFrontmatter, BuildingBlocksFrontmatterSchema, CURRENT_SCHEMA_VERSION, type ChangedFile, type DotnetProjectInfo, type DriftEntry, type DriftKind, type DriftOptions, type DriftSeverity, type ExportFormat, type ExtractedSymbol, type GenerateDatabaseResult, type GitRef, type IndexResult, type IndexerOptions, type InitProjectInput, type InsertActivityParams, type LatestQualitySnapshot, type LoadRolesResult, type MetricsResult, type Phase, PhaseSchema, type PhasesFile, PhasesFileSchema, type ProjectLanguage, QualityCategorySchema, QualityPrioritySchema, type QualityScenario, QualityScenarioSchema, QualityScenarioStatusSchema, type QualityScenariosFile, QualityScenariosFileSchema, type QueryMetricsParams, type ScenarioTestResult, type Service, type SessionTotals, type SymbolKind, type Task, type TaskFile, TaskFileSchema, type TaskInferenceResult, TaskSchema, type TestOutcome, type VerifyResult, addTaskToYaml, applyInferences, detectDrift, detectProjectLanguage, discoverDotnetServices, exportMetrics, generateAgentRoles, generateArc42, generateConfig, generateDatabase, generatePlan, generateSyncFiles, getChangedFiles, getHeadSha, getSessionTotals, getUncommittedChanges, indexPackageDependencies, indexProject, inferTaskStatuses, initializeSchema, insertActivity, loadConfig, loadRole, loadRoles, migrate, openDatabase, openMemoryDatabase, queryMetrics, refreshFromDocs, resolveRef, setSyncCommit, syncPhaseToYaml, syncScenarioToYaml, syncTaskToYaml, verifyScenarios, writeDriftLog };
+export { type ActivityRow, type AdrFrontmatter, AdrFrontmatterSchema, type AgentRole, AgentRoleSchema, type AggregatedRow, type ArcBridgeConfig, ArcBridgeConfigSchema, type BuildingBlock, BuildingBlockSchema, type BuildingBlocksFrontmatter, BuildingBlocksFrontmatterSchema, CURRENT_SCHEMA_VERSION, type ChangedFile, type Database, type DotnetProjectInfo, type DriftEntry, type DriftKind, type DriftOptions, type DriftSeverity, type ExportFormat, type ExtractedSymbol, type GenerateDatabaseResult, type GitRef, type IndexResult, type IndexerOptions, type InitProjectInput, type InsertActivityParams, type LatestQualitySnapshot, type LoadRolesResult, type MetricsResult, type Phase, PhaseSchema, type PhasesFile, PhasesFileSchema, type ProjectLanguage, QualityCategorySchema, QualityPrioritySchema, type QualityScenario, QualityScenarioSchema, QualityScenarioStatusSchema, type QualityScenariosFile, QualityScenariosFileSchema, type QueryMetricsParams, type ScenarioTestResult, type Service, type SessionTotals, type SymbolKind, type Task, type TaskFile, TaskFileSchema, type TaskInferenceResult, TaskSchema, type TestOutcome, type VerifyResult, addTaskToYaml, applyInferences, detectDrift, detectProjectLanguage, discoverDotnetServices, exportMetrics, generateAgentRoles, generateArc42, generateConfig, generateDatabase, generatePlan, generateSyncFiles, getChangedFiles, getHeadSha, getSessionTotals, getUncommittedChanges, indexPackageDependencies, indexProject, inferTaskStatuses, initializeSchema, insertActivity, loadConfig, loadRole, loadRoles, migrate, openDatabase, openMemoryDatabase, queryMetrics, refreshFromDocs, resolveRef, setSyncCommit, suppressSqliteWarning, syncPhaseToYaml, syncScenarioToYaml, syncTaskToYaml, transaction, verifyScenarios, writeDriftLog };

package/dist/index.js

@@ -210,15 +210,93 @@ var AgentRoleSchema = z6.object({
 });
 
 // src/db/connection.ts
-import Database from "better-sqlite3";
+import { DatabaseSync } from "node:sqlite";
+var suppressed = false;
+function suppressSqliteWarning() {
+  if (suppressed) return;
+  suppressed = true;
+  const origEmit = process.emit;
+  process.emit = function(event, ...args) {
+    if (event === "warning" && args[0]?.name === "ExperimentalWarning" && typeof args[0]?.message === "string" && args[0].message.includes("SQLite")) {
+      return true;
+    }
+    return origEmit.apply(process, [event, ...args]);
+  };
+}
+function sanitizeParams(params) {
+  return params.map((p) => p === void 0 ? null : p);
+}
 function openDatabase(dbPath) {
-  const db = new Database(dbPath);
-  db.pragma("journal_mode = WAL");
-  db.pragma("foreign_keys = ON");
+  const db = new DatabaseSync(dbPath);
+  db.exec("PRAGMA journal_mode = WAL");
+  db.exec("PRAGMA foreign_keys = ON");
+  patchPrepare(db);
   return db;
 }
 function openMemoryDatabase() {
-  return openDatabase(":memory:");
+  const db = new DatabaseSync(":memory:");
+  db.exec("PRAGMA foreign_keys = ON");
+  patchPrepare(db);
+  return db;
+}
+function patchPrepare(db) {
+  const originalPrepare = db.prepare.bind(db);
+  db.prepare = (sql) => {
+    const stmt = originalPrepare(sql);
+    const origRun = stmt.run.bind(stmt);
+    const origGet = stmt.get.bind(stmt);
+    const origAll = stmt.all.bind(stmt);
+    stmt.run = (...params) => origRun(...sanitizeParams(params));
+    stmt.get = (...params) => origGet(...sanitizeParams(params));
+    stmt.all = (...params) => origAll(...sanitizeParams(params));
+    return stmt;
+  };
+}
+function rejectAsync(fn) {
+  if (fn.constructor.name === "AsyncFunction") {
+    throw new Error(
+      "transaction() received an async function. Use a synchronous function \u2014 node:sqlite is synchronous and the transaction would commit before the async work completes."
+    );
+  }
+}
+var txDepth = /* @__PURE__ */ new WeakMap();
+function transaction(db, fn) {
+  rejectAsync(fn);
+  const depth = txDepth.get(db) ?? 0;
+  if (depth === 0) {
+    db.exec("BEGIN");
+    txDepth.set(db, 1);
+    try {
+      const result = fn();
+      db.exec("COMMIT");
+      return result;
+    } catch (err) {
+      try {
+        db.exec("ROLLBACK");
+      } catch {
+      }
+      throw err;
+    } finally {
+      txDepth.set(db, 0);
+    }
+  }
+  const name = `sp_${depth}`;
+  db.exec(`SAVEPOINT ${name}`);
+  txDepth.set(db, depth + 1);
+  try {
+    const result = fn();
+    db.exec(`RELEASE ${name}`);
+    return result;
+  } catch (err) {
+    try {
+      db.exec(`ROLLBACK TO ${name}`);
+      db.exec(`RELEASE ${name}`);
+    } catch {
+    }
+    throw err;
+  } finally {
+    txDepth.set(db, depth);
+  }
 }
 
 // src/db/schema.ts
@@ -477,12 +555,12 @@ function migrate(db) {
   }
   const pending = migrations.filter((m) => m.version > currentVersion).sort((a, b) => a.version - b.version);
   for (const migration of pending) {
-    db.transaction(() => {
+    transaction(db, () => {
       migration.up(db);
       db.prepare(
         "UPDATE arcbridge_meta SET value = ? WHERE key = 'schema_version'"
       ).run(String(migration.version));
-    })();
+    });
   }
 }
 
@@ -3275,7 +3353,7 @@ function refreshFromDocs(db, targetDir) {
   const scenarioStatusMap = new Map(
     existingScenarios.map((s) => [s.id, s.status])
   );
-  const refresh = db.transaction(() => {
+  const refresh = () => transaction(db, () => {
     db.prepare("DELETE FROM tasks").run();
     db.prepare("DELETE FROM phases").run();
     db.prepare("UPDATE contracts SET building_block = NULL").run();
@@ -3326,12 +3404,12 @@ function generateDatabase(targetDir, input) {
   upsert.run("project_name", input.name);
   upsert.run("project_type", input.template);
   upsert.run("last_full_index", (/* @__PURE__ */ new Date()).toISOString());
-  db.transaction(() => {
+  transaction(db, () => {
     allWarnings.push(...populateBuildingBlocks(db, targetDir));
     allWarnings.push(...populateQualityScenarios(db, targetDir));
     allWarnings.push(...populatePhases(db, targetDir));
     allWarnings.push(...populateAdrs(db, targetDir));
-  })();
+  });
   ensureGitignore(targetDir);
   return { db, warnings: allWarnings };
 }
@@ -4130,7 +4208,7 @@ function writeComponents(db, components) {
   const existingIds = new Set(
     db.prepare("SELECT id FROM symbols").all().map((r) => r.id)
   );
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const c of components) {
       if (!existingIds.has(c.symbolId)) continue;
       insert.run(
@@ -4144,7 +4222,6 @@ function writeComponents(db, components) {
       );
     }
   });
-  run();
 }
 
 // src/indexer/route-analyzer.ts
@@ -4273,7 +4350,7 @@ function writeRoutes(db, routes) {
       id, route_path, kind, http_methods, has_auth, parent_layout, service
     ) VALUES (?, ?, ?, ?, ?, ?, ?)
   `);
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const r of routes) {
       insert.run(
         r.id,
@@ -4286,7 +4363,6 @@ function writeRoutes(db, routes) {
       );
     }
   });
-  run();
 }
 
 // src/indexer/content-hash.ts
@@ -4320,7 +4396,7 @@ function removeSymbolsForFiles(db, filePaths) {
   const deleteComponents = db.prepare(
     "DELETE FROM components WHERE symbol_id IN (SELECT id FROM symbols WHERE file_path = ?)"
   );
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const fp of filePaths) {
       deleteDepsSource.run(fp);
       deleteDepsTarget.run(fp);
@@ -4328,7 +4404,6 @@ function removeSymbolsForFiles(db, filePaths) {
       deleteSymbols.run(fp);
     }
   });
-  run();
 }
 function writeSymbols(db, symbols, service, language = "typescript") {
   if (symbols.length === 0) return;
@@ -4348,7 +4423,7 @@ function writeSymbols(db, symbols, service, language = "typescript") {
     )
   `);
   const now = (/* @__PURE__ */ new Date()).toISOString();
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const s of symbols) {
       insert.run(
         s.id,
@@ -4372,7 +4447,6 @@ function writeSymbols(db, symbols, service, language = "typescript") {
       );
     }
   });
-  run();
 }
 function writeDependencies(db, dependencies) {
   if (dependencies.length === 0) return;
@@ -4380,12 +4454,11 @@ function writeDependencies(db, dependencies) {
     INSERT OR IGNORE INTO dependencies (source_symbol, target_symbol, kind)
     VALUES (?, ?, ?)
   `);
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const dep of dependencies) {
       insert.run(dep.sourceSymbolId, dep.targetSymbolId, dep.kind);
     }
   });
-  run();
 }
 
 // src/indexer/dotnet-indexer.ts
@@ -4582,7 +4655,7 @@ function indexDotnetProjectRoslyn(db, options) {
       INSERT OR REPLACE INTO routes (id, route_path, kind, http_methods, has_auth, service)
       VALUES (?, ?, ?, ?, ?, ?)
     `);
-    const runRoutes = db.transaction(() => {
+    transaction(db, () => {
       for (const route of output.routes) {
         insertRoute.run(
           route.id,
@@ -4594,7 +4667,6 @@ function indexDotnetProjectRoslyn(db, options) {
         );
       }
     });
-    runRoutes();
   }
   return {
     symbolsIndexed: output.symbols.length,
@@ -5513,7 +5585,7 @@ async function indexCSharpTreeSitter(db, options) {
       INSERT OR REPLACE INTO routes (id, route_path, kind, http_methods, has_auth, service)
       VALUES (?, ?, ?, ?, ?, ?)
     `);
-    const runRoutes = db.transaction(() => {
+    transaction(db, () => {
       for (const route of allRoutes) {
         insertRoute.run(
           route.id,
@@ -5525,7 +5597,6 @@ async function indexCSharpTreeSitter(db, options) {
         );
       }
     });
-    runRoutes();
   }
   return {
     symbolsIndexed: allNewSymbols.length,
@@ -5557,12 +5628,11 @@ function indexPackageDependencies(db, projectRoot, service = "main") {
   const insert = db.prepare(
     "INSERT OR IGNORE INTO package_dependencies (name, version, source, service) VALUES (?, ?, ?, ?)"
   );
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const dep of deps) {
       insert.run(dep.name, dep.version, dep.source, service);
     }
   });
-  run();
   return deps.length;
 }
 function parsePackageJson(filePath) {
@@ -5815,12 +5885,11 @@ function writeDriftLog(db, entries) {
     VALUES (?, ?, ?, ?, ?, ?)
   `);
   const now = (/* @__PURE__ */ new Date()).toISOString();
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const e of entries) {
       insert.run(now, e.kind, e.severity, e.description, e.affectedBlock, e.affectedFile);
     }
   });
-  run();
 }
 function detectUndocumentedModules(db, entries, ignorePaths = []) {
   const blocks = db.prepare("SELECT id, name, code_paths FROM building_blocks").all();
@@ -6139,12 +6208,11 @@ function applyInferences(db, inferences, projectRoot) {
     "UPDATE tasks SET status = ?, completed_at = CASE WHEN ? = 'done' THEN ? ELSE completed_at END WHERE id = ?"
   );
   const now = (/* @__PURE__ */ new Date()).toISOString();
-  const run = db.transaction(() => {
+  transaction(db, () => {
     for (const inf of inferences) {
       update.run(inf.inferredStatus, inf.inferredStatus, now, inf.taskId);
     }
   });
-  run();
   for (const inf of inferences) {
     const task = db.prepare("SELECT phase_id FROM tasks WHERE id = ?").get(inf.taskId);
     if (task) {
@@ -7086,9 +7154,11 @@ export {
   refreshFromDocs,
   resolveRef,
   setSyncCommit,
+  suppressSqliteWarning,
   syncPhaseToYaml,
   syncScenarioToYaml,
   syncTaskToYaml,
+  transaction,
   verifyScenarios,
   writeDriftLog
 };