Критические замечания

1. Изменение префикса лексикона в _build/build.php

- 'desc' => 'ms3_prop_' . $k,
+ 'desc' => 'ms_prop_' . $k,

Вопрос: Префикс лексикона изменён с ms3_prop_ на ms_prop_.

• Существуют ли лексиконы с префиксом ms_prop_?
• Это намеренное изменение или опечатка?

Если лексиконы ms_prop_* не существуют, описания свойств сниппетов перестанут отображаться.

2. Даунгрейд @primeuix/themes в package.json

- "@primeuix/themes": "^2.0.3",
+ "@primeuix/themes": "^1.0.0",

Вопрос: Версия понижена с 2.x до 1.x. Причина не указана в описании PR.

Возможные последствия:

• Несовместимость с PrimeVue 4.3.1
• Визуальные регрессии в компонентах

Пожалуйста, укажи причину даунгрейда.

Замечания средней важности

3. Удалён eslint-disable комментарий

// vueManager/src/utils/modx.js
- // eslint-disable-next-line no-undef
  return typeof ms3 !== 'undefined' ? ms3.config : null

Нужно убедиться, что глобальные переменные ms3, MODx, Ext добавлены в eslint.config.js → globals. Иначе ESLint будет выдавать ошибки no-undef.

Проверка: В текущем eslint.config.js вижу только:

globals: {
  Ext: 'readonly',
  MODx: 'readonly',
}

Переменная ms3 отсутствует — нужно добавить.

4. CSS Variables — возможное упрощение палитры

Файл _variables.scss содержит ~60 CSS переменных, что хорошо для consistency. Однако некоторые цвета очень близки:

Рекомендация: рассмотреть объединение близких цветов для упрощения поддержки.

5. Runtime injection стилей в formStyles.js

const el = document.createElement('style')
el.id = STYLE_ID
el.textContent = css
document.head.appendChild(el)

Комментарий в коде объясняет причину: "PrimeVue встраиваются в runtime и перезаписывают CSS из файлов".

Это работает, но добавляет complexity. Альтернатива — использовать !important в SCSS или увеличить специфичность селекторов.

Положительные моменты

• Dropdown → Select — правильная миграция под актуальные практики PrimeVue 4
• camelCase → kebab-case в атрибутах Vue — соответствует Vue 3 style guide
• px → rem — улучшает accessibility и масштабируемость
• Prettier + ESLint + Stylelint — единый code style для всей команды
• CSS переменные — упрощает кастомизацию темы
• eslint-plugin-simple-import-sort — автоматическая сортировка импортов улучшает читаемость
• Структурирование PHP контроллеров — добавлены комментарии к подключениям CSS/JS

Чеклист перед мержем

• Уточнить изменение ms3_prop_ → ms_prop_ в build.php
• Уточнить причину даунгрейда @primeuix/themes с 2.x на 1.x
• Добавить ms3 в ESLint globals
• Запустить npm run build — проверить сборку
• Визуальное тестирование основных страниц:
  • Список товаров
  • Карточка товара
  • Заказы
  • Настройки
  • Статусы

Резюме

PR содержит много полезных улучшений инфраструктуры и code style. Однако перед мержем необходимо:

1. Обязательно прояснить два критических момента (лексиконы и версия themes)
2. Желательно добавить ms3 в ESLint globals
3. Рекомендуется провести визуальное тестирование из-за большого объёма изменений

@biz87 можно глянуть