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

Follow-ups from #96 #98

New Issue
Open
opened 2026-10-10 09:02:47 +03:00 by aep · 1 comment
aep commented 2026-10-10 09:02:47 +03:00
Owner
Copy Link

Отложенные находки из PR #97 (файлы в контексте консолидации, DBML v5.18). Маркеры — из финального отчёта SDD-контроллера.

  • I-1 (важно, до перевыкладки контура): arm-core/web/src/services/api/files.ts:19 всё ещё шлёт ?consolidation_version_id=. Ядро параметр игнорирует, ошибки нет, но загрузка из встроенного фронта перестала переводить версию INITIATED → DOCUMENTS_PROCESSING. Синхронизировать arm-core/web с arm-web (PR arm-web #27: filesPath, маршруты консолидации, селектор, панель).
  • M-3 — порядок проверок 404 (консолидация → ЮЛ) у GET и POST /consolidations/{cid}/legal-entities/{le}/files различается; утечки нет, но стоит выровнять.
  • M-4 — OpenAPI 422 у загрузки описан как UploadResult, а для ENTITY_NOT_IN_PERIMETER тело — ErrorDetail; описать оба.
  • M-5 — основные спеки openspec/specs/data-model и directories-api упоминают «v5.17» как текущую; править при openspec archive console-files.
  • T3.2 — check_consolidation_for_upload читает members() в цикле по версиям; заменить одним EXISTS.
  • T3.3 — нет теста: две версии «в работе», ЮЛ ни в одной → 422.
  • T4.2 — нет теста автозамены: у обоих заменяемых файлов непустой consolidation_id (берётся первый по id).
  • T5.1 — CLAUDE.md, длинная строка про v5.18 внутри пункта «Первичные документы».

Контекст: RSHB/arm-core#96, PR #97; docs RSHB/arm-ka-docs#90; arm-web RSHB/arm-web#26.

Отложенные находки из PR #97 (файлы в контексте консолидации, DBML v5.18). Маркеры — из финального отчёта SDD-контроллера. - **`I-1` (важно, до перевыкладки контура):** `arm-core/web/src/services/api/files.ts:19` всё ещё шлёт `?consolidation_version_id=`. Ядро параметр игнорирует, ошибки нет, но загрузка из встроенного фронта перестала переводить версию `INITIATED → DOCUMENTS_PROCESSING`. Синхронизировать `arm-core/web` с arm-web (PR arm-web #27: `filesPath`, маршруты консолидации, селектор, панель). - `M-3` — порядок проверок 404 (консолидация → ЮЛ) у `GET` и `POST /consolidations/{cid}/legal-entities/{le}/files` различается; утечки нет, но стоит выровнять. - `M-4` — OpenAPI 422 у загрузки описан как `UploadResult`, а для `ENTITY_NOT_IN_PERIMETER` тело — `ErrorDetail`; описать оба. - `M-5` — основные спеки `openspec/specs/data-model` и `directories-api` упоминают «v5.17» как текущую; править при `openspec archive console-files`. - `T3.2` — `check_consolidation_for_upload` читает `members()` в цикле по версиям; заменить одним `EXISTS`. - `T3.3` — нет теста: две версии «в работе», ЮЛ ни в одной → 422. - `T4.2` — нет теста автозамены: у обоих заменяемых файлов непустой `consolidation_id` (берётся первый по `id`). - `T5.1` — `CLAUDE.md`, длинная строка про v5.18 внутри пункта «Первичные документы». Контекст: RSHB/arm-core#96, PR #97; docs RSHB/arm-ka-docs#90; arm-web RSHB/arm-web#26.
aep commented 2026-10-10 09:40:00 +03:00
Author
Owner
Copy Link

I-1 закрыт PR #101 (коммит 2ea8a96, merge 3414317): встроенный фронт переведён на /consolidations/{cid}/legal-entities/{le}/files, ?consolidation_version_id= больше не шлётся. Остальные пункты (M-3, M-4, M-5, T3.2, T3.3, T4.2, T5.1) открыты.

I-1 закрыт PR #101 (коммит 2ea8a96, merge 3414317): встроенный фронт переведён на /consolidations/{cid}/legal-entities/{le}/files, ?consolidation_version_id= больше не шлётся. Остальные пункты (M-3, M-4, M-5, T3.2, T3.3, T4.2, T5.1) открыты.
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#98
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?