would it make sense to make this a global (i.e. something like, `VALID_SEGMENT_ID = "^[a-zA-Z0-9]+\-*[a-zA-Z0-9]*$"`) so if people need to monkey-patch it in the future it's slightly easier?