fix(ui): перенести таблицу сетевых фильтров на ReUI v9
This commit is contained in:
1 parent
d47f7ab162
commit
45530cdb99
3 files changed
+55
-11
No files matched your search
@@ -1,23 +1,22 @@
|
|||||||
"use client"
|
"use client"
|
||||||
|
|
||||||
import { useMemo, useState } from "react"
|
import { useMemo, useState } from "react"
|
||||||
|
import { dataGridFeatures, type DataGridFeatures } from "@/components/reui/data-grid"
|
||||||
import {
|
import {
|
||||||
type ColumnDef,
|
type ColumnDef,
|
||||||
getCoreRowModel,
|
useTable,
|
||||||
getSortedRowModel,
|
} from "@tanstack/react-table"
|
||||||
useReactTable,
|
|
||||||
} from "@tanstack/react-table-v8"
|
|
||||||
import type { FilterRule, GreTunnel, Server } from "@/lib/data"
|
import type { FilterRule, GreTunnel, Server } from "@/lib/data"
|
||||||
import { Flag } from "@/components/flag"
|
import { Flag } from "@/components/flag"
|
||||||
import { cn } from "@/lib/utils"
|
import { cn } from "@/lib/utils"
|
||||||
import { Button } from "@/components/ui/button"
|
import { Button } from "@/components/ui/button"
|
||||||
import { DataGridShell } from "@/components/data-grids/shared/legacy-data-grid-shell"
|
import { DataGridShell } from "@/components/data-grids/shared/data-grid-shell"
|
||||||
import {
|
import {
|
||||||
DATA_GRID_CELL_PAD,
|
DATA_GRID_CELL_PAD,
|
||||||
DATA_GRID_CELL_PAD_FIRST,
|
DATA_GRID_CELL_PAD_FIRST,
|
||||||
DATA_GRID_CELL_PAD_LAST,
|
DATA_GRID_CELL_PAD_LAST,
|
||||||
} from "@/components/data-grids/shared/legacy-data-grid-layout"
|
} from "@/components/data-grids/shared/data-grid-layout"
|
||||||
import { DataGridSortHeader } from "@/components/data-grids/shared/legacy-data-grid-sort-header"
|
import { DataGridSortHeader } from "@/components/data-grids/shared/data-grid-sort-header"
|
||||||
import { EmptyState } from "@/components/empty-state"
|
import { EmptyState } from "@/components/empty-state"
|
||||||
import {
|
import {
|
||||||
AlertCircleIcon,
|
AlertCircleIcon,
|
||||||
@@ -210,7 +209,7 @@ function FiltersDataGrid({
|
|||||||
onMoveUp,
|
onMoveUp,
|
||||||
onMoveDown,
|
onMoveDown,
|
||||||
}: FiltersDataGridProps) {
|
}: FiltersDataGridProps) {
|
||||||
const columns = useMemo<ColumnDef<FilterRule>[]>(
|
const columns = useMemo<ColumnDef<DataGridFeatures, FilterRule>[]>(
|
||||||
() => [
|
() => [
|
||||||
{
|
{
|
||||||
id: "priority",
|
id: "priority",
|
||||||
@@ -242,6 +241,7 @@ function FiltersDataGrid({
|
|||||||
<button
|
<button
|
||||||
type="button"
|
type="button"
|
||||||
onClick={() => onMoveUp(row.original.id)}
|
onClick={() => onMoveUp(row.original.id)}
|
||||||
|
aria-label={`Поднять правило ${row.original.community}`}
|
||||||
disabled={row.index === 0}
|
disabled={row.index === 0}
|
||||||
className="text-muted-foreground/50 hover:text-foreground disabled:opacity-20"
|
className="text-muted-foreground/50 hover:text-foreground disabled:opacity-20"
|
||||||
>
|
>
|
||||||
@@ -250,6 +250,7 @@ function FiltersDataGrid({
|
|||||||
<button
|
<button
|
||||||
type="button"
|
type="button"
|
||||||
onClick={() => onMoveDown(row.original.id)}
|
onClick={() => onMoveDown(row.original.id)}
|
||||||
|
aria-label={`Опустить правило ${row.original.community}`}
|
||||||
disabled={isLast}
|
disabled={isLast}
|
||||||
className="text-muted-foreground/50 hover:text-foreground disabled:opacity-20"
|
className="text-muted-foreground/50 hover:text-foreground disabled:opacity-20"
|
||||||
>
|
>
|
||||||
@@ -349,11 +350,11 @@ function FiltersDataGrid({
|
|||||||
],
|
],
|
||||||
)
|
)
|
||||||
|
|
||||||
const table = useReactTable({
|
const table = useTable({
|
||||||
|
features: dataGridFeatures,
|
||||||
|
manualPagination: true,
|
||||||
data: rules,
|
data: rules,
|
||||||
columns,
|
columns,
|
||||||
getCoreRowModel: getCoreRowModel(),
|
|
||||||
getSortedRowModel: getSortedRowModel(),
|
|
||||||
getRowId: (row) => row.id,
|
getRowId: (row) => row.id,
|
||||||
})
|
})
|
||||||
|
|
||||||
|
|||||||
@@ -260,6 +260,14 @@ ReUI checklist получен; сохранены публичный API, ком
|
|||||||
|
|
||||||
Статические проверки блока 5: TypeScript прошёл; lint четырёх таблиц, фикстур и доменной модели — без замечаний. Расширенный lint `app/(main)/backups/page.tsx` обнаружил исходную ошибку `react-hooks/set-state-in-effect` на `void loadLive()` в эффекте (подтверждено в HEAD до правок). Эта ошибка не исправлялась в миграции; устранить при работе над состояниями данных на этапе 5. Не считать lint всей страницы успешным.
|
Статические проверки блока 5: TypeScript прошёл; lint четырёх таблиц, фикстур и доменной модели — без замечаний. Расширенный lint `app/(main)/backups/page.tsx` обнаружил исходную ошибку `react-hooks/set-state-in-effect` на `void loadLive()` в эффекте (подтверждено в HEAD до правок). Эта ошибка не исправлялась в миграции; устранить при работе над состояниями данных на этапе 5. Не считать lint всей страницы успешным.
|
||||||
|
|
||||||
|
## Этап 4 — блок 6: таблица сетевых Filters (2026-09-14)
|
||||||
|
|
||||||
|
`filters-data-grid` перенесён на установленную ReUI v9-основу с полным списком (`manualPagination: true`). Сохранены props, ID, обработчики и действующее отключение сортировки в live-режиме. Кнопки поднятия/опускания правила получили доступные подписи с community. Это миграция таблицы сетевых правил; старый компонент построения Filters/FilterQuery этим блоком не затронут.
|
||||||
|
|
||||||
|
TypeScript и целевой ESLint прошли. Браузерный сценарий `tests/e2e/network-filters-table.spec.ts` подтвердил 12 строк, исходный порядок и отсутствие кнопки сортировки в live-режиме, блокировку перемещения за границы, поиск/пустой результат/сброс. Первый вариант теста ошибочно ожидал доступную сортировку; ожидание исправлено под существующий контракт страницы. HTTP-моки не подтверждают перестановку и применение правил к RouterOS. Применён ранее полученный ReUI checklist; профильная приёмка операций и доступности остаётся этапу 9.
|
||||||
|
|
||||||
|
Квота: на старте использовано 64%, после статических проверок — 78% (остаток 22%). Новые блоки не начинать в резерве. Firewall не переносился: `firewall-rules-data-grid` использует DnD и требует чтения нового API и профильного браузерного теста перетаскивания с проверкой порядка. Следующий блок после проверки квоты — обе Firewall-таблицы; далее восемь таблиц диагностики/сборщика, общие legacy-потребители, старые Filters и удаление v8. Этап 4 остаётся открытым, ветку не выпускать. Сохранить локальный коммит без push.
|
||||||
|
|
||||||
## Карта миграции (исходная инвентаризация этапа 0)
|
## Карта миграции (исходная инвентаризация этапа 0)
|
||||||
|
|
||||||
Текущая связка: @tanstack/react-table ^8.21.3, @tanstack/react-virtual ^3.14.4, base-nova. Актуальный ReUI MCP ранее подтвердил useTable/dataGridFeatures для Data Grid и дерево FilterQuery вместо filters/onChange для Filters. Перед установкой перепроверить совместимые версии, сохранить семантику существующих фильтров (простые условия → AND) и закрепить lockfile.
|
Текущая связка: @tanstack/react-table ^8.21.3, @tanstack/react-virtual ^3.14.4, base-nova. Актуальный ReUI MCP ранее подтвердил useTable/dataGridFeatures для Data Grid и дерево FilterQuery вместо filters/onChange для Filters. Перед установкой перепроверить совместимые версии, сохранить семантику существующих фильтров (простые условия → AND) и закрепить lockfile.
|
||||||
|
|||||||
@@ -0,0 +1,35 @@
|
|||||||
|
import { test, expect } from "@playwright/test"
|
||||||
|
import type { FilterRule } from "../../lib/data"
|
||||||
|
|
||||||
|
test("Сетевые Filters: 12 правил, сохранение порядка, поиск и сброс", async ({ page }) => {
|
||||||
|
const rules: FilterRule[] = Array.from({ length: 12 }, (_, i) => ({
|
||||||
|
id: String(i + 1), community: `65001:${i + 1}`, gateway: "", gatewayTunnelId: "",
|
||||||
|
action: "blackhole", description: `Правило ${i + 1}`,
|
||||||
|
}))
|
||||||
|
const errors: string[] = []
|
||||||
|
page.on("pageerror", e => errors.push(e.message))
|
||||||
|
await page.route("**/health", route => route.fulfill({ json: { ok: true } }))
|
||||||
|
await page.route("**/api/**", route => {
|
||||||
|
const path = new URL(route.request().url()).pathname
|
||||||
|
return route.fulfill({ json: path === "/api/filters/rules" ? { rulesets: [{ serverId: "1", rules }], greTunnels: [], live: true }
|
||||||
|
: path === "/api/servers" ? [{ id: 1, name: "Тестовый роутер", host: "192.0.2.1", type: "exit-node", site: "TEST", country: "RU", asn: "", enabled: true, status: "online", latency: 1 }]
|
||||||
|
: path === "/api/auth/config" ? { required: false }
|
||||||
|
: path === "/api/filters/gre-tunnels" ? { tunnels: [] }
|
||||||
|
: path === "/api/recursive-routes" ? { routes: [] } : {} })
|
||||||
|
})
|
||||||
|
await page.goto("/filters")
|
||||||
|
const rows = page.locator("tr[data-row-id]")
|
||||||
|
await expect(rows).toHaveCount(12)
|
||||||
|
await expect(page.getByRole("button", { name: "Поднять правило 65001:1", exact: true })).toBeDisabled()
|
||||||
|
await expect(page.getByRole("button", { name: "Опустить правило 65001:12", exact: true })).toBeDisabled()
|
||||||
|
await expect(page.getByRole("button", { name: "Community", exact: true })).toHaveCount(0)
|
||||||
|
await expect(rows.first()).toContainText("65001:1")
|
||||||
|
await expect(rows.last()).toContainText("65001:12")
|
||||||
|
await page.getByPlaceholder("Community, gateway, описание…").fill("Правило 12")
|
||||||
|
await expect(rows).toHaveCount(1)
|
||||||
|
await page.getByPlaceholder("Community, gateway, описание…").fill("нет совпадений")
|
||||||
|
await expect(page.getByText("Ничего не найдено", { exact: true })).toBeVisible()
|
||||||
|
await page.getByPlaceholder("Community, gateway, описание…").fill("")
|
||||||
|
await expect(rows).toHaveCount(12)
|
||||||
|
expect(errors).toEqual([])
|
||||||
|
})
|
||||||
Reference in new issue
Block a user