Сессии по умолчанию хранятся в файлах с названием sess_{session_id} в директории, которую можно определить с помощью функции session_save_path .
Таким образом, теоретически можно циклически брать каждый файл и читать user_id внутри него.
Но это может быть ресурсоемко и поэтому легче было бы хранить сессии в базе, чтобы можно было найти нужного юзера обычным SELECT запросом по user_id

Необходимо добавить в условие передвижения ракет условие их уничтожения
Например, вот так

if (rocket.type === "left") {
  if (rocket.xCoord < parseInt(this.fieldEl.style.width)) {
    rocket.move();
  } else if (rocket.xCoord === parseInt(this.fieldEl.style.width)) {
    this.rockets.splice(index, 1);
    return;
  }
}

if (rocket.type === "right") {
  if (rocket.xCoord > 0) {
    rocket.move();
  } else if (rocket.xCoord === 0) {
    this.rockets.splice(index, 1);
    return;
  }
}

В ajax ты указал, что ожидаешь в ответ json с сервера (как и на сервере указал заголовки json), при этом на сервере выдаешь строку + json, из-за чего получается неверный формат данных. Убери echo success и addcomment (можешь перенести в тот же json) и будет тебе счастье
В идеале в таких случаях нужно всегда непосредственно перед выводом ответа сервера очищать буфер