The ignition needs to be at the red wire coming from the battery. between battery and where all the reds divide sort of speak. That way, iginition lock kills all. Like NotLob said, coils are live all the time now and you'll be able to kick it without key. Furthermore, coils will draw current, killing the battery over time.
rectifier needs ground
rest looks VERY minimal, but ok.