Общая оценка: APPROVE (с минорными замечаниями)

PR добавляет важную функциональность локализации PrimeVue компонентов. Изменения логичные и последовательные.

Что делает PR

1. Добавляет локализацию PrimeVue через @vuetools/usePrimeVueLocale
2. Обновляет все 18+ entry points — добавляет locale: getPrimeVueLocale() в конфигурацию PrimeVue
3. Обновляет vite.config.js — добавляет зависимость в externals
4. Bump версии — 1.3.0 → 1.4.0-beta1

Плюсы

1. Консистентность — изменения применены ко всем entry points единообразно
2. Правильный подход — использование external dependency через VueTools Import Map
3. Документация — CHANGELOG обновлён
4. Чеклист выполнен — ESLint, backward compatibility

Замечания

1. vite.config.js — проверить добавление зависимости

Убедиться, что @vuetools/usePrimeVueLocale добавлен в массив vuetoolsComposables:

const vuetoolsComposables = [
  '@vuetools/useApi',
  '@vuetools/useLexicon',
  '@vuetools/useModx',
  '@vuetools/usePermission',
  '@vuetools/usePrimeVueLocale'  // <-- должен быть добавлен
]

2. Несогласованность в импорте темы Aura

В разных entry файлах используются разные пути импорта:

// orders.js
import Aura from '@primevue/themes/aura';

// statuses.js, deliveries.js
import Aura from '@primeuix/themes/aura';

Рекомендация: Унифицировать импорт во всех файлах. Правильный путь для PrimeVue 4.x — @primeuix/themes/aura.

3. api-test.html — проверить добавление cultureKey

PR добавляет cultureKey: 'ru' в мок MODX конфигурации. Убедиться, что добавлено корректно:

window.MODx = {
  config: {
    MODAUTH: 'dev-token-12345',
    user_id: 1,
    username: 'admin',
    context_key: 'mgr',
    debug: true,
    cultureKey: 'ru'  // <-- должен быть добавлен
  },
  ...

4. Отсутствует Pinia в некоторых entry points

orders.js использует Pinia, а deliveries.js и statuses.js — нет. Это может быть намеренно (разные требования), но стоит проверить консистентность подхода.

5. Minor: Форматирование

PR упоминает "quote standardization and trailing comma adjustments". Убедиться, что ESLint прошёл без ошибок.

Вопросы к автору

1. Fallback локали: Если cultureKey не определён или недоступен — какой язык будет по умолчанию?

2. VueTools зависимость: Какая минимальная версия VueTools требуется для usePrimeVueLocale?

3. Поддерживаемые локали: Какие языки поддерживаются? Только ru/en или больше?

Итоговый чеклист

Рекомендация

APPROVE — можно мержить после проверки замечаний по импорту Aura и cultureKey в api-test.html.