@figliolia/galena 2.2.5 → 2.2.7

package/dist/cjs/Galena/Galena.js

@@ -63,9 +63,9 @@ class Galena extends Guards_1.Guards {
     constructor(middleware = []) {
         super();
         this.state = {};
-        this.subscriptions = new Map();
         this.middleware = [];
         this.IDs = new event_emitter_1.AutoIncrementingID();
+        this.subscriptions = new Map();
         this.middleware = middleware;
     }
     /**
@@ -176,7 +176,7 @@ class Galena extends Guards_1.Guards {
             stateSubscriptions.push([
                 key,
                 this.state[key].subscribe(() => {
-                    callback(this.state);
+                    void callback(this.state);
                 }),
             ]);
         }
@@ -195,8 +195,8 @@ class Galena extends Guards_1.Guards {
         if (IDs) {
             for (const [state, ID] of IDs) {
                 this.state[state].unsubscribe(ID);
-                this.subscriptions.delete(ID);
             }
+            this.subscriptions.delete(ID);
         }
     }
     /**
@@ -208,21 +208,13 @@ class Galena extends Guards_1.Guards {
      * subscriptions
      */
     reIndexSubscriptions(name) {
-        var _a;
         for (const [ID, unitSubscriptions] of this.subscriptions) {
-            for (const [state, subscriptionID] of unitSubscriptions) {
-                const callback = (_a = this.state[state]["emitter"]
-                    .get(state)) === null || _a === void 0 ? void 0 : _a.get(subscriptionID);
-                if (callback) {
-                    unitSubscriptions.push([
-                        name,
-                        this.state[name].subscribe(() => {
-                            void callback(this.state);
-                        }),
-                    ]);
-                    this.subscriptions.set(ID, unitSubscriptions);
-                    break;
-                }
+            const [state, listenerID] = unitSubscriptions[0];
+            const subscriptions = this.state[state]["emitter"].get(state);
+            const listener = subscriptions === null || subscriptions === void 0 ? void 0 : subscriptions.get(listenerID);
+            if (listener) {
+                unitSubscriptions.push([name, this.state[name].subscribe(listener)]);
+                this.subscriptions.set(ID, unitSubscriptions);
             }
         }
     }

package/dist/cjs/Middlewares/Profiler.js

@@ -29,7 +29,7 @@ class Profiler extends Middleware_1.Middleware {
             const endTime = performance.now();
             const diff = endTime - this.startTime;
             if (diff > this.threshold) {
-                console.warn("Slow state transition detected", nextState);
+                console.warn(`A slow state transition was detected on ${nextState.name}`, nextState);
                 console.warn(`The last transition took ${diff}ms`);
             }
         }

package/dist/mjs/Galena/Galena.js

@@ -58,9 +58,9 @@ import { Guards } from "./Guards.js";
  */
 export class Galena extends Guards {
     state = {};
-    subscriptions = new Map();
     middleware = [];
     IDs = new AutoIncrementingID();
+    subscriptions = new Map();
     constructor(middleware = []) {
         super();
         this.middleware = middleware;
@@ -173,7 +173,7 @@ export class Galena extends Guards {
             stateSubscriptions.push([
                 key,
                 this.state[key].subscribe(() => {
-                    callback(this.state);
+                    void callback(this.state);
                 }),
             ]);
         }
@@ -192,8 +192,8 @@ export class Galena extends Guards {
         if (IDs) {
             for (const [state, ID] of IDs) {
                 this.state[state].unsubscribe(ID);
-                this.subscriptions.delete(ID);
             }
+            this.subscriptions.delete(ID);
         }
     }
     /**
@@ -206,20 +206,12 @@ export class Galena extends Guards {
      */
     reIndexSubscriptions(name) {
         for (const [ID, unitSubscriptions] of this.subscriptions) {
-            for (const [state, subscriptionID] of unitSubscriptions) {
-                const callback = this.state[state]["emitter"]
-                    .get(state)
-                    ?.get(subscriptionID);
-                if (callback) {
-                    unitSubscriptions.push([
-                        name,
-                        this.state[name].subscribe(() => {
-                            void callback(this.state);
-                        }),
-                    ]);
-                    this.subscriptions.set(ID, unitSubscriptions);
-                    break;
-                }
+            const [state, listenerID] = unitSubscriptions[0];
+            const subscriptions = this.state[state]["emitter"].get(state);
+            const listener = subscriptions?.get(listenerID);
+            if (listener) {
+                unitSubscriptions.push([name, this.state[name].subscribe(listener)]);
+                this.subscriptions.set(ID, unitSubscriptions);
             }
         }
     }

package/dist/mjs/Middlewares/Profiler.js

@@ -27,7 +27,7 @@ export class Profiler extends Middleware {
             const endTime = performance.now();
             const diff = endTime - this.startTime;
             if (diff > this.threshold) {
-                console.warn("Slow state transition detected", nextState);
+                console.warn(`A slow state transition was detected on ${nextState.name}`, nextState);
                 console.warn(`The last transition took ${diff}ms`);
             }
         }

package/dist/types/Galena/Galena.d.ts

@@ -1,6 +1,7 @@
 import type { Middleware } from "../Middleware/Middleware";
 import { State } from "./State";
 import { Guards } from "./Guards";
+import type { Subscription } from "./types";
 /**
  * ## Galena
  *
@@ -58,9 +59,9 @@ import { Guards } from "./Guards";
  */
 export declare class Galena<T extends Record<string, State<any>> = Record<string, State<any>>> extends Guards {
     readonly state: T;
-    private readonly subscriptions;
     private readonly middleware;
     private readonly IDs;
+    private readonly subscriptions;
     constructor(middleware?: Middleware[]);
     /**
      * Compose State
@@ -139,7 +140,7 @@ export declare class Galena<T extends Record<string, State<any>> = Record<string
      * subscription, call `Galena.unsubscribeAll()` with the ID
      * returned
      */
-    subscribeAll(callback: (nextState: T) => void): string;
+    subscribeAll(callback: Subscription<T>): string;
     /**
      * Unsubscribe
      *

package/dist/types/Galena/State.d.ts

@@ -1,5 +1,6 @@
 import type { Middleware } from "../Middleware/Middleware";
 import { Scheduler } from "./Scheduler";
+import type { Subscription } from "./types";
 import { Priority } from "./types";
 /**
  * ### State
@@ -200,7 +201,7 @@ export declare class State<T extends any = any> extends Scheduler {
      * callback you provide will execute each time state changes.
      * Returns a unique identifier for your subscription
      */
-    subscribe(callback: (nextState: T) => void): string;
+    subscribe(callback: Subscription<T>): string;
     /**
      * Unsubscribe
      *

package/dist/types/Galena/types.d.ts

@@ -8,3 +8,5 @@ export declare enum Priority {
     "BATCHED" = 3
 }
 export type Task = () => void;
+export type SubscriptionTuple = [state: string, ID: string];
+export type Subscription<T> = (nextState: T) => void | Promise<void>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@figliolia/galena",
-  "version": "2.2.5",
+  "version": "2.2.7",
   "description": "A performant state management library supporting mutable state, batched updates, middleware and a rich development API",
   "main": "dist/cjs/index.js",
   "module": "dist/mjs/index.js",

package/src/Galena/Galena.ts

@@ -2,6 +2,7 @@ import { AutoIncrementingID } from "@figliolia/event-emitter";
 import type { Middleware } from "Middleware/Middleware";
 import { State } from "Galena/State";
 import { Guards } from "./Guards";
+import type { Subscription, SubscriptionTuple } from "./types";
 
 /**
  * ## Galena
@@ -62,12 +63,9 @@ export class Galena<
   T extends Record<string, State<any>> = Record<string, State<any>>
 > extends Guards {
   public readonly state = {} as T;
-  private readonly subscriptions = new Map<
-    string,
-    [state: string, ID: string][]
-  >();
   private readonly middleware: Middleware[] = [];
   private readonly IDs = new AutoIncrementingID();
+  private readonly subscriptions = new Map<string, SubscriptionTuple[]>();
   constructor(middleware: Middleware[] = []) {
     super();
     this.middleware = middleware;
@@ -200,13 +198,13 @@ export class Galena<
    * subscription, call `Galena.unsubscribeAll()` with the ID
    * returned
    */
-  public subscribeAll(callback: (nextState: T) => void) {
+  public subscribeAll(callback: Subscription<T>) {
     const stateSubscriptions: [state: string, ID: string][] = [];
     for (const key in this.state) {
       stateSubscriptions.push([
         key,
         this.state[key].subscribe(() => {
-          callback(this.state);
+          void callback(this.state);
         }),
       ]);
     }
@@ -226,8 +224,8 @@ export class Galena<
     if (IDs) {
       for (const [state, ID] of IDs) {
         this.state[state].unsubscribe(ID);
-        this.subscriptions.delete(ID);
       }
+      this.subscriptions.delete(ID);
     }
   }
 
@@ -241,20 +239,12 @@ export class Galena<
    */
   private reIndexSubscriptions(name: string) {
     for (const [ID, unitSubscriptions] of this.subscriptions) {
-      for (const [state, subscriptionID] of unitSubscriptions) {
-        const callback = this.state[state]["emitter"]
-          .get(state)
-          ?.get(subscriptionID);
-        if (callback) {
-          unitSubscriptions.push([
-            name,
-            this.state[name].subscribe(() => {
-              void callback(this.state);
-            }),
-          ]);
-          this.subscriptions.set(ID, unitSubscriptions);
-          break;
-        }
+      const [state, listenerID] = unitSubscriptions[0];
+      const subscriptions = this.state[state]["emitter"].get(state);
+      const listener = subscriptions?.get(listenerID);
+      if (listener) {
+        unitSubscriptions.push([name, this.state[name].subscribe(listener)]);
+        this.subscriptions.set(ID, unitSubscriptions);
       }
     }
   }

package/src/Galena/State.ts

@@ -2,6 +2,7 @@ import { MiddlewareEvents } from "Middleware/types";
 import type { Middleware } from "Middleware/Middleware";
 import { EventEmitter } from "@figliolia/event-emitter";
 import { Scheduler } from "./Scheduler";
+import type { Subscription } from "./types";
 import { Priority, type MutationEvent } from "./types";
 
 /**
@@ -260,7 +261,7 @@ export class State<T extends any = any> extends Scheduler {
    * callback you provide will execute each time state changes.
    * Returns a unique identifier for your subscription
    */
-  public subscribe(callback: (nextState: T) => void) {
+  public subscribe(callback: Subscription<T>) {
     return this.emitter.on(this.name, callback);
   }
 

package/src/Galena/types.ts

@@ -11,3 +11,7 @@ export enum Priority {
 }
 
 export type Task = () => void;
+
+export type SubscriptionTuple = [state: string, ID: string];
+
+export type Subscription<T> = (nextState: T) => void | Promise<void>;

package/src/Middlewares/Profiler.ts

@@ -30,7 +30,10 @@ export class Profiler extends Middleware {
       const endTime = performance.now();
       const diff = endTime - this.startTime;
       if (diff > this.threshold) {
-        console.warn("Slow state transition detected", nextState);
+        console.warn(
+          `A slow state transition was detected on ${nextState.name}`,
+          nextState
+        );
         console.warn(`The last transition took ${diff}ms`);
       }
     }