Update chat_handler.py

chat_handler.py

@@ -11,7 +11,7 @@ from pydantic import BaseModel
 import requests
 
 from prompt_builder import build_intent_prompt, build_parameter_prompt
-from utils import log
+from logger import log_info, log_error, log_warning, log_debug
 from api_executor import call_api as execute_api  
 from config_provider import ConfigProvider
 from validation_engine import validate
@@ -48,9 +48,9 @@ def _safe_intent_parse(raw: str) -> tuple[str, str]:
     # Remove 'assistant' suffix if exists
     if name.endswith("assistant"):
         name = name[:-9]  # Remove last 9 chars ("assistant")
-        log(f"🔧 Removed 'assistant' suffix from intent name")
+        log_info(f"🔧 Removed 'assistant' suffix from intent name")
     tail = raw[m.end():]
-    log(f"🎯 Parsed intent: {name}")
+    log_info(f"🎯 Parsed intent: {name}")
     return name, tail
 
 # ─────────────────────────  LLM SETUP ───────────────────────── #
@@ -61,9 +61,9 @@ def setup_llm_provider():
     try:
         from llm_factory import LLMFactory
         llm_provider = LLMFactory.create_provider()
-        log("✅ LLM provider initialized successfully")
+        log_info("✅ LLM provider initialized successfully")
     except Exception as e:
-        log(f"❌ Failed to initialize LLM provider: {e}")
+        log_error("❌ Failed to initialize LLM provider", e)
         raise
 
 # ─────────────────────────  LLM GENERATION ───────────────────────── #
@@ -86,8 +86,8 @@ async def llm_generate(s: Session, prompt: str, user_msg: str) -> str:
             if not version:
                 raise ValueError("No published version found")
         
-        log(f"🚀 Calling LLM for session {s.session_id[:8]}...")
-        log(f"📋 Prompt preview (first 200 chars): {prompt[:200]}...")
+        log_info(f"🚀 Calling LLM for session {s.session_id[:8]}...")
+        log_info(f"📋 Prompt preview (first 200 chars): {prompt[:200]}...")
         
         # Call the configured LLM provider
         raw = await llm_provider.generate(
@@ -96,14 +96,14 @@ async def llm_generate(s: Session, prompt: str, user_msg: str) -> str:
             context=s.chat_history[-10:] if s.chat_history else []
         )
         
-        log(f"🪄 LLM raw response: {raw[:100]}...")
+        log_info(f"🪄 LLM raw response: {raw[:100]}...")
         return raw
         
     except requests.exceptions.Timeout:
-        log(f"⏱️ LLM timeout for session {s.session_id[:8]}")
+        log_warning(f"⏱️ LLM timeout for session {s.session_id[:8]}")
         raise HTTPException(status_code=504, detail="LLM request timed out")
     except Exception as e:
-        log(f"❌ LLM error: {e}")
+        log_error("❌ LLM error", e)
         raise HTTPException(status_code=500, detail=f"LLM error: {str(e)}")
 
 # ─────────────────────────  PARAMETER EXTRACTION ───────────────────────── #
@@ -117,7 +117,7 @@ def _extract_parameters_from_response(raw: str, session: Session, intent_config)
             json_match = re.search(r'\{[^}]+\}', raw)
         
         if not json_match:
-            log("❌ No JSON found in response")
+            log_info("❌ No JSON found in response")
             return False
         
         json_str = json_match.group(1) if '```' in raw else json_match.group(0)
@@ -132,25 +132,25 @@ def _extract_parameters_from_response(raw: str, session: Session, intent_config)
             )
             
             if not param_config:
-                log(f"⚠️ Parameter config not found for: {param_name}")
+                log_info(f"⚠️ Parameter config not found for: {param_name}")
                 continue
             
             # Validate parameter
             if validate(str(param_value), param_config):
                 session.variables[param_config.variable_name] = str(param_value)
                 any_valid = True
-                log(f"✅ Extracted {param_name}={param_value} → {param_config.variable_name}")
+                log_info(f"✅ Extracted {param_name}={param_value} → {param_config.variable_name}")
             else:
-                log(f"❌ Invalid {param_name}={param_value}")
+                log_info(f"❌ Invalid {param_name}={param_value}")
         
         return any_valid
         
     except json.JSONDecodeError as e:
-        log(f"❌ JSON parsing error: {e}")
-        log(f"❌ Failed to parse: {raw[:200]}")
+        log_error("❌ JSON parsing error", e)
+        log_error(f"❌ Failed to parse: {raw[:200]}")
         return False
     except Exception as e:
-        log(f"❌ Parameter processing error: {e}")
+        log_error("❌ Parameter processing error", e)
         return False
 
 # ─────────────────────────  API EXECUTION ───────────────────────── #
@@ -162,17 +162,17 @@ async def _execute_api_call(session: Session, intent_config) -> str:
         api_config = cfg.get_api(api_name)
         
         if not api_config:
-            log(f"❌ API config not found: {api_name}")
+            log_info(f"❌ API config not found: {api_name}")
             session.reset_flow()
             return intent_config.fallback_error_prompt or "İşlem başarısız oldu."
         
-        log(f"📡 Calling API: {api_name}")
-        log(f"📦 API variables: {session.variables}")
+        log_info(f"📡 Calling API: {api_name}")
+        log_info(f"📦 API variables: {session.variables}")
         
         # Execute API call with session
         response = execute_api(api_config, session)
         api_json = response.json()
-        log(f"✅ API response: {api_json}")
+        log_info(f"✅ API response: {api_json}")
         
         # Humanize response
         session.state = "humanize"
@@ -189,11 +189,11 @@ async def _execute_api_call(session: Session, intent_config) -> str:
             return f"İşlem tamamlandı: {api_json}"
             
     except requests.exceptions.Timeout:
-        log(f"⏱️ API timeout: {api_name}")
+        log_warning(f"⏱️ API timeout: {api_name}")
         session.reset_flow()
         return intent_config.fallback_timeout_prompt or "İşlem zaman aşımına uğradı."
     except Exception as e:
-        log(f"❌ API call error: {e}")
+        log_error("❌ API call error", e)
         session.reset_flow()
         return intent_config.fallback_error_prompt or "İşlem sırasında bir hata oluştu."
 
@@ -240,7 +240,7 @@ async def start_session(req: StartRequest):
         if not llm_provider:
             from llm_factory import LLMFactory
             llm_provider = LLMFactory.create_provider()
-            log(f"🤖 LLM Provider created: {type(llm_provider).__name__}")
+            log_info(f"🤖 LLM Provider created: {type(llm_provider).__name__}")
         
         # Create session with version config
         session_id = session_store.create(req.project_name, version)
@@ -249,7 +249,7 @@ async def start_session(req: StartRequest):
         # Welcome prompt'u işle
         greeting = "Hoş geldiniz! Size nasıl yardımcı olabilirim?"
         if version.welcome_prompt:
-            log(f"🎉 Processing welcome prompt for session {session_id[:8]}...")
+            log_info(f"🎉 Processing welcome prompt for session {session_id[:8]}...")
             try:
                 # Welcome prompt'u LLM'e gönder
                 welcome_result = await llm_provider.generate(
@@ -260,19 +260,19 @@ async def start_session(req: StartRequest):
                 if welcome_result and welcome_result.strip():
                     greeting = welcome_result.strip()
             except Exception as e:
-                log(f"⚠️ Welcome prompt processing failed: {e}")
+                log_error("⚠️ Welcome prompt processing failed", e)
                 # Fallback to default greeting
         
         session.add_turn("assistant", greeting)
         
-        log(f"✅ Session created for project '{req.project_name}' version {version.no} (highest published)")
+        log_info(f"✅ Session created for project '{req.project_name}' version {version.no} (highest published)")
         
         return ChatResponse(session_id=session.session_id, answer=greeting)
         
     except HTTPException:
         raise
     except Exception as e:
-        log(f"❌ Session creation error: {e}")
+        log_error("❌ Session creation error", e)
         raise HTTPException(500, f"Session creation failed: {str(e)}")
 
 @router.post("/chat")
@@ -286,7 +286,7 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
         
         # Add user message to history
         session.add_message("user", req.message)
-        log(f"💬 User [{session.session_id[:8]}...]: {req.message}")
+        log_info(f"💬 User [{session.session_id[:8]}...]: {req.message}")
         
         # Get project and version config
         project = next((p for p in cfg.projects if p.name == session.project_name), None)
@@ -314,11 +314,11 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
                     session.current_intent = intent_name
                     session.intent_config = intent_config
                     session.state = "collect_params"
-                    log(f"🎯 Intent detected: {intent_name}")
+                    log_info(f"🎯 Intent detected: {intent_name}")
                     
                     # Check if parameters were already extracted
                     if tail and _extract_parameters_from_response(tail, session, intent_config):
-                        log("📦 Some parameters extracted from initial response")
+                        log_info("📦 Some parameters extracted from initial response")
                     
                     # Check what parameters are missing
                     missing_params = [
@@ -344,7 +344,7 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
                         session.add_message("assistant", clean_question)
                         return {"response": clean_question, "intent": intent_name, "state": "collecting_params"}
                 else:
-                    log(f"⚠️ Unknown intent: {intent_name}")
+                    log_info(f"⚠️ Unknown intent: {intent_name}")
             
             # No intent detected, return general response
             clean_response = _trim_response(raw)
@@ -405,7 +405,7 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
     except HTTPException:
         raise
     except Exception as e:
-        log(f"❌ Chat error: {e}")
+        log_error("❌ Chat error", e)
         import traceback
         traceback.print_exc()
         raise HTTPException(status_code=500, detail=str(e))
@@ -416,7 +416,7 @@ async def handle_new_message(session: Session, user_input: str) -> str:
         # Get version config from session
         version = session.get_version_config()
         if not version:
-            log("❌ Version config not found")
+            log_info("❌ Version config not found")
             return "Bir hata oluştu. Lütfen tekrar deneyin."
         
         # Get project config
@@ -432,7 +432,7 @@ async def handle_new_message(session: Session, user_input: str) -> str:
         
         # Empty response fallback
         if not raw:
-            log("⚠️ Empty response from LLM")
+            log_info("⚠️ Empty response from LLM")
             return "Üzgünüm, mesajınızı anlayamadım. Lütfen tekrar dener misiniz?"
         
         # Check for intent
@@ -446,11 +446,11 @@ async def handle_new_message(session: Session, user_input: str) -> str:
                 session.current_intent = intent_name
                 session.intent_config = intent_config
                 session.state = "collect_params"
-                log(f"🎯 Intent detected: {intent_name}")
+                log_info(f"🎯 Intent detected: {intent_name}")
                 
                 # Check if parameters were already extracted
                 if tail and _extract_parameters_from_response(tail, session, intent_config):
-                    log("📦 Some parameters extracted from initial response")
+                    log_info("📦 Some parameters extracted from initial response")
                 
                 # Check what parameters are missing
                 missing_params = [
@@ -476,14 +476,14 @@ async def handle_new_message(session: Session, user_input: str) -> str:
         return _trim_response(raw)
         
     except Exception as e:
-        log(f"❌ Error in handle_new_message: {e}")
+        log_error("❌ Error in handle_new_message", e)
         return "Bir hata oluştu. Lütfen tekrar deneyin."
 
 async def handle_parameter_followup(session: Session, user_input: str) -> str:
     """Handle parameter collection followup - for WebSocket"""
     try:
         if not session.intent_config:
-            log("⚠️ No intent config in session")
+            log_info("⚠️ No intent config in session")
             session.reset_flow()
             return "Üzgünüm, hangi işlem için bilgi istediğimi unuttum. Baştan başlayalım."
         
@@ -533,7 +533,7 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
             return _trim_response(param_question)
             
     except Exception as e:
-        log(f"❌ Error in handle_parameter_followup: {e}")
+        log_error("❌ Error in handle_parameter_followup", e)
         session.reset_flow()
         return "Bir hata oluştu. Lütfen tekrar deneyin."