Update app.py

app.py

@@ -3,7 +3,8 @@ import requests
 import os
 import json
 import time
-from moviepy.video.io.VideoFileClip import VideoFileClip
+# CORRECT, HIGH-LEVEL IMPORT for MoviePy. This brings in all editing functions.
+from moviepy.editor import VideoFileClip
 
 # --- 1. CONFIGURATION & CONSTANTS ---
 # Securely load API key from Hugging Face Space secrets.
@@ -92,18 +93,18 @@ def call_gpt4o_oneapi(transcript_content, prompt_template):
     except json.JSONDecodeError:
         return f"Error: Failed to decode JSON from API response.\nResponse Body: {response.text}"
 
-# --- 3. CORE ORCHESTRATOR FUNCTION (WITH GRADIO 4.x FIX) ---
+# --- 3. CORE ORCHESTRATOR FUNCTION ---
 
 def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progress()):
-    # The `video_file` and `srt_file` inputs are now file PATHS (strings), not objects.
     if not video_file or not srt_file:
         return "Error: Please upload both a video file and an SRT file.", None
     if not ONE_API_KEY or not ONE_API_URL:
         return "Error: API keys for OneAPI are not configured correctly in the Space secrets.", None
 
+    video = None
+    new_clip = None
     try:
         progress(0.1, desc="Reading SRT file...")
-        # FIX: Use `srt_file` directly, as it's now a path string.
         with open(srt_file, 'r', encoding='utf-8') as f:
             transcript_content = f.read()
 
@@ -145,20 +146,32 @@ def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progres
 
         progress(0.8, desc="Clipping video...")
         output_filename = "viral_clip.mp4"
-        # FIX: Use `video_file` directly, as it's now a path string.
-        with VideoFileClip(video_file) as video:
-            if end_time > video.duration:
-                end_time = video.duration
-                summary += f"\n\n⚠️ Warning: End time was beyond video duration, adjusted to {end_time:.2f}s."
-
-            new_clip = video.subclip(start_time, end_time)
-            new_clip.write_videofile(output_filename, codec="libx264", audio_codec="aac")
+        
+        # FIX: Load video using the correct high-level object
+        video = VideoFileClip(video_file)
+        
+        if end_time > video.duration:
+            end_time = video.duration
+            summary += f"\n\n⚠️ Warning: End time was beyond video duration, adjusted to {end_time:.2f}s."
+        
+        # Now .subclip() will exist because we imported from moviepy.editor
+        new_clip = video.subclip(start_time, end_time)
+        new_clip.write_videofile(output_filename, codec="libx264", audio_codec="aac")
 
         progress(1.0, desc="Done!")
         return summary, output_filename
 
     except Exception as e:
-        return f"An unexpected error occurred in the main process: {str(e)}", None
+        import traceback
+        tb_str = traceback.format_exc()
+        return f"An unexpected error occurred in the main process: {str(e)}\n\nTraceback:\n{tb_str}", None
+    
+    finally:
+        # Manually close the clips to release file resources, preventing locks.
+        if new_clip:
+            new_clip.close()
+        if video:
+            video.close()
 
 # --- 4. GRADIO UI DEFINITION ---
 with gr.Blocks(theme=gr.themes.Soft()) as demo:
@@ -192,4 +205,3 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
 
 if __name__ == "__main__":
     demo.launch(debug=True)
-