@pluv/persistence-cloudflare-transactional-storage 1.0.0 → 1.0.1

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @pluv/persistence-cloudflare-transactional-storage@1.0.0 build /home/runner/work/pluv/pluv/packages/persistence-cloudflare-transactional-storage
+> @pluv/persistence-cloudflare-transactional-storage@1.0.1 build /home/runner/work/pluv/pluv/packages/persistence-cloudflare-transactional-storage
 > tsup src/index.ts --format esm,cjs --dts
 
 CLI Building entry: src/index.ts
@@ -8,11 +8,11 @@
 CLI Target: es6
 ESM Build start
 CJS Build start
-CJS dist/index.js 10.29 KB
-CJS ⚡️ Build success in 69ms
-ESM dist/index.mjs 9.23 KB
-ESM ⚡️ Build success in 73ms
+ESM dist/index.mjs 10.28 KB
+ESM ⚡️ Build success in 72ms
+CJS dist/index.js 11.34 KB
+CJS ⚡️ Build success in 74ms
 DTS Build start
-DTS ⚡️ Build success in 3082ms
+DTS ⚡️ Build success in 3214ms
 DTS dist/index.d.mts 1.21 KB
 DTS dist/index.d.ts  1.21 KB

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @pluv/persistence-cloudflare-transactional-storage
 
+## 1.0.1
+
+### Patch Changes
+
+- Updated dependencies [c583be9]
+  - @pluv/io@1.0.1
+  - @pluv/types@1.0.1
+
 ## 1.0.0
 
 ### Major Changes

package/dist/index.js

@@ -84,6 +84,7 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
   }
   addUser(room, connectionId, user) {
     return __async(this, null, function* () {
+      var _a, _b;
       if (!this._initialized) return;
       if (!this._state) return;
       yield this._initialized;
@@ -91,6 +92,16 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
         yield this._state.storage.put(this._getUserKey(room, connectionId), user);
         return;
       }
+      const result = this._state.storage.sql.exec(
+        sql`
+                SELECT id
+                FROM ${SQLITE_USER_TABLE}
+                WHERE id = ?;
+            `,
+        connectionId
+      );
+      const existing = !!((_b = (_a = result.toArray()[0]) == null ? void 0 : _a.count) != null ? _b : 0);
+      if (existing) return;
       this._state.storage.sql.exec(
         sql`
                 INSERT INTO ${SQLITE_USER_TABLE} VALUES (?, ?, ?);
@@ -150,7 +161,10 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
           noCache: true,
           prefix: `${KV_USER_PREFIX}::${room}`
         });
-        const partitions = partitionByLength(Array.from(map.keys()), KV_CLOUDFLARE_DELETE_BATCH_LIMIT);
+        const partitions = partitionByLength(
+          Array.from(map.keys()),
+          KV_CLOUDFLARE_DELETE_BATCH_LIMIT
+        );
         yield this._state.storage.transaction((tx) => __async(this, null, function* () {
           yield partitions.reduce((promise, partition) => __async(this, null, function* () {
             yield promise;
@@ -199,7 +213,9 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
       if (!this._state) return null;
       yield this._initialized;
       if (this._mode === "kv") {
-        const user = yield this._state.storage.get(this._getUserKey(room, connectionId));
+        const user = yield this._state.storage.get(
+          this._getUserKey(room, connectionId)
+        );
         return user != null ? user : null;
       }
       const cursor = this._state.storage.sql.exec(
@@ -259,7 +275,9 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
       if (!this._state) return 0;
       yield this._initialized;
       if (this._mode) {
-        const storage = yield this._state.storage.list({ prefix: `${KV_USER_PREFIX}::${room}` });
+        const storage = yield this._state.storage.list({
+          prefix: `${KV_USER_PREFIX}::${room}`
+        });
         return storage.size;
       }
       const cursor = this._state.storage.sql.exec(
@@ -302,11 +320,35 @@ var PersistenceCloudflareTransactionalStorage = class extends import_io.Abstract
   }
   setStorageState(room, state) {
     return __async(this, null, function* () {
+      var _a, _b;
       if (!this._initialized) return;
       if (!this._state) return;
       yield this._initialized;
       if (this._mode === "kv") {
-        yield this._state.storage.put(this._getStorageKey(room), state, { allowConcurrency: true });
+        yield this._state.storage.put(this._getStorageKey(room), state, {
+          allowConcurrency: true
+        });
+        return;
+      }
+      const result = this._state.storage.sql.exec(
+        sql`
+                SELECT COUNT(*) as count
+                FROM ${SQLITE_STORAGE_TABLE}
+                WHERE room = ?;
+            `,
+        room
+      );
+      const existing = !!((_b = (_a = result.toArray()[0]) == null ? void 0 : _a.count) != null ? _b : 0);
+      if (existing) {
+        this._state.storage.sql.exec(
+          sql`
+                    UPDATE ${SQLITE_STORAGE_TABLE}
+                    SET data = ?
+                    WHERE room = ?;
+                `,
+          state,
+          room
+        );
         return;
       }
       this._state.storage.sql.exec(

package/dist/index.mjs

@@ -59,6 +59,7 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
   }
   addUser(room, connectionId, user) {
     return __async(this, null, function* () {
+      var _a, _b;
       if (!this._initialized) return;
       if (!this._state) return;
       yield this._initialized;
@@ -66,6 +67,16 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
         yield this._state.storage.put(this._getUserKey(room, connectionId), user);
         return;
       }
+      const result = this._state.storage.sql.exec(
+        sql`
+                SELECT id
+                FROM ${SQLITE_USER_TABLE}
+                WHERE id = ?;
+            `,
+        connectionId
+      );
+      const existing = !!((_b = (_a = result.toArray()[0]) == null ? void 0 : _a.count) != null ? _b : 0);
+      if (existing) return;
       this._state.storage.sql.exec(
         sql`
                 INSERT INTO ${SQLITE_USER_TABLE} VALUES (?, ?, ?);
@@ -125,7 +136,10 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
           noCache: true,
           prefix: `${KV_USER_PREFIX}::${room}`
         });
-        const partitions = partitionByLength(Array.from(map.keys()), KV_CLOUDFLARE_DELETE_BATCH_LIMIT);
+        const partitions = partitionByLength(
+          Array.from(map.keys()),
+          KV_CLOUDFLARE_DELETE_BATCH_LIMIT
+        );
         yield this._state.storage.transaction((tx) => __async(this, null, function* () {
           yield partitions.reduce((promise, partition) => __async(this, null, function* () {
             yield promise;
@@ -174,7 +188,9 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
       if (!this._state) return null;
       yield this._initialized;
       if (this._mode === "kv") {
-        const user = yield this._state.storage.get(this._getUserKey(room, connectionId));
+        const user = yield this._state.storage.get(
+          this._getUserKey(room, connectionId)
+        );
         return user != null ? user : null;
       }
       const cursor = this._state.storage.sql.exec(
@@ -234,7 +250,9 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
       if (!this._state) return 0;
       yield this._initialized;
       if (this._mode) {
-        const storage = yield this._state.storage.list({ prefix: `${KV_USER_PREFIX}::${room}` });
+        const storage = yield this._state.storage.list({
+          prefix: `${KV_USER_PREFIX}::${room}`
+        });
         return storage.size;
       }
       const cursor = this._state.storage.sql.exec(
@@ -277,11 +295,35 @@ var PersistenceCloudflareTransactionalStorage = class extends AbstractPersistenc
   }
   setStorageState(room, state) {
     return __async(this, null, function* () {
+      var _a, _b;
       if (!this._initialized) return;
       if (!this._state) return;
       yield this._initialized;
       if (this._mode === "kv") {
-        yield this._state.storage.put(this._getStorageKey(room), state, { allowConcurrency: true });
+        yield this._state.storage.put(this._getStorageKey(room), state, {
+          allowConcurrency: true
+        });
+        return;
+      }
+      const result = this._state.storage.sql.exec(
+        sql`
+                SELECT COUNT(*) as count
+                FROM ${SQLITE_STORAGE_TABLE}
+                WHERE room = ?;
+            `,
+        room
+      );
+      const existing = !!((_b = (_a = result.toArray()[0]) == null ? void 0 : _a.count) != null ? _b : 0);
+      if (existing) {
+        this._state.storage.sql.exec(
+          sql`
+                    UPDATE ${SQLITE_STORAGE_TABLE}
+                    SET data = ?
+                    WHERE room = ?;
+                `,
+          state,
+          room
+        );
         return;
       }
       this._state.storage.sql.exec(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pluv/persistence-cloudflare-transactional-storage",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "@pluv/io crdt storage persistence for cloudflare via transactional storage",
   "author": "leedavidcs",
   "license": "MIT",
@@ -17,16 +17,16 @@
     "access": "public"
   },
   "dependencies": {
-    "@pluv/io": "^1.0.0",
-    "@pluv/types": "^1.0.0"
+    "@pluv/io": "^1.0.1",
+    "@pluv/types": "^1.0.1"
   },
   "devDependencies": {
     "@cloudflare/workers-types": "^4.20250430.0",
     "eslint": "^9.25.1",
     "tsup": "^8.4.0",
     "typescript": "^5.8.3",
-    "@pluv/tsconfig": "^1.0.0",
-    "eslint-config-pluv": "^1.0.0"
+    "@pluv/tsconfig": "^1.0.1",
+    "eslint-config-pluv": "^1.0.1"
   },
   "scripts": {
     "build": "tsup src/index.ts --format esm,cjs --dts",

package/src/PersistenceCloudflareTransactionalStorage.ts

@@ -31,7 +31,11 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
         this._mode = mode;
     }
 
-    public async addUser(room: string, connectionId: string, user: JsonObject | null): Promise<void> {
+    public async addUser(
+        room: string,
+        connectionId: string,
+        user: JsonObject | null,
+    ): Promise<void> {
         if (!this._initialized) return;
         if (!this._state) return;
 
@@ -42,6 +46,19 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
             return;
         }
 
+        const result = this._state.storage.sql.exec<{ count: 0 }>(
+            sql`
+                SELECT id
+                FROM ${SQLITE_USER_TABLE}
+                WHERE id = ?;
+            `,
+            connectionId,
+        );
+
+        const existing = !!(result.toArray()[0]?.count ?? 0);
+
+        if (existing) return;
+
         this._state.storage.sql.exec(
             sql`
                 INSERT INTO ${SQLITE_USER_TABLE} VALUES (?, ?, ?);
@@ -108,7 +125,10 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
                 prefix: `${KV_USER_PREFIX}::${room}`,
             });
 
-            const partitions = partitionByLength(Array.from(map.keys()), KV_CLOUDFLARE_DELETE_BATCH_LIMIT);
+            const partitions = partitionByLength(
+                Array.from(map.keys()),
+                KV_CLOUDFLARE_DELETE_BATCH_LIMIT,
+            );
 
             await this._state.storage.transaction(async (tx) => {
                 await partitions.reduce(async (promise, partition) => {
@@ -165,7 +185,9 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
         await this._initialized;
 
         if (this._mode === "kv") {
-            const user = await this._state.storage.get<JsonObject | null>(this._getUserKey(room, connectionId));
+            const user = await this._state.storage.get<JsonObject | null>(
+                this._getUserKey(room, connectionId),
+            );
 
             return user ?? null;
         }
@@ -191,7 +213,9 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
         }
     }
 
-    public async getUsers(room: string): Promise<Map<[connectionId: string][0], JsonObject | null>> {
+    public async getUsers(
+        room: string,
+    ): Promise<Map<[connectionId: string][0], JsonObject | null>> {
         if (!this._initialized) return new Map();
         if (!this._state) return new Map();
 
@@ -238,7 +262,9 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
         await this._initialized;
 
         if (this._mode) {
-            const storage = await this._state.storage.list({ prefix: `${KV_USER_PREFIX}::${room}` });
+            const storage = await this._state.storage.list({
+                prefix: `${KV_USER_PREFIX}::${room}`,
+            });
 
             return storage.size;
         }
@@ -295,7 +321,34 @@ export class PersistenceCloudflareTransactionalStorage extends AbstractPersisten
         await this._initialized;
 
         if (this._mode === "kv") {
-            await this._state.storage.put(this._getStorageKey(room), state, { allowConcurrency: true });
+            await this._state.storage.put(this._getStorageKey(room), state, {
+                allowConcurrency: true,
+            });
+            return;
+        }
+
+        const result = this._state.storage.sql.exec<{ count: number }>(
+            sql`
+                SELECT COUNT(*) as count
+                FROM ${SQLITE_STORAGE_TABLE}
+                WHERE room = ?;
+            `,
+            room,
+        );
+
+        const existing = !!(result.toArray()[0]?.count ?? 0);
+
+        if (existing) {
+            this._state.storage.sql.exec(
+                sql`
+                    UPDATE ${SQLITE_STORAGE_TABLE}
+                    SET data = ?
+                    WHERE room = ?;
+                `,
+                state,
+                room,
+            );
+
             return;
         }
 

package/src/utils/castNumber.ts

@@ -1,3 +1,7 @@
 export const castNumber = (value: any): number => {
-    return typeof value === "number" ? value : typeof value === "string" ? Number.parseInt(value, 10) : Number(value);
+    return typeof value === "number"
+        ? value
+        : typeof value === "string"
+          ? Number.parseInt(value, 10)
+          : Number(value);
 };

package/src/utils/partitionByLength.ts

@@ -1,4 +1,7 @@
-export const partitionByLength = <T>(arr: readonly T[] | T[], length: number): readonly (readonly T[])[] => {
+export const partitionByLength = <T>(
+    arr: readonly T[] | T[],
+    length: number,
+): readonly (readonly T[])[] => {
     if (!arr.length) return [];
 
     const head = arr.slice(0, length);