I don't expect this change would show any noticeable difference without a manual rolling module like Real Dice, but in such a case, this should prevent a player from being prompted for input for the same dice multiple times. Essentially we're just using Midi's addRollTo helper function to avoid re-evaluating the already evaluated parts of the roll. I've tested with multiple effects that add bonus damage and everything seems to work okay.
I don't expect this change would show any noticeable difference without a manual rolling module like Real Dice, but in such a case, this should prevent a player from being prompted for input for the same dice multiple times. Essentially we're just using Midi's addRollTo helper function to avoid re-evaluating the already evaluated parts of the roll. I've tested with multiple effects that add bonus damage and everything seems to work okay.