Алексей Уколов

Ваш код очень бажный. Во-первых, вы не проверяете результат выполнения indexOf и проверка работает ровно наоборот. Во-вторых, значение поискового поля вы добавляете не в виде get-параметра, а прямо в тело адреса, что снова делает предыдущую проверку нерабочей. Ну и наконец, даже если всё это исправить, у вас всё равно будет баг - любой get-параметр, название которого заканчивается на "q", будет попадать под условие и подстановка не произойдёт.

Максимально тупое решение в лоб с перезагрузкой страницы:

if (
    window.location.search.indexOf('?q=') === -1 &&
    window.location.search.indexOf('&q=') === -1
) {
    window.location.search += (window.location.search ? '&' : '?' ) + 'q=' + searchInput.value;
}

В современном же js есть куда более удобные инструменты для решения этой задачи:
https://developer.mozilla.org/en-US/docs/Web/API/U...
https://developer.mozilla.org/ru/docs/Web/API/Hist...

Потому что defineProperty ожидает в третьем аргументе объект с ключом value. Что тут ещё можно добавить? Можно с тем же успехом спросить почему код не работает, если его написать на бумажке.

Потому что в JS ни один массив не равен другому массиву, даже если у них одинаковое содержимое. А indexOf ещё и строгое сравнение использует.
Перепишите логику нахождения номера строки на простой цикл или преобразуйте свой массив:

let row = names.map((row) => row[0]).indexOf(valToFind);

В js нет никаких "ассоциативных массивов", есть только объекты.

const total = {};

...
total[name] = age;
...

console.log(total.Алексей);

я хочу в итоге отсортировать этот массив по его значению (второму элементу)

Превратить объект в массив можно с помощью Object.entries(), а дальше уже сортируйте как хотите.

Вместо onclick следует использовать addEventListener.

Но в вашем случае, однако, можно обойтись и onclick - вся ваша логика должна быть в одном обработчике, просто запускаться по очереди то одна, то другая её ветка.

Нужно просто запретить событию подниматься выше:

$('li').on('click', function(event){
   var data = $(this).data('key');

   event.stopPropagation();
});

почему не пишут легкий в реализации SPA без всяких фреймворках

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

Ведь в кеш записывается весь result равный телу функции

С чего вы это взяли вообще? Что функция вернула, то и записывается в кеш. Вы же не саму её записали, а результат её вызова. Alert к этому результату не имеет никакого отношения и всегда возвращает undefined.

const moves = [];

... 
moves.push(JSON.stringify(jsonmove));
...

const datamove = "[\n" + moves.join(",\n") + "\n]";

Это если исходить из того, что вам нужен именно такой формат, когда каждый элемент массива на отдельной строке.
Иначе же можно упростить до

const moves = [];

... 
moves.push(jsonmove);
...

const datamove = JSON.stringify(moves);

На оба вопроса ответ один - первым в replacer передаётся весь объект целиком. Поэтому он и выводится, а если убрать return, то дальше обработка и не идёт.
Я же вам отвечал уже на этот вопрос: https://qna.habr.com/answer?answer_id=2165468#answ...

Например, вы хотите использовать дефолтное значение или значение из кэша, если не удалось получить данные с сервера.

Зачем итерировать по кривому формату данных, если можно просто сразу его превратить в нормальный?

setItems(json[0].pizzas)
...
items.map((pizza) => <PizzaBlock key={pizza.title} {...pizza} />)

Второй файл перезаписывает свойство. Используйте вместо свойства подписку на событие DOMContentLoaded.

В документации есть пример обработки закрытия: https://getbootstrap.com/docs/5.2/components/alert...
Но поскольку вам нужно, чтобы сообщение показывалось только когда в LocalStorage нет флага, то и создание алерта вам нужно реализовать через код: https://getbootstrap.com/docs/5.2/components/alert...

Где вы с клиента body-то передаёте? Вот то-то и оно.

Метод hasOwnProperty() возвращает логическое значение, указывающее, содержит ли объект указанное свойство.

Object.prototype.hasOwnProperty()

try {
  await fetch(...);
} catch (error) {
  // Обрабатываем ошибку как хотим
}

fetch(...)
  .catch((error) => /* Обрабатываем ошибку как хотим */);

const xhr = new XMLHttpRequest();
xhr.open(...);
xhr.send();

xhr.onload = function() {
  if (xhr.status != 200) {
    // Обрабатываем ошибку как хотим

    return;
  }

  ...
};

Я не понимаю, что тут может остаться непонятным с предыдущего вашего вопроса.
Цель - избаваться от циклических ссылок, которые делают сериализацию объекта невозможной. При этом ориентироваться требуется не на название поля, а на его значение.
Именно это и происходит в функции replacer, с поправкой на то, что при первом вызове в replacer передаётся весь объект целиком с пустым ключом:

Initially, the replacer function is called with an empty string as key representing the object being stringified.

https://developer.mozilla.org/en-US/docs/Web/JavaS...

key != "" && value == meetup

Переводится как "если ключ не пуст (т.е. не первая итерация, в которой value ссылается на объект целиком) и значение равно искомому объекту, его нужно пропустить и не сериализовывать".

В нормальном мире "кнопка" не должна никуда "вести". Кнопка запускает какой-либо процесс, за переход на другой URL же отвечает ссылка.
Если вы сделаете нормальную семантическую разметку: попадаем под "определённые условия", то рендерим div или выключенную кнопку (тут нужно смотреть по контексту), а не попадаем - обычную ссылку, то всё будет работать без всяких плясок с бубном.