вместо useEffect вам нужен useMemo
const sortedCompanyList = useMemo(
() => sort(companyList, activeIcon),
[companyList, activeIcon]
)
Желательно что функция sort(companyList, activeIcon) возвращала новый массив а не мутировала companyList.
Да, и почитайте про мемоизацию и хук useMemo