Update app.py

app.py

@@ -50,7 +50,7 @@ Your output MUST be a single, valid JSON object and nothing else. Do not include
 }}
 """
 
-# --- 2. LLM AGENT WRAPPER (WITH DEFINITIVE FIX) ---
+# --- 2. LLM AGENT WRAPPER ---
 
 def call_gpt4o_oneapi(transcript_content, prompt_template):
     """Makes a robust, custom API call to the one-api.ir service."""
@@ -69,26 +69,20 @@ def call_gpt4o_oneapi(transcript_content, prompt_template):
         response.raise_for_status()
         result = response.json()
         
-        # Check for the basic expected structure
         if "result" not in result or not isinstance(result.get("result"), list) or len(result["result"]) == 0:
             return f"Error: Unexpected JSON structure from API. 'result' list not found or empty.\nFull response: {json.dumps(result)}"
 
         first_item_in_result = result["result"][0]
 
-        # --- THE CRITICAL FIX IS HERE ---
-        # The API can return a dict OR a string inside the list. We must handle both.
         if isinstance(first_item_in_result, dict):
-            # This is the expected case: {"content": "..."}
             message_content = first_item_in_result.get("content")
             if message_content:
                 return message_content
             else:
                  return f"Error: 'content' key not found in API response dictionary.\nFull response: {json.dumps(result)}"
         elif isinstance(first_item_in_result, str):
-            # This is the alternate case: the string is the entire response.
             return first_item_in_result
         else:
-            # Handle any other unexpected type
             return f"Error: Unknown item type in API 'result' list.\nFull response: {json.dumps(result)}"
 
     except requests.exceptions.HTTPError as e:
@@ -98,9 +92,10 @@ 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 ---
+# --- 3. CORE ORCHESTRATOR FUNCTION (WITH GRADIO 4.x FIX) ---
 
 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:
@@ -108,7 +103,8 @@ def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progres
 
     try:
         progress(0.1, desc="Reading SRT file...")
-        with open(srt_file.name, 'r', encoding='utf-8') as f:
+        # 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()
 
         progress(0.2, desc="Preparing analysis prompt...")
@@ -122,7 +118,6 @@ def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progres
              return llm_response_str, None
 
         try:
-            # Clean potential markdown code fences from the AI response
             cleaned_response = llm_response_str.strip()
             if cleaned_response.startswith("```json"):
                 cleaned_response = cleaned_response[7:]
@@ -150,7 +145,8 @@ def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progres
 
         progress(0.8, desc="Clipping video...")
         output_filename = "viral_clip.mp4"
-        with VideoFileClip(video_file.name) as video:
+        # 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."
@@ -162,7 +158,6 @@ def generate_viral_clip(video_file, srt_file, analysis_mode, progress=gr.Progres
         return summary, output_filename
 
     except Exception as e:
-        # This generic catch-all is our final safety net.
         return f"An unexpected error occurred in the main process: {str(e)}", None
 
 # --- 4. GRADIO UI DEFINITION ---
@@ -197,3 +192,4 @@ with gr.Blocks(theme=gr.themes.Soft()) as demo:
 
 if __name__ == "__main__":
     demo.launch(debug=True)
+