I hate to be *this* nitpicky, but can we find a better name other than `variant_modifiers_hash`? Even something like `modifiers` would make sense. It's implicit that it's either a modification for the variant or the line item, and the code should make that clear. We don't need to include the class of the object in the variable name because this is Ruby.