Afterware function runs twice

[suneg]

I've use this handy little utility on a project, but I've come across an issue where the afterware functions are called twice.

Repro steps

1. Add the following code in the App function:

  const [test, setTest] = React.useState('test')

  React.useEffect(() => {
    console.log('todos changed')
    setTest(new Date())
  },[todos])

2. Run the app (npm start) and bring up the dev tools console
3. Click one of the check-boxes

Expected result:

logger before:
logger after:
todos changed

Actual result:

logger before:
logger after:
todos changed
logger after:

I'm unsure about the reason for it, but I've boiled it down to setting component state to an updated value, from an effect that relies on the state from the reducer. Clearly it does not make logical sense that the afterware would run more than once, or any other count than the middleware for that matter.

[rwieruch]

Can you check whether this change helps? I noticed this in my application as well, but forgot to update this repo here.

39dd042

[suneg]

That seems to do the trick. I was experimenting with

}, [afterwareFns.length, state]);

instead of

}, [afterwareFns, state]);

Which also seems to work. But that might have some undesired side-effects.