fix: upgrade to Node 20, fix Docker build, remove husky

Author: PMS61

Dockerfile

@@ -25,6 +25,15 @@ ENV NODE_ENV=production
 ENV HUSKY=0
 ENV NEXT_OUTPUT_STANDALONE=true
 
+# Required for build-time environment variable validation
+# These values are validated at runtime and can be placeholders for build
+ENV NEXT_PUBLIC_SUPABASE_URL=https://placeholder.supabase.co
+ENV SUPABASE_SERVICE_ROLE_KEY=placeholder_key_for_build_only
+ENV GOOGLE_CLIENT_ID=placeholder_for_build
+ENV GOOGLE_CLIENT_SECRET=placeholder_for_build
+ENV NEXTAUTH_SECRET=placeholder_for_build
+ENV JWT_SECRET=placeholder_for_build
+
 # Build application
 RUN npm run build
 

app/actions/auth/auth.ts

@@ -1,4 +1,4 @@
-import {supabaseAdmin} from '@/lib/supabase-admin';
+import { supabaseAdmin } from '@/lib/supabase-admin';
 import { cookies } from 'next/headers';
 import jwt from 'jsonwebtoken';
 

lib/auth.ts

@@ -26,10 +26,10 @@ export const authOptions: NextAuthOptions = {
           return false;
         }
         //extract dep and year from email
-        const dep = profile.email.split('@')[1].split('.')[0]; 
-        const match = profile.email.match(/_?b(\d+)@/); 
+        const dep = profile.email.split('@')[1].split('.')[0];
+        const match = profile.email.match(/_?b(\d+)@/);
         const year = match ? parseInt(match[1], 10) + 2000 : null;
-        
+
         // console.log(`Extracted department: ${dep}, year: ${year} from email: ${profile.email}`);
         // Sync user to Supabase
         try {
@@ -48,7 +48,7 @@ export const authOptions: NextAuthOptions = {
                 name: profile.name || user.name,
                 picture: profile.image || user.image,
                 branch:dep,
-                year: year, 
+                year: year,
                 is_admin: 0, // Explicitly set to 0, can only be changed via Supabase
               }]);
           } else {
@@ -59,7 +59,7 @@ export const authOptions: NextAuthOptions = {
                 name: profile.name || user.name,
                 picture: profile.image || user.image,
                 branch:dep,
-                year: year, 
+                year: year,
               })
               .eq('email', profile.email);
           }
@@ -104,7 +104,7 @@ export const authOptions: NextAuthOptions = {
       if (url.includes('/api/auth/callback')) {
         return baseUrl + "/dashboard";
       }
-      
+
       // Handle other redirects
       if (url.startsWith(baseUrl)) return url;
       else if (url.startsWith("/")) return new URL(url, baseUrl).toString();

lib/supabase-admin.ts

@@ -1,13 +1,35 @@
 // This file is for server-side Supabase operations only
-import { createClient } from '@supabase/supabase-js'
+import { createClient, SupabaseClient } from '@supabase/supabase-js'
 
-const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL!
-const supabaseServiceKey = process.env.SUPABASE_SERVICE_ROLE_KEY!
+// Lazy initialization to avoid build-time errors
+// Environment variables are validated at runtime when the client is first used
+let supabaseAdminInstance: SupabaseClient | null = null
 
-// Service client for server-side operations only
-export const supabaseAdmin = createClient(supabaseUrl, supabaseServiceKey, {
-  auth: {
-    autoRefreshToken: false,
-    persistSession: false
+function initSupabaseAdmin(): SupabaseClient {
+  const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL
+  const supabaseServiceKey = process.env.SUPABASE_SERVICE_ROLE_KEY
+
+  if (!supabaseUrl || !supabaseServiceKey) {
+    throw new Error('Missing required Supabase environment variables. Please check your .env file.')
   }
-})
+
+  return createClient(supabaseUrl, supabaseServiceKey, {
+    auth: {
+      autoRefreshToken: false,
+      persistSession: false
+    }
+  })
+}
+
+// Export a proxy that lazily initializes the client on first access
+export const supabaseAdmin = new Proxy({} as SupabaseClient, {
+  get: (_, prop) => {
+    if (!supabaseAdminInstance) {
+      supabaseAdminInstance = initSupabaseAdmin()
+    }
+    return (supabaseAdminInstance as any)[prop]
+  }
+})
+
+// Also export the getter function for explicit initialization
+export { initSupabaseAdmin as getSupabaseAdmin }