I think we should have a seperate shared file for this, I dont imagine people will be using both operator_variant_helper and this at the same time (and if they do they can include both), its not a huge amount of code but if we dont use operator_variant_helper I would prefer not to pull it in