From the one bg (so far) that I've used it in it's amazing. It also gets 2k more dps on a dummy than my current rogue pvp profile (I'm a 70 rogue twink)
One thing I would love is a sap, maybe something like this
Code:
-- Check for target
if UnitCanAttack("player","target") == nil then
TargetNearestEnemyPlayer()
end
-- Sap target
if UnitCanAttack("player","target")
and (UnitBuffID("target", 1784) or UnitBuffID("target", 5215))
and UnitDebuffID("target",6770) == nil
and UnitDebuffID("target",89775) == nil
and UnitDebuffID("target",18499) == nil
then
CastSpellByID(6770, "target")
end
Also, I recommend you refresh weakened armor before it finishes rather than waste 2 extra GCDs casting Expose Armor to get back to 3 stacks.
Finally, and this is a personal preference, have the user initiate combat. I don't want to target people to check them out then they run up to me and my stealth is lost. If you don't want to change this I can do it for me but I think that maybe other people would like it. The only reason I don't check the option to require combat before it runs is because I like the auto stealth and poison features.
Other than that everything works great!