@figliolia/galena 2.0.4 → 2.0.5

package/dist/Galena/Guards.d.ts

@@ -17,7 +17,7 @@ export declare class Guards {
      * developers know that they are attempting to manipulate a
      * unit of state that has not yet been initialized
      */
-    protected warnForUndefinedStates: (name: string, state: Record<string, State<any>>) => void;
+    protected warnForUndefinedStates<T extends Record<string, State<any>>>(name: string, state: T): void;
     /**
      * Guard Duplicate States
      *
@@ -25,12 +25,5 @@ export declare class Guards {
      * more than one state with the same name on a single
      * `Galena` instance
      */
-    protected guardDuplicateStates: (name: string, state: Record<string, State<any>>) => void;
-    /**
-     * Development Guard
-     *
-     * Wraps a provided function in a check for `process.env.NODE_ENV`
-     * equaling "development". Returns the callback
-     */
-    private developmentGuard;
+    protected guardDuplicateStates<T extends Record<string, State<any>>>(name: string, state: T): void;
 }

package/dist/Galena/Guards.js

@@ -11,45 +11,30 @@ exports.Guards = void 0;
  * of the library
  */
 class Guards {
-    constructor() {
-        /**
-         * Warn For Undefined States
-         *
-         * In Galena, it's normal to lazy initialize a unit of state
-         * in attached to a `Galena` instance. This warning lets
-         * developers know that they are attempting to manipulate a
-         * unit of state that has not yet been initialized
-         */
-        this.warnForUndefinedStates = this.developmentGuard((name, state) => {
-            if (!process.env.IGNORE_LAZY_STATE_WARNINGS && !(name in state)) {
-                console.warn(`A unit of state with the name "${name}" does not yet exist on this Galena instance. If this is expected, you can ignore this warning. To make these warnings go away, you can enable this process argument - "IGNORE_LAZY_STATE_WARNING=1"`);
-            }
-        });
-        /**
-         * Guard Duplicate States
-         *
-         * Throws an error if a developer attempts to create
-         * more than one state with the same name on a single
-         * `Galena` instance
-         */
-        this.guardDuplicateStates = this.developmentGuard((name, state) => {
-            if (name in state) {
-                throw new Error(`A unit of state with the name "${name}" already exists on this Galena instance. Please re-name this new unit of state to something unique`);
-            }
-        });
+    /**
+     * Warn For Undefined States
+     *
+     * In Galena, it's normal to lazy initialize a unit of state
+     * in attached to a `Galena` instance. This warning lets
+     * developers know that they are attempting to manipulate a
+     * unit of state that has not yet been initialized
+     */
+    warnForUndefinedStates(name, state) {
+        if (!(name in state)) {
+            console.warn(`A unit of state with the name "${name}" does not yet exist on this Galena instance. If this is expected, you can ignore this warning`);
+        }
     }
     /**
-     * Development Guard
+     * Guard Duplicate States
      *
-     * Wraps a provided function in a check for `process.env.NODE_ENV`
-     * equaling "development". Returns the callback
+     * Throws an error if a developer attempts to create
+     * more than one state with the same name on a single
+     * `Galena` instance
      */
-    developmentGuard(guard) {
-        return (...args) => {
-            if (process.env.NODE_ENV === "development") {
-                guard(...args);
-            }
-        };
+    guardDuplicateStates(name, state) {
+        if (name in state) {
+            console.warn(`A unit of state with the name "${name}" already exists on this Galena instance. Please re-name this new unit of state to something unique`);
+        }
     }
 }
 exports.Guards = Guards;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@figliolia/galena",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "description": "A performant state management library supporting mutable state, batched updates, middleware and a rich development API",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/Galena/Guards.ts

@@ -18,18 +18,16 @@ export class Guards {
    * developers know that they are attempting to manipulate a
    * unit of state that has not yet been initialized
    */
-  protected warnForUndefinedStates = this.developmentGuard(
-    <T extends Record<string, State<any>>, K extends Extract<keyof T, string>>(
-      name: K,
-      state: T
-    ) => {
-      if (!process.env.IGNORE_LAZY_STATE_WARNINGS && !(name in state)) {
-        console.warn(
-          `A unit of state with the name "${name}" does not yet exist on this Galena instance. If this is expected, you can ignore this warning. To make these warnings go away, you can enable this process argument - "IGNORE_LAZY_STATE_WARNING=1"`
-        );
-      }
+  protected warnForUndefinedStates<T extends Record<string, State<any>>>(
+    name: string,
+    state: T
+  ) {
+    if (!(name in state)) {
+      console.warn(
+        `A unit of state with the name "${name}" does not yet exist on this Galena instance. If this is expected, you can ignore this warning`
+      );
     }
-  );
+  }
 
   /**
    * Guard Duplicate States
@@ -38,30 +36,14 @@ export class Guards {
    * more than one state with the same name on a single
    * `Galena` instance
    */
-  protected guardDuplicateStates = this.developmentGuard(
-    <T extends Record<string, State<any>>, K extends Extract<keyof T, string>>(
-      name: K,
-      state: T
-    ) => {
-      if (name in state) {
-        throw new Error(
-          `A unit of state with the name "${name}" already exists on this Galena instance. Please re-name this new unit of state to something unique`
-        );
-      }
+  protected guardDuplicateStates<T extends Record<string, State<any>>>(
+    name: string,
+    state: T
+  ) {
+    if (name in state) {
+      console.warn(
+        `A unit of state with the name "${name}" already exists on this Galena instance. Please re-name this new unit of state to something unique`
+      );
     }
-  );
-
-  /**
-   * Development Guard
-   *
-   * Wraps a provided function in a check for `process.env.NODE_ENV`
-   * equaling "development". Returns the callback
-   */
-  private developmentGuard<F extends (...args: any[]) => void>(guard: F) {
-    return (...args: Parameters<F>) => {
-      if (process.env.NODE_ENV === "development") {
-        guard(...args);
-      }
-    };
   }
 }