From 0d2affaf6bc5b45906195a83db7b51968501b3df Mon Sep 17 00:00:00 2001 From: Denozordec Date: Thu, 1 Oct 2026 22:10:57 +0700 Subject: [PATCH] =?UTF-8?q?fix(evobgp):=20=D0=BF=D1=80=D0=BE=D0=B2=D0=B5?= =?UTF-8?q?=D1=80=D1=8F=D1=82=D1=8C=20=D0=BA=D0=B0=D1=82=D0=B0=D0=BB=D0=BE?= =?UTF-8?q?=D0=B3=20=D0=B8=20=D1=81=D0=BE=D1=85=D1=80=D0=B0=D0=BD=D1=8F?= =?UTF-8?q?=D1=82=D1=8C=20=D0=BD=D0=B0=D0=B2=D0=B8=D0=B3=D0=B0=D1=86=D0=B8?= =?UTF-8?q?=D1=8E=20=D0=BF=D1=80=D0=B8=20=D0=BE=D1=88=D0=B8=D0=B1=D0=BA?= =?UTF-8?q?=D0=B0=D1=85=20API?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/ux-ui-plan.md | 2 +- docs/ux-ui-progress.md | 10 +++++++++- docs/ux-ui-stage-11b.md | 9 +++++++++ lib/evobgp-catalog.test.ts | 17 +++++++++++++++++ lib/evobgp-catalog.ts | 22 ++++++++++++++++++++++ lib/evobgp-context.tsx | 5 +++-- tests/e2e/evobgp-catalog-error.spec.ts | 16 ++++++++++++++++ 7 files changed, 77 insertions(+), 4 deletions(-) create mode 100644 lib/evobgp-catalog.test.ts create mode 100644 lib/evobgp-catalog.ts create mode 100644 tests/e2e/evobgp-catalog-error.spec.ts diff --git a/docs/ux-ui-plan.md b/docs/ux-ui-plan.md index 7b5b71ac..fa668901 100644 --- a/docs/ux-ui-plan.md +++ b/docs/ux-ui-plan.md @@ -226,7 +226,7 @@ REST запрос → ожидание → подтверждённый output / | Пользователи | [Профиль](ux-ui/screens/users-profile-desktop.png), [ошибка](ux-ui/screens/users-error-desktop.png) | [390](ux-ui/screens/users-profile-mobile.png), [320](ux-ui/screens/users-profile-narrow.png) | | Контейнеры | [Обзор](ux-ui/screens/containers-overview-desktop.png), [снимок](ux-ui/screens/containers-export-desktop.png) | [390](ux-ui/screens/containers-export-mobile.png), [320](ux-ui/screens/containers-export-narrow.png) | -Актуальная точка продолжения: 11Б.3.3.5 — подключение strict settings readers, независимые known/error/time и dirty drafts. API основа проверена в 11Б.3.3.4; пока не подключена к странице. Независимые scheduler status/runs и достоверные KPI приняты в 11Б.3.3.3. Strict history/unknown/свежесть приняты в 11Б.3.3.2. Контекст/abort/revision и исходный lint исправлены в 11Б.3.3.1; далее независимые readers/dirty states и операции. GeoIP API принят в 11Б.3.2. Scheduler API основа проверена в 11Б.3.1, пока не подключена к странице. Совместимость JSONB/legacy snapshot и foreign guard проверены в 11Б.2; исходные lint/context ограничения страницы описаны в журнале. Общий runner исправлен в 11Б.1; карта оставшегося этапа — [11Б](ux-ui-stage-11b.md). Этап 11А принят локально в 11А.33: backups 71 основных + 28 source E2E, 27 frontend unit, backend профиль и production build. Backup scheduler cleanup проверен в 11А.32; общий dispatcher/journal status исправлен в 11Б.1, реальные интеграции остаются внешней приёмкой. S3/partial job/initial errors приняты в 11А.31.2.2. Диалоги ошибок приняты в 11А.31.2.1. Frame/первые 9 экранов приняты в 11А.31.1. URL/Filters контекст принят в 11А.30. Клиентский checkpoint restore принят в 11А.29.5. Backend outcome неизвестного импорта принят в 11А.29.4. Идентичность receipt принята в 11А.29.3. Backend cleanup удаления принят в 11А.29.2. Frontend delete/restore/download принят в 11А.29.1, затем URL/Frame/визуальная приёмка и scheduler cleanup. Frontend checkpoint/создание/serial polling приняты локально в 11А.28.2. API/основа наблюдения реализованы в 11А.28.1. Настройки расписания/S3 реализованы в 11А.27.1–11А.27.2. Каталоги/source/права реализованы в 11А.25, backend результат ручных операций исправлен в 11А.26. Сертификаты приняты локально в 11А.24, backups API стал строгим. State API, reader, snapshot merge и nullable представление подключены в 11А.14–11А.17.3; таблица требований — [этап 11А](ux-ui-stage-11a.md). Резервные копии и весь 11А приняты локально в 11А.33; внешние ограничения — журнал. +Каталог Evolution защищён от неполного HTTP-ответа в 11Б.3.3.4.1; testConnection receipt и состояния интеграции остаются в объёме 11Б. Актуальная точка продолжения: 11Б.3.3.5 — подключение strict settings readers, независимые known/error/time и dirty drafts. API основа проверена в 11Б.3.3.4; пока не подключена к странице. Независимые scheduler status/runs и достоверные KPI приняты в 11Б.3.3.3. Strict history/unknown/свежесть приняты в 11Б.3.3.2. Контекст/abort/revision и исходный lint исправлены в 11Б.3.3.1; далее независимые readers/dirty states и операции. GeoIP API принят в 11Б.3.2. Scheduler API основа проверена в 11Б.3.1, пока не подключена к странице. Совместимость JSONB/legacy snapshot и foreign guard проверены в 11Б.2; исходные lint/context ограничения страницы описаны в журнале. Общий runner исправлен в 11Б.1; карта оставшегося этапа — [11Б](ux-ui-stage-11b.md). Этап 11А принят локально в 11А.33: backups 71 основных + 28 source E2E, 27 frontend unit, backend профиль и production build. Backup scheduler cleanup проверен в 11А.32; общий dispatcher/journal status исправлен в 11Б.1, реальные интеграции остаются внешней приёмкой. S3/partial job/initial errors приняты в 11А.31.2.2. Диалоги ошибок приняты в 11А.31.2.1. Frame/первые 9 экранов приняты в 11А.31.1. URL/Filters контекст принят в 11А.30. Клиентский checkpoint restore принят в 11А.29.5. Backend outcome неизвестного импорта принят в 11А.29.4. Идентичность receipt принята в 11А.29.3. Backend cleanup удаления принят в 11А.29.2. Frontend delete/restore/download принят в 11А.29.1, затем URL/Frame/визуальная приёмка и scheduler cleanup. Frontend checkpoint/создание/serial polling приняты локально в 11А.28.2. API/основа наблюдения реализованы в 11А.28.1. Настройки расписания/S3 реализованы в 11А.27.1–11А.27.2. Каталоги/source/права реализованы в 11А.25, backend результат ручных операций исправлен в 11А.26. Сертификаты приняты локально в 11А.24, backups API стал строгим. State API, reader, snapshot merge и nullable представление подключены в 11А.14–11А.17.3; таблица требований — [этап 11А](ux-ui-stage-11a.md). Резервные копии и весь 11А приняты локально в 11А.33; внешние ограничения — журнал. | Экран | ПК | Телефон | | --- | --- | --- | diff --git a/docs/ux-ui-progress.md b/docs/ux-ui-progress.md index 6a83e35e..125887c6 100644 --- a/docs/ux-ui-progress.md +++ b/docs/ux-ui-progress.md @@ -28,7 +28,7 @@ | 9 | VPN, Firewall и Filters | Завершён в границах локальной приёмки 2026-09-30; блоки 9.1–9.64, ограничения в 9.64 | | 10 | Маршрутизация, диагностика, карта и терминал | Завершён в границах локальной приёмки 2026-10-01; блоки 10.1–10.71, внешние ограничения в 10.71 | | 11А | Пользователи, контейнеры, сертификаты и резервные копии | Завершён в границах локальной приёмки 2026-10-01; 11А.1–11А.33, внешние ограничения в 11А.33 | -| 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | В работе: 11Б.1–11Б.3.3.4; status/history/KPI и strict settings API основа проверены; далее подключение readers/dirty формы | +| 11Б | Оповещения, сбор данных, интеграции, релизы и настройки | В работе: 11Б.1–11Б.3.3.4.1; status/history/KPI, strict settings API основа и защита каталога Evolution проверены; далее readers/dirty формы | | 12 | Итоговая доступность, адаптивность, производительность и регрессия | Не начат | @@ -36,6 +36,14 @@ +## Блок 11Б.3.3.4.1 — проверка каталога Evolution (2026-10-01) + +- Каталог Evolution проверяется до записи в состояние: обязательные коллекции, поля строк, дата снимка, числовые счётчики и уникальность идентификаторов. Неполный HTTP 200 больше не обрушает боковую навигацию; в настройках EvoBGP отображается понятная ошибка. +- Подтверждённые пустые коллекции, false и zero сохраняются. Decoder совместим с DTO backend; неизвестные значения backend не переосмыслены. Последний снимок при ошибке пока не сохраняется: независимые состояния, свежесть и защита от гонок Evolution остаются в объёме этапа. +- Прошли 3 unit, 1 браузерный тест через обычные API-пути с подменой HTTP, ESLint изменённых файлов и production frontend/contracts build со встроенным TypeScript. В браузере проверены доступность журнала, переход в настройки EvoBGP и отсутствие pageerror при malformed каталоге. Реальный Evolution Portal не проверен. +- Следующий основной блок остаётся 11Б.3.3.5: подключение strict settings readers, независимые known/error/time и dirty drafts. Также выявлено: Evolution testConnection пока принимает любой успешный HTTP вместо подтверждения ok=true; исправление и профильная проверка ещё впереди. +- На завершении пятичасовой лимит использован на 42%, недельный показывает 100%. Новый блок не начат; изменения сохранены локально без push. + ## Блок 11Б.3.3.4 — строгие readers настроек сборщиков (2026-10-01) - Добавлен shared/api/collector-settings.ts: traffic/REST ping/uptime/internet path читаются по своим реальным GET DTO. False/zero/null сохраняются, missing/invalid не нормализуется defaults; PostgreSQL время приводится к тому же ISO instant. diff --git a/docs/ux-ui-stage-11b.md b/docs/ux-ui-stage-11b.md index 87db811c..82e8f87a 100644 --- a/docs/ux-ui-stage-11b.md +++ b/docs/ux-ui-stage-11b.md @@ -101,3 +101,12 @@ ReUI PRO/base-nova; существующие Frame, Data Grid v9, KpiStatGrid, S - Current UI всё ещё использует старые settings calls. Нельзя считать формы подтверждёнными или защищёнными от default submit по этим unit: подключение readers/known/error/time/dirty states, synchronous locks, RBAC и unknown mutation receipt ещё впереди. Generic CollectorSettingsDto на странице ошибочно требует retention у REST ping; при подключении исправить типы по отдельным DTO. - **Следующий блок 11Б.3.3.5:** подключить четыре strict settings reader и существующий certificate reader, независимые состояния/свежесть и dirty drafts; затем операции/права/source E2E. Evolution malformed snapshot crash и GeoIP/NetFlow формы остаются в scope 11Б. - На контрольной точке недельный лимит использован на 99%, пятичасовой на 36%. Проверенные блоки сохранены; для продолжения читать текущую квоту и этот журнал. + + +## Блок 11Б.3.3.4.1 — проверка каталога Evolution (2026-10-01) + +- Каталог Evolution проверяется до записи в состояние: обязательные коллекции, поля строк, дата снимка, числовые счётчики и уникальность идентификаторов. Неполный HTTP 200 больше не обрушает боковую навигацию; в настройках EvoBGP отображается понятная ошибка. +- Подтверждённые пустые коллекции, false и zero сохраняются. Decoder совместим с DTO backend; неизвестные значения backend не переосмыслены. Последний снимок при ошибке пока не сохраняется: независимые состояния, свежесть и защита от гонок Evolution остаются в объёме этапа. +- Прошли 3 unit, 1 браузерный тест через обычные API-пути с подменой HTTP, ESLint изменённых файлов и production frontend/contracts build со встроенным TypeScript. В браузере проверены доступность журнала, переход в настройки EvoBGP и отсутствие pageerror при malformed каталоге. Реальный Evolution Portal не проверен. +- Следующий основной блок остаётся 11Б.3.3.5: подключение strict settings readers, независимые known/error/time и dirty drafts. Также выявлено: Evolution testConnection пока принимает любой успешный HTTP вместо подтверждения ok=true; исправление и профильная проверка ещё впереди. +- На завершении пятичасовой лимит использован на 42%, недельный показывает 100%. Новый блок не начат; изменения сохранены локально без push. diff --git a/lib/evobgp-catalog.test.ts b/lib/evobgp-catalog.test.ts new file mode 100644 index 00000000..d04d61c4 --- /dev/null +++ b/lib/evobgp-catalog.test.ts @@ -0,0 +1,17 @@ +import assert from "node:assert/strict" +import { test } from "node:test" +import { parseEvoBgpCatalog } from "./evobgp-catalog" +const empty = { fetchedAt: "2026-10-01T00:00:00Z", modules: [], domains: [], ipRanges: [], asns: [], communities: [] } +const domain = { id: "domain-1", domain: "example.test", resolvedIp: "—", asn: "—", purpose: "", filter: "", updated: "—", enabled: false } +test("known empty and false stay explicit without invented rows", () => { + assert.deepEqual(parseEvoBgpCatalog(empty), empty) + assert.equal(parseEvoBgpCatalog({ ...empty, domains: [domain] }).domains[0].enabled, false) +}) +test("malformed or partial catalogue never reaches consumers as a snapshot", () => { + for (const raw of [{}, null, { ...empty, domains: undefined }, { ...empty, asns: {} }, { ...empty, fetchedAt: "bad" }, { ...empty, domains: [null] }, { ...empty, domains: [{ ...domain, enabled: undefined }] }, { ...empty, domains: [domain, domain] }]) assert.throws(() => parseEvoBgpCatalog(raw), /Каталог Evolution не подтверждён/) +}) +test("ASN and community zero counts are preserved, invalid counts rejected", () => { + const asn = { id: "asn-1", asn: "AS1", org: "", country: "", prefixes: 0, filter: "", updated: "—", enabled: false } + assert.equal(parseEvoBgpCatalog({ ...empty, asns: [asn] }).asns[0].prefixes, 0) + assert.throws(() => parseEvoBgpCatalog({ ...empty, asns: [{ ...asn, prefixes: -1 }] })) +}) diff --git a/lib/evobgp-catalog.ts b/lib/evobgp-catalog.ts new file mode 100644 index 00000000..d1e6063c --- /dev/null +++ b/lib/evobgp-catalog.ts @@ -0,0 +1,22 @@ +import { z } from "zod" +import type { EvoBgpCatalogSnapshot } from "./evobgp-context" +const text = z.string() +const id = text.min(1) +const count = z.number().int().nonnegative().safe() +function unique>(row: T) { + return z.array(row).refine(rows => new Set(rows.map(item => item.id)).size === rows.length) +} +const base = { id, asn: text, purpose: text, filter: text, updated: text, enabled: z.boolean() } +const schema = z.object({ + fetchedAt: text.refine(value => Number.isFinite(Date.parse(value))), + modules: unique(z.object({ id, name: text, type: text })), + domains: unique(z.object({ ...base, domain: text, resolvedIp: text })), + ipRanges: unique(z.object({ ...base, cidr: text, country: text })), + asns: unique(z.object({ id, asn: text, org: text, country: text, prefixes: count, filter: text, updated: text, enabled: z.boolean() })), + communities: unique(z.object({ id, value: text, name: text, description: text, type: z.enum(["standard", "no-export", "no-advertise", "local-as", "custom"]), filterIds: z.array(text), serverCount: count, prefixCount: count, action: z.enum(["permit", "deny", "local-pref", "metric"]), actionValue: z.number().finite().optional(), enabled: z.boolean() })), +}) +export function parseEvoBgpCatalog(raw: unknown): EvoBgpCatalogSnapshot { + const parsed = schema.safeParse(raw) + if (!parsed.success) throw new Error("Каталог Evolution не подтверждён: получен некорректный ответ сервера") + return parsed.data +} diff --git a/lib/evobgp-context.tsx b/lib/evobgp-context.tsx index c150c409..c04a8240 100644 --- a/lib/evobgp-context.tsx +++ b/lib/evobgp-context.tsx @@ -10,6 +10,7 @@ import { } from "react" import { useDataSource } from "@/lib/data-source" import type { Domain, IpRange, Asn } from "@/lib/data" +import { parseEvoBgpCatalog } from "@/lib/evobgp-catalog" import { ApiClientError, requestJson } from "@/shared/api/http-client" export interface EvoBgpCommunityRow { @@ -93,12 +94,12 @@ export function EvoBGPProvider({ children }: { children: React.ReactNode }) { setLoading(true) setError(null) try { - const data = await requestJson( + const data = await requestJson( backendUrl, "/api/evobgp/catalog", { method: "POST" }, ) - setSnapshot(data) + setSnapshot(parseEvoBgpCatalog(data)) } catch (e) { setSnapshot(null) setError(errorMessage(e, "Ошибка загрузки")) diff --git a/tests/e2e/evobgp-catalog-error.spec.ts b/tests/e2e/evobgp-catalog-error.spec.ts new file mode 100644 index 00000000..9b626e6f --- /dev/null +++ b/tests/e2e/evobgp-catalog-error.spec.ts @@ -0,0 +1,16 @@ +import { test, expect } from "@playwright/test" +test("Evolution: malformed catalogue не обрушает навигацию", async ({ page }) => { + const errors: string[] = []; let catalogs = 0; page.on("pageerror", error => errors.push(error.message)) + await page.route("**/health", r => r.fulfill({ json: { ok: true } })) + await page.route("**/api/**", r => { + const path = new URL(r.request().url()).pathname + if (path === "/api/auth/config") return r.fulfill({ json: { required: false } }) + if (path === "/api/evobgp/settings") return r.fulfill({ json: { baseUrl: "https://evo.example.test", enabled: true, secretConfigured: true } }) + if (path === "/api/evobgp/catalog") { catalogs++; return r.fulfill({ json: {} }) } + return r.fulfill({ status: 503, json: { error: "outside catalogue fixture" } }) + }) + await page.goto("/data-collection"); await expect.poll(() => catalogs).toBeGreaterThan(0) + await expect(page.getByText("Журнал прогонов", { exact: true })).toBeVisible() + await page.goto("/settings"); await page.getByRole("button", { name: "EvoBGP", exact: true }).click(); await expect(page.getByText("Каталог Evolution не подтверждён: получен некорректный ответ сервера", { exact: true })).toBeVisible() + expect(errors).toEqual([]) +})