IlyaMalyukov

Всё просто.
cart.push(products[id]); - вы добавляете товар не по id, а по индексу.

То есть products[5] в вашем случае равно id = 5 лишь случайно. заполните массив иначе и всё, уже не равен. И тем более, когда в cart не соответствует индексам вы не можете так же удалить cart.splice(cart[id], 1);
В функцию передаёте id, а удаляете по индексу.
При добавлении и удалении сначала нужно найти в массиве через findIndex(id == id_полученный_в_функции)) и уж делайте splice(результат findIndex)

короче, поменять:
cart.push(products.find((item) => { return item.id==id}));
и
cart.splice(cart.findIndex((item) =>{ return item.id == id }), 1);

У вас вызывается функция при нажатии на кнопку:

<button id="toCart" onclick = "addToCart()">Добавить в корзину</button>

Однако объявление этой функции находится внутри цикла и не попадает в область глобальной видимости.

// Добавление товаров в корзину 
  for(let j; j < products.length; i++) {
    function addToCart(j) {
      cart.push(products[j]);
      console.log(cart)
    };
  }

Поэтому при нажатии на кнопку выбрасывается ошибка "нет такой функции".

Скорее всего тут просто перепутаны местами строки. Должно быть:

// Добавление товаров в корзину 
function addToCart(j) {
    for(let j; j < products.length; i++) {
      cart.push(products[j]);
      console.log(cart)
    };
  }

не установлена связь между моделями User и Task: rusrails.ru/active-record-associations#svyaz-belongs_to

words = %w[apple banana]
words.find_index('is').class #=> NilClass

Скорее всего нет строки, которая содержит is, поэтому index равен nil, и попытка прибавить к нему единицу вызывает ошибку