Remove init action handling

piotrwitek · Dec 19, 2023 · 050e64f · 050e64f
1 parent 8be5b73
commit 050e64f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 37 deletions.
diff --git a/README.md b/README.md
@@ -398,15 +398,6 @@ store.dispatch(add(3));
 store.dispatch(reset());
 ```
 
-The `defaultHandler` isn't invoked at initialization of the redux store (default behavior).
-To enforce invoking the `defaultHandler` at initialization of the redux store, supply `true` as second argument to the `defaultHandler`.
-Note, that the initialization action doesn't have a payload.
-```ts
-const counterReducer = createReducer(0)
-  .handleAction(add, (state, action) => state + action.payload)
-  .defaultHandler((state, action) => state = 0, true);
-```
-
 #### Alternative usage with regular switch reducer
 
 First we need to start by generating a **tagged union type** of actions (`TodosAction`). It's very easy to do by using `ActionType` **type-helper**.

diff --git a/src/create-reducer.spec.snap.ts b/src/create-reducer.spec.snap.ts
@@ -106,8 +106,6 @@ const initialState = 0;
       fn(0, add(4)); // => 4
     });
     // @dts-jest:pass
-    counterReducer5(0, reduxInit()); // => 0
-    // @dts-jest:pass
     counterReducer5(0, {} as any); // => 1
     // @dts-jest:pass
     counterReducer5(0, add(4)); // => 4
@@ -299,8 +297,6 @@ const initialState = 0;
       fn(0, add(4)); // => 4
     });
     // @dts-jest:pass
-    counterReducer5(0, reduxInit()); // => 0;
-    // @dts-jest:pass
     counterReducer5(0, increment()); // => 1;
     // @dts-jest:pass
     counterReducer5(0, add(4)); // => 4;

diff --git a/src/create-reducer.spec.ts b/src/create-reducer.spec.ts
@@ -106,8 +106,6 @@ const initialState = 0;
       fn(0, add(4)); // => 4
     });
     // @dts-jest:pass
-    counterReducer5(0, reduxInit()); // => 0
-    // @dts-jest:pass
     counterReducer5(0, {} as any); // => 1
     // @dts-jest:pass
     counterReducer5(0, add(4)); // => 4
@@ -299,8 +297,6 @@ const initialState = 0;
       fn(0, add(4)); // => 4
     });
     // @dts-jest:pass
-    counterReducer5(0, reduxInit()); // => 0;
-    // @dts-jest:pass
     counterReducer5(0, increment()); // => 1;
     // @dts-jest:pass
     counterReducer5(0, add(4)); // => 4;

diff --git a/src/create-reducer.ts b/src/create-reducer.ts
@@ -46,8 +46,7 @@ type HandleDefaultActionChainApi<
   THandledAction extends ReturnType<TActionCreator>,
   TOutputAction extends Exclude<TInputAction, THandledAction>
 >(
-  reducer: (state: TState, action: THandledAction) => TState,
-  executeAtInitialization?: boolean
+  reducer: (state: TState, action: THandledAction) => TState
 ) => [TOutputAction] extends [Action]
   ? Reducer<TState, TRootAction> & {
       handlers: Record<
@@ -110,8 +109,7 @@ type RootAction = Types extends { RootAction: infer T } ? T : any;
 export function createReducer<TState, TRootAction extends Action = RootAction>(
   initialState: TState,
   initialHandlers: InitialHandler<TState, TRootAction> = {},
-  defaultReducer?: Reducer<TState, TRootAction>,
-  defaultReducerExecutedAtInitialization: boolean = false
+  defaultReducer?: Reducer<TState, TRootAction>
 ) {
   const handlers: any = {
     ...initialHandlers,
@@ -131,12 +129,7 @@ export function createReducer<TState, TRootAction extends Action = RootAction>(
         );
       }
       return reducer(state, action);
-    } else if (
-      defaultReducer &&
-      (defaultReducerExecutedAtInitialization ||
-        (!defaultReducerExecutedAtInitialization &&
-          !initializationActionTypes.test(action.type)))
-    ) {
+    } else if (defaultReducer && !initializationActionTypes.test(action.type)) {
       return defaultReducer(state, action);
     } else {
       return state;
@@ -179,16 +172,8 @@ export function createReducer<TState, TRootAction extends Action = RootAction>(
     | HandleActionChainApi<TState, TRootAction, TRootAction>
     | HandleTypeChainApi<TState, TRootAction, TRootAction>;
 
-  const defaultHandler = ((
-    reducer: any,
-    executeAtInitialization: boolean = false
-  ) => {
-    return createReducer<TState, TRootAction>(
-      initialState,
-      handlers,
-      reducer,
-      executeAtInitialization
-    );
+  const defaultHandler = ((reducer: any) => {
+    return createReducer<TState, TRootAction>(initialState, handlers, reducer);
   }) as HandleDefaultActionChainApi<TState, TRootAction, TRootAction>;
 
   const chainApi = Object.assign(rootReducer, {