Revert environment variable handling - SUPABASE_SERVICE_ROLE_KEY should always be provided

This reverts commit 985e49f. The SUPABASE_SERVICE_ROLE_KEY environment variable
should be properly configured in all build environments rather than being
handled as optional.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: kiwicopple

website/data/static-path-queries.ts

@@ -1,56 +1,35 @@
-import supabaseAdmin, { isAdminAvailable } from '~/lib/supabase-admin'
+import supabaseAdmin from '~/lib/supabase-admin'
 
 // [Alaister]: These functions are to be called server side only
 // as they bypass RLS. They will not work client side.
 
 export async function getAllProfiles() {
-  // During build time, if admin client isn't available, return empty array
-  if (!isAdminAvailable()) {
-    console.warn('SUPABASE_SERVICE_ROLE_KEY not available during build - skipping profile paths generation')
-    return []
-  }
-
-  try {
-    const [{ data: organizations }, { data: accounts }] = await Promise.all([
-      supabaseAdmin
-        .from('organizations')
-        .select('handle')
-        .order('created_at', { ascending: false })
-        .limit(500)
-        .returns<{ handle: string }[]>(),
-      supabaseAdmin
-        .from('accounts')
-        .select('handle')
-        .order('created_at', { ascending: false })
-        .limit(500)
-        .returns<{ handle: string }[]>(),
-    ])
+  const [{ data: organizations }, { data: accounts }] = await Promise.all([
+    supabaseAdmin
+      .from('organizations')
+      .select('handle')
+      .order('created_at', { ascending: false })
+      .limit(500)
+      .returns<{ handle: string }[]>(),
+    supabaseAdmin
+      .from('accounts')
+      .select('handle')
+      .order('created_at', { ascending: false })
+      .limit(500)
+      .returns<{ handle: string }[]>(),
+  ])
 
-    return [...(organizations ?? []), ...(accounts ?? [])]
-  } catch (error) {
-    console.error('Error fetching profiles for static paths:', error)
-    return []
-  }
+  return [...(organizations ?? []), ...(accounts ?? [])]
 }
 
 export async function getAllPackages() {
-  // During build time, if admin client isn't available, return empty array
-  if (!isAdminAvailable()) {
-    console.warn('SUPABASE_SERVICE_ROLE_KEY not available during build - skipping package paths generation')
-    return []
-  }
+  const { data } = await supabaseAdmin
 
-  try {
-    const { data } = await supabaseAdmin
-      .from('packages')
-      .select('handle,partial_name')
-      .order('created_at', { ascending: false })
-      .limit(1000)
-      .returns<{ handle: string; partial_name: string }[]>()
+    .from('packages')
+    .select('handle,partial_name')
+    .order('created_at', { ascending: false })
+    .limit(1000)
+    .returns<{ handle: string; partial_name: string }[]>()
 
-    return data ?? []
-  } catch (error) {
-    console.error('Error fetching packages for static paths:', error)
-    return []
-  }
+  return data ?? []
 }

website/lib/supabase-admin.ts

@@ -5,16 +5,13 @@ if (!process.env.NEXT_PUBLIC_SUPABASE_URL) {
   throw new Error('Missing NEXT_PUBLIC_SUPABASE_URL environment variable')
 }
 
-// During build time, we might not have the service role key
-// In that case, we'll create a client that will be used later at runtime
+if (!process.env.SUPABASE_SERVICE_ROLE_KEY) {
+  throw new Error('Missing SUPABASE_SERVICE_ROLE_KEY environment variable')
+}
+
 const supabaseAdmin = createClient<Database>(
   process.env.NEXT_PUBLIC_SUPABASE_URL,
-  process.env.SUPABASE_SERVICE_ROLE_KEY || 'dummy-key-for-build'
+  process.env.SUPABASE_SERVICE_ROLE_KEY
 )
 
-// Helper function to check if admin operations are available
-export const isAdminAvailable = () => {
-  return !!process.env.SUPABASE_SERVICE_ROLE_KEY
-}
-
 export default supabaseAdmin