Rename cellRefCache to cellRefHolder because the use of the word cache indicate that it is for speeding-up some kind of operation, but in reality it is necessary due to the nature of the structure of the ExpressionHandler and math JS.
I change move ExpressionHandler initialization to resolveCell instead of sheet constructor, the reason is that the field cellRefHolder is operation scoped meaning that it needed to be cleared in evaluate which feels semantically incorrect. Another reason is that ExpressionHandler didn't even feel like part of a sheet because it is not holding any information and just a class that encapsulate certain functions.
Some important things to note:
cellRefCachetocellRefHolderbecause the use of the wordcacheindicate that it is for speeding-up some kind of operation, but in reality it is necessary due to the nature of the structure of theExpressionHandlerand math JS.ExpressionHandlerinitialization toresolveCellinstead ofsheetconstructor, the reason is that the fieldcellRefHolderis operation scoped meaning that it needed to be cleared inevaluatewhich feels semantically incorrect. Another reason is thatExpressionHandlerdidn't even feel like part of asheetbecause it is not holding any information and just a class that encapsulate certain functions.