@powersync/common 1.22.0 → 1.22.1

package/lib/client/AbstractPowerSyncDatabase.js

@@ -390,13 +390,13 @@ export class AbstractPowerSyncDatabase extends BaseObserver {
             if (writeCheckpoint) {
                 const check = await tx.execute(`SELECT 1 FROM ${PSInternalTable.CRUD} LIMIT 1`);
                 if (!check.rows?.length) {
-                    await tx.execute(`UPDATE ${PSInternalTable.BUCKETS} SET target_op = ? WHERE name='$local'`, [
+                    await tx.execute(`UPDATE ${PSInternalTable.BUCKETS} SET target_op = CAST(? as INTEGER) WHERE name='$local'`, [
                         writeCheckpoint
                     ]);
                 }
             }
             else {
-                await tx.execute(`UPDATE ${PSInternalTable.BUCKETS} SET target_op = ? WHERE name='$local'`, [
+                await tx.execute(`UPDATE ${PSInternalTable.BUCKETS} SET target_op = CAST(? as INTEGER) WHERE name='$local'`, [
                     this.bucketStorageAdapter.getMaxOpId()
                 ]);
             }

package/lib/client/sync/bucket/SqliteBucketStorage.js

@@ -63,7 +63,7 @@ export class SqliteBucketStorage extends BaseObserver {
      */
     startSession() { }
     async getBucketStates() {
-        const result = await this.db.getAll('SELECT name as bucket, cast(last_op as TEXT) as op_id FROM ps_buckets WHERE pending_delete = 0');
+        const result = await this.db.getAll("SELECT name as bucket, cast(last_op as TEXT) as op_id FROM ps_buckets WHERE pending_delete = 0 AND name != '$local'");
         return result;
     }
     async saveSyncData(batch) {
@@ -204,9 +204,7 @@ export class SqliteBucketStorage extends BaseObserver {
         this.compactCounter = 0;
     }
     async updateLocalTarget(cb) {
-        const rs1 = await this.db.getAll("SELECT target_op FROM ps_buckets WHERE name = '$local' AND target_op = ?", [
-            MAX_OP_ID
-        ]);
+        const rs1 = await this.db.getAll("SELECT target_op FROM ps_buckets WHERE name = '$local' AND target_op = CAST(? as INTEGER)", [MAX_OP_ID]);
         if (!rs1.length) {
             // Nothing to update
             return false;

package/lib/db/schema/Schema.js

@@ -11,6 +11,16 @@ export class Schema {
     tables;
     constructor(tables) {
         if (Array.isArray(tables)) {
+            /*
+              We need to validate that the tables have a name here because a user could pass in an array
+              of Tables that don't have a name because they are using the V2 syntax.
+              Therefore, 'convertToClassicTables' won't be called on the tables resulting in a runtime error.
+            */
+            for (const table of tables) {
+                if (table.name === '') {
+                    throw new Error("It appears you are trying to create a new Schema with an array instead of an object. Passing in an object instead of an array into 'new Schema()' may resolve your issue.");
+                }
+            }
             this.tables = tables;
         }
         else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@powersync/common",
-  "version": "1.22.0",
+  "version": "1.22.1",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"