Процедуры контроля (Code Review) и автоматизация

Code Review чеклист

Обязательные проверки ревьюера

• В новом UI-коде отсутствуют hard-coded значения:

  • Цвета (#hex, rgb())
  • Отступы (16px, 1rem напрямую)
  • Размеры (width: 200px)

• Используются дизайн-токены или допустимые UI-константы:

  • var(--color-primary) или className="text-primary"

• Новые значения не копируются из legacy-кода:

  • Если затронут старый компонент - код отрефакторен

• Иконки используются через Icon компонент:

  • Нет прямого импорта SVG

• Inline-SVG применяется только с обоснованием:

  • Комментарий объясняет причину (сложная анимация)

• Нейминг вариантов универсальный (не привязан к экрану)

• Компоненты не содержат продуктовой логики

• Wrapper-компоненты оправданы (содержат бизнес-логику)

Автоматизация

Статические проверки (ESLint / Stylelint)

Запрет hard-coded значений в коде

• Пример Stylelint конфига – stylelint.ts
• Пример ESLint правила – no-color-literals

Запрет прямого импорта .svg файлов в обход Icon-компонента (если применимо)

• Пример ESLint правила – no-svg-import