pi-mega-compact 0.11.0 → 0.11.1

package/dist/src/store/sqlite/schema.js

@@ -105,10 +105,6 @@ export function initSchema(db) {
     );
     CREATE INDEX IF NOT EXISTS idx_raptor_session ON raptor_nodes(session_id);
     CREATE INDEX IF NOT EXISTS idx_raptor_parent ON raptor_nodes(parent_id);
-    -- S42D/QA perf: session-scoped built_at ordering so the per-search cache
-    -- invalidation MAX(built_at) is index-satisfied.
-    CREATE INDEX IF NOT EXISTS idx_raptor_session_built
-      ON raptor_nodes(session_id, built_at DESC);
 
     -- S42D: structured RAPTOR build history. One row per tree build; the
     -- freshness check (buildHistory.ts) uses completed_at + leaf_count to skip
@@ -375,6 +371,11 @@ export function initSchema(db) {
     // S25: RAPTOR freshness-guard timestamp. Additive; old DBs have NULL → 0 →
     // treated as stale → flat fallback (safe).
     ensureColumn(db, "raptor_nodes", "built_at", "INTEGER");
+    // S42D/QA perf: session-scoped built_at ordering. Must follow ensureColumn
+    // above — old DBs already have raptor_nodes without built_at, so CREATE TABLE
+    // IF NOT EXISTS is a no-op that doesn't add the column, and moving this index
+    // inside the DDL block caused "no such column: built_at" on upgrade.
+    db.exec("CREATE INDEX IF NOT EXISTS idx_raptor_session_built ON raptor_nodes(session_id, built_at DESC)");
     // S43: turn_index on raw_transcript so a message points directly at its
     // conversation turn (otherwise it must be inferred from seq ordering). NULL
     // for legacy rows — turns written before S43 have no turn link.
@@ -390,7 +391,9 @@ export function initSchema(db) {
     for (const d of ACHIEVEMENT_DEFS) {
         seedAch.run(d.id, d.title, d.description, d.hidden ? 1 : 0, d.icon);
     }
-    const v = db.prepare("SELECT value FROM meta WHERE key='schema_version'").get();
+    const v = db
+        .prepare("SELECT value FROM meta WHERE key='schema_version'")
+        .get();
     if (!v) {
         db.prepare("INSERT INTO meta(key, value) VALUES(?, ?)").run("schema_version", String(SCHEMA_VERSION));
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-mega-compact",
-  "version": "0.11.0",
+  "version": "0.11.1",
   "description": "Layered, local, vector-backed context compressor for pi — supersede/collapse/cluster compaction with deduped inline recall.",
   "type": "module",
   "license": "BSD-3-Clause",

package/src/store/sqlite/schema.ts

@@ -1,7 +1,7 @@
 /**
  * schema.ts — table creation, migrations, `ensureColumn`, PRAGMA setup.
  */
-import { DatabaseSync } from "node:sqlite";
+import type { DatabaseSync } from "node:sqlite";
 import { ACHIEVEMENT_DEFS } from "../../game/scoring.js";
 
 const SCHEMA_VERSION = 2;
@@ -13,14 +13,21 @@ const SCHEMA_VERSION = 2;
  * input), so the unavoidable identifier interpolation here does not violate
  * PREVENT-002 (no external data reaches this SQL).
  */
-export function ensureColumn(db: DatabaseSync, table: string, column: string, decl: string): void {
-  const cols = db.prepare(`PRAGMA table_info(${table})`).all() as Array<{ name: string }>;
-  if (cols.some((c) => c.name === column)) return;
-  db.exec(`ALTER TABLE ${table} ADD COLUMN ${column} ${decl}`);
+export function ensureColumn(
+	db: DatabaseSync,
+	table: string,
+	column: string,
+	decl: string,
+): void {
+	const cols = db.prepare(`PRAGMA table_info(${table})`).all() as Array<{
+		name: string;
+	}>;
+	if (cols.some((c) => c.name === column)) return;
+	db.exec(`ALTER TABLE ${table} ADD COLUMN ${column} ${decl}`);
 }
 
 export function initSchema(db: DatabaseSync): void {
-  db.exec(`
+	db.exec(`
     CREATE TABLE IF NOT EXISTS context_chunks (
       id                  TEXT NOT NULL,
       session_id         TEXT NOT NULL,
@@ -111,10 +118,6 @@ export function initSchema(db: DatabaseSync): void {
     );
     CREATE INDEX IF NOT EXISTS idx_raptor_session ON raptor_nodes(session_id);
     CREATE INDEX IF NOT EXISTS idx_raptor_parent ON raptor_nodes(parent_id);
-    -- S42D/QA perf: session-scoped built_at ordering so the per-search cache
-    -- invalidation MAX(built_at) is index-satisfied.
-    CREATE INDEX IF NOT EXISTS idx_raptor_session_built
-      ON raptor_nodes(session_id, built_at DESC);
 
     -- S42D: structured RAPTOR build history. One row per tree build; the
     -- freshness check (buildHistory.ts) uses completed_at + leaf_count to skip
@@ -364,45 +367,55 @@ export function initSchema(db: DatabaseSync): void {
     CREATE INDEX IF NOT EXISTS idx_perf_samples_ts ON perf_samples(ts);
     CREATE INDEX IF NOT EXISTS idx_perf_samples_kind_ts ON perf_samples(kind, ts);
   `);
-  // Idempotent column migrations. `CREATE TABLE IF NOT EXISTS` is a no-op on a
-  // pre-existing table, so new columns added to context_chunks after a store was
-  // first created (e.g. original_token_estimate in v0.4.2) must be ALTERed in for
-  // databases created by an older version — otherwise repoStats()/upsert crash
-  // with "no such column" and the extension fails to load. Additive only.
-  ensureColumn(db, "context_chunks", "original_token_estimate", "INTEGER");
-  // S27 Task 6: content_ref column in raw_transcript for dedup_mirror references.
-  ensureColumn(db, "raw_transcript", "content_ref", "TEXT");
-  // S20 memory-RAG extension: additive columns for auto-review ops. Idempotent —
-  // only alters DBs created by an older version that lack these columns.
-  ensureColumn(db, "memories", "category", "TEXT");
-  ensureColumn(db, "memories", "target", "TEXT");
-  ensureColumn(db, "memories", "last_referenced", "INTEGER");
-  ensureColumn(db, "memories", "source_turn", "INTEGER");
-  // S25: RAPTOR freshness-guard timestamp. Additive; old DBs have NULL → 0 →
-  // treated as stale → flat fallback (safe).
-  ensureColumn(db, "raptor_nodes", "built_at", "INTEGER");
-  // S43: turn_index on raw_transcript so a message points directly at its
-  // conversation turn (otherwise it must be inferred from seq ordering). NULL
-  // for legacy rows — turns written before S43 have no turn link.
-  ensureColumn(db, "raw_transcript", "turn_index", "INTEGER");
-  // S43: conversation_id + last_turn_id on session_state (legacy DBs have NULL).
-  ensureColumn(db, "session_state", "conversation_id", "TEXT");
-  ensureColumn(db, "session_state", "last_turn_id", "INTEGER");
-  // S35: idempotent seed of the 9 achievement rows. ON CONFLICT(id) DO
-  // NOTHING so a re-open never clobbers an already-unlocked row's
-  // unlocked_at. No user input reaches this SQL (PREVENT-002 safe).
-  const seedAch = db.prepare(
-    `INSERT INTO game_achievements (id, title, description, hidden, icon)
+	// Idempotent column migrations. `CREATE TABLE IF NOT EXISTS` is a no-op on a
+	// pre-existing table, so new columns added to context_chunks after a store was
+	// first created (e.g. original_token_estimate in v0.4.2) must be ALTERed in for
+	// databases created by an older version — otherwise repoStats()/upsert crash
+	// with "no such column" and the extension fails to load. Additive only.
+	ensureColumn(db, "context_chunks", "original_token_estimate", "INTEGER");
+	// S27 Task 6: content_ref column in raw_transcript for dedup_mirror references.
+	ensureColumn(db, "raw_transcript", "content_ref", "TEXT");
+	// S20 memory-RAG extension: additive columns for auto-review ops. Idempotent —
+	// only alters DBs created by an older version that lack these columns.
+	ensureColumn(db, "memories", "category", "TEXT");
+	ensureColumn(db, "memories", "target", "TEXT");
+	ensureColumn(db, "memories", "last_referenced", "INTEGER");
+	ensureColumn(db, "memories", "source_turn", "INTEGER");
+	// S25: RAPTOR freshness-guard timestamp. Additive; old DBs have NULL → 0 →
+	// treated as stale → flat fallback (safe).
+	ensureColumn(db, "raptor_nodes", "built_at", "INTEGER");
+	// S42D/QA perf: session-scoped built_at ordering. Must follow ensureColumn
+	// above — old DBs already have raptor_nodes without built_at, so CREATE TABLE
+	// IF NOT EXISTS is a no-op that doesn't add the column, and moving this index
+	// inside the DDL block caused "no such column: built_at" on upgrade.
+	db.exec(
+		"CREATE INDEX IF NOT EXISTS idx_raptor_session_built ON raptor_nodes(session_id, built_at DESC)",
+	);
+	// S43: turn_index on raw_transcript so a message points directly at its
+	// conversation turn (otherwise it must be inferred from seq ordering). NULL
+	// for legacy rows — turns written before S43 have no turn link.
+	ensureColumn(db, "raw_transcript", "turn_index", "INTEGER");
+	// S43: conversation_id + last_turn_id on session_state (legacy DBs have NULL).
+	ensureColumn(db, "session_state", "conversation_id", "TEXT");
+	ensureColumn(db, "session_state", "last_turn_id", "INTEGER");
+	// S35: idempotent seed of the 9 achievement rows. ON CONFLICT(id) DO
+	// NOTHING so a re-open never clobbers an already-unlocked row's
+	// unlocked_at. No user input reaches this SQL (PREVENT-002 safe).
+	const seedAch = db.prepare(
+		`INSERT INTO game_achievements (id, title, description, hidden, icon)
      VALUES (?, ?, ?, ?, ?) ON CONFLICT(id) DO NOTHING`,
-  );
-  for (const d of ACHIEVEMENT_DEFS) {
-    seedAch.run(d.id, d.title, d.description, d.hidden ? 1 : 0, d.icon);
-  }
+	);
+	for (const d of ACHIEVEMENT_DEFS) {
+		seedAch.run(d.id, d.title, d.description, d.hidden ? 1 : 0, d.icon);
+	}
 
-  const v = db.prepare("SELECT value FROM meta WHERE key='schema_version'").get() as
-    | { value: string }
-    | undefined;
-  if (!v) {
-    db.prepare("INSERT INTO meta(key, value) VALUES(?, ?)").run("schema_version", String(SCHEMA_VERSION));
-  }
+	const v = db
+		.prepare("SELECT value FROM meta WHERE key='schema_version'")
+		.get() as { value: string } | undefined;
+	if (!v) {
+		db.prepare("INSERT INTO meta(key, value) VALUES(?, ?)").run(
+			"schema_version",
+			String(SCHEMA_VERSION),
+		);
+	}
 }