Вот тут почему то отличается именование класса с префиксом
.search-room__filter-item должно быть .js-search-room__filter-item, а у тебя .js-search-room__dropdown-date
Проверь пожалуйста, чтобы по всему проекту класс с префиксом соответствовал классу без префикса

А почему класс CardForm лежит в папке с глобальными скриптами?

Нужно стараться делать данные не изменяемыми.
К примеру здесь происходит мутирование pug объекта attributes, чтобы этого избежать можно ведь создать новый объект, к примеру

const params = {
      type: 'button',
      title: 'likeCount',
      'data-active': active,
      class: [blockName, `js-${blockName}`, active && `${blockName}_active`],
}

И затем уже передавать этот объект button&attributes(params).

const btnTag = {
      'link': 'a';
      'button': 'button'
};
#{btnTag[buttonType]}&attributes(attributes) #{btnText}

Но это если много вариаций будет, в данном случае так как вариации всего две, вполне можно обойтись тернарным оператором const btnTag = btnType === 'link' ? 'a' : 'button';

Все картинки, шрифты и т. д. должны иметь уникальное название в рамках проекта и не содержать кириллических символов. Если в проекте нет индивидуальных соглашений, то они должны именоваться в стиле lower-case-hyphenated.

Сейчас плагины где-то инициализируются где-то внутри блока, где-то на страницах сайта. Нужно привести инициализацию к одному виду и разделить реализацию и инициализацию. Можно к примеру создать в блоках создать init.js и там производить инициализацию

По нашим стандартам нельзя использовать отрицательные margin (best practices)

При открытии "Дополнительных параметров" почему-то на долю секунды появляется полоса прокрутки.

Использовать сокращения в именовании не очень хорошо, т.к. это ухудшает читаемость и поиск по проекту

Вот тут из контекста конечно можно понять, что означает это число в скобочках, но хорошей практикой считается держать такие вещи в константах

Файлы с классами нужно именовать исходя из названия классов, то есть если у тебя класс называется ButtonLike, то и файл должен быть ButtonLike.js

Нужно добавить футер на основные страницы сайта для экранов 768px и меньше, который есть в макетах.

Можно не передавать пустой объект в миксин, если указано значение по умолчанию

Тут неверное именование обработчика

Обработчики событий для БЭМ-элементов, а также для компонент, вложенных в БЭМ-элементы, именуются по шаблону handleElementNameEventName, где ElementName - имя БЭМ-элемента.

Тут обработчик на mousedown, а в именовании click

Тут нужны модификаторы ключ-значение, я так понимаю, что для инпутов small сделал булевым, потому что другие размеры не используются, но мне кажется, что это все равно по смыслу немного не то, т.к. булевый модификатор чаще всего отражает состояние, например, disabled и тут понятно, что кнопка может быть либо задизейблина, либо активна, третьего вариант нет. А с размером ты можешь и large добавить и medium и еще что-нибудь, т.е. важно значение модификатора, а не просто его наличие

С кнопками может быть вообще стоит дата атрибуты использовать вместо модификаторов, если они для стилизации не используются, а только поведение отражают


)