colors.forEach(color => {
      if (color.checked === true) {
        selectedColors.push(color.value);
      }
    }

この文の if (color.checked ・・・のcolorが何を意味しているのか分かりません。
checkedがチェックされている事を判断する関数であるのなら、if(colors.checked・・・
になりそうなのですが…。