Update app.py

app.py

@@ -5,11 +5,57 @@ import re
 import time
 import glob
 import shutil
+# --- START: New Imports for Pyflakes Integration ---
+from pyflakes.api import check
+from pyflakes.reporter import Reporter
+import io
+# --- END: New Imports ---
 
 # ---------------------------------------------------------
 # 1. Helper Functions (Dependencies for Rendering)
 # ---------------------------------------------------------
 
+class ErrorReporter(Reporter):
+    """Custom reporter to capture pyflakes errors into a list."""
+    def __init__(self):
+        self.errors = []
+        # Suppress the default print-to-stdout behavior
+        super().__init__(io.StringIO(), io.StringIO())
+
+    def unexpectedError(self, filename, msg):
+        self.errors.append(f"Unexpected Error in {filename}: {msg}")
+
+    def syntaxError(self, filename, msg, lineno, offset, text):
+        self.errors.append(f"Syntax Error in {filename} (Line {lineno}): {msg}")
+
+    def flake(self, message):
+        self.errors.append(str(message))
+
+def check_code_with_pyflakes(code_str: str) -> str:
+    """
+    Analyzes a string of Python code using pyflakes and returns a formatted error report.
+    Returns an empty string if no errors are found.
+    """
+    reporter = ErrorReporter()
+    check(code_str, "scene.py", reporter=reporter)
+    
+    if not reporter.errors:
+        return "" # No errors found
+        
+    # Format the error list into a user-friendly string
+    error_report = "Static analysis found potential errors:\n"
+    for err in reporter.errors:
+        # Example raw error: 'scene.py:5:21: undefined name 'Cirle''
+        match = re.search(r'scene\.py:(\d+):.*(undefined name.*)', str(err))
+        if match:
+            line_num = match.group(1)
+            error_msg = match.group(2)
+            error_report += f"- Line {line_num}: {error_msg}\n"
+        else:
+            error_report += f"- {err}\n"
+            
+    return error_report
+
 def cleanup_media_directory():
     """Wipes the media directory to prevent caching issues."""
     media_dir = 'media'
@@ -115,15 +161,17 @@ def run_manim(code_str, orientation, quality, timeout):
 
     # Strategy B: Look for a Static Image (.png) and convert to Video
     media_image_base = os.path.join("media", "images", "scene")
-    expected_image_name = output_filename + ".png" 
+    expected_image_name = output_filename.replace('.mp4', '.png')
     found_image_path = None
 
     if os.path.exists(media_image_base):
         for root, _, files in os.walk(media_image_base):
-            if expected_image_name in files:
-                found_image_path = os.path.join(root, expected_image_name)
-                break
-    
+            # Manim Community v0.18.0 changed naming for -o. Let's find the single png.
+            png_files = [f for f in files if f.endswith(".png")]
+            if len(png_files) == 1:
+                 found_image_path = os.path.join(root, png_files[0])
+                 break
+
     if found_image_path:
         print(f"🖼️ Static scene detected (0 animations). converting image to video: {found_image_path}", flush=True)
         converted_video_path = os.path.join(os.path.dirname(found_image_path), output_filename)
@@ -151,6 +199,16 @@ def run_manim(code_str, orientation, quality, timeout):
 def render_video_from_code(code, orientation, quality, timeout, preview_factor):
     """Renders a video from a given Manim code string."""
     try:
+        # --- START: Integrated Pyflakes Pre-Check ---
+        print("🕵️ Running static analysis with pyflakes...", flush=True)
+        static_errors = check_code_with_pyflakes(code)
+        if static_errors:
+            print(f"❌ Static analysis failed:\n{static_errors}", flush=True)
+            # Return the error report immediately without trying to render
+            return None, static_errors, gr.Button(visible=True)
+        print("✅ Static analysis passed.", flush=True)
+        # --- END: Integrated Pyflakes Pre-Check ---
+
         cleanup_media_directory()
 
         if not code or "from manim import" not in code:
@@ -184,16 +242,16 @@ class GenScene(Scene):
 """
 
 with gr.Blocks(title="Manim Render API") as demo:
-    # Hidden components to define the API signature
-    code_input = gr.Code(label="Python Code", language="python", value=DEFAULT_CODE,visible=False)
-    orientation_opt = gr.Radio(choices=["Landscape (16:9)", "Portrait (9:16)"], value="Portrait (9:16)", label="Orientation",visible=False)
-    quality_opt = gr.Dropdown(choices=["Preview (360p)", "480p", "720p", "1080p", "4k"], value="Preview (360p)", label="Quality", visible=False)
-    timeout_input = gr.Number(label="Render Timeout (seconds)", value=60, visible=False)
-    preview_speed_factor_input = gr.Number(label="Preview Speed Factor", value=0.5,visible=False)
+    # Components are now visible for direct interaction
+    code_input = gr.Code(label="Python Code", language="python", value=DEFAULT_CODE, visible=True)
+    orientation_opt = gr.Radio(choices=["Landscape (16:9)", "Portrait (9:16)"], value="Portrait (9:16)", label="Orientation", visible=True)
+    quality_opt = gr.Dropdown(choices=["Preview (360p)", "480p", "720p", "1080p", "4k"], value="Preview (360p)", label="Quality", visible=True)
+    timeout_input = gr.Number(label="Render Timeout (seconds)", value=60, visible=True)
+    preview_speed_factor_input = gr.Number(label="Preview Speed Factor", value=0.5, visible=True)
 
     video_output = gr.Video(label="Result")
     status_output = gr.Textbox(label="Status/Logs")
-    fix_btn_output = gr.Button("Fix Error & Re-render", variant="stop",visible=False)
+    fix_btn_output = gr.Button("Fix Error & Re-render", variant="stop", visible=True)
 
     render_btn = gr.Button("Render")