karaoke-gen 0.96.0__py3-none-any.whl → 0.101.0__py3-none-any.whl

backend/api/routes/audio_search.py

@@ -33,10 +33,12 @@ from backend.services.audio_search_service import (
     NoResultsError,
     DownloadError,
 )
+from backend.services.theme_service import get_theme_service
 from backend.config import get_settings
 from backend.version import VERSION
 from backend.api.dependencies import require_auth
 from backend.services.auth_service import UserType, AuthResult
+from backend.middleware.tenant import get_tenant_config_from_request
 from pathlib import Path
 
 logger = logging.getLogger(__name__)
@@ -478,17 +480,25 @@ async def search_audio(
 ):
     """
     Search for audio by artist and title, creating a new job.
-    
+
     This endpoint:
     1. Creates a job in PENDING state
     2. Searches for audio using flacfetch
     3. Either returns search results for user selection, or
     4. If auto_download=True, automatically selects best and starts processing
-    
+
     Use cases:
     - Interactive mode (default): Returns results, user calls /select endpoint
     - Auto mode (auto_download=True): Automatically selects and downloads best
     """
+    # Check tenant feature flag
+    tenant_config = get_tenant_config_from_request(request)
+    if tenant_config and not tenant_config.features.audio_search:
+        raise HTTPException(
+            status_code=403,
+            detail="Audio search is not available for this portal"
+        )
+
     try:
         # Apply default distribution settings
         settings = get_settings()
@@ -534,9 +544,18 @@ async def search_audio(
         # Extract request metadata
         request_metadata = extract_request_metadata(request, created_from="audio_search")
 
+        # Apply default theme if none specified
+        # This ensures all karaoke videos use the Nomad theme by default
+        effective_theme_id = body.theme_id
+        if effective_theme_id is None:
+            theme_service = get_theme_service()
+            effective_theme_id = theme_service.get_default_theme_id()
+            if effective_theme_id:
+                logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Resolve CDG/TXT defaults based on theme
         resolved_cdg, resolved_txt = _resolve_cdg_txt_defaults(
-            body.theme_id, body.enable_cdg, body.enable_txt
+            effective_theme_id, body.enable_cdg, body.enable_txt
         )
 
         # Use authenticated user's email
@@ -552,7 +571,7 @@ async def search_audio(
         job_create = JobCreate(
             artist=effective_display_artist,  # Display value for title screens, filenames
             title=effective_display_title,    # Display value for title screens, filenames
-            theme_id=body.theme_id,
+            theme_id=effective_theme_id,
             color_overrides=body.color_overrides or {},
             enable_cdg=resolved_cdg,
             enable_txt=resolved_txt,
@@ -575,6 +594,8 @@ async def search_audio(
             auto_download=body.auto_download,
             request_metadata=request_metadata,
             non_interactive=body.non_interactive,
+            # Tenant scoping
+            tenant_id=tenant_config.id if tenant_config else None,
         )
         job = job_manager.create_job(job_create)
         job_id = job.job_id
@@ -591,11 +612,11 @@ async def search_audio(
         # If theme is set and no custom style files are being uploaded, prepare theme style now
         # This copies the theme's style_params.json to the job folder so LyricsTranscriber
         # can access the style configuration for preview videos
-        if body.theme_id and not body.style_files:
+        if effective_theme_id and not body.style_files:
             from backend.api.routes.file_upload import _prepare_theme_for_job
             try:
                 style_params_path, theme_style_assets, youtube_desc = _prepare_theme_for_job(
-                    job_id, body.theme_id, body.color_overrides
+                    job_id, effective_theme_id, body.color_overrides
                 )
                 theme_update = {
                     'style_params_gcs_path': style_params_path,
@@ -604,9 +625,9 @@ async def search_audio(
                 if youtube_desc and not effective_youtube_description:
                     theme_update['youtube_description_template'] = youtube_desc
                 job_manager.update_job(job_id, theme_update)
-                logger.info(f"Applied theme '{body.theme_id}' to job {job_id}")
+                logger.info(f"Applied theme '{effective_theme_id}' to job {job_id}")
             except Exception as e:
-                logger.warning(f"Failed to prepare theme '{body.theme_id}' for job {job_id}: {e}")
+                logger.warning(f"Failed to prepare theme '{effective_theme_id}' for job {job_id}: {e}")
                 # Continue without theme - job can still be processed with defaults
 
         # Handle style file uploads if provided

backend/api/routes/file_upload.py

@@ -34,6 +34,7 @@ from backend.version import VERSION
 from backend.services.metrics import metrics
 from backend.api.dependencies import require_auth
 from backend.services.auth_service import UserType, AuthResult
+from backend.middleware.tenant import get_tenant_config_from_request
 
 logger = logging.getLogger(__name__)
 router = APIRouter(tags=["jobs"])
@@ -446,6 +447,14 @@ async def upload_and_create_job(
     The style_params JSON can reference the uploaded images/fonts by their original
     filenames, and the backend will update the paths to GCS locations.
     """
+    # Check tenant feature flag
+    tenant_config = get_tenant_config_from_request(request)
+    if tenant_config and not tenant_config.features.file_upload:
+        raise HTTPException(
+            status_code=403,
+            detail="File upload is not available for this portal"
+        )
+
     try:
         # Validate main audio file type
         file_ext = Path(file.filename).suffix.lower()
@@ -562,9 +571,18 @@ async def upload_and_create_job(
                     detail=f"Invalid color_overrides JSON: {e}"
                 )
 
+        # Apply default theme if none specified
+        # This ensures all karaoke videos use the Nomad theme by default
+        effective_theme_id = theme_id
+        if effective_theme_id is None:
+            theme_service = get_theme_service()
+            effective_theme_id = theme_service.get_default_theme_id()
+            if effective_theme_id:
+                logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Resolve CDG/TXT defaults based on theme
         resolved_cdg, resolved_txt = _resolve_cdg_txt_defaults(
-            theme_id, enable_cdg, enable_txt
+            effective_theme_id, enable_cdg, enable_txt
         )
 
         # Check if any custom style files are being uploaded (overrides theme)
@@ -596,7 +614,7 @@ async def upload_and_create_job(
             artist=artist,
             title=title,
             filename=file.filename,
-            theme_id=theme_id,
+            theme_id=effective_theme_id,
             color_overrides=parsed_color_overrides,
             enable_cdg=resolved_cdg,
             enable_txt=resolved_txt,
@@ -623,10 +641,12 @@ async def upload_and_create_job(
             request_metadata=request_metadata,
             # Non-interactive mode
             non_interactive=non_interactive,
+            # Tenant scoping
+            tenant_id=tenant_config.id if tenant_config else None,
         )
         job = job_manager.create_job(job_create)
         job_id = job.job_id
-        
+
         # Record job creation metric
         metrics.record_job_created(job_id, source="upload")
 
@@ -638,16 +658,16 @@ async def upload_and_create_job(
         theme_style_params_path = None
         theme_style_assets = {}
         theme_youtube_desc = None
-        if theme_id and not has_custom_style_files:
+        if effective_theme_id and not has_custom_style_files:
             try:
                 theme_style_params_path, theme_style_assets, theme_youtube_desc = _prepare_theme_for_job(
-                    job_id, theme_id, parsed_color_overrides or None
+                    job_id, effective_theme_id, parsed_color_overrides or None
                 )
-                logger.info(f"Applied theme '{theme_id}' to job {job_id}")
+                logger.info(f"Applied theme '{effective_theme_id}' to job {job_id}")
             except HTTPException:
                 raise  # Re-raise validation errors (e.g., theme not found)
             except Exception as e:
-                logger.warning(f"Failed to prepare theme '{theme_id}' for job {job_id}: {e}")
+                logger.warning(f"Failed to prepare theme '{effective_theme_id}' for job {job_id}: {e}")
                 # Continue without theme - job can still be processed with defaults
 
         # Upload main audio file to GCS
@@ -1020,6 +1040,14 @@ async def create_job_with_upload_urls(
     - Works with any HTTP client (no HTTP/2 required)
     - Resumable uploads possible with GCS
     """
+    # Check tenant feature flag
+    tenant_config = get_tenant_config_from_request(request)
+    if tenant_config and not tenant_config.features.file_upload:
+        raise HTTPException(
+            status_code=403,
+            detail="File upload is not available for this portal"
+        )
+
     try:
         # Validate files list
         if not body.files:
@@ -1097,9 +1125,18 @@ async def create_job_with_upload_urls(
         # Get original audio filename
         audio_file = audio_files[0]
 
+        # Apply default theme if none specified
+        # This ensures all karaoke videos use the Nomad theme by default
+        effective_theme_id = body.theme_id
+        if effective_theme_id is None:
+            theme_service = get_theme_service()
+            effective_theme_id = theme_service.get_default_theme_id()
+            if effective_theme_id:
+                logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Resolve CDG/TXT defaults based on theme
         resolved_cdg, resolved_txt = _resolve_cdg_txt_defaults(
-            body.theme_id, body.enable_cdg, body.enable_txt
+            effective_theme_id, body.enable_cdg, body.enable_txt
         )
 
         # Check if style_params is being uploaded (overrides theme)
@@ -1113,7 +1150,7 @@ async def create_job_with_upload_urls(
             artist=body.artist,
             title=body.title,
             filename=audio_file.filename,
-            theme_id=body.theme_id,
+            theme_id=effective_theme_id,
             color_overrides=body.color_overrides or {},
             enable_cdg=resolved_cdg,
             enable_txt=resolved_txt,
@@ -1135,6 +1172,8 @@ async def create_job_with_upload_urls(
             other_stems_models=body.other_stems_models,
             request_metadata=request_metadata,
             non_interactive=body.non_interactive,
+            # Tenant scoping
+            tenant_id=tenant_config.id if tenant_config else None,
         )
         job = job_manager.create_job(job_create)
         job_id = job.job_id
@@ -1145,9 +1184,9 @@ async def create_job_with_upload_urls(
         logger.info(f"Created job {job_id} for {body.artist} - {body.title} (signed URL upload flow)")
 
         # If theme is set and no style_params uploaded, prepare theme style now
-        if body.theme_id and not has_style_params_upload:
+        if effective_theme_id and not has_style_params_upload:
             style_params_path, style_assets, youtube_desc = _prepare_theme_for_job(
-                job_id, body.theme_id, body.color_overrides
+                job_id, effective_theme_id, body.color_overrides
             )
             # Update job with theme style data
             update_data = {
@@ -1157,7 +1196,7 @@ async def create_job_with_upload_urls(
             if youtube_desc and not body.youtube_description:
                 update_data['youtube_description_template'] = youtube_desc
             job_manager.update_job(job_id, update_data)
-            logger.info(f"Applied theme '{body.theme_id}' to job {job_id}")
+            logger.info(f"Applied theme '{effective_theme_id}' to job {job_id}")
         
         # Generate signed upload URLs for each file
         upload_urls = []
@@ -1463,6 +1502,14 @@ async def create_job_from_url(
     Note: YouTube rate limiting may cause occasional download failures.
     The backend will retry automatically.
     """
+    # Check tenant feature flag
+    tenant_config = get_tenant_config_from_request(request)
+    if tenant_config and not tenant_config.features.youtube_url:
+        raise HTTPException(
+            status_code=403,
+            detail="URL-based job creation is not available for this portal"
+        )
+
     try:
         # Validate URL
         if not _validate_url(body.url):
@@ -1521,9 +1568,18 @@ async def create_job_from_url(
         artist = body.artist
         title = body.title
 
+        # Apply default theme if none specified
+        # This ensures all karaoke videos use the Nomad theme by default
+        effective_theme_id = body.theme_id
+        if effective_theme_id is None:
+            theme_service = get_theme_service()
+            effective_theme_id = theme_service.get_default_theme_id()
+            if effective_theme_id:
+                logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Resolve CDG/TXT defaults based on theme
         resolved_cdg, resolved_txt = _resolve_cdg_txt_defaults(
-            body.theme_id, body.enable_cdg, body.enable_txt
+            effective_theme_id, body.enable_cdg, body.enable_txt
         )
 
         # Prefer authenticated user's email over request body
@@ -1535,7 +1591,7 @@ async def create_job_from_url(
             artist=artist,
             title=title,
             filename=None,  # No file uploaded
-            theme_id=body.theme_id,
+            theme_id=effective_theme_id,
             color_overrides=body.color_overrides or {},
             enable_cdg=resolved_cdg,
             enable_txt=resolved_txt,
@@ -1556,6 +1612,8 @@ async def create_job_from_url(
             other_stems_models=body.other_stems_models,
             request_metadata=request_metadata,
             non_interactive=body.non_interactive,
+            # Tenant scoping
+            tenant_id=tenant_config.id if tenant_config else None,
         )
         job = job_manager.create_job(job_create)
         job_id = job.job_id
@@ -1564,9 +1622,9 @@ async def create_job_from_url(
         metrics.record_job_created(job_id, source="url")
 
         # If theme is set, prepare theme style now
-        if body.theme_id:
+        if effective_theme_id:
             style_params_path, style_assets, youtube_desc = _prepare_theme_for_job(
-                job_id, body.theme_id, body.color_overrides
+                job_id, effective_theme_id, body.color_overrides
             )
             # Update job with theme style data
             update_data = {
@@ -1576,7 +1634,7 @@ async def create_job_from_url(
             if youtube_desc and not body.youtube_description:
                 update_data['youtube_description_template'] = youtube_desc
             job_manager.update_job(job_id, update_data)
-            logger.info(f"Applied theme '{body.theme_id}' to job {job_id}")
+            logger.info(f"Applied theme '{effective_theme_id}' to job {job_id}")
         
         logger.info(f"Created URL-based job {job_id} for URL: {body.url}")
         if artist:
@@ -1676,6 +1734,14 @@ async def create_finalise_only_job(
     
     The endpoint returns signed URLs for uploading all the prep files.
     """
+    # Check tenant feature flag - finalise-only requires file upload capability
+    tenant_config = get_tenant_config_from_request(request)
+    if tenant_config and not tenant_config.features.file_upload:
+        raise HTTPException(
+            status_code=403,
+            detail="File upload is not available for this portal"
+        )
+
     try:
         # Validate files list
         if not body.files:
@@ -1768,9 +1834,18 @@ async def create_finalise_only_job(
         # Extract request metadata
         request_metadata = extract_request_metadata(request, created_from="finalise_only_upload")
 
+        # Apply default theme if none specified
+        # This ensures all karaoke videos use the Nomad theme by default
+        effective_theme_id = body.theme_id
+        if effective_theme_id is None:
+            theme_service = get_theme_service()
+            effective_theme_id = theme_service.get_default_theme_id()
+            if effective_theme_id:
+                logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Resolve CDG/TXT defaults based on theme
         resolved_cdg, resolved_txt = _resolve_cdg_txt_defaults(
-            body.theme_id, body.enable_cdg, body.enable_txt
+            effective_theme_id, body.enable_cdg, body.enable_txt
         )
 
         # Check if style_params is being uploaded (overrides theme)
@@ -1784,7 +1859,7 @@ async def create_finalise_only_job(
             artist=body.artist,
             title=body.title,
             filename="finalise_only",  # No single audio file - using prep outputs
-            theme_id=body.theme_id,
+            theme_id=effective_theme_id,
             color_overrides=body.color_overrides or {},
             enable_cdg=resolved_cdg,
             enable_txt=resolved_txt,
@@ -1798,6 +1873,8 @@ async def create_finalise_only_job(
             finalise_only=True,
             keep_brand_code=body.keep_brand_code,
             request_metadata=request_metadata,
+            # Tenant scoping
+            tenant_id=tenant_config.id if tenant_config else None,
         )
         job = job_manager.create_job(job_create)
         job_id = job.job_id
@@ -1808,9 +1885,9 @@ async def create_finalise_only_job(
         logger.info(f"Created finalise-only job {job_id} for {body.artist} - {body.title}")
 
         # If theme is set and no style_params uploaded, prepare theme style now
-        if body.theme_id and not has_style_params_upload:
+        if effective_theme_id and not has_style_params_upload:
             style_params_path, style_assets, youtube_desc = _prepare_theme_for_job(
-                job_id, body.theme_id, body.color_overrides
+                job_id, effective_theme_id, body.color_overrides
             )
             # Update job with theme style data
             update_data = {
@@ -1820,7 +1897,7 @@ async def create_finalise_only_job(
             if youtube_desc and not body.youtube_description:
                 update_data['youtube_description_template'] = youtube_desc
             job_manager.update_job(job_id, update_data)
-            logger.info(f"Applied theme '{body.theme_id}' to job {job_id}")
+            logger.info(f"Applied theme '{effective_theme_id}' to job {job_id}")
         
         # Generate signed upload URLs for each file
         upload_urls = []

backend/api/routes/health.py

@@ -13,6 +13,9 @@ from backend.services.flacfetch_client import get_flacfetch_client
 from backend.services.email_service import get_email_service
 from backend.services.stripe_service import get_stripe_service
 from backend.services.encoding_service import get_encoding_service
+from backend.services.spacy_preloader import get_preloaded_model, is_model_preloaded
+from backend.services.nltk_preloader import get_preloaded_cmudict, is_cmudict_preloaded
+from backend.services.langfuse_preloader import get_preloaded_langfuse_handler, is_langfuse_preloaded, is_langfuse_configured
 
 router = APIRouter()
 logger = logging.getLogger(__name__)
@@ -334,6 +337,68 @@ async def detailed_health_check() -> Dict[str, Any]:
     }
 
 
+@router.get("/health/preload-status")
+async def preload_status() -> Dict[str, Any]:
+    """
+    Check status of preloaded resources for performance optimization.
+
+    Use this endpoint to verify that NLTK, SpaCy, and Langfuse resources
+    were successfully preloaded at container startup. If any show as
+    not preloaded, check Cloud Run startup logs for errors.
+
+    Expected state after successful deployment:
+    - spacy.preloaded: true
+    - nltk.preloaded: true
+    - langfuse.preloaded: true (if configured) or configured: false
+    """
+    # SpaCy status
+    spacy_model = get_preloaded_model("en_core_web_sm")
+    spacy_status = {
+        "preloaded": is_model_preloaded("en_core_web_sm"),
+        "model": "en_core_web_sm",
+    }
+    if spacy_model:
+        spacy_status["vocab_size"] = len(spacy_model.vocab)
+
+    # NLTK status
+    cmudict = get_preloaded_cmudict()
+    nltk_status = {
+        "preloaded": is_cmudict_preloaded(),
+        "resource": "cmudict",
+    }
+    if cmudict:
+        nltk_status["entries"] = len(cmudict)
+
+    # Langfuse status
+    langfuse_handler = get_preloaded_langfuse_handler()
+    langfuse_status = {
+        "configured": is_langfuse_configured(),
+        "preloaded": is_langfuse_preloaded(),
+    }
+    if langfuse_handler:
+        langfuse_status["handler_type"] = type(langfuse_handler).__name__
+
+    # Overall status
+    all_preloaded = (
+        spacy_status["preloaded"]
+        and nltk_status["preloaded"]
+        and (langfuse_status["preloaded"] or not langfuse_status["configured"])
+    )
+
+    return {
+        "status": "ok" if all_preloaded else "degraded",
+        "message": "All resources preloaded" if all_preloaded else "Some resources not preloaded - check startup logs",
+        "spacy": spacy_status,
+        "nltk": nltk_status,
+        "langfuse": langfuse_status,
+        "performance_impact": {
+            "spacy_preload": "Saves ~60s on first lyrics correction",
+            "nltk_preload": "Saves ~100-150s on SyllablesMatchHandler init",
+            "langfuse_preload": "Saves ~200s on AgenticCorrector init",
+        }
+    }
+
+
 @router.get("/readiness")
 async def readiness_check() -> Dict[str, str]:
     """Readiness check endpoint for Cloud Run."""

backend/api/routes/internal.py

@@ -371,6 +371,12 @@ async def check_idle_reminder(
         add_span_event("already_sent")
         return {"status": "already_sent", "job_id": job_id, "message": "Reminder already sent"}
 
+    # Skip reminders for made-for-you jobs (admin handles these directly, no intermediate customer emails)
+    if getattr(job, 'made_for_you', False):
+        logger.info(f"[job:{job_id}] Made-for-you job, skipping customer reminder (admin handles)")
+        add_span_event("made_for_you_skip")
+        return {"status": "skipped", "job_id": job_id, "message": "Made-for-you job - admin handles directly"}
+
     # Check if user has an email
     if not job.user_email:
         logger.warning(f"[job:{job_id}] No user email, cannot send reminder")

backend/api/routes/jobs.py

@@ -11,7 +11,7 @@ import asyncio
 import logging
 import httpx
 from typing import List, Optional, Dict, Any, Tuple
-from fastapi import APIRouter, HTTPException, BackgroundTasks, Depends
+from fastapi import APIRouter, HTTPException, BackgroundTasks, Depends, Request
 
 from backend.models.job import Job, JobCreate, JobResponse, JobStatus
 from backend.models.requests import (
@@ -25,10 +25,13 @@ from backend.models.requests import (
 from backend.services.job_manager import JobManager
 from backend.services.worker_service import get_worker_service
 from backend.services.storage_service import StorageService
+from backend.services.theme_service import get_theme_service
 from backend.config import get_settings
 from backend.api.dependencies import require_admin, require_auth, require_instrumental_auth
 from backend.services.auth_service import UserType, AuthResult
 from backend.services.metrics import metrics
+from backend.middleware.tenant import get_tenant_from_request
+from backend.utils.test_data import is_test_email
 
 
 logger = logging.getLogger(__name__)
@@ -92,11 +95,22 @@ async def create_job(
         settings = get_settings()
         effective_enable_youtube_upload = request.enable_youtube_upload if request.enable_youtube_upload is not None else settings.default_enable_youtube_upload
 
+        # Apply default theme - all jobs require a theme
+        theme_service = get_theme_service()
+        effective_theme_id = theme_service.get_default_theme_id()
+        if not effective_theme_id:
+            raise HTTPException(
+                status_code=422,
+                detail="No default theme configured. Please contact support or specify a theme_id."
+            )
+        logger.info(f"Applying default theme: {effective_theme_id}")
+
         # Create job with all preferences
         job_create = JobCreate(
             url=str(request.url),
             artist=request.artist,
             title=request.title,
+            theme_id=effective_theme_id,  # Required - all jobs must have a theme
             enable_cdg=request.enable_cdg,
             enable_txt=request.enable_txt,
             enable_youtube_upload=effective_enable_youtube_upload,
@@ -177,11 +191,13 @@ def _check_job_ownership(job: Job, auth_result: AuthResult) -> bool:
 
 @router.get("", response_model=List[Job])
 async def list_jobs(
+    request: Request,
     status: Optional[JobStatus] = None,
     environment: Optional[str] = None,
     client_id: Optional[str] = None,
     created_after: Optional[str] = None,
     created_before: Optional[str] = None,
+    exclude_test: bool = True,
     limit: int = 100,
     auth_result: AuthResult = Depends(require_auth)
 ) -> List[Job]:
@@ -189,6 +205,7 @@ async def list_jobs(
     List jobs with optional filters.
 
     Regular users only see their own jobs. Admins see all jobs.
+    Users on tenant portals only see jobs from their tenant.
 
     Args:
         status: Filter by job status (pending, complete, failed, etc.)
@@ -196,6 +213,7 @@ async def list_jobs(
         client_id: Filter by request_metadata.client_id (customer identifier)
         created_after: Filter jobs created after this ISO datetime (e.g., 2024-01-01T00:00:00Z)
         created_before: Filter jobs created before this ISO datetime
+        exclude_test: If True (default), exclude jobs from test users (admin only)
         limit: Maximum number of jobs to return (default 100)
 
     Returns:
@@ -232,6 +250,10 @@ async def list_jobs(
                 logger.warning("Non-admin auth without user_email, returning empty job list")
                 return []
 
+        # Get tenant_id from request for portal scoping
+        # Tenant users only see jobs from their tenant
+        tenant_id = get_tenant_from_request(request)
+
         jobs = job_manager.list_jobs(
             status=status,
             environment=environment,
@@ -239,9 +261,14 @@ async def list_jobs(
             created_after=created_after_dt,
             created_before=created_before_dt,
             user_email=user_email_filter,
+            tenant_id=tenant_id,
             limit=limit
         )
 
+        # Filter out test user jobs if exclude_test is True (admin only)
+        if exclude_test and auth_result.is_admin:
+            jobs = [j for j in jobs if not is_test_email(j.user_email or "")]
+
         logger.debug(f"Listed {len(jobs)} jobs for user={auth_result.user_email}, admin={auth_result.is_admin}")
         return jobs
     except HTTPException:

backend/api/routes/review.py

@@ -412,7 +412,10 @@ async def generate_preview_video(
 
     # Check if GCE preview encoding is enabled
     use_gce_preview = encoding_service.is_preview_enabled
-    logger.info(f"Job {job_id}: Generating preview video (GCE preview: {use_gce_preview})")
+
+    # Check if user wants theme background image (slower) or black background (faster, default)
+    use_background_image = updated_data.get("use_background_image", False)
+    logger.info(f"Job {job_id}: Generating preview video (GCE: {use_gce_preview}, background image: {use_background_image})")
 
     # Use tracing and job_log_context for full observability
     with create_span("generate-preview-video", {"job_id": job_id, "use_gce": use_gce_preview}) as span:
@@ -498,12 +501,16 @@ async def generate_preview_video(
                                 # Get background image and font from style assets if available
                                 style_assets = job.style_assets or {}
 
+                                # Only use background image if user explicitly requested it
+                                # Default is black background for faster preview generation (~10s vs ~30-60s)
                                 background_image_gcs_path = None
-                                for key in ["karaoke_background", "style_karaoke_background"]:
-                                    if key in style_assets:
-                                        background_image_gcs_path = f"gs://{bucket_name}/{style_assets[key]}"
-                                        gce_span.set_attribute("background_image", background_image_gcs_path)
-                                        break
+                                if use_background_image:
+                                    for key in ["karaoke_background", "style_karaoke_background"]:
+                                        if key in style_assets:
+                                            background_image_gcs_path = f"gs://{bucket_name}/{style_assets[key]}"
+                                            gce_span.set_attribute("background_image", background_image_gcs_path)
+                                            break
+                                gce_span.set_attribute("use_background_image", use_background_image)
 
                                 font_gcs_path = None
                                 for key in ["font", "style_font"]: