Update chat_handler.py

chat_handler.py

@@ -369,10 +369,10 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
                     
                     # Check what parameters are missing
                     missing_params = [
-                        p for p in intent_config.parameters
+                        p.name for p in intent_config.parameters
                         if p.required and p.variable_name not in session.variables
                     ]
-                    
+
                     if not missing_params:
                         # All required parameters collected, execute API
                         response = await _execute_api_call(session, intent_config)
@@ -380,16 +380,31 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
                         return {"response": response, "intent": intent_name, "state": "completed"}
                     else:
                         # Need to collect more parameters
+                        # Get parameter collection config
+                        cfg = ConfigProvider.get()
+                        collection_config = cfg.global_config.llm_provider.settings.get("parameter_collection_config", {})
+                        max_params = collection_config.get("max_params_per_question", 2)
+                        
+                        # Decide which parameters to ask
+                        params_to_ask = missing_params[:max_params]
+                        
                         param_prompt = build_parameter_prompt(
-                            intent_config, 
-                            session.variables, 
-                            session.chat_history,
-                            project.default_locale
+                            version=version,
+                            intent_config=intent_config,
+                            chat_history=session.chat_history,
+                            collected_params=session.variables,
+                            missing_params=missing_params,
+                            params_to_ask=params_to_ask,
+                            max_params=max_params,
+                            project_locale=project.default_locale,
+                            unanswered_params=session.unanswered_parameters
                         )
+                        
                         param_question = await llm_generate(session, param_prompt, req.message)
                         clean_question = _trim_response(param_question)
                         session.add_message("assistant", clean_question)
-                        return {"response": clean_question, "intent": intent_name, "state": "collecting_params"}
+                        return {"response": clean_question, "intent": intent_name, "state": "collecting_params"}                
+
                 else:
                     log_info(f"⚠️ Unknown intent: {intent_name}")
             
@@ -422,7 +437,7 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
             
             # Check what parameters are still missing
             missing_params = [
-                p for p in intent_config.parameters
+                p.name for p in intent_config.parameters
                 if p.required and p.variable_name not in session.variables
             ]
             
@@ -433,11 +448,24 @@ async def chat(req: ChatRequest, x_session_id: str = Header(...)):
                 return {"response": response, "intent": session.current_intent, "state": "completed"}
             else:
                 # Still need more parameters
+                # Get parameter collection config
+                cfg = ConfigProvider.get()
+                collection_config = cfg.global_config.llm_provider.settings.get("parameter_collection_config", {})
+                max_params = collection_config.get("max_params_per_question", 2)
+                
+                # Decide which parameters to ask
+                params_to_ask = missing_params[:max_params]
+                
                 param_prompt = build_parameter_prompt(
-                    intent_config, 
-                    session.variables, 
-                    session.chat_history,
-                    project.default_locale
+                    version=version,
+                    intent_config=intent_config,
+                    chat_history=session.chat_history,
+                    collected_params=session.variables,
+                    missing_params=missing_params,
+                    params_to_ask=params_to_ask,
+                    max_params=max_params,
+                    project_locale=project.default_locale,
+                    unanswered_params=session.unanswered_parameters
                 )
                 param_question = await llm_generate(session, param_prompt, req.message)
                 clean_question = _trim_response(param_question)
@@ -514,7 +542,7 @@ async def handle_new_message(session: Session, user_input: str) -> str:
                 
                 # Check what parameters are missing
                 missing_params = [
-                    p for p in intent_config.parameters
+                    p.name for p in intent_config.parameters
                     if p.required and p.variable_name not in session.variables
                 ]
                 
@@ -523,11 +551,23 @@ async def handle_new_message(session: Session, user_input: str) -> str:
                     return await _execute_api_call(session, intent_config)
                 else:
                     # Need to collect more parameters
+                    cfg = ConfigProvider.get()
+                    collection_config = cfg.global_config.llm_provider.settings.get("parameter_collection_config", {})
+                    max_params = collection_config.get("max_params_per_question", 2)
+                    
+                    # Decide which parameters to ask
+                    params_to_ask = missing_params[:max_params]
+                    
                     param_prompt = build_parameter_prompt(
-                        intent_config, 
-                        session.variables, 
-                        session.chat_history,
-                        project.default_locale
+                        version=version,
+                        intent_config=intent_config,
+                        chat_history=session.chat_history,
+                        collected_params=session.variables,
+                        missing_params=missing_params,
+                        params_to_ask=params_to_ask,
+                        max_params=max_params,
+                        project_locale=project.default_locale,
+                        unanswered_params=session.unanswered_parameters
                     )
                     param_question = await llm_generate(session, param_prompt, user_input)
                     return _trim_response(param_question)
@@ -538,7 +578,7 @@ async def handle_new_message(session: Session, user_input: str) -> str:
     except Exception as 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:
@@ -549,11 +589,15 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
         
         intent_config = session.intent_config
         
-        # Get project config
+        # Get project config and version
         project = next((p for p in cfg.projects if p.name == session.project_name), None)
         if not project:
             return "Proje konfigürasyonu bulunamadı."
         
+        version = session.get_version_config()
+        if not version:
+            return "Versiyon konfigürasyonu bulunamadı."
+        
         # Try to extract parameters from user message
         param_prompt = f"""
         Extract parameters from user message: "{user_input}"
@@ -574,7 +618,7 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
         
         # Check what parameters are still missing
         missing_params = [
-            p for p in intent_config.parameters
+            p.name for p in intent_config.parameters  # p.name olmalı, sadece p değil
             if p.required and p.variable_name not in session.variables
         ]
         
@@ -583,11 +627,23 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
             return await _execute_api_call(session, intent_config)
         else:
             # Still need more parameters
+            cfg = ConfigProvider.get()
+            collection_config = cfg.global_config.llm_provider.settings.get("parameter_collection_config", {})
+            max_params = collection_config.get("max_params_per_question", 2)
+            
+            # Decide which parameters to ask
+            params_to_ask = missing_params[:max_params]
+            
             param_prompt = build_parameter_prompt(
-                intent_config, 
-                session.variables, 
-                session.chat_history,
-                project.default_locale
+                version=version,
+                intent_config=intent_config,
+                chat_history=session.chat_history,
+                collected_params=session.variables,
+                missing_params=missing_params,
+                params_to_ask=params_to_ask,
+                max_params=max_params,
+                project_locale=project.default_locale,
+                unanswered_params=session.unanswered_parameters
             )
             param_question = await llm_generate(session, param_prompt, user_input)
             return _trim_response(param_question)
@@ -596,7 +652,7 @@ async def handle_parameter_followup(session: Session, user_input: str) -> str:
         log_error("❌ Error in handle_parameter_followup", e)
         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 = {