Упростить js скрипт?
Отделите данные от скрипта, будет компактнее. Да и много повторений
const tags = ['screen', 'goods', 'gallery', 'reviews', 'contacts'];
function enableBlock(blockTag) {
    document.querySelector('.blocks__phone-bg').classList.add(`blocks__phone-bg--${blockTag}`);
    document.querySelector('.blocks__phone-bg').className = `blocks__phone-bg blocks__phone-bg--${blockTag}`;
    tags.forEach(tag=>{
      let elem = document.querySelector(`.blocks__item[data-item="${tag}"]`);
      if (!elem){
         return;
      };
      if (blockTag===tag){
        elem.classList.add('blocks__item--active');
      }else{
        elem.className = 'blocks__item';
      };
    });
}
// Пусть, список будет в строке разделенной вертикальной чертой
"contacts|goods|gallery|reviews|screen".split`|`
// Находим каждый элемент
.map(data=>document.querySelector(`.blocks__item[data-item="${data}"]`))
// Вешаем на элементы событие mouseenter
.forEach(el=>el.addEventListener('mouseenter',e=>{
  // Класс активного элемента надо удалить только у активного элемента
  document.querySelector`.blocks__item.blocks__item--active`.classList.remove`blocks__item--active`;
  // А элементу, вызвавшему это событие надо добавить класс активности
  e.target.classList.add`blocks__item--active`;
  // Еще где-то какому-то элементу присвоим класс, частично формирующийся из атрибута элемента события 
  document.querySelector`.blocks__phone-bg`.className = `blocks__phone-bg blocks__phone-bg--${e.target.getAttribute`data-item`}`;
});
// Вот и все!