diff --git a/app/(main)/filters/page.tsx b/app/(main)/filters/page.tsx index b2fd1bd8..e3672849 100644 --- a/app/(main)/filters/page.tsx +++ b/app/(main)/filters/page.tsx @@ -1676,11 +1676,17 @@ function FiltersPage() { } const openEdit = (rule: FilterRule) => { if (mutationsLocked || applyLock.current) return + // The observed catalog can have new DB IDs after refresh. Recover only an + // explicit recursive reference, never reinterpret a named GRE interface. + const routes = recRoutesByServer[selectedServerId] ?? [] + const recursive = isRecursiveGatewayRef(rule.gatewayTunnelId) + ? routes.find(r => r.id === rule.gatewayTunnelId.slice(4)) ?? pickRecursiveRouteByGatewayHop(routes, rule.gateway) + : undefined setSheetInitial({ action: rule.action ?? "route", community: rule.community, gateway: rule.gateway, - gatewayTunnelId: rule.gatewayTunnelId, + gatewayTunnelId: recursive ? `rec:${recursive.id}` : rule.gatewayTunnelId, gatewayKind: isRecursiveGatewayRef(rule.gatewayTunnelId) ? "recursive" : "gre", description: rule.description, }) diff --git a/components/data-grids/filters-data-grid.tsx b/components/data-grids/filters-data-grid.tsx index bc1e4dfe..71728db8 100644 --- a/components/data-grids/filters-data-grid.tsx +++ b/components/data-grids/filters-data-grid.tsx @@ -77,7 +77,7 @@ function FilterGatewayCell({ ? recursiveRoutes.find((r) => r.id === rule.gatewayTunnelId.slice(4)) : undefined const recRowByHop = - !isBlackhole && !(rule.gatewayTunnelId ?? "").trim() && rule.gateway.trim() + !isBlackhole && !recRowByRef && (isRecRef || !(rule.gatewayTunnelId ?? "").trim()) && rule.gateway.trim() ? pickRecursiveRouteByGatewayHop(recursiveRoutes, rule.gateway) : undefined const recRow = recRowByRef ?? recRowByHop diff --git a/docs/ux-ui-progress.md b/docs/ux-ui-progress.md index 3151525a..88b65c6d 100644 --- a/docs/ux-ui-progress.md +++ b/docs/ux-ui-progress.md @@ -1,6 +1,6 @@ # UX/UI: журнал реализации -Актуальное состояние 2026-10-15: Production266 — с отдельного разрешения создан online-индекс retention; нагрузка БД существенно снизилась в двух повторных замерах. Карта исправлена локально (265), frontend-патч ещё не развёрнут. Проверка трафика и причин остальных тайм-аутов продолжается отдельно. Лабораторная приёмка264 завершена в документированных границах стенда:1396 основных UI и221 context-сценарий;120 live default states30 экранов. Runtime стенда backend258/frontend261. Непроверенные внешние группы сохранены в итоговом акте. Production v2.0.0 подтверждён через UI; [описание релиза](releases/2.0.0.md). +Актуальное состояние 2026-10-15: Production268 — локально исправлено отображение рекурсивных шлюзов Filters после смены ID каталога;6 профильных E2E/TypeScript passed. Карта и миграция retention опубликованы в main; исправлен CI PostgreSQL, backend-test2467/8417 success. Установка этих frontend-исправлений в production ещё не проверена. Online-индекс266 применён отдельно с разрешения владельца; нагрузка БД снизилась в двух повторных замерах. Проверка трафика и причин остальных тайм-аутов продолжается отдельно. Лабораторная приёмка264 завершена в документированных границах стенда:1396 основных UI и221 context-сценарий;120 live default states30 экранов. Runtime стенда backend258/frontend261. Непроверенные внешние группы сохранены в итоговом акте. Production v2.0.0 подтверждён через UI; [описание релиза](releases/2.0.0.md). Общий план: [UX/UI по этапам и пятичасовым окнам](ux-ui-plan.md). План задаёт объём, этот журнал фиксирует выполненные блоки и ограничения. Записи расположены от новых к старым; подробная история этапа12 также сохранена в [ux-ui-stage-12.md](ux-ui-stage-12.md). @@ -38,11 +38,17 @@ | 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | Завершён в границах локальной приёмки 2026-10-03; ограничения и evidence в11Б.7 | | 12 | Итоговая доступность, адаптивность, производительность и регрессия | Завершён2026-10-15 в границах предоставленного стенда:1396 UI/221 context,120 live states30 экранов, production builds/ReUI checklist и реальные интеграции. Ограничения — в [акте](ux-ui-acceptance-candidate.md); production-выпуск не выполнялся. | +## Production268 — рекурсивные шлюзы Filters после обновления каталога (2026-10-15) + +По production-скриншоту Filters показывает «рекурсивный маршрут (нет строки в списке)», хотя каталог формы содержит шлюз. Найден механизм: legacy GET /recursive-routes пересоздаёт строки recursive_routes, а правила могут сохранять прежние rec:id; таблица допускала поиск по hop только при пустой ссылке. Для явной рекурсивной ссылки с отсутствующим ID теперь используется уже имеющийся выбор активного маршрута по dst-address/hop и минимальной distance в каталоге выбранного устройства. Редактирование восстанавливает актуальный rec:id и выделение карточки. Именованный GRE не переопределяется по совпавшему IP; при реальном отсутствии маршрута предупреждение сохранено. Точная ссылка по ID сохраняет приоритет. + +6 E2E passed: две новые регрессии и существующие проверки gateway failure/save. Новый сценарий проверяет отображение, выделение, отсутствие записи до подтверждения и payload с новым rec:id; второй — настоящий orphan и GRE с совпавшим IP. TypeScript passed; scoped ESLint0 errors/11 прежних unused-import warnings в filters/page.tsx. Непривилегированный первый запуск E2E не поднял доступный локальный сервер и остановлен; проверки выполнены повторно с разрешением запуска локального сервера/браузера. Backend и production-конфигурация не менялись; механизм пересоздания cache ID этим frontend-патчем не перерабатывается. Исправление сохранено локально, без push/deploy. Следующий шаг — опубликовать этот патч и проверить production Filters после обновления frontend. + ## Production267 — подключение PostgreSQL в CI (2026-10-15) После push main backend-test2466/8410 завершился ECONNREFUSED: PostgreSQL публиковался на loopback Docker-хоста, но тестовый процесс работал в отдельном контейнере Gitea runner. Прежние SQL-тесты пропускали подключение; новый обязательный retention-тест выявил проблему. PostgreSQL теперь присоединяется к сети контейнера задания и доступен по уникальному DNS alias; для host runner сохранён вариант с назначенным Docker портом. Удаление общего mm-pg исключено, cleanup затрагивает только контейнер текущего запуска. Перед тестами выполняется SELECT1 из процесса задания; REQUIRE_TEST_POSTGRES=1 запрещает скрытый skip при недоступности/ошибке bootstrap. -Workflow YAML/scoped ESLint/diff check прошли. Проверены оба режима helper на недоступном localhost-порту: локально false/skip, в обязательном режиме исходная ошибка подключения выходит наружу. Полная CI-проверка сети и SQL-профилей должна подтвердиться следующим запуском; успешный тест retention на реальном тестовом PostgreSQL описан в266. Production этим блоком напрямую не изменялся. +Workflow YAML/scoped ESLint/diff check прошли. Проверены оба режима helper на недоступном localhost-порту: локально false/skip, в обязательном режиме исходная ошибка подключения выходит наружу. После push042407f0 [backend-test2467/8417](https://git.shx.one/denozord/MikrotikManager/actions/runs/2467/jobs/8417) завершён success, включая retention20тыс. строк; обязательное подключение из тестового процесса подтверждено. Production этим блоком напрямую не изменялся. ## Production266 — online-индекс retention и проверка нагрузки (2026-10-15) diff --git a/tests/e2e/filters-recursive-reference.spec.ts b/tests/e2e/filters-recursive-reference.spec.ts new file mode 100644 index 00000000..62eb0e62 --- /dev/null +++ b/tests/e2e/filters-recursive-reference.spec.ts @@ -0,0 +1,58 @@ +import { test, expect } from "@playwright/test" + +test("Filters: обновлённый ID рекурсивного маршрута не скрывает шлюз и выбор в форме", async ({ page }) => { + const rule = { id: "rule-1", community: "65001:120", gateway: "10.9.9.2", gatewayTunnelId: "rec:42", action: "route", description: "Маршрут" } + let writes = 0 + let savedGatewayRef: string | undefined + await page.route("**/health", r => r.fulfill({ json: { ok: true } })) + await page.route("**/api/**", r => { + const path = new URL(r.request().url()).pathname + if (r.request().method() !== "GET") { + writes++ + const payload = r.request().postDataJSON() + savedGatewayRef = payload.rules[0].gatewayTunnelId + return r.fulfill({ json: { ok: true, rules: payload.rules } }) + } + return r.fulfill({ json: path === "/api/filters/rules" ? { rulesets: [{ serverId: "1", rules: [rule] }], live: true } + : path === "/api/servers" ? [{ id: 1, name: "Роутер", host: "192.0.2.1", type: "jump-host", enabled: true, country: "RU", status: "online", latency: 1 }] + : path === "/api/filters/gre-tunnels" ? { tunnels: [] } + : path === "/api/recursive-routes" ? { routes: [{ id: "99", dstAddress: "10.9.9.2/32", gateway: "192.0.2.2%gre-test", distance: 1, routingTable: "main", comment: "Рекурсивный тест", disabled: false }], stale: false } + : path === "/api/auth/config" ? { required: false } : {} }) + }) + await page.goto("/filters") + const row = page.locator("tr[data-row-id]").first() + await expect(row).toContainText("10.9.9.2/32") + await expect(row).not.toContainText("нет строки в списке") + await row.getByRole("button", { name: "Редактировать правило", exact: true }).click() + const dialog = page.getByRole("dialog") + await expect(dialog).toBeVisible() + await expect(dialog.getByRole("tab", { name: /Рекурсив/ })).toHaveAttribute("aria-selected", "true") + await expect(dialog.getByRole("button", { name: /10\.9\.9\.2\/32/ })).toHaveClass(/border-primary/) + await expect(dialog.getByText("Выберите рекурсивный маршрут", { exact: true })).toHaveCount(0) + expect(writes).toBe(0) + await dialog.getByRole("button", { name: "Сохранить", exact: true }).click() + await expect(dialog).toHaveCount(0) + expect(writes).toBe(1) + expect(savedGatewayRef).toBe("rec:99") +}) + +test("Filters: отсутствующий маршрут и именованный GRE не подменяются по совпавшему IP", async ({ page }) => { + const rules = [ + { id: "orphan", community: "65001:120", gateway: "10.9.9.3", gatewayTunnelId: "rec:42", action: "route", description: "Нет маршрута" }, + { id: "gre", community: "65001:130", gateway: "10.9.9.2", gatewayTunnelId: "gre-test", action: "route", description: "Это GRE" }, + ] + await page.route("**/health", r => r.fulfill({ json: { ok: true } })) + await page.route("**/api/**", r => { + const path = new URL(r.request().url()).pathname + return r.fulfill({ json: path === "/api/filters/rules" ? { rulesets: [{ serverId: "1", rules }], live: true } + : path === "/api/servers" ? [{ id: 1, name: "Роутер", host: "192.0.2.1", type: "jump-host", enabled: true, country: "RU", status: "online", latency: 1 }] + : path === "/api/filters/gre-tunnels" ? { tunnels: [] } + : path === "/api/recursive-routes" ? { routes: [{ id: "99", dstAddress: "10.9.9.2/32", gateway: "192.0.2.2", distance: 1, routingTable: "main", comment: "", disabled: false }], stale: false } + : path === "/api/auth/config" ? { required: false } : {} }) + }) + await page.goto("/filters") + const rows = page.locator("tr[data-row-id]") + await expect(rows).toHaveCount(2) + await expect(rows.nth(0)).toContainText("нет строки в списке") + await expect(rows.nth(1)).not.toContainText("recursive") +})