fix: Prevent re-adding existing columns during schema migration and update bplist-parser dependency.

Author: miccy

bun.lock

@@ -229,12 +229,12 @@ const startDbWorker =
       readonly id: Id;
       readonly processedAt: Millis;
     }> = [];
-    const processedRequestIdTtl: Millis = 5 * 60 * 1000;
+    const processedRequestIdTtl = 5 * 60 * 1000;
 
     const { port } = run.deps;
 
     port.onMessage = ({ callbackId, request, evoluPortId }) => {
-      const now = clock.get();
+      const now = run.deps.time.now();
 
       // Evict expired callback IDs based on time-to-live.
       while (processedRequestIdsOrder.length > 0) {
@@ -493,9 +493,11 @@ const validateColumnValue =
     );
   };
 
-const systemColumnsWithoutOwnerId = systemColumns.difference(
-  new Set(["ownerId"]),
-);
+const systemColumnsWithoutOwnerId: ReadonlySet<string> = (() => {
+  const columns = new Set(systemColumns);
+  columns.delete("ownerId");
+  return columns;
+})();
 
 const applyColumnChange =
   (deps: SqliteDep) =>

packages/common/src/local-first/Db.ts

@@ -513,7 +513,8 @@ export const ensureDbSchema =
       if (!currentColumns) {
         queries.push(createAppTable(tableName, newColumns));
       } else {
-        for (const newColumn of newColumns.difference(currentColumns)) {
+        for (const newColumn of newColumns) {
+          if (currentColumns.has(newColumn)) continue;
           queries.push(sql`
             alter table ${sql.identifier(tableName)}
             add column ${sql.identifier(newColumn)} any;