@pluv/client 0.38.12 → 0.38.14

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @pluv/client@0.38.12 build /home/runner/work/pluv/pluv/packages/client
+> @pluv/client@0.38.14 build /home/runner/work/pluv/pluv/packages/client
 > 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
-ESM dist/index.mjs 48.06 KB
-ESM ⚡️ Build success in 117ms
-CJS dist/index.js 49.38 KB
-CJS ⚡️ Build success in 117ms
+CJS dist/index.js 49.20 KB
+CJS ⚡️ Build success in 128ms
+ESM dist/index.mjs 47.88 KB
+ESM ⚡️ Build success in 134ms
 DTS Build start
-DTS ⚡️ Build success in 3216ms
+DTS ⚡️ Build success in 3169ms
 DTS dist/index.d.mts 23.23 KB
 DTS dist/index.d.ts  23.23 KB

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @pluv/client
 
+## 0.38.14
+
+### Patch Changes
+
+- 886a1f9: Fix cases where the user's own presence was getting updated excessively, and without re-triggering subscriptions.
+  - @pluv/crdt@0.38.14
+  - @pluv/types@0.38.14
+
+## 0.38.13
+
+### Patch Changes
+
+- @pluv/crdt@0.38.13
+- @pluv/types@0.38.13
+
 ## 0.38.12
 
 ### Patch Changes

package/dist/index.js

@@ -347,9 +347,7 @@ var UsersManager = class {
   patchPresence(connectionId, patch) {
     var _a, _b;
     if (((_a = this._myself) == null ? void 0 : _a.connectionId) === connectionId) {
-      const presence2 = __spreadValues(__spreadValues({}, this._myPresence), patch);
-      this._myself.presence = presence2;
-      this._myPresence = presence2;
+      this.updateMyPresence(patch);
       return;
     }
     const other = this._others.get(connectionId);
@@ -400,8 +398,8 @@ var UsersManager = class {
   /**
    * @description This method need not care about being connected.
    */
-  updateMyPresence(presence) {
-    this._myPresence = __spreadValues(__spreadValues({}, this._myPresence), presence);
+  updateMyPresence(patch) {
+    this._myPresence = __spreadValues(__spreadValues({}, this._myPresence), patch);
     if (!this._myself) return;
     this._myself.presence = this._myPresence;
   }
@@ -874,10 +872,9 @@ var PluvRoom = class extends AbstractRoom {
     if (!connectionId) return;
     if (!this._state.webSocket) throw new Error("Could not find WebSocket");
     const data = message.data;
-    this._usersManager.patchPresence(connectionId, data.presence);
-    const myPresence = this._usersManager.myPresence;
     const myself = (_a = this._usersManager.myself) != null ? _a : null;
     if ((myself == null ? void 0 : myself.connectionId) === connectionId) return;
+    this._usersManager.patchPresence(connectionId, data.presence);
     const other = this._usersManager.getOther(connectionId);
     const others = this._usersManager.getOthers();
     this._otherNotifier.subject(connectionId).next(other);

package/dist/index.mjs

@@ -316,9 +316,7 @@ var UsersManager = class {
   patchPresence(connectionId, patch) {
     var _a, _b;
     if (((_a = this._myself) == null ? void 0 : _a.connectionId) === connectionId) {
-      const presence2 = __spreadValues(__spreadValues({}, this._myPresence), patch);
-      this._myself.presence = presence2;
-      this._myPresence = presence2;
+      this.updateMyPresence(patch);
       return;
     }
     const other = this._others.get(connectionId);
@@ -369,8 +367,8 @@ var UsersManager = class {
   /**
    * @description This method need not care about being connected.
    */
-  updateMyPresence(presence) {
-    this._myPresence = __spreadValues(__spreadValues({}, this._myPresence), presence);
+  updateMyPresence(patch) {
+    this._myPresence = __spreadValues(__spreadValues({}, this._myPresence), patch);
     if (!this._myself) return;
     this._myself.presence = this._myPresence;
   }
@@ -843,10 +841,9 @@ var PluvRoom = class extends AbstractRoom {
     if (!connectionId) return;
     if (!this._state.webSocket) throw new Error("Could not find WebSocket");
     const data = message.data;
-    this._usersManager.patchPresence(connectionId, data.presence);
-    const myPresence = this._usersManager.myPresence;
     const myself = (_a = this._usersManager.myself) != null ? _a : null;
     if ((myself == null ? void 0 : myself.connectionId) === connectionId) return;
+    this._usersManager.patchPresence(connectionId, data.presence);
     const other = this._usersManager.getOther(connectionId);
     const others = this._usersManager.getOthers();
     this._otherNotifier.subject(connectionId).next(other);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pluv/client",
-  "version": "0.38.12",
+  "version": "0.38.14",
   "description": "@pluv/io frontend client",
   "author": "leedavidcs",
   "license": "MIT",
@@ -18,15 +18,15 @@
   },
   "dependencies": {
     "wonka": "^6.3.5",
-    "@pluv/crdt": "^0.38.12",
-    "@pluv/types": "^0.38.12"
+    "@pluv/crdt": "^0.38.14",
+    "@pluv/types": "^0.38.14"
   },
   "devDependencies": {
     "eslint": "^8.57.1",
     "tsup": "^8.4.0",
     "typescript": "^5.8.3",
-    "@pluv/tsconfig": "^0.38.12",
-    "eslint-config-pluv": "^0.38.12"
+    "@pluv/tsconfig": "^0.38.14",
+    "eslint-config-pluv": "^0.38.14"
   },
   "scripts": {
     "build": "tsup src/index.ts --format esm,cjs --dts",

package/src/PluvRoom.ts

@@ -785,9 +785,6 @@ export class PluvRoom<
 
         const data = message.data as BaseIOEventRecord<InferIOAuthorize<TIO>>["$presenceUpdated"];
 
-        this._usersManager.patchPresence(connectionId, data.presence as TPresence);
-
-        const myPresence = this._usersManager.myPresence;
         const myself = this._usersManager.myself ?? null;
 
         /**
@@ -799,6 +796,8 @@ export class PluvRoom<
          */
         if (myself?.connectionId === connectionId) return;
 
+        this._usersManager.patchPresence(connectionId, data.presence as TPresence);
+
         const other = this._usersManager.getOther(connectionId);
         const others = this._usersManager.getOthers();
 

package/src/UsersManager.ts

@@ -52,13 +52,7 @@ export class UsersManager<TIO extends IOLike, TPresence extends JsonObject = {}>
 
     public patchPresence(connectionId: string, patch: Partial<TPresence>): void {
         if (this._myself?.connectionId === connectionId) {
-            const presence = {
-                ...this._myPresence,
-                ...patch,
-            } as TPresence;
-
-            this._myself.presence = presence;
-            this._myPresence = presence;
+            this.updateMyPresence(patch);
 
             return;
         }
@@ -135,8 +129,8 @@ export class UsersManager<TIO extends IOLike, TPresence extends JsonObject = {}>
     /**
      * @description This method need not care about being connected.
      */
-    public updateMyPresence(presence: Partial<TPresence>): void {
-        this._myPresence = { ...this._myPresence, ...presence };
+    public updateMyPresence(patch: Partial<TPresence>): void {
+        this._myPresence = { ...this._myPresence, ...patch };
 
         if (!this._myself) return;