hippo-memory 1.69.0 → 1.71.0

package/dist/{store-cards.js → store/cards.js}

@@ -1,10 +1,11 @@
-import { generateId } from './memory.js';
-import { closeHippoDb } from './db.js';
-import { rowToSessionHandoff, isHandoffOutcome } from './handoff.js';
-import { CARD_TRANSITIONS, CARD_LEASE_MS } from './card.js';
+import { generateId } from '../core/memory.js';
+import { withWriteScope, withWriteScopeOr } from '../db/index.js';
+import { rowToSessionHandoff, isHandoffOutcome } from '../core/handoff.js';
+import { CARD_LEASE_MS, assertCardTransition } from '../core/card.js';
 import { assertTenantId } from './tenant.js';
-import { openStore } from './store/open.js';
-import { HANDOFF_COLUMNS } from './store/handoffs.js';
+import { onHandle, openStore } from './open.js';
+import { chunked } from './entry-reads.js';
+import { HANDOFF_COLUMNS } from './handoffs.js';
 const CARD_COLUMNS = 'id, title, status, assignee_runtime, repo, contract, budget, lease_until, heartbeat_at, created_at, updated_at, tenant_id, scope';
 function rowToCard(row) {
     return {
@@ -77,27 +78,27 @@ function closeLiveRun(db, tenantId, cardId, outcome, now) {
     WHERE card = ? AND tenant_id = ? AND ended IS NULL
   `).run(now, outcome, now, cardId, tenantId);
 }
-// The single status-mutating seam (rule 15): CARD_TRANSITIONS is the one
-// runtime authority, so a hand-copied wrong `from` list fails fast here.
-export function transitionCard(db, tenantId, cardId, from, to, extra) {
-    for (const status of from) {
-        if (!CARD_TRANSITIONS[status].includes(to)) {
-            throw new Error(`illegal card transition: ${status} -> ${to}`);
-        }
-    }
+function transitionCard(db, tenantId, cardId, options) {
+    return transitionCards(db, tenantId, [cardId], options);
+}
+/** transitionCard for every card in `cardIds`, as one statement with one timestamp; returns how many moved. */
+function transitionCards(db, tenantId, cardIds, options) {
+    const { from, to, extra } = options;
+    assertCardTransition(from, to);
     const now = new Date().toISOString();
     // Lease columns follow status: set on the move to running, cleared on every other move (rule 15).
     const lease = to === 'running' ? [leaseUntilFrom(now), now] : [null, null];
     const fromPlaceholders = from.map(() => '?').join(', ');
     const sql = `
     UPDATE cards SET status = ?, updated_at = ?, lease_until = ?, heartbeat_at = ?${extra?.setSql ? `, ${extra.setSql}` : ''}
-    WHERE id = ? AND tenant_id = ? AND status IN (${fromPlaceholders})${extra?.whereSql ? ` AND ${extra.whereSql}` : ''}
+    WHERE id IN (${cardIds.map(() => '?').join(', ')}) AND tenant_id = ? AND status IN (${fromPlaceholders})${extra?.whereSql ? ` AND ${extra.whereSql}` : ''}
   `;
-    const params = [to, now, ...lease, ...(extra?.params ?? []), cardId, tenantId, ...from];
+    const params = [to, now, ...lease, ...(extra?.params ?? []), ...cardIds, tenantId, ...from];
     const result = db.prepare(sql).run(...params);
     return Number(result.changes ?? 0);
 }
-/** Creates a card; status is ready with no deps or once every dependsOn id is done, else backlog. An unknown dependsOn id throws and commits nothing. A repeated dependsOn id is recorded once. */
+/** Creates a card; status is ready with no deps or once every dependsOn id is done, else backlog.
+ * An unknown dependsOn id throws and commits nothing. A repeated dependsOn id is recorded once. */
 export function createCard(hippoRoot, tenantId, input) {
     assertTenantId('createCard', tenantId);
     if (input.title.trim() === '') {
@@ -106,28 +107,13 @@ export function createCard(hippoRoot, tenantId, input) {
     if (input.budget !== undefined && !(Number.isSafeInteger(input.budget) && input.budget > 0)) {
         throw new Error(`Invalid budget: ${input.budget} (expected a positive integer)`);
     }
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         const dependsOn = [...new Set(input.dependsOn ?? [])];
         let id = '';
-        db.exec('BEGIN IMMEDIATE');
-        try {
+        withWriteScope(db, 'create_card', () => {
             // Probe runs inside the transaction (mirrors batchWriteAndDelete): a parent
             // completing between an outside-the-lock read and the INSERT would strand the child.
-            let allParentsDone = true;
-            if (dependsOn.length > 0) {
-                const placeholders = dependsOn.map(() => '?').join(', ');
-                // SAFETY: rows' shape matches the two columns named in the SELECT below.
-                const rows = db.prepare(`SELECT id, status FROM cards WHERE tenant_id = ? AND id IN (${placeholders})`).all(tenantId, ...dependsOn);
-                const found = new Map(rows.map((r) => [r.id, r.status]));
-                // Pre-check before any write: a typo'd --depends-on can never commit a card row.
-                for (const parentId of dependsOn) {
-                    if (!found.has(parentId)) {
-                        throw new Error(`unknown parent card id: ${parentId}`);
-                    }
-                }
-                allParentsDone = dependsOn.every((pid) => found.get(pid) === 'done');
-            }
+            const allParentsDone = everyParentDone(db, tenantId, dependsOn);
             id = generateId('card');
             const now = new Date().toISOString();
             const status = dependsOn.length === 0 || allParentsDone ? 'ready' : 'backlog';
@@ -135,42 +121,42 @@ export function createCard(hippoRoot, tenantId, input) {
         INSERT INTO cards (id, title, status, repo, contract, budget, created_at, updated_at, tenant_id)
         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)
       `).run(id, input.title, status, input.repo ?? null, input.contract ?? null, input.budget ?? null, now, now, tenantId);
-            for (const parentId of dependsOn) {
-                db.prepare(`
-          INSERT INTO card_deps (parent, child, tenant_id, created_at) VALUES (?, ?, ?, ?)
-        `).run(parentId, id, tenantId, now);
-            }
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
+            const insertDep = db.prepare(`
+        INSERT INTO card_deps (parent, child, tenant_id, created_at) VALUES (?, ?, ?, ?)
+      `);
+            for (const parentId of dependsOn)
+                insertDep.run(parentId, id, tenantId, now);
+        });
         return loadCardRow(db, tenantId, id);
+    }, openStore);
+}
+/** Whether every dependsOn card is done, true for none; throws on an id that is no card of the tenant. Call inside the write scope. */
+function everyParentDone(db, tenantId, dependsOn) {
+    if (dependsOn.length === 0)
+        return true;
+    const placeholders = dependsOn.map(() => '?').join(', ');
+    // SAFETY: rows' shape matches the two columns named in the SELECT below.
+    const rows = db.prepare(`SELECT id, status FROM cards WHERE tenant_id = ? AND id IN (${placeholders})`).all(tenantId, ...dependsOn);
+    const found = new Map(rows.map((r) => [r.id, r.status]));
+    // Pre-check before any write: a typo'd --depends-on can never commit a card row.
+    for (const parentId of dependsOn) {
+        if (!found.has(parentId)) {
+            throw new Error(`unknown parent card id: ${parentId}`);
+        }
     }
-    finally {
-        closeHippoDb(db);
-    }
+    return dependsOn.every((pid) => found.get(pid) === 'done');
 }
 /** Returns the card row for id, or null if it does not exist under this tenant. */
 export function loadCard(hippoRoot, tenantId, id) {
     assertTenantId('loadCard', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         return loadCardRow(db, tenantId, id);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
 /** Lists cards for this tenant, optionally filtered to one status, newest-updated first. */
 export function listCards(hippoRoot, tenantId, opts = {}) {
     assertTenantId('listCards', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         const conditions = ['tenant_id = ?'];
         const params = [tenantId];
         if (opts.status) {
@@ -182,153 +168,109 @@ export function listCards(hippoRoot, tenantId, opts = {}) {
       SELECT ${CARD_COLUMNS} FROM cards WHERE ${conditions.join(' AND ')} ORDER BY updated_at DESC, id DESC
     `).all(...params);
         return rows.map(rowToCard);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
 /** Returns this card's parent and child ids from card_deps. */
 export function loadCardDeps(hippoRoot, tenantId, id) {
     assertTenantId('loadCardDeps', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         // SAFETY: rows' shape matches the single `parent` column named in the SELECT below.
         const parents = db.prepare(`SELECT parent FROM card_deps WHERE tenant_id = ? AND child = ?`).all(tenantId, id).map((r) => r.parent);
         // SAFETY: rows' shape matches the single `child` column named in the SELECT below.
         const children = db.prepare(`SELECT child FROM card_deps WHERE tenant_id = ? AND parent = ?`).all(tenantId, id).map((r) => r.child);
         return { parents, children };
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
 /** Returns this card's run history, most recent first. */
 export function loadCardRuns(hippoRoot, tenantId, id) {
     assertTenantId('loadCardRuns', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         // SAFETY: rows' shape matches CardRunRow.
         const rows = db.prepare(`
       SELECT id, card, runtime, session_id, started, ended, outcome
       FROM card_runs WHERE tenant_id = ? AND card = ? ORDER BY started DESC, id DESC
     `).all(tenantId, id);
         return rows.map(rowToCardRun);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
 /** Returns this card's comments, most recent first. */
 export function loadCardComments(hippoRoot, tenantId, id) {
     assertTenantId('loadCardComments', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         // SAFETY: rows' shape matches CardCommentRow.
         const rows = db.prepare(`
       SELECT id, card_id, author, body, created_at
       FROM card_comments WHERE tenant_id = ? AND card_id = ? ORDER BY created_at DESC, id DESC
     `).all(tenantId, id);
         return rows.map(rowToCardComment);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
 /** Read side of the card <-> handoff round trip: the newest handoff filed against this card. */
 export function loadLatestHandoffForCard(hippoRoot, tenantId, cardId) {
     assertTenantId('loadLatestHandoffForCard', tenantId);
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         // SAFETY: row's shape matches HANDOFF_COLUMNS.
         const row = db.prepare(`
       SELECT ${HANDOFF_COLUMNS} FROM session_handoffs
       WHERE tenant_id = ? AND card_id = ? ORDER BY created_at DESC, id DESC LIMIT 1
     `).get(tenantId, cardId);
         return row ? rowToSessionHandoff(row) : null;
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
-/** Atomic claim: WHERE status IN (ready, blocked) AND assignee_runtime IS NULL decides the race. Throws on an unknown card id; returns null for a card not ready/blocked or already claimed. Sets a CARD_LEASE_MS lease and returns the new run's id as runId. */
+/** Atomic claim: WHERE status IN (ready, blocked) AND assignee_runtime IS NULL decides the race. Throws on an unknown card id;
+ * returns null for a card not ready/blocked or already claimed. Sets a CARD_LEASE_MS lease and returns the new run's id as runId. */
 export function claimCard(hippoRoot, tenantId, id, runtime, sessionId) {
     assertTenantId('claimCard', tenantId);
     if (runtime.trim() === '') {
         throw new Error('runtime must not be empty');
     }
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        let runId = 0;
-        try {
-            const changes = transitionCard(db, tenantId, id, ['ready', 'blocked'], 'running', {
-                setSql: 'assignee_runtime = ?',
-                whereSql: 'assignee_runtime IS NULL',
-                params: [runtime],
+    return onHandle(hippoRoot, (db) => {
+        const runId = withWriteScopeOr(db, 'claim_card', (rollback) => {
+            const changes = transitionCard(db, tenantId, id, {
+                from: ['ready', 'blocked'],
+                to: 'running',
+                extra: { setSql: 'assignee_runtime = ?', whereSql: 'assignee_runtime IS NULL', params: [runtime] },
             });
             if (changes === 0) {
                 if (!loadCardRow(db, tenantId, id)) {
                     throw new Error(`unknown card id: ${id}`);
                 }
-                db.exec('ROLLBACK');
-                return null;
+                return rollback(null);
             }
             const now = new Date().toISOString();
             const insert = db.prepare(`
         INSERT INTO card_runs (card, runtime, session_id, started, created_at, updated_at, tenant_id)
         VALUES (?, ?, ?, ?, ?, ?, ?)
       `).run(id, runtime, sessionId ?? null, now, now, now, tenantId);
-            runId = Number(insert.lastInsertRowid ?? 0);
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-        return { ...loadCardRow(db, tenantId, id), runId };
-    }
-    finally {
-        closeHippoDb(db);
-    }
+            return Number(insert.lastInsertRowid ?? 0);
+        });
+        return runId === null ? null : { ...loadCardRow(db, tenantId, id), runId };
+    }, openStore);
 }
-/** Moves a running card's lease to CARD_LEASE_MS from now and records the heartbeat; updated_at is left alone. Throws on an unknown card id or a run id that is not a positive integer; returns null unless the card is running and runId is its live run. */
+/** Moves a running card's lease to CARD_LEASE_MS from now and records the heartbeat; updated_at is left alone. Throws on an
+ * unknown card id or a run id that is not a positive integer; returns null unless the card is running and runId is its live run. */
 export function heartbeatCard(hippoRoot, tenantId, id, runId) {
     assertTenantId('heartbeatCard', tenantId);
     assertRunId(runId);
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        try {
+    return onHandle(hippoRoot, (db) => {
+        const beat = withWriteScopeOr(db, 'heartbeat_card', (rollback) => {
             const card = loadCardRow(db, tenantId, id);
             if (!card) {
                 throw new Error(`unknown card id: ${id}`);
             }
             if (card.status !== 'running' || !isLiveRun(db, tenantId, id, runId)) {
-                db.exec('ROLLBACK');
-                return null;
+                return rollback(false);
             }
             const now = new Date().toISOString();
             db.prepare(`UPDATE cards SET lease_until = ?, heartbeat_at = ? WHERE id = ? AND tenant_id = ?`)
                 .run(leaseUntilFrom(now), now, id, tenantId);
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-        return loadCardRow(db, tenantId, id);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+            return true;
+        });
+        return beat ? loadCardRow(db, tenantId, id) : null;
+    }, openStore);
 }
-/** Requires the card be running; closes the live run as blocked and files reason as a comment. Throws on an unknown card id; returns null for a card not running. When runId is given, returns null unless it is the card's live run. */
+/** Requires the card be running; closes the live run as blocked and files reason as a comment. Throws on an unknown
+ * card id; returns null for a card not running. When runId is given, returns null unless it is the card's live run. */
 export function blockCard(hippoRoot, tenantId, id, reason, runId) {
     assertTenantId('blockCard', tenantId);
     if (reason.trim() === '') {
@@ -336,72 +278,48 @@ export function blockCard(hippoRoot, tenantId, id, reason, runId) {
     }
     if (runId !== undefined)
         assertRunId(runId);
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        try {
+    return onHandle(hippoRoot, (db) => {
+        const blocked = withWriteScopeOr(db, 'block_card', (rollback) => {
             const allowed = runId === undefined || isLiveRun(db, tenantId, id, runId);
-            const changes = allowed ? transitionCard(db, tenantId, id, ['running'], 'blocked', { setSql: 'assignee_runtime = NULL' }) : 0;
+            const changes = allowed ? transitionCard(db, tenantId, id, { from: ['running'], to: 'blocked', extra: { setSql: 'assignee_runtime = NULL' } }) : 0;
             if (changes === 0) {
                 if (!loadCardRow(db, tenantId, id)) {
                     throw new Error(`unknown card id: ${id}`);
                 }
-                db.exec('ROLLBACK');
-                return null;
+                return rollback(false);
             }
             const now = new Date().toISOString();
             // Close the interrupted run here so completeCard's ended IS NULL scope only ever matches the live run.
             closeLiveRun(db, tenantId, id, 'blocked', now);
             insertCardComment(db, tenantId, id, 'system', reason);
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-        return loadCardRow(db, tenantId, id);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+            return true;
+        });
+        return blocked ? loadCardRow(db, tenantId, id) : null;
+    }, openStore);
 }
-/** Requires the card be running; moves it to review, clearing its lease and heartbeat and keeping its live run. When runId is given, returns null unless it is the card's live run. Throws on an unknown card id; returns null for a card not running. */
+/** Requires the card be running; moves it to review, clearing its lease and heartbeat and keeping its live run. When runId is
+ * given, returns null unless it is the card's live run. Throws on an unknown card id; returns null for a card not running. */
 export function reviewCard(hippoRoot, tenantId, id, runId) {
     assertTenantId('reviewCard', tenantId);
     if (runId !== undefined)
         assertRunId(runId);
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        try {
+    return onHandle(hippoRoot, (db) => {
+        const moved = withWriteScopeOr(db, 'review_card', (rollback) => {
             const allowed = runId === undefined || isLiveRun(db, tenantId, id, runId);
-            const changes = allowed ? transitionCard(db, tenantId, id, ['running'], 'review') : 0;
+            const changes = allowed ? transitionCard(db, tenantId, id, { from: ['running'], to: 'review' }) : 0;
             if (changes === 0) {
                 if (!loadCardRow(db, tenantId, id)) {
                     throw new Error(`unknown card id: ${id}`);
                 }
-                db.exec('ROLLBACK');
-                return null;
-            }
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
+                return rollback(false);
             }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-        return loadCardRow(db, tenantId, id);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+            return true;
+        });
+        return moved ? loadCardRow(db, tenantId, id) : null;
+    }, openStore);
 }
-/** Requires the card be in review; closes the live run with outcome. Outcome 'success' moves the card to done and, in the same transaction, promotes any child whose parents are now all done; 'failure' or 'partial' moves it to shelved and promotes nothing. Throws on an unknown card id; returns null for a card not in review. When runId is given, returns null unless it is the card's live run. */
+/** Closes the live run of a card in review: 'success' moves it to done and promotes children whose parents are all done; 'failure'/'partial' shelves it.
+ * Throws on an unknown card id; returns null for a card not in review, or when `runId` is given and is not the card's live run. */
 export function completeCard(hippoRoot, tenantId, id, outcome, runId) {
     assertTenantId('completeCard', tenantId);
     if (!isHandoffOutcome(outcome)) {
@@ -409,66 +327,48 @@ export function completeCard(hippoRoot, tenantId, id, outcome, runId) {
     }
     if (runId !== undefined)
         assertRunId(runId);
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        let promotedChildren = [];
-        try {
+    return onHandle(hippoRoot, (db) => {
+        const promoted = withWriteScopeOr(db, 'complete_card', (rollback) => {
             const target = outcome === 'success' ? 'done' : 'shelved';
             const allowed = runId === undefined || isLiveRun(db, tenantId, id, runId);
-            const changes = allowed ? transitionCard(db, tenantId, id, ['review'], target) : 0;
+            const changes = allowed ? transitionCard(db, tenantId, id, { from: ['review'], to: target }) : 0;
             if (changes === 0) {
                 if (!loadCardRow(db, tenantId, id)) {
                     throw new Error(`unknown card id: ${id}`);
                 }
-                db.exec('ROLLBACK');
-                return null;
+                return rollback(null);
             }
             const now = new Date().toISOString();
             closeLiveRun(db, tenantId, id, outcome, now);
             // Not best-effort (rule 12): promotion runs in this same transaction, so a
             // card can never be `done` with an un-evaluated child.
-            if (target === 'done') {
-                // SAFETY: rows' shape matches the single `child` column named in the SELECT below.
-                const children = db.prepare(`SELECT child FROM card_deps WHERE tenant_id = ? AND parent = ?`).all(tenantId, id).map((r) => r.child);
-                for (const childId of children) {
-                    // SAFETY: row's shape matches the single `status` column named in the SELECT below.
-                    const child = db.prepare(`SELECT status FROM cards WHERE tenant_id = ? AND id = ?`).get(tenantId, childId);
-                    if (!child || child.status !== 'backlog')
-                        continue;
-                    // SAFETY: rows' shape matches the single `parent` column named in the SELECT below.
-                    const parents = db.prepare(`SELECT parent FROM card_deps WHERE tenant_id = ? AND child = ?`).all(tenantId, childId).map((r) => r.parent);
-                    const placeholders = parents.map(() => '?').join(', ');
-                    // SAFETY: row's shape matches the single `c` column named in the SELECT below.
-                    const doneCount = db.prepare(`SELECT COUNT(*) as c FROM cards WHERE tenant_id = ? AND id IN (${placeholders}) AND status = 'done'`).get(tenantId, ...parents).c;
-                    if (doneCount === parents.length) {
-                        transitionCard(db, tenantId, childId, ['backlog'], 'ready');
-                        promotedChildren.push(childId);
-                    }
-                }
-            }
-            db.exec('COMMIT');
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-        return { card: loadCardRow(db, tenantId, id), promotedChildren };
-    }
-    finally {
-        closeHippoDb(db);
-    }
+            return target === 'done' ? promoteUnblockedChildren(db, tenantId, id) : [];
+        });
+        return promoted === null ? null : { card: loadCardRow(db, tenantId, id), promotedChildren: promoted };
+    }, openStore);
+}
+/** Moves to ready each backlog child of `parentId` whose parents are all done, and returns their ids. Call inside the write scope. */
+function promoteUnblockedChildren(db, tenantId, parentId) {
+    // The child's status and its open parents are result columns, not joins, so the rows keep the order of a plain read of the parent's children.
+    // SAFETY: rows' shape matches the three columns named in the SELECT below.
+    const children = db.prepare(`
+    SELECT d.child AS child,
+      (SELECT c.status FROM cards c WHERE c.tenant_id = ? AND c.id = d.child) AS status,
+      (SELECT COUNT(*) FROM card_deps p WHERE p.tenant_id = ? AND p.child = d.child
+        AND NOT EXISTS (SELECT 1 FROM cards pc WHERE pc.tenant_id = ? AND pc.id = p.parent AND pc.status = 'done')) AS open_parents
+    FROM card_deps d WHERE d.tenant_id = ? AND d.parent = ?
+  `).all(tenantId, tenantId, tenantId, tenantId, parentId);
+    const unblocked = children.filter((r) => r.status === 'backlog' && r.open_parents === 0).map((r) => r.child);
+    for (const ids of chunked(unblocked))
+        transitionCards(db, tenantId, ids, { from: ['backlog'], to: 'ready' });
+    return unblocked;
 }
-/** Returns to ready every running card of the tenant whose lease has expired or is missing: clears its assignee, closes its live run as 'reclaimed' and leaves its handoffs alone, all in one write transaction. Returns the reclaimed card ids in id order. */
+/** Returns to ready every running card of the tenant whose lease has expired or is missing: clears its assignee, closes its live
+ * run as 'reclaimed' and leaves its handoffs alone, all in one write transaction. Returns the reclaimed card ids in id order. */
 export function reclaimExpiredCards(hippoRoot, tenantId) {
     assertTenantId('reclaimExpiredCards', tenantId);
-    const db = openStore(hippoRoot);
-    try {
-        db.exec('BEGIN IMMEDIATE');
-        try {
+    return onHandle(hippoRoot, (db) => {
+        return withWriteScope(db, 'reclaim_expired_cards', () => {
             // Read lease times under the write lock, so a heartbeat that committed while we waited wins.
             const now = new Date().toISOString();
             // SAFETY: rows' shape matches the single `id` column named in the SELECT below.
@@ -478,23 +378,12 @@ export function reclaimExpiredCards(hippoRoot, tenantId) {
         ORDER BY id
       `).all(tenantId, now).map((r) => r.id);
             for (const id of ids) {
-                transitionCard(db, tenantId, id, ['running'], 'ready', { setSql: 'assignee_runtime = NULL' });
+                transitionCard(db, tenantId, id, { from: ['running'], to: 'ready', extra: { setSql: 'assignee_runtime = NULL' } });
                 closeLiveRun(db, tenantId, id, 'reclaimed', now);
             }
-            db.exec('COMMIT');
             return ids;
-        }
-        catch (error) {
-            try {
-                db.exec('ROLLBACK');
-            }
-            catch { /* commit may have already rolled back */ }
-            throw error;
-        }
-    }
-    finally {
-        closeHippoDb(db);
-    }
+        });
+    }, openStore);
 }
 /** Appends a comment to cardId in any card status; throws if cardId is not a card of this tenant. */
 export function addCardComment(hippoRoot, tenantId, cardId, author, body) {
@@ -502,12 +391,8 @@ export function addCardComment(hippoRoot, tenantId, cardId, author, body) {
     if (body.trim() === '') {
         throw new Error('body must not be empty');
     }
-    const db = openStore(hippoRoot);
-    try {
+    return onHandle(hippoRoot, (db) => {
         return insertCardComment(db, tenantId, cardId, author, body);
-    }
-    finally {
-        closeHippoDb(db);
-    }
+    }, openStore);
 }
-//# sourceMappingURL=store-cards.js.map
+//# sourceMappingURL=cards.js.map