Remind me why we're not just doing a BotFactory class in a separate file? Seems cleaner and more SRP-ish to me. Then this function and the botType function would both live there...