Well, the repeat loop isn't going to repeat, it's just going to go to the Return at the end of the sub.
A Repeat is used to do something until a criteria is met then move on (or simply leave the loop and never go back via a goto or gosub that doesn't return, both of which would be poor coding practice in most cases). So, where would you need to repeat the series of tasks until? THAT is where you close your loop and give it a criteria - e.g. "Until %numbercrafted = 10" or "until %domynutsitch = #FALSE", etc.