Maybe I am missing something, but why the second chip? The 18 has plenty of pins.
With 12 volt on the LEDs you may want to think about some sort of isolation just in case something goes wrong the PIC will not be over amped or over volted? (mosfet, opto, transistor)
Other than that it looks like it should work.