В коде функция мутирует массив и возвращает его длину вместо самого массива — как это исправить правильно?

Если возвращается результат push, это длина массива. Для неизменяемого добавления верните новый массив через spread.

Например, вместо return items.push(item) используйте return [...items, item]. Если мутация разрешена контрактом, можно выполнить items.push(item); return items;. Выбор зависит от требований: состояние React/Redux обычно обновляют неизменяемо, а локальный внутренний массив иногда допустимо менять.