Олег Черр

Рекомендую именовать классы так, как удобно лично вам (или вашей команде).
Лично мне очень не нравятся записи с двойным тире или подчёркиванием, и уж тем более модификаторы "ключ-значение". Смотрится очень громоздко, особенно когда у одного элемента несколько подобных классов.

.menu__item--active {}
.menu__item_state_active {}

Поэтому настоятельно рекомендую подстраивать бэм под себя. Подумайте над задачами, с которыми вы сталкиваетесь сейчас и столкнётесь в будущем, и решите их самым лаконичным способом. Вот, например, моя модификация бэм. Использую её уже несколько лет. Разработка стала очень приятной.

Что касается именования классов, то я бы сделал так:

<ul class="l-menu">
  <li class="l-menu-item">Пункт</li>
  <li class="l-menu-item l-menu-item_active">Пункт</li>
  <li class="l-menu-item l-menu-item_submenu">
    Пункт с подменю
    <ul class="l-menu-submenu">
      <li class="l-menu-subitem">Пункт подменю</li>
      <li class="l-menu-subitem">Пункт подменю</li>
      <li class="l-menu-subitem l-subitem_active">Пункт подменю</li>
    </ul>
  </li>
  <li class="l-menu-item">Пункт</li>
</ul>

Вообще, не нужно бояться дорабатывать методологии "под себя". Удобство разработки и поддержки – вещь в значительной степени субъективная.
Лично я использую свой доработанный бэм-подход и он мне очень нравится.