Not sure if I understood correctly, but looks like current signature behaves correct? Takewhile doesn't change what source emits but only limits when to stop emit. Would you mind explain bit as I may miss something?

Not sure if I understood correctly, but looks like current signature behaves correct? Takewhile doesn't change what source emits but only limits when to stop emit. Would you mind explain bit as I may miss something?

If the inclusive flag is set to false (which it's its default value) then if the takeWhile operator is used with the Boolean constructor the resulting observable won't emit falsy values. For instance:

of('foo', 'bar', null).pipe(
  takeWhile(Boolean)
)

will only emit strings, it would never emit null.

That's why I think that it makes sense to change the typings of the Boolean constructor (when the inclusive flag is set to false), so that the typings of the resulting Observable exclude the falsy values, like filter does.

Thank you, @josepot!