bug fix

Author: zakandrewking

src/app/PuzzleClient.tsx

@@ -253,7 +253,7 @@ export default function PuzzleClient() {
     } catch (error) {
       console.error("Failed to send completion email:", error);
     }
-  }, []);
+  }, [gridSize, elapsedTime, moveCount]);
 
   // Whenever tiles change, check if puzzle is solved
   useEffect(() => {

supabase/README.md

@@ -19,10 +19,27 @@ serve(async (req) => {
     // Get the request data
     const data: PuzzleCompletionData = await req.json();
 
+    // Validate the data
+    console.log("Received puzzle completion data:", JSON.stringify(data));
+
+    if (!data.gridSize || data.gridSize <= 0) {
+      throw new Error(`Invalid grid size: ${data.gridSize}`);
+    }
+
+    if (data.timeInSeconds === undefined || data.timeInSeconds < 0) {
+      throw new Error(`Invalid time: ${data.timeInSeconds}`);
+    }
+
+    if (data.moveCount === undefined || data.moveCount < 0) {
+      throw new Error(`Invalid move count: ${data.moveCount}`);
+    }
+
     // Format the time
     const minutes = Math.floor(data.timeInSeconds / 60);
     const seconds = data.timeInSeconds % 60;
-    const timeFormatted = `${minutes} minutes and ${seconds} seconds`;
+    const timeFormatted = `${minutes} ${
+      minutes === 1 ? "minute" : "minutes"
+    } and ${seconds} ${seconds === 1 ? "second" : "seconds"}`;
 
     // Your email service configuration
     const RESEND_EMAIL_SERVICE_API_KEY = Deno.env.get(
@@ -40,43 +57,57 @@ serve(async (req) => {
     }`;
     const body = `Completed the ${data.gridSize}x${data.gridSize} puzzle in ${timeFormatted} with ${data.moveCount} moves.`;
 
-    // Send the email using your preferred email service
-    // This example uses a generic fetch approach - you would replace this with your specific email service
+    console.log("Sending email with subject:", subject);
+    console.log("Email body:", body);
 
+    // Send the email using Resend
     const response = await fetch("https://api.resend.com/emails", {
       method: "POST",
       headers: {
         "Content-Type": "application/json",
         Authorization: `Bearer ${RESEND_EMAIL_SERVICE_API_KEY}`,
       },
       body: JSON.stringify({
-        from: "[email protected]",
+        from: "Brainsx3 Robots <[email protected]>",
         to: NOTIFICATION_EMAIL,
         subject: subject,
         text: body,
       }),
     });
 
     if (!response.ok) {
-      const error = await response.text();
-      throw new Error(`Failed to send email: ${error}`);
+      const errorText = await response.text();
+      console.error("Email service error:", errorText);
+      throw new Error(`Failed to send email: ${errorText}`);
     }
 
     // Return a success response
     return new Response(
       JSON.stringify({
         success: true,
         message: "Email notification sent successfully",
+        data: {
+          subject,
+          body,
+          recipient: NOTIFICATION_EMAIL,
+        },
       }),
       {
         headers: { ...corsHeaders, "Content-Type": "application/json" },
         status: 200,
       }
     );
   } catch (error) {
+    // Log the error
+    console.error("Error in send-puzzle-completion-email:", error);
+
     // Return an error response
     return new Response(
-      JSON.stringify({ success: false, error: error.message }),
+      JSON.stringify({
+        success: false,
+        error: error instanceof Error ? error.message : String(error),
+        stack: error instanceof Error ? error.stack : undefined,
+      }),
       {
         headers: { ...corsHeaders, "Content-Type": "application/json" },
         status: 500,