Тем что короче

Но реализация конечно там веселая

Они не равноценные

Object.assign — это функция, которая модифицирует и возвращает целевой объект?

Вполне можно. Не используйте геттеры.

Да

Я советую гуглить по именам правил для линтера. Они все оформлены как "пакеты", у всех в документации рационализация, т.е. причины, по которым такое правило вообще существует. У префер-обжект-спреда такое:

"Introduced in ES2018, object spread is a declarative alternative which may perform better than the more dynamic, imperative Object.assign."

"Введённый в ЕС2018, обжект спред является декларативной альтернативой, которая может отрабатывать быстрее, чем более динамический, императивный Обжект.ассигн"

Т.е. код становится короче и чище, что хорошо, становится более декларативным (т.е. говорит, что сделать, а не как сделать; что хорошо) и делает работу с данными более иммутабельной (т.е. с созданием новых копий данных вместо изменения существующих; что хорошо, т.к. мутации данных приводят к целому классу сложноотлаживаемых багов)

(Хотя, конечно, байтодрочеров от такого кощунства может и удар хватить)

И отхватввает

У нас запрещён

И асигн тоже правда :))

Есть client.onmessage внутри которого постоянно прилетает data с сервера, по вебсокету
Прилетает обычно что-то по типу { type: string }
В данный момент, у меня структура такая: онмеседж это функция, которая на каждое сообщение вызывается и как параметр принимает все те сообщения от сервера, и внутри неё много if'ов, которые выполняют разные действия зависимо от typ'а в прилетевших данных

Как можно убрать все эти if'ы и вынести каждый из них в отдельный модуль (вообще в отдельный файл, чтобы вместо всех тех условий вся логика ифов была разбито по разным файлам)
Надеюсь понятно описал...

Интересно какие варианты решения сможете предложить

У меня пока только один: написать один класс, который будет принимать как параметр каждое сообщение от сервера и передавать всем своим наследникам, а наследники уже будут обрабатывать.. просто раскинуть их по разным файлам и всё
Но это я на ходу придумал, возможно это не лучшее решение и есть какие-то паттерны, которые вы практикуете в подобных ситуациях, поделитесь пожалуйста своим мнением по этому поводу

Сделать типа патерн матчинга на обектах

Будет дикт, в который регистрируются обработчики

И потом по:

handlers[message.type].process(message)

Обрабатывают

Ну с проверкой ошибок конечно типа:

onMessage(msg) {
    if(!(msg.type in handlers))
          throw 'Unhandled type:' + msg.type;

    handlers[msg.type].process(msg);

}

Ну или как миделвары

Перебирая все.