вообще все не бесплатно, вопрос в балансе

не смешите мои пейсы

Это ещё и побочный эффект. Обрезание очень дорого. Особенно с большими массивами

Мутация же будет, не ?

у основного массива

Да

не помню, чтобы речь шла о действительно больших массивах.
К тому же, ] и не говорил, что это хороший способ, просто сказал, что он существует)

в коде выше он скопирован спредом

еще можно выбросить ошибку

и обернуть редюс в трайкетч )))

оборвал. Решение

И создать для этого обёртку, что бы игнорировать только допустимое исключение

Я бы сказал, что выход из редьюса по некоему абстрактному условию - это вообще идеологически неверно, потому что нельзя выйти из свертки, не закончив её.

Согласен

Подскажите плиз правильно ли я понял этот код:

var w = 1, z = 2;

function foo(
 x = w + 1,
 y = x + 2,
 z = z + 3
) {
 console.log(x, y, z);
};

foo();

У аргументов в js есть своя область видимости, типа:

{ let x = w + 1, y = x + 2, z = z + 3 }

Начнется инициализация x, она начнет искать w в области видимости аргументов, не найдет там w, поднимется вверх и найдет w = 1, y инициализируется, так как x уже есть.

А вот z попытается найти z в области видимости аргументов, найдет z (поэтому поднятия вверх не будет), но так как z не инициализирована, а мы пытаемся ее использовать, то упадет ошибка.

Так?

нет, тут дело в том, что именованные аргументы функций - не то же самое, что переменные. Они не всплывают, и z не будет искаться снаружи функции

тут бы стоило разграничить определение функции и ее вызов, а то смешались в кучу кони, люди

const itemCollect =(data)=>{
           data.map((item)=>{
               item['items'] =[];
               makeItemCollect(item,data)
           })
       };
       const makeItemCollect =(dataItem,data)=>{
           data.map((item)=>{
               if (dataItem.id===item.parent_id){
                   dataItem.item=[...item];
                   makeItemCollect(item,data)
               }
           });
       }

Как ты смирительную рубашку снял?