From b8d90897a4b5ce0c06aec55466a4c3aabd0c5770 Mon Sep 17 00:00:00 2001
From: David Gwilliam <dhgwilliam@gmail.com>
Date: Tue, 31 Mar 2026 00:50:56 -0700
Subject: [PATCH] fix: await params in Next.js 16 routes and pages for
 TypeScript compatibility

---
 src/app/admin/tournaments/[id]/edit/page.tsx  |  4 ++-
 src/app/admin/tournaments/[id]/entry/page.tsx | 32 +++++++++++++++----
 src/app/admin/tournaments/[id]/page.tsx       |  4 ++-
 .../admin/tournaments/[id]/results/page.tsx   |  4 ++-
 src/app/api/admin/players/[id]/route.ts       |  5 +--
 .../api/tournaments/[id]/games/bulk/route.ts  |  5 +--
 .../tournaments/[id]/participants/route.ts    |  5 +--
 src/app/api/tournaments/[id]/route.ts         | 13 +++++---
 src/app/api/users/[id]/role/route.ts          | 10 +++---
 src/app/api/users/[id]/route.ts               | 10 +++---
 src/app/players/[id]/profile/page.tsx         |  4 ++-
 src/app/players/[id]/schedule/page.tsx        |  4 ++-
 src/lib/prisma.ts                             | 17 +++++++++-
 13 files changed, 85 insertions(+), 32 deletions(-)

diff --git a/src/app/admin/tournaments/[id]/edit/page.tsx b/src/app/admin/tournaments/[id]/edit/page.tsx
index 82461a2..d1f6d63 100644
--- a/src/app/admin/tournaments/[id]/edit/page.tsx
+++ b/src/app/admin/tournaments/[id]/edit/page.tsx
@@ -12,7 +12,9 @@ interface PageProps {
 }
 
 export default async function EditTournamentPage({ params }: PageProps) {
-  const tournamentId = parseInt(params.id, 10)
+  // Next.js 16 requires awaiting params
+  const { id } = await params
+  const tournamentId = parseInt(id, 10)
   
   if (isNaN(tournamentId)) {
     notFound()
diff --git a/src/app/admin/tournaments/[id]/entry/page.tsx b/src/app/admin/tournaments/[id]/entry/page.tsx
index 0a5161b..550c286 100644
--- a/src/app/admin/tournaments/[id]/entry/page.tsx
+++ b/src/app/admin/tournaments/[id]/entry/page.tsx
@@ -31,23 +31,41 @@ interface GameEntry {
   score2: number
 }
 
-export default function TournamentEntryPage({ params }: { params: { id: string } }) {
+export default function TournamentEntryPage({ params }: { params: Promise<{ id: string }> }) {
   const router = useRouter()
   const [tournament, setTournament] = useState<Tournament | null>(null)
+  const [tournamentId, setTournamentId] = useState<number | null>(null)
   const [gameText, setGameText] = useState("")
   const [parsedGames, setParsedGames] = useState<GameEntry[]>([])
   const [error, setError] = useState("")
   const [success, setSuccess] = useState("")
   const [isLoading, setIsLoading] = useState(false)
 
+  // Parse params and validate tournamentId
   useEffect(() => {
-    loadTournament()
+    async function parseParams() {
+      const { id } = await params
+      const parsedId = parseInt(id, 10)
+      if (isNaN(parsedId)) {
+        router.push('/admin/tournaments')
+      } else {
+        setTournamentId(parsedId)
+      }
+    }
+    parseParams()
+  }, [params, router])
+
+  // Load tournament when tournamentId is available
+  useEffect(() => {
+    if (tournamentId) {
+      loadTournament()
+    }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [])
+  }, [tournamentId])
 
   const loadTournament = async () => {
     try {
-      const response = await fetch(`/api/tournaments/${params.id}`)
+      const response = await fetch(`/api/tournaments/${tournamentId}`)
       const data = await response.json()
       if (response.ok) {
         setTournament(data.tournament)
@@ -103,7 +121,7 @@ export default function TournamentEntryPage({ params }: { params: { id: string }
     setIsLoading(true)
 
     try {
-      const response = await fetch(`/api/tournaments/${params.id}/games/bulk`, {
+      const response = await fetch(`/api/tournaments/${tournamentId}/games/bulk`, {
         method: "POST",
         headers: {
           "Content-Type": "application/json",
@@ -154,7 +172,7 @@ export default function TournamentEntryPage({ params }: { params: { id: string }
         <div className="px-4 py-6 sm:px-0">
           <div className="mb-6">
             <button
-              onClick={() => router.push(`/admin/tournaments/${params.id}`)}
+              onClick={() => router.push(`/admin/tournaments/${tournamentId}`)}
               className="text-green-600 hover:text-green-800 mb-4"
             >
               ← Back to Tournament
@@ -281,7 +299,7 @@ export default function TournamentEntryPage({ params }: { params: { id: string }
 
                 <div className="flex justify-end space-x-3">
                   <button
-                    onClick={() => router.push(`/admin/tournaments/${params.id}`)}
+              onClick={() => router.push(`/admin/tournaments/${tournamentId}`)}
                     className="px-4 py-2 border border-gray-300 rounded-md shadow-sm text-sm font-medium text-gray-700 bg-white hover:bg-gray-50 focus:outline-none focus:ring-2 focus:ring-offset-2 focus:ring-green-500"
                   >
                     Cancel
diff --git a/src/app/admin/tournaments/[id]/page.tsx b/src/app/admin/tournaments/[id]/page.tsx
index f037076..d4a153d 100644
--- a/src/app/admin/tournaments/[id]/page.tsx
+++ b/src/app/admin/tournaments/[id]/page.tsx
@@ -13,7 +13,9 @@ interface PageProps {
 }
 
 export default async function TournamentDetailPage({ params }: PageProps) {
-  const tournamentId = parseInt(params.id, 10)
+  // Next.js 16 requires awaiting params
+  const { id } = await params
+  const tournamentId = parseInt(id, 10)
   
   if (isNaN(tournamentId)) {
     notFound()
diff --git a/src/app/admin/tournaments/[id]/results/page.tsx b/src/app/admin/tournaments/[id]/results/page.tsx
index a22b657..15e702e 100644
--- a/src/app/admin/tournaments/[id]/results/page.tsx
+++ b/src/app/admin/tournaments/[id]/results/page.tsx
@@ -12,7 +12,9 @@ interface PageProps {
 }
 
 export default async function TournamentResultsPage({ params }: PageProps) {
-  const tournamentId = parseInt(params.id, 10)
+  // Next.js 16 requires awaiting params
+  const { id } = await params
+  const tournamentId = parseInt(id, 10)
   
   if (isNaN(tournamentId)) {
     notFound()
diff --git a/src/app/api/admin/players/[id]/route.ts b/src/app/api/admin/players/[id]/route.ts
index 0370c15..7c0969d 100644
--- a/src/app/api/admin/players/[id]/route.ts
+++ b/src/app/api/admin/players/[id]/route.ts
@@ -10,10 +10,11 @@ import { hasRole } from "@/lib/permissions";
  */
 export async function PATCH(
   request: Request,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
-    const playerId = parseInt(params.id);
+    const { id } = await params
+    const playerId = parseInt(id);
 
     // Validate player ID
     if (isNaN(playerId)) {
diff --git a/src/app/api/tournaments/[id]/games/bulk/route.ts b/src/app/api/tournaments/[id]/games/bulk/route.ts
index bdd342a..1393f62 100644
--- a/src/app/api/tournaments/[id]/games/bulk/route.ts
+++ b/src/app/api/tournaments/[id]/games/bulk/route.ts
@@ -8,10 +8,11 @@ const K_FACTOR = 32; // Standard K-factor for Elo calculations
 
 export async function POST(
   request: Request,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
-    const tournamentId = parseInt(params.id, 10);
+    const { id } = await params
+    const tournamentId = parseInt(id, 10);
     
     if (isNaN(tournamentId)) {
       return NextResponse.json(
diff --git a/src/app/api/tournaments/[id]/participants/route.ts b/src/app/api/tournaments/[id]/participants/route.ts
index 63c5292..daa261e 100644
--- a/src/app/api/tournaments/[id]/participants/route.ts
+++ b/src/app/api/tournaments/[id]/participants/route.ts
@@ -4,10 +4,11 @@ import { canManageTournament } from "@/lib/permissions";
 
 export async function POST(
   request: Request,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
-    const tournamentId = parseInt(params.id, 10);
+    const { id } = await params
+    const tournamentId = parseInt(id, 10);
     
     if (isNaN(tournamentId)) {
       return NextResponse.json(
diff --git a/src/app/api/tournaments/[id]/route.ts b/src/app/api/tournaments/[id]/route.ts
index 714881e..cc05214 100644
--- a/src/app/api/tournaments/[id]/route.ts
+++ b/src/app/api/tournaments/[id]/route.ts
@@ -4,9 +4,9 @@ import { canManageTournament, canDeleteTournament } from "@/lib/permissions";
 import { getTournamentStatus } from "@/lib/tournamentUtils";
 
 interface RouteParams {
-  params: {
+  params: Promise<{
     id: string;
-  };
+  }>;
 }
 
 /**
@@ -16,7 +16,8 @@ interface RouteParams {
  */
 export async function GET(request: Request, { params }: RouteParams) {
   try {
-    const tournamentId = parseInt(params.id);
+    const { id } = await params
+    const tournamentId = parseInt(id);
 
     // Validate tournament ID
     if (isNaN(tournamentId)) {
@@ -105,7 +106,8 @@ export async function GET(request: Request, { params }: RouteParams) {
  */
 export async function PUT(request: Request, { params }: RouteParams) {
   try {
-    const tournamentId = parseInt(params.id);
+    const { id } = await params
+    const tournamentId = parseInt(id);
 
     // Validate tournament ID
     if (isNaN(tournamentId)) {
@@ -219,7 +221,8 @@ export async function PUT(request: Request, { params }: RouteParams) {
  */
 export async function DELETE(request: Request, { params }: RouteParams) {
   try {
-    const tournamentId = parseInt(params.id);
+    const { id } = await params
+    const tournamentId = parseInt(id);
 
     // Validate tournament ID
     if (isNaN(tournamentId)) {
diff --git a/src/app/api/users/[id]/role/route.ts b/src/app/api/users/[id]/role/route.ts
index 6a4ba37..69b47b7 100644
--- a/src/app/api/users/[id]/role/route.ts
+++ b/src/app/api/users/[id]/role/route.ts
@@ -5,9 +5,10 @@ import { canAssignTournamentAdmin } from '@/lib/permissions';
 
 export async function GET(
   request: NextRequest,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
+    const { id } = await params
     const session = await getSession();
     
     if (!session) {
@@ -25,12 +26,12 @@ export async function GET(
       return NextResponse.json({ error: 'Requesting user not found' }, { status: 404 });
     }
 
-    if (session.user.id !== params.id && requestingUser.role !== 'club_admin') {
+    if (session.user.id !== id && requestingUser.role !== 'club_admin') {
       return NextResponse.json({ error: 'Forbidden' }, { status: 403 });
     }
 
     const user = await prisma.user.findUnique({
-      where: { id: params.id },
+      where: { id },
       select: { role: true }
     });
 
@@ -46,9 +47,10 @@ export async function GET(
 
 export async function PATCH(
   request: NextRequest,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
+    const { id } = await params
     // Check if the current user has permission to assign roles
     const permission = await canAssignTournamentAdmin();
     if (!permission.allowed) {
diff --git a/src/app/api/users/[id]/route.ts b/src/app/api/users/[id]/route.ts
index e8ce157..263fa27 100644
--- a/src/app/api/users/[id]/route.ts
+++ b/src/app/api/users/[id]/route.ts
@@ -5,9 +5,10 @@ import { canEditUserProfiles } from '@/lib/permissions';
 
 export async function GET(
   request: NextRequest,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
+    const { id } = await params
     const session = await getSession();
     
     if (!session) {
@@ -25,13 +26,13 @@ export async function GET(
     }
 
     // Allow if requesting own profile or if user is club_admin
-    const canView = session.user.id === params.id || requestingUser.role === 'club_admin';
+    const canView = session.user.id === id || requestingUser.role === 'club_admin';
     if (!canView) {
       return NextResponse.json({ error: 'Forbidden' }, { status: 403 });
     }
 
     const user = await prisma.user.findUnique({
-      where: { id: params.id },
+      where: { id },
       include: {
         player: true,
       }
@@ -49,9 +50,10 @@ export async function GET(
 
 export async function PATCH(
   request: NextRequest,
-  { params }: { params: { id: string } }
+  { params }: { params: Promise<{ id: string }> }
 ) {
   try {
+    const { id } = await params
     const session = await getSession();
     
     if (!session) {
diff --git a/src/app/players/[id]/profile/page.tsx b/src/app/players/[id]/profile/page.tsx
index 2422889..add386b 100644
--- a/src/app/players/[id]/profile/page.tsx
+++ b/src/app/players/[id]/profile/page.tsx
@@ -10,7 +10,9 @@ interface PageProps {
 }
 
 export default async function PlayerProfilePage({ params }: PageProps) {
-  const playerId = parseInt(params.id, 10)
+  // Next.js 16 requires awaiting params
+  const { id } = await params
+  const playerId = parseInt(id, 10)
   
   if (isNaN(playerId)) {
     notFound()
diff --git a/src/app/players/[id]/schedule/page.tsx b/src/app/players/[id]/schedule/page.tsx
index 8548000..5ce4c19 100644
--- a/src/app/players/[id]/schedule/page.tsx
+++ b/src/app/players/[id]/schedule/page.tsx
@@ -10,7 +10,9 @@ interface PageProps {
 }
 
 export default async function PlayerSchedulePage({ params }: PageProps) {
-  const playerId = parseInt(params.id, 10)
+  // Next.js 16 requires awaiting params
+  const { id } = await params
+  const playerId = parseInt(id, 10)
   
   if (isNaN(playerId)) {
     notFound()
diff --git a/src/lib/prisma.ts b/src/lib/prisma.ts
index b605608..ffa962e 100644
--- a/src/lib/prisma.ts
+++ b/src/lib/prisma.ts
@@ -8,6 +8,21 @@ const globalForPrisma = globalThis as unknown as {
 
 const adapter = new PrismaPg({ connectionString: process.env.DATABASE_URL })
 
-export const prisma = globalForPrisma.prisma ?? new PrismaClient({ adapter })
+// Create PrismaClient with adapter
+const createPrismaClient = () => {
+  const client = new PrismaClient({ adapter })
+  
+  // Add logging if in development
+  if (process.env.NODE_ENV !== 'production') {
+    client.$on('query', (e) => {
+      console.log('Query:', e.query)
+      console.log('Duration:', e.duration, 'ms')
+    })
+  }
+  
+  return client
+}
+
+export const prisma = globalForPrisma.prisma ?? createPrismaClient()
 
 if (process.env.NODE_ENV !== 'production') globalForPrisma.prisma = prisma