What is the matter with the petattack portion of this macro? Only attack my target if it does not already have a target
#showtooltip
/cast Shadow Bolt
/petattack [target=pettarget, noexists]
You’re telling your pet to attack its own target, not your own target. Well, you’re telling your pet to attack its own target, but only if that target doesn’t exist.
You should be able to use the /stopmacro command to do what you’re looking for. Maybe something like the following:
#showtooltip
/cast Shadow Bolt
/stopmacro [@pettarget, exists]
/petattack