Merge pull request #36 from MisbahAN/final-backend-voice

finallll tryyyy

Author: mehaksahjwani

api/lessons/[uid].ts

@@ -57,6 +57,7 @@ export default async function handler(
     const { uid } = req.query;
 
     if (!uid || typeof uid !== "string") {
+      setCorsHeaders(res);
       return res.status(400).json({ error: "Missing or invalid uid parameter" });
     }
 
@@ -68,13 +69,15 @@ export default async function handler(
     const user = await usersCollection.findOne({ firebaseUid: uid });
 
     if (!user) {
+      setCorsHeaders(res);
       return res.status(404).json({ error: "User not found" });
     }
 
     // Get user's lessons
     const lessonData = await lessonsCollection.findOne({ firebaseUid: uid });
 
     if (!lessonData) {
+      setCorsHeaders(res);
       return res.status(404).json({
         error: "No lessons found",
         message: "Lessons have not been generated for this user yet"
@@ -166,6 +169,7 @@ export default async function handler(
 
   } catch (error) {
     console.error("Error fetching lessons:", error);
+    setCorsHeaders(res); // Ensure CORS headers on error responses
     return res.status(500).json({
       error: "Failed to fetch lessons",
       message: error instanceof Error ? error.message : "Unknown error",

api/lessons/generate.ts

@@ -124,11 +124,10 @@ Q&A: ${qaText}
 
 REQUIREMENTS:
 - ${lessonCount} progressive lessons (${isFirstBatch ? 'fundamentals' : 'intermediate to advanced'})
-- Each lesson: 2-3 reading cards (100-150 words each), 2 MCQ cards (4 options), 1 summary card
-- Canadian context (TFSA, RRSP)
+- Each lesson: 1-2 reading cards (50-100 words each), 2 MCQ cards (4 options), 1 summary card
+- Canadian context (TFSA, RRSP), Finny's voice
 - XP: Lessons 1-3 = 30, 4-7 = 35-40, 8-10 = 40-50
-- Engaging titles, Finny's voice
-${isFirstBatch ? '' : '- These lessons should build on fundamentals from lessons 1-3'}
+${isFirstBatch ? '' : '- Build on fundamentals from lessons 1-3'}
 
 Return ONLY JSON (no markdown):
 {
@@ -144,10 +143,10 @@ Return ONLY JSON (no markdown):
       "prerequisites": [],
       "xpReward": ${startLesson <= 3 ? 30 : startLesson <= 7 ? 35 : 40},
       "cards": [
-        {"type": "reading", "content": "100-150 word explanation"},
-        {"type": "mcq", "question": "Question?", "options": ["A", "B", "C", "D"], "correctIndex": 0, "explanation": "Why correct"},
-        {"type": "reading", "content": "100-150 words"},
-        {"type": "mcq", "question": "Question?", "options": ["A", "B", "C", "D"], "correctIndex": 1, "explanation": "Why correct"},
+        {"type": "reading", "content": "50-100 word explanation"},
+        {"type": "mcq", "question": "Question?", "options": ["A", "B", "C", "D"], "correctIndex": 0, "explanation": "Brief explanation"},
+        {"type": "reading", "content": "50-100 words"},
+        {"type": "mcq", "question": "Question?", "options": ["A", "B", "C", "D"], "correctIndex": 1, "explanation": "Brief explanation"},
         {"type": "summary", "xpEarned": ${startLesson <= 3 ? 30 : startLesson <= 7 ? 35 : 40}, "streakDays": 1}
       ]
     }
@@ -163,18 +162,18 @@ Return ONLY JSON (no markdown):
     try {
       const model = genAI.getGenerativeModel({ model: modelName });
 
-      // Use minimal thinking for faster models to speed up generation
+      // Use minimal thinking for ALL flash/lite models to speed up generation
       const generationConfig: { thinkingConfig: { thinkingLevel: "MINIMAL" } } | undefined = 
-        modelName === "gemini-3-flash-preview" 
+        (modelName.includes("flash") || modelName.includes("lite"))
         ? { 
             thinkingConfig: { 
               thinkingLevel: "MINIMAL" as const 
             } 
           }
         : undefined;
 
-      // Set timeout: 30s for fast models, 60s for slower ones
-      const timeout = modelName.includes("flash") || modelName.includes("lite") ? 30000 : 60000;
+      // Set timeout: 20s for fast models, 40s for slower ones (reduced for faster failure)
+      const timeout = modelName.includes("flash") || modelName.includes("lite") ? 20000 : 40000;
 
       const generateContentPromise = model.generateContent(
         prompt, 
@@ -376,6 +375,7 @@ export default async function handler(
 
     if (!firebaseUid) {
       console.error("=== API ERROR: Missing firebaseUid ===");
+      setCorsHeaders(res);
       return res.status(400).json({ error: "Missing firebaseUid" });
     }
 
@@ -391,6 +391,7 @@ export default async function handler(
 
     if (!user) {
       console.error("=== API ERROR: User not found ===");
+      setCorsHeaders(res);
       return res.status(404).json({ error: "User not found" });
     }
 
@@ -399,6 +400,7 @@ export default async function handler(
     console.log("onboardingCompleted:", user.onboardingCompleted);
     if (!user.onboardingCompleted) {
       console.error("=== API ERROR: Onboarding not completed ===");
+      setCorsHeaders(res);
       return res.status(400).json({
         error: "Onboarding not completed",
         message: "User must complete onboarding before generating lessons"
@@ -510,6 +512,7 @@ export default async function handler(
 
   } catch (error) {
     console.error("Error generating lessons:", error);
+    setCorsHeaders(res); // Ensure CORS headers on error responses
     return res.status(500).json({
       error: "Failed to generate lessons",
       message: error instanceof Error ? error.message : "Unknown error",