Update libs/hooks/debounce/src/lib/debounce-hook.ts

Co-authored-by: Lukas Reining <[email protected]>
Signed-off-by: Todd Baert <[email protected]>

Author: toddbaert

libs/hooks/debounce/src/lib/debounce-hook.ts

@@ -158,33 +158,32 @@ export class DebounceHook<T extends FlagValue = FlagValue> implements Hook {
     // the cache key is a concatenation of the result of calling keyGenCallback and the stage
     const dynamicKey = keyGenCallback();
 
-    // if the keyGenCallback returns nothing, we don't do any caching
-    if (dynamicKey) {
-      const cacheKeySuffix = stage;
-      const cacheKey = `${dynamicKey}::${cacheKeySuffix}`;
-      const got = this.cache.get(cacheKey);
+    // if the keyGenCallback returns nothing, we don't do any caching 
+    if (!dynamicKey) {
+      hookCallback.call(this.innerHook);
+    }
+    
+    const cacheKeySuffix = stage;
+    const cacheKey = `${dynamicKey}::${cacheKeySuffix}`;
+    const got = this.cache.get(cacheKey);
 
-      if (got) {
-        // throw cached errors
-        if (got instanceof CachedError) {
-          throw got;
-        }
-        return;
-      } else {
-        try {
-          hookCallback.call(this.innerHook);
-          this.cache.set(cacheKey, true);
-        } catch (error: unknown) {
-          if (this.cacheErrors) {
-            // cache error
-            this.cache.set(cacheKey, new CachedError(error));
-          }
-          throw error;
-        }
-        return;
+    if (got) {
+      // throw cached errors
+      if (got instanceof CachedError) {
+        throw got;
       }
-    } else {
+      return;
+    } 
+    
+    try {
       hookCallback.call(this.innerHook);
-    }
+      this.cache.set(cacheKey, true);
+    } catch (error: unknown) {
+      if (this.cacheErrors) {
+        // cache error
+        this.cache.set(cacheKey, new CachedError(error));
+      }
+      throw error;
+    }    
   }
 }