fix: environmental variables missing in GitHub actions + remove some consoles

Author: danieldanielecki

app/api/get-ip/route.ts

@@ -24,7 +24,6 @@ export async function GET(request: NextRequest) {
 
     return NextResponse.json({ ip });
   } catch (error) {
-    console.error("Error getting IP:", error);
     return NextResponse.json({ error: "Failed to get IP" }, { status: 500 });
   }
 }

app/api/graphql/route.ts

@@ -58,7 +58,6 @@ const wrappedHandler = async (req: Request) => {
   try {
     return await handler(req);
   } catch (error) {
-    console.error("GraphQL Error:", error);
     return new Response(
       JSON.stringify({
         errors: [{ message: "Internal server error" }],

hooks/useSecureTrial.tsx

@@ -126,6 +126,11 @@ export const useSecureTrial = () => {
   // Check if user already has an active trial
   const checkExistingTrial = async (): Promise<TrialRecord | null> => {
     try {
+      // Check if Appwrite is available
+      if (!databases) {
+        return null;
+      }
+
       const sessionId = await getSessionId();
       const deviceFingerprint = getDeviceFingerprint();
       const ipAddress = await getUserIP();
@@ -194,14 +199,18 @@ export const useSecureTrial = () => {
 
       return null;
     } catch (error) {
-      console.error("Error checking existing trial:", error);
       return null;
     }
   };
 
   // Create a new trial record
   const createTrial = async (): Promise<TrialRecord | null> => {
     try {
+      // Check if Appwrite is available
+      if (!databases) {
+        return null;
+      }
+
       const sessionId = await getSessionId();
       const deviceFingerprint = getDeviceFingerprint();
       const ipAddress = await getUserIP();
@@ -254,7 +263,6 @@ export const useSecureTrial = () => {
 
       return trialRecord as unknown as TrialRecord;
     } catch (error) {
-      console.error("Error creating trial:", error);
       // Clear the creation flag on error
       if (typeof window !== "undefined") {
         localStorage.removeItem("creating_trial");
@@ -266,6 +274,11 @@ export const useSecureTrial = () => {
   // Mark trial as expired
   const expireTrial = async (trialId: string) => {
     try {
+      // Check if Appwrite is available
+      if (!databases) {
+        return;
+      }
+
       await databases.updateDocument(
         DATABASE_ID,
         TRIAL_COLLECTION_ID,
@@ -337,7 +350,6 @@ export const useSecureTrial = () => {
           }
         }
       } catch (error) {
-        console.error("Error initializing trial:", error);
         // Don't block access on error - allow trial to proceed
         setTrialStartTime(Date.now());
         setTimeRemaining(TRIAL_DURATION_MS);

lib/appwrite/auth.ts

@@ -14,13 +14,23 @@ export interface AuthError {
 }
 
 export class AuthService {
+  // Check if Appwrite is available
+  private static checkAppwriteAvailable() {
+    if (!account) {
+      throw new Error(
+        "Appwrite is not initialized. Please check your environment variables.",
+      );
+    }
+  }
+
   // Email OTP Authentication
   static async createEmailOTPSession(
     email: string,
   ): Promise<{ success: boolean; error?: AuthError; userId?: string }> {
     try {
+      this.checkAppwriteAvailable();
       // Create email token (sends 6-digit OTP to email)
-      const sessionToken = await account.createEmailToken(ID.unique(), email);
+      const sessionToken = await account!.createEmailToken(ID.unique(), email);
       return { success: true, userId: sessionToken.userId };
     } catch (error: any) {
       return {
@@ -39,8 +49,9 @@ export class AuthService {
     otp: string,
   ): Promise<{ success: boolean; error?: AuthError }> {
     try {
+      this.checkAppwriteAvailable();
       // Create session with userId and OTP
-      await account.createSession(userId, otp);
+      await account!.createSession(userId, otp);
       return { success: true };
     } catch (error: any) {
       return {
@@ -58,7 +69,8 @@ export class AuthService {
     secret: string,
   ): Promise<{ success: boolean; error?: AuthError }> {
     try {
-      await account.updateMagicURLSession(userId, secret);
+      this.checkAppwriteAvailable();
+      await account!.updateMagicURLSession(userId, secret);
       return { success: true };
     } catch (error: any) {
       return {
@@ -77,8 +89,9 @@ export class AuthService {
     error?: AuthError;
   }> {
     try {
+      this.checkAppwriteAvailable();
       const redirectUrl = `${window.location.origin}/auth/callback`;
-      const url = await account.createOAuth2Session(
+      const url = await account!.createOAuth2Session(
         "google" as any,
         redirectUrl,
         redirectUrl,
@@ -107,8 +120,9 @@ export class AuthService {
     error?: AuthError;
   }> {
     try {
+      this.checkAppwriteAvailable();
       const redirectUrl = `${window.location.origin}/auth/callback`;
-      const url = await account.createOAuth2Session(
+      const url = await account!.createOAuth2Session(
         "apple" as any,
         redirectUrl,
         redirectUrl,
@@ -135,7 +149,8 @@ export class AuthService {
     error?: AuthError;
   }> {
     try {
-      const user = await account.get();
+      this.checkAppwriteAvailable();
+      const user = await account!.get();
       return {
         user: {
           $id: user.$id,
@@ -157,7 +172,8 @@ export class AuthService {
   // Sign out
   static async signOut(): Promise<{ success: boolean; error?: AuthError }> {
     try {
-      await account.deleteSession("current");
+      this.checkAppwriteAvailable();
+      await account!.deleteSession("current");
       return { success: true };
     } catch (error: any) {
       return {
@@ -173,7 +189,8 @@ export class AuthService {
   // Check if user is authenticated
   static async isAuthenticated(): Promise<boolean> {
     try {
-      await account.get();
+      this.checkAppwriteAvailable();
+      await account!.get();
       return true;
     } catch {
       return false;

lib/appwrite/config.ts

@@ -1,10 +1,20 @@
 import { Client, Account, Databases } from "appwrite";
 
-const client = new Client()
-  .setEndpoint(process.env.NEXT_PUBLIC_APPWRITE_ENDPOINT || "")
-  .setProject(process.env.NEXT_PUBLIC_APPWRITE_PROJECT_ID || "");
+// Only initialize Appwrite client if we have the required environment variables
+// This prevents build-time errors during static generation
+const isClientSide = typeof window !== "undefined";
+const hasRequiredEnvVars =
+  process.env.NEXT_PUBLIC_APPWRITE_ENDPOINT &&
+  process.env.NEXT_PUBLIC_APPWRITE_PROJECT_ID;
 
-export const account = new Account(client);
-export const databases = new Databases(client);
+const client =
+  isClientSide && hasRequiredEnvVars
+    ? new Client()
+        .setEndpoint(process.env.NEXT_PUBLIC_APPWRITE_ENDPOINT!)
+        .setProject(process.env.NEXT_PUBLIC_APPWRITE_PROJECT_ID!)
+    : null;
+
+export const account = client ? new Account(client) : null;
+export const databases = client ? new Databases(client) : null;
 
 export default client;