hippo-memory 1.57.0 → 1.58.0

package/dist/connectors/slack/web-client.js

@@ -1,7 +1,8 @@
-import { fetchWithRetry } from './ratelimit.js';
+import { fetchWithRetry, isRetryableStatus } from '../../http-retry.js';
 function isJsonString(value) {
     return typeof value === 'string';
 }
+const SLACK_TIMEOUT_MS = 30_000;
 export function slackHistoryFetcher(token, fetchImpl) {
     return async ({ channelId, cursor, oldest }) => {
         const url = new URL('https://slack.com/api/conversations.history');
@@ -11,11 +12,12 @@ export function slackHistoryFetcher(token, fetchImpl) {
             url.searchParams.set('cursor', cursor);
         if (oldest)
             url.searchParams.set('oldest', oldest);
-        const r = await fetchWithRetry({
-            url: url.toString(),
-            init: { method: 'GET', headers: { authorization: `Bearer ${token}` } },
-            fetchImpl,
+        const r = await fetchWithRetry(url, { method: 'GET', headers: { authorization: `Bearer ${token}` } }, {
+            timeoutMs: SLACK_TIMEOUT_MS,
+            fetchFn: fetchImpl,
         });
+        if (isRetryableStatus(r.status))
+            throw new Error(`slack: still rate-limited or unavailable (HTTP ${r.status})`);
         // SAFETY: body is the Slack `conversations.history` response; per the
         // documented shape it's `{ ok, error?, messages?, response_metadata? }`.
         const body = (await r.json());

package/dist/consolidate.d.ts

@@ -6,6 +6,7 @@
  * 2. Merge pass  - find episodic entries with high text overlap, create semantic summaries
  * 3. Stats tracking
  */
+import { MemoryEntry, type DecayOptions } from './memory.js';
 export interface ConsolidationResult {
     decayed: number;
     removed: number;
@@ -45,4 +46,13 @@ export declare function consolidate(hippoRoot: string, options?: {
     now?: Date;
     fetcher?: typeof fetch;
 }): Promise<ConsolidationResult>;
+/** Maps i to each j > i, ascending, whose text overlap with i reaches the merge threshold; every text needs at least one token. */
+export declare function mergePartners(contents: readonly string[]): (i: number) => number[];
+/** Pairs of live non-semantic memories that contradict each other, in survivor order. */
+export declare function detectConflicts(entries: MemoryEntry[], now: Date, decayOpts?: DecayOptions, rescuedIds?: Set<string>): Array<{
+    memory_a_id: string;
+    memory_b_id: string;
+    reason: string;
+    score: number;
+}>;
 //# sourceMappingURL=consolidate.d.ts.map

package/dist/consolidate.js

@@ -9,8 +9,8 @@
 import { evalNow, isRecallBoostAblated } from './ablation.js';
 import { Layer, calculateStrength, canAutoDelete, createMemory, markRetrieved } from './memory.js';
 import { loadAllEntries, batchWriteAndDelete, appendConsolidationRun, replaceDetectedConflicts, loadSessionDecayContext, incrementSleepCount, findPromotableSessions, traceExistsForSession, listSessionEvents, memoriesBackingObjects, } from './store.js';
-import { textOverlap } from './search.js';
 import { tokenize } from './tokenize.js';
+import { jaccardMinShared, overlapPartners } from './overlap-index.js';
 import { compareEntryIdentity } from './compare.js';
 import { duplicateKey, mergedText } from './same-text.js';
 import { successorAfterRetirement } from './merged-row.js';
@@ -26,7 +26,7 @@ import { renderTraceContent } from './trace.js';
 import { resolveTenantId } from './tenant.js';
 import { rescueSet, rankNonPinnedByTenant, validateWeights } from './memory-value.js';
 import { MEMORY_VALUE_WEIGHTS, SOURCE_ARTIFACT_SHA256 } from './memory-value-weights.js';
-import { appendAuditEvent } from './audit.js';
+import { appendAuditEvent, reportAuditWriteFailure } from './audit.js';
 import { migrateDefaultHalfLife, LEGACY_TYPED_HALF_LIFE } from './half-life-migration.js';
 import { derivationScope, commonDerivationScope, derivationPartitionKey } from './recall-scope.js';
 import { isQuarantineScope } from './quarantine.js';
@@ -419,8 +419,8 @@ export async function consolidate(hippoRoot, options = {}) {
                                 },
                             });
                         }
-                        catch {
-                            // Best-effort — mirrors store.ts's audit() semantics.
+                        catch (error) {
+                            reportAuditWriteFailure('reject_refusal', String(error));
                         }
                         continue;
                     }
@@ -708,17 +708,14 @@ export async function consolidate(hippoRoot, options = {}) {
         for (const [, tenantCandidates] of mergeCandidatesByTenant) {
             const mergeTenant = tenantCandidates[0].tenantId;
             const mergeScope = derivationScope(tenantCandidates[0].scope);
+            const partnersOf = mergePartners(tenantCandidates.map((e) => e.content));
             for (let i = 0; i < tenantCandidates.length; i++) {
                 if (used.has(tenantCandidates[i].id) || tenantCandidates[i].content.length > MERGE_MAX_CHARS)
                     continue;
                 const related = [tenantCandidates[i]];
-                for (let j = i + 1; j < tenantCandidates.length; j++) {
-                    if (used.has(tenantCandidates[j].id))
-                        continue;
-                    const overlap = textOverlap(tenantCandidates[i].content, tenantCandidates[j].content);
-                    if (overlap >= MERGE_OVERLAP_THRESHOLD) {
+                for (const j of partnersOf(i)) {
+                    if (!used.has(tenantCandidates[j].id))
                         related.push(tenantCandidates[j]);
-                    }
                 }
                 const cluster = [];
                 let clusterChars = 0;
@@ -793,8 +790,8 @@ export async function consolidate(hippoRoot, options = {}) {
                                 },
                             });
                         }
-                        catch {
-                            // Best-effort — mirrors store.ts's audit() semantics.
+                        catch (error) {
+                            reportAuditWriteFailure('reject_refusal', String(error));
                         }
                         continue;
                     }
@@ -911,8 +908,9 @@ export async function consolidate(hippoRoot, options = {}) {
                                     : {},
                             });
                         }
-                        catch {
+                        catch (error) {
                             auditFailures++;
+                            reportAuditWriteFailure('mv_rescue', String(error), entry.id);
                         }
                     }
                     if (auditFailures > 0) {
@@ -965,7 +963,14 @@ function pickStrongestValence(entries) {
     }
     return 'neutral';
 }
-function detectConflicts(entries, now, decayOpts = {}, 
+/** Maps i to each j > i, ascending, whose text overlap with i reaches the merge threshold; every text needs at least one token. */
+export function mergePartners(contents) {
+    const sets = contents.map((text) => new Set(tokenize(text)));
+    const candidatesOf = overlapPartners(sets, jaccardMinShared(MERGE_OVERLAP_THRESHOLD));
+    return (i) => candidatesOf(i).filter((j) => jaccardSets(sets[i], sets[j]) >= MERGE_OVERLAP_THRESHOLD);
+}
+/** Pairs of live non-semantic memories that contradict each other, in survivor order. */
+export function detectConflicts(entries, now, decayOpts = {}, 
 // LC2-E3 (opt-in, default off): ids rescued by this cycle's decay pass.
 // detectConflicts recomputes its own strength>=DECAY_THRESHOLD survivor
 // filter independently of the decay pass above; without this bypass,
@@ -978,8 +983,10 @@ rescuedIds = new Set()) {
         && !isQuarantineScope(entry.scope ?? null)
         && (rescuedIds.has(entry.id) || calculateStrength(entry, now, decayOpts) >= DECAY_THRESHOLD));
     const detected = [];
+    const profiles = survivors.map((entry) => conflictProfile(entry.content));
+    const partnersOf = overlapPartners(profiles.map((p) => p.distinct), jaccardMinShared(CONFLICT_OVERLAP_THRESHOLD, CONFLICT_MIN_RARE_SHARED));
     for (let i = 0; i < survivors.length; i++) {
-        for (let j = i + 1; j < survivors.length; j++) {
+        for (const j of partnersOf(i)) {
             // Traces are variants of each other, not contradictions. Two
             // strategies for the same task can both be valid; conflict detection
             // exists for stated-rule disagreement, not strategy diversity.
@@ -989,7 +996,7 @@ rescuedIds = new Set()) {
                 continue;
             if ([survivors[i], survivors[j]].some((e) => e.tags.includes('extracted') || e.tags.includes('session-digest')))
                 continue;
-            const reasonAndScore = describeConflict(survivors[i], survivors[j]);
+            const reasonAndScore = describeConflict(profiles[i], profiles[j]);
             if (!reasonAndScore)
                 continue;
             detected.push({
@@ -1002,25 +1009,26 @@ rescuedIds = new Set()) {
     }
     return detected;
 }
+function conflictProfile(text) {
+    const opening = openingWindow(text);
+    // Polarity is measured only in the first POLARITY_WINDOW_WORDS, so a stray
+    // negation deep in a prose memory doesn't flip the intent.
+    // Pad with spaces so space-delimited patterns match words at the start/end.
+    return { distinct: distinctiveTokens(text), polarity: inferConflictPolarity(opening), window: ' ' + opening.toLowerCase() + ' ' };
+}
 function describeConflict(a, b) {
-    const aDistinct = distinctiveTokens(a.content);
-    const bDistinct = distinctiveTokens(b.content);
     // Jaccard on stopword-stripped tokens. Defer the threshold check until we
     // know whether an explicit polarity pair is present (lower bar for those).
-    const overlapScore = jaccardSets(aDistinct, bDistinct);
+    const overlapScore = jaccardSets(a.distinct, b.distinct);
     // Require at least N shared distinctive tokens so two short memories sharing
     // only "the project name" don't register.
     let shared = 0;
-    for (const t of aDistinct)
-        if (bDistinct.has(t))
+    for (const t of a.distinct)
+        if (b.distinct.has(t))
             shared++;
     if (shared < CONFLICT_MIN_RARE_SHARED)
         return null;
-    // Polarity is measured only in the first POLARITY_WINDOW_WORDS, so a stray
-    // negation deep in a prose memory doesn't flip the intent.
-    const polarityA = inferConflictPolarity(openingWindow(a.content));
-    const polarityB = inferConflictPolarity(openingWindow(b.content));
-    const conflictType = classifyConflictType(a.content, b.content, polarityA, polarityB);
+    const conflictType = classifyConflictType(a.window, b.window, a.polarity, b.polarity);
     if (!conflictType)
         return null;
     if (overlapScore < CONFLICT_OVERLAP_THRESHOLD)
@@ -1052,14 +1060,8 @@ function jaccardSets(a, b) {
 function openingWindow(text) {
     return text.split(/\s+/).slice(0, POLARITY_WINDOW_WORDS).join(' ');
 }
-function classifyConflictType(aText, bText, aPolarity, bPolarity) {
-    // Classifier scans only the opening window of each memory so " on " and
-    // " off " used as English prepositions deep in a long prose memory don't
-    // trigger an enabled/disabled flag. The opening window is where a rule or
-    // declaration is typically stated.
-    // Pad with spaces so space-delimited patterns match words at the start/end.
-    const a = ' ' + openingWindow(aText).toLowerCase() + ' ';
-    const b = ' ' + openingWindow(bText).toLowerCase() + ' ';
+// Takes opening windows only, so " on " and " off " as prepositions deep in long prose don't read as enabled/disabled.
+function classifyConflictType(a, b, aPolarity, bPolarity) {
     // Tightened tokens: require whole-word boundaries so " on " alone doesn't
     // match "on/off". Pair only `enabled` ↔ `disabled` and explicit on/off in
     // imperative context.

package/dist/customer-notes.js

@@ -19,6 +19,7 @@
  *
  * Lifecycle: active -> superseded (a corrected version) or active -> closed (retired).
  */
+import { BadRequestError, ConflictError, NotFoundError } from './api-errors.js';
 import { openHippoDb, closeHippoDb } from './db.js';
 import { writeEntry } from './store.js';
 import { assertTenantId } from './tenant.js';
@@ -47,21 +48,21 @@ export const MAX_CHANGE_SUMMARY_LEN = 4096;
 function validateNoteFields(customer, note, changeSummary) {
     const normalizedCustomer = (customer ?? '').trim();
     if (normalizedCustomer.length === 0)
-        throw new Error('saveCustomerNote: customer is required');
+        throw new BadRequestError('saveCustomerNote: customer is required');
     if (/[\r\n]/.test(normalizedCustomer)) {
-        throw new Error('saveCustomerNote: customer must be a single line (no newlines)');
+        throw new BadRequestError('saveCustomerNote: customer must be a single line (no newlines)');
     }
     if (normalizedCustomer.length > MAX_CUSTOMER_LEN) {
-        throw new Error(`saveCustomerNote: customer exceeds the ${MAX_CUSTOMER_LEN}-char cap`);
+        throw new BadRequestError(`saveCustomerNote: customer exceeds the ${MAX_CUSTOMER_LEN}-char cap`);
     }
     if (!note || note.trim().length === 0) {
-        throw new Error('saveCustomerNote: note is required');
+        throw new BadRequestError('saveCustomerNote: note is required');
     }
     if (note.length > MAX_NOTE_LEN) {
-        throw new Error(`saveCustomerNote: note exceeds the ${MAX_NOTE_LEN}-char cap`);
+        throw new BadRequestError(`saveCustomerNote: note exceeds the ${MAX_NOTE_LEN}-char cap`);
     }
     if (changeSummary !== undefined && changeSummary.length > MAX_CHANGE_SUMMARY_LEN) {
-        throw new Error(`saveCustomerNote: changeSummary exceeds the ${MAX_CHANGE_SUMMARY_LEN}-char cap`);
+        throw new BadRequestError(`saveCustomerNote: changeSummary exceeds the ${MAX_CHANGE_SUMMARY_LEN}-char cap`);
     }
     return { customer: normalizedCustomer };
 }
@@ -137,10 +138,10 @@ export function saveCustomerNote(hippoRoot, tenantId, opts, actor = 'cli') {
                 // shape for the matching row, or undefined when no note/tenant pair matches.
                 const pred = db.prepare(`SELECT status, version FROM customer_notes WHERE id = ? AND tenant_id = ?`).get(opts.supersedesNoteId, tenantId);
                 if (!pred) {
-                    throw new Error(`saveCustomerNote: note ${opts.supersedesNoteId} to supersede not found for tenant ${tenantId}`);
+                    throw new NotFoundError(`saveCustomerNote: note ${opts.supersedesNoteId} to supersede not found for tenant ${tenantId}`);
                 }
                 if (pred.status !== 'active') {
-                    throw new Error(`saveCustomerNote: note ${opts.supersedesNoteId} is not active (status='${pred.status}'); only active notes can be superseded.`);
+                    throw new ConflictError(`saveCustomerNote: note ${opts.supersedesNoteId} is not active (status='${pred.status}'); only active notes can be superseded.`);
                 }
                 version = pred.version + 1;
             }
@@ -158,7 +159,7 @@ export function saveCustomerNote(hippoRoot, tenantId, opts, actor = 'cli') {
           WHERE id = ? AND tenant_id = ? AND status = 'active' AND id != ?
         `).run(noteId, now, opts.supersedesNoteId, tenantId, noteId);
                 if (sup.changes === 0) {
-                    throw new Error(`saveCustomerNote: note ${opts.supersedesNoteId} could not be superseded (no longer active or self-reference).`);
+                    throw new ConflictError(`saveCustomerNote: note ${opts.supersedesNoteId} could not be superseded (no longer active or self-reference).`);
                 }
                 appendAuditEvent(db, {
                     tenantId,
@@ -221,9 +222,9 @@ export function closeCustomerNote(hippoRoot, tenantId, id, actor = 'cli') {
                 // shape, or undefined when the id/tenant pair doesn't exist.
                 const existing = db.prepare(`SELECT status FROM customer_notes WHERE id = ? AND tenant_id = ?`).get(id, tenantId);
                 if (!existing) {
-                    throw new Error(`closeCustomerNote: note ${id} not found for tenant ${tenantId}`);
+                    throw new NotFoundError(`closeCustomerNote: note ${id} not found for tenant ${tenantId}`);
                 }
-                throw new Error(`closeCustomerNote: note ${id} is not active (status='${existing.status}'); only active notes can be closed.`);
+                throw new ConflictError(`closeCustomerNote: note ${id} is not active (status='${existing.status}'); only active notes can be closed.`);
             }
             // SAFETY: SELECT ${NOTE_COLS} projects exactly the CustomerNoteRow columns;
             // .get() returns that row for the just-updated id, or undefined only in an
@@ -231,7 +232,7 @@ export function closeCustomerNote(hippoRoot, tenantId, id, actor = 'cli') {
             const row = db.prepare(`SELECT ${NOTE_COLS} FROM customer_notes WHERE id = ? AND tenant_id = ?`)
                 .get(id, tenantId);
             if (!row)
-                throw new Error(`closeCustomerNote: note ${id} not found after UPDATE`);
+                throw new NotFoundError(`closeCustomerNote: note ${id} not found after UPDATE`);
             appendAuditEvent(db, {
                 tenantId,
                 actor,
@@ -284,7 +285,7 @@ export function loadCustomerNotes(hippoRoot, tenantId, opts = {}) {
     assertTenantId('loadCustomerNotes', tenantId);
     const limit = opts.limit ?? 100;
     if (opts.status && !VALID_NOTE_STATES.has(opts.status)) {
-        throw new Error(`loadCustomerNotes: status must be one of ${Array.from(VALID_NOTE_STATES).join('|')}; got ${opts.status}`);
+        throw new BadRequestError(`loadCustomerNotes: status must be one of ${Array.from(VALID_NOTE_STATES).join('|')}; got ${opts.status}`);
     }
     const db = openHippoDb(hippoRoot);
     try {

package/dist/dag.js

@@ -2,6 +2,7 @@ import { createMemory, Layer } from './memory.js';
 import { writeEntry, loadAllDirtySummaries, loadChildrenOfSummary, applyRebuildResult, clearSummaryDirtyAfterBuild, } from './store.js';
 import { RejectedValueError } from './rejection.js';
 import { redactSecrets } from './secret-detect.js';
+import { fetchWithRetry, llmTimeoutMs } from './http-retry.js';
 import { derivationScope, derivationPartitionKey } from './recall-scope.js';
 import { loadConfig } from './config.js';
 import { neverAutoShareTags } from './shared.js';
@@ -52,7 +53,7 @@ export async function generateDagSummary(label, factContents, opts) {
         .replace('{facts}', factsBlock);
     let res;
     try {
-        res = await fetchFn('https://api.anthropic.com/v1/messages', {
+        res = await fetchWithRetry('https://api.anthropic.com/v1/messages', {
             method: 'POST',
             headers: {
                 'content-type': 'application/json',
@@ -64,7 +65,7 @@ export async function generateDagSummary(label, factContents, opts) {
                 max_tokens: 400,
                 messages: [{ role: 'user', content: prompt }],
             }),
-        });
+        }, { timeoutMs: llmTimeoutMs(), fetchFn });
     }
     catch (err) {
         opts.onError?.(`request failed: ${err instanceof Error ? err.message : String(err)}`);

package/dist/dashboard.js

@@ -29,7 +29,7 @@ function buildDashboardData(hippoRoot) {
     // render resolved conflicts as faded historical context, not just open
     // conflicts. The open_conflicts stat below still counts only 'open' rows
     // to preserve the existing badge meaning.
-    const conflicts = listMemoryConflicts(hippoRoot, '*');
+    const conflicts = listMemoryConflicts(hippoRoot, '*', tenantId);
     // D4 v1.12.10: tenant-scope peer discovery in the dashboard (matches the
     // tenantId already used for loadAllEntries on line 72).
     const peers = listPeers(undefined, tenantId);

package/dist/db.d.ts

@@ -10,6 +10,8 @@ export interface DatabaseSyncLike {
     exec(sql: string): void;
     prepare(sql: string): StatementSyncLike;
     close(): void;
+    readonly isOpen?: boolean;
+    readonly isTransaction?: boolean;
 }
 export declare function getHippoDbPath(hippoRoot: string): string;
 export declare function getCurrentSchemaVersion(): number;
@@ -17,6 +19,16 @@ export declare function getCurrentSchemaVersion(): number;
 export declare class IncompatibleBinaryError extends Error {
 }
 export declare function isSqliteBusy(error: unknown): boolean;
+export declare function execWithBusyRetry(db: DatabaseSyncLike, sql: string, timeoutMs?: number): void;
+/** Lock wait for hook commands: under the 5 s prompt-hook budget even after a few skipped writes, and far above a normal write's hold. */
+export declare const HOOK_DB_WAIT_MS = 1000;
+/** A busy store made a command skip work: warn once per process (the holder is usually `hippo sleep`). */
+export declare function noteStoreBusy(skipped: string): void;
+/** Runs `fn` with one handle per store: openHippoDb reuses it and closeHippoDb leaves it open until `fn` settles or the process exits.
+ *  `busyWaitMs` is the lock wait of every open inside `fn` that does not pass its own. */
+export declare function withSharedStoreHandles<T>(fn: () => T | Promise<T>, opts?: {
+    busyWaitMs?: number;
+}): Promise<T>;
 /** `busyWaitMs` shortens every lock wait of this open, for a hook that must finish inside its own timeout. */
 export declare function openHippoDb(hippoRoot: string, opts?: {
     busyWaitMs?: number;

package/dist/db.js

@@ -6,6 +6,7 @@ import { createPhysicsTable } from './physics-state.js';
 import { cleanupArchivedMirrors } from './raw-archive-mirror-cleanup.js';
 import { PACKAGE_VERSION, compareSemver } from './version.js';
 import { deriveOriginProject, originFromSource, isGlobalStoreRoot } from './project-identity.js';
+import { log } from './log.js';
 const require = createRequire(import.meta.url);
 // SAFETY: node:sqlite's DatabaseSync constructor genuinely has this shape at
 // runtime (Node's built-in synchronous SQLite module); there are no bundled
@@ -2607,7 +2608,7 @@ export function isSqliteBusy(error) {
 // busy_timeout covers neither of this file's two contended statements: SQLite
 // skips the busy handler for `PRAGMA journal_mode` and for a write that upgrades
 // a deferred read snapshot. Both need an explicit wait instead.
-function execWithBusyRetry(db, sql, timeoutMs = 30000) {
+export function execWithBusyRetry(db, sql, timeoutMs = 30000) {
     const deadline = Date.now() + timeoutMs;
     const idle = new Int32Array(new SharedArrayBuffer(4));
     for (;;) {
@@ -2622,8 +2623,66 @@ function execWithBusyRetry(db, sql, timeoutMs = 30000) {
         }
     }
 }
+// Hook commands run on every prompt, so inside withSharedStoreHandles each store pays its pragmas, migration check and mirror cleanup once.
+const sharedHandles = new Map();
+const sharedSet = new WeakSet();
+let shareDepth = 0;
+let shareBusyWaitMs;
+/** Lock wait for hook commands: under the 5 s prompt-hook budget even after a few skipped writes, and far above a normal write's hold. */
+export const HOOK_DB_WAIT_MS = 1000;
+/** A busy store made a command skip work: warn once per process (the holder is usually `hippo sleep`). */
+export function noteStoreBusy(skipped) {
+    log.once('store-busy', 'warn', `store busy (another hippo process holds the write lock); ${skipped}`);
+    // A lock held past one full wait belongs to a long transaction, so the hook's later writes skip at once.
+    for (const db of sharedHandles.values()) {
+        if (db.isOpen !== false)
+            db.exec('PRAGMA busy_timeout = 0');
+    }
+}
+function closeSharedStoreHandles() {
+    for (const db of sharedHandles.values()) {
+        sharedSet.delete(db);
+        if (db.isOpen !== false)
+            db.close();
+    }
+    sharedHandles.clear();
+}
+/** Runs `fn` with one handle per store: openHippoDb reuses it and closeHippoDb leaves it open until `fn` settles or the process exits.
+ *  `busyWaitMs` is the lock wait of every open inside `fn` that does not pass its own. */
+export async function withSharedStoreHandles(fn, opts) {
+    if (shareDepth++ === 0) {
+        process.once('exit', closeSharedStoreHandles);
+        shareBusyWaitMs = opts?.busyWaitMs;
+    }
+    try {
+        return await fn();
+    }
+    finally {
+        if (--shareDepth === 0) {
+            process.off('exit', closeSharedStoreHandles);
+            closeSharedStoreHandles();
+            shareBusyWaitMs = undefined;
+        }
+    }
+}
 /** `busyWaitMs` shortens every lock wait of this open, for a hook that must finish inside its own timeout. */
 export function openHippoDb(hippoRoot, opts) {
+    if (shareDepth === 0)
+        return openOwnHippoDb(hippoRoot, opts);
+    const busyWaitMs = opts?.busyWaitMs ?? shareBusyWaitMs;
+    const key = `${path.resolve(getHippoDbPath(hippoRoot))}\0${busyWaitMs ?? ''}`;
+    const shared = sharedHandles.get(key);
+    if (shared?.isOpen && !shared.isTransaction)
+        return shared;
+    const db = openOwnHippoDb(hippoRoot, { busyWaitMs });
+    // An open nested inside a transaction gets its own connection, as it did before sharing.
+    if (!shared?.isOpen) {
+        sharedHandles.set(key, db);
+        sharedSet.add(db);
+    }
+    return db;
+}
+function openOwnHippoDb(hippoRoot, opts) {
     fs.mkdirSync(hippoRoot, { recursive: true });
     const db = new DatabaseSync(getHippoDbPath(hippoRoot));
     const busyWaitMs = opts?.busyWaitMs;
@@ -2937,6 +2996,8 @@ function backfillFtsIndex(db) {
   `);
 }
 export function closeHippoDb(db) {
+    if (sharedSet.has(db))
+        return;
     db.close();
 }
 export function getMeta(db, key, fallback = '') {

package/dist/decisions.js

@@ -23,6 +23,7 @@
  * 'write_entry' (store.ts:1196) via the afterWrite hook, so a failure in any
  * step rolls all of them back. Pattern matches savePrediction (predictions.ts).
  */
+import { BadRequestError, ConflictError, NotFoundError } from './api-errors.js';
 import { openHippoDb, closeHippoDb } from './db.js';
 import { writeEntry } from './store.js';
 import { assertTenantId } from './tenant.js';
@@ -73,7 +74,7 @@ const DECISION_COLS = `
 export function saveDecision(hippoRoot, tenantId, opts, actor = 'cli') {
     assertTenantId('saveDecision', tenantId);
     if (!opts.decisionText)
-        throw new Error('saveDecision: decisionText is required');
+        throw new BadRequestError('saveDecision: decisionText is required');
     const now = new Date().toISOString();
     const content = opts.context
         ? `${opts.decisionText}\n\nContext: ${opts.context}`
@@ -103,10 +104,10 @@ export function saveDecision(hippoRoot, tenantId, opts, actor = 'cli') {
                 // SAFETY: row shape matches the single `status` column named in the SELECT below.
                 const pred = db.prepare(`SELECT status FROM decisions WHERE id = ? AND tenant_id = ?`).get(opts.supersedesDecisionId, tenantId);
                 if (!pred) {
-                    throw new Error(`saveDecision: decision ${opts.supersedesDecisionId} to supersede not found for tenant ${tenantId}`);
+                    throw new NotFoundError(`saveDecision: decision ${opts.supersedesDecisionId} to supersede not found for tenant ${tenantId}`);
                 }
                 if (pred.status !== 'active') {
-                    throw new Error(`saveDecision: decision ${opts.supersedesDecisionId} is not active (status='${pred.status}'); only active decisions can be superseded.`);
+                    throw new ConflictError(`saveDecision: decision ${opts.supersedesDecisionId} is not active (status='${pred.status}'); only active decisions can be superseded.`);
                 }
             }
             const result = db.prepare(`
@@ -127,7 +128,7 @@ export function saveDecision(hippoRoot, tenantId, opts, actor = 'cli') {
           WHERE id = ? AND tenant_id = ? AND status = 'active' AND id != ?
         `).run(decisionId, now, opts.supersedesDecisionId, tenantId, decisionId);
                 if (sup.changes === 0) {
-                    throw new Error(`saveDecision: decision ${opts.supersedesDecisionId} could not be superseded (no longer active or self-reference).`);
+                    throw new BadRequestError(`saveDecision: decision ${opts.supersedesDecisionId} could not be superseded (no longer active or self-reference).`);
                 }
                 appendAuditEvent(db, {
                     tenantId,
@@ -191,15 +192,15 @@ export function closeDecision(hippoRoot, tenantId, id, actor = 'cli') {
                 // SAFETY: row shape matches the single `status` column named in the SELECT above.
                 const existing = db.prepare(`SELECT status FROM decisions WHERE id = ? AND tenant_id = ?`).get(id, tenantId);
                 if (!existing) {
-                    throw new Error(`closeDecision: decision ${id} not found for tenant ${tenantId}`);
+                    throw new NotFoundError(`closeDecision: decision ${id} not found for tenant ${tenantId}`);
                 }
-                throw new Error(`closeDecision: decision ${id} is not active (status='${existing.status}'); only active decisions can be closed.`);
+                throw new ConflictError(`closeDecision: decision ${id} is not active (status='${existing.status}'); only active decisions can be closed.`);
             }
             // SAFETY: row's shape matches the columns named in DECISION_COLS above.
             const row = db.prepare(`SELECT ${DECISION_COLS} FROM decisions WHERE id = ? AND tenant_id = ?`)
                 .get(id, tenantId);
             if (!row)
-                throw new Error(`closeDecision: decision ${id} not found after UPDATE`);
+                throw new NotFoundError(`closeDecision: decision ${id} not found after UPDATE`);
             appendAuditEvent(db, {
                 tenantId,
                 actor,
@@ -256,7 +257,7 @@ export function loadDecisions(hippoRoot, tenantId, opts = {}) {
         let rows;
         if (opts.status) {
             if (!VALID_DECISION_STATES.has(opts.status)) {
-                throw new Error(`loadDecisions: status must be one of ${Array.from(VALID_DECISION_STATES).join('|')}; got ${opts.status}`);
+                throw new BadRequestError(`loadDecisions: status must be one of ${Array.from(VALID_DECISION_STATES).join('|')}; got ${opts.status}`);
             }
             // SAFETY: rows' shape matches the columns named in DECISION_COLS above.
             rows = db.prepare(`

package/dist/doctor.js

@@ -10,6 +10,7 @@ import * as path from 'node:path';
 import { findHippoStoreDir } from './project-identity.js';
 import { getGlobalRoot } from './shared.js';
 import { isInitialized } from './store.js';
+import { loadConfig } from './config.js';
 import { openHippoDbReadOnly, closeHippoDb, getSchemaVersion, getCurrentSchemaVersion, countTableRows, IncompatibleBinaryError } from './db.js';
 import { REPLAY_AFTER_MS, TRANSCRIPT_FILL_WINDOW_MS } from './compaction-record.js';
 import { isEmbeddingAvailable } from './embeddings.js';
@@ -208,6 +209,10 @@ export function runDoctor(opts) {
                 closeHippoDb(db);
         }
     }
+    const holdoutRateBp = store === null ? 0 : loadConfig(store).pilot.holdoutRateBp;
+    if (holdoutRateBp > 0) {
+        checks.push({ id: 'pilot', status: 'info', detail: `pilot holdout on: about ${holdoutRateBp / 100}% of sessions get no memories pushed by hippo (pilot.holdoutRateBp=${holdoutRateBp})` });
+    }
     const claudeDir = path.join(home, '.claude');
     if (fs.existsSync(claudeDir)) {
         const settings = readJson(path.join(claudeDir, 'settings.json'));

package/dist/embedding-provider.js

@@ -33,6 +33,7 @@
 import { getEmbedding, isEmbeddingAvailable, resolveEmbeddingModel, DEFAULT_EMBEDDING_MODEL, } from './embeddings.js';
 import { loadConfig } from './config.js';
 import { redactSecrets } from './secret-detect.js';
+import { fetchWithRetry } from './http-retry.js';
 export const API_PROVIDER_KINDS = ['openai', 'voyage', 'cohere'];
 function isApiProviderKind(x) {
     return x === 'openai' || x === 'voyage' || x === 'cohere';
@@ -184,15 +185,14 @@ class ApiEmbeddingProvider {
         const url = `${this.baseUrl.replace(/\/$/, '')}/${spec.path}`;
         let resp;
         try {
-            resp = await fetch(url, {
+            resp = await fetchWithRetry(url, {
                 method: 'POST',
                 headers: {
                     'content-type': 'application/json',
                     authorization: `Bearer ${key}`,
                 },
                 body: JSON.stringify(spec.buildBody(this.model, chunk.map(redactSecrets), role)),
-                signal: AbortSignal.timeout(REQUEST_TIMEOUT_MS),
-            });
+            }, { timeoutMs: REQUEST_TIMEOUT_MS });
         }
         catch (err) {
             const msg = err instanceof Error ? err.message : String(err);

package/dist/embeddings.d.ts

@@ -4,6 +4,7 @@
  * Falls back silently if the library is not installed.
  */
 import { MemoryEntry } from './memory.js';
+import { type EmbeddingProvider } from './embedding-provider.js';
 export declare const DEFAULT_EMBEDDING_MODEL = "Xenova/all-MiniLM-L6-v2";
 export declare const EMBEDDING_MODEL_META_KEY = "embedding_model";
 /**
@@ -114,8 +115,7 @@ export declare function getEmbedding(text: string, model?: string, role?: Embedd
  */
 export declare function cosineSimilarity(a: number[], b: number[]): number;
 /**
- * Load the cached embedding index from disk.
- * Returns an empty object if the file doesn't exist or is corrupt.
+ * Load the cached embedding index; `{}` when the file is missing. A corrupt file is moved aside and rebuilt on the next embed; any other read error throws, so nothing saves over an index it could not read.
  */
 export declare function loadEmbeddingIndex(hippoRoot: string): Record<string, number[]>;
 /**
@@ -129,7 +129,7 @@ export declare function embedMemory(hippoRoot: string, entry: MemoryEntry, model
 /**
  * Embed all entries in hippoRoot that don't already have cached vectors.
  * Prunes orphaned embeddings for memories that no longer exist.
- * Returns the count of newly embedded entries.
+ * Returns the count of newly embedded entries. `provider` defaults to the store's configured one.
  */
-export declare function embedAll(hippoRoot: string, model?: string): Promise<number>;
+export declare function embedAll(hippoRoot: string, model?: string, provider?: EmbeddingProvider): Promise<number>;
 //# sourceMappingURL=embeddings.d.ts.map