Как лучше делать логически проверки для триггеров? Через TriggerAddAction или же через if...then...endif прямо в коде уже функции action? Интересует именно в плане быстродействия

ифзенелз в функции кондишн, которую использовать вместо экшн.

Если нужно просто проверить условие - то условный оператор if. Condition конечно же медленее, и там есть счетчик на срабатывание триггера.

А например.. у меня триггерная абилка с отловом unit is attacked. Мне надо проверить наличие абилки у атакующего, и тогда активировать триггер. Куда это лучше встасить?

Перед экшеном, в общем условии. Т.к. нет других действий, без выполнения условия.

Если вы отлавливаете само нанесение урона - то факт атаки там вообще не причем, там надо динамически регистрировать событие.