Follow-ups from #42 #48

New Issue

aep · 2026-10-07T17:24:24+03:00

aep commented

2026-10-07 17:24:24 +03:00

Отложенное из PR #46 (arm-core) и RSHB/arm-ka-docs#51, финальное ревью ветки feat/42-front-api-gaps (opus, 07.10.2026).

M-1 — тест osv_accounts на документ с period_quarter = NULL и с пустым osv_account_code (решение №4 не покрыто). Откуда: финальное ревью.
M-2 — тест подхвата подтверждённых документов ЮЛ вне ГСК при явном legal_entity_ids; тест порядка «422 периметра раньше 409 периода» (решение №5). Откуда: финальное ревью.
M-3 — OpenAPI-тест maxItems: 500 и nullable у ConsolidationCreate.legal_entity_ids. Откуда: финальное ревью.
M-4 (docs) — context/10-user-journey.md шаг 2.1, колонка «БД / статус»: «consolidation_member из отмеченных ЮЛ (без списка — из состава ГСК)». Откуда: финальное ревью.
M-5 (docs) — dev/api-contract-files-directories.md: вычисляемые поля outside_group/closed у LegalEntityOut. Откуда: финальное ревью.
M-6 (docs) — FILE_TOO_LARGE: «по умолчанию 50 МиБ» (настройка UPLOAD_MAX_BYTES). Откуда: финальное ревью.
M-7 (docs) — модалка [42]: список ЮЛ ГК дочитывать по offset (limit ≤ 200, а периметр до 500). Откуда: финальное ревью.
M-8 — design D-3, сигнатура osv_accounts(quarters) — исправлено при архивации (3b4f329), пункт закрыт.
M-9 — нумерация tasks.md (п. 4/5) не совпадала с планом; архив уже в main, исправлять не нужно — только учесть в следующих планах.
O-1 (docs) — make check в arm-ka-docs падает на validate-openspec: Makefile ссылается на заархивированный slice-0-skeleton (и на main). Откуда: исполнение Task 4.
O-2 — PATCH /directories/legal-entities/{id} с {"entity_type": null} → validate_inn(inn, None) → KeyError в INN_LENGTH[None] → 500 (services/legal_entities.py:332, domain/inn.py:28); нужно 422 или запрет null в LegalEntityUpdate. Существовало до ветки. Откуда: финальное ревью.

Отложенное из PR #46 (arm-core) и RSHB/arm-ka-docs#51, финальное ревью ветки feat/42-front-api-gaps (opus, 07.10.2026). - **M-1** — тест `osv_accounts` на документ с `period_quarter = NULL` и с пустым `osv_account_code` (решение №4 не покрыто). Откуда: финальное ревью. - **M-2** — тест подхвата подтверждённых документов ЮЛ вне ГСК при явном `legal_entity_ids`; тест порядка «422 периметра раньше 409 периода» (решение №5). Откуда: финальное ревью. - **M-3** — OpenAPI-тест `maxItems: 500` и nullable у `ConsolidationCreate.legal_entity_ids`. Откуда: финальное ревью. - **M-4** (docs) — `context/10-user-journey.md` шаг 2.1, колонка «БД / статус»: «consolidation_member из отмеченных ЮЛ (без списка — из состава ГСК)». Откуда: финальное ревью. - **M-5** (docs) — `dev/api-contract-files-directories.md`: вычисляемые поля `outside_group`/`closed` у `LegalEntityOut`. Откуда: финальное ревью. - **M-6** (docs) — `FILE_TOO_LARGE`: «по умолчанию 50 МиБ» (настройка `UPLOAD_MAX_BYTES`). Откуда: финальное ревью. - **M-7** (docs) — модалка [42]: список ЮЛ ГК дочитывать по `offset` (`limit ≤ 200`, а периметр до 500). Откуда: финальное ревью. - **M-8** — design D-3, сигнатура `osv_accounts(quarters)` — **исправлено при архивации (3b4f329)**, пункт закрыт. - **M-9** — нумерация `tasks.md` (п. 4/5) не совпадала с планом; архив уже в main, исправлять не нужно — только учесть в следующих планах. - **O-1** (docs) — `make check` в arm-ka-docs падает на `validate-openspec`: Makefile ссылается на заархивированный `slice-0-skeleton` (и на main). Откуда: исполнение Task 4. - **O-2** — `PATCH /directories/legal-entities/{id}` с `{"entity_type": null}` → `validate_inn(inn, None)` → `KeyError` в `INN_LENGTH[None]` → 500 (`services/legal_entities.py:332`, `domain/inn.py:28`); нужно 422 или запрет `null` в `LegalEntityUpdate`. Существовало до ветки. Откуда: финальное ревью.

aep commented

2026-10-07 18:12:09 +03:00

Из arm-ka-docs PR #57 (слит, c20fe57):

M-5, M-6, M-7 (docs) — закрыты: dev/api-contract-files-directories.md — outside_group/closed у LegalEntityOut, FILE_TOO_LARGE по настройке UPLOAD_MAX_BYTES (50 МиБ по умолчанию), дочитывание ЮЛ модалки [42] по offset.
O-2 — не воспроизводится на main: PATCH /directories/legal-entities/{id} с {"entity_type": null} отвечает 422 массивом (LegalEntityUpdate._required_fields_are_not_null, schemas/directories.py:183) раньше validate_inn; KeyError в сервисе недостижим через API. Предлагаю пункт закрыть как rejected (или оставить как защиту в сервисе — по желанию).
Остальные пункты (M-1…M-4, M-9, O-1) — открыты.

Из arm-ka-docs PR #57 (слит, c20fe57): - **M-5, M-6, M-7** (docs) — закрыты: dev/api-contract-files-directories.md — outside_group/closed у LegalEntityOut, FILE_TOO_LARGE по настройке UPLOAD_MAX_BYTES (50 МиБ по умолчанию), дочитывание ЮЛ модалки [42] по offset. - **O-2** — не воспроизводится на main: PATCH /directories/legal-entities/{id} с {"entity_type": null} отвечает 422 массивом (LegalEntityUpdate._required_fields_are_not_null, schemas/directories.py:183) раньше validate_inn; KeyError в сервисе недостижим через API. Предлагаю пункт закрыть как rejected (или оставить как защиту в сервисе — по желанию). Остальные пункты (M-1…M-4, M-9, O-1) — открыты.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: RSHB/arm-core#48