refactor(orders): decompose OrdersController + ServiceRegistry factory map

[Ibochkarev]

Описание

Декомпозиция OrdersController (#338): list/mutation/products/presenter вынесены в ManagerOrder* сервисы, контроллер — тонкий HTTP-слой с Response. Рефакторинг ServiceRegistry (#345): явная карта фабрик ServiceRegistryFactories вместо switch/in_array для wiring; зарегистрированы manager order сервисы и ms3_extra_fields.

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

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

Связанные Issues

Closes #338
Closes #345

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

php -l core/components/minishop3/src/Controllers/Api/Manager/OrdersController.php
php -l core/components/minishop3/src/ServiceRegistry.php
php -l core/components/minishop3/src/ServiceRegistryFactories.php
# exit 0

cd core/components/minishop3 && composer test:smoke
# OK smoke tests (27), включая ManagerOrdersDecompositionTest, ServiceRegistryFactoryMapTest

• Ручное тестирование
• Автоматические тесты (composer test:smoke)
• Тестирование на разных версиях PHP/MODX

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

• MiniShop3: ветка feat/issue-338-345-orders-registry
• MODX: n/a (smoke без полной установки)
• PHP: 8.2+

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

До	После
OrdersController ~1804 LOC	~404 LOC

Чеклист

• Код соответствует стилю проекта
• Добавлены/обновлены комментарии в сложных местах
• Изменения не ломают существующую функциональность
• Лексиконы — не требовались
• PHPStan — полный stan в CI
• ESLint — Vue не трогали
• CHANGELOG — по политике репо при релизе

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

#338 AC

Критерий	Результат
OrdersController < 1000 строк	404 LOC (smoke)
mgr REST без breaking changes	envelope сохранён; deleteProduct → data: null
Нет новой логики в manager.php closures	n/a
php -l + smoke	exit 0

#345 AC

Критерий	Результат
Нет switch/in_array wiring	ServiceRegistryFactoryMapTest
Orders/ExtraFields в $modx->services	50 factories = defaultServices keys
Boot без регрессий	smoke green

Новые DI-ключи

ms3_manager_order_presenter, ms3_manager_order_list, ms3_manager_order_mutation, ms3_manager_order_products, ms3_manager_order_cost_recalculator, ms3_extra_fields

Follow-up (не блокирует)

• Вынести getLogs / finalize / recalculateCost оркестрацию из контроллера
• Shared success/error/getOrderLog base для mutation/products
• GridConfigController → ManagerOrderListService::getDirectFilterKeys()

[biz87]

Спасибо! Это последний крупный в цикле — держим его на конец, как ты и планировал. Перед мержем нужен ребейз на финальную beta: за цикл в OrdersController и ServiceRegistry влилось много, и декомпозиция (+2010/−1629) с ними конфликтует.

Что нужно сохранить/свести при декомпозиции:

OrdersController:

• fix: проверка связи delivery/payment при оформлении заказа #459 — проверка совместимости пары delivery/payment (getDeliveryPaymentPairError) на create/update + finalize;
• perf(orders): conditional Address JOIN и отдельный stats endpoint #469 — conditional Address JOIN (needsAddressJoin) + отдельный endpoint GET /orders/stats (stats убраны с hot-path getList);
• feat(api): лексикон для сообщений CategoryProductsController #467/feat(mgr-api): document-level ACL для category products #473/feat(mgr-api): category-scoped PUT для inline-edit product data #474 — lexicon-хелперы + ACL (если пересекается с order-контроллером).

ServiceRegistry (factory-map этого PR пересекается с реестром — нужен union регистраций):

• refactor(di): OptionSync/Loader + ManagerOrderCostRecalculator in ServiceRegistry #477 (DI recalculator/OptionSync), feat(settings): consolidate dual API combo lists, drop member processors #490 (SettingsComboListService, payments-active), refactor(mgr-api): общий CRUD deliveries ↔ payments #509 (Ms3ReferenceCrudService), refactor(mgr-api): ModelFieldsController → ModelField services #517 (ModelField services), GridConfigService: request-scoped memoize getGridConfig #488 (GridConfig memoize), fix(di): OptionCategoryService в ms3_option_service (Settings/Category Options) #535 (OptionCategoryService fix), feat(order): публичный sessionless API программного создания заказов #508 (ms3_programmatic_order), refactor(options): разрез OptionLoaderService (Phase B #365) #514/refactor(product): разрез ProductDataService (Phase C #365) #515 (Option/ProductData распил).

После ребейза провалидирую по полному протоколу: PHPStan + smoke + PHPUnit, особое внимание ServiceRegistryDiTest (все DI-ключи резолвятся) и order-тестам (#459 pair, #469 stats/JOIN). Как будет готово на финале — вольём.

[Ibochkarev]

Rebase на актуальную beta выполнен (по комментарию @biz87).

Что сохранено при сведении

• fix: проверка связи delivery/payment при оформлении заказа #459 pair delivery/payment: ManagerOrderMutationService (create/update) + OrderFinalizeService
• perf(orders): conditional Address JOIN и отдельный stats endpoint #469 stats/JOIN: getStats → ManagerOrderListService::getOrdersStats, needsAddressJoin через ManagerOrderListQueryService
• DI union в ServiceRegistry + ServiceRegistryFactories (68 ключей), в т.ч. beta-only: ms3_programmatic_order, ms3_settings_combo_list, ms3_validation_service, ms3_product_*, ms3_customer_order, ms3_option_category_service
• fix(di): OptionCategoryService в ms3_option_service (Settings/Category Options) #535: ms3_option_service резолвит ms3_option_category_service (не ms3_category_option_service)

Gates

php tests/ServiceRegistryDiTest.php              # OK
php tests/ServiceRegistryFactoryMapTest.php      # OK (68)
php tests/ManagerOrdersDecompositionTest.php     # OK (412 lines)
php tests/DeliveryPaymentAvailabilityTest.php    # OK
composer test:smoke                              # OK (69)

OrdersController после декомпозиции: 412 LOC.