We can simplify quite a bit of logic by reusing some property names between atom instances and selectors. Change the evaluationFinished mod to simply use node instead of different instance and cache keys.
Change prevEvaluationReasons and _prevEvaluationResons to simply prevReasons.
Change nextEvaluationReasons and _nextEvaluationReasons to simply nextReasons.
This includes making those fields non-underscore-prefixed in the AtomInstance class and therefore adding documentation for them.
This is a breaking change - the evaluationFinished mod event's payload shape has changed and some already-public fields in the SelectorCache class are renamed:
Description
We can simplify quite a bit of logic by reusing some property names between atom instances and selectors. Change the
evaluationFinishedmod to simply usenodeinstead of differentinstanceandcachekeys.Change
prevEvaluationReasonsand_prevEvaluationResonsto simplyprevReasons.Change
nextEvaluationReasonsand_nextEvaluationReasonsto simplynextReasons.This includes making those fields non-underscore-prefixed in the
AtomInstanceclass and therefore adding documentation for them.This is a breaking change - the
evaluationFinishedmod event's payload shape has changed and some already-public fields in theSelectorCacheclass are renamed:evaluationFinishedAction.payload.cache->evaluationFinishedAction.payload.nodeevaluationFinishedAction.payload.instance->evaluationFinishedAction.payload.nodeselectorCache.prevEvaluationReasons->selectorCache.prevReasonsselectorCache.nextEvaluationReasons->selectorCache.nextReasons