diff --git a/frontend/src/app/site-finder/page.tsx b/frontend/src/app/site-finder/page.tsx index 8febd2b8..2a20afc0 100644 --- a/frontend/src/app/site-finder/page.tsx +++ b/frontend/src/app/site-finder/page.tsx @@ -1,9 +1,9 @@ "use client"; -import { useEffect, useState } from "react"; +import { Suspense, useEffect, useState } from "react"; import dynamic from "next/dynamic"; import Link from "next/link"; -import { useRouter, useSearchParams } from "next/navigation"; +import { usePathname, useRouter, useSearchParams } from "next/navigation"; import type { FeatureCollection } from "geojson"; import { CadInput } from "@/components/site-finder/CadInput"; @@ -80,19 +80,21 @@ function scoreKpiColor(score: number, max: number): KpiColor { // ── Page ────────────────────────────────────────────────────────────────────── -export default function SiteFinderPage() { +function SiteFinderContent() { const router = useRouter(); + const pathname = usePathname(); const searchParams = useSearchParams(); - const initialTab: TabId = (() => { - const t = searchParams.get("tab"); - return isTabId(t) ? t : "overview"; - })(); const { mutate, data, isPending, error, isIdle } = useSiteAnalysis(); const [isochrones, setIsochrones] = useState( undefined, ); - const [tab, setTabState] = useState(initialTab); + // Lazy init: считаем initialTab один раз на mount (useState всё равно + // игнорирует initializer после первого render — не тратим CPU). + const [tab, setTabState] = useState(() => { + const t = searchParams.get("tab"); + return isTabId(t) ? t : "overview"; + }); // Sync tab → URL query (?tab=env). overview = default, не пишем в URL. function setTab(next: TabId) { @@ -104,15 +106,17 @@ export default function SiteFinderPage() { params.set("tab", next); } const qs = params.toString(); - router.replace(qs ? `?${qs}` : "?", { scroll: false }); + // Используем pathname вместо голого "?" — Next 15 App Router не всегда + // чисто чистит query при `router.replace("?")` (dangling `?` остаётся). + router.replace(qs ? `${pathname}?${qs}` : pathname, { scroll: false }); } - // Listen to back/forward navigation — reflect URL tab in state + // Listen to back/forward navigation — reflect URL tab in state. + // setState с тем же значением React сам бэйлит через Object.is, поэтому + // guard по tab не нужен (и не вводит stale-closure через eslint-disable). useEffect(() => { const urlTab = searchParams.get("tab"); - const validTab: TabId = isTabId(urlTab) ? urlTab : "overview"; - if (validTab !== tab) setTabState(validTab); - // eslint-disable-next-line react-hooks/exhaustive-deps + setTabState(isTabId(urlTab) ? urlTab : "overview"); }, [searchParams]); function handleAnalyze(cadNum: string) { @@ -377,3 +381,13 @@ export default function SiteFinderPage() { ); } + +// useSearchParams() в Next 15 App Router throw'ит при static rendering без +// boundary — поэтому оборачиваем page-level. +export default function SiteFinderPage() { + return ( + + + + ); +}