Update src/app.py

src/app.py

@@ -481,35 +481,32 @@ with tab3:
             else:
                 valid_question_found = False
                 attempts = 0
+                last_error = None
+
                 while not valid_question_found and attempts < 5:
                     attempts += 1
-                    # Pass the focus topic here!
                     q_ctx = quiz.get_document_context(st.session_state.username, topic_filter=focus_topic)
                     
+                    # ERROR HANDLING SWITCH
+                    if q_ctx and "error" in q_ctx:
+                        last_error = q_ctx["error"]
+                        break # Stop trying, we know why it failed (e.g., Topic not found)
+                    
                     if q_ctx:
                         prompt = quiz.construct_question_generation_prompt(q_ctx["context_text"])
-                        response_text, usage = query_model_universal([{"role": "user", "content": prompt}], 300, model_choice, st.session_state.get("user_openai_key"))
+                        question_text, usage = query_model_universal([{"role": "user", "content": prompt}], 300, model_choice, st.session_state.get("user_openai_key"))
                         
-                        # PARSE OUTPUT
-                        if "UNABLE" not in response_text and "QUOTE:" in response_text:
-                            # Split into Question and Quote
-                            parts = response_text.split("QUOTE:")
-                            q_text = parts[0].replace("QUESTION:", "").strip()
-                            quote_text = parts[1].strip()
-                            
-                            # SAFETY CHECK: Is the quote actually in the text?
-                            # We use a loose check (in case of minor whitespace diffs)
-                            # We take the first 20 chars of the quote to verify location
-                            if quote_text[:20] in q_ctx["context_text"]:
-                                valid_question_found = True
-                                qs["active"] = True
-                                qs["question_data"] = q_ctx
-                                qs["generated_question_text"] = q_text
-                                qs["feedback"] = None
+                        if "UNABLE" not in question_text and len(question_text) > 10:
+                            valid_question_found = True; qs["active"] = True; qs["question_data"] = q_ctx; qs["generated_question_text"] = question_text; qs["feedback"] = None
                 
                 if not valid_question_found:
-                    if focus_topic: st.warning(f"No documents found containing '{focus_topic}'. Try a different keyword.")
-                    else: st.warning("Could not generate a question. Documents may be too sparse.")
+                    # SPECIFIC USER FEEDBACK
+                    if last_error == "topic_not_found":
+                        st.warning(f"Could not find any documents containing the topic: **'{focus_topic}'**. \n\nCheck your spelling or try a broader term.")
+                    elif focus_topic: 
+                        st.warning(f"Found documents with '{focus_topic}', but the AI struggled to form a question. Try again or check the document content.")
+                    else: 
+                        st.warning("Could not generate a question. Your 'Knowledge & Tools' local cache might be empty or corrupted. \n\n**Try clicking '🔄 Resync from Pinecone' in Tab 2.**")
 
     if st.session_state.quiz_trigger:
         st.session_state.quiz_trigger = False