From 27a41c7a1bb9d09e6b24dfcf9af3e92175536da1 Mon Sep 17 00:00:00 2001
From: Paulo Cesar <email@pocesar.e4ward.com>
Date: Sat, 24 Dec 2016 16:46:26 -0200
Subject: [PATCH 1/3] add inferred parameters in createAction

---
 redux-actions/index.d.ts             | 55 ++++++++++++++++++++--------
 redux-actions/redux-actions-tests.ts | 37 +++++++++++++++++++
 redux-actions/tslint.json            |  1 +
 scripts/new-package.js               |  1 +
 4 files changed, 78 insertions(+), 16 deletions(-)
 create mode 100644 redux-actions/tslint.json

diff --git a/redux-actions/index.d.ts b/redux-actions/index.d.ts
index b5415bca9d..54f373ead0 100644
--- a/redux-actions/index.d.ts
+++ b/redux-actions/index.d.ts
@@ -1,4 +1,4 @@
-// Type definitions for redux-actions v0.8.0
+// Type definitions for redux-actions 1.2
 // Project: https://github.com/acdlite/redux-actions
 // Definitions by: Jack Hsu <https://github.com/jaysoo>, Alex Gorbatchev <https://github.com/alexgorbatchev>
 // Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
@@ -26,18 +26,46 @@ declare namespace ReduxActions {
         [actionType: string]: Reducer<State, Payload>;
     }
 
-    interface Reducer<State, Payload> {
-        (state: State, action: Action<Payload>): State;
-    }
+    type Reducer<State, Payload> = (state: State, action: Action<Payload>) => State;
 
-    interface ReducerMeta<State, Payload, Meta> extends Reducer<State, Payload> {
-        (state: State, action: ActionMeta<Payload, Meta>): State;
-    }
+    type ReducerMeta<State, Payload, Meta> = (state: State, action: ActionMeta<Payload, Meta>) => State;
+
+    /** argument inferring borrowed from lodash definitions */
+    type Function0<R> = () => R;
+    type Function1<T1, R> = (t1: T1) => R;
+    type Function2<T1, T2, R> = (t1: T1, t2: T2) => R;
+    type Function3<T1, T2, T3, R>  = (t1: T1, t2: T2, t3: T3) => R;
+    type Function4<T1, T2, T3, T4, R> = (t1: T1, t2: T2, t3: T3, t4: T4) => R;
+    type FunctionAny<R> = (...args: any[]) => R;
 
     export function createAction<Payload>(
         actionType: string,
-        payloadCreator?: (...args: any[]) => Payload,
-    ): (...args: any[]) => Action<Payload>;
+        payloadCreator: Function0<Payload>
+    ): Function0<Action<Payload>>;
+
+    export function createAction<Payload, Arg1>(
+        actionType: string,
+        payloadCreator: Function1<Arg1, Payload>
+    ): Function1<Arg1, Action<Payload>>;
+
+    export function createAction<Payload, Arg1, Arg2>(
+        actionType: string,
+        payloadCreator: Function2<Arg1, Arg2, Payload>
+    ): Function2<Arg1, Arg2, Action<Payload>>;
+
+    export function createAction<Payload, Arg1, Arg2, Arg3>(
+        actionType: string,
+        payloadCreator: Function3<Arg1, Arg2, Arg3, Payload>
+    ): Function3<Arg1, Arg2, Arg3, Action<Payload>>;
+
+    export function createAction<Payload, Arg1, Arg2, Arg3, Arg4>(
+        actionType: string,
+        payloadCreator: Function4<Arg1, Arg2, Arg3, Arg4, Payload>
+    ): Function4<Arg1, Arg2, Arg3, Arg4, Action<Payload>>;
+
+    export function createAction<Payload>(
+        actionType: string
+    ): FunctionAny<Action<Payload>>;
 
     export function createAction<Payload, Meta>(
         actionType: string,
@@ -45,13 +73,8 @@ declare namespace ReduxActions {
         metaCreator: (...args: any[]) => Meta
     ): (...args: any[]) => ActionMeta<Payload, Meta>;
 
-    export function handleAction<StateAndPayload>(
-        actionType: { toString: () => string },
-        reducer: Reducer<StateAndPayload, StateAndPayload> | ReducerMap<StateAndPayload, StateAndPayload>
-    ): Reducer<StateAndPayload, StateAndPayload>;
-
     export function handleAction<State, Payload>(
-        actionType: { toString(): string },
+        actionType: string | Function0<Action<Payload>> | Function1<any, Action<Payload>> | Function2<any, any, Action<Payload>> | Function3<any, any, any, Action<Payload>> | Function4<any, any, any, any, Action<Payload>> | FunctionAny<Action<Payload>>,
         reducer: Reducer<State, Payload> | ReducerMap<State, Payload>
     ): Reducer<State, Payload>;
 
@@ -71,6 +94,6 @@ declare namespace ReduxActions {
     ): Reducer<State, Payload>;
 
     export function combineActions(
-        ...actionTypes: { toString(): string }[]
+        ...actionTypes: Array<{ toString(): string }>
     ): { toString(): string };
 }
diff --git a/redux-actions/redux-actions-tests.ts b/redux-actions/redux-actions-tests.ts
index 4e585f9d9a..65ef3282ed 100644
--- a/redux-actions/redux-actions-tests.ts
+++ b/redux-actions/redux-actions-tests.ts
@@ -101,3 +101,40 @@ const typedActionHandlerWithReduceMap = ReduxActions.handleAction<TypedState, Ty
 );
 
 typedState = typedActionHandlerWithReduceMap({ value: 0 }, typedIncrementByActionWithMeta(10));
+
+const act = ReduxActions.createAction<string>('ACTION1')
+act('hello').payload === 'hello'
+
+const act2 = ReduxActions.createAction('ACTION2', (s: {load: boolean}) => s)
+act2({load: true}).payload.load == true
+
+const act3 = ReduxActions.createAction('ACTION3', (s: string) => ({s}))
+act3('hello').payload.s == 'hello'
+
+ReduxActions.handleAction<{ hello: string }, string>(act, (state, action) => {
+    return { hello: action.payload }
+})
+
+ReduxActions.handleAction<{ hello: { load: boolean } }, { load: boolean }>(act2, (state, action) => {
+    return { hello: action.payload }
+})
+
+ReduxActions.handleAction(act3, (state, action) => {
+    return { hello: action.payload.s }
+})
+
+/* can't do this until it lands in 2.2
+ReduxActions.handleAction(act, (state, action) => {
+    action.payload === 'hello'
+    return {}
+})
+
+ReduxActions.handleAction(act2, (state, action) => {
+    action.payload.load === true
+    return {}
+})
+
+ReduxActions.handleAction(act3, (state, action) => {
+    action.payload.s == 'hello'
+    return {}
+})*/
diff --git a/redux-actions/tslint.json b/redux-actions/tslint.json
new file mode 100644
index 0000000000..2221e40e4a
--- /dev/null
+++ b/redux-actions/tslint.json
@@ -0,0 +1 @@
+{ "extends": "../tslint.json" }
\ No newline at end of file
diff --git a/scripts/new-package.js b/scripts/new-package.js
index e235714fe0..c3fb666592 100644
--- a/scripts/new-package.js
+++ b/scripts/new-package.js
@@ -74,6 +74,7 @@ function getTSConfig() {
             "module": "commonjs",
             "target": "es6",
             "noImplicitAny": true,
+            "noImplicitThis": true,
             "strictNullChecks": true,
             "baseUrl": "../",
             "typeRoots": [

From ab6200ebeea6e486e2ec1e7caa89295e9998885b Mon Sep 17 00:00:00 2001
From: Paulo Cesar <email@pocesar.e4ward.com>
Date: Sun, 25 Dec 2016 08:13:09 -0200
Subject: [PATCH 2/3] meaningful function names and better next/throw

---
 redux-actions/index.d.ts             | 66 +++++++++++++++++-----------
 redux-actions/redux-actions-tests.ts | 16 +++++--
 2 files changed, 53 insertions(+), 29 deletions(-)

diff --git a/redux-actions/index.d.ts b/redux-actions/index.d.ts
index 54f373ead0..bd195c1c18 100644
--- a/redux-actions/index.d.ts
+++ b/redux-actions/index.d.ts
@@ -23,49 +23,65 @@ declare namespace ReduxActions {
     }
 
     interface ReducerMap<State, Payload> {
-        [actionType: string]: Reducer<State, Payload>;
+        [actionType: string]: Reducer<State, Payload> | ReducerNextThrow<State, Payload>;
     }
 
+    interface ReducerMapMeta<State, Payload, Meta> {
+        [actionType: string]: Reducer<State, Payload> | ReducerNextThrow<State, Payload>;
+    }
+
+    interface ReducerNextThrow<State, Payload> {
+        next?(state: State, action: Action<Payload>): State;
+        throw?(state: State, action: Action<Payload>): State;
+    }
+
+    interface ReducerNextThrowMeta<State, Payload, Meta> {
+        next?(state: State, action: ActionMeta<Payload, Meta>): State;
+        throw?(state: State, action: ActionMeta<Payload, Meta>): State;
+    }
+
+    type ActionFunctions<Payload> = ActionFunction0<Action<Payload>> | ActionFunction1<any, Action<Payload>> | ActionFunction2<any, any, Action<Payload>> | ActionFunction3<any, any, any, Action<Payload>> | ActionFunction4<any, any, any, any, Action<Payload>> | ActionFunctionAny<Action<Payload>>
+
     type Reducer<State, Payload> = (state: State, action: Action<Payload>) => State;
 
     type ReducerMeta<State, Payload, Meta> = (state: State, action: ActionMeta<Payload, Meta>) => State;
 
     /** argument inferring borrowed from lodash definitions */
-    type Function0<R> = () => R;
-    type Function1<T1, R> = (t1: T1) => R;
-    type Function2<T1, T2, R> = (t1: T1, t2: T2) => R;
-    type Function3<T1, T2, T3, R>  = (t1: T1, t2: T2, t3: T3) => R;
-    type Function4<T1, T2, T3, T4, R> = (t1: T1, t2: T2, t3: T3, t4: T4) => R;
-    type FunctionAny<R> = (...args: any[]) => R;
+    type ActionFunction0<R> = () => R;
+    type ActionFunction1<T1, R> = (t1: T1) => R;
+    type ActionFunction2<T1, T2, R> = (t1: T1, t2: T2) => R;
+    type ActionFunction3<T1, T2, T3, R>  = (t1: T1, t2: T2, t3: T3) => R;
+    type ActionFunction4<T1, T2, T3, T4, R> = (t1: T1, t2: T2, t3: T3, t4: T4) => R;
+    type ActionFunctionAny<R> = (...args: any[]) => R;
 
     export function createAction<Payload>(
         actionType: string,
-        payloadCreator: Function0<Payload>
-    ): Function0<Action<Payload>>;
+        payloadCreator: ActionFunction0<Payload>
+    ): ActionFunction0<Action<Payload>>;
 
     export function createAction<Payload, Arg1>(
         actionType: string,
-        payloadCreator: Function1<Arg1, Payload>
-    ): Function1<Arg1, Action<Payload>>;
+        payloadCreator: ActionFunction1<Arg1, Payload>
+    ): ActionFunction1<Arg1, Action<Payload>>;
 
     export function createAction<Payload, Arg1, Arg2>(
         actionType: string,
-        payloadCreator: Function2<Arg1, Arg2, Payload>
-    ): Function2<Arg1, Arg2, Action<Payload>>;
+        payloadCreator: ActionFunction2<Arg1, Arg2, Payload>
+    ): ActionFunction2<Arg1, Arg2, Action<Payload>>;
 
     export function createAction<Payload, Arg1, Arg2, Arg3>(
         actionType: string,
-        payloadCreator: Function3<Arg1, Arg2, Arg3, Payload>
-    ): Function3<Arg1, Arg2, Arg3, Action<Payload>>;
+        payloadCreator: ActionFunction3<Arg1, Arg2, Arg3, Payload>
+    ): ActionFunction3<Arg1, Arg2, Arg3, Action<Payload>>;
 
     export function createAction<Payload, Arg1, Arg2, Arg3, Arg4>(
         actionType: string,
-        payloadCreator: Function4<Arg1, Arg2, Arg3, Arg4, Payload>
-    ): Function4<Arg1, Arg2, Arg3, Arg4, Action<Payload>>;
+        payloadCreator: ActionFunction4<Arg1, Arg2, Arg3, Arg4, Payload>
+    ): ActionFunction4<Arg1, Arg2, Arg3, Arg4, Action<Payload>>;
 
     export function createAction<Payload>(
         actionType: string
-    ): FunctionAny<Action<Payload>>;
+    ): ActionFunctionAny<Action<Payload>>;
 
     export function createAction<Payload, Meta>(
         actionType: string,
@@ -74,26 +90,26 @@ declare namespace ReduxActions {
     ): (...args: any[]) => ActionMeta<Payload, Meta>;
 
     export function handleAction<State, Payload>(
-        actionType: string | Function0<Action<Payload>> | Function1<any, Action<Payload>> | Function2<any, any, Action<Payload>> | Function3<any, any, any, Action<Payload>> | Function4<any, any, any, any, Action<Payload>> | FunctionAny<Action<Payload>>,
-        reducer: Reducer<State, Payload> | ReducerMap<State, Payload>
+        actionType: string | ActionFunctions<Payload>,
+        reducer: Reducer<State, Payload> | ReducerNextThrow<State, Payload>
     ): Reducer<State, Payload>;
 
     export function handleAction<State, Payload, Meta>(
         actionType: { toString(): string },
-        reducer: ReducerMeta<State, Payload, Meta> | ReducerMap<State, Payload>
+        reducer: ReducerMeta<State, Payload, Meta> | ReducerNextThrowMeta<State, Payload, Meta>
     ): Reducer<State, Payload>;
 
     export function handleActions<StateAndPayload>(
         reducerMap: ReducerMap<StateAndPayload, StateAndPayload>,
-        initialState?: StateAndPayload
+        initialState: StateAndPayload
     ): Reducer<StateAndPayload, StateAndPayload>;
 
     export function handleActions<State, Payload>(
         reducerMap: ReducerMap<State, Payload>,
-        initialState?: State
+        initialState: State
     ): Reducer<State, Payload>;
 
     export function combineActions(
-        ...actionTypes: Array<{ toString(): string }>
-    ): { toString(): string };
+        ...actionTypes: Array<ActionFunctions<any>>
+    ): Array<ActionFunctions<any>>;
 }
diff --git a/redux-actions/redux-actions-tests.ts b/redux-actions/redux-actions-tests.ts
index 65ef3282ed..f5a2d41e18 100644
--- a/redux-actions/redux-actions-tests.ts
+++ b/redux-actions/redux-actions-tests.ts
@@ -34,13 +34,17 @@ state = actionHandlerWithReduceMap(0, multiplyAction(10));
 const actionsHandler = ReduxActions.handleActions<number, number>({
     'INCREMENT': (state: number, action: ReduxActions.Action<number>) => state + action.payload,
     'MULTIPLY': (state: number, action: ReduxActions.Action<number>) => state * action.payload
-});
+}, 0);
 
 state = actionsHandler(0, { type: 'INCREMENT' });
 
 const actionsHandlerWithInitialState = ReduxActions.handleActions<number, number>({
-    'INCREMENT': (state: number, action: ReduxActions.Action<number>) => state + action.payload,
-    'MULTIPLY': (state: number, action: ReduxActions.Action<number>) => state * action.payload
+    'INCREMENT': {
+        next: (state: number, action: ReduxActions.Action<number>) => state + action.payload,
+    },
+    'MULTIPLY': {
+        next: (state: number, action: ReduxActions.Action<number>) => state * action.payload
+    }
 }, 0);
 
 state = actionsHandlerWithInitialState(0, { type: 'INCREMENT' });
@@ -123,7 +127,11 @@ ReduxActions.handleAction(act3, (state, action) => {
     return { hello: action.payload.s }
 })
 
-/* can't do this until it lands in 2.2
+ReduxActions.handleAction(ReduxActions.combineActions(act, act3, act2), () => {
+
+})
+
+/* can't do this until it lands in 2.2, HKTs
 ReduxActions.handleAction(act, (state, action) => {
     action.payload === 'hello'
     return {}

From 126dc5b597174fdef031ba11e28d1dd6a8ca2352 Mon Sep 17 00:00:00 2001
From: Paulo Cesar <email@pocesar.e4ward.com>
Date: Sun, 25 Dec 2016 08:48:31 -0200
Subject: [PATCH 3/3] fix missing semicolon

---
 redux-actions/index.d.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/redux-actions/index.d.ts b/redux-actions/index.d.ts
index bd195c1c18..6776f2b621 100644
--- a/redux-actions/index.d.ts
+++ b/redux-actions/index.d.ts
@@ -40,7 +40,7 @@ declare namespace ReduxActions {
         throw?(state: State, action: ActionMeta<Payload, Meta>): State;
     }
 
-    type ActionFunctions<Payload> = ActionFunction0<Action<Payload>> | ActionFunction1<any, Action<Payload>> | ActionFunction2<any, any, Action<Payload>> | ActionFunction3<any, any, any, Action<Payload>> | ActionFunction4<any, any, any, any, Action<Payload>> | ActionFunctionAny<Action<Payload>>
+    type ActionFunctions<Payload> = ActionFunction0<Action<Payload>> | ActionFunction1<any, Action<Payload>> | ActionFunction2<any, any, Action<Payload>> | ActionFunction3<any, any, any, Action<Payload>> | ActionFunction4<any, any, any, any, Action<Payload>> | ActionFunctionAny<Action<Payload>>;
 
     type Reducer<State, Payload> = (state: State, action: Action<Payload>) => State;