Update app.py

app.py

@@ -7,6 +7,59 @@ import shutil
 import glob
 import base64
 import tempfile
+import requests
+import json
+from datetime import datetime
+
+# ---------------------------------------------------------
+# 0. Error Logging Helper (GitHub Gist)
+# ---------------------------------------------------------
+
+def log_to_gist(content):
+    """
+    Silently appends the error content to a GitHub Gist file.
+    Uses environment variables for security.
+    """
+    token = os.getenv("GITHUB_TOKEN")
+    gist_id = os.getenv("GIST_ID")
+    
+    if not token or not gist_id:
+        print("⚠️ GitHub Token or Gist ID not set. Skipping error logging.", flush=True)
+        return
+
+    filename = "manim_error_history.txt"
+    url = f"https://api.github.com/gists/{gist_id}"
+    headers = {
+        "Authorization": f"token {token}",
+        "Accept": "application/vnd.github.v3+json"
+    }
+    
+    try:
+        # 1. Fetch existing content
+        resp = requests.get(url, headers=headers, timeout=5)
+        if resp.status_code != 200:
+            print(f"⚠️ Failed to fetch Gist history: {resp.status_code}", flush=True)
+            return
+            
+        current_data = resp.json()
+        old_content = current_data.get('files', {}).get(filename, {}).get('content', "")
+        
+        # 2. Format new entry (Newest at the top)
+        timestamp = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
+        new_entry = f"{'='*40}\nDATE: {timestamp}\n{content}\n\n"
+        updated_content = new_entry + old_content
+        
+        # 3. Update Gist
+        payload = {"files": {filename: {"content": updated_content}}}
+        patch_resp = requests.patch(url, headers=headers, data=json.dumps(payload), timeout=5)
+        
+        if patch_resp.status_code == 200:
+            print(f"📋 Error logged to GitHub history. URL: https://gist.github.com/{gist_id}", flush=True)
+        else:
+            print(f"⚠️ Failed to update Gist: {patch_resp.text}", flush=True)
+            
+    except Exception as e:
+        print(f"⚠️ Gist Logging Exception: {str(e)}", flush=True)
 
 # ---------------------------------------------------------
 # 1. Helper Functions
@@ -15,37 +68,28 @@ import tempfile
 def modify_animation_times(code: str, factor: float) -> str:
     """
     Safely scales all numeric run_time and wait() values by a given factor.
-    This single, safe function is now used for both Pre-check and Preview modes.
     """
     print(f"⚡ Scaling animation times by a factor of {factor}...", flush=True)
-    # A safe minimum to prevent Manim from crashing with run_time=0
     MIN_RUN_TIME = 0.01 
     
     def scale_match(m, is_wait):
         try:
             val = float(m.group(2))
             new_val = val * factor
-            # For waits, we can go low. For run_time, we must stay above zero.
             final_val = new_val if is_wait else max(new_val, MIN_RUN_TIME)
             return f"{m.group(1)}{final_val:.3f}"
         except ValueError:
-            # This happens if run_time is a variable. We safely ignore it.
             return m.group(0)
 
-    # This safe regex only matches explicit numbers, preventing SyntaxErrors.
     code = re.sub(r"(run_time\s*=\s*)(\d+\.?\d*)", lambda m: scale_match(m, False), code)
     code = re.sub(r"(self\.wait\s*\(\s*)(\d+\.?\d*)", lambda m: scale_match(m, True), code)
     return code
 
 def run_manim_pre_check(code_str: str) -> (bool, str):
     """
-    Runs Manim with '-s'. This is the clean and safe version.
-    - Uses the safe `modify_animation_times` function, so no special error handling is needed.
-    - Any error that occurs is now considered a real, critical error.
-    - Soft Pass on Timeout remains as a fallback.
+    Runs Manim with '-s'. 
     """
     print("🕵️ Running fast pre-check with 'manim -s'...", flush=True)
-    # Using a factor of 0.2 provides a good speedup without being overly aggressive.
     fast_code = modify_animation_times(code_str, factor=0.1)
     
     with open("scene_pre_check.py", "w", encoding="utf-8") as f:
@@ -59,8 +103,10 @@ def run_manim_pre_check(code_str: str) -> (bool, str):
             print("✅ Pre-check passed.", flush=True)
             return True, "Pre-check successful."
         else:
-            # With our safe regex, any error should be treated as a real problem.
             stderr_log = process.stderr.decode('utf-8', 'ignore')
+            # --- LOGGING TO GITHUB ---
+            log_to_gist(f"PRE-CHECK FAILED:\n{stderr_log}")
+            
             print(f"❌ Pre-check failed with a critical error.\n{stderr_log}", flush=True)
             return False, f"⚠️ ERROR: Your code failed the pre-check.\n\n--- ERROR LOG ---\n{stderr_log}"
                 
@@ -89,7 +135,7 @@ def get_resolution_flags(orientation, quality):
 
 def run_manim(code_str, orientation, quality, timeout):
     """
-    Executes Manim. Partial stitching on Timeout. No recovery on other errors.
+    Executes Manim. Partial stitching on Timeout.
     """
     timeout_sec = float(timeout) if timeout and float(timeout) > 0 else None
     print(f"🎬 Starting Full Render: {orientation} @ {quality} (Timeout: {timeout_sec}s)...", flush=True)
@@ -158,6 +204,10 @@ def run_manim(code_str, orientation, quality, timeout):
                 print(f"✅ Video Render Success: {found_video_path}", flush=True)
                 return found_video_path, f"✅ Rendering Successful\n\n{full_logs}", True
     
+    # --- LOGGING TO GITHUB ---
+    # If we reached here, the video was not found or failed completely.
+    log_to_gist(f"RENDER FAILED:\n{full_logs}")
+    
     return None, f"❌ Failure: Video file was not created.\n\n{full_logs}", False
 
 # ---------------------------------------------------------
@@ -182,6 +232,8 @@ def render_video_from_code(code, orientation, quality, timeout, preview_factor):
         video_path, logs, success = run_manim(code_to_render, orientation, quality, timeout)
         return video_path, logs, gr.Button(visible=not success)
     except Exception as e:
+        # --- LOGGING TO GITHUB ---
+        log_to_gist(f"CRITICAL APP EXCEPTION:\n{str(e)}")
         return None, f"Rendering failed: {str(e)}", gr.Button(visible=True)
 
 # ---------------------------------------------------------