[voraa]

Тогда вот это совсем непонятно

45 else if (arr[objLocal]?.id!==data[openItemIndex]?.id){
markMovieAsFavorite(objLocal)
console.log("dob")
}
Как я понимаю, надо узнать, содержится ли в arr (вернее узнать, что не содержится) объект с id: data[openItemIndex]?.id. Так?
Тогда
if (! arr.find(el => el.id == data[openItemIndex].id) ) {

И это непонятно.
markMovieAsFavorite(undefined)
Зачем туда запихивать лишний элемент undefined? И потом везде таскать ?.

Ну напишите вы чуть больше кода

const markMovieAsFavorite = movieId => {
const a = [...favotireMoviesIds]
if (movieId) a.push(movieId)
localStorage.setItem('favorite-movies', JSON.stringify(a));
}

А какой смысл хранить в localStorage все структуры вместе с текстовыми описаниями? Одних индексов не достаточно? По ним же всегда можно найти нужный объект.