[tradein] scraper_kit Group F3: системный аудит config-footgun паттерна во всех providers/* #2351

New issue

Closed

opened 2026-07-03 23:51:13 +00:00 by lekss361 · 0 comments

lekss361 commented

2026-07-03 23:51:13 +00:00

Owner

Родительский: #2309 (Group F). Read-only разведка, НЕ пишет фиксы сама — производит список находок.

Мотивация

За эпик #2277 (Groups B/C/E1-E4) нашли 5 независимых инстансов одного класса бага: kit-функция
с сетевым/прокси-поведением, у которой параметр config/endpoint либо optional-и-молча-теряет-
-поведение, либо отсутствует в сигнатуре вообще — потому что legacy читал settings.* implicit,
а kit требует explicit DI. Каждый раз находили по одному, при миграции конкретного caller'а.
См. #2322, #2330 (уже заведены, F1/F2) — но это может быть не всё.

Scope

Пройтись по ВСЕМ функциям в tradein-mvp/packages/scraper-kit/src/scraper_kit/providers/*,
которые:

Создают BrowserFetcher/httpx-client/сессию с сетевым I/O
Имеют config/proxy_provider/delay_provider/endpoint параметр (optional ИЛИ отсутствующий)

Для каждой — проверить: если вызвать БЕЗ этого параметра (или он не проброшен из caller), теряется
ли реально proxy/throttle/endpoint-поведение (сравнить с legacy-эквивалентом, если есть). НЕ
чинить на месте — только составить список находок с file:line + severity.

Output

Отчёт (список находок), каждая находка ≥ severity "может сломать прод-caller" → отдельный
Forgejo issue (как #2322/#2330), заведённый под #2309. Находки cosmetic/theoretical — просто
перечислить в комментарии к этому issue, не заводить отдельно.

DoD

Полный список функций-кандидатов (grep по сигнатурам + сетевому I/O)
Для каждой — вердикт (footgun confirmed / not applicable / needs manual check) с обоснованием
Новые issues заведены под #2309 для каждого confirmed footgun

Не пересекается по файлам с F1/F2 (read-only) — можно параллельно. Должен завершиться ДО F4
(структурный рефактор) — F4 не должен строиться поверх ещё не найденных багов того же класса.

Refs #2309

Родительский: #2309 (Group F). Read-only разведка, НЕ пишет фиксы сама — производит список находок. ## Мотивация За эпик #2277 (Groups B/C/E1-E4) нашли 5 независимых инстансов одного класса бага: kit-функция с сетевым/прокси-поведением, у которой параметр `config`/`endpoint` либо optional-и-молча-теряет- -поведение, либо отсутствует в сигнатуре вообще — потому что legacy читал `settings.*` implicit, а kit требует explicit DI. Каждый раз находили по одному, при миграции конкретного caller'а. См. #2322, #2330 (уже заведены, F1/F2) — но это может быть не всё. ## Scope Пройтись по ВСЕМ функциям в `tradein-mvp/packages/scraper-kit/src/scraper_kit/providers/*`, которые: - Создают `BrowserFetcher`/httpx-client/сессию с сетевым I/O - Имеют `config`/`proxy_provider`/`delay_provider`/`endpoint` параметр (optional ИЛИ отсутствующий) Для каждой — проверить: если вызвать БЕЗ этого параметра (или он не проброшен из caller), теряется ли реально proxy/throttle/endpoint-поведение (сравнить с legacy-эквивалентом, если есть). НЕ чинить на месте — только составить список находок с file:line + severity. ## Output Отчёт (список находок), каждая находка ≥ severity "может сломать прод-caller" → отдельный Forgejo issue (как #2322/#2330), заведённый под #2309. Находки cosmetic/theoretical — просто перечислить в комментарии к этому issue, не заводить отдельно. ## DoD - [ ] Полный список функций-кандидатов (grep по сигнатурам + сетевому I/O) - [ ] Для каждой — вердикт (footgun confirmed / not applicable / needs manual check) с обоснованием - [ ] Новые issues заведены под #2309 для каждого confirmed footgun Не пересекается по файлам с F1/F2 (read-only) — можно параллельно. Должен завершиться ДО F4 (структурный рефактор) — F4 не должен строиться поверх ещё не найденных багов того же класса. Refs #2309