[epic] Аудит дублирования кода: дедуп и упрощение

[axisrow]

Зонтичный трекер по результатам аудита всего проекта на дублирование и возможности упрощения (xhigh-effort: 4 параллельных аудитора по областям + ручная верификация чтением кода). Рабочее дерево чистое на коммите f77b52e.

Кодовая база зрелая — utils.py, _lifecycle.py (17 файлов), _batch.py показывают, что дедуп уже идёт. Осталось несколько пластов, где паттерн доказан, а тиражируется руками.

Дочерние задачи (по убыванию ROI)

• refactor(_bidding_strategy): дедуп budget-билдеров и пар search/network стратегий (~2000 строк) #581 — closed by refactor(bidding): dedup budget builders; make YES-only divergence explicit (#581) #590; follow-ups _bidding_strategy: дедуп _assemble_*_strategy_block (dynamic/smart → паттерн unified) #591 (assemble*), _bidding_strategy: дедуп пар build_*_search/network_strategy через config-объект #592 (search/network pairs) — _bidding_strategy.py (~2000 строк дублей): 4 копии _build_*_custom_period_budget, 4 копии _build_*_exploration_budget (со скрытым YES-only расхождением ⚠️), 6 пар build_*_search/*_network_strategy, 3 копии _assemble_*_strategy_block.
• refactor(commands): фабрика make_get_command для идентичного скелета get (26 файлов) #582 — closed by refactor(commands): make_get_command factory for the v5 get skeleton (#582) #586; follow-ups make_get_command: мигрировать get с вложенными полями + criteria-limit (Группы 2-3) #587 (groups 2-3), make_get_command: мигрировать сложные get-команды (Группа 4) #588 (group 4), commands dedup: lifecycle-обёртки (#11), make_set_bids_command (#12), execute_add (#13) #589 (helpers fix(ads): stop sending implicit Type field in add command #11/fix+test: stop sending implicit Type field in add commands; add dry-run coverage #12/feat(ads get): add --text-ad-fields for type-specific field retrieval #13) — commands/: фабрика make_get_command для идентичного get-скелета (26 файлов, 72 вхождения if dry_run); локальные lifecycle-обёртки; дубль set-bids; boilerplate add.
• refactor(v4, coverage): общие v4-хелперы и единая captcha-детекция #583 — closed by refactor(v4,coverage): shared v4 option helpers + single captcha registry (#583) #585 — v4 + coverage: общие v4-хелперы (_non_empty байт-в-байт, парсинг ID=VALUE ×3, парсинг списков ID ×3); единая captcha-детекция между wsdl_coverage.py и reports_coverage.py.

Рекомендованный порядок

1. refactor(_bidding_strategy): дедуп budget-билдеров и пар search/network стратегий (~2000 строк) #581 — максимум дублей в одном файле + скрытое поведенческое расхождение, опасное для копипаст-поддержки.
2. refactor(commands): фабрика make_get_command для идентичного скелета get (26 файлов) #582 — фабрика make_get_command (паттерн уже доказан _lifecycle.py).
3. refactor(v4, coverage): общие v4-хелперы и единая captcha-детекция #583 — быстрые низкорисковые победы.

Намеренно НЕ выносится

• CAMPAIGN_TYPE_BUILDERS реестр — задел под расширение (campaigns bidding strategies: TextCampaign search branch #361–369), не дубль.
• parse_ids vs add_criteria_csv и контракт пустого SelectionCriteria — стиль/контракт ошибок, вне темы дедупа.

Дисциплина при рефакторинге

Все находки сохраняют CLI-поверхность 1:1: i18n-ключи, тексты ошибок и --dry-run payload должны остаться байт-идентичными. Гейты test_wsdl_parity_gate.py, test_dry_run.py, test_api_coverage.py — обязательны к прохождению.

[axisrow]

Фаза 1 завершена — все три дочерние issue смерджены ✅

Issue	PR	Эффект (нетто строк)
#583 — v4-хелперы + captcha	#585	+274 / −197
#582 — фабрика make_get_command	#586	+198 / −214
#581 — budget-билдеры	#590	+62 / −235

Каждый PR: байт-идентичная CLI-поверхность (флаги, тексты ошибок, i18n-ключи, --dry-run payload), /simplify + cloud cycle-review (@claude + @codex, оба без блокирующих замечаний), зелёный CI, squash-merge.

Заметные побочные улучшения: api.resolve_module_create_client чинит латентный patchability-дефект фабрик; скрытое YES-only расхождение в exploration-budget стало явным required-аргументом; captcha-маркеры и v4-валидаторы сведены в единые реестры.

Фаза 2 — follow-up (вынесено по правилу ≤1000 добавленных строк на PR)

• refactor(commands): фабрика make_get_command для идентичного скелета get (26 файлов) #582 → make_get_command: мигрировать get с вложенными полями + criteria-limit (Группы 2-3) #587 (get группы 2-3), make_get_command: мигрировать сложные get-команды (Группа 4) #588 (get группа 4), commands dedup: lifecycle-обёртки (#11), make_set_bids_command (#12), execute_add (#13) #589 (lifecycle-обёртки / make_set_bids / execute_add)
• refactor(_bidding_strategy): дедуп budget-билдеров и пар search/network стратегий (~2000 строк) #581 → _bidding_strategy: дедуп _assemble_*_strategy_block (dynamic/smart → паттерн unified) #591 (_assemble_*_strategy_block), _bidding_strategy: дедуп пар build_*_search/network_strategy через config-объект #592 (пары search/network через config-объект; + переиспользование новых budget-хелперов в dynamic/smart/mobile)

Намеренно НЕ трогалось: реестр CAMPAIGN_TYPE_BUILDERS, build_*_base, parse_ids/SelectionCriteria-контракты (вне темы дедупа).