diff --git a/CLAUDE.md b/CLAUDE.md index 54f0321..874225d 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -41,7 +41,7 @@ SPA кредитного аналитика АРМ КА: справочники - `src/utils/` — чистые функции без React/store/API: деньги (`money.ts`, суммы — строки), подписи статусов/типов/периодов (`labels.ts`), скачивание и имя файла (`download.ts`). - `src/auth/` — `AuthGate` (профиль `/me`), `DevLogin` (`POST /dev/token`; форма показывается в любой сборке, заглушка «вход через РАИСА» — после 404 «Not Found» от бэкенда, то есть когда dev-эмитент выключен), `token.ts`. - `src/store/` — Redux Toolkit: `store.ts`, слайсы `companyGroupsSlice` и `consolidations` (+ `thunks.ts`). -- `src/layouts/` — страницы (маршруты в `src/App.tsx`, в т.ч. `/documents/:document_id` и `/groups/:group_id/versions/:version_id/osv`), `src/widgets/` — блоки экранов (`DocumentReview` — проверка ББ/ОФР/ОСВ со сканом, `ConsolidatedOsv` — сводная ОСВ, `FilesDrawer` — файлы ЮЛ), `src/components/` — примитивы +- `src/layouts/` — страницы (маршруты в `src/App.tsx`, в т.ч. `/documents/:document_id` и `/groups/:group_id/versions/:version_id/osv`), `src/widgets/` — блоки экранов (`DocumentReview` — проверка ББ/ОФР/ОСВ со сканом, `ConsolidatedOsv` — сводная ОСВ, `FilesDrawer` — файлы ЮЛ (стол консолидации / все файлы ЮЛ, подпись консолидации, селектор `store/slices/consolidations/selectors.ts`)), `src/components/` — примитивы (`GenericTable`, `TableTag`, `PageTitle`), `src/types/` — DTO API. ## Команды diff --git a/docs/superpowers/plans/2026-10-09-console-files-web.md b/docs/superpowers/plans/2026-10-09-console-files-web.md new file mode 100644 index 0000000..8dba6b8 --- /dev/null +++ b/docs/superpowers/plans/2026-10-09-console-files-web.md @@ -0,0 +1,552 @@ +# Файлы в контексте консолидации — план для arm-web + +> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking. + +**Goal:** Панель «Первичные документы» [39] в версии показывает рабочий стол консолидации, загружает файлы в +консолидацию, умеет переключаться на «Все документы организации» с подписью консолидации у файла и не даёт загружать +в версию не «в работе». + +**Architecture:** Чистые функции (селектор консолидации по версии, подпись консолидации) с vitest; сервис файлов +переключает маршруты по наличию `consolidationId`; хук списка принимает `consolidationId`; панель держит +`showAll`. Компоненты не тестируются автоматически — ручной прогон на стенде в Playwright со сверкой с макетом [39]/[40]. + +**Tech Stack:** React 18 + TypeScript strict, antd 6, Redux Toolkit, vitest, biome, rsbuild (`make check`). + +**Spec:** `openspec/changes/console-files/` (`proposal.md`, `design.md` D-1…D-6, `specs/primary-files-ui/spec.md`); +контракт ядра — `../docs/dev/frontend-api-guide.md` (запись 09.10, docs #90) и `arm-core/openspec/changes/console-files/`. + +## Global Constraints + +- Worktree `/Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web`, ветка `feat/26-console-files` от `test`; + PR — в `test`. Git — `git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web …`. +- Все запросы — через `src/api/client.ts` (`apiFetch`); поля типов — только из схем arm-core (`FileOut.consolidation_id: string | null`). +- biome: ширина 140, одинарные кавычки, табы (как в репозитории); `tsc --noEmit` без ошибок; `make check` зелёный перед PR. +- Маршруты: `GET/POST /consolidations/{cid}/legal-entities/{le}/files`; без консолидации — `/legal-entities/{le}/files`; + query `consolidation_version_id` не отправлять. +- Статусы «в работе»: `INITIATED`, `DOCUMENTS_PROCESSING`, `DATA_CORRECTION`. +- Подписи: «Все документы организации», «вне консолидации», «другая консолидация», «<год> К» (`quarterShortLabel`), + подсказка «Загрузка доступна только в версии в работе». +- Пользователь слаб во фронте: в PR объяснить решения простым языком; код-ревью TS/React — сам исполнитель. +- Коммит заканчивается строкой `Co-Authored-By: Claude Fable 5.1 `. + +## Review Focus + +1. Переход по прямой ссылке `?version=` до загрузки списка консолидаций: селектор вернёт `null`, панель не должна + падать и должна работать в режиме «все» без переключателя. Тест `findCurrentConsolidation` на пустом списке в Task 1. +2. Файл текущей консоли в режиме «все» не должен получать подпись, файл другой консоли — должен. Тест + `consolidationLabel` в Task 1. +3. Переключение режима при идущем опросе: старый ответ не должен затирать новый список. Хук уже защищён `requestRef`; + `consolidationId` добавляется в зависимости эффекта сброса — проверяется ручным сценарием Task 4 (переключить во + время «Обработка»). +4. Версия `CANCELLED`: кнопка загрузки недоступна, стол при этом показывается. Ручной сценарий Task 4. +5. `uploadFiles` с консолидацией при 422 `UploadResult` по-прежнему возвращает результат, а не бросает. Тест в Task 2. + +--- + +### Task 1: Селектор консолидации и подпись консолидации (чистые функции) + +**Files:** +- Create: `src/store/slices/consolidations/selectors.ts`, `src/store/slices/consolidations/selectors.test.ts`, + `src/widgets/FilesDrawer/consolidationLabel.ts`, `src/widgets/FilesDrawer/consolidationLabel.test.ts` +- Modify: `src/types/consolidations/consolidations.ts` (экспорт `IConsolidationVersion`), `src/types/api.ts` (`FileOut.consolidation_id`) + +**Interfaces:** +- Produces: `findCurrentConsolidation(consolidations: IConsolidation[], versionId: string | null): CurrentConsolidation | null` + с `CurrentConsolidation = { consolidation_id: string; year: number; quarter: QuarterEnum; version_status: ConsStatus }`; + `IN_WORK_STATUSES: ReadonlySet`; `isInWork(status: ConsStatus): boolean`; + `useCurrentConsolidation(): CurrentConsolidation | null`; + `consolidationLabel(fileConsolidationId: string | null, consolidations: IConsolidation[], currentConsolidationId: string | null): string | null`. + +- [ ] **Step 1: Тесты (красные)** + +`src/store/slices/consolidations/selectors.test.ts`: + +```ts +import { describe, expect, it } from 'vitest'; +import type { IConsolidation } from '../../../types/consolidations/consolidations'; +import { findCurrentConsolidation, isInWork } from './selectors'; + +const cons: IConsolidation[] = [ + { + consolidation_id: 'c-2024q3', + year: 2024, + quarter: 'Q3', + reporting_date: '2024-10-01', + versions: [ + { id: 'v1', version_no: 1, status: 'VERIFIED', scenario: 'CREATE', approach: 'BASIC', lock: null, created_at: null, members_count: 2 }, + { id: 'v2', version_no: 2, status: 'INITIATED', scenario: 'CREATE', approach: 'BASIC', lock: null, created_at: null, members_count: 2 }, + ], + }, + { + consolidation_id: 'c-2024q4', + year: 2024, + quarter: 'Q4', + reporting_date: '2025-01-01', + versions: [{ id: 'v3', version_no: 1, status: 'CANCELLED', scenario: 'CREATE', approach: 'BASIC', lock: null, created_at: null, members_count: 2 }], + }, +]; + +describe('findCurrentConsolidation', () => { + it('находит консолидацию и статус версии по id версии', () => { + expect(findCurrentConsolidation(cons, 'v2')).toEqual({ consolidation_id: 'c-2024q3', year: 2024, quarter: 'Q3', version_status: 'INITIATED' }); + expect(findCurrentConsolidation(cons, 'v3')).toEqual({ consolidation_id: 'c-2024q4', year: 2024, quarter: 'Q4', version_status: 'CANCELLED' }); + }); + it('null без версии, при неизвестной версии и при пустом списке (прямая ссылка до загрузки)', () => { + expect(findCurrentConsolidation(cons, null)).toBeNull(); + expect(findCurrentConsolidation(cons, 'nope')).toBeNull(); + expect(findCurrentConsolidation([], 'v2')).toBeNull(); + }); +}); + +describe('isInWork', () => { + it('в работе — три статуса', () => { + expect(isInWork('INITIATED')).toBe(true); + expect(isInWork('DOCUMENTS_PROCESSING')).toBe(true); + expect(isInWork('DATA_CORRECTION')).toBe(true); + expect(isInWork('VERIFIED')).toBe(false); + expect(isInWork('CANCELLED')).toBe(false); + }); +}); +``` + +`src/widgets/FilesDrawer/consolidationLabel.test.ts`: + +```ts +import { describe, expect, it } from 'vitest'; +import type { IConsolidation } from '../../types/consolidations/consolidations'; +import { consolidationLabel } from './consolidationLabel'; + +const cons: IConsolidation[] = [ + { consolidation_id: 'c-2024q3', year: 2024, quarter: 'Q3', reporting_date: '2024-10-01', versions: [] }, + { consolidation_id: 'c-2025q1', year: 2025, quarter: 'Q1', reporting_date: '2025-04-01', versions: [] }, +]; + +describe('consolidationLabel', () => { + it('файл текущей консоли — без подписи', () => { + expect(consolidationLabel('c-2024q3', cons, 'c-2024q3')).toBeNull(); + }); + it('другая консоль из списка — год и квартал', () => { + expect(consolidationLabel('c-2025q1', cons, 'c-2024q3')).toBe('2025 К1'); + }); + it('без контекста и неизвестный id', () => { + expect(consolidationLabel(null, cons, 'c-2024q3')).toBe('вне консолидации'); + expect(consolidationLabel('ghost', cons, 'c-2024q3')).toBe('другая консолидация'); + }); + it('без текущей консоли подпись есть у всех, кроме null-текущей', () => { + expect(consolidationLabel('c-2024q3', cons, null)).toBe('2024 К3'); + expect(consolidationLabel(null, cons, null)).toBe('вне консолидации'); + }); +}); +``` + +Run: `npx vitest run src/store/slices/consolidations/selectors.test.ts src/widgets/FilesDrawer/consolidationLabel.test.ts 2>&1 | tail -8` +Expected: FAIL — модули не найдены. + +- [ ] **Step 2: Типы** + +`src/types/consolidations/consolidations.ts`: `interface IConsolidationVersion` → `export interface IConsolidationVersion`. +`src/types/api.ts`, `FileOut`: после `uploaded_at` добавить +`/** консолидация, в контексте которой загружен файл (v5.18); null — без контекста */ consolidation_id: string | null;`. +Во всех фабриках `FileOut` в тестах (`src/services/api/files.test.ts`, `src/widgets/FilesDrawer/documentRows.test.ts`) +добавить `consolidation_id: null`. + +- [ ] **Step 3: Реализация** + +`src/store/slices/consolidations/selectors.ts`: + +```ts +import type { ConsStatus, QuarterEnum } from '../../../types/consolidations/common_types'; +import type { IConsolidation } from '../../../types/consolidations/consolidations'; +import { useAppSelector } from '../../store'; + +/** Консолидация открытой версии и статус этой версии (design D-1): выводится из списка ГК, в сторе не хранится. */ +export interface CurrentConsolidation { + consolidation_id: string; + year: number; + quarter: QuarterEnum; + version_status: ConsStatus; +} + +/** Статусы «в работе» (§9.1): только в них разрешена загрузка файлов в консолидацию. */ +export const IN_WORK_STATUSES: ReadonlySet = new Set(['INITIATED', 'DOCUMENTS_PROCESSING', 'DATA_CORRECTION']); + +export const isInWork = (status: ConsStatus): boolean => IN_WORK_STATUSES.has(status); + +export function findCurrentConsolidation(consolidations: IConsolidation[], versionId: string | null): CurrentConsolidation | null { + if (!versionId) return null; + for (const c of consolidations) { + const version = c.versions.find((v) => v.id === versionId); + if (version) return { consolidation_id: c.consolidation_id, year: c.year, quarter: c.quarter, version_status: version.status }; + } + return null; +} + +export function useCurrentConsolidation(): CurrentConsolidation | null { + const consolidations = useAppSelector((s) => s.consolidations.consolidations); + const versionId = useAppSelector((s) => s.consolidations.current_version_id); + return findCurrentConsolidation(consolidations, versionId); +} +``` + +`src/widgets/FilesDrawer/consolidationLabel.ts`: + +```ts +import type { IConsolidation } from '../../types/consolidations/consolidations'; +import { quarterShortLabel } from '../../utils/labels'; + +/** Подпись консолидации у файла в режиме «Все документы организации» (design D-4): null — файл текущей консоли. */ +export function consolidationLabel( + fileConsolidationId: string | null, + consolidations: IConsolidation[], + currentConsolidationId: string | null, +): string | null { + if (fileConsolidationId === null) return 'вне консолидации'; + if (fileConsolidationId === currentConsolidationId) return null; + const found = consolidations.find((c) => c.consolidation_id === fileConsolidationId); + return found ? `${found.year} ${quarterShortLabel(found.quarter)}` : 'другая консолидация'; +} +``` + +Если `quarterShortLabel` принимает тип `Quarter` из `types/api`, а не `QuarterEnum`, — оба это `'Q1' | … | 'Q4'`, +передавать как есть; при ошибке типов привести `found.quarter as Quarter` с импортом типа. + +Run: `npx vitest run src/store/slices/consolidations/selectors.test.ts src/widgets/FilesDrawer/consolidationLabel.test.ts 2>&1 | tail -6 && npx tsc --noEmit && npx biome check src 2>&1 | tail -3` +Expected: PASS 7 тестов; tsc и biome чистые. + +- [ ] **Step 4: Коммит** + +```bash +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web add src/store/slices/consolidations/selectors.ts src/store/slices/consolidations/selectors.test.ts src/widgets/FilesDrawer/consolidationLabel.ts src/widgets/FilesDrawer/consolidationLabel.test.ts src/types src/services/api/files.test.ts src/widgets/FilesDrawer/documentRows.test.ts +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web commit -q -m "$(cat <<'EOF' +store: консолидация открытой версии и подпись консолидации у файла (#26) + +Co-Authored-By: Claude Fable 5.1 +EOF +)" +``` + +--- + +### Task 2: Сервис файлов — маршруты консолидации + +**Files:** +- Modify: `src/services/api/files.ts`, `src/services/api/files.test.ts` + +**Interfaces:** +- Produces: `uploadFiles(legalEntityId: string, files: File[], consolidationId: string | null): Promise`; + `listLegalEntityFiles(legalEntityId: string, consolidationId: string | null): Promise`; + `filesPath(legalEntityId: string, consolidationId: string | null): string`. + +- [ ] **Step 1: Тесты (красные)** + +В `src/services/api/files.test.ts` блок `describe('uploadFiles')` переписать: + +```ts +describe('filesPath', () => { + it('с консолидацией — маршрут консолидации, без неё — маршрут ЮЛ', () => { + expect(filesPath('le', 'c1')).toBe('/consolidations/c1/legal-entities/le/files'); + expect(filesPath('le', null)).toBe('/legal-entities/le/files'); + }); +}); + +describe('uploadFiles', () => { + afterEach(() => vi.restoreAllMocks()); + + it('422 с UploadResult возвращается как результат (и с консолидацией)', async () => { + const client = await import('../../api/client'); + vi.spyOn(client, 'apiFetch').mockRejectedValue(new ApiError(422, undefined, 'x', { accepted: [], rejected: result.rejected })); + await expect(uploadFiles('le', [new File(['a'], 'dup.xlsx')], 'c1')).resolves.toEqual({ accepted: [], rejected: result.rejected }); + }); + + it('422 с detail и 409 пробрасываются', async () => { + const client = await import('../../api/client'); + const error = new ApiError(409, 'CONSOLIDATION_NOT_EDITABLE', 'нет версии в работе', { detail: { code: 'CONSOLIDATION_NOT_EDITABLE' } }); + vi.spyOn(client, 'apiFetch').mockRejectedValue(error); + await expect(uploadFiles('le', [new File(['a'], 'a.xlsx')], 'c1')).rejects.toBe(error); + }); + + it('консолидация — в пути, query нет, лишних полей формы нет', async () => { + const client = await import('../../api/client'); + const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue(result); + await uploadFiles('le', [new File(['a'], 'a.xlsx'), new File(['b'], 'b.pdf')], 'c1'); + const [path, init] = spy.mock.calls[0]; + expect(path).toBe('/consolidations/c1/legal-entities/le/files'); + const form = init?.body as FormData; + expect(form.getAll('files')).toHaveLength(2); + const keys: string[] = []; + form.forEach((_, key) => keys.push(key)); + expect(keys.every((k) => k === 'files')).toBe(true); + }); + + it('без консолидации — маршрут ЮЛ без query', async () => { + const client = await import('../../api/client'); + const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue(result); + await uploadFiles('le', [new File(['a'], 'a.xlsx')], null); + expect(spy.mock.calls[0][0]).toBe('/legal-entities/le/files'); + }); +}); + +describe('listLegalEntityFiles', () => { + afterEach(() => vi.restoreAllMocks()); + it('стол консолидации и список ЮЛ', async () => { + const client = await import('../../api/client'); + const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue({ items: [], total: 0, limit: 100, offset: 0 }); + await listLegalEntityFiles('le', 'c1'); + await listLegalEntityFiles('le', null); + expect(spy.mock.calls[0][0]).toBe('/consolidations/c1/legal-entities/le/files?limit=100'); + expect(spy.mock.calls[1][0]).toBe('/legal-entities/le/files?limit=100'); + }); +}); +``` + +Импорт дополнить `filesPath`, `listLegalEntityFiles`. + +Run: `npx vitest run src/services/api/files.test.ts 2>&1 | tail -8` +Expected: FAIL — `filesPath` не экспортируется, пути с query. + +- [ ] **Step 2: Реализация** + +В `src/services/api/files.ts`: + +```ts +/** Маршрут файлов ЮЛ: с консолидацией — рабочий стол / загрузка в консолидацию, без неё — файлы ЮЛ (design D-2). */ +export function filesPath(legalEntityId: string, consolidationId: string | null): string { + return consolidationId + ? `/consolidations/${encodeURIComponent(consolidationId)}/legal-entities/${legalEntityId}/files` + : `/legal-entities/${legalEntityId}/files`; +} +``` + +`uploadFiles(legalEntityId, files, consolidationId: string | null)`: вместо `query` — `apiFetch(filesPath(legalEntityId, consolidationId), { method: 'POST', body: form })`. +`listLegalEntityFiles(legalEntityId, consolidationId: string | null)`: `apiFetch>(`${filesPath(legalEntityId, consolidationId)}?limit=100`)`. +Docstring `uploadFiles` — «в консолидацию или без контекста». + +Run: `npx vitest run src/services/api/files.test.ts 2>&1 | tail -6 && npx tsc --noEmit 2>&1 | tail -5` +Expected: тесты PASS; tsc покажет ошибки в вызовах `useLegalEntityFiles`/`UploadDocumentsModal` — это ожидаемо до Task 3 +(если ошибок нет — убедиться, что сигнатура действительно изменилась). + +- [ ] **Step 3: Коммит** + +```bash +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web add src/services/api/files.ts src/services/api/files.test.ts +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web commit -q -m "$(cat <<'EOF' +api: загрузка и список файлов по маршрутам консолидации (#26) + +Co-Authored-By: Claude Fable 5.1 +EOF +)" +``` + +--- + +### Task 3: Панель — стол, переключатель, подпись, недоступная загрузка + +**Files:** +- Modify: `src/widgets/FilesDrawer/useLegalEntityFiles.ts`, `src/widgets/FilesDrawer/FilesDrawer.tsx`, + `src/widgets/FilesDrawer/FilesList.tsx`, `src/widgets/FilesDrawer/documentRows.ts`, + `src/widgets/FilesDrawer/documentRows.test.ts`, `src/widgets/FilesDrawer/UploadDocumentsModal.tsx` + +**Interfaces:** +- Consumes: `useCurrentConsolidation`, `isInWork`, `consolidationLabel` (Task 1); `uploadFiles`, `listLegalEntityFiles` (Task 2). +- Produces: `useLegalEntityFiles(legalEntityId: string | null, consolidationId: string | null, open: boolean)`; + `buildDocumentRows(files: FileOut[], labelFor?: (f: FileOut) => string | null): DocumentRow[]`; + `FilesList({ files, loading, labelFor? })`; `UploadDocumentsModal` принимает `consolidationId: string | null` вместо `versionId`. + +- [ ] **Step 1: Тест строк с подписью (красный)** + +В `src/widgets/FilesDrawer/documentRows.test.ts` добавить: + +```ts + it('подпись консолидации добавляется в subtitle последней, null — не добавляется', () => { + const rows = buildDocumentRows( + [file({ id: 'f1', documents: [doc()] }), file({ id: 'f2', documents: [], consolidation_id: 'c9' })], + (f) => (f.id === 'f2' ? 'вне консолидации' : null), + ); + expect(rows[0].subtitle.endsWith('вне консолидации')).toBe(false); + expect(rows[1].subtitle.endsWith(' · вне консолидации')).toBe(true); + }); +``` + +Run: `npx vitest run src/widgets/FilesDrawer/documentRows.test.ts 2>&1 | tail -6` +Expected: FAIL — второй аргумент игнорируется (tsc: лишний аргумент). + +- [ ] **Step 2: `documentRows.ts` и `FilesList.tsx`** + +`buildDocumentRows(files, labelFor?)`: `const extra = labelFor?.(f) ?? null;` в начале `flatMap`; в обеих ветках +`subtitle: join([... , extra])` (для файла без документов — `join([f.file_type, date, extra])`, для документа — +`join([typeText, f.file_type, date, extra])`). +`FilesList({ files, loading, labelFor }: { files: FileOut[]; loading: boolean; labelFor?: (f: FileOut) => string | null })` +→ `buildDocumentRows(files, labelFor)`. + +Run: `npx vitest run src/widgets/FilesDrawer/documentRows.test.ts 2>&1 | tail -4` +Expected: PASS. + +- [ ] **Step 3: Хук** + +`useLegalEntityFiles(legalEntityId, consolidationId, open)`: `listLegalEntityFiles(legalEntityId, consolidationId)` в +`reload` (зависимости `[legalEntityId, consolidationId]`); эффект сброса — зависимости `[open, legalEntityId, consolidationId, reload]`, +комментарий «при смене ЮЛ, режима (консолидация/все) или закрытии панели…». Docstring: «Файлы рабочего стола консолидации +(`consolidationId`) или все файлы ЮЛ (`null`)». + +- [ ] **Step 4: Панель и модалка** + +`UploadDocumentsModal`: prop `versionId` → `consolidationId: string | null`; `uploadFiles(legalEntityId, files, consolidationId)`. + +`FilesDrawer.tsx`: + +```tsx +import { CloseOutlined, UploadOutlined } from '@ant-design/icons'; +import { Button, Drawer, Switch, Tooltip, Typography } from 'antd'; +import { useEffect, useState } from 'react'; +import { fetchLegalEntity } from '../../services/api/directories/company_groups'; +import { isInWork, useCurrentConsolidation } from '../../store/slices/consolidations/selectors'; +import { useAppSelector } from '../../store/store'; +import type { FileOut, LegalEntityBrief } from '../../types/api'; +import { consolidationLabel } from './consolidationLabel'; +import { FilesList } from './FilesList'; +import { UploadDocumentsModal } from './UploadDocumentsModal'; +import { useLegalEntityFiles } from './useLegalEntityFiles'; + +const UPLOAD_DISABLED_HINT = 'Загрузка доступна только в версии в работе'; + +export function DocumentsDrawer({ open, onClose }: { open: boolean; onClose: () => void }) { + const [uploadOpen, setUploadOpen] = useState(false); + const [showAll, setShowAll] = useState(false); + const [entity, setEntity] = useState(null); + const { current_legal_entity_id, consolidations } = useAppSelector((s) => s.consolidations); + const current = useCurrentConsolidation(); + // консолидация не определена (версии нет в списке ГК) — режим «все» без переключателя (design D-3) + const desk = current !== null && !showAll; + const consolidationId = desk ? current.consolidation_id : null; + const { files, loading, reload } = useLegalEntityFiles(current_legal_entity_id, consolidationId, open); + const canUpload = current !== null && isInWork(current.version_status); + const labelFor = desk ? undefined : (f: FileOut) => consolidationLabel(f.consolidation_id, consolidations, current?.consolidation_id ?? null); + + useEffect(() => { + setShowAll(false); // новая панель открывается столом консоли + setEntity(null); + if (!open || !current_legal_entity_id) return; + let cancelled = false; + fetchLegalEntity(current_legal_entity_id) + .then((e) => { + if (!cancelled) setEntity(e); + }) + .catch(() => { + // без имени: шапка остаётся с подписью, панель не падает + }); + return () => { + cancelled = true; + }; + }, [open, current_legal_entity_id]); + + const uploadButton = ( + + ); + + return ( + +
+ {canUpload ? uploadButton : {uploadButton}} + {current !== null && ( + + + Все документы организации + + )} +
+ + {current_legal_entity_id && ( + setUploadOpen(false)} + onUploaded={() => void reload()} + /> + )} +
+ ); +} +``` + +Шапку `Drawer` (title, extra, size, closable) оставить как сейчас. `Tooltip` на disabled-кнопке требует обёртки +`` — antd не вешает события на disabled button. + +Run: `make check 2>&1 | tail -12` +Expected: biome, tsc, vitest, build — зелёные. + +- [ ] **Step 5: Коммит** + +```bash +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web add src/widgets/FilesDrawer +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web commit -q -m "$(cat <<'EOF' +drawer: рабочий стол консолидации, переключатель «Все документы организации», загрузка только в версии в работе (#26) + +Co-Authored-By: Claude Fable 5.1 +EOF +)" +``` + +--- + +### Task 4: Ручной прогон на стенде, README, PR + +**Files:** +- Modify: `README.md` (раздел про панель файлов, если есть упоминание `consolidation_version_id` — `grep -n consolidation_version_id README.md docs`), `CLAUDE.md` (строка про `FilesDrawer`) + +- [ ] **Step 1: Предусловие — стенд с ядром #96** + +Проверить: `curl -s -o /dev/null -w '%{http_code}' localhost:8000/api/v1/openapi.json` и в OpenAPI есть путь +`/api/v1/consolidations/{consolidation_id}/legal-entities/{legal_entity_id}/files` (`curl -s localhost:8000/api/v1/openapi.json | grep -c "legal-entities/{legal_entity_id}/files"` ≥ 2). +Если стенда с ядром #96 нет — ручной прогон записать в ledger как `deferred` (O-1: стенд не обновлён), выполнить +остальные шаги и указать это в PR. + +- [ ] **Step 2: Ручной сценарий в Playwright (сверка с макетом `../docs/mockup/screens/` для [39]/[40])** + +`make start` (порт 3000, прокси на :8000), затем через Playwright MCP: +1. Войти (`ka_go`), открыть ГК «Русский бисквит» → консолидация 2024 Q3 → строка ЮЛ → панель. Ожидание: заголовок + «Первичные документы», список стола, переключатель «Все документы организации» выключен, кнопка загрузки активна. +2. Включить переключатель: появились файлы других периодов с подписями «2024 К4» / «вне консолидации»; у файлов + текущей консоли подписи нет. Скриншот. +3. Выключить, загрузить файл из `../fixtures/osv/rb/2024-Q3/` через [40]: модалка без полей типа/периода, итог + «Принято: 1», в столе строка «Обработка», через ≤15 с — документ. Во время «Обработка» переключить режим туда и + обратно — список не «прыгает» на старый ответ (Review Focus 3). +4. Открыть панель из версии `CANCELLED` (создать через API `POST …/status {CANCELLED, comment}` на тестовой консоли + или использовать существующую): кнопка недоступна, подсказка при наведении; стол показывается (Review Focus 4). +5. Прямая ссылка `/groups/{id}?version={v}` в новой вкладке → панель открывается без ошибок в консоли браузера. + +Скриншоты шагов 1, 2, 4 — в описание PR (пути в `.superpowers/sdd//`, не в git). + +- [ ] **Step 3: README / CLAUDE.md** + +`grep -rn "consolidation_version_id" README.md CLAUDE.md docs scripts 2>/dev/null` — заменить упоминания на маршруты консолидации +(в `scripts/demo-prepare.sh` загрузка через `curl` должна идти на `/consolidations/$C/legal-entities/$LE/files`, где `C` — `consolidation_id` из ответа `POST /consolidations`); +в `CLAUDE.md` в строке про `FilesDrawer` дописать «(стол консолидации / все файлы ЮЛ, подпись консолидации, селектор +`store/slices/consolidations/selectors.ts`)». + +- [ ] **Step 4: Проверка, коммит, push, PR** + +```bash +cd /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web && make check 2>&1 | tail -15 +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web add README.md CLAUDE.md +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web commit -q -m "$(cat <<'EOF' +docs: панель файлов — маршруты консолидации (#26) + +Co-Authored-By: Claude Fable 5.1 +EOF +)" +git -C /Users/andrey.epifancev/projects/awp-ca/.wt/console-files/arm-web push -u origin feat/26-console-files +``` + +PR — через superpowers:finishing-a-development-branch (`tea pulls create --login home --repo RSHB/arm-web --base test +--head feat/26-console-files …`): заголовок «Панель «Первичные документы»: рабочий стол консолидации и загрузка в +консолидацию (#26)»; в описании — ссылки на issue #26, docs #90, arm-core #96, объяснение решений простым языком +(селектор вместо поля стора, переключатель, подпись), «Rulings I made», `deferred`, вывод `make check`, скриншоты. diff --git a/openspec/changes/console-files/.openspec.yaml b/openspec/changes/console-files/.openspec.yaml new file mode 100644 index 0000000..8126432 --- /dev/null +++ b/openspec/changes/console-files/.openspec.yaml @@ -0,0 +1,2 @@ +schema: spec-driven +created: 2026-10-09 diff --git a/openspec/changes/console-files/design.md b/openspec/changes/console-files/design.md new file mode 100644 index 0000000..ecd4085 --- /dev/null +++ b/openspec/changes/console-files/design.md @@ -0,0 +1,63 @@ +## Context + +Панель [39] (`widgets/FilesDrawer/FilesDrawer.tsx`) берёт `current_legal_entity_id` и `current_version_id` из стора, +грузит `GET /legal-entities/{id}/files?limit=100` хуком `useLegalEntityFiles` (опрос раз в 5 с, пока есть +`UPLOADED`/`PROCESSING`) и загружает файлы через `uploadFiles(legalEntityId, files, versionId)` с query +`consolidation_version_id`. Список консолидаций ГК (`IConsolidation[]` с `consolidation_id`, `year`, `quarter`, +`versions[] {id, status}`) уже лежит в сторе `consolidations`. Сквозной дизайн — `../docs/openspec/changes/console-files/design.md`. + +## Decisions + +**D-1. Консолидация и статус версии — селектор, не новое поле стора.** `store/slices/consolidations/selectors.ts`: +чистая функция `findCurrentConsolidation(consolidations, versionId) -> { consolidation_id, year, quarter, version_status } +| null` (ищет версию по `id` в `versions[]`) и хук-обёртка `useCurrentConsolidation()` через `useAppSelector`. Поле +`current_consolidation_id`, упомянутое в сквозном дизайне, не вводится: оно выводится из уже хранимого и не может +рассинхронизироваться с `?version=` (R-1). + +**D-2. Один сервис, два маршрута.** `listLegalEntityFiles(legalEntityId, consolidationId: string | null)` и +`uploadFiles(legalEntityId, files, consolidationId: string | null)`: при `consolidationId` путь +`/consolidations/{cid}/legal-entities/{le}/files`, иначе `/legal-entities/{le}/files`. Обработка 422 `UploadResult` +без изменений. + +**D-3. Режим панели — локальное состояние `showAll`.** `useLegalEntityFiles(legalEntityId, consolidationId, open)`: +смена `consolidationId` сбрасывает список и перечитывает (как смена ЮЛ). В панели переключатель antd `Switch` с +подписью «Все документы организации» под кнопкой загрузки; `consolidationId = showAll ? null : current.consolidation_id`. +Если консолидация не найдена (версия не из списка) — панель работает в режиме «все» и переключатель скрыт. + +**D-4. Подпись консолидации у файла.** `widgets/FilesDrawer/consolidationLabel.ts`: +`consolidationLabel(fileConsolidationId, consolidations, currentConsolidationId) -> string | null`: `null`, если файл в +текущей консоли; «вне консолидации» при `null`; «2025 К1» (`year` + `quarterShortLabel`) по списку; «другая +консолидация», если id нет в списке. `buildDocumentRows(files, labelFor?)` добавляет результат в `subtitle` последним +элементом; `FilesList` получает `labelFor` от панели только в режиме «все». + +**D-5. Загрузка только в версии «в работе».** `IN_WORK_STATUSES = ['INITIATED', 'DOCUMENTS_PROCESSING', 'DATA_CORRECTION']` +в `selectors.ts`; кнопка «Загрузить документы» `disabled`, если `version_status` не в списке, с `Tooltip` «Загрузка +доступна только в версии в работе». Модалка получает `consolidationId` вместо `versionId`. + +**D-6. Опрос** без изменений: тот же хук, тот же интервал; перечитывается активный список. + +## Risks / Trade-offs + +- До выката arm-core#96 на стенд панель получит 404 от маршрутов консолидации — ветка вливается после ядра. +- `limit=100` остаётся: стол консоли меньше списка ЮЛ, ограничение стало мягче. + +## Decisions made without discussion + +1. 09.10.2026 — **R-1:** консолидация и статус версии выводятся селектором из `consolidations[]` + `current_version_id`, + а не хранятся отдельным полем `current_consolidation_id` (сквозной дизайн docs #90 называл поле). Цена ошибки: + низкая — если список ГК в сторе пуст (прямой переход по ссылке до загрузки списка), панель секунду работает в + режиме «все»; список грузится той же страницей. +2. 09.10.2026 — **R-2:** подпись консолидации показывается только в режиме «Все документы организации» и только у файлов + не из текущей консоли; в режиме стола подписи нет (все файлы — этой консоли или подхваченные). Цена ошибки: низкая. +3. 09.10.2026 — **R-3:** неизвестный `consolidation_id` (нет в списке ГК) подписывается «другая консолидация», а не + скрывается. Цена ошибки: нулевая. +4. 10.10.2026 — **R-4:** ручной прогон на стенде отложен до выката arm-core#96. Цена ошибки: UI-дефект всплывёт позже; + PR не вливать до прогона. +5. 10.10.2026 — **R-5:** trailer коммита 5b7b02a «Claude Haiku 5.5» вместо «Claude Fable 5.1» оставлен. Цена ошибки: + неверная атрибуция в одном коммите. +6. 10.10.2026 — **R-6:** промежуточное состояние (tsc-ошибка и versionId в UploadDocumentsModal между Task 2 и Task 3) + принято, чинится в Task 3. Цена ошибки: нулевая, закрыто в Task 3. +7. 10.10.2026 — **R-7:** `consolidation_id` в scripts/demo-prepare.sh берётся из карточки версии + `GET /consolidations/versions/$V`, а не из `POST /consolidations`. Цена ошибки: скрипт демо не запускался на живом ядре. +8. 10.10.2026 — **R-8:** после создания консолидации перечитывается список консолидаций группы (I-1). Цена ошибки: + правка выходит за Impact proposal, лишний запрос. diff --git a/openspec/changes/console-files/proposal.md b/openspec/changes/console-files/proposal.md new file mode 100644 index 0000000..8f90ce7 --- /dev/null +++ b/openspec/changes/console-files/proposal.md @@ -0,0 +1,51 @@ +## Why + +Панель «Первичные документы» [39], открытая из строки ЮЛ в карточке версии, показывает все файлы ЮЛ +(`GET /legal-entities/{id}/files`), в том числе загруженные в другие консолидации ГК. Решения владельца 09.10.2026 +(RSHB/arm-ka-docs#90, сквозной change `../docs/openspec/changes/console-files/`; ядро — RSHB/arm-core#96): файл +загружается в консолидацию (ГК + период) и образует её «рабочий стол»; ядро даёт маршруты +`POST/GET /consolidations/{cid}/legal-entities/{le}/files`, а query `consolidation_version_id` у +`POST /legal-entities/{id}/files` удаляет. Issue — RSHB/arm-web#26. + +## What Changes + +Экраны: [39] панель «Первичные документы», [40] модалка «Загрузка документов». Эндпоинты arm-core: +`GET /consolidations/{cid}/legal-entities/{le}/files` (новый), `POST /consolidations/{cid}/legal-entities/{le}/files` +(новый), `GET /legal-entities/{id}/files` (без изменений, в `FileOut` добавлено `consolidation_id`), +`POST /legal-entities/{id}/files` (без query версии). + +- Консолидация и статус открытой версии выводятся из стора (`consolidations[]` + `current_version_id`) селектором; + отдельного поля в сторе нет. +- `services/api/files.ts`: `listLegalEntityFiles(legalEntityId, consolidationId)` и + `uploadFiles(legalEntityId, files, consolidationId)` — с консолидацией маршруты `/consolidations/{cid}/…`, без неё + маршруты ЮЛ; query `consolidation_version_id` исчезает. +- Панель [39]: по умолчанию рабочий стол консолидации; переключатель «Все документы организации» переключает на список + ЮЛ; в режиме «все» у файла в подписи — его консолидация («2025 К1» по списку консолидаций ГК, «вне консолидации» при + `null`, «другая консолидация» при неизвестном id). Опрос раз в 5 с — по активному списку. +- Кнопка «Загрузить документы» недоступна, если открытая версия не «в работе» (`INITIATED`, `DOCUMENTS_PROCESSING`, + `DATA_CORRECTION`), с подсказкой. +- Типы: `FileOut.consolidation_id: string | null`. +- Спека `primary-files-ui`: три требования переписаны/добавлены. + +## Non-goals + +- Экран [60] «Монозаёмщики» и панель без контекста консолидации — не делаются (их пока нет во фронте). +- Пометка «документ не подходит версии» в панели — нет (чек-лист версии, OQ-120). +- Перенос файла между консолидациями, удаление файла из панели — нет (как и сейчас). +- Изменение экрана проверки документа [41] и сводной ОСВ — нет. +- `arm-core/web` (дубль фронта в ядре) не трогается. + +## Capabilities + +### New Capabilities + +### Modified Capabilities +- `primary-files-ui`: загрузка в консолидацию, рабочий стол и переключатель «Все документы организации», недоступная + загрузка вне версии «в работе». + +## Impact + +- `src/services/api/files.ts` (+ `files.test.ts`), `src/widgets/FilesDrawer/{FilesDrawer,FilesList,UploadDocumentsModal}.tsx`, + `src/widgets/FilesDrawer/{useLegalEntityFiles,documentRows}.ts` (+ тест), новый `src/widgets/FilesDrawer/consolidationLabel.ts` + (+ тест), новый `src/store/slices/consolidations/selectors.ts` (+ тест), `src/types/api.ts`. +- Зависимость: ядро с RSHB/arm-core#96 на стенде; до его выката панель получит 404 на маршрутах консолидации. diff --git a/openspec/changes/console-files/specs/primary-files-ui/spec.md b/openspec/changes/console-files/specs/primary-files-ui/spec.md new file mode 100644 index 0000000..32c0c0b --- /dev/null +++ b/openspec/changes/console-files/specs/primary-files-ui/spec.md @@ -0,0 +1,62 @@ +## MODIFIED Requirements + +### Requirement: Загрузка файлов ЮЛ в контексте версии +Панель «Файлы ЮЛ» [39]/[40], открытая из строки ЮЛ карточки версии, SHALL загружать выбранные файлы одним запросом +`POST /consolidations/{consolidation_id}/legal-entities/{legal_entity_id}/files` (multipart, поле `files` повторяется), +где `consolidation_id` — консолидация открытой версии (из списка консолидаций ГК в сторе по `current_version_id`), +без других полей формы и без query `consolidation_version_id`. Тип и период документа не спрашиваются. Кнопка +«Загрузить документы» SHALL быть недоступна с подсказкой «Загрузка доступна только в версии в работе», если статус +открытой версии не `INITIATED`, `DOCUMENTS_PROCESSING` или `DATA_CORRECTION`. После ответа панель SHALL показать итог: +принятые файлы (`accepted[].name`) и отклонённые с причиной (`rejected[].name` + `message`). Ответ 422 с телом +`UploadResult` SHALL трактоваться как «ни один файл не принят» с тем же показом `rejected[]`; ответ 422 с `detail` +(`ENTITY_NOT_IN_PERIMETER`), 409 (`CONSOLIDATION_NOT_EDITABLE`) и прочие ошибки — показом `message`. + +#### Scenario: Часть файлов отклонена +- **WHEN** аналитик загружает ОСВ-файл и файл, уже загруженный ранее у этого ЮЛ +- **THEN** первый показан принятым, второй — отклонённым с текстом причины `DUPLICATE_FILE` от сервера + +#### Scenario: Все файлы отклонены +- **WHEN** сервер отвечает 422 с телом `UploadResult` +- **THEN** панель показывает каждый отклонённый файл с причиной, а не общую ошибку + +#### Scenario: Загрузка идёт в консолидацию +- **WHEN** панель открыта из версии консолидации 2024 Q3 и аналитик загружает файл +- **THEN** запрос уходит на `/consolidations/{id консолидации 2024 Q3}/legal-entities/{id ЮЛ}/files` без query; после + приёма файл появляется в столе панели + +#### Scenario: Версия не в работе +- **WHEN** панель открыта из версии в статусе `CANCELLED` +- **THEN** кнопка «Загрузить документы» недоступна, при наведении — подсказка + +### Requirement: Список файлов и документов ЮЛ +Панель SHALL по умолчанию показывать рабочий стол консолидации открытой версии — +`GET /consolidations/{consolidation_id}/legal-entities/{id}/files` (новые первыми): имя, статус файла по-русски +(«Обработка» для `UPLOADED`/`PROCESSING`, «Ошибка» с причиной из `job` для `ERROR`) и вложенные `documents[]` — тип +(«ОСВ <счёт>», «Бух. баланс», «ОФР», «Расшифровка», «Прочее»), период («2024 К1», «2024 · 9 мес.», «период не +определён»), статус документа («Требуется проверка», «Загружен» для `CONFIRMED`, «Ошибка»). Документы `OSV`, `BALANCE`, +`INCOME` SHALL быть ссылками на `/documents/:document_id`. Переключатель «Все документы организации» SHALL переключать +панель на `GET /legal-entities/{id}/files`; в этом режиме у файла не из текущей консолидации в подписи SHALL стоять его +консолидация: «<год> К» по списку консолидаций ГК, «вне консолидации» при `consolidation_id = null`, «другая +консолидация», если id нет в списке. Если консолидация открытой версии не определена (версии нет в списке ГК), панель +SHALL работать в режиме «все» без переключателя. + +#### Scenario: PDF с двумя формами +- **WHEN** файл `aep_6.pdf` распознан +- **THEN** под файлом видны два документа — «Бух. баланс» и «ОФР» со статусом «Требуется проверка», каждый открывает + экран проверки + +#### Scenario: Стол и все документы +- **WHEN** у ЮЛ есть файлы, загруженные в консолидации 2024 Q3 и 2024 Q4, и файл, загруженный без контекста; панель + открыта из версии 2024 Q3 +- **THEN** по умолчанию виден только файл 2024 Q3; после переключателя «Все документы организации» видны все три, у + файла 2024 Q4 подпись «2024 К4», у файла без контекста — «вне консолидации», у файла 2024 Q3 подписи нет + +### Requirement: Опрос до окончания распознавания +Пока панель открыта и среди файлов активного списка (стол или все документы) есть `UPLOADED` или `PROCESSING`, панель +SHALL перечитывать этот список раз в 5 секунд и SHALL прекратить опрос, когда таких файлов нет или панель закрыта. +Переключение режима или смена ЮЛ SHALL сбрасывать список и перечитывать его. При закрытии панели карточка версии SHALL +перечитываться (обновить ячейки чек-листа). + +#### Scenario: Опрос останавливается +- **WHEN** все файлы активного списка перешли в `PROCESSED` или `ERROR` +- **THEN** запросы списка файлов прекращаются diff --git a/scripts/demo-prepare.sh b/scripts/demo-prepare.sh index aca0dd2..da39d16 100755 --- a/scripts/demo-prepare.sh +++ b/scripts/demo-prepare.sh @@ -1,5 +1,5 @@ #!/usr/bin/env bash -# Подготовка демо: загрузить квартальные ОСВ РБ и Счастливочной за 2024 Q1–Q3 в версию и подтвердить ОСВ-документы. +# Подготовка демо: загрузить квартальные ОСВ РБ и Счастливочной за 2024 Q1–Q3 в консолидацию версии и подтвердить ОСВ-документы. # Использование: scripts/demo-prepare.sh [email] (стенд arm-core на localhost:8000, DEBUG=true) # Каталог фикстур: FIXTURES_DIR (по умолчанию ../../fixtures относительно репозитория arm-web, т.е. workspace/fixtures). set -euo pipefail @@ -9,7 +9,10 @@ API="${API:-localhost:8000/api/v1}" F="${FIXTURES_DIR:-$(cd "$(dirname "$0")/../.." && pwd)/fixtures}/osv" H="Authorization: Bearer $(curl -sf -X POST "$API/dev/token" -H 'Content-Type: application/json' -d "{\"email\":\"$EMAIL\"}" | python3 -c 'import sys,json;print(json.load(sys.stdin)["access_token"])')" -members=$(curl -sf "$API/consolidations/versions/$V" -H "$H" | python3 -c ' +card=$(curl -sf "$API/consolidations/versions/$V" -H "$H") +# загрузка идёт в консолидацию (период), а не в версию: consolidation_id берём из карточки версии +C=$(echo "$card" | python3 -c 'import sys,json;print(json.load(sys.stdin)["consolidation_id"])') +members=$(echo "$card" | python3 -c ' import sys,json for m in json.load(sys.stdin)["members"]: print(m["id"], m["inn"], m["name"])') # ЮЛ ищем по ИНН (в справочнике имена могут быть в верхнем регистре) @@ -19,7 +22,7 @@ LE_SCH=$(le_by_inn 3528104476) [ -n "$LE_RB" ] && [ -n "$LE_SCH" ] || { echo "В периметре версии нет РБ или Счастливочной:"; echo "$members"; exit 1; } upload() { for f in "$F/$2"/2024-Q{1,2,3}/*; do - curl -s -X POST "$API/legal-entities/$1/files?consolidation_version_id=$V" -H "$H" -F "files=@$f" \ + curl -s -X POST "$API/consolidations/$C/legal-entities/$1/files" -H "$H" -F "files=@$f" \ | python3 -c 'import sys,json;d=json.load(sys.stdin);print("принят" if d.get("accepted") else d.get("rejected") or d)' done; } upload "$LE_RB" rb @@ -29,7 +32,7 @@ echo "Жду распознавания…" for _ in $(seq 1 60); do pending=0 for LE in $LE_RB $LE_SCH; do - n=$(curl -sf "$API/legal-entities/$LE/files?limit=100" -H "$H" | python3 -c 'import sys,json;print(sum(f["status"] in ("UPLOADED","PROCESSING") for f in json.load(sys.stdin)["items"]))') + n=$(curl -sf "$API/consolidations/$C/legal-entities/$LE/files?limit=100" -H "$H" | python3 -c 'import sys,json;print(sum(f["status"] in ("UPLOADED","PROCESSING") for f in json.load(sys.stdin)["items"]))') pending=$((pending + n)) done [ "$pending" -eq 0 ] && break @@ -37,7 +40,7 @@ for _ in $(seq 1 60); do done for LE in $LE_RB $LE_SCH; do - curl -sf "$API/legal-entities/$LE/files?limit=100" -H "$H" | python3 -c ' + curl -sf "$API/consolidations/$C/legal-entities/$LE/files?limit=100" -H "$H" | python3 -c ' import sys,json for f in json.load(sys.stdin)["items"]: for d in f["documents"]: diff --git a/src/services/api/files.test.ts b/src/services/api/files.test.ts index 3a73fa9..07fce57 100644 --- a/src/services/api/files.test.ts +++ b/src/services/api/files.test.ts @@ -1,7 +1,7 @@ import { afterEach, describe, expect, it, vi } from 'vitest'; import { ApiError } from '../../api/client'; import type { FileOut, UploadResult } from '../../types/api'; -import { hasPendingFiles, isUploadResult, summarizeUpload, uploadFiles } from './files'; +import { filesPath, hasPendingFiles, isUploadResult, listLegalEntityFiles, summarizeUpload, uploadFiles } from './files'; const result: UploadResult = { accepted: [{ file_id: 'f1', job_id: 'j1', name: 'osv.xlsx' }], @@ -35,6 +35,7 @@ describe('hasPendingFiles', () => { version_no: 1, size_bytes: null, uploaded_at: null, + consolidation_id: null, job: null, documents: [], }); @@ -48,28 +49,37 @@ describe('hasPendingFiles', () => { }); }); +describe('filesPath', () => { + it('с консолидацией — маршрут консолидации, без неё — маршрут ЮЛ', () => { + expect(filesPath('le', 'c1')).toBe('/consolidations/c1/legal-entities/le/files'); + expect(filesPath('le', null)).toBe('/legal-entities/le/files'); + }); +}); + describe('uploadFiles', () => { afterEach(() => vi.restoreAllMocks()); - it('422 с UploadResult возвращается как результат', async () => { + it('422 с UploadResult возвращается как результат (и с консолидацией)', async () => { const client = await import('../../api/client'); vi.spyOn(client, 'apiFetch').mockRejectedValue(new ApiError(422, undefined, 'x', { accepted: [], rejected: result.rejected })); - await expect(uploadFiles('le', [new File(['a'], 'dup.xlsx')], null)).resolves.toEqual({ accepted: [], rejected: result.rejected }); + await expect(uploadFiles('le', [new File(['a'], 'dup.xlsx')], 'c1')).resolves.toEqual({ accepted: [], rejected: result.rejected }); }); - it('422 с detail пробрасывается', async () => { + it('422 с detail и 409 пробрасываются', async () => { const client = await import('../../api/client'); - const error = new ApiError(422, 'ENTITY_NOT_IN_PERIMETER', 'ЮЛ вне периметра', { detail: { code: 'ENTITY_NOT_IN_PERIMETER' } }); + const error = new ApiError(409, 'CONSOLIDATION_NOT_EDITABLE', 'нет версии в работе', { + detail: { code: 'CONSOLIDATION_NOT_EDITABLE' }, + }); vi.spyOn(client, 'apiFetch').mockRejectedValue(error); - await expect(uploadFiles('le', [new File(['a'], 'a.xlsx')], 'v1')).rejects.toBe(error); + await expect(uploadFiles('le', [new File(['a'], 'a.xlsx')], 'c1')).rejects.toBe(error); }); - it('версия попадает в query, лишних полей формы нет', async () => { + it('консолидация — в пути, query нет, лишних полей формы нет', async () => { const client = await import('../../api/client'); const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue(result); - await uploadFiles('le', [new File(['a'], 'a.xlsx'), new File(['b'], 'b.pdf')], 'v1'); + await uploadFiles('le', [new File(['a'], 'a.xlsx'), new File(['b'], 'b.pdf')], 'c1'); const [path, init] = spy.mock.calls[0]; - expect(path).toBe('/legal-entities/le/files?consolidation_version_id=v1'); + expect(path).toBe('/consolidations/c1/legal-entities/le/files'); const form = init?.body as FormData; expect(form.getAll('files')).toHaveLength(2); const keys: string[] = []; @@ -77,10 +87,22 @@ describe('uploadFiles', () => { expect(keys.every((k) => k === 'files')).toBe(true); }); - it('без версии — без query', async () => { + it('без консолидации — маршрут ЮЛ без query', async () => { const client = await import('../../api/client'); const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue(result); await uploadFiles('le', [new File(['a'], 'a.xlsx')], null); expect(spy.mock.calls[0][0]).toBe('/legal-entities/le/files'); }); }); + +describe('listLegalEntityFiles', () => { + afterEach(() => vi.restoreAllMocks()); + it('стол консолидации и список ЮЛ', async () => { + const client = await import('../../api/client'); + const spy = vi.spyOn(client, 'apiFetch').mockResolvedValue({ items: [], total: 0, limit: 100, offset: 0 }); + await listLegalEntityFiles('le', 'c1'); + await listLegalEntityFiles('le', null); + expect(spy.mock.calls[0][0]).toBe('/consolidations/c1/legal-entities/le/files?limit=100'); + expect(spy.mock.calls[1][0]).toBe('/legal-entities/le/files?limit=100'); + }); +}); diff --git a/src/services/api/files.ts b/src/services/api/files.ts index 9ff317e..921f864 100644 --- a/src/services/api/files.ts +++ b/src/services/api/files.ts @@ -10,15 +10,21 @@ export function isUploadResult(body: unknown): body is UploadResult { ); } -/** Загрузка файлов ЮЛ одним запросом; 422 «ни один не принят» возвращается как результат, прочие ошибки — throw. */ -export async function uploadFiles(legalEntityId: string, files: File[], versionId: string | null): Promise { +/** Маршрут файлов ЮЛ: с консолидацией — файлы в контексте консолидации, без неё — файлы ЮЛ (design D-2). */ +export function filesPath(legalEntityId: string, consolidationId: string | null): string { + return consolidationId + ? `/consolidations/${encodeURIComponent(consolidationId)}/legal-entities/${legalEntityId}/files` + : `/legal-entities/${legalEntityId}/files`; +} + +/** Загрузка файлов ЮЛ одним запросом (в консолидацию или без контекста); 422 «ни один не принят» возвращается как результат, прочие ошибки — throw. */ +export async function uploadFiles(legalEntityId: string, files: File[], consolidationId: string | null): Promise { const form = new FormData(); for (const file of files) { form.append('files', file); } - const query = versionId ? `?consolidation_version_id=${encodeURIComponent(versionId)}` : ''; try { - return await apiFetch(`/legal-entities/${legalEntityId}/files${query}`, { method: 'POST', body: form }); + return await apiFetch(filesPath(legalEntityId, consolidationId), { method: 'POST', body: form }); } catch (e: unknown) { if (e instanceof ApiError && e.status === 422 && isUploadResult(e.body)) { return e.body; @@ -34,8 +40,8 @@ export function summarizeUpload(result: UploadResult): { accepted: string[]; rej }; } -export async function listLegalEntityFiles(legalEntityId: string): Promise { - const page = await apiFetch>(`/legal-entities/${legalEntityId}/files?limit=100`); +export async function listLegalEntityFiles(legalEntityId: string, consolidationId: string | null): Promise { + const page = await apiFetch>(`${filesPath(legalEntityId, consolidationId)}?limit=100`); return page.items; } diff --git a/src/store/slices/consolidations/selectors.test.ts b/src/store/slices/consolidations/selectors.test.ts new file mode 100644 index 0000000..91a429a --- /dev/null +++ b/src/store/slices/consolidations/selectors.test.ts @@ -0,0 +1,84 @@ +import { describe, expect, it } from 'vitest'; +import type { IConsolidation } from '../../../types/consolidations/consolidations'; +import { findCurrentConsolidation, isInWork } from './selectors'; + +const cons: IConsolidation[] = [ + { + consolidation_id: 'c-2024q3', + year: 2024, + quarter: 'Q3', + reporting_date: '2024-10-01', + versions: [ + { + id: 'v1', + version_no: 1, + status: 'VERIFIED', + scenario: 'CREATE', + approach: 'BASIC', + lock: null, + created_at: null, + members_count: 2, + }, + { + id: 'v2', + version_no: 2, + status: 'INITIATED', + scenario: 'CREATE', + approach: 'BASIC', + lock: null, + created_at: null, + members_count: 2, + }, + ], + }, + { + consolidation_id: 'c-2024q4', + year: 2024, + quarter: 'Q4', + reporting_date: '2025-01-01', + versions: [ + { + id: 'v3', + version_no: 1, + status: 'CANCELLED', + scenario: 'CREATE', + approach: 'BASIC', + lock: null, + created_at: null, + members_count: 2, + }, + ], + }, +]; + +describe('findCurrentConsolidation', () => { + it('находит консолидацию и статус версии по id версии', () => { + expect(findCurrentConsolidation(cons, 'v2')).toEqual({ + consolidation_id: 'c-2024q3', + year: 2024, + quarter: 'Q3', + version_status: 'INITIATED', + }); + expect(findCurrentConsolidation(cons, 'v3')).toEqual({ + consolidation_id: 'c-2024q4', + year: 2024, + quarter: 'Q4', + version_status: 'CANCELLED', + }); + }); + it('null без версии, при неизвестной версии и при пустом списке (прямая ссылка до загрузки)', () => { + expect(findCurrentConsolidation(cons, null)).toBeNull(); + expect(findCurrentConsolidation(cons, 'nope')).toBeNull(); + expect(findCurrentConsolidation([], 'v2')).toBeNull(); + }); +}); + +describe('isInWork', () => { + it('в работе — три статуса', () => { + expect(isInWork('INITIATED')).toBe(true); + expect(isInWork('DOCUMENTS_PROCESSING')).toBe(true); + expect(isInWork('DATA_CORRECTION')).toBe(true); + expect(isInWork('VERIFIED')).toBe(false); + expect(isInWork('CANCELLED')).toBe(false); + }); +}); diff --git a/src/store/slices/consolidations/selectors.ts b/src/store/slices/consolidations/selectors.ts new file mode 100644 index 0000000..ee50b7f --- /dev/null +++ b/src/store/slices/consolidations/selectors.ts @@ -0,0 +1,31 @@ +import type { ConsStatus, QuarterEnum } from '../../../types/consolidations/common_types'; +import type { IConsolidation } from '../../../types/consolidations/consolidations'; +import { useAppSelector } from '../../store'; + +/** Консолидация открытой версии и статус этой версии (design D-1): выводится из списка ГК, в сторе не хранится. */ +export interface CurrentConsolidation { + consolidation_id: string; + year: number; + quarter: QuarterEnum; + version_status: ConsStatus; +} + +/** Статусы «в работе» (§9.1): только в них разрешена загрузка файлов в консолидацию. */ +export const IN_WORK_STATUSES: ReadonlySet = new Set(['INITIATED', 'DOCUMENTS_PROCESSING', 'DATA_CORRECTION']); + +export const isInWork = (status: ConsStatus): boolean => IN_WORK_STATUSES.has(status); + +export function findCurrentConsolidation(consolidations: IConsolidation[], versionId: string | null): CurrentConsolidation | null { + if (!versionId) return null; + for (const c of consolidations) { + const version = c.versions.find((v) => v.id === versionId); + if (version) return { consolidation_id: c.consolidation_id, year: c.year, quarter: c.quarter, version_status: version.status }; + } + return null; +} + +export function useCurrentConsolidation(): CurrentConsolidation | null { + const consolidations = useAppSelector((s) => s.consolidations.consolidations); + const versionId = useAppSelector((s) => s.consolidations.current_version_id); + return findCurrentConsolidation(consolidations, versionId); +} diff --git a/src/types/api.ts b/src/types/api.ts index 91e2c7d..222a082 100644 --- a/src/types/api.ts +++ b/src/types/api.ts @@ -63,6 +63,8 @@ export interface FileOut { version_no: number; size_bytes: number | null; uploaded_at: string | null; + /** консолидация, в контексте которой загружен файл (v5.18); null — без контекста */ + consolidation_id: string | null; job: JobBrief | null; documents: DocumentBrief[]; } diff --git a/src/types/consolidations/consolidations.ts b/src/types/consolidations/consolidations.ts index 23bebbb..f546b70 100644 --- a/src/types/consolidations/consolidations.ts +++ b/src/types/consolidations/consolidations.ts @@ -1,6 +1,6 @@ import type { Approach, ConsRussianStatus, ConsStatus, LockOut, QuarterEnum, Scenario } from './common_types'; -interface IConsolidationVersion { +export interface IConsolidationVersion { id: string; version_no: number; status: ConsStatus; diff --git a/src/widgets/FilesDrawer/FilesDrawer.tsx b/src/widgets/FilesDrawer/FilesDrawer.tsx index 1ac9e2a..3026379 100644 --- a/src/widgets/FilesDrawer/FilesDrawer.tsx +++ b/src/widgets/FilesDrawer/FilesDrawer.tsx @@ -1,20 +1,34 @@ import { CloseOutlined, UploadOutlined } from '@ant-design/icons'; -import { Button, Drawer, Typography } from 'antd'; +import { Button, Drawer, Switch, Tooltip, Typography } from 'antd'; import { useEffect, useState } from 'react'; import { fetchLegalEntity } from '../../services/api/directories/company_groups'; +import { isInWork, useCurrentConsolidation } from '../../store/slices/consolidations/selectors'; import { useAppSelector } from '../../store/store'; -import type { LegalEntityBrief } from '../../types/api'; +import type { FileOut, LegalEntityBrief } from '../../types/api'; import { FilesList } from './FilesList'; import { UploadDocumentsModal } from './UploadDocumentsModal'; +import { consolidationLabel } from './consolidationLabel'; import { useLegalEntityFiles } from './useLegalEntityFiles'; +const UPLOAD_DISABLED_HINT = 'Загрузка доступна только в версии в работе'; + export function DocumentsDrawer({ open, onClose }: { open: boolean; onClose: () => void }) { const [uploadOpen, setUploadOpen] = useState(false); const [entity, setEntity] = useState(null); - const { current_version_id, current_legal_entity_id } = useAppSelector((s) => s.consolidations); - const { files, loading, reload } = useLegalEntityFiles(current_legal_entity_id, open); + const [showAll, setShowAll] = useState(false); + const { current_legal_entity_id, consolidations } = useAppSelector((s) => s.consolidations); + const current = useCurrentConsolidation(); + // консолидация не определена (версии нет в списке ГК) — режим «все» без переключателя (design D-3) + const desk = current !== null && !showAll; + const consolidationId = desk ? current.consolidation_id : null; + const { files, loading, reload } = useLegalEntityFiles(current_legal_entity_id, consolidationId, open); + const canUpload = current !== null && isInWork(current.version_status); + const labelFor = desk + ? undefined + : (f: FileOut) => consolidationLabel(f.consolidation_id, consolidations, current?.consolidation_id ?? null); useEffect(() => { + setShowAll(false); // новая панель открывается столом консоли setEntity(null); if (!open || !current_legal_entity_id) return; let cancelled = false; @@ -30,6 +44,17 @@ export function DocumentsDrawer({ open, onClose }: { open: boolean; onClose: () }; }, [open, current_legal_entity_id]); + const uploadButton = ( + + ); + return ( }> -
- +
+ {canUpload ? ( + uploadButton + ) : ( + + {uploadButton} + + )} + {current !== null && ( + + + Все документы организации + + )}
- + {current_legal_entity_id && ( setUploadOpen(false)} diff --git a/src/widgets/FilesDrawer/FilesList.tsx b/src/widgets/FilesDrawer/FilesList.tsx index 41f46c0..8e4eaf8 100644 --- a/src/widgets/FilesDrawer/FilesList.tsx +++ b/src/widgets/FilesDrawer/FilesList.tsx @@ -57,9 +57,9 @@ function StatusPill({ row }: { row: DocumentRow }) { const HEADER = { fontSize: 11, letterSpacing: 0.5, color: '#8c8c8c', textTransform: 'uppercase' as const }; const head = (text: string) => {text}; -export function FilesList({ files, loading }: { files: FileOut[]; loading: boolean }) { +export function FilesList({ files, loading, labelFor }: { files: FileOut[]; loading: boolean; labelFor?: (f: FileOut) => string | null }) { const navigate = useNavigate(); - const rows = buildDocumentRows(files); + const rows = buildDocumentRows(files, labelFor); const columns: ColumnsType = [ { diff --git a/src/widgets/FilesDrawer/UploadDocumentsModal.tsx b/src/widgets/FilesDrawer/UploadDocumentsModal.tsx index a5dddf2..ee9b608 100644 --- a/src/widgets/FilesDrawer/UploadDocumentsModal.tsx +++ b/src/widgets/FilesDrawer/UploadDocumentsModal.tsx @@ -7,7 +7,7 @@ import { summarizeUpload, uploadFiles } from '../../services/api/files'; type Props = { legalEntityId: string; - versionId: string | null; + consolidationId: string | null; organizationName?: string | null; open: boolean; onClose: () => void; @@ -16,7 +16,7 @@ type Props = { type Summary = ReturnType; -export function UploadDocumentsModal({ legalEntityId, versionId, organizationName, open, onClose, onUploaded }: Props) { +export function UploadDocumentsModal({ legalEntityId, consolidationId, organizationName, open, onClose, onUploaded }: Props) { const [fileList, setFileList] = useState([]); const [uploading, setUploading] = useState(false); const [summary, setSummary] = useState(null); @@ -36,7 +36,7 @@ export function UploadDocumentsModal({ legalEntityId, versionId, organizationNam const files = fileList.map((f) => f.originFileObj).filter((f): f is NonNullable => Boolean(f)); setUploading(true); try { - const result = summarizeUpload(await uploadFiles(legalEntityId, files, versionId)); + const result = summarizeUpload(await uploadFiles(legalEntityId, files, consolidationId)); setSummary(result); if (result.accepted.length > 0) onUploaded(); else message.error('Ни один файл не принят'); diff --git a/src/widgets/FilesDrawer/consolidationLabel.test.ts b/src/widgets/FilesDrawer/consolidationLabel.test.ts new file mode 100644 index 0000000..3fc6447 --- /dev/null +++ b/src/widgets/FilesDrawer/consolidationLabel.test.ts @@ -0,0 +1,25 @@ +import { describe, expect, it } from 'vitest'; +import type { IConsolidation } from '../../types/consolidations/consolidations'; +import { consolidationLabel } from './consolidationLabel'; + +const cons: IConsolidation[] = [ + { consolidation_id: 'c-2024q3', year: 2024, quarter: 'Q3', reporting_date: '2024-10-01', versions: [] }, + { consolidation_id: 'c-2025q1', year: 2025, quarter: 'Q1', reporting_date: '2025-04-01', versions: [] }, +]; + +describe('consolidationLabel', () => { + it('файл текущей консоли — без подписи', () => { + expect(consolidationLabel('c-2024q3', cons, 'c-2024q3')).toBeNull(); + }); + it('другая консоль из списка — год и квартал', () => { + expect(consolidationLabel('c-2025q1', cons, 'c-2024q3')).toBe('2025 К1'); + }); + it('без контекста и неизвестный id', () => { + expect(consolidationLabel(null, cons, 'c-2024q3')).toBe('вне консолидации'); + expect(consolidationLabel('ghost', cons, 'c-2024q3')).toBe('другая консолидация'); + }); + it('без текущей консолидации подпись есть у всех файлов', () => { + expect(consolidationLabel('c-2024q3', cons, null)).toBe('2024 К3'); + expect(consolidationLabel(null, cons, null)).toBe('вне консолидации'); + }); +}); diff --git a/src/widgets/FilesDrawer/consolidationLabel.ts b/src/widgets/FilesDrawer/consolidationLabel.ts new file mode 100644 index 0000000..b406016 --- /dev/null +++ b/src/widgets/FilesDrawer/consolidationLabel.ts @@ -0,0 +1,14 @@ +import type { IConsolidation } from '../../types/consolidations/consolidations'; +import { quarterShortLabel } from '../../utils/labels'; + +/** Подпись консолидации у файла в режиме «Все документы организации» (design D-4): null — файл текущей консоли. */ +export function consolidationLabel( + fileConsolidationId: string | null, + consolidations: IConsolidation[], + currentConsolidationId: string | null, +): string | null { + if (fileConsolidationId === null) return 'вне консолидации'; + if (fileConsolidationId === currentConsolidationId) return null; + const found = consolidations.find((c) => c.consolidation_id === fileConsolidationId); + return found ? `${found.year} ${quarterShortLabel(found.quarter)}` : 'другая консолидация'; +} diff --git a/src/widgets/FilesDrawer/documentRows.test.ts b/src/widgets/FilesDrawer/documentRows.test.ts index 33c6c9a..9b20410 100644 --- a/src/widgets/FilesDrawer/documentRows.test.ts +++ b/src/widgets/FilesDrawer/documentRows.test.ts @@ -26,6 +26,7 @@ const file = (over: Partial = {}): FileOut => ({ version_no: 1, size_bytes: 10, uploaded_at: '2024-03-12T10:00:00Z', + consolidation_id: null, job: null, documents: [], ...over, @@ -104,4 +105,13 @@ describe('buildDocumentRows', () => { expect(rows[0].subtitle).toBe('Прочий документ · PDF · 12.03.2024'); expect(rows[0].linkable).toBe(false); }); + + it('подпись консолидации добавляется в subtitle последней, null — не добавляется', () => { + const rows = buildDocumentRows( + [file({ id: 'f1', documents: [doc()] }), file({ id: 'f2', documents: [], consolidation_id: 'c9' })], + (f) => (f.id === 'f2' ? 'вне консолидации' : null), + ); + expect(rows[0].subtitle.endsWith('вне консолидации')).toBe(false); + expect(rows[1].subtitle.endsWith(' · вне консолидации')).toBe(true); + }); }); diff --git a/src/widgets/FilesDrawer/documentRows.ts b/src/widgets/FilesDrawer/documentRows.ts index 785392a..bfeef40 100644 --- a/src/widgets/FilesDrawer/documentRows.ts +++ b/src/widgets/FilesDrawer/documentRows.ts @@ -36,8 +36,9 @@ function documentRowStatus(status: DocumentStatus): RowStatus { } /** Файлы → строки таблицы: одна строка на документ, файл без документов — одна строка на файл. */ -export function buildDocumentRows(files: FileOut[]): DocumentRow[] { +export function buildDocumentRows(files: FileOut[], labelFor?: (f: FileOut) => string | null): DocumentRow[] { return files.flatMap((f): DocumentRow[] => { + const extra = labelFor?.(f) ?? null; const name = f.original_name ?? 'без имени'; const date = formatMoscowDate(f.uploaded_at); if (f.documents.length === 0) { @@ -48,7 +49,7 @@ export function buildDocumentRows(files: FileOut[]): DocumentRow[] { documentId: null, name, fileType: f.file_type, - subtitle: join([f.file_type, date]), + subtitle: join([f.file_type, date, extra]), period: '—', status: failed ? 'error' : 'processing', statusLabel: failed ? 'Ошибка' : 'Обработка', @@ -68,7 +69,7 @@ export function buildDocumentRows(files: FileOut[]): DocumentRow[] { documentId: d.id, name, fileType: f.file_type, - subtitle: join([typeText, f.file_type, date]), + subtitle: join([typeText, f.file_type, date, extra]), period: periodLabel(d.period_year, d.period_quarter, d.is_cumulative), periodCheck: d.period_check, counterpartyCheck: d.counterparty_check, diff --git a/src/widgets/FilesDrawer/useLegalEntityFiles.ts b/src/widgets/FilesDrawer/useLegalEntityFiles.ts index df6e399..c877f8e 100644 --- a/src/widgets/FilesDrawer/useLegalEntityFiles.ts +++ b/src/widgets/FilesDrawer/useLegalEntityFiles.ts @@ -6,8 +6,11 @@ import type { FileOut } from '../../types/api'; const POLL_MS = 5000; -/** Файлы ЮЛ; пока панель открыта и есть UPLOADED/PROCESSING — перечитываем раз в 5 с. */ -export function useLegalEntityFiles(legalEntityId: string | null, open: boolean) { +/** + * Файлы рабочего стола консолидации (`consolidationId`) или все файлы ЮЛ (`null`); + * пока панель открыта и есть UPLOADED/PROCESSING — перечитываем раз в 5 с. + */ +export function useLegalEntityFiles(legalEntityId: string | null, consolidationId: string | null, open: boolean) { const [files, setFiles] = useState([]); const [loading, setLoading] = useState(false); const requestRef = useRef(0); @@ -20,7 +23,7 @@ export function useLegalEntityFiles(legalEntityId: string | null, open: boolean) inFlightRef.current = true; setLoading(true); try { - const items = await listLegalEntityFiles(legalEntityId); + const items = await listLegalEntityFiles(legalEntityId, consolidationId); if (requestId === requestRef.current) { setFiles(items); errorShownRef.current = false; @@ -37,10 +40,11 @@ export function useLegalEntityFiles(legalEntityId: string | null, open: boolean) setLoading(false); } } - }, [legalEntityId]); + }, [legalEntityId, consolidationId]); + // смена режима (consolidationId) приходит через reload: он пересоздаётся и перезапускает этот эффект useEffect(() => { - // при смене ЮЛ / закрытии панели: старые ответы не применяем, файлы прежнего ЮЛ очищаем + // при смене ЮЛ, режима (консолидация/все) или закрытии панели: старые ответы не применяем, прежние файлы очищаем requestRef.current++; inFlightRef.current = false; errorShownRef.current = false; diff --git a/src/widgets/Sidebar/Sidebar.tsx b/src/widgets/Sidebar/Sidebar.tsx index 7e75df4..a615a1e 100644 --- a/src/widgets/Sidebar/Sidebar.tsx +++ b/src/widgets/Sidebar/Sidebar.tsx @@ -15,6 +15,8 @@ import { NavLink, useNavigate } from 'react-router-dom'; import { USERS_ADMIN } from '../../auth/permissions'; import { Logo } from '../../components/Logo'; import { createConsolidation } from '../../services/api/consolidations'; +import { getConsolidationsListThunk } from '../../store/slices/consolidations/thunks'; +import { useAppDispatch } from '../../store/store'; import { quarterLabel } from '../../utils/labels'; import { NewConsolidationModal } from '../NewConsoleModal/NewConsoleModal'; import styles from './styles.module.css'; @@ -165,6 +167,7 @@ const Section = memo(function Section({ export function Sidebar() { const navigate = useNavigate(); + const dispatch = useAppDispatch(); const [modalOpen, setModalOpen] = useState(false); const [isCollapsed, setIsCollapsed] = useState(false); @@ -218,6 +221,8 @@ export function Sidebar() { }} onOk={async (values) => { const card = await createConsolidation(values); + // перечитываем список: на странице группы он сам не обновится, и новая консолидация не станет текущей + await dispatch(getConsolidationsListThunk(values.company_group_id)); setModalOpen(false); message.success(`Создана консолидация ${values.year}, ${quarterLabel(values.quarter)}`); navigate(`/groups/${values.company_group_id}?version=${card.id}`);