Follow-ups from #80 #83

New Issue

aep · 2026-10-09T11:57:31+03:00

aep commented

2026-10-09 11:57:31 +03:00

Отложенные находки по #80 (PR #81, архив #82; фронт RSHB/arm-web#20, docs RSHB/arm-ka-docs#82). Маркеры — из ledger SDD и финального ревью.

Вопросы владельцу

M-6 — данные справочника 66н: строка 2411 записана с префиксом «в т.ч.»; в ОФР 66н нет строки 2530 — нужна ли она? (финальное ревью, opus)
O-4 — после ввода суммы в шаблонную строку (1700) только в одной колонке по остальным колонкам появляется нарушение «1700 = 1300 + 1400 + 1500 — ожидалось 0,00»: строка есть, пустая ячейка = 0 (правило run_checks). Сигнал верный, но шумный — оставить или пропускать проверку для пустой ячейки созданной строки? (Task 8, стенд)
R-13 / M-2 — пустая созданная строка учитывается в has_lines: форма без сумм может стать RECOGNIZED после ручного создания и очистки строки (противоречит D-5). (финальное ревью)

Ядро

M-3 — аудит не пишет создание строки (только value:<id>). (финальное ревью)
M-4 — редакция определяется по дате документа до правки шапки в том же PATCH. (финальное ревью)
T1.1 — повтор id редакции в rules/statement_lines.yaml молча перезаписывается. (ревью Task 1)
T1.2 — LineCatalog.editions — изменяемый dict в frozen dataclass. (ревью Task 1)
T1.3 — null-имя строки превращается в "None" (нет проверки пустого имени). (ревью Task 1)
T2.2 — точное строковое сравнение кодов строк без нормализации. (ревью Task 2)
T3.1 — недостижимая ветка if target is None в _apply_cells. (ревью Task 3)
T3.2 — цепочка присваивания created[...] читается тяжело. (ревью Task 3)

Тесты

T2.4 — тест «GET не пишет» сверяет только число строк. (ревью Task 2)
T2.5 — тест правит reporting_date через общую сессию. (ревью Task 2)
T3.3 — тест Focus 1 не проверяет пустоту аудита. (ревью Task 3)
T3.4 — тест Focus 2 не проверяет аудит с двумя ключами. (ревью Task 3)
T3.5 — нет теста line_id чужой формы. (ревью Task 3)
T4.1 — нет явного теста «0 = заполнено» для созданной строки. (ревью Task 4)
T4.2 — тест Task 4 зависит от хелперов Task 2/3. (ревью Task 4)

Документация

T5.2 — абзац про форму в CLAUDE.md слишком длинный. (ревью Task 5)
T5.3 — слабые обоснования R-9/R-10 в design.md (архив). (ревью Task 5)
без маркера (ре-ревью fix wave) — README: у UNKNOWN_CELL код 422 в обратных кавычках, у соседних строк — без.

Не проверено на стенде (покрыто тестами)

O-5 — «Подтвердить данные» с шаблонными строками на стенде не нажималось (необратимо на демо-данных).
O-6 — редакция 66н (ББ 2024 Q3, aep_6) на стенде не проверялась — в БД стенда только формы 1 кв. 2025.

Связанные

Фронт: RSHB/arm-web — follow-up по arm-web#19.
Словарь scan-gateway (fsbu4, сведение копий справочника): RSHB/arm-scan-gateway — отдельная issue.
Замечено попутно: arm-core/web/src/types/api.ts расходится с arm-web по counterparty_entity_inn (к #80 не относится).

🤖 Generated with Claude Code

Отложенные находки по #80 (PR #81, архив #82; фронт RSHB/arm-web#20, docs RSHB/arm-ka-docs#82). Маркеры — из ledger SDD и финального ревью. ## Вопросы владельцу - [ ] **M-6** — данные справочника 66н: строка 2411 записана с префиксом «в т.ч.»; в ОФР 66н нет строки 2530 — нужна ли она? (финальное ревью, opus) - [ ] **O-4** — после ввода суммы в шаблонную строку (1700) только в одной колонке по остальным колонкам появляется нарушение «1700 = 1300 + 1400 + 1500 — ожидалось 0,00»: строка есть, пустая ячейка = 0 (правило `run_checks`). Сигнал верный, но шумный — оставить или пропускать проверку для пустой ячейки созданной строки? (Task 8, стенд) - [ ] **R-13 / M-2** — пустая созданная строка учитывается в `has_lines`: форма без сумм может стать RECOGNIZED после ручного создания и очистки строки (противоречит D-5). (финальное ревью) ## Ядро - [ ] **M-3** — аудит не пишет создание строки (только `value:<id>`). (финальное ревью) - [ ] **M-4** — редакция определяется по дате документа до правки шапки в том же PATCH. (финальное ревью) - [ ] **T1.1** — повтор `id` редакции в `rules/statement_lines.yaml` молча перезаписывается. (ревью Task 1) - [ ] **T1.2** — `LineCatalog.editions` — изменяемый dict в frozen dataclass. (ревью Task 1) - [ ] **T1.3** — null-имя строки превращается в `"None"` (нет проверки пустого имени). (ревью Task 1) - [ ] **T2.2** — точное строковое сравнение кодов строк без нормализации. (ревью Task 2) - [ ] **T3.1** — недостижимая ветка `if target is None` в `_apply_cells`. (ревью Task 3) - [ ] **T3.2** — цепочка присваивания `created[...]` читается тяжело. (ревью Task 3) ## Тесты - [ ] **T2.4** — тест «GET не пишет» сверяет только число строк. (ревью Task 2) - [ ] **T2.5** — тест правит `reporting_date` через общую сессию. (ревью Task 2) - [ ] **T3.3** — тест Focus 1 не проверяет пустоту аудита. (ревью Task 3) - [ ] **T3.4** — тест Focus 2 не проверяет аудит с двумя ключами. (ревью Task 3) - [ ] **T3.5** — нет теста `line_id` чужой формы. (ревью Task 3) - [ ] **T4.1** — нет явного теста «0 = заполнено» для созданной строки. (ревью Task 4) - [ ] **T4.2** — тест Task 4 зависит от хелперов Task 2/3. (ревью Task 4) ## Документация - [ ] **T5.2** — абзац про форму в `CLAUDE.md` слишком длинный. (ревью Task 5) - [ ] **T5.3** — слабые обоснования R-9/R-10 в design.md (архив). (ревью Task 5) - [ ] без маркера (ре-ревью fix wave) — README: у `UNKNOWN_CELL` код `422` в обратных кавычках, у соседних строк — без. ## Не проверено на стенде (покрыто тестами) - [ ] **O-5** — «Подтвердить данные» с шаблонными строками на стенде не нажималось (необратимо на демо-данных). - [ ] **O-6** — редакция 66н (ББ 2024 Q3, aep_6) на стенде не проверялась — в БД стенда только формы 1 кв. 2025. ## Связанные - Фронт: RSHB/arm-web — follow-up по arm-web#19. - Словарь scan-gateway (fsbu4, сведение копий справочника): RSHB/arm-scan-gateway — отдельная issue. - Замечено попутно: `arm-core/web/src/types/api.ts` расходится с arm-web по `counterparty_entity_inn` (к #80 не относится). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

aep commented

2026-10-09 11:57:38 +03:00

Связанные follow-up: фронт — RSHB/arm-web#21, словарь scan-gateway — RSHB/arm-scan-gateway#25.