Are reaction callbacks automatically run in (trans)actions?

[Dorosty]

Apologies in advance if this has been answered/documented before. I couldn't find an answer in the docs.

Are reaction callbacks automatically run in (trans)actions? What I mean is, is this necessary?

autorun(action(() => { /****/ }))
reaction(getDataFromState, action(data => { /****/ }))
when(predicate, action(() => { /****/ }))

Or does this suffice?

autorun(() => { /****/ })
reaction(getDataFromState, data => { /****/ })
when(predicate, () => { /****/ })

[urugator]

Are reaction callbacks automatically run in (trans)actions?

yes

What I mean is, is this necessary?

no

Or does this suffice?

yes

However you should avoid mutating state in reactions, therefore it shouldn't matter most of the time.

[Dorosty]

Thanks! Is the same thing true for generator functions? If I pass one as a callback will it automatically get wrapped with flow?

[urugator]

No, interesting idea though.

[Dorosty]

Last question. Are callbacks passed to onBecomeObserved and onBecomeUnobserved also run as actions?

[urugator]

No, observe/intercept callbacks neither.

[urugator]

Btw you can inspect _getGlobalState().inBatch, non-zero value means you're in batch (usually in action, but not necessary).