do not clear the queue as it may still be in use

ibnlanre · Nov 20, 2023 · f7d832b · f7d832b
1 parent e5e5574
commit f7d832b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/component/atom.ts b/src/component/atom.ts
@@ -154,7 +154,6 @@ export function atom<
   const executeQueue = (useArgs: UseArgs) => {
     dispose("rerun");
     queue.forEach((fn) => fn(...useArgs));
-    queue.clear();
   };
 
   const setValueWithArgs = (value: SetStateAction<State>) => {

diff --git a/src/utilities/useShallowEffect.ts b/src/utilities/useShallowEffect.ts
@@ -75,11 +75,11 @@ function useShallowCompare(dependencies?: DependencyList): [number] {
  *
  * @returns {() => () => void} A function to trigger the debounced effect.
  */
-const debounce = (effect: EffectCallback, delay: number): (() => void) => {
+const debounce = (effect: EffectCallback, delay: number) => {
   let destructor: ReturnType<EffectCallback>;
   let timeout: NodeJS.Timeout;
 
-  const later = (): (() => void) => {
+  const later = () => {
     if (timeout) clearTimeout(timeout);
 
     timeout = setTimeout(() => {
@@ -109,5 +109,8 @@ export function useDebouncedShallowEffect(
   dependencies: DependencyList = [],
   delay: number
 ): void {
-  useEffect(debounce(effect, delay), useShallowCompare(dependencies));
+  useEffect(
+    delay ? debounce(effect, delay) : effect,
+    useShallowCompare(dependencies)
+  );
 }