@pluv/client 0.38.1 → 0.38.3

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @pluv/client@0.38.1 build /home/runner/work/pluv/pluv/packages/client
+> @pluv/client@0.38.3 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
-CJS dist/index.js 48.91 KB
-CJS ⚡️ Build success in 115ms
-ESM dist/index.mjs 47.59 KB
-ESM ⚡️ Build success in 129ms
+CJS dist/index.js 49.48 KB
+CJS ⚡️ Build success in 118ms
+ESM dist/index.mjs 48.17 KB
+ESM ⚡️ Build success in 119ms
 DTS Build start
-DTS ⚡️ Build success in 3357ms
-DTS dist/index.d.mts 23.13 KB
-DTS dist/index.d.ts  23.13 KB
+DTS ⚡️ Build success in 3259ms
+DTS dist/index.d.mts 23.16 KB
+DTS dist/index.d.ts  23.16 KB

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @pluv/client
 
+## 0.38.3
+
+### Patch Changes
+
+- 8346273: Disallow the `# @pluv/client character in router event names.
+  - @pluv/crdt@0.38.3
+  - @pluv/types@0.38.3
+
+## 0.38.2
+
+### Patch Changes
+
+- de35f0b: Require `PluvRouter` event names to be formatted as valid JavaScript variable names.
+  - @pluv/crdt@0.38.2
+  - @pluv/types@0.38.2
+
 ## 0.38.1
 
 ### Patch Changes

package/dist/index.d.mts

@@ -86,6 +86,7 @@ declare class PluvRouter<TIO extends IOLike = IOLike, TPresence extends JsonObje
     };
     constructor(events: TEvents);
     static merge<TRouters extends PluvRouter<any, any, any, any>[]>(...routers: TRouters): MergedRouter<TRouters>;
+    private _isValidEventName;
 }
 
 interface StateNotifierSubjects<TIO extends IOLike, TPresence extends JsonObject> {

package/dist/index.d.ts

@@ -86,6 +86,7 @@ declare class PluvRouter<TIO extends IOLike = IOLike, TPresence extends JsonObje
     };
     constructor(events: TEvents);
     static merge<TRouters extends PluvRouter<any, any, any, any>[]>(...routers: TRouters): MergedRouter<TRouters>;
+    private _isValidEventName;
 }
 
 interface StateNotifierSubjects<TIO extends IOLike, TPresence extends JsonObject> {

package/dist/index.js

@@ -249,6 +249,12 @@ var OtherNotifier = class {
 var PluvRouter = class _PluvRouter {
   constructor(events) {
     this._defs = { events: {} };
+    const invalidName = Object.keys(events).find((name) => !this._isValidEventName(name));
+    if (typeof invalidName === "string") {
+      throw new Error(
+        `Invalid event name. Event names must be formatted as valid JavaScript variable names: "${invalidName}"`
+      );
+    }
     this._defs = { events };
   }
   static merge(...routers) {
@@ -258,6 +264,9 @@ var PluvRouter = class _PluvRouter {
     }
     return new _PluvRouter(events);
   }
+  _isValidEventName(name) {
+    return /^[a-z_$][a-z0-9_$]*$/gi.test(name);
+  }
 };
 
 // src/StateNotifier.ts
@@ -1256,6 +1265,10 @@ var PluvClient = class {
     return new PluvProcedure();
   }
   router(events) {
+    const invalidName = Object.keys(events).find((name) => name.includes("$"));
+    if (typeof invalidName === "string") {
+      throw new Error(`Invalid event name. Event names must not contain $: "${invalidName}"`);
+    }
     return new PluvRouter(events);
   }
   _logDebug(...data) {

package/dist/index.mjs

@@ -218,6 +218,12 @@ var OtherNotifier = class {
 var PluvRouter = class _PluvRouter {
   constructor(events) {
     this._defs = { events: {} };
+    const invalidName = Object.keys(events).find((name) => !this._isValidEventName(name));
+    if (typeof invalidName === "string") {
+      throw new Error(
+        `Invalid event name. Event names must be formatted as valid JavaScript variable names: "${invalidName}"`
+      );
+    }
     this._defs = { events };
   }
   static merge(...routers) {
@@ -227,6 +233,9 @@ var PluvRouter = class _PluvRouter {
     }
     return new _PluvRouter(events);
   }
+  _isValidEventName(name) {
+    return /^[a-z_$][a-z0-9_$]*$/gi.test(name);
+  }
 };
 
 // src/StateNotifier.ts
@@ -1225,6 +1234,10 @@ var PluvClient = class {
     return new PluvProcedure();
   }
   router(events) {
+    const invalidName = Object.keys(events).find((name) => name.includes("$"));
+    if (typeof invalidName === "string") {
+      throw new Error(`Invalid event name. Event names must not contain $: "${invalidName}"`);
+    }
     return new PluvRouter(events);
   }
   _logDebug(...data) {

package/package.json

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

package/src/PluvClient.ts

@@ -147,6 +147,12 @@ export class PluvClient<
     public router<TEvents extends PluvRouterEventConfig<TIO, TPresence, TStorage> = {}>(
         events: TEvents,
     ): PluvRouter<TIO, TPresence, TStorage, TEvents> {
+        const invalidName = Object.keys(events).find((name) => name.includes("$"));
+
+        if (typeof invalidName === "string") {
+            throw new Error(`Invalid event name. Event names must not contain $: "${invalidName}"`);
+        }
+
         return new PluvRouter<TIO, TPresence, TStorage, TEvents>(events);
     }
 

package/src/PluvRouter.ts

@@ -53,6 +53,14 @@ export class PluvRouter<
     readonly _defs: { events: TEvents } = { events: {} as TEvents };
 
     constructor(events: TEvents) {
+        const invalidName = Object.keys(events).find((name) => !this._isValidEventName(name));
+
+        if (typeof invalidName === "string") {
+            throw new Error(
+                `Invalid event name. Event names must be formatted as valid JavaScript variable names: "${invalidName}"`,
+            );
+        }
+
         this._defs = { events };
     }
 
@@ -67,4 +75,8 @@ export class PluvRouter<
 
         return new PluvRouter<any, any, any, any>(events) as MergedRouter<TRouters>;
     }
+
+    private _isValidEventName(name: string): boolean {
+        return /^[a-z_$][a-z0-9_$]*$/gi.test(name);
+    }
 }