Чем плохо массовое использование async функций?
Помогите привести доводы почему это надо отрефакторить и убрать ненужные async await.
То есть у вас самого доводов никаких нет? А зачем тогда вы хотите это переделать?

Для начала стоит спросить ради чего так сделано. 
Если это реально кто-то наговнокодил и налепил async не понимая что это и зачем, то довода "это лишний код" должно быть достаточно. 
Ну можете еще сказать что это заворачивает все в лишние промисы, делает выполнение дольше на какие-то микросекунды и усложняет отладку и может портить стектрейсы.
Вы вот когда рашили, что асинх там не нужен, чем руководствовались? Подумайте об этом, вспомните и будут вам доводы, которые вы ищете.
Конечно, можно просто ждать, когда вопрос увидит телепат, который по аватарке узрит, что там за код и почему он не нужен.