From 459f07dde2f4107edb4bda575c23fc8d5a5877f8 Mon Sep 17 00:00:00 2001 From: Kevin Turcios Date: Wed, 15 Apr 2026 05:23:46 -0500 Subject: [PATCH] fix: remove redundant auth check from roadmap page The page-level isTeamMember() check breaks PPR (Partial Prerendering): at build time there's no session, so the prerender resolves as a redirect/404 and the static shell poisons all runtime responses with 404. Team member gating is already handled by proxy.ts middleware (lines 50-58), matching how /observability and other team-gated routes work. --- js/cf-webapp/src/app/roadmap/page.tsx | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/js/cf-webapp/src/app/roadmap/page.tsx b/js/cf-webapp/src/app/roadmap/page.tsx index 9ec88dd85..79befe54a 100644 --- a/js/cf-webapp/src/app/roadmap/page.tsx +++ b/js/cf-webapp/src/app/roadmap/page.tsx @@ -1,5 +1,3 @@ -import { isTeamMember } from "@/app/utils/auth" -import { redirect } from "next/navigation" import { Repeat, Wrench, @@ -476,10 +474,9 @@ function Metrics() { } export default async function RoadmapPage() { - const teamMember = await isTeamMember() - if (!teamMember) { - redirect("/login") - } + // Team member gating is handled by proxy.ts middleware — no auth check here. + // Adding one breaks PPR: at build time there's no session, so the prerender + // would resolve as a redirect/404 and the static shell poisons runtime responses. return (