shave 9 bytes (#94)

Author: charkour

packages/zundo/src/index.ts

@@ -32,10 +32,10 @@ declare module 'zustand/vanilla' {
   }
 }
 
-const zundoImpl = <TState>(
+export const temporal = (<TState>(
   config: StateCreator<TState, [], []>,
   {
-    partialize = (state: TState) => state,
+    partialize = (state) => state,
     handleSet = (handleSetCb) => handleSetCb,
     ...restOptions
   } = {} as ZundoOptions<TState>,
@@ -86,7 +86,6 @@ const zundoImpl = <TState>(
     );
   };
   return configWithTemporal as StateCreator<TState, [], []>;
-};
+}) as unknown as Zundo;
 
-export const temporal = zundoImpl as unknown as Zundo;
 export type { ZundoOptions, Zundo, TemporalState };

packages/zundo/src/temporal.ts

@@ -28,39 +28,34 @@ export const createVanillaTemporal = <TState>(
           // Fastest way to clone an array on Chromium. Needed to create a new array reference
           const pastStates = get().pastStates.slice();
           const futureStates = get().futureStates.slice();
-          if (pastStates.length === 0) {
-            return;
-          }
-
-          // Based on the steps, get values from the pastStates array and push them to the futureStates array
-          for (let i = 0; i < steps; i++) {
-            const pastState = pastStates.pop();
-            if (pastState) {
-              futureStates.push(partialize(userGet()));
-              userSet(pastState);
+          if (pastStates.length) {
+            // Based on the steps, get values from the pastStates array and push them to the futureStates array
+            for (let i = 0; i < steps; i++) {
+              const pastState = pastStates.pop();
+              if (pastState) {
+                futureStates.push(partialize(userGet()));
+                userSet(pastState);
+              }
             }
-          }
 
-          set({ pastStates, futureStates });
+            set({ pastStates, futureStates });
+          }
         },
         redo: (steps = 1) => {
           // Fastest way to clone an array on Chromium. Needed to create a new array reference
           const pastStates = get().pastStates.slice();
           const futureStates = get().futureStates.slice();
-          if (futureStates.length === 0) {
-            return;
-          }
-
-          // Based on the steps, get values from the futureStates array and push them to the pastStates array
-          for (let i = 0; i < steps; i++) {
-            const futureState = futureStates.pop();
-            if (futureState) {
-              pastStates.push(partialize(userGet()));
-              userSet(futureState);
+          if (futureStates.length) {
+            // Based on the steps, get values from the futureStates array and push them to the pastStates array
+            for (let i = 0; i < steps; i++) {
+              const futureState = futureStates.pop();
+              if (futureState) {
+                pastStates.push(partialize(userGet()));
+                userSet(futureState);
+              }
             }
+            set({ pastStates, futureStates });
           }
-
-          set({ pastStates, futureStates });
         },
         clear: () => {
           set({ pastStates: [], futureStates: [] });
@@ -78,10 +73,10 @@ export const createVanillaTemporal = <TState>(
         // Internal properties
         __onSave: onSave,
         __handleSet: (pastState) => {
-          const trackingStatus = get().trackingStatus,
-            onSave = get().__onSave,
-            pastStates = get().pastStates.slice(),
-            currentState = partialize(userGet());
+          const trackingStatus = get().trackingStatus;
+          const onSave = get().__onSave;
+          const pastStates = get().pastStates.slice();
+          const currentState = partialize(userGet());
           if (
             trackingStatus === 'tracking' &&
             !equality?.(currentState, pastState)

packages/zundo/src/types.ts

@@ -65,5 +65,5 @@ export type Write<T, U> = Omit<T, keyof U> & U;
 
 export type TemporalState<TState> = Omit<
   TemporalStateWithInternals<TState>,
-  '__onSave' | '__handleUserSet'
+  '__onSave' | '__handleSet'
 >;