lekss361/gendesign

Fork 0

fix(export): «НДС (паркинг)» → «НДС (паркинг + коммерция)» в full_report экспортёрах (follow-up #2445-C1) #2457

New issue

Closed

opened 2026-07-05 19:23:32 +00:00 by bot-backend · 1 comment

bot-backend commented

2026-07-05 19:23:32 +00:00

Collaborator

Follow-up от PR #2455 (epic #2445, C1). Тот PR починил caveat в concept-PDF (generative/exporters/pdf.py). Ревьюер нашёл related мислейбл в другом семействе экспортёров (site-finder full-report):

backend/app/services/exporters/full_report_html.py:1339
backend/app/services/exporters/full_report_docx.py:870

Оба рендерят строку НДС с меткой «НДС (паркинг)», хотя vat_rub теперь включает и добавленную стоимость коммерции/офисов (после добавления office-support в financial.py/teap.py). Метка неполная/вводит в заблуждение — в том же духе, что C1, но это table-row label, а не противоречивый caveat-параграф.

Scope

Переименовать метку → «НДС (паркинг + коммерция)» в обоих файлах, согласовав с concept-экспортёром (pdf.py уже использует «нежилое: паркинг + коммерция»). Проверить, нет ли рядом такого же stale-caveat про «только машиноместа».

Estimated: S (2 маленькие правки метки + возможно caveat рядом).

Refs #2445, #2455

Follow-up от PR #2455 (epic #2445, C1). Тот PR починил caveat в concept-PDF (`generative/exporters/pdf.py`). Ревьюер нашёл related мислейбл в **другом семействе экспортёров** (site-finder full-report): - `backend/app/services/exporters/full_report_html.py:1339` - `backend/app/services/exporters/full_report_docx.py:870` Оба рендерят строку НДС с меткой **«НДС (паркинг)»**, хотя `vat_rub` теперь включает и добавленную стоимость коммерции/офисов (после добавления office-support в `financial.py`/`teap.py`). Метка неполная/вводит в заблуждение — в том же духе, что C1, но это table-row label, а не противоречивый caveat-параграф. ## Scope Переименовать метку → «НДС (паркинг + коммерция)» в обоих файлах, согласовав с concept-экспортёром (`pdf.py` уже использует «нежилое: паркинг + коммерция»). Проверить, нет ли рядом такого же stale-caveat про «только машиноместа». Estimated: S (2 маленькие правки метки + возможно caveat рядом). Refs #2445, #2455

bot-backend referenced this issue

2026-07-05 20:10:08 +00:00

EPIC: аудит кода ПТИЦЫ (мульти-агентный code review 2026-07-05) — 18 подтверждённых находок #2445

bot-backend referenced this issue

2026-07-05 20:16:03 +00:00

EPIC: аудит кода ПТИЦЫ (мульти-агентный code review 2026-07-05) — 18 подтверждённых находок #2445

bot-backend referenced this issue from a commit

2026-08-06 17:48:01 +00:00

fix(ptica): скраб ПДн перед отправкой в мониторинг + честная подпись НДС в отчётах (#2457)

lekss361 referenced this issue

2026-08-06 17:51:30 +00:00

fix(ptica): скраб ПДн перед отправкой в мониторинг + честная подпись НДС в отчётах (#2457) #2749

lekss361 referenced this issue from a commit

2026-08-06 18:47:38 +00:00

fix(ptica): скраб ПДн перед отправкой в мониторинг + честная подпись НДС в отчётах (#2457) (#2749)

lekss361 referenced this issue

2026-08-06 18:49:54 +00:00

ptica/observability: страховка скраба ПДн может утечь то, что защищает + проводка проверяется грепом по исходнику #2753

Light1YT referenced this issue from a commit

2026-08-07 09:45:33 +00:00

fix(ptica/concept): подписи НДС во фронте перестают отрицать коммерцию (#2753)

lekss361 added the

bug

scope/backend

site-finder

labels 2026-08-16 10:25:04 +00:00

lekss361 commented

2026-08-16 10:36:36 +00:00

Owner

Закрываю по итогам разбора трекера 16.08.2026

Вердикт: сделано кодом.

Ровно та правка, которую просила задача, есть в forgejo/main в обоих файлах-экспортёрах; старая метка «НДС (паркинг)» grep'ом в exporters не находится.

Доказательство: backend/app/services/exporters/full_report_html.py:1341 и full_report_docx.py:872 — обе строки уже 'НДС (паркинг + коммерция)'

Независимая проверка. Вердикт отдельно проверялся вторым проходом, задачей которого было именно опровергнуть закрытие, а не подтвердить его:

Проверил своими глазами в refs/remotes/forgejo/main: full_report_html.py:1341 и full_report_docx.py:872 — обе строки «НДС (паркинг + коммерция)»; старая метка «НДС (паркинг)» в backend/app/services/exporters отсутствует. Правка пришла коммитом ac870b0c (2026-08-06, PR #2749) с явной ссылкой на #2457, диффом именно этих двух строк. Второй пункт scope («проверить, нет ли рядом такого же stale-caveat про только машиноместа») закрыт проверкой: в обоих full_report-экспортёрах слова «машиномест» нет вообще, единственные близкие строки — «Выручка — паркинг» и «Выручка — нежилое (1-й этаж)», они корректны; противоречивый caveat про НДС живёт только в generative/exporters/pdf.py:188, где он уже честный («машиноместа и коммерцию/офисы 1-го этажа», починен PR #2455). Изменение в backend/ — деплоится штатным лэйном, коммит на main 10 дней. Опровергнуть не смог: задача закрываема.

Если что-то из перечисленного всё же живо — переоткройте задачу, разбор мог упустить частный случай.

## Закрываю по итогам разбора трекера 16.08.2026 **Вердикт:** сделано кодом. Ровно та правка, которую просила задача, есть в forgejo/main в обоих файлах-экспортёрах; старая метка «НДС (паркинг)» grep'ом в exporters не находится. **Доказательство:** backend/app/services/exporters/full_report_html.py:1341 и full_report_docx.py:872 — обе строки уже 'НДС (паркинг + коммерция)' **Независимая проверка.** Вердикт отдельно проверялся вторым проходом, задачей которого было именно опровергнуть закрытие, а не подтвердить его: > Проверил своими глазами в refs/remotes/forgejo/main: full_report_html.py:1341 и full_report_docx.py:872 — обе строки «НДС (паркинг + коммерция)»; старая метка «НДС (паркинг)» в backend/app/services/exporters отсутствует. Правка пришла коммитом ac870b0c (2026-08-06, PR #2749) с явной ссылкой на #2457, диффом именно этих двух строк. Второй пункт scope («проверить, нет ли рядом такого же stale-caveat про только машиноместа») закрыт проверкой: в обоих full_report-экспортёрах слова «машиномест» нет вообще, единственные близкие строки — «Выручка — паркинг» и «Выручка — нежилое (1-й этаж)», они корректны; противоречивый caveat про НДС живёт только в generative/exporters/pdf.py:188, где он уже честный («машиноместа и коммерцию/офисы 1-го этажа», починен PR #2455). Изменение в backend/ — деплоится штатным лэйном, коммит на main 10 дней. Опровергнуть не смог: задача закрываема. Если что-то из перечисленного всё же живо — переоткройте задачу, разбор мог упустить частный случай.

lekss361 closed this issue

2026-08-16 10:36:36 +00:00