hippo-memory 1.70.0 → 1.72.0

package/dist/db/migrations/v32.js

@@ -52,10 +52,8 @@ const TRG_PROCESSES_TENANT_MATCH_UPDATE = `
             END;
           END
         `;
-// Cross-tenant safety vs the successor process (self-FK; verbatim mirror
-// of the v30 decisions supersede trigger). superseded_by is set only via
-// the supersede UPDATE; the successor must share the tenant. The successor
-// row already exists in the same transaction when this fires.
+// Cross-tenant safety vs the successor process (self-FK; mirrors the v30 supersede trigger): superseded_by is set only via the supersede UPDATE
+// and the successor, already present in the same transaction, must share the tenant.
 const TRG_PROCESSES_SUPERSEDE_TENANT_MATCH_UPDATE = `
           CREATE TRIGGER IF NOT EXISTS trg_processes_supersede_tenant_match_update
           BEFORE UPDATE ON processes
@@ -71,24 +69,8 @@ const TRG_PROCESSES_SUPERSEDE_TENANT_MATCH_UPDATE = `
 export const v32 = {
     version: 32,
     up: (db) => {
-        // Process first-class object.
-        // A process is a "living process map": a named, ordered list of steps that
-        // evolves. Unlike incident (open->resolved->closed, no supersede), process
-        // REUSES the v30 decisions supersede path as its delta mechanism: a process
-        // evolves by being superseded by a NEW VERSION that records what changed
-        // (change_summary) and the full new state (steps), carrying a derived
-        // version counter. So this table combines the v31 incidents tenant-match
-        // trigger pair (vs the referenced memory) WITH the v30 decisions
-        // superseded_by self-FK + supersede tenant-match trigger.
-        //
-        // status (active|superseded|closed): superseded carries a self-FK
-        // superseded_by to the successor version; closed is a terminal
-        // retire-without-successor (only an active head closes). The memory mirror
-        // is kept for recall but is not authoritative; memory_id is NULLABLE with
-        // ON DELETE SET NULL so forget/consolidate/archive does not lose a process.
-        // steps is a JSON-encoded array of step strings (scoped v1; a normalized
-        // process_steps table is deferred). version is server-derived
-        // (predecessor.version + 1); change_summary is set on a successor row only.
+        // Process first-class object (living process map): it evolves via the v30 supersede path, each version recording change_summary and full steps (JSON).
+        // Combines the v31 tenant-match trigger pair with the v30 superseded_by self-FK; version is server-derived (predecessor + 1); only an active head closes.
         if (!tableExists(db, 'processes')) {
             db.exec(PROCESSES_TABLE);
             db.exec(IDX_PROCESSES_TENANT_STATUS);

package/dist/db/migrations/v33.js

@@ -75,21 +75,8 @@ const TRG_POLICIES_SUPERSEDE_TENANT_MATCH_UPDATE = `
 export const v33 = {
     version: 33,
     up: (db) => {
-        // Policy first-class object.
-        // The "bi-temporal-first" object type: a named rule/statement that is in
-        // force over an EFFECTIVE-TIME range (valid_from required, valid_to nullable
-        // = open-ended) and evolves via the v32 processes supersede machinery
-        // (superseded_by self-FK + supersede tenant-match trigger + version +
-        // change_summary). This table = the v32 processes table MINUS `steps`
-        // (a policy has policy_text, not an ordered step list) PLUS the first-class
-        // effective-time columns valid_from/valid_to. Valid-time is the queryable
-        // axis (the as-of query loadPoliciesAsOf); transaction-time is present via
-        // created_at + the supersede chain's superseded_at (time-travel deferred).
-        //
-        // All date inputs are normalized to standard ISO-8601 datetime
-        // (toISOString) at the store boundary before persist/compare, so the
-        // fixed-width values sort lexically and the half-open [valid_from, valid_to)
-        // as-of comparison is correct.
+        // Policy first-class object: a rule in force over an effective-time range (valid_to nullable = open-ended); supersedes like v32 processes.
+        // Dates are ISO-8601 (toISOString) at the store boundary so values sort lexically and the half-open [valid_from, valid_to) as-of comparison is correct.
         if (!tableExists(db, 'policies')) {
             db.exec(POLICIES_TABLE);
             db.exec(IDX_POLICIES_TENANT_STATUS);

package/dist/db/migrations/v34.js

@@ -69,17 +69,8 @@ const TRG_SKILLS_SUPERSEDE_TENANT_MATCH_UPDATE = `
 export const v34 = {
     version: 34,
     up: (db) => {
-        // Skill first-class object.
-        // A skill is a reusable, agent-followable capability: an `instructions` body
-        // + an optional `trigger_text` (when to apply), evolving via the v32
-        // processes supersede machinery (superseded_by self-FK + supersede
-        // tenant-match trigger + version + change_summary). This table = the v32
-        // processes table MINUS `steps` (a skill's content is a single instructions
-        // body) PLUS `instructions` (NOT NULL) and `trigger_text`. "Executable" is
-        // scoped to an agent-followable instruction that EXPORTS into the agent's
-        // in-force rules (AGENTS.md / CLAUDE.md) via exportSkills; literal code
-        // execution is deferred. NOTE: the column is `trigger_text`, NOT `trigger`,
-        // because TRIGGER is a SQLite reserved keyword.
+        // Skill first-class object: `instructions` plus optional `trigger_text`, superseding like v32 processes; exportSkills writes it to AGENTS.md / CLAUDE.md.
+        // The column is `trigger_text`, not `trigger`, because TRIGGER is a SQLite reserved keyword.
         if (!tableExists(db, 'skills')) {
             db.exec(SKILLS_TABLE);
             db.exec(IDX_SKILLS_TENANT_STATUS);

package/dist/db/migrations/v35.js

@@ -72,17 +72,8 @@ const TRG_PROJECT_BRIEFS_SUPERSEDE_TENANT_MATCH_UPDATE = `
 export const v35 = {
     version: 35,
     up: (db) => {
-        // project_brief first-class object. A project_brief is the
-        // living, repo-scoped summary of a repository's state: a `summary` body
-        // scoped to a `repo`, evolving via the v34 skills supersede machinery
-        // (superseded_by self-FK + supersede tenant-match trigger + version +
-        // change_summary). This table = the v34 skills table with
-        // skill_name/trigger_text replaced by `repo` (the repo-scoping dimension)
-        // PLUS `summary` (the brief body). The distinguishing op (refreshBrief, in
-        // src/objects/project-briefs.ts) auto-assembles the summary from the repo's receipts
-        // (memory rows tagged path:<repo>); it needs no schema support beyond `repo`.
-        // All column names were checked against SQLite reserved words:
-        // repo/summary/version/status/etc. are non-reserved.
+        // project_brief first-class object: repo-scoped `summary` superseding like v34 skills; refreshBrief (objects/project-briefs.ts) builds it from receipts.
+        // The schema needs only `repo`; column names were checked against SQLite reserved words.
         if (!tableExists(db, 'project_briefs')) {
             db.exec(PROJECT_BRIEFS_TABLE);
             db.exec(IDX_PROJECT_BRIEFS_TENANT_STATUS);

package/dist/db/migrations/v36.js

@@ -72,17 +72,8 @@ const TRG_CUSTOMER_NOTES_SUPERSEDE_TENANT_MATCH_UPDATE = `
 export const v36 = {
     version: 36,
     up: (db) => {
-        // customer_note first-class object. A customer_note is a
-        // discrete note recorded against an account/customer entity, evolving via the
-        // v35 project_briefs supersede machinery (superseded_by self-FK + supersede
-        // tenant-match trigger + version + change_summary). This table = the v35
-        // project_briefs table with repo/summary replaced by `customer` (the
-        // entity-scoping dimension; a free-form account/customer id - the entities
-        // table is unbuilt, so a FK is deferred) PLUS `note` (the note body).
-        // MANY notes per customer (each its own supersede chain), unlike the
-        // one-summary-per-repo project_brief. All column names checked against SQLite
-        // reserved words: customer/note/
-        // version/status/etc. are non-reserved.
+        // customer_note first-class object: a note on a customer, superseding like v35 project_briefs; `customer` is a free-form id (no FK: no entities table).
+        // Many notes per customer, each its own supersede chain. Column names were checked against SQLite reserved words.
         if (!tableExists(db, 'customer_notes')) {
             db.exec(CUSTOMER_NOTES_TABLE);
             db.exec(IDX_CUSTOMER_NOTES_TENANT_STATUS);

package/dist/db/migrations/v37.js

@@ -41,9 +41,8 @@ const GRAPH_EXTRACTION_QUEUE_TABLE = `
             FOREIGN KEY (memory_id) REFERENCES memories(id) ON DELETE CASCADE
           )
         `;
-// entities guard: source_kind must equal the FK'd memory's actual kind (so a
-// raw memory or a lying source_kind both ABORT), and tenant must match. Both
-// INSERT and UPDATE (UPDATE fires when memory_id/source_kind/tenant_id change).
+// entities guard: source_kind must equal the FK'd memory's actual kind (raw or a lying source_kind ABORTs) and tenant must match,
+// on INSERT and on UPDATE (when memory_id/source_kind/tenant_id change).
 const TRG_ENTITIES_CONSOLIDATED_ONLY_INSERT = `
           CREATE TRIGGER IF NOT EXISTS trg_entities_consolidated_only_insert
           BEFORE INSERT ON entities
@@ -139,16 +138,8 @@ const TRG_GRAPH_QUEUE_CONSOLIDATED_ONLY_UPDATE = `
             END;
           END
         `;
-// Reverse guard: the graph-table triggers
-// only fire on writes to the GRAPH tables. They do NOT fire when an
-// already-indexed memory is later mutated. So 'UPDATE memories SET kind=raw'
-// (or a tenant change) on a memory the graph references would silently leave
-// entity/relation/queue rows pointing at a raw / cross-tenant memory while
-// their source_kind stays 'distilled' - bypassing the central 'graph never
-// indexes raw' invariant after insertion. This trigger closes that direction:
-// a memory cannot be reclassified to raw, nor moved cross-tenant, WHILE the
-// graph references it (rebuild/remove the graph rows first). Cheap: the EXISTS
-// checks are only evaluated when kind actually becomes raw or tenant changes.
+// Reverse guard: graph triggers fire only on graph-table writes, so reclassifying an indexed memory to raw (or tenant) would break the no-raw invariant.
+// Block both while the graph references the memory; the EXISTS checks run only when kind becomes raw or tenant changes.
 const TRG_MEMORIES_GRAPH_REFERENCED_GUARD = `
           CREATE TRIGGER IF NOT EXISTS trg_memories_graph_referenced_guard
           BEFORE UPDATE ON memories
@@ -162,13 +153,8 @@ const TRG_MEMORIES_GRAPH_REFERENCED_GUARD = `
             SELECT RAISE(ABORT, 'cannot change the kind or tenant of a memory while the graph references it (E3.3 graph-on-consolidated guard); a graph-referenced memory is immutable in kind/tenant - rebuild/remove the graph rows first, or rebuild them after supersession');
           END
         `;
-// Second reverse guard: an entity that is
-// a relation endpoint cannot be moved cross-tenant. The entity UPDATE trigger
-// validates the entity against its source memory, but an existing relation
-// pointing at the entity is NOT re-validated, so a raw 'UPDATE entities SET
-// tenant_id=?, memory_id=?' to another tenant would leave a tenant-A relation
-// pointing at a tenant-B entity. Block the tenant move while the entity is
-// referenced by any relation (rebuild the relations first).
+// Second reverse guard: the entity UPDATE trigger does not re-validate existing relations, so a cross-tenant entity move would leave a tenant-A relation
+// pointing at a tenant-B entity. Block the tenant move while any relation references the entity.
 const TRG_ENTITIES_NO_TENANT_MOVE_WHEN_REFERENCED = `
           CREATE TRIGGER IF NOT EXISTS trg_entities_no_tenant_move_when_referenced
           BEFORE UPDATE ON entities
@@ -181,19 +167,8 @@ const TRG_ENTITIES_NO_TENANT_MOVE_WHEN_REFERENCED = `
 export const v37 = {
     version: 37,
     up: (db) => {
-        // Graph-on-consolidated guard.
-        // The graph layer (entities + relations) sits ON TOP OF consolidated state and
-        // must NEVER index the raw layer. The substrate: entities + relations +
-        // graph_extraction_queue, each FK-ing to memories and guarded so they can only
-        // reference CONSOLIDATED memories (kind IN ('distilled','superseded')), never
-        // kind='raw'. New tables -> real CHECK constraints (unlike the ALTER'd memories,
-        // whose kind CHECK lives in triggers). The kind/source MATCH (source_kind ==
-        // the FK'd memory's actual kind) cannot be a CHECK (CHECK can't subquery), so it
-        // is a BEFORE INSERT *and* BEFORE UPDATE trigger (the subquery-capable pattern
-        // from the v30 decisions / predictions tenant-match triggers). Both INSERT and
-        // UPDATE are guarded: an INSERT-only guard is bypassable via a raw SQL UPDATE
-        // that moves a row onto a raw memory. All column
-        // names checked vs SQL reserved words: rel_type avoids REFERENCES.
+        // Graph-on-consolidated guard: entities, relations and graph_extraction_queue may reference only consolidated memories (distilled|superseded), never raw.
+        // The kind match needs a subquery, so it is a BEFORE INSERT and UPDATE trigger (INSERT-only is bypassable). rel_type avoids the reserved word REFERENCES.
         if (!tableExists(db, 'entities')) {
             db.exec(ENTITIES_TABLE);
             db.exec(`CREATE INDEX IF NOT EXISTS idx_entities_tenant ON entities(tenant_id)`);

package/dist/db/migrations/v38.js

@@ -33,12 +33,8 @@ const RELATIONS_TABLE = `
           FOREIGN KEY (memory_id) REFERENCES memories(id) ON DELETE SET NULL
         )
       `;
-// entities guard (dual-provenance): at least one valid provenance, no raw.
-//  - all-null  -> ABORT.
-//  - memory path (memory_id NOT NULL): source_kind == the FK'd memory's kind
-//    (raw / lying source_kind ABORT) AND tenant-match.
-//  - object path (memory_id NULL): the (type,id) points at an EXISTING same-tenant
-//    object row whose status is active|superseded (explicit 4-way CASE per table).
+// entities guard (dual-provenance): at least one valid provenance, no raw. Memory path: source_kind must equal the FK'd memory's kind, plus tenant-match.
+// Object path (memory_id NULL): (type,id) must point at an existing same-tenant object with status active|superseded (4-way CASE per table); all-null aborts.
 const TRG_ENTITIES_CONSOLIDATED_ONLY_INSERT = `
         CREATE TRIGGER IF NOT EXISTS trg_entities_consolidated_only_insert
         BEFORE INSERT ON entities
@@ -201,10 +197,7 @@ const TRG_RELATIONS_CONSOLIDATED_ONLY_UPDATE = `
           END;
         END
       `;
-// Recreated VERBATIM from v37 (logic unchanged): an entity that is a relation
-// endpoint cannot be moved cross-tenant while referenced. trg_memories_graph_referenced_guard
-// (on memories) and trg_graph_queue_* (on graph_extraction_queue) are NOT recreated
-// here: those tables are not dropped by v38, so the triggers survive.
+// Recreated verbatim from v37 (relation endpoints cannot move cross-tenant). The memories and graph queue triggers survive: those tables are not dropped.
 const TRG_ENTITIES_NO_TENANT_MOVE_WHEN_REFERENCED = `
         CREATE TRIGGER IF NOT EXISTS trg_entities_no_tenant_move_when_referenced
         BEFORE UPDATE ON entities
@@ -217,34 +210,8 @@ const TRG_ENTITIES_NO_TENANT_MOVE_WHEN_REFERENCED = `
 export const v38 = {
     version: 38,
     up: (db) => {
-        // Object provenance: anchor graph entity/relation provenance to the authoritative
-        // first-class object (decision/policy/customer-note/project-brief) instead of the
-        // decaying memory mirror. An in-force object must STAY in the graph after its mirror memory is forgotten or
-        // consolidation-pruned. The graph is a PURE DERIVED CACHE (clearGraph + rebuild on
-        // every `graph extract` / `sleep`), so v38 DROPs+recreates entities/relations (no
-        // data copy) and the next extract repopulates. graph_extraction_queue is untouched.
-        //
-        // Two provenance paths, "at least one, no raw":
-        //  - memory path (memory_id NOT NULL): source_kind must equal the FK'd memory's live
-        //    kind and that kind is distilled|superseded (raw still ABORTs) + tenant-match.
-        //  - object path (memory_id NULL): source_object_type/id must reference an EXISTING
-        //    same-tenant object row whose status is active|superseded (not closed). Such objects
-        //    are consolidated BY CONSTRUCTION, so the no-raw invariant still holds.
-        //  - all-null is rejected.
-        //
-        // memory_id is now NULLABLE with ON DELETE SET NULL (was NOT NULL / CASCADE), so a
-        // mirror forget/consolidate nulls the recall pointer without dropping the row. NOTE
-        // (empirically verified, contradicts the SQLite docs): node:sqlite DOES fire the
-        // BEFORE UPDATE guard from the FK SET NULL action even with recursive_triggers OFF.
-        // So the *_consolidated_only_UPDATE triggers deliberately OMIT the all-null ABORT
-        // (kept on INSERT) - otherwise a mirror delete of a memory-only row would be blocked.
-        // See the per-trigger comments below. A SET NULL leaves source_kind at its old value
-        // by design (the object path is distilled-by-construction; source_kind is only
-        // re-checked when memory_id NOT NULL).
-        // source_object_id is a SOFT (type,id) pointer (no hard FK) the rebuild re-validates,
-        // so a legitimate object hard-delete is never blocked; a `closed` object row drops at next
-        // extract. SQLite cannot parametrize a table name in a trigger, so the object-path
-        // validation is an explicit 4-way CASE (one arm per object table).
+        // Anchor graph provenance to the authoritative object, not the decaying mirror memory; the graph is a derived cache, so entities/relations are recreated.
+        // memory_id is NULLABLE ON DELETE SET NULL; node:sqlite fires the BEFORE UPDATE guard on that SET NULL, so the UPDATE triggers omit the all-null ABORT.
         recreateGraphTables(db);
         createGraphGuards(db);
     },

package/dist/db/migrations/v39.js

@@ -6,25 +6,17 @@ import { tableHasColumn } from '../tables.js';
 export const v39 = {
     version: 39,
     up: (db, ctx) => {
-        // Memory scope isolation.
-        // origin_project: '<name>' = owned by that project, '' = user-global,
+        // Memory scope isolation. origin_project: '<name>' = owned by that project, '' = user-global,
         // NULL = legacy/unknown (ambient context treats NULL as deny).
         if (!tableHasColumn(db, 'memories', 'origin_project')) {
             db.exec(`ALTER TABLE memories ADD COLUMN origin_project TEXT`);
         }
-        // Backfill on store-location evidence only (no path-tag guessing):
-        // 1. Rows shared from a project carry source 'shared:<project>:<ts>' -
-        //    take <project>; a share whose <project> is the home dir basename
-        //    maps to '' (user-global).
-        // 2. Every other row was written into THIS store, so it takes the
-        //    store's own origin: `<project>/.hippo` -> '<project>', the
-        //    home/global store -> '' (user-global).
-        // Rows stay NULL only when no hippoRoot was provided.
+        // Backfill from store location only (no path-tag guessing): 'shared:<project>:<ts>' rows take <project> (home-dir basename maps to '' user-global);
+        // every other row takes this store's origin (`<project>/.hippo` -> '<project>', global store -> ''). Rows stay NULL only when no hippoRoot was provided.
         const hippoRoot = ctx?.hippoRoot;
         if (hippoRoot) {
-            // Provenance-source evidence first (shared:<project>: / promoted:<localRoot>,
-            // parsed by the same helper the markdown-import stamp uses), then the
-            // store's own location for everything else.
+            // Provenance-source evidence first (shared:<project>: / promoted:<localRoot>, parsed by the markdown-import stamp's helper),
+            // then the store's own location for everything else.
             const homeName = path.basename(os.homedir()).toLowerCase();
             // SAFETY: sourcedRows' shape matches the two columns (id, source)
             // named in the SELECT above.
@@ -36,18 +28,15 @@ export const v39 = {
                     continue;
                 setOrigin.run(origin, row.id);
             }
-            // The global root itself is ALWAYS user-global (''), regardless of
-            // what surrounds it on disk - a HIPPO_HOME inside a dotfiles git
-            // repo must not stamp the whole corpus with that repo's name.
+            // The global root itself is always user-global (''), whatever surrounds it on disk: a HIPPO_HOME inside a dotfiles git repo
+            // must not stamp the whole corpus with that repo's name.
             const storeOrigin = isGlobalStoreRoot(hippoRoot)
                 ? ''
                 : deriveOriginProject(path.dirname(hippoRoot));
             db.prepare(`UPDATE memories SET origin_project = ? WHERE origin_project IS NULL`).run(storeOrigin);
         }
-        // Rollback-safety guard (v24 precedent): a pre-isolation binary opening
-        // this DB would ignore origin_project and the secret veto and resume
-        // injecting cross-project rows. 1.24.0 is the first version with the
-        // isolation behavior. Forward-only - never lower an existing minimum.
+        // Rollback-safety guard: a pre-isolation binary would ignore origin_project and the secret veto and resume injecting cross-project rows.
+        // 1.24.0 is the first version with isolation; forward-only, never lower an existing minimum.
         raiseMinBinary(db, '1.24.0');
     },
 };

package/dist/db/migrations/v40.js

@@ -48,15 +48,8 @@ const RECALL_TRACES_SCHEMA_SQL = `
 export const v40 = {
     version: 40,
     up: (db) => {
-        // Retrieval-trace persistence.
-        // Follows the v18 goal_recall_log style: one parent trace row per
-        // recall, a WITHOUT ROWID child table for the ranked results, and a
-        // separate append-only outcomes table so audit_log pruning can never
-        // erase training data.
-        //
-        // No min_compatible_binary bump, on purpose: an old binary ignores these tables and
-        // recordTraceOutcome re-validates trace ids, so its stale writes are skipped; a bump
-        // would lock old binaries out of the whole store for an observability feature.
+        // Retrieval-trace persistence: one trace row per recall, a WITHOUT ROWID results table, append-only outcomes (audit_log pruning must not erase them).
+        // No min_compatible_binary bump on purpose: old binaries ignore these tables and recordTraceOutcome re-validates trace ids; a bump would lock them out.
         db.exec(RECALL_TRACES_SCHEMA_SQL);
     },
 };

package/dist/db/migrations/v41.js

@@ -1,30 +1,8 @@
 export const v41 = {
     version: 41,
     up: (db) => {
-        // Rejected-value tombstone.
-        // Additive table, template = v40 above. A human who rejects a fact gets
-        // a durable say: the write-path guard in upsertEntryRow refuses any
-        // write that would re-introduce a value whose normalized digest
-        // matches a row here.
-        //
-        // No raw content and no preview stored. Follows the raw_archive
-        // redaction precedent (raw-archive.ts payload is {redacted:true, ...}):
-        // a rejected value may itself be a secret or PII ("never store my key
-        // again") — persisting it in the tombstone would defeat the point. The
-        // human sees the content at reject time (CLI echoes it); afterwards
-        // `reason` is the human-readable identity.
-        //
-        // No FK on source_memory_id (v40 precedent above: tombstone outlives
-        // the row it was sourced from) — provenance only.
-        //
-        // Reserved-word check on column names:
-        // tenant/digest/reason/rejected/source/normalized/chars are non-reserved.
-        //
-        // No min_compatible_binary bump, a deliberate tradeoff: an old binary sharing a synced store
-        // writes WITHOUT the guard until upgraded; documented in
-        // MEMORY_ENVELOPE.md rather than hard-locking every old binary out of
-        // the store, which is disproportionate for the dominant single-user
-        // single-binary deployment.
+        // Rejected-value tombstone: upsertEntryRow refuses writes matching a row's digest. No raw content stored (may be secret); no FK: it outlives its row.
+        // No min_compatible_binary bump, a deliberate tradeoff: an old binary writes without the guard until upgraded (see MEMORY_ENVELOPE.md).
         db.exec(`
         CREATE TABLE IF NOT EXISTS rejected_values (
           tenant_id  TEXT NOT NULL,

package/dist/db/migrations/v43.js

@@ -61,7 +61,7 @@ const CARDS_SCHEMA_SQL = `
 export const v43 = {
     version: 43,
     up: (db) => {
-        // W2a work-queue cards (trajectories/01M2D5VSYJFK4YXQ0RG2NGCPYJ/plan.md). Additive only, v41 precedent.
+        // W2a work-queue cards. Additive only, v41 precedent.
         db.exec(CARDS_SCHEMA_SQL);
     },
 };

package/dist/db/migrations/v44.js

@@ -1,14 +1,8 @@
 export const v44 = {
     version: 44,
     up: (db) => {
-        // Dormant memories (src/store/dormant.ts): with `dormant.enabled`, the sleep
-        // decay pass moves a faded memory here instead of deleting it. The row
-        // leaves `memories` in the same transaction, so recall, context and
-        // every sleep pass stop seeing it; entry_json is the full MemoryEntry
-        // snapshot `hippo dormant restore` writes back. No FK to memories (the
-        // memories row is gone by design). Additive only, v41 precedent: no
-        // min_compatible_binary bump. An older binary ignores the table and
-        // keeps deleting faded memories as it always did.
+        // Dormant memories (src/store/dormant.ts): with `dormant.enabled`, sleep moves a faded memory here; `hippo dormant restore` writes entry_json back.
+        // No FK to memories (the row is gone by design). Additive only: no min_compatible_binary bump, an older binary ignores the table and keeps deleting.
         db.exec(`
         CREATE TABLE IF NOT EXISTS dormant_memories (
           tenant_id  TEXT NOT NULL DEFAULT 'default',

package/dist/db/migrations/v45.js

@@ -1,13 +1,8 @@
 export const v45 = {
     version: 45,
     up: (db) => {
-        // Token ledger (src/store/token-ledger.ts): one row per block of
-        // memory text hippo hands an agent (hook, CLI, MCP, HTTP). `event` is 'inject'
-        // (sent), 'skip' (unchanged since the session's last inject, not sent), 'reset'
-        // (compaction dropped earlier injections, so the next one must be sent) or
-        // 'reread' (re-read by later calls, booked at session end per call day). block_hash
-        // lets the per-prompt hook skip an unchanged block. Rows older than the retention
-        // window are pruned on write. Additive only: no min_compatible_binary bump.
+        // Token ledger (src/store/token-ledger.ts): one row per block of memory text handed to an agent; `event` is inject, skip, reset (compaction) or reread.
+        // block_hash lets the per-prompt hook skip an unchanged block; rows past retention are pruned on write. Additive: no min_compatible_binary bump.
         db.exec(`
         CREATE TABLE IF NOT EXISTS token_ledger (
           id         INTEGER PRIMARY KEY AUTOINCREMENT,

package/dist/db/migrations/v47.js

@@ -1,9 +1,8 @@
 export const v47 = {
     version: 47,
     up: (db) => {
-        // Scope grants (src/store/auth.ts): a member key reads a
-        // restricted scope only by an explicit row here. Additive only: no
-        // min_compatible_binary bump.
+        // Scope grants (src/store/auth.ts): a member key reads a restricted scope only by an explicit row here.
+        // Additive only: no min_compatible_binary bump.
         db.exec(`
         CREATE TABLE IF NOT EXISTS api_key_scope_grants (
           key_id     TEXT NOT NULL,

package/dist/db/migrations/v51.js

@@ -1,7 +1,7 @@
 export const v51 = {
     version: 51,
     up: (db) => {
-        // Both indexes serve loadAmbientCandidates (src/store.ts); their WHERE text must match its SQL for the planner to pick them.
+        // Both indexes serve loadAmbientCandidates (src/store/candidates.ts); their WHERE text must match its SQL for the planner to pick them.
         // The drift index stays empty on a healthy store, so the drift probe is one index seek instead of a tenant scan. Additive only.
         db.exec(`
         CREATE INDEX IF NOT EXISTS idx_memories_pinned

package/dist/db/migrations/v55.d.ts

@@ -0,0 +1,3 @@
+import type { Migration } from './types.js';
+export declare const v55: Migration;
+//# sourceMappingURL=v55.d.ts.map

package/dist/db/migrations/v55.js

@@ -0,0 +1,17 @@
+import { tableHasColumn, tableExists } from '../tables.js';
+export const v55 = {
+    version: 55,
+    up: (db) => {
+        // No backfill and no floor raise: an older binary names its INSERT columns, so it still writes.
+        if (!tableExists(db, 'delivery_events'))
+            return;
+        if (!tableHasColumn(db, 'delivery_events', 'tool_use_id'))
+            db.exec('ALTER TABLE delivery_events ADD COLUMN tool_use_id TEXT');
+        if (!tableHasColumn(db, 'delivery_events', 'failure_id'))
+            db.exec('ALTER TABLE delivery_events ADD COLUMN failure_id INTEGER');
+        db.exec(`CREATE UNIQUE INDEX IF NOT EXISTS idx_delivery_events_tool_call
+        ON delivery_events(tenant_id, session_id, tool_use_id)
+        WHERE tool_use_id IS NOT NULL AND turn_seq IS NOT NULL`);
+    },
+};
+//# sourceMappingURL=v55.js.map

package/dist/db/open.d.ts

@@ -24,8 +24,6 @@ export declare function withSqliteAllowed<T>(fn: () => T): T;
 export declare function withSqliteOffLoop<T>(fn: () => T): T;
 /** Leaves withSqliteOffLoop's block and nothing else: another store's block and the folder marker still refuse, which withSqliteAllowed would waive. */
 export declare function outsideSqliteOffLoop<T>(fn: () => T): T;
-/** First line of a best-effort catch around a hippo.db open: an unported path must fail closed, not fall back silently. */
-export declare function rethrowIfSqliteBlocked<E>(err: E): void;
 /** store init --db, store copy --db and serve --db write this file in the hippo root; its text is the store kind. */
 export declare const OTHER_STORE_MARKER = "other-store";
 /** Throws where hippo.db must not open; checked on every open, not cached, since the marker can appear while a process runs. */
@@ -40,5 +38,7 @@ export declare function openHippoDbWithFacts(hippoRoot: string, opts?: {
 }): OpenedDb;
 /** Open an existing store without changing it: no mkdir, WAL switch, migration or mirror cleanup. Throws when hippo.db is missing. */
 export declare function openHippoDbReadOnly(hippoRoot: string): DatabaseSyncLike;
+/** Runs `fn` on an existing hippo.db opened without migrations, for a repair of a store whose schema is behind; closes it after. */
+export declare function withUnmigratedDb<T>(hippoRoot: string, readOnly: boolean, fn: (db: DatabaseSyncLike) => T): T;
 export declare function closeHippoDb(db: DatabaseSyncLike): void;
 //# sourceMappingURL=open.d.ts.map

package/dist/db/open.js

@@ -1,10 +1,11 @@
 import { AsyncLocalStorage } from 'node:async_hooks';
 import { existsSync, readFileSync } from 'node:fs';
 import { join } from 'node:path';
-import { errorMessage, log } from '../util/log.js';
+import { errorFields, errorMessage, log } from '../util/log.js';
 import { DatabaseSync } from './sqlite.js';
 import { tableExists } from './tables.js';
 import { assertBinaryCompatible } from './migrate.js';
+import { DEFAULT_BUSY_WAIT_MS } from './busy.js';
 import { connectWithFacts, getHippoDbPath } from './connect.js';
 import { currentRequestStores, isScopedHandle, runWithRequestStores } from './request-stores.js';
 import { OtherStoreFolderError, SqliteBlockedError } from '../util/sqlite-blocked.js';
@@ -49,11 +50,6 @@ export function withSqliteOffLoop(fn) {
 export function outsideSqliteOffLoop(fn) {
     return sqliteBlockedBy.getStore() === OFF_LOOP ? sqliteBlockedBy.exit(fn) : fn();
 }
-/** First line of a best-effort catch around a hippo.db open: an unported path must fail closed, not fall back silently. */
-export function rethrowIfSqliteBlocked(err) {
-    if (err instanceof SqliteBlockedError)
-        throw err;
-}
 /** store init --db, store copy --db and serve --db write this file in the hippo root; its text is the store kind. */
 export const OTHER_STORE_MARKER = 'other-store';
 const markerWaived = new AsyncLocalStorage();
@@ -95,7 +91,7 @@ export function openHippoDbReadOnly(hippoRoot) {
     assertSqliteAllowed(hippoRoot);
     const db = new DatabaseSync(getHippoDbPath(hippoRoot), { readOnly: true });
     try {
-        db.exec('PRAGMA busy_timeout = 5000');
+        db.exec(`PRAGMA busy_timeout = ${DEFAULT_BUSY_WAIT_MS}`);
         if (tableExists(db, 'meta'))
             assertBinaryCompatible(db);
         return db;
@@ -104,12 +100,28 @@ export function openHippoDbReadOnly(hippoRoot) {
         try {
             db.close();
         }
-        catch {
-            // Best effort only.
+        catch (closeErr) {
+            log.error(`openHippoDbReadOnly: closing the handle after a failed open failed: ${errorMessage(closeErr)}`, errorFields(closeErr));
         }
         throw error;
     }
 }
+/** Runs `fn` on an existing hippo.db opened without migrations, for a repair of a store whose schema is behind; closes it after. */
+export function withUnmigratedDb(hippoRoot, readOnly, fn) {
+    // Skips openHippoDb, so it takes the same refusal itself.
+    assertSqliteAllowed(hippoRoot);
+    const file = getHippoDbPath(hippoRoot);
+    if (!existsSync(file))
+        throw new Error(`No existing Hippo database at ${file}`);
+    const db = new DatabaseSync(file, { readOnly });
+    try {
+        db.exec(`PRAGMA busy_timeout = ${DEFAULT_BUSY_WAIT_MS}`);
+        return fn(db);
+    }
+    finally {
+        db.close();
+    }
+}
 export function closeHippoDb(db) {
     if (isScopedHandle(db))
         return;

package/dist/db/physics-state.d.ts

@@ -1,17 +1,10 @@
-/**
- * Physics state persistence for Hippo.
- * Stores and loads particle state (position, velocity, mass, charge, temperature)
- * in SQLite using BLOB columns for 384-dim vectors.
- */
+/** Physics state persistence: particle state (position, velocity, mass, charge, temperature) in SQLite, with BLOB columns for 384-dim vectors. */
 import type { DatabaseSyncLike } from './index.js';
 import type { MemoryEntry } from '../core/memory.js';
 import type { PhysicsParticle } from '../core/physics.js';
 export declare function float32ToBuffer(arr: number[]): Buffer;
 export declare function bufferToFloat32(buf: Buffer | Uint8Array): number[];
-/**
- * Create the memory_physics table (migration v8).
- * Call this from db.ts MIGRATIONS array.
- */
+/** Create the memory_physics table (migration v8). */
 export declare function createPhysicsTable(db: DatabaseSyncLike): void;
 /**
  * Load physics state for specific memory IDs (or all if no IDs given).
@@ -21,19 +14,10 @@ export declare function loadPhysicsState(db: DatabaseSyncLike, memoryIds?: strin
  * Save physics state for multiple particles (batch upsert).
  */
 export declare function savePhysicsState(db: DatabaseSyncLike, particles: PhysicsParticle[]): void;
-/**
- * Initialize physics state for a memory from its embedding vector.
- * Returns the new particle (does not persist — caller must save).
- */
+/** Initialize physics state for a memory from its embedding vector; returns the new particle without persisting (the caller must save). */
 export declare function initializeParticle(entry: MemoryEntry, embedding: number[], now?: Date): PhysicsParticle;
-/**
- * Reset all physics states from original embeddings.
- * Drops existing physics data and re-initializes from the embedding index.
- */
-export declare function resetAllPhysicsState(db: DatabaseSyncLike, entries: MemoryEntry[], embeddingIndex: Record<string, number[]>, now?: Date): number;
-/**
- * Refresh mass, charge, and temperature for existing particles
- * based on current memory attributes (called during consolidation).
- */
+/** Drops every particle, then seeds one per entry that has an embedding; `entries` may be read lazily, as each batch is written before the next is read. */
+export declare function resetAllPhysicsState(db: DatabaseSyncLike, entries: Iterable<MemoryEntry>, embeddingIndex: Record<string, number[]>, now?: Date): number;
+/** Refresh mass, charge and temperature for existing particles from current memory attributes (called during consolidation). */
 export declare function refreshParticleProperties(particles: PhysicsParticle[], entries: Map<string, MemoryEntry>, now?: Date): void;
 //# sourceMappingURL=physics-state.d.ts.map