latticesql 4.3.6 → 4.3.8

package/dist/cli.js

@@ -58419,8 +58419,18 @@ init_adapter();
 var DELETED_AT_SENTINEL = "internal:upgrade:deleted-at-empty-to-null:v1";
 var FILES_PATH_SENTINEL = "internal:upgrade:files-path-to-local-ref:v1";
 async function upgradeLegacyData(db) {
-  await normalizeEmptyDeletedAt(db);
-  await backfillFilesPath(db);
+  await runUpgradeStep("deleted_at normalization", () => normalizeEmptyDeletedAt(db));
+  await runUpgradeStep("files path backfill", () => backfillFilesPath(db));
+}
+async function runUpgradeStep(name, fn) {
+  try {
+    await fn();
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    console.warn(
+      `[lattice] open-time data upgrade step "${name}" skipped (will retry on next open): ${msg}`
+    );
+  }
 }
 async function tablesWithDeletedAt(db) {
   if (db.getDialect() === "postgres") {
@@ -58444,14 +58454,40 @@ async function normalizeEmptyDeletedAt(db) {
     await db.migrate([
       {
         version: `${DELETED_AT_SENTINEL}:all`,
+        // TYPE-AWARE: only a text-like deleted_at column can hold the legacy ''
+        // sentinel. A column that is a real `timestamptz` (some clouds' deleted_at
+        // is) MUST be skipped: the predicate `deleted_at = ''` forces Postgres to
+        // parse `''::timestamptz` at PLAN time — invalid input that throws regardless
+        // of data (a timestamptz column can't even hold ''), which aborted the entire
+        // workspace open. We BLACKLIST the non-text types (timestamp/date/time,
+        // numeric, boolean, json, uuid, bytea, xml) rather than allow-list text — so
+        // text-like columns we don't enumerate (citext, a DOMAIN over text, a custom
+        // text type) are still normalized instead of silently leaving '' rows that
+        // 4.x's `deleted_at IS NULL` predicate would read as DELETED. PER-TABLE FAULT
+        // ISOLATION is the backstop: each table's UPDATE runs in its own
+        // subtransaction, so an exotic type that still errors on `= ''` (enum, inet,
+        // interval, array) — or a blocking trigger/lock — is warned + skipped, never
+        // fatal to the open.
         sql: `DO $LATTICE_DAU$
   DECLARE r record;
 BEGIN
   FOR r IN
     SELECT table_name FROM information_schema.columns
-     WHERE table_schema = current_schema() AND column_name = 'deleted_at'
+     WHERE table_schema = current_schema()
+       AND column_name = 'deleted_at'
+       AND data_type NOT IN (
+         'timestamp with time zone', 'timestamp without time zone',
+         'date', 'time with time zone', 'time without time zone',
+         'integer', 'bigint', 'smallint', 'numeric', 'decimal',
+         'real', 'double precision', 'boolean', 'json', 'jsonb',
+         'bytea', 'uuid', 'xml'
+       )
   LOOP
-    EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    BEGIN
+      EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    EXCEPTION WHEN others THEN
+      RAISE WARNING 'lattice: skipped deleted_at normalization for %: %', r.table_name, SQLERRM;
+    END;
   END LOOP;
 END $LATTICE_DAU$;`
       }
@@ -58467,23 +58503,18 @@ END $LATTICE_DAU$;`
   });
   if (migrations.length > 0) await db.migrate(migrations);
 }
-async function filesTableHasPath(db) {
-  if (db.getDialect() === "postgres") {
-    const rows2 = await allAsyncOrSync(
-      db.adapter,
-      `SELECT 1 AS x FROM information_schema.columns
-        WHERE table_schema = current_schema() AND table_name = 'files' AND column_name = 'path'`
-    );
-    return rows2.length > 0;
-  }
-  const rows = await allAsyncOrSync(
-    db.adapter,
-    `SELECT 1 AS x FROM pragma_table_info('files') WHERE name = 'path'`
-  );
-  return rows.length > 0;
+async function filesColumns(db) {
+  const sql = db.getDialect() === "postgres" ? `SELECT column_name AS name FROM information_schema.columns
+          WHERE table_schema = current_schema() AND table_name = 'files'` : `SELECT name FROM pragma_table_info('files')`;
+  const rows = await allAsyncOrSync(db.adapter, sql);
+  return new Set(rows.map((r6) => r6.name));
 }
 async function backfillFilesPath(db) {
-  if (!await filesTableHasPath(db)) return;
+  const cols = await filesColumns(db);
+  if (!cols.has("path")) return;
+  for (const col of ["ref_kind", "ref_uri", "ref_provider", "blob_path"]) {
+    if (!cols.has(col)) await addColumnAsyncOrSync(db.adapter, "files", col, "TEXT");
+  }
   await db.migrate([
     {
       version: FILES_PATH_SENTINEL,
@@ -82106,7 +82137,7 @@ function printHelp() {
   );
 }
 function getVersion() {
-  if (true) return "4.3.6";
+  if (true) return "4.3.8";
   try {
     const pkgPath = new URL("../package.json", import.meta.url).pathname;
     const pkg = JSON.parse(readFileSync26(pkgPath, "utf-8"));

package/dist/desktop-entry.js

@@ -58113,8 +58113,18 @@ init_adapter();
 var DELETED_AT_SENTINEL = "internal:upgrade:deleted-at-empty-to-null:v1";
 var FILES_PATH_SENTINEL = "internal:upgrade:files-path-to-local-ref:v1";
 async function upgradeLegacyData(db) {
-  await normalizeEmptyDeletedAt(db);
-  await backfillFilesPath(db);
+  await runUpgradeStep("deleted_at normalization", () => normalizeEmptyDeletedAt(db));
+  await runUpgradeStep("files path backfill", () => backfillFilesPath(db));
+}
+async function runUpgradeStep(name, fn) {
+  try {
+    await fn();
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    console.warn(
+      `[lattice] open-time data upgrade step "${name}" skipped (will retry on next open): ${msg}`
+    );
+  }
 }
 async function tablesWithDeletedAt(db) {
   if (db.getDialect() === "postgres") {
@@ -58138,14 +58148,40 @@ async function normalizeEmptyDeletedAt(db) {
     await db.migrate([
       {
         version: `${DELETED_AT_SENTINEL}:all`,
+        // TYPE-AWARE: only a text-like deleted_at column can hold the legacy ''
+        // sentinel. A column that is a real `timestamptz` (some clouds' deleted_at
+        // is) MUST be skipped: the predicate `deleted_at = ''` forces Postgres to
+        // parse `''::timestamptz` at PLAN time — invalid input that throws regardless
+        // of data (a timestamptz column can't even hold ''), which aborted the entire
+        // workspace open. We BLACKLIST the non-text types (timestamp/date/time,
+        // numeric, boolean, json, uuid, bytea, xml) rather than allow-list text — so
+        // text-like columns we don't enumerate (citext, a DOMAIN over text, a custom
+        // text type) are still normalized instead of silently leaving '' rows that
+        // 4.x's `deleted_at IS NULL` predicate would read as DELETED. PER-TABLE FAULT
+        // ISOLATION is the backstop: each table's UPDATE runs in its own
+        // subtransaction, so an exotic type that still errors on `= ''` (enum, inet,
+        // interval, array) — or a blocking trigger/lock — is warned + skipped, never
+        // fatal to the open.
         sql: `DO $LATTICE_DAU$
   DECLARE r record;
 BEGIN
   FOR r IN
     SELECT table_name FROM information_schema.columns
-     WHERE table_schema = current_schema() AND column_name = 'deleted_at'
+     WHERE table_schema = current_schema()
+       AND column_name = 'deleted_at'
+       AND data_type NOT IN (
+         'timestamp with time zone', 'timestamp without time zone',
+         'date', 'time with time zone', 'time without time zone',
+         'integer', 'bigint', 'smallint', 'numeric', 'decimal',
+         'real', 'double precision', 'boolean', 'json', 'jsonb',
+         'bytea', 'uuid', 'xml'
+       )
   LOOP
-    EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    BEGIN
+      EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    EXCEPTION WHEN others THEN
+      RAISE WARNING 'lattice: skipped deleted_at normalization for %: %', r.table_name, SQLERRM;
+    END;
   END LOOP;
 END $LATTICE_DAU$;`
       }
@@ -58161,23 +58197,18 @@ END $LATTICE_DAU$;`
   });
   if (migrations.length > 0) await db.migrate(migrations);
 }
-async function filesTableHasPath(db) {
-  if (db.getDialect() === "postgres") {
-    const rows2 = await allAsyncOrSync(
-      db.adapter,
-      `SELECT 1 AS x FROM information_schema.columns
-        WHERE table_schema = current_schema() AND table_name = 'files' AND column_name = 'path'`
-    );
-    return rows2.length > 0;
-  }
-  const rows = await allAsyncOrSync(
-    db.adapter,
-    `SELECT 1 AS x FROM pragma_table_info('files') WHERE name = 'path'`
-  );
-  return rows.length > 0;
+async function filesColumns(db) {
+  const sql = db.getDialect() === "postgres" ? `SELECT column_name AS name FROM information_schema.columns
+          WHERE table_schema = current_schema() AND table_name = 'files'` : `SELECT name FROM pragma_table_info('files')`;
+  const rows = await allAsyncOrSync(db.adapter, sql);
+  return new Set(rows.map((r6) => r6.name));
 }
 async function backfillFilesPath(db) {
-  if (!await filesTableHasPath(db)) return;
+  const cols = await filesColumns(db);
+  if (!cols.has("path")) return;
+  for (const col of ["ref_kind", "ref_uri", "ref_provider", "blob_path"]) {
+    if (!cols.has(col)) await addColumnAsyncOrSync(db.adapter, "files", col, "TEXT");
+  }
   await db.migrate([
     {
       version: FILES_PATH_SENTINEL,
@@ -81684,7 +81715,7 @@ ${e6.stack ?? ""}`
 }
 
 // src/desktop-entry.ts
-var VERSION2 = true ? "4.3.6" : "unknown";
+var VERSION2 = true ? "4.3.8" : "unknown";
 export {
   VERSION2 as VERSION,
   ensureRootForGui,

package/dist/index.cjs

@@ -62650,8 +62650,18 @@ init_adapter();
 var DELETED_AT_SENTINEL = "internal:upgrade:deleted-at-empty-to-null:v1";
 var FILES_PATH_SENTINEL = "internal:upgrade:files-path-to-local-ref:v1";
 async function upgradeLegacyData(db) {
-  await normalizeEmptyDeletedAt(db);
-  await backfillFilesPath(db);
+  await runUpgradeStep("deleted_at normalization", () => normalizeEmptyDeletedAt(db));
+  await runUpgradeStep("files path backfill", () => backfillFilesPath(db));
+}
+async function runUpgradeStep(name, fn) {
+  try {
+    await fn();
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    console.warn(
+      `[lattice] open-time data upgrade step "${name}" skipped (will retry on next open): ${msg}`
+    );
+  }
 }
 async function tablesWithDeletedAt(db) {
   if (db.getDialect() === "postgres") {
@@ -62675,14 +62685,40 @@ async function normalizeEmptyDeletedAt(db) {
     await db.migrate([
       {
         version: `${DELETED_AT_SENTINEL}:all`,
+        // TYPE-AWARE: only a text-like deleted_at column can hold the legacy ''
+        // sentinel. A column that is a real `timestamptz` (some clouds' deleted_at
+        // is) MUST be skipped: the predicate `deleted_at = ''` forces Postgres to
+        // parse `''::timestamptz` at PLAN time — invalid input that throws regardless
+        // of data (a timestamptz column can't even hold ''), which aborted the entire
+        // workspace open. We BLACKLIST the non-text types (timestamp/date/time,
+        // numeric, boolean, json, uuid, bytea, xml) rather than allow-list text — so
+        // text-like columns we don't enumerate (citext, a DOMAIN over text, a custom
+        // text type) are still normalized instead of silently leaving '' rows that
+        // 4.x's `deleted_at IS NULL` predicate would read as DELETED. PER-TABLE FAULT
+        // ISOLATION is the backstop: each table's UPDATE runs in its own
+        // subtransaction, so an exotic type that still errors on `= ''` (enum, inet,
+        // interval, array) — or a blocking trigger/lock — is warned + skipped, never
+        // fatal to the open.
         sql: `DO $LATTICE_DAU$
   DECLARE r record;
 BEGIN
   FOR r IN
     SELECT table_name FROM information_schema.columns
-     WHERE table_schema = current_schema() AND column_name = 'deleted_at'
+     WHERE table_schema = current_schema()
+       AND column_name = 'deleted_at'
+       AND data_type NOT IN (
+         'timestamp with time zone', 'timestamp without time zone',
+         'date', 'time with time zone', 'time without time zone',
+         'integer', 'bigint', 'smallint', 'numeric', 'decimal',
+         'real', 'double precision', 'boolean', 'json', 'jsonb',
+         'bytea', 'uuid', 'xml'
+       )
   LOOP
-    EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    BEGIN
+      EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    EXCEPTION WHEN others THEN
+      RAISE WARNING 'lattice: skipped deleted_at normalization for %: %', r.table_name, SQLERRM;
+    END;
   END LOOP;
 END $LATTICE_DAU$;`
       }
@@ -62698,23 +62734,18 @@ END $LATTICE_DAU$;`
   });
   if (migrations.length > 0) await db.migrate(migrations);
 }
-async function filesTableHasPath(db) {
-  if (db.getDialect() === "postgres") {
-    const rows2 = await allAsyncOrSync(
-      db.adapter,
-      `SELECT 1 AS x FROM information_schema.columns
-        WHERE table_schema = current_schema() AND table_name = 'files' AND column_name = 'path'`
-    );
-    return rows2.length > 0;
-  }
-  const rows = await allAsyncOrSync(
-    db.adapter,
-    `SELECT 1 AS x FROM pragma_table_info('files') WHERE name = 'path'`
-  );
-  return rows.length > 0;
+async function filesColumns(db) {
+  const sql = db.getDialect() === "postgres" ? `SELECT column_name AS name FROM information_schema.columns
+          WHERE table_schema = current_schema() AND table_name = 'files'` : `SELECT name FROM pragma_table_info('files')`;
+  const rows = await allAsyncOrSync(db.adapter, sql);
+  return new Set(rows.map((r6) => r6.name));
 }
 async function backfillFilesPath(db) {
-  if (!await filesTableHasPath(db)) return;
+  const cols = await filesColumns(db);
+  if (!cols.has("path")) return;
+  for (const col of ["ref_kind", "ref_uri", "ref_provider", "blob_path"]) {
+    if (!cols.has(col)) await addColumnAsyncOrSync(db.adapter, "files", col, "TEXT");
+  }
   await db.migrate([
     {
       version: FILES_PATH_SENTINEL,

package/dist/index.js

@@ -62350,8 +62350,18 @@ init_adapter();
 var DELETED_AT_SENTINEL = "internal:upgrade:deleted-at-empty-to-null:v1";
 var FILES_PATH_SENTINEL = "internal:upgrade:files-path-to-local-ref:v1";
 async function upgradeLegacyData(db) {
-  await normalizeEmptyDeletedAt(db);
-  await backfillFilesPath(db);
+  await runUpgradeStep("deleted_at normalization", () => normalizeEmptyDeletedAt(db));
+  await runUpgradeStep("files path backfill", () => backfillFilesPath(db));
+}
+async function runUpgradeStep(name, fn) {
+  try {
+    await fn();
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    console.warn(
+      `[lattice] open-time data upgrade step "${name}" skipped (will retry on next open): ${msg}`
+    );
+  }
 }
 async function tablesWithDeletedAt(db) {
   if (db.getDialect() === "postgres") {
@@ -62375,14 +62385,40 @@ async function normalizeEmptyDeletedAt(db) {
     await db.migrate([
       {
         version: `${DELETED_AT_SENTINEL}:all`,
+        // TYPE-AWARE: only a text-like deleted_at column can hold the legacy ''
+        // sentinel. A column that is a real `timestamptz` (some clouds' deleted_at
+        // is) MUST be skipped: the predicate `deleted_at = ''` forces Postgres to
+        // parse `''::timestamptz` at PLAN time — invalid input that throws regardless
+        // of data (a timestamptz column can't even hold ''), which aborted the entire
+        // workspace open. We BLACKLIST the non-text types (timestamp/date/time,
+        // numeric, boolean, json, uuid, bytea, xml) rather than allow-list text — so
+        // text-like columns we don't enumerate (citext, a DOMAIN over text, a custom
+        // text type) are still normalized instead of silently leaving '' rows that
+        // 4.x's `deleted_at IS NULL` predicate would read as DELETED. PER-TABLE FAULT
+        // ISOLATION is the backstop: each table's UPDATE runs in its own
+        // subtransaction, so an exotic type that still errors on `= ''` (enum, inet,
+        // interval, array) — or a blocking trigger/lock — is warned + skipped, never
+        // fatal to the open.
         sql: `DO $LATTICE_DAU$
   DECLARE r record;
 BEGIN
   FOR r IN
     SELECT table_name FROM information_schema.columns
-     WHERE table_schema = current_schema() AND column_name = 'deleted_at'
+     WHERE table_schema = current_schema()
+       AND column_name = 'deleted_at'
+       AND data_type NOT IN (
+         'timestamp with time zone', 'timestamp without time zone',
+         'date', 'time with time zone', 'time without time zone',
+         'integer', 'bigint', 'smallint', 'numeric', 'decimal',
+         'real', 'double precision', 'boolean', 'json', 'jsonb',
+         'bytea', 'uuid', 'xml'
+       )
   LOOP
-    EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    BEGIN
+      EXECUTE format('UPDATE %I SET deleted_at = NULL WHERE deleted_at = ''''', r.table_name);
+    EXCEPTION WHEN others THEN
+      RAISE WARNING 'lattice: skipped deleted_at normalization for %: %', r.table_name, SQLERRM;
+    END;
   END LOOP;
 END $LATTICE_DAU$;`
       }
@@ -62398,23 +62434,18 @@ END $LATTICE_DAU$;`
   });
   if (migrations.length > 0) await db.migrate(migrations);
 }
-async function filesTableHasPath(db) {
-  if (db.getDialect() === "postgres") {
-    const rows2 = await allAsyncOrSync(
-      db.adapter,
-      `SELECT 1 AS x FROM information_schema.columns
-        WHERE table_schema = current_schema() AND table_name = 'files' AND column_name = 'path'`
-    );
-    return rows2.length > 0;
-  }
-  const rows = await allAsyncOrSync(
-    db.adapter,
-    `SELECT 1 AS x FROM pragma_table_info('files') WHERE name = 'path'`
-  );
-  return rows.length > 0;
+async function filesColumns(db) {
+  const sql = db.getDialect() === "postgres" ? `SELECT column_name AS name FROM information_schema.columns
+          WHERE table_schema = current_schema() AND table_name = 'files'` : `SELECT name FROM pragma_table_info('files')`;
+  const rows = await allAsyncOrSync(db.adapter, sql);
+  return new Set(rows.map((r6) => r6.name));
 }
 async function backfillFilesPath(db) {
-  if (!await filesTableHasPath(db)) return;
+  const cols = await filesColumns(db);
+  if (!cols.has("path")) return;
+  for (const col of ["ref_kind", "ref_uri", "ref_provider", "blob_path"]) {
+    if (!cols.has(col)) await addColumnAsyncOrSync(db.adapter, "files", col, "TEXT");
+  }
   await db.migrate([
     {
       version: FILES_PATH_SENTINEL,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "latticesql",
-  "version": "4.3.6",
+  "version": "4.3.8",
   "description": "Persistent structured memory for AI agent systems — pluggable SQLite or Postgres backend, LLM context bridge",
   "type": "module",
   "main": "./dist/index.js",