Fix critical NoneType errors in hybrid LLM service

- Remove duplicate and broken method definitions
- Properly implement analyze_task_complexity method with keywords
- Add null checks for provider.value and complexity.value access
- Fix method logic confusion between task complexity and provider selection
- Ensure all methods return valid enum values

hybrid_llm_service.py

@@ -85,10 +85,6 @@ class HybridLLMService:
 
     def analyze_task_complexity(self, message: str) -> TaskComplexity:
         """Analyze if a task requires complex reasoning or simple response"""
-    
-    def determine_task_complexity(self, message: str, context: str = "") -> TaskComplexity:
-        """Determine task complexity - alias for analyze_task_complexity for compatibility"""
-        return self.analyze_task_complexity(message)
         complex_keywords = [
             'analyze', 'compare', 'evaluate', 'scenario', 'chart', 'graph', 
             'visualization', 'complex', 'detailed analysis', 'multi-step',
@@ -111,6 +107,10 @@ class HybridLLMService:
             return TaskComplexity.COMPLEX
         
         return TaskComplexity.SIMPLE
+    
+    def determine_task_complexity(self, message: str, context: str = "") -> TaskComplexity:
+        """Determine task complexity - alias for analyze_task_complexity for compatibility"""
+        return self.analyze_task_complexity(message)
 
     def choose_llm_provider(self, message: str) -> LLMProvider:
         """Choose the best LLM provider based on task complexity and availability"""
@@ -146,7 +146,9 @@ At least one of these API keys is required for the Voice Bot to function properl
         provider = self.choose_llm_provider(message)
         complexity = self.analyze_task_complexity(message)
         
-        logger.info(f"🎯 Using {provider.value} for {complexity.value} task")
+        provider_name = provider.value if provider else "unknown"
+        complexity_name = complexity.value if complexity else "unknown"
+        logger.info(f"🎯 Using {provider_name} for {complexity_name} task")
         
         # If system_prompt is provided, prepend it to the context
         if system_prompt:
@@ -169,7 +171,8 @@ At least one of these API keys is required for the Voice Bot to function properl
                     return "I apologize, but no AI providers are currently available. Please check your API keys."
                     
         except Exception as e:
-            logger.error(f"❌ Error with {provider.value}: {e}")
+            provider_name = provider.value if provider else "unknown"
+            logger.error(f"❌ Error with {provider_name}: {e}")
             
             # Try fallback provider
             if provider == LLMProvider.GROQ and self.gemini_llm:
@@ -260,7 +263,8 @@ At least one of these API keys is required for the Voice Bot to function properl
                     yield "No AI providers are currently available."
                     
         except Exception as e:
-            logger.error(f"❌ Streaming error with {provider.value}: {e}")
+            provider_name = provider.value if provider else "unknown"
+            logger.error(f"❌ Streaming error with {provider_name}: {e}")
             
             # Try fallback
             if provider == LLMProvider.GROQ and self.gemini_llm: