Update chat_handler.py

chat_handler.py

@@ -155,7 +155,7 @@ def _extract_parameters_from_response(raw: str, session: Session, intent_config)
 
 # ─────────────────────────  API EXECUTION ───────────────────────── #
 async def _execute_api_call(session: Session, intent_config) -> str:
-    """Execute API call and return humanized response"""
+    """Execute API call and return humanized response with better error handling"""
     try:
         session.state = "call_api"
         api_name = intent_config.action
@@ -164,7 +164,7 @@ async def _execute_api_call(session: Session, intent_config) -> str:
         if not api_config:
             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."
+            return get_user_friendly_error("api_error", {"api_name": api_name})
         
         log_info(f"📡 Calling API: {api_name}")
         log_info(f"📦 API variables: {session.variables}")
@@ -191,11 +191,11 @@ async def _execute_api_call(session: Session, intent_config) -> str:
     except requests.exceptions.Timeout:
         log_warning(f"⏱️ API timeout: {api_name}")
         session.reset_flow()
-        return intent_config.fallback_timeout_prompt or "İşlem zaman aşımına uğradı."
+        return get_user_friendly_error("api_timeout")
     except Exception as 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."
+        return get_user_friendly_error("api_error", {"api_name": api_name})
 
 # ─────────────────────────  REQUEST MODELS ───────────────────────── #
 class ChatRequest(BaseModel):
@@ -282,8 +282,25 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
         # Get session
         session = session_store.get(x_session_id)
         if not session:
-            raise HTTPException(status_code=404, detail="Session not found or expired")
+            # Better error message
+            raise HTTPException(
+                status_code=404, 
+                detail=get_user_friendly_error("session_not_found")
+            )
+        
+        # Session expiry check ekle
+        if session.is_expired():
+            session_store.delete_session(x_session_id)
+            raise HTTPException(
+                status_code=401,
+                detail=get_user_friendly_error("session_expired")
+            )
         
+        # Update last activity
+        session.last_activity = datetime.utcnow().isoformat()
+        session_store.update_session(session)
+        
+        # Mevcut kod devam ediyor...
         # Add user message to history
         session.add_message("user", req.message)
         log_info(f"💬 User [{session.session_id[:8]}...]: {req.message}")
@@ -291,13 +308,19 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
         # Get project and version config
         project = next((p for p in cfg.projects if p.name == session.project_name), None)
         if not project:
-            raise HTTPException(status_code=404, detail=f"Project '{session.project_name}' not found")
+            raise HTTPException(
+                status_code=404, 
+                detail=get_user_friendly_error("project_not_found")
+            )
         
         version = session.get_version_config()
         if not version:
-            raise HTTPException(status_code=400, detail="Version config not found in session")
+            raise HTTPException(
+                status_code=400, 
+                detail=get_user_friendly_error("version_not_found")
+            )
         
-        # Process based on current state
+        # Process based on current state - MEVCUT KODU KORUYORUZ
         if session.state == "idle":
             # Build intent detection prompt
             prompt = build_intent_prompt(version, session.chat_history, project.default_locale)
@@ -400,15 +423,28 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
         else:
             # Unknown state, reset
             session.reset_flow()
-            return {"response": "Bir hata oluştu, lütfen tekrar deneyin.", "state": "error"}
+            return {"response": get_user_friendly_error("internal_error"), "state": "error"}
             
     except HTTPException:
         raise
+    except requests.exceptions.Timeout:
+        # Better timeout error
+        log_error(f"Timeout in chat for session {x_session_id[:8]}")
+        return {
+            "response": get_user_friendly_error("llm_timeout"),
+            "state": "error",
+            "error": True
+        }
     except Exception as e:
         log_error("❌ Chat error", e)
         import traceback
         traceback.print_exc()
-        raise HTTPException(status_code=500, detail=str(e))
+        # Better generic error
+        return {
+            "response": get_user_friendly_error("internal_error"),
+            "state": "error",
+            "error": True
+        }
 
 async def handle_new_message(session: Session, user_input: str) -> str:
     """Handle new message (not parameter followup) - for WebSocket"""
@@ -537,5 +573,72 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
         session.reset_flow()
         return "Bir hata oluştu. Lütfen tekrar deneyin."
 
+def get_user_friendly_error(error_type: str, context: dict = None) -> str:
+    """Get user-friendly error messages"""
+    error_messages = {
+        "session_not_found": "Oturumunuz bulunamadı. Lütfen yeni bir konuşma başlatın.",
+        "project_not_found": "Proje konfigürasyonu bulunamadı. Lütfen yönetici ile iletişime geçin.",
+        "version_not_found": "Proje versiyonu bulunamadı. Lütfen geçerli bir versiyon seçin.",
+        "intent_not_found": "Üzgünüm, ne yapmak istediğinizi anlayamadım. Lütfen daha açık bir şekilde belirtir misiniz?",
+        "api_timeout": "İşlem zaman aşımına uğradı. Lütfen tekrar deneyin.",
+        "api_error": "İşlem sırasında bir hata oluştu. Lütfen daha sonra tekrar deneyin.",
+        "parameter_validation": "Girdiğiniz bilgide bir hata var. Lütfen kontrol edip tekrar deneyin.",
+        "llm_error": "Sistem yanıt veremedi. Lütfen biraz sonra tekrar deneyin.",
+        "llm_timeout": "Sistem meşgul. Lütfen birkaç saniye bekleyip tekrar deneyin.",
+        "session_expired": "Oturumunuz zaman aşımına uğradı. Lütfen yeni bir konuşma başlatın.",
+        "rate_limit": "Çok fazla istek gönderdiniz. Lütfen biraz bekleyin.",
+        "internal_error": "Beklenmeyen bir hata oluştu. Lütfen yönetici ile iletişime geçin."
+    }
+    
+    message = error_messages.get(error_type, error_messages["internal_error"])
+    
+    # Context bilgisi varsa ekle
+    if context:
+        if error_type == "parameter_validation" and "field" in context:
+            message = f"{context['field']} alanı için {message}"
+        elif error_type == "api_error" and "api_name" in context:
+            message = f"{context['api_name']} servisi için {message}"
+    
+    return message
+
+def validate_parameter_with_message(param_config, value, locale="tr") -> tuple[bool, str]:
+    """Validate parameter and return user-friendly message"""
+    try:
+        # Type validation
+        if param_config.type == "int":
+            try:
+                int(value)
+            except ValueError:
+                return False, f"Lütfen geçerli bir sayı girin."
+                
+        elif param_config.type == "float":
+            try:
+                float(value)
+            except ValueError:
+                return False, f"Lütfen geçerli bir ondalık sayı girin."
+                
+        elif param_config.type == "date":
+            # Date parsing with locale support
+            from locale_manager import LocaleManager
+            parsed_date = LocaleManager.parse_date_expression(value, locale)
+            if not parsed_date:
+                return False, f"Lütfen geçerli bir tarih girin (örn: yarın, 15 Haziran, 2025-06-15)."
+                
+        elif param_config.type == "bool":
+            if value.lower() not in ["evet", "hayır", "yes", "no", "true", "false"]:
+                return False, f"Lütfen 'evet' veya 'hayır' olarak cevaplayın."
+        
+        # Regex validation
+        if param_config.validation_regex:
+            import re
+            if not re.match(param_config.validation_regex, value):
+                return False, param_config.invalid_prompt or "Girdiğiniz değer geçerli formatta değil."
+        
+        return True, ""
+        
+    except Exception as e:
+        log_error(f"Parameter validation error", e)
+        return False, "Değer kontrol edilirken bir hata oluştu."
+        
 # Initialize LLM on module load
 setup_llm_provider()