Вместо return; сделайте return "";
ESlint исходит из того, что Ваша функция должна возвращать строку, вместо этого Вы возвращаете undefined. Очень желательно придерживаться того, чтобы функция всегда возвращала один и тот же тип данных, либо выбрасывала исключение при ошибке, тогда, когда Вам захочется вернуть null/undefined.
почему бы не заменить условие, которое повторяется много раз?