@jungjaehoon/mama-core 2.4.1 → 4.1.0

package/dist/db-manager.js

@@ -2,10 +2,9 @@
 /**
  * MAMA Database Manager (SQLite-only)
  *
- * SQLite-exclusive database interface for MAMA Plugin.
- * Uses better-sqlite3 for local storage.
+ * SQLite-exclusive database interface. Uses better-sqlite3 for local storage.
  *
- * PostgreSQL support is only available in the legacy mcp-server repository.
+ * There is no PostgreSQL path.
  *
  * Features:
  * - WAL mode for better concurrency
@@ -18,43 +17,36 @@
  * @date 2026-02-01
  * @source-of-truth packages/mama-core/src/db-manager.js (mama-core)
  */
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.openDatabase = exports.isTestMode = exports.declareProductionDatabasePath = exports.clearProductionDatabasePaths = exports.assertTestProcessIsNotUsingRealDb = void 0;
 exports.assertEmbeddingSchemeCurrent = assertEmbeddingSchemeCurrent;
 exports.initDB = initDB;
-exports.assertTestProcessIsNotUsingRealDb = assertTestProcessIsNotUsingRealDb;
 exports.getDB = getDB;
 exports.getAdapter = getAdapter;
+exports.buildMemoryScopeId = buildMemoryScopeId;
 exports.closeDB = closeDB;
 exports.ensureMemoryScope = ensureMemoryScope;
-exports.insertEmbedding = insertEmbedding;
-exports.vectorSearch = vectorSearch;
-exports.insertDecisionWithEmbedding = insertDecisionWithEmbedding;
-exports.queryDecisionGraph = queryDecisionGraph;
-exports.querySemanticEdges = querySemanticEdges;
-exports.queryVectorSearch = queryVectorSearch;
+exports.prepareDecisionEmbedding = prepareDecisionEmbedding;
+exports.insertPreparedDecision = insertPreparedDecision;
 exports.updateDecisionOutcome = updateDecisionOutcome;
-exports.getPreparedStmt = getPreparedStmt;
 exports.getDbPath = getDbPath;
 exports.resetDBState = resetDBState;
-exports.isTestMode = isTestMode;
-exports.fts5Search = fts5Search;
-const path_1 = __importDefault(require("path"));
-const os_1 = __importDefault(require("os"));
 const debug_logger_js_1 = require("./debug-logger.js");
-const progress_indicator_js_1 = require("./progress-indicator.js");
-const index_js_1 = require("./db-adapter/index.js");
-const embeddings_js_1 = require("./embeddings.js");
-// Database adapter instance (singleton)
-let dbAdapter = null;
-let dbConnection = null;
-let isInitialized = false;
-let initializingPromise = null; // Single-flight guard for concurrent callers
+const database_js_1 = require("./storage/database.js");
+var database_js_2 = require("./storage/database.js");
+Object.defineProperty(exports, "assertTestProcessIsNotUsingRealDb", { enumerable: true, get: function () { return database_js_2.assertTestProcessIsNotUsingRealDb; } });
+Object.defineProperty(exports, "clearProductionDatabasePaths", { enumerable: true, get: function () { return database_js_2.clearProductionDatabasePaths; } });
+Object.defineProperty(exports, "declareProductionDatabasePath", { enumerable: true, get: function () { return database_js_2.declareProductionDatabasePath; } });
+Object.defineProperty(exports, "isTestMode", { enumerable: true, get: function () { return database_js_2.isTestMode; } });
+var database_js_3 = require("./storage/database.js");
+Object.defineProperty(exports, "openDatabase", { enumerable: true, get: function () { return database_js_3.openDatabase; } });
+const embedder_js_1 = require("./embedding/embedder.js");
+// One process-wide handle for callers that have not been given a database of
+// their own yet. `openDatabase` owns the lifetime; this module only holds the
+// handle and hands its adapter out.
+let handle = null;
+let openingPromise = null; // Single-flight guard for concurrent callers
 // Migration directory (moved to src/db/migrations for M1.2)
-const MIGRATIONS_DIR = path_1.default.join(__dirname, '..', 'db', 'migrations');
-const REAL_USER_DB_PATH = path_1.default.join(os_1.default.homedir(), '.claude', 'mama-memory.db');
 /**
  * Initialize SQLite database adapter and connect
  *
@@ -96,13 +88,13 @@ function assertEmbeddingSchemeCurrent(adapter) {
         // embedding_meta missing (pre-042). runMigrations creates it, so this path
         // only hits if migrations did not run; fall through to the vector check.
     }
-    if (scheme === embeddings_js_1.EMBEDDING_PREFIX_SCHEME)
+    if (scheme === embedder_js_1.EMBEDDING_PREFIX_SCHEME)
         return; // current
     const legacyVectors = countRows(adapter, 'embeddings') + countRows(adapter, 'wiki_page_embeddings');
     if (legacyVectors > 0) {
-        const dbPath = resolveAdapterDbPath(adapter) ?? '<db>';
+        const dbPath = (0, database_js_1.resolveAdapterDbPath)(adapter) ?? '<db>';
         throw new Error(`[embedding-guard] ${dbPath} holds ${legacyVectors} vectors written WITHOUT the e5 ` +
-            `query/passage prefix (scheme='${scheme}', code requires '${embeddings_js_1.EMBEDDING_PREFIX_SCHEME}'). ` +
+            `query/passage prefix (scheme='${scheme}', code requires '${embedder_js_1.EMBEDDING_PREFIX_SCHEME}'). ` +
             `Cosine search would be non-discriminative. Re-embed before starting:\n` +
             `  MAMA_DB_PATH="${dbPath}" node packages/mama-core/scripts/re-embed-migration.mjs`);
     }
@@ -110,112 +102,48 @@ function assertEmbeddingSchemeCurrent(adapter) {
     try {
         adapter
             .prepare("INSERT OR REPLACE INTO embedding_meta (key, value) VALUES ('embedding_prefix_scheme', ?)")
-            .run(embeddings_js_1.EMBEDDING_PREFIX_SCHEME);
+            .run(embedder_js_1.EMBEDDING_PREFIX_SCHEME);
     }
     catch {
         // best-effort; if embedding_meta truly missing, nothing to guard yet
     }
 }
-async function initDB() {
-    assertTestProcessIsNotUsingRealDb();
-    // Already initialized - return immediately
-    if (isInitialized) {
-        return dbConnection;
-    }
-    // Single-flight guard: If initialization is in progress, wait for it
-    if (initializingPromise) {
-        return initializingPromise;
-    }
-    // Start initialization and store promise for concurrent callers
-    initializingPromise = (async () => {
-        try {
-            (0, progress_indicator_js_1.logSearching)('Initializing database...');
-            // Create SQLite adapter
-            dbAdapter = (0, index_js_1.createAdapter)();
-            assertTestProcessIsNotUsingRealDb(resolveAdapterDbPath(dbAdapter), 'adapter.getDbPath()');
-            // Connect to database
-            dbConnection = await dbAdapter.connect();
-            // Run migrations (includes 012-create-checkpoints-table.sql)
-            await dbAdapter.runMigrations(MIGRATIONS_DIR);
-            // Reload vector cache after migrations (new tables/rows may now exist)
-            if (typeof dbAdapter.reloadVectorCache === 'function') {
-                dbAdapter.reloadVectorCache();
-            }
-            assertEmbeddingSchemeCurrent(dbAdapter); // fail loud on legacy vectors + new code
-            isInitialized = true;
-            (0, debug_logger_js_1.info)(`[db-manager] Database initialized (${dbAdapter.constructor.name})`);
-            (0, progress_indicator_js_1.logComplete)('Database ready');
-            return dbConnection;
-        }
-        catch (error) {
-            // Clear state on failure so retry is possible
-            initializingPromise = null;
-            dbAdapter = null;
-            dbConnection = null;
-            isInitialized = false;
-            const message = error instanceof Error ? error.message : String(error);
-            throw new Error(`Failed to initialize database: ${message}`);
-        }
-    })();
-    return initializingPromise;
-}
-function assertTestProcessIsNotUsingRealDb(effectivePath, effectivePathSource = 'adapter') {
-    if (!isDatabaseBoundaryTestMode()) {
-        return;
-    }
-    const configuredPaths = [
-        { name: 'MAMA_DB_PATH', value: process.env.MAMA_DB_PATH },
-        { name: 'MAMA_DATABASE_PATH', value: process.env.MAMA_DATABASE_PATH },
-    ];
-    if (effectivePath) {
-        configuredPaths.push({ name: effectivePathSource, value: effectivePath });
-    }
-    for (const configuredPath of configuredPaths) {
-        if (!configuredPath.value) {
-            continue;
-        }
-        const resolvedPath = path_1.default.resolve(expandHomePath(configuredPath.value));
-        if (resolvedPath === path_1.default.resolve(REAL_USER_DB_PATH)) {
-            throw new Error(`[db-boundary] Refusing to initDB against real DB ${REAL_USER_DB_PATH} ` +
-                `from a test process (${configuredPath.name}=${configuredPath.value}). ` +
-                'Set MAMA_DB_PATH to a temporary path before initDB.');
-        }
-    }
-}
-function resolveAdapterDbPath(adapter) {
-    if (typeof adapter.getDbPath === 'function') {
-        return adapter.getDbPath();
-    }
-    return adapter.dbPath;
-}
-function isDatabaseBoundaryTestMode() {
-    return isTestMode();
-}
-function expandHomePath(value) {
-    const home = os_1.default.homedir();
-    if (value === '~') {
-        return home;
-    }
-    if (value.startsWith('~/')) {
-        return path_1.default.join(home, value.slice(2));
-    }
-    return value.replaceAll('${HOME}', home);
+async function initDB(options = {}) {
+    if (handle) {
+        return handle.connection;
+    }
+    if (openingPromise) {
+        return (await openingPromise).connection;
+    }
+    // The handle is stored inside the chain, before `openingPromise` is cleared.
+    // Assigning it after the await left one microtask in which both guards read
+    // null, and a concurrent caller landing there would open a second adapter and
+    // run migrations again on the same file.
+    openingPromise = (0, database_js_1.openDatabase)(options.migrations ? { migrations: options.migrations } : {})
+        .then((opened) => {
+        handle = opened;
+        return opened;
+    })
+        .finally(() => {
+        openingPromise = null;
+    });
+    return (await openingPromise).connection;
 }
 /**
  * Get database connection (singleton pattern)
  *
  * Returns better-sqlite3 Database instance
  *
- * Note: Synchronous for backward compatibility with memory-store.js
+ * Note: Synchronous because callers assume a resolved adapter, not a promise.
  * Will throw if database not initialized
  *
  * @returns SQLite database connection
  */
 function getDB() {
-    if (!dbConnection) {
+    if (!handle) {
         throw new Error('Database not initialized. Call await initDB() first.');
     }
-    return dbConnection;
+    return handle.connection;
 }
 /**
  * Get database adapter instance
@@ -225,10 +153,10 @@ function getDB() {
  * @returns Adapter instance
  */
 function getAdapter() {
-    if (!dbAdapter) {
+    if (!handle) {
         throw new Error('Database adapter not initialized. Call await initDB() first.');
     }
-    return dbAdapter;
+    return handle.adapter;
 }
 function buildMemoryScopeId(kind, externalId) {
     return `scope_${kind}_${Buffer.from(externalId).toString('base64url')}`;
@@ -239,17 +167,19 @@ function buildMemoryScopeId(kind, externalId) {
  * Call this on process exit
  */
 async function closeDB() {
-    if (dbAdapter) {
-        await dbAdapter.disconnect();
-        dbAdapter = null;
-        dbConnection = null;
-        isInitialized = false;
-        initializingPromise = null; // Clear to allow re-initialization
+    if (handle) {
+        await handle.close();
+        handle = null;
+        openingPromise = null; // Clear to allow re-initialization
         (0, debug_logger_js_1.info)('[db-manager] Database connection closed');
     }
 }
-async function ensureMemoryScope(kind, externalId) {
-    const adapter = getAdapter();
+/**
+ * The single write boundary for memory_scopes. Callers inside a transaction
+ * pass their adapter through; the public wrapper resolves the global adapter
+ * for the call sites that predate the adapter-taking boundary.
+ */
+function ensureMemoryScope(adapter, kind, externalId) {
     const id = buildMemoryScopeId(kind, externalId);
     adapter
         .prepare(`
@@ -259,388 +189,54 @@ async function ensureMemoryScope(kind, externalId) {
         .run(id, kind, externalId);
     return id;
 }
-/**
- * Insert embedding into vector search table
- *
- * Stores embedding in the embeddings table for vector similarity search
- *
- * @param decisionRowid - SQLite rowid
- * @param embedding - 1024-dim embedding vector
- */
-async function insertEmbedding(decisionRowid, embedding) {
-    const adapter = getAdapter();
-    try {
-        adapter.insertEmbedding(decisionRowid, embedding);
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        // Graceful degradation: Log warning but don't fail
-        (0, debug_logger_js_1.error)(`[db-manager] Failed to insert embedding (vector search unavailable): ${message}`);
-    }
-}
-/**
- * Perform vector similarity search
- *
- * Returns empty array if vector search not available (no keyword fallback)
- *
- * @param queryEmbedding - Query embedding (1024-dim)
- * @param limit - Max results to return (default: 5)
- * @param threshold - Minimum similarity threshold (default: 0.7)
- * @returns Array of decisions with similarity scores, or empty array
- */
-async function vectorSearch(queryEmbedding, limit = 5, threshold = 0.7, topicPrefix, excludeStatuses) {
-    const adapter = getAdapter();
-    try {
-        // Brute-force cosine similarity over all embeddings (with optional topic/status pre-filter)
-        const results = await adapter.vectorSearch(queryEmbedding, limit * 3, topicPrefix, excludeStatuses);
-        if (!results || results.length === 0) {
-            return []; // No keyword fallback - fast fail
-        }
-        const stmt = adapter.prepare(`SELECT * FROM decisions WHERE rowid = ?`);
-        const decisions = [];
-        for (const row of results) {
-            const decision = stmt.get(row.rowid);
-            if (!decision) {
-                continue;
-            }
-            const similarity = row.similarity ?? Math.max(0, 1.0 - (row.distance ?? 1));
-            const distance = row.distance ?? Math.max(0, 1.0 - similarity);
-            if (similarity >= threshold) {
-                decisions.push({
-                    ...decision,
-                    distance,
-                    similarity,
-                });
-            }
-            if (decisions.length >= limit) {
-                break;
-            }
-        }
-        return decisions;
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        (0, debug_logger_js_1.error)(`[db-manager] Vector search failed: ${message}`);
-        return []; // No keyword fallback - fast fail
-    }
-}
-/**
- * Insert decision with embedding
- *
- * Combined operation: Insert decision + Generate embedding + Insert embedding
- * SQLite-only implementation
- *
- * @param decision - Decision object
- * @returns Decision ID
- */
-async function insertDecisionWithEmbedding(decision) {
-    const adapter = getAdapter();
-    const { generateEnhancedEmbedding } = await import('./embeddings.js');
-    // Validate event_date before any DB operations
-    if (decision.event_date !== null && decision.event_date !== undefined) {
-        const d = decision.event_date;
-        if (!/^\d{4}-\d{2}-\d{2}$/.test(d) || isNaN(new Date(d).getTime())) {
-            throw new Error(`Invalid event_date: must be ISO 8601 YYYY-MM-DD (got: ${d})`);
-        }
-    }
-    if (decision.event_datetime !== null && decision.event_datetime !== undefined) {
-        if (typeof decision.event_datetime !== 'number' ||
+/** Generate the optional vector before opening a synchronous SQLite transaction. */
+async function prepareDecisionEmbedding(decision) {
+    if (decision.event_date !== null &&
+        decision.event_date !== undefined &&
+        (!/^\d{4}-\d{2}-\d{2}$/.test(decision.event_date) ||
+            Number.isNaN(new Date(decision.event_date).getTime()))) {
+        throw new Error(`Invalid event_date: must be ISO 8601 YYYY-MM-DD (got: ${decision.event_date})`);
+    }
+    if (decision.event_datetime !== null &&
+        decision.event_datetime !== undefined &&
+        (typeof decision.event_datetime !== 'number' ||
             !Number.isFinite(decision.event_datetime) ||
-            decision.event_datetime <= 0) {
-            throw new Error(`Invalid event_datetime: must be a positive millisecond timestamp (got: ${decision.event_datetime})`);
-        }
-    }
-    try {
-        // Generate embedding BEFORE transaction (required for SQLite's sync transaction)
-        // Note: Redact topic for privacy - only log length
-        (0, debug_logger_js_1.info)(`[db-manager] Generating embedding for decision (topic length: ${decision.topic?.length || 0})`);
-        let embedding = null;
-        try {
-            // e5 passage role (default) - stored vector
-            embedding = await generateEnhancedEmbedding({
-                topic: decision.topic,
-                decision: decision.decision,
-                reasoning: decision.reasoning || undefined,
-                outcome: decision.outcome || undefined,
-                confidence: decision.confidence,
-            });
-            (0, debug_logger_js_1.info)(`[db-manager] Embedding generated: ${embedding ? embedding.length : 'null'} dimensions`);
-        }
-        catch (embGenErr) {
-            // Non-fatal: save decision without embedding (e.g. ONNX model unavailable on CI)
-            const message = embGenErr instanceof Error ? embGenErr.message : String(embGenErr);
-            (0, debug_logger_js_1.error)(`[db-manager] ⚠️ Embedding generation failed, saving without vector: ${message}`);
-        }
-        // SQLite: Synchronous transaction including embedding
-        adapter.transaction(() => {
-            // Prepare INSERT statement
-            const stmt = adapter.prepare(`
-        INSERT INTO decisions (
-          id, topic, decision, reasoning,
-          outcome, failure_reason, limitation,
-          user_involvement, session_id,
-          supersedes, superseded_by, refined_from,
-          confidence, created_at, updated_at,
-          needs_validation, validation_attempts, last_validated_at, usage_count,
-          trust_context, usage_success, usage_failure, time_saved,
-          evidence, alternatives, risks, event_date, event_datetime,
-          agent_id, model_run_id, envelope_hash, gateway_call_id, source_refs_json, provenance_json
-        ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `);
-            const insertResult = stmt.run(decision.id, decision.topic, decision.decision, decision.reasoning || null, decision.outcome || null, decision.failure_reason || null, decision.limitation || null, decision.user_involvement || null, decision.session_id || null, decision.supersedes || null, decision.superseded_by || null, decision.refined_from ? JSON.stringify(decision.refined_from) : null, decision.confidence !== undefined ? decision.confidence : 0.5, 
-            // IMPORTANT: All timestamps are stored in milliseconds (Date.now()).
-            // The schema DEFAULT uses unixepoch() (seconds) but is never used
-            // since all inserts go through this function which always provides ms.
-            decision.created_at || Date.now(), decision.updated_at || Date.now(), decision.needs_validation !== undefined ? decision.needs_validation : 0, decision.validation_attempts || 0, decision.last_validated_at || null, decision.usage_count || 0, decision.trust_context || null, decision.usage_success || 0, decision.usage_failure || 0, decision.time_saved || 0, decision.evidence || null, decision.alternatives || null, decision.risks || null, decision.event_date || null, decision.event_datetime ?? null, decision.agent_id ?? null, decision.model_run_id ?? null, decision.envelope_hash ?? null, decision.gateway_call_id ?? null, decision.source_refs_json ?? null, decision.provenance_json ?? null);
-            const rowid = Number(insertResult.lastInsertRowid);
-            // Insert embedding in same transaction to ensure rowid matching
-            (0, debug_logger_js_1.info)(`[db-manager] Vector search enabled: ${adapter.vectorSearchEnabled}`);
-            if (adapter.vectorSearchEnabled && embedding) {
-                try {
-                    (0, debug_logger_js_1.info)(`[db-manager] Inserting embedding for rowid: ${rowid}`);
-                    adapter.insertEmbedding(rowid, embedding);
-                    (0, debug_logger_js_1.info)(`[db-manager] ✅ Embedding inserted successfully`);
-                }
-                catch (embErr) {
-                    const message = embErr instanceof Error ? embErr.message : String(embErr);
-                    // Log but don't fail transaction if embedding fails
-                    (0, debug_logger_js_1.error)(`[db-manager] ❌ Embedding insert failed: ${message}`);
-                }
-            }
-            else {
-                (0, debug_logger_js_1.info)(`[db-manager] ⚠️  Vector search disabled, skipping embedding`);
-            }
-            return rowid;
-        });
-        if (process.env.MAMA_DEBUG) {
-            (0, debug_logger_js_1.info)(`[db-manager] Decision stored: ${decision.id}`);
-        }
-        return decision.id;
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        throw new Error(`Failed to insert decision with embedding: ${message}`);
-    }
+            decision.event_datetime <= 0)) {
+        throw new Error(`Invalid event_datetime: must be a positive millisecond timestamp (got: ${decision.event_datetime})`);
+    }
+    const { generateEnhancedEmbedding, isForceTier3Enabled } = await import('./embedding/embedder.js');
+    if (isForceTier3Enabled()) {
+        // Tier-3 is an explicit no-vector mode, not an embedder failure.
+        return null;
+    }
+    // Failure-first: an embedder error aborts the write instead of committing a
+    // decision row without its required vector.
+    return await generateEnhancedEmbedding({
+        topic: decision.topic,
+        decision: decision.decision,
+        reasoning: decision.reasoning || undefined,
+        outcome: decision.outcome || undefined,
+        confidence: decision.confidence,
+    });
 }
-/**
- * Query decision graph for topic
- *
- * Recursive CTE to traverse supersedes chain
- * SQLite implementation using WITH RECURSIVE
- *
- * @param topic - Decision topic to query
- * @returns Array of decisions (ordered by recency)
- */
-async function queryDecisionGraph(topic) {
-    const adapter = getAdapter();
-    try {
-        // Story 014.14 Fix: Prioritize exact topic match over fuzzy matching
-        // First try exact match, then fallback to fuzzy if no results
-        // Try exact match first
-        let stmt = adapter.prepare(`
-      WITH RECURSIVE decision_chain AS (
-        -- Base case: Get current decision (not superseded)
-        SELECT * FROM decisions
-        WHERE topic = ? AND superseded_by IS NULL
-
-        UNION ALL
-
-        -- Recursive case: Get previous decisions
-        SELECT d.* FROM decisions d
-        JOIN decision_chain dc ON d.id = dc.supersedes
-      )
-      SELECT * FROM decision_chain
-      ORDER BY created_at DESC
-    `);
-        let decisions = stmt.all(topic);
-        // If no exact match, try fuzzy matching as fallback
-        if (decisions.length === 0) {
-            const topicKeyword = topic.split('_')[0];
-            stmt = adapter.prepare(`
-        WITH RECURSIVE decision_chain AS (
-          -- Base case: Get current decision (not superseded)
-          SELECT * FROM decisions
-          WHERE topic LIKE ? || '%' AND superseded_by IS NULL
-
-          UNION ALL
-
-          -- Recursive case: Get previous decisions
-          SELECT d.* FROM decisions d
-          JOIN decision_chain dc ON d.id = dc.supersedes
-        )
-        SELECT * FROM decision_chain
-        ORDER BY created_at DESC
-      `);
-            decisions = stmt.all(topicKeyword);
-        }
-        // Join with decision_edges to include relationships
-        // Prepare statement once outside loop for performance
-        const edgesStmt = adapter.prepare(`
-      SELECT * FROM decision_edges
-      WHERE from_id = ?
-        AND (approved_by_user = 1 OR approved_by_user IS NULL)
-    `);
-        for (const decision of decisions) {
-            decision.edges = edgesStmt.all(decision.id);
-            // Parse refined_from JSON if exists
-            if (decision.refined_from) {
-                try {
-                    decision.refined_from =
-                        typeof decision.refined_from === 'string'
-                            ? JSON.parse(decision.refined_from)
-                            : decision.refined_from;
-                }
-                catch {
-                    decision.refined_from = [];
-                }
-            }
-        }
-        return decisions;
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        throw new Error(`Decision graph query failed: ${message}`);
-    }
-}
-/**
- * Query semantic edges for a list of decisions
- *
- * Returns both outgoing (from_id) and incoming (to_id) edges
- * for refines and contradicts relationships
- *
- * @param decisionIds - Decision IDs to query edges for
- * @returns Categorized edges
- */
-async function querySemanticEdges(decisionIds) {
-    const adapter = getAdapter();
-    if (!decisionIds || decisionIds.length === 0) {
-        return {
-            refines: [],
-            refined_by: [],
-            contradicts: [],
-            contradicted_by: [],
-            // Story 2.1: Extended edge types
-            builds_on: [],
-            built_on_by: [],
-            debates: [],
-            debated_by: [],
-            synthesizes: [],
-            synthesized_by: [],
-        };
-    }
-    try {
-        // Build placeholders for IN clause
-        const placeholders = decisionIds.map(() => '?').join(',');
-        // Story 2.1: Include new edge types in query
-        const edgeTypes = ['refines', 'contradicts', 'builds_on', 'debates', 'synthesizes'];
-        const edgeTypePlaceholders = edgeTypes.map(() => '?').join(',');
-        // Query outgoing edges (from_id = decision)
-        const outgoingStmt = adapter.prepare(`
-      SELECT e.*, d.topic, d.decision, d.confidence, d.created_at
-      FROM decision_edges e
-      JOIN decisions d ON e.to_id = d.id
-      WHERE e.from_id IN (${placeholders})
-        AND e.relationship IN (${edgeTypePlaceholders})
-        AND (e.approved_by_user = 1 OR e.approved_by_user IS NULL)
-      ORDER BY e.created_at DESC
-    `);
-        const outgoingEdges = outgoingStmt.all(...decisionIds, ...edgeTypes);
-        // Query incoming edges (to_id = decision)
-        const incomingStmt = adapter.prepare(`
-      SELECT e.*, d.topic, d.decision, d.confidence, d.created_at
-      FROM decision_edges e
-      JOIN decisions d ON e.from_id = d.id
-      WHERE e.to_id IN (${placeholders})
-        AND e.relationship IN (${edgeTypePlaceholders})
-        AND (e.approved_by_user = 1 OR e.approved_by_user IS NULL)
-      ORDER BY e.created_at DESC
-    `);
-        const incomingEdges = incomingStmt.all(...decisionIds, ...edgeTypes);
-        // Categorize edges (original + v1.3 extended)
-        const refines = outgoingEdges.filter((e) => e.relationship === 'refines');
-        const refined_by = incomingEdges.filter((e) => e.relationship === 'refines');
-        const contradicts = outgoingEdges.filter((e) => e.relationship === 'contradicts');
-        const contradicted_by = incomingEdges.filter((e) => e.relationship === 'contradicts');
-        // Story 2.1: New edge type categories
-        const builds_on = outgoingEdges.filter((e) => e.relationship === 'builds_on');
-        const built_on_by = incomingEdges.filter((e) => e.relationship === 'builds_on');
-        const debates = outgoingEdges.filter((e) => e.relationship === 'debates');
-        const debated_by = incomingEdges.filter((e) => e.relationship === 'debates');
-        const synthesizes = outgoingEdges.filter((e) => e.relationship === 'synthesizes');
-        const synthesized_by = incomingEdges.filter((e) => e.relationship === 'synthesizes');
-        return {
-            refines,
-            refined_by,
-            contradicts,
-            contradicted_by,
-            builds_on,
-            built_on_by,
-            debates,
-            debated_by,
-            synthesizes,
-            synthesized_by,
-        };
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        throw new Error(`Semantic edges query failed: ${message}`);
-    }
-}
-/**
- * Query vector search with time window and threshold
- *
- * Story 014.14: AC #1 - Vector Search for Related Decisions
- *
- * @param params - Search parameters
- * @returns Results with similarity scores and decision data
- */
-async function queryVectorSearch(params) {
-    const { query, limit = 10, threshold = 0.75, timeWindow = 90 * 24 * 60 * 60 * 1000 } = params;
-    const adapter = getAdapter();
-    const { generateEmbedding } = await import('./embeddings.js');
-    try {
-        // Generate embedding for query
-        const embedding = await generateEmbedding(query, 'query');
-        const cutoffTime = Date.now() - timeWindow;
-        const candidates = await adapter.vectorSearch(embedding, limit * 5);
-        if (!candidates || candidates.length === 0) {
-            return [];
-        }
-        const stmt = adapter.prepare(`SELECT * FROM decisions WHERE rowid = ?`);
-        const results = [];
-        for (const candidate of candidates) {
-            const decision = stmt.get(candidate.rowid);
-            if (!decision) {
-                continue;
-            }
-            if (decision.created_at < cutoffTime) {
-                continue;
-            }
-            const similarity = candidate.similarity ?? Math.max(0, 1 - (candidate.distance ?? 1));
-            const distance = candidate.distance ?? Math.max(0, 1 - similarity);
-            if (similarity < threshold) {
-                continue;
-            }
-            // Filter out superseded decisions unless explicitly requested
-            if (!params.includeSuperseded && decision.superseded_by) {
-                continue;
-            }
-            results.push({
-                ...decision,
-                similarity,
-                distance,
-            });
-            if (results.length >= limit) {
-                break;
-            }
-        }
-        return results;
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        (0, debug_logger_js_1.error)(`[db-manager] queryVectorSearch failed: ${message}`);
-        return []; // Return empty array on error (graceful degradation)
+/** Insert one decision and its already-prepared vector inside the caller's transaction. */
+function insertPreparedDecision(adapter, decision, embedding) {
+    const stmt = adapter.prepare(`
+    INSERT INTO decisions (
+      id, topic, decision, reasoning, outcome, failure_reason, limitation,
+      user_involvement, session_id, supersedes, superseded_by, refined_from,
+      confidence, created_at, updated_at, needs_validation, validation_attempts,
+      last_validated_at, usage_count, trust_context, usage_success, usage_failure,
+      time_saved, evidence, alternatives, risks, event_date, event_datetime,
+      agent_id, model_run_id, envelope_hash, gateway_call_id, source_refs_json, provenance_json
+    ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+  `);
+    const result = stmt.run(decision.id, decision.topic, decision.decision, decision.reasoning || null, decision.outcome || null, decision.failure_reason || null, decision.limitation || null, decision.user_involvement || null, decision.session_id || null, decision.supersedes || null, decision.superseded_by || null, decision.refined_from ? JSON.stringify(decision.refined_from) : null, decision.confidence ?? 0.5, decision.created_at || Date.now(), decision.updated_at || Date.now(), decision.needs_validation ?? 0, decision.validation_attempts || 0, decision.last_validated_at || null, decision.usage_count || 0, decision.trust_context || null, decision.usage_success || 0, decision.usage_failure || 0, decision.time_saved || 0, decision.evidence || null, decision.alternatives || null, decision.risks || null, decision.event_date || null, decision.event_datetime ?? null, decision.agent_id ?? null, decision.model_run_id ?? null, decision.envelope_hash ?? null, decision.gateway_call_id ?? null, decision.source_refs_json ?? null, decision.provenance_json ?? null);
+    const rowid = Number(result.lastInsertRowid);
+    if (embedding) {
+        adapter.insertEmbedding(rowid, embedding);
     }
+    return rowid;
 }
 /**
  * Update decision outcome
@@ -649,20 +245,18 @@ async function queryVectorSearch(params) {
  * @param outcomeData - Outcome data
  */
 async function updateDecisionOutcome(decisionId, outcomeData) {
-    const adapter = getAdapter();
     try {
-        const stmt = adapter.prepare(`
-      UPDATE decisions
-      SET
-        outcome = ?,
-        failure_reason = ?,
-        limitation = ?,
-        duration_days = ?,
-        confidence = COALESCE(?, confidence),
-        updated_at = ?
-      WHERE id = ?
-    `);
-        stmt.run(outcomeData.outcome ?? null, outcomeData.failure_reason ?? null, outcomeData.limitation ?? null, outcomeData.duration_days ?? null, outcomeData.confidence !== undefined ? outcomeData.confidence : null, Date.now(), decisionId);
+        // Append-only: the outcome change is an authored judgment record; the
+        // maintained decisions projection columns move in the same transaction.
+        // Dynamic import avoids a module cycle with memory/write-adapters.
+        const { appendOutcomeAmendment } = await import('./memory/write-adapters.js');
+        await appendOutcomeAmendment(decisionId, {
+            outcome: outcomeData.outcome ?? null,
+            failureReason: outcomeData.failure_reason ?? null,
+            limitation: outcomeData.limitation ?? null,
+            confidence: outcomeData.confidence ?? null,
+            durationDays: outcomeData.duration_days ?? null,
+        }, { adapter: getAdapter() });
         (0, debug_logger_js_1.info)(`[db-manager] Decision outcome updated: ${decisionId} → ${outcomeData.outcome}`);
     }
     catch (error) {
@@ -670,58 +264,16 @@ async function updateDecisionOutcome(decisionId, outcomeData) {
         throw new Error(`Failed to update decision outcome: ${message}`);
     }
 }
-/**
- * Get prepared statement
- *
- * For backward compatibility with memory-store.js
- * Returns a compatibility shim that proxies to adapter.prepare()
- *
- * @param sql - SQL statement
- * @returns Statement-like object with run/get/all methods
- */
-function getPreparedStmt(sql) {
-    if (!dbAdapter) {
-        (0, debug_logger_js_1.warn)('[db-manager] getPreparedStmt() called before initialization');
-        // Return no-op object for feature detection (won't throw)
-        return {
-            run: () => ({ changes: 0, lastInsertRowid: 0 }),
-            get: () => null,
-            all: () => [],
-        };
-    }
-    // Proxy to adapter.prepare() for actual usage
-    try {
-        return dbAdapter.prepare(sql);
-    }
-    catch (error) {
-        const message = error instanceof Error ? error.message : String(error);
-        (0, debug_logger_js_1.warn)(`[db-manager] getPreparedStmt() failed: ${message}`);
-        // Return no-op object on error (graceful degradation)
-        return {
-            run: () => ({ changes: 0, lastInsertRowid: 0 }),
-            get: () => null,
-            all: () => [],
-        };
-    }
-}
 /**
  * Get database file path
  *
  * @returns Actual database path or 'Not initialized'
  */
 function getDbPath() {
-    if (!dbAdapter) {
+    if (!handle) {
         return 'Not initialized';
     }
-    // Use adapter's getDbPath method if available, fallback to description
-    if (typeof dbAdapter.getDbPath === 'function') {
-        return dbAdapter.getDbPath();
-    }
-    // Fallback: try to get path from adapter properties
-    if (dbAdapter.dbPath) {
-        return dbAdapter.dbPath;
-    }
-    return `${dbAdapter.constructor.name} (path unavailable)`;
+    return handle.dbPath;
 }
 // Note: Removed auto-registered SIGINT/SIGTERM handlers that called process.exit(0)
 // This was causing issues with host cleanup in parent processes.
@@ -737,54 +289,20 @@ function getDbPath() {
  */
 function resetDBState(options = {}) {
     const { disconnect = true } = options;
-    if (disconnect && dbAdapter) {
+    if (disconnect && handle) {
         try {
-            dbAdapter.disconnect();
+            handle.adapter.disconnect();
         }
         catch {
             // Ignore disconnect errors during reset
         }
     }
-    dbAdapter = null;
-    dbConnection = null;
-    isInitialized = false;
-    initializingPromise = null;
+    handle = null;
+    openingPromise = null;
 }
 /**
  * Check if running in test mode
  *
  * Returns true if MAMA_TEST_MODE, VITEST, or NODE_ENV=test env vars are set
  */
-function isTestMode() {
-    return Boolean(process.env.MAMA_TEST_MODE || process.env.VITEST || process.env.NODE_ENV === 'test');
-}
-/**
- * FTS5 keyword search on decisions table.
- * Returns matching decision IDs with BM25 rank scores.
- */
-async function fts5Search(query, limit = 10) {
-    const adapter = getAdapter();
-    // Check if FTS5 table exists (swallow errors — table may not exist yet)
-    let tableCheck;
-    try {
-        tableCheck = adapter
-            .prepare("SELECT name FROM sqlite_master WHERE type='table' AND name='decisions_fts'")
-            .get();
-    }
-    catch {
-        return [];
-    }
-    if (!tableCheck)
-        return [];
-    // Query execution — let errors propagate to the caller
-    const stmt = adapter.prepare(`
-    SELECT d.id, rank
-    FROM decisions_fts
-    JOIN decisions d ON decisions_fts.rowid = d.rowid
-    WHERE decisions_fts MATCH ?
-    ORDER BY rank
-    LIMIT ?
-  `);
-    return stmt.all(query, limit);
-}
 //# sourceMappingURL=db-manager.js.map