RSHB/arm-core
9
0
Fork 0
Code Issues 24 Pull Requests 4 Actions Packages Projects Releases Wiki Activity

Follow-ups from #36 #40

New Issue
Open
opened 2026-10-07 14:23:13 +03:00 by aep · 1 comment
aep commented 2026-10-07 14:23:13 +03:00
Owner
Copy Link

Отложенные находки Среза 3b (PR #39, change slice-3b-statements-api, решения №1–25 в design.md). Маркер — одна строка — откуда.

Поведение и данные

  • R-2 — пересчёт статуса формы после правки не учитывает предупреждения маппинга (хранить их на документе — нужен DBML v5.17); документ с предупреждением может стать RECOGNIZED после правки — рулинг контроллера, design №12.
  • N-2 — _clear_file не удаляет комментарии документов при повторном маппинге файла (комментарии повисают на удалённых document) — финальное ревью.
  • R-9 (риск, принят) — CSV чек-листа не обезвреживает формулы Excel: название ЮЛ, начинающееся с =, +, -, @, откроется как формула — рулинг контроллера, design №19.
  • M-4 — правка ИНН формы грузит CounterpartyIndex целиком ради имён владельца — финальное ревью.

Код и тесты

  • T1.2 — rule not in sublines — линейный скан в statement_checks.run_checks — ревью Task 1.
  • T1.3 — docstring'и statement_checks.py обёрнуты посреди фразы — ревью Task 1.
  • T3.2 — сортировка документов по enum в map_golden_snapshot даёт порядок Postgres, а не алфавитный (тесты учитывают множествами) — ревью Task 3.
  • T6.4 (остаток) — ключи аудита правки формы value:<uuid> без кода строки и колонки — ревью Task 6.
  • T7.4 — DocumentService.confirm строит StatementService ради hard_failures (связность сервисов) — ревью Task 7.
  • PB.3 — освобождение S3-потока зависит от GC, если ответ не был вызван — ревью пакета B.
  • PB.4 — aclose потока под shield без таймаута — ревью пакета B.

Открытые вопросы владельцу (в docs/context/07-open-questions.md): OQ-153 (журналировать ли скачивание первички и CSV), OQ-154 (автозамена файла с формами на уровне файла).

Отложенные находки Среза 3b (PR #39, change `slice-3b-statements-api`, решения №1–25 в `design.md`). Маркер — одна строка — откуда. ## Поведение и данные - **R-2** — пересчёт статуса формы после правки не учитывает предупреждения маппинга (хранить их на документе — нужен DBML v5.17); документ с предупреждением может стать `RECOGNIZED` после правки — рулинг контроллера, design №12. - **N-2** — `_clear_file` не удаляет комментарии документов при повторном маппинге файла (комментарии повисают на удалённых `document`) — финальное ревью. - **R-9** (риск, принят) — CSV чек-листа не обезвреживает формулы Excel: название ЮЛ, начинающееся с `=`, `+`, `-`, `@`, откроется как формула — рулинг контроллера, design №19. - **M-4** — правка ИНН формы грузит `CounterpartyIndex` целиком ради имён владельца — финальное ревью. ## Код и тесты - **T1.2** — `rule not in sublines` — линейный скан в `statement_checks.run_checks` — ревью Task 1. - **T1.3** — docstring'и `statement_checks.py` обёрнуты посреди фразы — ревью Task 1. - **T3.2** — сортировка документов по enum в `map_golden_snapshot` даёт порядок Postgres, а не алфавитный (тесты учитывают множествами) — ревью Task 3. - **T6.4** (остаток) — ключи аудита правки формы `value:<uuid>` без кода строки и колонки — ревью Task 6. - **T7.4** — `DocumentService.confirm` строит `StatementService` ради `hard_failures` (связность сервисов) — ревью Task 7. - **PB.3** — освобождение S3-потока зависит от GC, если ответ не был вызван — ревью пакета B. - **PB.4** — `aclose` потока под `shield` без таймаута — ревью пакета B. Открытые вопросы владельцу (в `docs/context/07-open-questions.md`): OQ-153 (журналировать ли скачивание первички и CSV), OQ-154 (автозамена файла с формами на уровне файла).
aep commented 2026-10-25 19:41:01 +03:00
Author
Owner
Copy Link

R-9 (CSV чек-листа без нейтрализации формул) вынес в AURORA-1817.

R-9 (CSV чек-листа без нейтрализации формул) вынес в AURORA-1817.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
PotapovaA Raykov-MS SalminMP Veronika Yashin1994 aep grog maxim tsygankoviva
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-core#40
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?