Fix(@inquirer/core): Reduce memory consumption of usePrefix in loadin…

…g mode
SBoudrias · Jun 19, 2024 · 3d15d50 · 3d15d50
1 parent 9735dea
commit 3d15d50
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/packages/core/src/lib/use-prefix.mts b/packages/core/src/lib/use-prefix.mts
@@ -16,20 +16,21 @@ export function usePrefix({
 
   useEffect((): void | (() => unknown) => {
     if (isLoading) {
-      const timeout = setTimeout(
+      let inc = -1;
+      const interval = setInterval(
         AsyncResource.bind(() => {
-          setTick(tick + 1);
+          inc = inc + 1;
+          setTick(inc % spinner.frames.length);
         }),
         spinner.interval,
       );
 
-      return () => clearTimeout(timeout);
+      return () => clearInterval(interval);
     }
-  }, [isLoading, tick]);
+  }, [isLoading]);
 
   if (isLoading) {
-    const frame = tick % spinner.frames.length;
-    return spinner.frames[frame]!;
+    return spinner.frames[tick]!;
   }
 
   return prefix;