encumbrance.zouit_count приходит из parcels.py как len(zouit_rows) — число ЗАПИСЕЙ
cad_zouit, пересёкших участок. Типы лежат отдельно, в zouit_types, и показаны
строкой ниже. Оба экспортёра подписывали это число как «Кол-во типов ЗОУИТ».
Расхождение не редкое — померил по реальным отчётам (analysis_runs):
разборов с ЗОУИТ 1637
где записей != типов 717 (43.8%)
среднее отношение 1.35x
максимум по проду 7 записей при 3 типах
То есть почти в половине отчётов с ЗОУИТ читателю показывали число, которое на
треть больше того, что обещает подпись.
Пункт эпика предлагал обратное — заменить значение на len(zouit_types). Это
сделало бы фолбэк несогласованным с основным путём: у источника счётчик считает
записи, и таким он приходит в чат-контекст (chat/retrieval.py) тоже. Врёт
подпись, её и правлю.
Тест формулирует инвариант по смыслу, а не сверкой с выбранной строкой: ЕСЛИ
подпись обещает типы, показанное число обязано равняться числу типов. Такая
формулировка переживёт разумное переименование и не даст «починить» тест
подгонкой подписи. Проверяются оба формата — HTML и DOCX.
Против origin/main (5 записей, 2 типа):
html: подпись «Кол-во типов ЗОУИТ» показывает 5, типов 2 → падает
docx: то же самое → падает
число записей всё ещё в сводке — контроль, зелёный с обеих сторон
перечисление типов на месте — контроль, зелёный с обеих сторон
Контроли не для симметрии: первый ловит «починку», которая выкинула бы счётчик
вместо переименования, второй — потерю перечисления типов.
Прогоны: tests/services/exporters + tests/services/chat — 302 passed rc=0.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
excel.py рендерил _scenario_deficit_index (скаляр) в таблицу сценариев под
шапкой «(12 мес)» — при fallback на другой горизонт подпись лгала (то же что
docx/pptx до 764db617). Добавлены _scenario_deficit_horizon и
_scenario_deficit_cell (зеркало report_pdf.py/764db617): при fallback ячейка
несёт «(гор. N мес)». _build_scenarios_sheet переключён на _scenario_deficit_cell.
Тесты: два новых кейса — fallback (6 мес → аннотация) и primary (12 мес → скаляр).
#1244 (security): внешние/скрейпинг-строки (comm_name из DOM.РФ, headline/usp_text)
с ведущим = + - @ \t \r писались как есть → openpyxl сохранял как формулы
(data_type='f'), исполнялись при открытии в Excel/LibreOffice. _sanitize_formula
префиксует такие строки апострофом (OWASP CSV-injection escape); числа/даты/bool
не трогаются. _write_kv labels тоже санитизируются. Подтверждено на openpyxl 3.1.5.
#1245 (concurrency): async ask() вызывал sync get_report_for_chat() (sync SQLAlchemy
тянет крупный JSONB §22-отчёт) напрямую — блокировал event loop, в отличие от
LLM-ветки (run_in_threadpool). Обёрнуто в run_in_threadpool.
Closes#1244Closes#1245
render_report_docx (python-docx) mirrors report_md/report_pdf section order &
content, reuses report_pdf pure helpers (DRY), graceful on thin/empty report.
Widen /forecast/export format Literal to include docx → Word attachment.
Add python-docx dep + regenerate uv.lock (uv sync --frozen passes). Part of #959.
render_report_telegram_summary (pure, no new deps, DRY-reuses report_pdf str
helpers) + `tg` format on GET /{cad}/forecast/export → inline text/plain snippet
(no attachment, copy-paste-ready). md/json unchanged; no-run 404, bad format 422.
Graceful on thin/empty reports. Part of EPIC #959.
render_report_markdown (pure, no new runtime deps) reuses report_pdf's str
helpers (DRY), + GET /{cad_num}/forecast/export?format=md|json. No forecast run
→ 404; graceful on thin/empty reports; GFM-safe table escaping. PDF/XLSX already
existed; this adds the cheapest no-dep formats. Part of EPIC #959.
deficit_index pins to -1.0 for every ЕКБ segment (12mo demand flow vs multi-year
supply stock → log-ratio clamps) → zero discriminating power, though the oversupply
is partly real. Add MOI (gross competing supply / demand_per_mo), the real-estate
absorption standard, as an additive non-saturating companion that DISCRIMINATES
(Уралмаш 42mo … Чермет 109mo) where deficit cannot. deficit_index math kept exactly
as-is (honest absolute: -1 = genuinely oversupplied); docstrings clarify -1 is common
and MOI is the discriminating companion (no recalibration). _gross_supply extract-method
(single source of truth; _project_supply behavior byte-identical, code-review-verified).
Surface MOI in §22 future_market (passthrough) + exec_summary key_numbers/verdict.
Guards: no demand → None, no supply → 0. Prod: MOI varies 42→109mo, deficit stays -1.
Discrimination test pins MOI separating two segments both at deficit -1. Refs #952.
REOPENED. PDF + Excel exporters read non-existent dict keys, so demand/supply/
scenario columns silently rendered "—". Tests passed only because the fixtures
were stale (hand-typed the same wrong keys → fixture agreed with buggy exporter).
- future_market: demand/supply → projected_demand_units/projected_supply_units
- scenarios: drop non-existent per-scenario "overall"; show primary-horizon
deficit_index from ScenarioForecast.forecasts (scoring.overall was NOT broken)
- Excel #991: add missing future_supply (index + breakdown) + confidence.factors
sections; add future_supply to PDF for parity
- tests: rebuild forecast/scenario fixtures from real DemandSupplyForecast /
ScenarioForecast as_dict(); contract-key regression guards fail on key-drift
(verified: reintroducing old keys fails the new tests). 28 passed.
Refs #989#991
