Update admin_routes.py

admin_routes.py

@@ -28,6 +28,22 @@ from config_models import VersionConfig, IntentConfig, LLMConfiguration
 security = HTTPBearer()
 router = APIRouter(tags=["admin"])
 
+# ===================== Decarators =====================
+def handle_exceptions(func):
+    """Decorator to handle exceptions consistently"""
+    @wraps(func)
+    async def wrapper(*args, **kwargs):
+        try:
+            return await func(*args, **kwargs)
+        except FlareException:
+            # Let global handlers deal with our custom exceptions
+            raise
+        except Exception as e:
+            # Log and convert unexpected exceptions to HTTP 500
+            log_error(f"❌ Unexpected error in {func.__name__}", e)
+            raise HTTPException(status_code=500, detail=str(e))
+    return wrapper
+
 # ===================== Models =====================
 class LoginRequest(BaseModel):
     username: str
@@ -125,6 +141,7 @@ class TestRequest(BaseModel):
 
 # ===================== Auth Endpoints =====================
 @router.post("/login", response_model=LoginResponse)
+@handle_exceptions
 async def login(request: LoginRequest):
     """User login endpoint"""
     cfg = ConfigProvider.get()
@@ -159,6 +176,7 @@ async def login(request: LoginRequest):
     return LoginResponse(token=token, username=request.username)
 
 @router.post("/change-password")
+@handle_exceptions
 async def change_password(
     request: ChangePasswordRequest,
     username: str = Depends(verify_token)
@@ -205,6 +223,7 @@ async def change_password(
 
 # ===================== Locales Endpoints =====================
 @router.get("/locales")
+@handle_exceptions
 async def get_available_locales(username: str = Depends(verify_token)):
     """Get all system-supported locales"""
     from locale_manager import LocaleManager
@@ -217,6 +236,7 @@ async def get_available_locales(username: str = Depends(verify_token)):
     }
 
 @router.get("/locales/{locale_code}")
+@handle_exceptions
 async def get_locale_details(
     locale_code: str,
     username: str = Depends(verify_token)
@@ -233,6 +253,7 @@ async def get_locale_details(
     
 # ===================== Environment Endpoints =====================
 @router.get("/environment")
+@handle_exceptions
 async def get_environment(username: str = Depends(verify_token)):
     """Get environment configuration with provider info"""
     cfg = ConfigProvider.get()
@@ -359,6 +380,7 @@ async def get_environment(username: str = Depends(verify_token)):
     return response
 
 @router.put("/environment")
+@handle_exceptions
 async def update_environment(
     update: EnvironmentUpdate,
     username: str = Depends(verify_token)
@@ -403,12 +425,14 @@ async def update_environment(
 
 # ===================== Project Endpoints =====================
 @router.get("/projects/names")
+@handle_exceptions
 def list_enabled_projects():
     """Get list of enabled project names for chat"""
     cfg = ConfigProvider.get()
     return [p.name for p in cfg.projects if p.enabled and not getattr(p, 'deleted', False)]
 
 @router.get("/projects")
+@handle_exceptions
 async def list_projects(
     include_deleted: bool = False,
     username: str = Depends(verify_token)
@@ -424,6 +448,7 @@ async def list_projects(
     return [p.model_dump() for p in projects]
 
 @router.get("/projects/{project_id}")
+@handle_exceptions
 async def get_project(
     project_id: int,
     username: str = Depends(verify_token)
@@ -436,6 +461,7 @@ async def get_project(
     return project.model_dump()
 
 @router.post("/projects")
+@handle_exceptions
 async def create_project(
     project: ProjectCreate,
     username: str = Depends(verify_token)
@@ -459,67 +485,50 @@ async def create_project(
             status_code=400,
             detail="Default locale must be one of the supported locales"
         )
-    
-    try:
-        # Debug log for project creation
-        log_debug(f"🔍 Creating project '{project.name}' with default_locale: {project.default_locale}")
-        
-        new_project = ConfigProvider.create_project(project.model_dump(), username)
-        
-        # Debug log for initial version
-        if new_project.versions:
-            initial_version = new_project.versions[0]
-            log_debug(f"🔍 Initial version created - no: {initial_version.no}, published: {initial_version.published}, type: {type(initial_version.published)}")
-        
-        log_info(f"✅ Project '{project.name}' created by {username}")
-        return new_project.model_dump()
-        
-    except DuplicateResourceError:
-        raise HTTPException(status_code=409, detail=f"Project with name '{project.name}' already exists")
-    except Exception as e:
-        log_error(f"❌ Error creating project", e)
-        raise HTTPException(status_code=500, detail=str(e))
 
+    # Debug log for project creation
+    log_debug(f"🔍 Creating project '{project.name}' with default_locale: {project.default_locale}")
+    
+    new_project = ConfigProvider.create_project(project.model_dump(), username)
+    
+    # Debug log for initial version
+    if new_project.versions:
+        initial_version = new_project.versions[0]
+        log_debug(f"🔍 Initial version created - no: {initial_version.no}, published: {initial_version.published}, type: {type(initial_version.published)}")
+    
+    log_info(f"✅ Project '{project.name}' created by {username}")
+    return new_project.model_dump()
+    
 @router.put("/projects/{project_id}")
+@handle_exceptions
 async def update_project(
     project_id: int,
     update: ProjectUpdate,
     username: str = Depends(verify_token)
 ):
     """Update existing project with race condition handling"""
-    try:
-        log_info(f"🔍 Update request for project {project_id} by {username}")
-        log_info(f"🔍 Received last_update_date: {update.last_update_date}")
-        
-        # Mevcut project'i al ve durumunu logla
-        current_project = ConfigProvider.get_project(project_id)
-        if current_project:
-            log_info(f"🔍 Current project last_update_date: {current_project.last_update_date}")
-            log_info(f"🔍 Current project last_update_user: {current_project.last_update_user}")
-        
-        # Optimistic locking kontrolü
-        result = ConfigProvider.update_project(
-            project_id, 
-            update.model_dump(),
-            username,
-            expected_last_update=update.last_update_date
-        )
-        
-        log_info(f"✅ Project {project_id} updated by {username}")
-        return result     
-    except RaceConditionError as e:
-        log_warning(f"⚠️ Race condition detected for project {project_id}")
-        raise HTTPException(
-            status_code=409,
-            detail=e.to_http_detail()
-        )
-    except ResourceNotFoundError:
-        raise HTTPException(status_code=404, detail="Project not found")
-    except Exception as e:
-        log_error(f"❌ Error updating project {project_id}", e)
-        raise HTTPException(status_code=500, detail=str(e))
+    log_info(f"🔍 Update request for project {project_id} by {username}")
+    log_info(f"🔍 Received last_update_date: {update.last_update_date}")
+    
+    # Mevcut project'i al ve durumunu logla
+    current_project = ConfigProvider.get_project(project_id)
+    if current_project:
+        log_info(f"🔍 Current project last_update_date: {current_project.last_update_date}")
+        log_info(f"🔍 Current project last_update_user: {current_project.last_update_user}")
+    
+    # Optimistic locking kontrolü
+    result = ConfigProvider.update_project(
+        project_id, 
+        update.model_dump(),
+        username,
+        expected_last_update=update.last_update_date
+    )
+    
+    log_info(f"✅ Project {project_id} updated by {username}")
+    return result     
 
 @router.delete("/projects/{project_id}")
+@handle_exceptions
 async def delete_project(project_id: int, username: str = Depends(verify_token)):
     """Delete project (soft delete)"""
     ConfigProvider.delete_project(project_id, username)
@@ -537,134 +546,125 @@ async def toggle_project(project_id: int, username: str = Depends(verify_token))
 
 # ===================== Import/Export Endpoints =====================
 @router.get("/projects/{project_id}/export")
+@handle_exceptions
 async def export_project(
     project_id: int,
     username: str = Depends(verify_token)
 ):
     """Export project as JSON"""
-    try:
-        project = ConfigProvider.get_project(project_id)
-        if not project:
-            raise HTTPException(status_code=404, detail="Project not found")
-        
-        # Prepare export data
-        export_data = {
-            "name": project.name,
-            "caption": project.caption,
-            "icon": project.icon,
-            "description": project.description,
-            "default_locale": project.default_locale,
-            "supported_locales": project.supported_locales,
-            "timezone": project.timezone,
-            "region": project.region,
-            "versions": []
-        }
-        
-        # Add versions (only non-deleted)
-        for version in project.versions:
-            if not getattr(version, 'deleted', False):
-                version_data = {
-                    "caption": version.caption,
-                    "description": getattr(version, 'description', ''),
-                    "general_prompt": version.general_prompt,
-                    "welcome_prompt": getattr(version, 'welcome_prompt', None),
-                    "llm": version.llm.model_dump() if version.llm else {},
-                    "intents": [intent.model_dump() for intent in version.intents]
-                }
-                export_data["versions"].append(version_data)
-        
-        log_info(f"✅ Project '{project.name}' exported by {username}")
-        
-        return export_data
-        
-    except Exception as e:
-        log_error(f"❌ Error exporting project {project_id}", e)
-        raise HTTPException(status_code=500, detail=str(e))
-
+    project = ConfigProvider.get_project(project_id)
+    if not project:
+        raise HTTPException(status_code=404, detail="Project not found")
+    
+    # Prepare export data
+    export_data = {
+        "name": project.name,
+        "caption": project.caption,
+        "icon": project.icon,
+        "description": project.description,
+        "default_locale": project.default_locale,
+        "supported_locales": project.supported_locales,
+        "timezone": project.timezone,
+        "region": project.region,
+        "versions": []
+    }
+    
+    # Add versions (only non-deleted)
+    for version in project.versions:
+        if not getattr(version, 'deleted', False):
+            version_data = {
+                "caption": version.caption,
+                "description": getattr(version, 'description', ''),
+                "general_prompt": version.general_prompt,
+                "welcome_prompt": getattr(version, 'welcome_prompt', None),
+                "llm": version.llm.model_dump() if version.llm else {},
+                "intents": [intent.model_dump() for intent in version.intents]
+            }
+            export_data["versions"].append(version_data)
+    
+    log_info(f"✅ Project '{project.name}' exported by {username}")
+    
+    return export_data
+    
 @router.post("/projects/import")
+@handle_exceptions
 async def import_project(
     project_data: dict = Body(...),
     username: str = Depends(verify_token)
 ):
     """Import project from JSON"""
-    try:
-        # Validate required fields
-        if not project_data.get('name'):
-            raise HTTPException(status_code=400, detail="Project name is required")
+    # Validate required fields
+    if not project_data.get('name'):
+        raise HTTPException(status_code=400, detail="Project name is required")
+    
+    # Check for duplicate name
+    cfg = ConfigProvider.get()
+    if any(p.name == project_data['name'] for p in cfg.projects if not p.deleted):
+        raise HTTPException(
+            status_code=409, 
+            detail=f"Project with name '{project_data['name']}' already exists"
+        )
+    
+    # Create project
+    new_project_data = {
+        "name": project_data['name'],
+        "caption": project_data.get('caption', project_data['name']),
+        "icon": project_data.get('icon', 'folder'),
+        "description": project_data.get('description', ''),
+        "default_locale": project_data.get('default_locale', 'tr'),
+        "supported_locales": project_data.get('supported_locales', ['tr']),
+        "timezone": project_data.get('timezone', 'Europe/Istanbul'),
+        "region": project_data.get('region', 'tr-TR')
+    }
+    
+    # Create project
+    new_project = ConfigProvider.create_project(new_project_data, username)
+    
+    # Import versions
+    if 'versions' in project_data and project_data['versions']:
+        # Remove the initial version that was auto-created
+        if new_project.versions:
+            new_project.versions.clear()
         
-        # Check for duplicate name
-        cfg = ConfigProvider.get()
-        if any(p.name == project_data['name'] for p in cfg.projects if not p.deleted):
-            raise HTTPException(
-                status_code=409, 
-                detail=f"Project with name '{project_data['name']}' already exists"
+        # Add imported versions
+        for idx, version_data in enumerate(project_data['versions']):
+            version = VersionConfig(
+                no=idx + 1,
+                caption=version_data.get('caption', f'Version {idx + 1}'),
+                description=version_data.get('description', ''),
+                published=False,  # Imported versions are unpublished
+                deleted=False,
+                general_prompt=version_data.get('general_prompt', ''),
+                welcome_prompt=version_data.get('welcome_prompt'),
+                llm=LLMConfiguration(**version_data.get('llm', {
+                    'repo_id': '',
+                    'generation_config': {
+                        'max_new_tokens': 512,
+                        'temperature': 0.7,
+                        'top_p': 0.9
+                    },
+                    'use_fine_tune': False,
+                    'fine_tune_zip': ''
+                })),
+                intents=[IntentConfig(**intent) for intent in version_data.get('intents', [])],
+                created_date=get_current_timestamp(),
+                created_by=username
             )
+            new_project.versions.append(version)
         
-        # Create project
-        new_project_data = {
-            "name": project_data['name'],
-            "caption": project_data.get('caption', project_data['name']),
-            "icon": project_data.get('icon', 'folder'),
-            "description": project_data.get('description', ''),
-            "default_locale": project_data.get('default_locale', 'tr'),
-            "supported_locales": project_data.get('supported_locales', ['tr']),
-            "timezone": project_data.get('timezone', 'Europe/Istanbul'),
-            "region": project_data.get('region', 'tr-TR')
-        }
-        
-        # Create project
-        new_project = ConfigProvider.create_project(new_project_data, username)
-        
-        # Import versions
-        if 'versions' in project_data and project_data['versions']:
-            # Remove the initial version that was auto-created
-            if new_project.versions:
-                new_project.versions.clear()
-            
-            # Add imported versions
-            for idx, version_data in enumerate(project_data['versions']):
-                version = VersionConfig(
-                    no=idx + 1,
-                    caption=version_data.get('caption', f'Version {idx + 1}'),
-                    description=version_data.get('description', ''),
-                    published=False,  # Imported versions are unpublished
-                    deleted=False,
-                    general_prompt=version_data.get('general_prompt', ''),
-                    welcome_prompt=version_data.get('welcome_prompt'),
-                    llm=LLMConfiguration(**version_data.get('llm', {
-                        'repo_id': '',
-                        'generation_config': {
-                            'max_new_tokens': 512,
-                            'temperature': 0.7,
-                            'top_p': 0.9
-                        },
-                        'use_fine_tune': False,
-                        'fine_tune_zip': ''
-                    })),
-                    intents=[IntentConfig(**intent) for intent in version_data.get('intents', [])],
-                    created_date=get_current_timestamp(),
-                    created_by=username
-                )
-                new_project.versions.append(version)
-            
-            # Update version counter
-            new_project.version_id_counter = len(new_project.versions) + 1
-            
-            # Save updated project
-            ConfigProvider.save(cfg, username)
-        
-        log_info(f"✅ Project '{new_project.name}' imported by {username}")
+        # Update version counter
+        new_project.version_id_counter = len(new_project.versions) + 1
         
-        return {"success": True, "project_id": new_project.id, "project_name": new_project.name}
-        
-    except DuplicateResourceError as e:
-        raise HTTPException(status_code=409, detail=str(e))
-    except Exception as e:
-        log_error(f"❌ Error importing project", e)
-        raise HTTPException(status_code=500, detail=str(e))
+        # Save updated project
+        ConfigProvider.save(cfg, username)
+    
+    log_info(f"✅ Project '{new_project.name}' imported by {username}")
+    
+    return {"success": True, "project_id": new_project.id, "project_name": new_project.name}
 
 # ===================== Version Endpoints =====================
 @router.get("/projects/{project_id}/versions")
+@handle_exceptions
 async def list_versions(
     project_id: int,
     include_deleted: bool = False,
@@ -684,6 +684,7 @@ async def list_versions(
     return [v.model_dump() for v in versions]
 
 @router.post("/projects/{project_id}/versions")
+@handle_exceptions
 async def create_version(
     project_id: int,
     version_data: VersionCreate,
@@ -696,6 +697,7 @@ async def create_version(
     return new_version.model_dump()
 
 @router.put("/projects/{project_id}/versions/{version_no}")
+@handle_exceptions
 async def update_version(
     project_id: int,
     version_no: int,
@@ -704,51 +706,25 @@ async def update_version(
     username: str = Depends(verify_token)
 ):
     """Update version with race condition handling"""
-    try:
-        log_debug(f"🔍 Version update request - project: {project_id}, version: {version_no}, user: {username}")
-       
-        # Force parametresi kontrolü
-        if force:
-            log_warning(f"⚠️ Force update requested for version {version_no} by {username}")
-        
-        result = ConfigProvider.update_version(
-            project_id,
-            version_no,
-            update.model_dump(),
-            username,
-            expected_last_update=update.last_update_date if not force else None
-        )
-        
-        log_info(f"✅ Version {version_no} updated by {username}")
-        return result
-
-    except ValidationError as e:
-        log_error(f"❌ Validation error for version {version_no}: {str(e)}")
-        raise HTTPException(status_code=400, detail=str(e))    
-    except RaceConditionError as e:
-        if force:
-            # Force modunda race condition'ı yoksay
-            result = ConfigProvider.update_version(
-                project_id,
-                version_no,
-                update.model_dump(),
-                username,
-                expected_last_update=None
-            )
-            return result
-        else:
-            log_warning(f"⚠️ Race condition detected for version {version_no}")
-            raise HTTPException(
-                status_code=409,
-                detail=e.to_http_detail()
-            )
-    except ResourceNotFoundError:
-        raise HTTPException(status_code=404, detail="Version not found")
-    except Exception as e:
-        log_error(f"❌ Error updating version {version_no}", e)
-        raise HTTPException(status_code=500, detail=str(e))
+    log_debug(f"🔍 Version update request - project: {project_id}, version: {version_no}, user: {username}")
+   
+    # Force parametresi kontrolü
+    if force:
+        log_warning(f"⚠️ Force update requested for version {version_no} by {username}")
+    
+    result = ConfigProvider.update_version(
+        project_id,
+        version_no,
+        update.model_dump(),
+        username,
+        expected_last_update=update.last_update_date if not force else None
+    )
+    
+    log_info(f"✅ Version {version_no} updated by {username}")
+    return result
         
 @router.post("/projects/{project_id}/versions/{version_no}/publish")
+@handle_exceptions
 async def publish_version(
     project_id: int,
     version_no: int,
@@ -773,6 +749,7 @@ async def publish_version(
     return {"success": True}
 
 @router.delete("/projects/{project_id}/versions/{version_no}")
+@handle_exceptions
 async def delete_version(
     project_id: int,
     version_no: int,
@@ -785,6 +762,7 @@ async def delete_version(
     return {"success": True}
 
 @router.get("/projects/{project_name}/versions")
+@handle_exceptions
 async def get_project_versions(
     project_name: str,
     username: str = Depends(verify_token)
@@ -818,6 +796,7 @@ async def get_project_versions(
     }
 
 @router.get("/projects/{project_id}/versions/{version1_id}/compare/{version2_id}")
+@handle_exceptions
 async def compare_versions(
     project_id: int,
     version1_no: int,
@@ -878,6 +857,7 @@ async def compare_versions(
     
 # ===================== API Endpoints =====================
 @router.get("/apis")
+@handle_exceptions
 async def list_apis(
     include_deleted: bool = False,
     username: str = Depends(verify_token)
@@ -893,6 +873,7 @@ async def list_apis(
     return [a.model_dump() for a in apis]
 
 @router.post("/apis")
+@handle_exceptions
 async def create_api(api: APICreate, username: str = Depends(verify_token)):
     """Create new API"""
     new_api = ConfigProvider.create_api(api.model_dump(), username)
@@ -901,36 +882,25 @@ async def create_api(api: APICreate, username: str = Depends(verify_token)):
     return new_api.model_dump()
 
 @router.put("/apis/{api_name}")
+@handle_exceptions
 async def update_api(
     api_name: str,
     update: APIUpdate,
     username: str = Depends(verify_token)
 ):
     """Update API configuration with race condition handling"""
-    try:
-        result = ConfigProvider.update_api(
-            api_name,
-            update.model_dump(),
-            username,
-            expected_last_update=update.last_update_date
-        )
-        
-        log_info(f"✅ API '{api_name}' updated by {username}")
-        return result
-        
-    except RaceConditionError as e:
-        log_warning(f"⚠️ Race condition detected for API '{api_name}'")
-        raise HTTPException(
-            status_code=409,
-            detail=e.to_http_detail()
-        )
-    except ResourceNotFoundError:
-        raise HTTPException(status_code=404, detail="API not found")
-    except Exception as e:
-        log_error(f"❌ Error updating API '{api_name}'", e)
-        raise HTTPException(status_code=500, detail=str(e))
+    result = ConfigProvider.update_api(
+        api_name,
+        update.model_dump(),
+        username,
+        expected_last_update=update.last_update_date
+    )
+    
+    log_info(f"✅ API '{api_name}' updated by {username}")
+    return result
 
 @router.delete("/apis/{api_name}")
+@handle_exceptions
 async def delete_api(api_name: str, username: str = Depends(verify_token)):
     """Delete API (soft delete)"""
     ConfigProvider.delete_api(api_name, username)
@@ -939,6 +909,7 @@ async def delete_api(api_name: str, username: str = Depends(verify_token)):
     return {"success": True}
 
 @router.post("/validate/regex")
+@handle_exceptions
 async def validate_regex(
     request: dict = Body(...),
     username: str = Depends(verify_token)
@@ -947,31 +918,20 @@ async def validate_regex(
     pattern = request.get("pattern", "")
     test_value = request.get("test_value", "")
     
-    try:
-        import re
-        compiled_regex = re.compile(pattern)
-        matches = bool(compiled_regex.match(test_value))
-        
-        return {
-            "valid": True,
-            "matches": matches,
-            "pattern": pattern,
-            "test_value": test_value
-        }
-    except DuplicateResourceError as e:
-        # Let the exception handler deal with it
-        raise
-    except Exception as e:
-        return {
-            "valid": False,
-            "matches": False,
-            "error": str(e),
-            "pattern": pattern,
-            "test_value": test_value
-        }
+    import re
+    compiled_regex = re.compile(pattern)
+    matches = bool(compiled_regex.match(test_value))
+    
+    return {
+        "valid": True,
+        "matches": matches,
+        "pattern": pattern,
+        "test_value": test_value
+    }
 
 # ===================== Test Endpoints =====================
 @router.post("/test/run-all")
+@handle_exceptions
 async def run_all_tests(
     request: TestRequest,
     username: str = Depends(verify_token)
@@ -992,6 +952,7 @@ async def run_all_tests(
     }
 
 @router.get("/test/status/{test_run_id}")
+@handle_exceptions
 async def get_test_status(
     test_run_id: str,
     username: str = Depends(verify_token)
@@ -1011,6 +972,7 @@ async def get_test_status(
 
 # ===================== Activity Log =====================
 @router.get("/activity-log")
+@handle_exceptions
 async def get_activity_log(
     limit: int = Query(100, ge=1, le=1000),
     entity_type: Optional[str] = None,
@@ -1034,6 +996,7 @@ class TTSRequest(BaseModel):
     language: Optional[str] = "tr-TR"
 
 @router.post("/tts/generate")
+@handle_exceptions
 async def generate_tts(
     request: TTSRequest,
     username: str = Depends(verify_token)
@@ -1043,71 +1006,62 @@ async def generate_tts(
     from tts_preprocessor import TTSPreprocessor
     import base64
     
-    try:
-        # Create TTS provider
-        tts_provider = TTSFactory.create_provider()
-        
-        if not tts_provider:
-            # Return empty response for no TTS
-            return Response(
-                content=b"",
-                media_type="audio/mpeg",
-                headers={"X-TTS-Status": "disabled"}
-            )
-        
-        log_info(f"🎤 TTS request: '{request.text[:50]}...' with provider: {tts_provider.get_provider_name()}")
-        
-        # Preprocess text if needed
-        preprocessor = TTSPreprocessor(language=request.language)
-        processed_text = preprocessor.preprocess(
-            request.text,
-            tts_provider.get_preprocessing_flags()
-        )
-        
-        # Generate audio
-        audio_data = await tts_provider.synthesize(
-            text=processed_text,
-            voice_id=request.voice_id
-        )
-        
-        # Return audio as binary response
+    # Create TTS provider
+    tts_provider = TTSFactory.create_provider()
+    
+    if not tts_provider:
+        # Return empty response for no TTS
         return Response(
-            content=audio_data,
+            content=b"",
             media_type="audio/mpeg",
-            headers={
-                "Content-Disposition": 'inline; filename="tts_output.mp3"',
-                "X-TTS-Provider": tts_provider.get_provider_name()
-            }
+            headers={"X-TTS-Status": "disabled"}
         )
-        
-    except Exception as e:
-        log_error("❌ TTS generation error", e)
-        raise HTTPException(status_code=500, detail=str(e))
+    
+    log_info(f"🎤 TTS request: '{request.text[:50]}...' with provider: {tts_provider.get_provider_name()}")
+    
+    # Preprocess text if needed
+    preprocessor = TTSPreprocessor(language=request.language)
+    processed_text = preprocessor.preprocess(
+        request.text,
+        tts_provider.get_preprocessing_flags()
+    )
+    
+    # Generate audio
+    audio_data = await tts_provider.synthesize(
+        text=processed_text,
+        voice_id=request.voice_id
+    )
+    
+    # Return audio as binary response
+    return Response(
+        content=audio_data,
+        media_type="audio/mpeg",
+        headers={
+            "Content-Disposition": 'inline; filename="tts_output.mp3"',
+            "X-TTS-Provider": tts_provider.get_provider_name()
+        }
+    )
 
 @router.get("/tts/voices")
+@handle_exceptions
 async def get_tts_voices(username: str = Depends(verify_token)):
     """Get available TTS voices"""
     from tts_factory import TTSFactory
     
-    try:
-        tts_provider = TTSFactory.create_provider()
-        
-        if not tts_provider:
-            return {"voices": []}
-        
-        voices = tts_provider.get_supported_voices()
-        
-        # Convert dict to list format
-        voice_list = [
-            {"id": voice_id, "name": voice_name}
-            for voice_id, voice_name in voices.items()
-        ]
-        
-        return {"voices": voice_list}
-        
-    except Exception as e:
-        log_error("❌ Error getting TTS voices", e)
+    tts_provider = TTSFactory.create_provider()
+    
+    if not tts_provider:
         return {"voices": []}
+    
+    voices = tts_provider.get_supported_voices()
+    
+    # Convert dict to list format
+    voice_list = [
+        {"id": voice_id, "name": voice_name}
+        for voice_id, voice_name in voices.items()
+    ]
+    
+    return {"voices": voice_list}
 
 # ===================== Helper Functions =====================
 async def notify_llm_startup(project, version):