Update app.py

app.py

@@ -344,39 +344,83 @@ def create_video(script_text: str, generate_script: bool, music_path: str | None
         if not video_paths:
             raise RuntimeError("No se pudieron descargar videos de Pexels")
         
-        # Paso 4: Procesar videos - VERSIÓN SIMPLIFICADA
+        # Paso 4: Procesar videos - MANEJO CORRECTO DE CLIPS NULOS
         update_task_progress(task_id, f"Paso 4/7: Procesando {len(video_paths)} videos...")
         video_clips = []
         
-        for path in video_paths:
+        for i, path in enumerate(video_paths):
             clip = None
             try:
-                # Cargar el video
+                # Cargar el video - SI FALLA, CONTINUAR CON EL SIGUIENTE
                 clip = VideoFileClip(path)
                 if clip is None:
+                    logger.error(f"Video {i+1} no se pudo cargar: {path}")
                     continue
                 
-                # Tomar máximo 8 segundos de cada clip
+                # Verificar que el clip tenga duración
+                if clip.duration <= 0:
+                    logger.error(f"Video {i+1} tiene duración inválida: {path}")
+                    clip.close()
+                    continue
+                
+                # Tomar máximo 8 segundos - SI FALLA, CONTINUAR
                 duration = min(8, clip.duration)
                 processed_clip = clip.subclip(0, duration)
                 
-                # Normalizar el clip
+                if processed_clip is None:
+                    logger.error(f"No se pudo recortar el video {i+1}: {path}")
+                    clip.close()
+                    continue
+                
+                # Normalizar el clip - SI FALLA, CONTINUAR
                 processed_clip = normalize_clip(processed_clip)
                 
                 if processed_clip is not None:
                     video_clips.append(processed_clip)
+                    logger.info(f"Video {i+1} procesado exitosamente: {path}")
+                else:
+                    logger.error(f"No se pudo normalizar el video {i+1}: {path}")
+                    processed_clip.close()
+                    clip.close()
                     
             except Exception as e:
-                logger.error(f"Error procesando video {path}: {e}")
+                logger.error(f"Error procesando video {i+1} {path}: {e}")
             finally:
+                # Asegurarse de cerrar el clip original si existe
                 if clip is not None:
                     clip.close()
         
-        if not video_clips:
-            raise RuntimeError("No se pudieron procesar los videos")
+        logger.info(f"Videos procesados exitosamente: {len(video_clips)} de {len(video_paths)}")
         
-        # Concatenar videos
-        base_video = concatenate_videoclips(video_clips, method="chain")
+        # Si no hay videos válidos, crear video de respaldo
+        if not video_clips:
+            logger.warning("No se procesaron videos válidos, creando video de respaldo...")
+            base_video = ColorClip(
+                size=TARGET_RESOLUTION, 
+                color=(0, 0, 0), 
+                duration=video_duration
+            )
+            base_video.fps = TARGET_FPS
+        else:
+            # Concatenar videos - MANEJO DE ERROR
+            try:
+                base_video = concatenate_videoclips(video_clips, method="chain")
+                if base_video is None:
+                    raise RuntimeError("La concatenación devolvió None")
+                logger.info(f"Videos concatenados exitosamente. Duración: {base_video.duration}s")
+            except Exception as e:
+                logger.error(f"Error concatenando videos: {e}")
+                # Cerrar todos los clips antes de crear respaldo
+                for clip in video_clips:
+                    if clip is not None:
+                        clip.close()
+                # Crear video de respaldo
+                base_video = ColorClip(
+                    size=TARGET_RESOLUTION, 
+                    color=(0, 0, 0), 
+                    duration=video_duration
+                )
+                base_video.fps = TARGET_FPS
         
         # Extender video si es más corto que el audio
         if base_video.duration < video_duration:
@@ -429,7 +473,8 @@ def create_video(script_text: str, generate_script: bool, music_path: str | None
         base_video.close()
         final_video.close()
         for clip in video_clips:
-            clip.close()
+            if clip is not None:
+                clip.close()
         
         return output_path