ok, for your purposes everything looks good then. Except for one line.

Code:
/castrandom [combat] Devouring Plague, Shadow Word: Death
when I put [mod] that is exactly what should have been in. [mod] means if you press a modifier that line will continue, if you do not it will bypass for the next.
However, one thing I did not think of before, that I am unsure if it will work, I will test it later.
If the /castrandom list is full of spells on cool down, is that considered as a negative conditional and thus pass on to the next line?
Like I said I will test it when I get home in about an hour and a half. But if it does that could resolve your macro and one I was working on.