Update rag_service.py

rag_service.py

@@ -152,7 +152,7 @@ class RAGService:
             )
 
     def ask_question(self, question: str):
-        """Process a question and return response"""
+    """Process a question and return response"""
         try:
             # Check if RAG retrieval finds relevant context
             context_length = self._check_context_length(question)
@@ -171,10 +171,10 @@ class RAGService:
                 # First, try to get a normal response
                 try:
                     response = self.chain_with_sources.invoke(question)
-                    result = response['response']
+                    result = response.get('response') if response else None
                     
-                    # Check if response seems incomplete or ambiguous
-                    if self._is_response_ambiguous(result, question):
+                    # Check if response is None or invalid
+                    if self._is_response_invalid(result):
                         return self._generate_counter_questions(question, context_text)
                     
                     return result
@@ -192,19 +192,15 @@ class RAGService:
             print(f"Error in ask_question: {e}")
             return f"I encountered an error processing your question. Could you please rephrase it more clearly?"
 
-    def _is_response_ambiguous(self, response, question):
-        """Check if the response seems ambiguous or incomplete"""
-        if not response or len(response.strip()) < 20:
+    def _is_response_invalid(self, response):
+        """Check if the response is None or invalid"""
+        # Check if response is None, empty, or too short
+        if response is None:
+            return True
+        if not response or len(response.strip()) < 5:
             return True
         
-        # Check for ambiguous response indicators
-        ambiguous_indicators = [
-            "i'm not sure", "unclear", "ambiguous", "could you clarify",
-            "multiple interpretations", "not specific enough", "vague"
-        ]
-        
-        response_lower = response.lower()
-        return any(indicator in response_lower for indicator in ambiguous_indicators)
+        return False
 
     def _generate_counter_questions(self, original_question, context_text):
         """Generate counter questions based on retrieved context"""