fix(site-finder): метка источника весов выводится из результата резолва, а не из входа (#2811) #2817

Merged
bot-backend merged 1 commit from fix/2811-weights-source-honesty into main 2026-08-10 10:34:40 +00:00

1 commit

Author SHA1 Message Date
bot-backend
34d5fb76b7 fix(site-finder): метка источника весов выводится из результата резолва, а не из входа (#2811)
All checks were successful
CI Trade-In / changes (pull_request) Successful in 9s
CI / changes (pull_request) Successful in 9s
CI Trade-In / backend-tests (pull_request) Has been skipped
CI Trade-In / browser-tests (pull_request) Has been skipped
CI Trade-In / frontend-checks (pull_request) Has been skipped
CI / frontend-tests (pull_request) Has been skipped
CI / openapi-codegen-check (pull_request) Successful in 2m18s
CI / backend-tests (pull_request) Successful in 16m50s
`/analyze` ставил `weights_profile.source = "profile"` по условию
`profile_id is not None` — то есть по факту, что параметр пришёл. Нашёлся ли
профиль, никто не спрашивал, а `resolve_weights` при промахе молча уходил вниз
по лестнице приоритетов (единственный след — `logger.debug`, которого на проде
нет). Ответ утверждал, что оценка посчитана по профилю, хотя веса были другие,
и опровергнуть это по ответу было нечем.

Воспроизведено живым запросом на проде (все три способа промаха: owner не
передан, чужой профиль, несуществующий id) и найдено в истории: ран
analysis_runs #4000 от 2026-08-07 — source='profile', profile_id=1, при
tram_stop=-0.5 (системный; у профиля 1 он -0.4).

- `resolve_weights` возвращает `ResolvedWeights(weights, source)` — вызывающий
  не может взять веса и не взять источник.
- Запрошенный, но не применённый профиль пишется `logger.warning` с id.
- В ответе появился `weights_profile.requested_profile_applied` (True/False/None):
  «что просили» и «что получилось» остаются разными величинами, source не
  схлопывается в "system".
- HTTP-код не менялся: profile_id для /analyze — необязательный модификатор,
  а не адресуемый ресурс; 404/422 превратил бы гонку «профиль удалили между
  списком и анализом» в отказ вместо честно помеченного ответа.

Refs #2811
2026-08-10 15:15:13 +05:00