Don't filter like this

bad.js

const produce = ['Apple', 'Potato', 'Banana', 'Cucumber']

// Using a temporary value to build up our result.
const fruits = []

// Bad: Using filter that has a side-effect (non-pure function).
produce.filter(p => {
  if (['Apple', 'Banana'].includes(p)) {
    fruits.push(p)
  }
})

// Voila
console.log(fruits)

better.js

const produce = ['Apple', 'Potato', 'Banana', 'Cucumber']

// Better: filter the collection, return matches to build up fruits.
const fruits = produce.filter(p => {
  if (['Apple', 'Banana'].includes(p)) {
    return p
  }
})

// Voila
console.log(fruits)

AAAAAAAAAAAAHHHHHHHHHHHHHHHHHGGGGGGGGGGGGGG.

Yup, that's a code smell. Flag it.