push*(repeater) type API about repeater HOT 3 OPEN

Hmmm. So first things first: your use-case is probably best served by an async generator and not a repeater:

async function *gen() {
  await checkSomethingAsynchronously();
  yield *someOtherRepeater;
} 

Think of it like the Promise constructor: if you have a promise already then using the Promise constructor is probably a mistake. Same goes for repeaters: if you have an async iterator, you should probably be using an async generator and not a repeater. Choosing between the two can be tricky and I’ve thought about making a flow-chart for the different use-cases. It really depends on the sources that you’re pulling from. If they’re callback-based, use repeaters obviously. If they’re promises or async iterators, then it depends on how complicated the pulling algorithm is. If you’re simply pulling in series, then async generators and yield/yield* work. If you’re pulling from them concurrently, then repeaters can help.

didn't we come to the conclusion awhile back that using for..await like this is dangerous

The problem was with using async generators and for...await. Calling return on an async generator object paused at the top of a for...await loop wouldn’t work until the loop resumed, potentially causing deadlocks. The same would be true with a Repeater, but you could mitigate this issue by racing iteration with the stop promise.

from repeater.

Oh yeah, I didn't think of just using an async generator.

Calling return on an async generator object paused at the top of a for...await loop wouldn’t work until the loop resumed, potentially causing deadlocks.

I wonder if this is the case for yield * also though?

from repeater.

I think I checked and found that yield * propagates return into the inner iterator immediately.

from repeater.