From 92ba474ca92121ab90a8195dd38439a011e082d0 Mon Sep 17 00:00:00 2001
From: Denozordec <shatskiy.blog@gmail.com>
Date: Wed, 30 Sep 2026 10:24:35 +0700
Subject: [PATCH] =?UTF-8?q?fix(ui):=20=D0=B8=D1=81=D0=BA=D0=BB=D1=8E=D1=87?=
 =?UTF-8?q?=D0=B8=D1=82=D1=8C=20=D0=BA=D0=BE=D0=BD=D1=84=D0=BB=D0=B8=D0=BA?=
 =?UTF-8?q?=D1=82=D1=8B=20=D0=B2=D0=BE=D1=81=D1=81=D1=82=D0=B0=D0=BD=D0=BE?=
 =?UTF-8?q?=D0=B2=D0=BB=D0=B5=D0=BD=D0=B8=D1=8F=20Firewall?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/(main)/firewall/page.tsx            | 12 ++--
 docs/ux-ui-plan.md                      |  4 +-
 docs/ux-ui-progress.md                  | 10 +++-
 docs/ux-ui-stage-9.md                   |  4 ++
 tests/e2e/firewall-history-lock.spec.ts | 75 +++++++++++++++++++++++++
 tests/e2e/firewall-move.spec.ts         |  2 +
 6 files changed, 100 insertions(+), 7 deletions(-)
 create mode 100644 tests/e2e/firewall-history-lock.spec.ts

diff --git a/app/(main)/firewall/page.tsx b/app/(main)/firewall/page.tsx
index c93282a4..c6b0b011 100644
--- a/app/(main)/firewall/page.tsx
+++ b/app/(main)/firewall/page.tsx
@@ -2025,7 +2025,9 @@ function FirewallPageInner() {
   }, [isLive, historyServerId, apiFetch])
 
   const restoreRevision = useCallback(async (id: string) => {
-    if (!isLive || !historyServerId) return
+    if (!isLive || !historyServerId) throw new Error("Выберите устройство в режиме Live")
+    if (mutationLock.current) throw new Error("Дождитесь завершения текущей операции Firewall")
+    mutationLock.current = true
     setHistoryRestoring(true)
     try {
       const res = await apiFetch<{ ok: boolean }>(
@@ -2039,6 +2041,7 @@ function FirewallPageInner() {
     } catch (err) {
       throw err
     } finally {
+      mutationLock.current = false
       setHistoryRestoring(false)
     }
   }, [isLive, historyServerId, apiFetch, loadLive, loadRevisions])
@@ -2492,8 +2495,8 @@ function FirewallPageInner() {
               <Button
                 variant="outline"
                 size="sm"
-                onClick={() => { void loadLive() }}
-                disabled={!isLive || dataLoading}
+                onClick={() => { if (!mutationLock.current) void loadLive() }}
+                disabled={!isLive || dataLoading || busy || togglePending || deletePending || movePending || historyRestoring}
                 title={!isLive ? "Включите Live в источнике данных" : "Обновить правила с устройств"}
               >
                 <RefreshCwIcon className={cn("size-4", dataLoading && "animate-spin")} />
@@ -2503,10 +2506,11 @@ function FirewallPageInner() {
                 variant="outline"
                 size="sm"
                 onClick={() => {
+                  if (mutationLock.current) return
                   setHistoryOpen(true)
                   void loadRevisions()
                 }}
-                disabled={!isLive || !historyServerId || dataLoading}
+                disabled={!isLive || !historyServerId || dataLoading || busy || togglePending || deletePending || movePending || historyRestoring}
                 title={!historyServerId ? "Выберите сервер, чтобы смотреть историю" : "История версий и откат на CHR"}
               >
                 <HistoryIcon className="size-4" />
diff --git a/docs/ux-ui-plan.md b/docs/ux-ui-plan.md
index acbfed0c..68272a29 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.29 — IPsec/GRE/VXLAN и переключение Firewall); этапы 10–12 не начаты. Этап 11 разделён: 11А — пользователи, контейнеры, сертификаты, резервирование; 11Б — оповещения, сбор данных, интеграции, релизы, настройки.
+По журналу этапы 0–8 завершены в границах локальной приёмки; внешние ограничения сохраняются. Этап 9 начат (блоки 9.1–9.30 — IPsec/GRE/VXLAN и переключение Firewall); этапы 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. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. В 9.26 подтверждены включение/отключение address-list с общим замком переключений. В 9.27 удаление правил/address-list перенесено в диалог с подтверждением API и защитой повторов. В 9.28 создание/редактирование правил и address-list требуют подтверждения API, сохраняют ввод/ошибку и блокируют повторы/закрытие. В 9.29 перемещение Firewall требует ok=true, откатывает порядок при ошибке и использует общий замок. Далее — согласование восстановления истории с мутациями и гонки загрузки Firewall, затем Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми.
+1. Этап 9: карта разделов и схема сохранены; удаление, импорт, создание, переключение, контекст экспорта и выбор пира WireGuard исправлены в 9.1–9.7; в 9.8 проверено подтверждение восстановления. В 9.9 исправлены разметка и сохранение подтверждения при ошибке у всех шести потребителей. В 9.10 исправлены ошибки и поздние ответы загрузки истории WireGuard. В 9.11 подтверждение восстановления проверено у остальных пяти потребителей. В 9.12 обработаны ошибка/повтор/поздние ответы истории IPsec. В 9.13 подключены ошибка/повтор/защита поздних ответов остальных четырёх разделов и общая валидация ревизий всех шести. В 9.14 исправлены подтверждение инициализации IPsec, сохранение формы и защита повторной отправки. В 9.15 проверены создание/редактирование пользователей IPsec и сохранение формы. В 9.16 проверены получение/повторный экспорт сертификатов, поздние ответы и соответствие отображаемого пароля архиву. В 9.17 подтверждены ответы удаления четырёх типов объектов IPsec и исправлен диалог повторов. В 9.18 исправлено подтверждение сохранения peer IPsec и поведение формы. В 9.19 исправлено подтверждение удаления GRE и сохранение диалога при ошибке. В 9.20 исправлено подтверждение включения/выключения GRE и защита повторов. В 9.21 исправлены подтверждение создания/сохранения GRE и защита формы. В 9.22 исправлены контейнер ответа, первоначальные KPI и сохранение данных при ошибке VXLAN. В 9.23 backend сообщает частичные сбои VXLAN, UI сохраняет данные неответивших устройств и показывает неизвестное состояние. В 9.24 добавлены схемы payload и тесты гонок VXLAN. В 9.25 подтверждены включение/отключение правил Firewall и защита повторов. В 9.26 подтверждены включение/отключение address-list с общим замком переключений. В 9.27 удаление правил/address-list перенесено в диалог с подтверждением API и защитой повторов. В 9.28 создание/редактирование правил и address-list требуют подтверждения API, сохраняют ввод/ошибку и блокируют повторы/закрытие. В 9.29 перемещение Firewall требует ok=true, откатывает порядок при ошибке и использует общий замок. В 9.30 восстановление истории подключено к общему замку, ручное обновление и открытие истории заблокированы во время записи. Далее — схема/гонки загрузки Firewall, затем Filters; ограничения счётчика, свежести и записи VXLAN остаются открытыми; загрузка данных IPsec и общая приёмка остаются открытыми.
 2. Сохранять контекст устройства, введённые значения при ошибке и подтверждать результат операций по API; локальные моки проверяют клиентские сценарии, реальные сетевые операции требуют тестового окружения.
 3. В этапе 12 выполнить полный аудит доступности (скринридер, контраст), производительности и сквозную регрессию. Ограничения этапа 8 — удалённые серверы месячного каталога, отсутствие экспорта и автоматического переподключения SSE — остаются явно зафиксированными, без заявления о реализации.
 
diff --git a/docs/ux-ui-progress.md b/docs/ux-ui-progress.md
index 822c67f8..69d77c10 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.29 — IPsec/GRE/VXLAN и переключение Firewall |
+| 9 | VPN, Firewall и Filters | В работе: блоки 9.1–9.30 — IPsec/GRE/VXLAN и переключение Firewall |
 | 10 | Маршрутизация, диагностика, карта и терминал | Не начат |
 | 11А | Пользователи, контейнеры, сертификаты и резервные копии | Не начат |
 | 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Не начат |
 | 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат |
 
+## Блок 9.30 — общий замок истории Firewall (2026-09-30)
+
+- Восстановление истории захватывает mutationLock до ответа API, обновления правил и повторного чтения истории. Ошибка освобождает замок; при занятой операции обработчик отклоняет восстановление с понятным сообщением, не выдавая успешное завершение. Отсутствие Live/устройства также сообщает ошибку.
+- Кнопки «История» и «Обновить» disabled во время сохранения, удаления, переключения, переноса и восстановления. Их обработчики дополнительно проверяют синхронный замок. Уже запущенные запросы загрузки пока не отменяются и требуют защиты поколением запроса в следующем блоке.
+- Прошли 26 разных E2E в двух прогонах: 23 сценария операций Firewall и три профильных сценария истории. Новый сценарий проверяет двойной запрос, 403, освобождение замка, задержку чтения правил/истории после успеха, недоступность конфликтов и последующее сохранение. Первая попытка нового теста остановилась на неоднозначном alert-селекторе; после уточнения селектора профильный прогон прошёл. TypeScript прошёл; изменённые тесты lint-чистые, исходные 4 ошибки/2 предупреждения страницы не изменились относительно HEAD. Diff-check проверен перед коммитом. Последний production build — 9.21, повтор не выполнялся.
+- Далее — схема ответа и гонки загрузки Firewall: сохранять последние данные при ошибке, неизвестные первоначальные KPI, поздние ответы/смена backend и конфликт ранее запущенного чтения с мутацией. Затем Filters. Реальные RouterOS/JWT не проверены; этап 9 открыт.
+- Квота: 28% пятичасовой/4% недельной на старте, 31%/5% после проверок. Локальный коммит без push; чужие изменения сохранены.
+
 ## Блок 9.29 — перемещение Firewall (2026-09-30)
 
 - POST перемещения требует ok=true. Оптимистический порядок откатывается при HTTP-ошибке, пустом ответе и ok=false. Status/alert содержат контекст правила; повторный запрос очищает ошибку.
diff --git a/docs/ux-ui-stage-9.md b/docs/ux-ui-stage-9.md
index 45e0e4cc..6a2b2371 100644
--- a/docs/ux-ui-stage-9.md
+++ b/docs/ux-ui-stage-9.md
@@ -157,3 +157,7 @@ POST/PUT правил и address-list требуют ok=true. Ошибка со
 ## Блок 9.29 — перемещение Firewall (2026-09-30)
 
 POST требует ok=true, ошибка возвращает исходный порядок. Общий mutationLock предотвращает конфликтующие операции до обновления списка. Все 23 E2E и TypeScript прошли, новых замечаний lint нет. Далее — общий замок восстановления истории и схема/гонки загрузки Firewall, затем Filters. Реальные устройства не проверены; последний build — 9.21.
+
+## Блок 9.30 — общий замок истории Firewall (2026-09-30)
+
+Восстановление удерживает mutationLock до чтения правил и истории, ошибка освобождает замок. История/ручное обновление недоступны во время записи. Прошли 26 разных E2E в двух прогонах, TypeScript и lint тестов; исходные замечания страницы сохранены. Далее — схема/гонки загрузки Firewall, затем Filters. Реальные устройства не проверены; последний build — 9.21.
diff --git a/tests/e2e/firewall-history-lock.spec.ts b/tests/e2e/firewall-history-lock.spec.ts
new file mode 100644
index 00000000..3eab442d
--- /dev/null
+++ b/tests/e2e/firewall-history-lock.spec.ts
@@ -0,0 +1,75 @@
+import { test, expect } from "@playwright/test"
+
+test("Firewall: восстановление удерживает общий замок до обновления правил и истории", async ({ page }) => {
+  let attempts = 0
+  let revisionsReads = 0
+  let writes = 0
+  let release: (() => void) | undefined
+  let releaseRead: (() => void) | undefined
+  let releaseHistory: (() => void) | undefined
+  const gate = new Promise<void>(resolve => { release = resolve })
+  const readGate = new Promise<void>(resolve => { releaseRead = resolve })
+  const historyGate = new Promise<void>(resolve => { releaseHistory = resolve })
+  const rule = { id: "rule-1", rosId: "*R/1", serverId: "1", serverName: "Роутер", family: "ip", table: "filter", chain: "forward", action: "accept", proto: "tcp", src: "", dst: "", port: "443", iface: "", comment: "Правило тест", enabled: true, hits: 1 }
+  await page.route("**/health", route => route.fulfill({ json: { ok: true } }))
+  await page.route("**/api/**", async route => {
+    const path = new URL(route.request().url()).pathname
+    if (path.endsWith("/restore")) {
+      attempts++
+      expect(decodeURIComponent(path)).toBe("/api/firewall/revisions/rev/1/restore")
+      expect(route.request().postDataJSON()).toEqual({ serverId: "1" })
+      if (attempts === 1) { await gate; return route.fulfill({ status: 403, json: { error: "Восстановление запрещено тест" } }) }
+      return route.fulfill({ json: { ok: true } })
+    }
+    if (route.request().method() !== "GET") { writes++; return route.fulfill({ json: { ok: true } }) }
+    if (path === "/api/firewall/all" && attempts === 2) await readGate
+    if (path.endsWith("/revisions")) {
+      revisionsReads++
+      if (revisionsReads === 2) await historyGate
+    }
+    return route.fulfill({ json: path === "/api/auth/config" ? { required: false }
+      : path === "/api/firewall/all" ? { rules: [rule], addressLists: [] }
+      : path.endsWith("/revisions") ? { revisions: [{ id: "rev/1", serverId: "1", section: "firewall", source: "apply", fingerprint: "fixture", createdAt: "2026-09-29T00:00:00Z", note: null, itemCount: 1 }] }
+      : path === "/api/servers" ? [{ id: 1, name: "Роутер", host: "192.0.2.1", type: "exit-node", enabled: true, country: "RU", status: "online" }] : {} })
+  })
+  await page.goto("/firewall?server=1")
+  await expect(page.locator("tr[data-row-id]")).toHaveCount(1)
+  await page.getByRole("button", { name: "История", exact: true }).click()
+  const sheet = page.getByRole("dialog")
+  await sheet.getByRole("button", { name: "Откатить", exact: true }).click()
+  const dialog = page.getByRole("alertdialog")
+  const restore = dialog.getByRole("button", { name: "Откатить", exact: true })
+  await restore.evaluate(button => { (button as HTMLButtonElement).click(); (button as HTMLButtonElement).click() })
+  await expect.poll(() => attempts).toBe(1)
+  const locked = async () => {
+    await expect(page.locator("tr[data-row-id]").getByRole("switch", { includeHidden: true })).toBeDisabled()
+    await expect(page.getByRole("button", { name: "Обновить", exact: true, includeHidden: true })).toBeDisabled()
+    await expect(page.getByRole("button", { name: "История", exact: true, includeHidden: true })).toBeDisabled()
+    await expect(dialog.getByRole("button", { name: "Отмена" })).toBeDisabled()
+  }
+  await locked()
+  // Проверяем защиту обработчика открытия формы даже при программном клике за модальным слоем.
+  await page.getByRole("button", { name: "Новое правило", exact: true, includeHidden: true }).evaluate(button => (button as HTMLButtonElement).click())
+  expect(writes).toBe(0)
+  await page.keyboard.press("Escape")
+  await expect(dialog).toBeVisible()
+  release!()
+  await expect(dialog.getByRole("alert").filter({ hasText: "Восстановление запрещено тест" })).toBeVisible()
+  await expect(page.locator("tr[data-row-id]").getByRole("switch", { includeHidden: true })).toBeEnabled()
+  await restore.click()
+  await expect.poll(() => attempts).toBe(2)
+  await locked()
+  releaseRead!()
+  await expect.poll(() => revisionsReads).toBe(2)
+  await locked()
+  releaseHistory!()
+  await expect(dialog).toHaveCount(0)
+  await page.keyboard.press("Escape")
+  await expect(sheet).toHaveCount(0)
+  await expect(page.getByRole("button", { name: "Обновить", exact: true })).toBeEnabled()
+  await page.getByRole("button", { name: "Действия: forward accept", exact: true }).click()
+  await page.getByRole("menuitem", { name: "Редактировать", exact: true }).click()
+  await page.getByRole("dialog").getByRole("button", { name: "Сохранить", exact: true }).click()
+  await expect.poll(() => writes).toBe(1)
+  await expect(page.getByRole("dialog")).toHaveCount(0)
+})
diff --git a/tests/e2e/firewall-move.spec.ts b/tests/e2e/firewall-move.spec.ts
index 4338b93e..b5d5034d 100644
--- a/tests/e2e/firewall-move.spec.ts
+++ b/tests/e2e/firewall-move.spec.ts
@@ -47,6 +47,8 @@ for (const direction of ["down", "up"] as const) {
     await expect.poll(() => attempts).toBe(1)
     const waiting = page.getByRole("status").filter({ hasText: "Ожидание перемещения правила" })
     await expect(waiting).toBeVisible()
+    await expect(page.getByRole("button", { name: "История", exact: true })).toBeDisabled()
+    await expect(page.getByRole("button", { name: "Обновить", exact: true })).toBeDisabled()
     await expect(page.getByRole("button", { name: "Перетащите для изменения порядка строк", exact: true })).toHaveCount(0)
     for (const toggle of await rows.getByRole("switch").all()) await expect(toggle).toBeDisabled()
     await rows.first().getByRole("button", { name: "Действия: forward accept", exact: true }).click()