fix(undo-redo): clear previous item

Author: GregOnNet

libs/ngrx-toolkit/src/lib/with-undo-redo.ts

@@ -5,7 +5,8 @@ import {
   withComputed,
   withHooks,
   withMethods,
-  EmptyFeatureResult, SignalStoreFeatureResult
+  EmptyFeatureResult,
+  SignalStoreFeatureResult,
 } from '@ngrx/signals';
 import { Signal, effect, signal, untracked, isSignal } from '@angular/core';
 import { capitalize } from './with-data-service';
@@ -16,7 +17,7 @@ export type NormalizedUndoRedoOptions = {
   maxStackSize: number;
   collections?: string[];
   keys: string[];
-  skip: number,
+  skip: number;
 };
 
 const defaultOptions: NormalizedUndoRedoOptions = {
@@ -41,29 +42,35 @@ type NonNever<T> = T extends never ? never : T;
 
 type ExtractEntityCollection<T> = T extends `${infer U}Entities` ? U : never;
 
-type ExtractEntityCollections<Store extends SignalStoreFeatureResult> = NonNever<{
-  [K in keyof Store['props']]: ExtractEntityCollection<K>;
-}[keyof Store['props']]>;
+type ExtractEntityCollections<Store extends SignalStoreFeatureResult> =
+  NonNever<
+    {
+      [K in keyof Store['props']]: ExtractEntityCollection<K>;
+    }[keyof Store['props']]
+  >;
 
-type OptionsForState<Store extends SignalStoreFeatureResult> = Partial<Omit<NormalizedUndoRedoOptions, 'collections' | 'keys'>> & {
+type OptionsForState<Store extends SignalStoreFeatureResult> = Partial<
+  Omit<NormalizedUndoRedoOptions, 'collections' | 'keys'>
+> & {
   collections?: ExtractEntityCollections<Store>[];
   keys?: (keyof Store['state'])[];
 };
 
-export function withUndoRedo<
-  Input extends EmptyFeatureResult>(options?: OptionsForState<Input>): SignalStoreFeature<
+export function withUndoRedo<Input extends EmptyFeatureResult>(
+  options?: OptionsForState<Input>
+): SignalStoreFeature<
   Input,
   EmptyFeatureResult & {
-  props: {
-    canUndo: Signal<boolean>;
-    canRedo: Signal<boolean>;
-  };
-  methods: {
-    undo: () => void;
-    redo: () => void;
-    clearStack: () => void;
-  };
-}
+    props: {
+      canUndo: Signal<boolean>;
+      canRedo: Signal<boolean>;
+    };
+    methods: {
+      undo: () => void;
+      redo: () => void;
+      clearStack: () => void;
+    };
+  }
 > {
   let previous: StackItem | null = null;
   let skipOnce = false;
@@ -130,14 +137,19 @@ export function withUndoRedo<
       clearStack(): void {
         undoStack.splice(0);
         redoStack.splice(0);
+
+        previous = null;
+
         updateInternal();
       },
     })),
     withHooks({
       onInit(store) {
         effect(() => {
           const cand = keys.reduce((acc, key) => {
-            const s = (store as Record<string | keyof Input['state'], unknown>)[key];
+            const s = (store as Record<string | keyof Input['state'], unknown>)[
+              key
+            ];
             if (s && isSignal(s)) {
               return {
                 ...acc,