Name Surname

ну ты передаешь handleChange параметром в функцию как props, а потом сразу его перезаписываешь своей функцией, линтеру это не нравится, да и зачем тогда пропсом это передавать, если потом не использовать

Значение по-умолчанию для первого аргумента.
Демо.

for (let i = 0; i < password.length; i + 1) {

если запустить код то страница крашнется, codesandbox на счет этого говорит что циклов слишком много

Ну да, i + 1 - счётчик не меняет своего значения, цикл получается бесконечным.

Должно быть или i += 1 или i++.

return pass;

Наверное, всё-таки должно быть return text.

buts[i].addEventListener('click', Addclass(i));
Так вы вызываете функцию Addclass, и ее возвращаемое значение передаете в addEventListener, а должны передавать функцию, которую нужно вызвать при клике.
Вообще, вам не нужен togs, просто у текущего кликнутого элемента берите children, и добавляйте нужный класс.

Опять текст ошибки нужно угадать? Ну здесь моей телепатии хватает, даже запускать не придется.
Скрипт подключен в заголовке, он ищет элементы, которых еще нет на странице.
Не находит их.
Капец.

У тебя нода древняя как говно мамонта(4 версии, сейчас уже 10 ая), поставь последнюю:

curl -sL https://deb.nodesource.com/setup_10.x | sudo -E bash -
sudo apt-get install -y nodejs

чудо что в ней вообще что-то запустилось

.menu-list li {
  flex-grow: 1;
}

https://jsfiddle.net/34gjbk4s/6/

https://jsfiddle.net/p9vxv5u9/