Looks like combinators dont work as intended in redux-saga

[krutoo]

Hi, i noticed that next code:

import { all, takeLatest } from "typed-redux-saga";

function* mySaga() {
  yield* all([
    // parallel calls
    takeLatest("HI", sayHi),
    takeLatest("BYE", sayBye),
  ]);

  console.log('after')
}

Dont works - it not prints "after" when saga run

I also noticed that this is works fine when macro version used.

I Described thoughts in readme of repo with attempt to fix that behavior:
https://github.com/krutoo/typed-redux-saga

You can also looks on tests folder in repo and check that with typed-redux-saga it is not works but with redux-saga and with fixed version it works.

Can we add a fix or something like that to typed-redux-saga?

[krutoo]

After some research into the problem I found out a few details about how redux-saga works

Here's what @Vijah told me on discord:

takeLatest, takeLeading, takeEvery are not intended to be used inside all/race.

The interesting part about forks is that they are non-blocking, BUT the parent task will never finish as a result of waiting after the fork.

As a result, my guess is that takeLatest returns a fork (a task) which does not block the all, but will cause the parent generator (takeLatestWrapper) to never finish as a result of the task running indefinitely.

Also in discord @neurosnap tolds me that there are situations when it makes sense to use all() and takeLatest together:

There are cases where you want to use all() with take* helpers: for mounting a root task that spawns subtasks and never completes: https://github.com/neurosnap/redux-saga-creator/blob/master/index.ts