@vandeurenglenn/little-pubsub 1.4.3 → 1.4.5

package/index.d.ts

@@ -1,11 +1,16 @@
 export default class LittlePubSub {
-    subscribers: {};
+    subscribers: {
+        [index: string]: {
+            value?: any;
+            handlers?: Function[];
+        };
+    };
     verbose: boolean;
     constructor(verbose?: boolean);
     _handleContext(handler: Function, context?: Function): Function;
     hasSubscribers(event: string): boolean;
     subscribe(event: string, handler: Function, context?: Function): void;
     unsubscribe(event: string, handler: Function, context?: Function): void;
-    publish(event: string, change: string | number | boolean | object | Array<any>): void;
+    publish(event: string, value: string | number | boolean | object | Array<any>): void;
     once(event: string): Promise<string | number | boolean | object | Array<any>>;
 }

package/index.js

@@ -1,4 +1,4 @@
-class LittlePubSub {
+export default class LittlePubSub {
     subscribers = {};
     verbose;
     constructor(verbose) {
@@ -28,13 +28,16 @@ class LittlePubSub {
         if (this.subscribers[event].handlers.length === 0)
             delete this.subscribers[event];
     }
-    publish(event, change) {
-        if (this.verbose && this.hasSubscribers(event) || this.subscribers?.[event].value !== change) {
-            this.subscribers[event].value = change;
-            this.subscribers[event].handlers.forEach((handler) => {
-                handler(change, this.subscribers[event].value);
-            });
-        }
+    publish(event, value) {
+        // always set value even when having no subscribers
+        if (!this.hasSubscribers(event))
+            this.subscribers[event] = {};
+        const oldValue = this.subscribers[event]?.value;
+        this.subscribers[event].value = value;
+        if (this.verbose || oldValue !== value)
+            for (const handler of this.subscribers[event].handlers) {
+                handler(value, oldValue);
+            }
     }
     once(event) {
         return new Promise((resolve) => {
@@ -46,5 +49,3 @@ class LittlePubSub {
         });
     }
 }
-
-export { LittlePubSub as default };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vandeurenglenn/little-pubsub",
-  "version": "1.4.3",
+  "version": "1.4.5",
   "description": "Publish & Subscribe",
   "main": "./index.js",
   "types": "./index.d.ts",
@@ -17,18 +17,17 @@
   "private": false,
   "flat": true,
   "scripts": {
-    "build": "rollup -c",
+    "build": "npx tsc",
     "test": "node test.js"
   },
   "keywords": [
     "publish",
     "subscribe",
-    "pubsub"
+    "pubsub",
+    "once"
   ],
   "devDependencies": {
-    "@rollup/plugin-typescript": "^10.0.1",
-    "rollup": "^3.5.1",
     "tape": "^4.13.0",
-    "tslib": "^2.4.1"
+    "typescript": "^5.1.6"
   }
 }

package/src/index.ts

@@ -1,5 +1,5 @@
 export default class LittlePubSub  {
-  subscribers: {} = {}
+  subscribers: {[index: string]: { value?: any, handlers?: Function[] }} = {}
   verbose: boolean
   
   constructor(verbose?: boolean) {
@@ -30,16 +30,19 @@ export default class LittlePubSub  {
     context = this._handleContext(handler, context)
     const index = this.subscribers[event].handlers.indexOf(handler.bind(context));
     this.subscribers[event].handlers.splice(index);
-    if (this.subscribers[event].handlers.length === 0) delete this.subscribers[event];    
+    if (this.subscribers[event].handlers.length === 0) delete this.subscribers[event]; 
   }
 
-  publish(event: string, change: string | number | boolean | object | Array<any>): void {
-    if (this.verbose && this.hasSubscribers(event) || this.subscribers?.[event].value !== change) {
-      this.subscribers[event].value = change;
-      this.subscribers[event].handlers.forEach((handler: Function) => {
-        handler(change, this.subscribers[event].value)
-      })
-    }
+  publish(event: string, value: string | number | boolean | object | Array<any>): void {
+    // always set value even when having no subscribers
+    if (!this.hasSubscribers(event)) this.subscribers[event] = {}
+    const oldValue = this.subscribers[event]?.value
+    this.subscribers[event].value = value;
+    
+    if (this.verbose || oldValue !== value) 
+      for (const handler of this.subscribers[event].handlers) {
+        handler(value, oldValue)
+      }
   }
 
   once(event: string): Promise<string | number | boolean | object | Array<any>> {

package/test.js

@@ -1,5 +1,5 @@
 import test from 'tape'
-import PubSub  from './dist/index.js'
+import PubSub  from './index.js'
 
 test('pubsub is defined', tape => {
   tape.plan(1)
@@ -19,15 +19,9 @@ test('pubsub is defined', tape => {
   })
 
   test('pubsub once', async (tape) => {
-    tape.plan(2)
+    tape.plan(1)
     setTimeout(() => pubsub.publish('on', true) ,1000)
     let value = await pubsub.once('on')
-    console.log(value);
-    tape.ok(Boolean(Object.keys(pubsub.subscribers).length === 0))
+    tape.ok((value === true))
   })
-  
-  // test('classIs', tape => {
-  //   tape.plan(1)
-  //   tape.ok(PubSub.isLittlePubSub(pubsub))
-  // })
 });

package/tsconfig.json

@@ -3,7 +3,7 @@
       "outDir": "./",
       "esModuleInterop": true,
       "allowSyntheticDefaultImports": true,
-      "allowJs": true,
+      "allowJs": false,
       "declaration": true,
       "target": "ES2022",
       "module": "es2022"

package/rollup.config.js

@@ -1,14 +0,0 @@
-import typescript from '@rollup/plugin-typescript';
-import tsconfig from './tsconfig.json' assert { type: 'json'}
-
-export default [
-	// CommonJS version, for Node, Browserify & Webpack
-	{
-		input: ['src/index.ts'],
-		output: {
-			dir: './',
-			format: 'es'
-		},
-		plugins: [typescript(tsconfig)]
-	}
-];