Skip to content

fix(vue-manager): clean vue-dist before Vite build#398

Open
Ibochkarev wants to merge 1 commit into
betafrom
feat/issue-340-clean-vue-dist
Open

fix(vue-manager): clean vue-dist before Vite build#398
Ibochkarev wants to merge 1 commit into
betafrom
feat/issue-340-clean-vue-dist

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Перед npm run build полностью очищаются js/mgr/vue-dist и css/mgr/vue-dist. Старый prebuild удалял только index*.min.js, поэтому хешированные chunks (FileBrowser-*, request-*, ActionsColumn-*, …) копились.

emptyOutDir: true нельзя: outDir в Vite — корень репозитория (../). В конфиге явно emptyOutDir: false + комментарий. Vendor-сборка не трогалась.

vue-dist в .gitignore — в PR нет удаления артефактов из git, только скрипт очистки.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите):

Связанные Issues

Closes #340

Как это было протестировано?

  • Ручное тестирование
  • Автоматические тесты (PHPStan, ESLint)
  • Тестирование на разных версиях PHP/MODX

Gate E:

  • cd vueManager && npm run lint → exit 0
  • npm run build ×2 → exit 0
  • До: 132 JS-файла, 5.7M (js/mgr/vue-dist)
  • После clean+build: 52 JS-файла, 1.9M; CSS 29 файлов, 920K
  • Второй build: те же счётчики (stable=yes)
  • Entry smoke на диске: orders, order, customers, deliveries, payments, vendors, statuses, links, options, category-products, help, main → OK
  • ActionsColumn-*.min.js копий: 1 (было десятки)

Конфигурация тестирования:

  • MiniShop3: branch feat/issue-340-clean-vue-dist
  • MODX: n/a (build-only; browser smoke менеджера — на ревью)
  • PHP: n/a

Скриншоты (если применимо)

До После
n/a n/a

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок
  • ESLint проходит без ошибок (для JS/Vue изменений)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

  • Ручной smoke в менеджере (orders / products / settings) перед merge желателен — AC issue.
  • npm run clean:vue-dist можно вызывать отдельно без сборки.

Replace partial index*.min.js prebuild wipe with full rimraf of js/css
vue-dist so hashed orphan chunks cannot accumulate. Keep emptyOutDir
false — outDir is the repo root.
@Ibochkarev Ibochkarev added priority: medium Средний приоритет javascript Pull requests that update javascript code enhancement New feature or request tech-debt Maintainability / refactor / architecture debt labels Jul 16, 2026
@Ibochkarev Ibochkarev requested a review from biz87 July 16, 2026 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request javascript Pull requests that update javascript code priority: medium Средний приоритет tech-debt Maintainability / refactor / architecture debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Vite: emptyOutDir / очистка orphan chunks в vue-dist

1 participant