fix(#234): NSPD harvest ETA badge + SETNX dedupe (Sub-PR C) #250

Merged

lekss361 merged 3 commits from fix/234-nspd-harvest-eta-badge into main

2026-05-17 06:35:41 +00:00

Author	SHA1	Message	Date
lekss361	81cf274e80	fix(#234 ): address re-review nits — Settings singleton + docstring sync 1. _acquire_harvest_lock: Settings() instance → singleton 'settings' import from app.core.config — соответствует convention (scrape_kn.py, objective.py, celery_app.py все используют singleton). 2. _trigger_harvest docstring: убрана incorrect фраза «остальные ... видят harvest_eta_seconds от первого запроса» (код возвращает eta=None для denied-lock callers). Заменена на actual behavior: graceful no-data UI с ПКК fallback на FE. PR #250 re-review followup (issues #3 + docstring sync).	2026-05-17 09:31:43 +03:00
lekss361	b99ac76463	Merge remote-tracking branch 'forgejo/main' into fix/234-nspd-harvest-eta-badge # Conflicts: # backend/app/services/site_finder/quarter_dump_lookup.py	2026-05-17 09:19:12 +03:00
lekss361	990c109756	fix(#234 ): NSPD harvest ETA badge + SETNX dedupe (Sub-PR C) Backend (quarter_dump_lookup.py): - _acquire_harvest_lock: Redis SETNX TTL=120s на quarter, защищает от burst N concurrent analyze, ставящих N одинаковых harvest task в очередь - _trigger_harvest: использует lock перед apply_async, возвращает False если lock уже взят (другой запрос триггернул раньше) - make_empty_result/EMPTY_DUMP_RESULT: новое поле harvest_eta_seconds в nspd_dump dict, типичный harvest_quarter = 60с - /analyze: пробрасывает поле через nspd_dump dict (нет typed schema — response_model=None для /analyze endpoint, dict уходит как есть) Frontend (NspdFreshnessBadge, NspdZoningBlock): - Countdown «НСПД: загрузка ~Nс» вместо бесконечного спиннера - После остановки countdown (remaining=0) NspdZoningBlock показывает «загрузка дольше обычного» + ссылку на ПКК вместо infinite skeleton Tests: 5 новых unit + 2 для empty_result schema (всего +7, pass) Closes #234 (UX-side; data-side resolves когда Sub-PR B + D merged).	2026-05-17 09:13:06 +03:00

Author

SHA1

Message

Date

lekss361

81cf274e80

fix(#234 ): address re-review nits — Settings singleton + docstring sync

1. _acquire_harvest_lock: Settings() instance → singleton 'settings' import
   from app.core.config — соответствует convention (scrape_kn.py, objective.py,
   celery_app.py все используют singleton).
2. _trigger_harvest docstring: убрана incorrect фраза «остальные ... видят
   harvest_eta_seconds от первого запроса» (код возвращает eta=None для
   denied-lock callers). Заменена на actual behavior: graceful no-data UI
   с ПКК fallback на FE.

PR #250 re-review followup (issues #3 + docstring sync).

2026-05-17 09:31:43 +03:00

lekss361

b99ac76463

Merge remote-tracking branch 'forgejo/main' into fix/234-nspd-harvest-eta-badge

# Conflicts:
#	backend/app/services/site_finder/quarter_dump_lookup.py

2026-05-17 09:19:12 +03:00

lekss361

990c109756

fix(#234 ): NSPD harvest ETA badge + SETNX dedupe (Sub-PR C)

Backend (quarter_dump_lookup.py):
- _acquire_harvest_lock: Redis SETNX TTL=120s на quarter, защищает от burst
  N concurrent analyze, ставящих N одинаковых harvest task в очередь
- _trigger_harvest: использует lock перед apply_async, возвращает False если
  lock уже взят (другой запрос триггернул раньше)
- make_empty_result/EMPTY_DUMP_RESULT: новое поле harvest_eta_seconds в
  nspd_dump dict, типичный harvest_quarter = 60с
- /analyze: пробрасывает поле через nspd_dump dict (нет typed schema —
  response_model=None для /analyze endpoint, dict уходит как есть)

Frontend (NspdFreshnessBadge, NspdZoningBlock):
- Countdown «НСПД: загрузка ~Nс» вместо бесконечного спиннера
- После остановки countdown (remaining=0) NspdZoningBlock показывает
  «загрузка дольше обычного» + ссылку на ПКК вместо infinite skeleton

Tests: 5 новых unit + 2 для empty_result schema (всего +7, pass)

Closes #234 (UX-side; data-side resolves когда Sub-PR B + D merged).

2026-05-17 09:13:06 +03:00