@powersync/common 1.22.0 → 1.22.2

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/Column.d.ts

@@ -12,7 +12,7 @@ export type BaseColumnType<T extends number | string | null> = {
 };
 export type ColumnsType = Record<string, BaseColumnType<any>>;
 export type ExtractColumnValueType<T extends BaseColumnType<any>> = T extends BaseColumnType<infer R> ? R : unknown;
-export declare const MAX_AMOUNT_OF_COLUMNS = 63;
+export declare const MAX_AMOUNT_OF_COLUMNS = 1999;
 export declare const column: {
     text: BaseColumnType<string | null>;
     integer: BaseColumnType<number | null>;

package/lib/db/schema/Column.js

@@ -14,9 +14,9 @@ const integer = {
 const real = {
     type: ColumnType.REAL
 };
-// There is maximum of 127 arguments for any function in SQLite. Currently we use json_object which uses 1 arg per key (column name)
-// and one per value, which limits it to 63 arguments.
-export const MAX_AMOUNT_OF_COLUMNS = 63;
+// powersync-sqlite-core limits the number of column per table to 1999, due to internal SQLite limits.
+// In earlier versions this was limited to 63.
+export const MAX_AMOUNT_OF_COLUMNS = 1999;
 export const column = {
     text,
     integer,

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.2",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"
@@ -59,7 +59,7 @@
   "scripts": {
     "build": "tsc -b && rollup -c rollup.config.mjs",
     "build:prod": "tsc -b --sourceMap false && rollup -c rollup.config.mjs --sourceMap false",
-    "clean": "rm -rf lib dist tsconfig.tsbuildinfo",
+    "clean": "rm -rf lib dist tsconfig.tsbuildinfo node_modules",
     "test": "vitest"
   }
 }