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

fix(web): синхронизация встроенного фронта с arm-web@test (a3c0010) (#139) #140

Merged
aep merged 1 commits from feat/139-web-sync-a3c0010 into main 2026-10-19 13:56:18 +03:00
Conversation 1 Commits 1 Files Changed 22 +892 -75
aep commented 2026-10-19 13:54:41 +03:00
Owner
Copy Link

Закрывает #139.

Встроенный фронт arm-core/web отставал от RSHB/arm-web@test (a3c0010). Диапазон arm-web 8345f5e..a3c0010
перенесён скриптом docs/scripts/sync-web.sh без правок; после переноса web/ и arm-web идентичны по набору синхронизации.

Коммиты arm-web (3):

  • ff1e45c file id as option value for react rendering
  • 0242486 fix(#49): зелёный make check и исправления после ревью (AURORA-144)
  • 0fdbd80 feat: add comments frontend, checklist modal, files delete, select options in document review

Файлы (22):

  • web/src/index.tsx
  • web/src/layouts/DocumentPage.tsx
  • web/src/services/api/consolidations.ts
  • web/src/services/api/documents.ts
  • web/src/services/api/files.ts
  • web/src/types/consolidations/version_members.ts
  • web/src/widgets/ConsoleLegalEntities/ChecklistModal.tsx
  • web/src/widgets/ConsoleLegalEntities/ConsoleLegalEntities.tsx
  • web/src/widgets/ConsoleLegalEntities/checklist_utils.test.ts
  • web/src/widgets/ConsoleLegalEntities/checklist_utils.tsx
  • web/src/widgets/DocumentReview/CommentModal.tsx
  • web/src/widgets/DocumentReview/StatementReview.tsx
  • web/src/widgets/DocumentReview/reviewCounters.test.ts
  • web/src/widgets/DocumentReview/reviewCounters.ts
  • web/src/widgets/DocumentReview/reviewView.test.ts
  • web/src/widgets/DocumentReview/reviewView.ts
  • web/src/widgets/FilesDrawer/DeleteModal.tsx
  • web/src/widgets/FilesDrawer/FilesDrawer.tsx
  • web/src/widgets/FilesDrawer/FilesList.tsx
  • web/src/widgets/FilesDrawer/ReplaceDocModal.tsx
  • web/src/widgets/FilesDrawer/documentRows.test.ts
  • web/src/widgets/FilesDrawer/documentRows.ts

Проверки в worktree: Tests 331 passed (331); tsc --noEmit чисто; biome check src — 7 предупреждений (на main — 7); rsbuild build собирается.

Rulings I made

  • R-1 — служебные файлы arm-web (CLAUDE.md, .claude/, openspec/, docs/, scripts/, Makefile, ARCHITECTURE.md, .gitignore, lockfile) не переносятся: UI-спеки и демо-стенд живут в arm-web.
  • R-2 — bounded-задача без OpenSpec change: перенос готового, проревьюенного в arm-web кода.

Ревью перенесённого кода против бэкенда — за исполнителем, находки с маркерами в комментариях к PR.

🤖 Generated with Claude Code

Закрывает #139. Встроенный фронт `arm-core/web` отставал от `RSHB/arm-web@test` (a3c0010). Диапазон arm-web 8345f5e..a3c0010 перенесён скриптом `docs/scripts/sync-web.sh` без правок; после переноса `web/` и arm-web идентичны по набору синхронизации. Коммиты arm-web (3): - ff1e45c file id as option value for react rendering - 0242486 fix(#49): зелёный make check и исправления после ревью (AURORA-144) - 0fdbd80 feat: add comments frontend, checklist modal, files delete, select options in document review Файлы (22): - web/src/index.tsx - web/src/layouts/DocumentPage.tsx - web/src/services/api/consolidations.ts - web/src/services/api/documents.ts - web/src/services/api/files.ts - web/src/types/consolidations/version_members.ts - web/src/widgets/ConsoleLegalEntities/ChecklistModal.tsx - web/src/widgets/ConsoleLegalEntities/ConsoleLegalEntities.tsx - web/src/widgets/ConsoleLegalEntities/checklist_utils.test.ts - web/src/widgets/ConsoleLegalEntities/checklist_utils.tsx - web/src/widgets/DocumentReview/CommentModal.tsx - web/src/widgets/DocumentReview/StatementReview.tsx - web/src/widgets/DocumentReview/reviewCounters.test.ts - web/src/widgets/DocumentReview/reviewCounters.ts - web/src/widgets/DocumentReview/reviewView.test.ts - web/src/widgets/DocumentReview/reviewView.ts - web/src/widgets/FilesDrawer/DeleteModal.tsx - web/src/widgets/FilesDrawer/FilesDrawer.tsx - web/src/widgets/FilesDrawer/FilesList.tsx - web/src/widgets/FilesDrawer/ReplaceDocModal.tsx - web/src/widgets/FilesDrawer/documentRows.test.ts - web/src/widgets/FilesDrawer/documentRows.ts Проверки в worktree: Tests 331 passed (331); `tsc --noEmit` чисто; `biome check src` — 7 предупреждений (на main — 7); `rsbuild build` собирается. ## Rulings I made - R-1 — служебные файлы arm-web (CLAUDE.md, .claude/, openspec/, docs/, scripts/, Makefile, ARCHITECTURE.md, .gitignore, lockfile) не переносятся: UI-спеки и демо-стенд живут в arm-web. - R-2 — bounded-задача без OpenSpec change: перенос готового, проревьюенного в arm-web кода. Ревью перенесённого кода против бэкенда — за исполнителем, находки с маркерами в комментариях к PR. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
aep added 1 commit 2026-10-19 13:54:42 +03:00 
Перенесён диапазон arm-web 8345f5e..a3c0010 (3 коммитов, 22 файлов набора
синхронизации) скриптом docs/scripts/sync-web.sh. После переноса web/ и
arm-web@test идентичны по набору. Коммиты arm-web:
- ff1e45c file id as option value for react rendering
- 0242486 fix(#49): зелёный make check и исправления после ревью (AURORA-144)
- 0fdbd80 feat: add comments frontend, checklist modal, files delete, select options in document review

Co-Authored-By: sync-web.sh <noreply@aepif.ru>
aep commented 2026-10-19 13:55:32 +03:00
Author
Owner
Copy Link

Ревью перенесённого кода против бэкенда (arm-web 8345f5e..a3c0010, PR arm-web #49):

  • GET /consolidations/versions/{id}/checklist — есть; тип ChecklistBackendShape (rules: document_type, account, is_mandatory; rows: legal_entity_id/name/inn, cells) совпадает с ChecklistOut.
  • POST /documents/{id}/comments {text} — есть, CommentIn (1–4000 символов после обрезки).
  • DELETE /files/{file_id} — есть; DeleteModal передаёт id файла (targetRow.fileId), не документа.
  • Замена документа — штатная загрузка в ту же консолидацию (uploadFiles, автозамена ядра §9.5 п. 3) + причина комментарием к заменяемому документу; новых эндпоинтов нет.

Проверки скрипта: vitest 331 passed, tsc чисто, biome 7 предупреждений (как на main), build ок. Замечаний нет.

Ревью перенесённого кода против бэкенда (arm-web 8345f5e..a3c0010, PR arm-web #49): - `GET /consolidations/versions/{id}/checklist` — есть; тип `ChecklistBackendShape` (`rules`: document_type, account, is_mandatory; `rows`: legal_entity_id/name/inn, cells) совпадает с `ChecklistOut`. - `POST /documents/{id}/comments {text}` — есть, `CommentIn` (1–4000 символов после обрезки). - `DELETE /files/{file_id}` — есть; `DeleteModal` передаёт id файла (`targetRow.fileId`), не документа. - Замена документа — штатная загрузка в ту же консолидацию (`uploadFiles`, автозамена ядра §9.5 п. 3) + причина комментарием к заменяемому документу; новых эндпоинтов нет. Проверки скрипта: vitest 331 passed, tsc чисто, biome 7 предупреждений (как на main), build ок. Замечаний нет.
aep merged commit 5bec434e1f into main 2026-10-19 13:56:18 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
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#140
No description provided.
Delete Branch "feat/139-web-sync-a3c0010"

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?