From 3dd4ad34eeead194a84d8fd7e761e6fe043d80be Mon Sep 17 00:00:00 2001 From: Denozordec Date: Tue, 29 Sep 2026 13:58:26 +0700 Subject: [PATCH] =?UTF-8?q?fix(ui):=20=D0=BF=D1=80=D0=BE=D0=B2=D0=B5=D1=80?= =?UTF-8?q?=D1=8F=D1=82=D1=8C=20=D0=BF=D0=BE=D0=B4=D1=82=D0=B2=D0=B5=D1=80?= =?UTF-8?q?=D0=B6=D0=B4=D0=B5=D0=BD=D0=B8=D0=B5=20=D0=B2=D0=BE=D1=81=D1=81?= =?UTF-8?q?=D1=82=D0=B0=D0=BD=D0=BE=D0=B2=D0=BB=D0=B5=D0=BD=D0=B8=D1=8F=20?= =?UTF-8?q?=D1=81=D0=B5=D1=82=D0=B5=D0=B2=D1=8B=D1=85=20=D0=BD=D0=B0=D1=81?= =?UTF-8?q?=D1=82=D1=80=D0=BE=D0=B5=D0=BA?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- app/(main)/filters/page.tsx | 1 + app/(main)/firewall/page.tsx | 3 ++- app/(main)/gre/page.tsx | 3 ++- app/(main)/ipsec/page.tsx | 3 ++- app/(main)/recursive-routes/page.tsx | 1 + docs/ux-ui-plan.md | 4 ++-- docs/ux-ui-progress.md | 10 +++++++++- docs/ux-ui-stage-9.md | 6 +++++- tests/e2e/config-history.spec.ts | 10 +++++++++- 9 files changed, 33 insertions(+), 8 deletions(-) diff --git a/app/(main)/filters/page.tsx b/app/(main)/filters/page.tsx index cca9cece..1a7c3b71 100644 --- a/app/(main)/filters/page.tsx +++ b/app/(main)/filters/page.tsx @@ -1532,6 +1532,7 @@ export default function FiltersPage() { `/api/filters/revisions/${encodeURIComponent(id)}/restore`, { method: "POST", body: JSON.stringify({ serverId: selectedServerId }) }, ) + if (res?.ok !== true) throw new Error("Сервер не подтвердил восстановление") if (res.rules) { setRulesets((p) => p.map((rs) => rs.serverId === selectedServerId ? { ...rs, rules: res.rules ?? [] } : rs)) } else { diff --git a/app/(main)/firewall/page.tsx b/app/(main)/firewall/page.tsx index 63bee655..c6c9ef7d 100644 --- a/app/(main)/firewall/page.tsx +++ b/app/(main)/firewall/page.tsx @@ -1998,10 +1998,11 @@ function FirewallPageInner() { if (!isLive || !historyServerId) return setHistoryRestoring(true) try { - await apiFetch( + const res = await apiFetch<{ ok: boolean }>( `/api/firewall/revisions/${encodeURIComponent(id)}/restore`, { method: "POST", body: JSON.stringify({ serverId: historyServerId }) }, ) + if (res?.ok !== true) throw new Error("Сервер не подтвердил восстановление") toast.success("Версия применена на роутер") await loadLive() await loadRevisions() diff --git a/app/(main)/gre/page.tsx b/app/(main)/gre/page.tsx index 2ceaabe1..405c3da6 100644 --- a/app/(main)/gre/page.tsx +++ b/app/(main)/gre/page.tsx @@ -393,10 +393,11 @@ function GrePageContent({ selectedServerId, selectedTunnelId, allTunnelsHref, on if (!isLive || !historyServerId) return setHistoryRestoring(true) try { - await apiFetch( + const res = await apiFetch<{ ok: boolean }>( `/api/gre/revisions/${encodeURIComponent(id)}/restore`, { method: "POST", body: JSON.stringify({ serverId: historyServerId }) }, ) + if (res?.ok !== true) throw new Error("Сервер не подтвердил восстановление") toast.success("Версия применена на роутер") await loadLive() await loadRevisions() diff --git a/app/(main)/ipsec/page.tsx b/app/(main)/ipsec/page.tsx index 524b28dd..ff94327a 100644 --- a/app/(main)/ipsec/page.tsx +++ b/app/(main)/ipsec/page.tsx @@ -266,7 +266,8 @@ export default function IpsecPage() { if (!isLive || !historyServerId) return setHistoryRestoring(true) try { - await restoreIpsecRevision(backendUrl, id, historyServerId) + const res = await restoreIpsecRevision(backendUrl, id, historyServerId) + if (res?.ok !== true) throw new Error("Сервер не подтвердил восстановление") toast.success("Версия применена на роутер") await loadLive() await loadRevisions() diff --git a/app/(main)/recursive-routes/page.tsx b/app/(main)/recursive-routes/page.tsx index 46ba27a9..3ee2aa32 100644 --- a/app/(main)/recursive-routes/page.tsx +++ b/app/(main)/recursive-routes/page.tsx @@ -496,6 +496,7 @@ export default function RecursiveRoutesPage() { `/api/recursive-routes/revisions/${encodeURIComponent(id)}/restore`, { method: "POST", body: JSON.stringify({ serverId: selectedServerId }) }, ) + if (res?.ok !== true) throw new Error("Сервер не подтвердил восстановление") setRows(res.routes ?? []) setLiveStale(false) toast.success("Версия применена на роутер") diff --git a/docs/ux-ui-plan.md b/docs/ux-ui-plan.md index 1f4a7560..19501d88 100644 --- a/docs/ux-ui-plan.md +++ b/docs/ux-ui-plan.md @@ -80,7 +80,7 @@ flowchart LR ## Текущая точка продолжения -По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.10 — операции и история WireGuard); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки. +По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.11 — история и подтверждение восстановления); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки. Подробные планы: [этап 6](ux-ui-stage-6.md), [этап 7](ux-ui-stage-7.md), [этап 8 и итоговая матрица](ux-ui-stage-8.md), [этап 9](ux-ui-stage-9.md). @@ -88,7 +88,7 @@ flowchart LR Ближайший оставшийся объём: -1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. Далее — подтверждение восстановления и состояния истории остальных разделов; затем IPsec/GRE/VXLAN/Firewall/Filters. +1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. Далее — их состояния загрузки истории: ошибка, повтор и поздние ответы; затем IPsec/GRE/VXLAN/Firewall/Filters. 2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения. 3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации. diff --git a/docs/ux-ui-progress.md b/docs/ux-ui-progress.md index 661bda9f..4fd33728 100644 --- a/docs/ux-ui-progress.md +++ b/docs/ux-ui-progress.md @@ -25,12 +25,20 @@ | 6 | Навигация и дашборд | Завершён 2026-09-20 в границах локальной приёмки; ограничения ниже | | 7 | Серверы и мониторинг: детали, формы, URL-контекст, права | Завершён в границах локальной приёмки 2026-09-23; внешние ограничения ниже | | 8 | Трафик и статистика | Завершён 2026-09-28 в границах локальной приёмки; ограничения в блоке 8.13 | -| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.10 — операции и загрузка истории WireGuard | +| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.11 — история и подтверждение восстановления | | 10 | Маршрутизация, диагностика, карта и терминал | Не начат | | 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат | | 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат | | 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат | +## Блок 9.11 — подтверждение восстановления остальных разделов (2026-09-29) + +- IPsec, GRE, Firewall, Filters и рекурсивные маршруты проверяют ok=true до изменения таблиц, обновления данных и сообщения успеха. Контракты всех пяти backend-маршрутов проверены по коду: они возвращают такое подтверждение. WireGuard уже использовал эту проверку. +- При ok=false, пустом/неподтверждённом ответе выбрасывается ошибка в общий диалог; выбранная ревизия сохраняется для повтора. Сетевые правила и композиция ReUI не изменены; получен audit checklist. +- Расширенные 6 E2E истории прошли: 403 с задержкой/двойным нажатием, ok=false, пустой ответ, затем успех. Lint теста, TypeScript и diff-check прошли. Диагностики пяти страниц сравнены с HEAD: прежние 7 ошибок и 31 предупреждение сохраняются, новых нет; единственное текстовое отличие Firewall — сдвиг строки прежнего set-state-in-effect с 2042 на 2043. Полный lint страниц остаётся красным по причинам блока 9.9. +- Полная регрессия/build не повторялись для этих условий; последний production build — 9.9. Проверка ok не является полной валидацией вложенных rules/routes; реальный RouterOS/JWT и транзакционность восстановления не проверены. +- Далее — состояния загрузки истории IPsec/GRE/Firewall/Filters/рекурсивных маршрутов: ошибка вместо пустого списка, повтор и поздние ответы. На контрольной точке использовано 80% пятичасового и 75% недельного лимита. Этап 9 открыт; локальный коммит без push. + ## Блок 9.10 — состояния загрузки истории WireGuard (2026-09-29) - Ошибка истории больше не подменяется пустым списком. ConfigHistorySheet получил необязательные loadError/onRetry; WireGuard показывает ошибку через role=alert с повтором. Пустое состояние доступно только после успешного пустого ответа. Во время ошибки прежние ревизии не предлагаются для отката; данные остаются в состоянии, но скрыты до успешной загрузки. diff --git a/docs/ux-ui-stage-9.md b/docs/ux-ui-stage-9.md index b1908272..e668a0da 100644 --- a/docs/ux-ui-stage-9.md +++ b/docs/ux-ui-stage-9.md @@ -1,6 +1,6 @@ # Этап 9: VPN и сетевые операции -Статус: начат 2026-09-28, выполнены блоки 9.1–9.10. Основа — существующие ReUI PRO/base-nova, Frame, ServerTileRail, DataGrid и формы. Правила применения сетевой конфигурации не меняем. +Статус: начат 2026-09-28, выполнены блоки 9.1–9.11. Основа — существующие ReUI PRO/base-nova, Frame, ServerTileRail, DataGrid и формы. Правила применения сетевой конфигурации не меняем. ## Схема и референсы принятого плана @@ -81,3 +81,7 @@ ## Блок 9.10 — загрузка истории WireGuard (2026-09-29) Ошибка отделена от пустого списка и имеет повтор; ревизии проверяются по схеме и выбранному серверу. Закрытие/размонтирование и новое поколение запроса исключают поздние результат/ошибку/finally. Три новых E2E, lint и TypeScript прошли; итог регрессии в журнале. Остальные страницы пока не подключены к новым loadError/onRetry. Далее — подтверждение восстановления остальных разделов и их состояния истории. + +## Блок 9.11 — подтверждение восстановления остальных разделов (2026-09-29) + +IPsec/GRE/Firewall/Filters/рекурсивные маршруты требуют ok=true до применения ответа в UI; WireGuard уже проверял подтверждение. 6 расширенных сценариев истории прошли (403, ok=false, пустой ответ, успех), как и TypeScript/lint теста/diff-check. Исходные ошибки lint страниц не изменились. Далее — ошибка/повтор/поздние ответы загрузки истории этих пяти разделов. Полная валидация rules/routes и реальная операция RouterOS остаются вне этого блока. diff --git a/tests/e2e/config-history.spec.ts b/tests/e2e/config-history.spec.ts index 2ee55f0f..4922e58b 100644 --- a/tests/e2e/config-history.spec.ts +++ b/tests/e2e/config-history.spec.ts @@ -19,6 +19,8 @@ for (const section of ["wireguard", "ipsec", "gre", "firewall", "filters", "recu await gate return route.fulfill({ status: 403, json: { error: "Восстановление запрещено тест" } }) } + if (attempts === 2) return route.fulfill({ json: { ok: false, routes: [], rules: [] } }) + if (attempts === 3) return route.fulfill({ json: {} }) return route.fulfill({ json: { ok: true, routes: [], rules: [] } }) } const json = path.endsWith("/revisions") ? { revisions: [{ id: "rev/1", serverId: "1", section, source: "apply", fingerprint: "fixture", createdAt: "2026-09-29T00:00:00Z", note: null, itemCount: 1 }] } @@ -49,9 +51,15 @@ for (const section of ["wireguard", "ipsec", "gre", "firewall", "filters", "recu release!() await expect(dialog.getByRole("alert").filter({ hasText: "Восстановление запрещено тест" })).toBeVisible() await expect(dialog.getByRole("button", { name: "Откатить", exact: true })).toBeEnabled() + for (let i = 0; i < 2; i++) { + await dialog.getByRole("button", { name: "Откатить", exact: true }).click() + await expect.poll(() => attempts).toBe(i + 2) + await expect(dialog.getByRole("alert").filter({ hasText: "Сервер не подтвердил восстановление" })).toBeVisible() + await expect(page.getByText("Версия применена на роутер", { exact: true })).toHaveCount(0) + } await dialog.getByRole("button", { name: "Откатить", exact: true }).click() await expect(dialog).toHaveCount(0) - expect(attempts).toBe(2) + expect(attempts).toBe(4) await sheet.getByRole("button", { name: "Откатить", exact: true }).click() await expect(dialog.getByText("Восстановление запрещено тест")).toHaveCount(0) await page.keyboard.press("Escape")