Use immer within redux-actions

[mhubig]

Hi together,

I found combining redux-actions with immer to be a very powerful combination to avoid redux boilerplate code. See the example. But I think it would be even more elegant if handleActions would natively use immer to produce the new state.

So this

import produce from 'immer';

const reducer = handleActions(
	{
		[combineActions(
			actionCreators.shoppinglist.fetchDefaultShoppinglist,
			actionCreators.shoppinglist.modifyShoppinglist
		)]: produce((state, { payload }) => {
			state.loading = payload;
		}),
	},
	initialState
);

could be written as:

const reducer = handleActions(
	{
		[combineActions(
			actionCreators.shoppinglist.fetchDefaultShoppinglist,
			actionCreators.shoppinglist.modifyShoppinglist
		)]: ((state, { payload }) => state.loading = payload,
        },
	initialState
);

[vinnymac]

This would be a performance hit and not an insignificant API change to make for this lib in my opinion. Right now it does the same job whether or not you use immer, which means it is more flexible, and remains as fast as the user needs it to be.

If you do want to avoid the boilerplate while still using immer, you can just simply wrap handleActions yourself like so.

import produce from 'immer'
import { handleActions as raHandleActions } from 'redux-actions'

export const handleActions = (actions, state) => raHandleActions(
  Object.keys(actions).reduce((acc, key) => {
    acc[key] = produce(actions[key])

    return acc
  }, {}),
  state
)

Then whenever you need to use it you would just do

import { handleActions } from 'utils/redux-actions'

const reducer = handleActions(
	{
		[combineActions(
			actionCreators.shoppinglist.fetchDefaultShoppinglist,
			actionCreators.shoppinglist.modifyShoppinglist
		)]: ((state, { payload }) => state.loading = payload,
        },
	initialState
);

[raythree]

I also wrap handleActions as above when I want to use immer. It would be really nice however if you could optionally pass "produce" to handleActions as an option, and have it optionally wrap the returned reducer with it. I'm pretty sure that redux-starter-kit always wraps "createReducer" with immer, and includes it as a dependency. Supporting it in redux-actions like this would be really convenient and not require immer as a dependency or peer dependency:

import { handleActions } from 'redux-actions';
import produce from 'immer';

const mutableReducer = handleActions(reducerMap, defaultState, {produce});

[h0jeZvgoxFepBQ2C]

that would be pretty nice @raythree

You don't want to submit a PR, don't you? :D
I would love to use this feature too, but not skilled enough to make this PR..