Внедрить фабрики объектов вместо прямых new #14

2026-09-13T03:42:26+03:00

stepan commented

2026-09-13 03:42:26 +03:00

#12

Что сделано

Создание сервисов с порт-интерфейсом перенесено в фабрики IXxxFactory/XxxFactory (метод Create() возвращает интерфейс готового объекта):
- ISecretCipherFactory — AesGcmSecretCipher;
- ITenantLimitStoreFactory — TenantLimitStore;
- IAiClassifierFactory — BudgetedAiClassifier;
- IAiToolsFactory — BudgetedAiTools;
- IFileStorageFactory — LocalFileStorage/MinioFileStorage.
Регистраторы AddDealPersistence, AddDealIntegrations, AddDealSecurity, AddDealFileStorage переведены на фабрики; DiscoveryWorkerService больше не имеет fallback new DiscoverySearchErrorCounter() (счётчик — обязательная DI-зависимость).
Правило фабрик/билдеров и список исключений добавлены в docs/spec/Код-стайл-Дейл.md (§4.1).
Добавлен тест FileStorageFactoryTests (выбор Local/MinIO).

Исключения (осознанно вне правила)

EF-конфигурации IEntityTypeConfiguration — метаданные модели EF;
Ai/Ml/TelegramGrpcConnection — обёртки ресурсов с IDisposable без порт-интерфейса;
статичные хэлперы и объекты без порт-интерфейса, создаваемые контейнером.

Проверки

Сборка 5 решений: 0 предупреждений / 0 ошибок.
Тесты: core 1342, telegram 130, ai 52, ml 38, storage 9 — зелёные.

Затрагивает core (инфраструктура, регистраторы, Discovery); фронт не трогался.

https://gitea.khomegeneric.keenetic.pro/rust/Deal/issues/12 ## Что сделано - Создание сервисов с порт-интерфейсом перенесено в фабрики `IXxxFactory`/`XxxFactory` (метод `Create()` возвращает интерфейс готового объекта): - `ISecretCipherFactory` — `AesGcmSecretCipher`; - `ITenantLimitStoreFactory` — `TenantLimitStore`; - `IAiClassifierFactory` — `BudgetedAiClassifier`; - `IAiToolsFactory` — `BudgetedAiTools`; - `IFileStorageFactory` — `LocalFileStorage`/`MinioFileStorage`. - Регистраторы `AddDealPersistence`, `AddDealIntegrations`, `AddDealSecurity`, `AddDealFileStorage` переведены на фабрики; `DiscoveryWorkerService` больше не имеет fallback `new DiscoverySearchErrorCounter()` (счётчик — обязательная DI-зависимость). - Правило фабрик/билдеров и список исключений добавлены в `docs/spec/Код-стайл-Дейл.md` (§4.1). - Добавлен тест `FileStorageFactoryTests` (выбор Local/MinIO). ## Исключения (осознанно вне правила) - EF-конфигурации `IEntityTypeConfiguration` — метаданные модели EF; - `Ai/Ml/TelegramGrpcConnection` — обёртки ресурсов с `IDisposable` без порт-интерфейса; - статичные хэлперы и объекты без порт-интерфейса, создаваемые контейнером. ## Проверки - Сборка 5 решений: 0 предупреждений / 0 ошибок. - Тесты: core 1342, telegram 130, ai 52, ml 38, storage 9 — зелёные. Затрагивает core (инфраструктура, регистраторы, Discovery); фронт не трогался.

stepan added 2 commits 2026-09-13 03:42:27 +03:00

Внедрить фабрики объектов вместо прямого new d172b6408b

Создание сервисов с порт-интерфейсом перенесено в IXxxFactory:
шифр секретов, хранилище лимитов, ИИ-классификатор, ИИ-инструменты,
файловое хранилище (Local/MinIO). Регистраторы и DiscoveryWorkerService
больше не создают реализации напрямую. Добавлен тест FileStorageFactory.

Зафиксировать правило фабрик и билдеров в код-стайле

ci / build-test (pull_request) Successful in 2m49s

Details

f788b6d39b

stepan requested review from rust 2026-09-13 03:42:28 +03:00

stepan referenced this pull request

2026-09-13 03:42:29 +03:00

Внедрить фабрики объектов вместо прямых new #12

stepan added 1 commit 2026-09-13 03:54:38 +03:00

Стабилизировать ingress-тесты без глобального env

ci / build-test (pull_request) Successful in 2m50s

Details

3bcf4074b5

TelegramIngressTestHost больше не мутирует процессную переменную
DEAL_SERVICE_TOKEN: токен задаётся in-memory конфигурацией хоста.
Три ingress-тест-класса шли параллельно и перетирали env друг другу.

stepan referenced this pull request

2026-09-13 03:54:46 +03:00

Внедрить фабрики объектов вместо прямых new #12

stepan added 1 commit 2026-09-13 03:59:34 +03:00

Сделать реализацию интерфейсов фабрик явной

ci / build-test (pull_request) Successful in 3m9s

Details

c7789c0b21

По код-стайлу §11 метод Create реализуется явно
(ISecretCipher ISecretCipherFactory.Create()), интерфейс вызывается
только по порту; тест фабрики типизирован интерфейсом.

stepan referenced this pull request

2026-09-13 03:59:44 +03:00

Проверить явную реализацию интерфейсов и дублирование summary #15

stepan added 2 commits 2026-09-13 04:10:25 +03:00

Убрать самари конструкторов фабрик 418f6efe39

Запретить самари конструкторов в код-стайле

ci / build-test (pull_request) Successful in 2m52s

Details

58df018ea4

stepan added 2 commits 2026-09-13 04:17:46 +03:00

Убрать XML-доки из реализаций фабрик a9d26b6ffc

Реализации интерфейсов — без XML-доков (код-стайл)