update

Dockerfile

@@ -3,7 +3,7 @@ FROM node:20-alpine AS frontend-build
 WORKDIR /frontend
 
 # Accept build arguments for Vite environment variables
-# Hugging Face Spaces will automatically pass environment variables as build args
+# Hugging Face Spaces should pass these as build args, but we'll also check environment
 ARG VITE_FIREBASE_API_KEY
 ARG VITE_FIREBASE_AUTH_DOMAIN
 ARG VITE_FIREBASE_PROJECT_ID
@@ -12,7 +12,8 @@ ARG VITE_FIREBASE_MESSAGING_SENDER_ID
 ARG VITE_FIREBASE_APP_ID
 ARG VITE_API_BASE_URL
 
-# Set as environment variables so they're available to the build script
+# Export as environment variables so the build script can access them
+# Use ARG value if provided, otherwise it will be empty and script will check env
 ENV VITE_FIREBASE_API_KEY=$VITE_FIREBASE_API_KEY
 ENV VITE_FIREBASE_AUTH_DOMAIN=$VITE_FIREBASE_AUTH_DOMAIN
 ENV VITE_FIREBASE_PROJECT_ID=$VITE_FIREBASE_PROJECT_ID
@@ -25,25 +26,13 @@ ENV VITE_API_BASE_URL=$VITE_API_BASE_URL
 COPY frontend/package*.json ./
 RUN npm install
 
-# Copy rest of frontend
+# Copy rest of frontend (including build script)
 COPY frontend/ .
 
-# Create .env file from build arguments and build
-# Hugging Face Spaces will pass these as build arguments from environment variables
-RUN echo "Checking environment variables..." && \
-    [ -z "$VITE_FIREBASE_API_KEY" ] && echo "WARNING: VITE_FIREBASE_API_KEY is not set" || echo "✓ VITE_FIREBASE_API_KEY is set" && \
-    [ -z "$VITE_FIREBASE_AUTH_DOMAIN" ] && echo "WARNING: VITE_FIREBASE_AUTH_DOMAIN is not set" || echo "✓ VITE_FIREBASE_AUTH_DOMAIN is set" && \
-    [ -z "$VITE_FIREBASE_PROJECT_ID" ] && echo "WARNING: VITE_FIREBASE_PROJECT_ID is not set" || echo "✓ VITE_FIREBASE_PROJECT_ID is set" && \
-    echo "VITE_FIREBASE_API_KEY=${VITE_FIREBASE_API_KEY}" > .env && \
-    echo "VITE_FIREBASE_AUTH_DOMAIN=${VITE_FIREBASE_AUTH_DOMAIN}" >> .env && \
-    echo "VITE_FIREBASE_PROJECT_ID=${VITE_FIREBASE_PROJECT_ID}" >> .env && \
-    echo "VITE_FIREBASE_STORAGE_BUCKET=${VITE_FIREBASE_STORAGE_BUCKET}" >> .env && \
-    echo "VITE_FIREBASE_MESSAGING_SENDER_ID=${VITE_FIREBASE_MESSAGING_SENDER_ID}" >> .env && \
-    echo "VITE_FIREBASE_APP_ID=${VITE_FIREBASE_APP_ID}" >> .env && \
-    echo "VITE_API_BASE_URL=${VITE_API_BASE_URL}" >> .env && \
-    echo "Created .env file with environment variables" && \
-    echo "Contents of .env file:" && \
-    cat .env && \
+# Make build script executable and run it to create .env file
+# Hugging Face Spaces makes environment variables available during build
+RUN chmod +x build-env.sh && \
+    ./build-env.sh && \
     npm run build
 # Vite will output to /frontend/dist by default
 

frontend/build-env.sh

@@ -1,22 +1,43 @@
 #!/bin/sh
-# Script to create .env file from environment variables for Vite build
-# This is used in Docker build when environment variables are available
+# Build script that creates .env file from environment variables
+# This works with Hugging Face Spaces which makes env vars available during build
 
-# Debug: Check if variables are set (without exposing values)
-echo "Checking environment variables..."
-[ -z "$VITE_FIREBASE_API_KEY" ] && echo "WARNING: VITE_FIREBASE_API_KEY is not set" || echo "✓ VITE_FIREBASE_API_KEY is set"
-[ -z "$VITE_FIREBASE_AUTH_DOMAIN" ] && echo "WARNING: VITE_FIREBASE_AUTH_DOMAIN is not set" || echo "✓ VITE_FIREBASE_AUTH_DOMAIN is set"
-[ -z "$VITE_FIREBASE_PROJECT_ID" ] && echo "WARNING: VITE_FIREBASE_PROJECT_ID is not set" || echo "✓ VITE_FIREBASE_PROJECT_ID is set"
+set -e  # Exit on error
 
-cat > .env << EOF
-VITE_FIREBASE_API_KEY=${VITE_FIREBASE_API_KEY:-}
-VITE_FIREBASE_AUTH_DOMAIN=${VITE_FIREBASE_AUTH_DOMAIN:-}
-VITE_FIREBASE_PROJECT_ID=${VITE_FIREBASE_PROJECT_ID:-}
-VITE_FIREBASE_STORAGE_BUCKET=${VITE_FIREBASE_STORAGE_BUCKET:-}
-VITE_FIREBASE_MESSAGING_SENDER_ID=${VITE_FIREBASE_MESSAGING_SENDER_ID:-}
-VITE_FIREBASE_APP_ID=${VITE_FIREBASE_APP_ID:-}
-VITE_API_BASE_URL=${VITE_API_BASE_URL:-}
-EOF
+echo "=== Checking environment variables ==="
+echo "Note: Variables can come from ARG (build args) or ENV (environment)"
 
-echo "Created .env file with environment variables"
+# Check and report status - check both ARG (from ENV) and direct environment
+API_KEY="${VITE_FIREBASE_API_KEY}"
+AUTH_DOMAIN="${VITE_FIREBASE_AUTH_DOMAIN}"
+PROJECT_ID="${VITE_FIREBASE_PROJECT_ID}"
+STORAGE_BUCKET="${VITE_FIREBASE_STORAGE_BUCKET}"
+MESSAGING_SENDER_ID="${VITE_FIREBASE_MESSAGING_SENDER_ID}"
+APP_ID="${VITE_FIREBASE_APP_ID}"
+API_BASE_URL="${VITE_API_BASE_URL}"
 
+[ -z "$API_KEY" ] && echo "WARNING: VITE_FIREBASE_API_KEY is not set" || echo "✓ VITE_FIREBASE_API_KEY is set"
+[ -z "$AUTH_DOMAIN" ] && echo "WARNING: VITE_FIREBASE_AUTH_DOMAIN is not set" || echo "✓ VITE_FIREBASE_AUTH_DOMAIN is set"
+[ -z "$PROJECT_ID" ] && echo "WARNING: VITE_FIREBASE_PROJECT_ID is not set" || echo "✓ VITE_FIREBASE_PROJECT_ID is set"
+[ -z "$STORAGE_BUCKET" ] && echo "WARNING: VITE_FIREBASE_STORAGE_BUCKET is not set" || echo "✓ VITE_FIREBASE_STORAGE_BUCKET is set"
+[ -z "$MESSAGING_SENDER_ID" ] && echo "WARNING: VITE_FIREBASE_MESSAGING_SENDER_ID is not set" || echo "✓ VITE_FIREBASE_MESSAGING_SENDER_ID is set"
+[ -z "$APP_ID" ] && echo "WARNING: VITE_FIREBASE_APP_ID is not set" || echo "✓ VITE_FIREBASE_APP_ID is set"
+[ -z "$API_BASE_URL" ] && echo "WARNING: VITE_API_BASE_URL is not set" || echo "✓ VITE_API_BASE_URL is set"
+
+echo ""
+echo "=== Creating .env file ==="
+
+# Create .env file (use empty string if variable is not set)
+echo "VITE_FIREBASE_API_KEY=${API_KEY}" > .env
+echo "VITE_FIREBASE_AUTH_DOMAIN=${AUTH_DOMAIN}" >> .env
+echo "VITE_FIREBASE_PROJECT_ID=${PROJECT_ID}" >> .env
+echo "VITE_FIREBASE_STORAGE_BUCKET=${STORAGE_BUCKET}" >> .env
+echo "VITE_FIREBASE_MESSAGING_SENDER_ID=${MESSAGING_SENDER_ID}" >> .env
+echo "VITE_FIREBASE_APP_ID=${APP_ID}" >> .env
+echo "VITE_API_BASE_URL=${API_BASE_URL}" >> .env
+
+echo "✓ Created .env file"
+echo ""
+echo "=== Contents of .env file ==="
+cat .env
+echo ""