Handler State Become Stale for Actions That Produce Long Running Flows

[pedroql]

Handler State Become Stale for Actions That Produce Long Running Flows

[pedroql]

This is covered in the documentation:

Note: although the handler receives the current state when an action is performed, if the handler performs a slow operation, the handler will no longer know the most recent state. This is why the handler can’t mutate the state directly.

From https://pedroql.github.io/mvflow/#reducer

The reason we can't have the handler receiving the current state is because there's no way to guarantee that this is and remains the most up to date state while the handler does something with it because there could be multiple coroutines accessing and mutating the state.

Instead, MVFlow ensures that calls to the reducer don't interleave so it's the only safe place to update the state.

We do have unit tests for the scenarios you highlight here.

--

You need to update your code sample and use the mutations in a different way. In your mutations, you pass the desired final value. Instead, you should pass what you want to do to the state. So it goes from emit(MyMutation.IncrementCount(state.count + 1)) to emit(MyMutation.IncrementCount(1)). And your reducer (with a renamed property) becomes state.copy(count = state.count+ mutation.changeValue).

This is what we mean when we say

Keep also in mind that Mutations should indicate how to change the state, but should not rely on/assume what the current state is (as of when the action was emitted).

MVFlow.kt#L28

--

Let me know if you have ideas to improve the documentation so other people don't miss this important detail!