Fix file permission issues for Docker deployment

- Add tempfile.mkdtemp() for system temp directory creation
- Add file permission management with os.chmod()
- Update all file paths to use absolute paths
- Fix upload endpoint temp file handling
- Fix download endpoint to use OUTPUTS_DIR
- Resolve Permission denied errors in HF Spaces

highlights_api.py

@@ -72,9 +72,20 @@ class JobStatus(BaseModel):
 active_jobs = {}
 completed_jobs = {}
 
-# Create output directories
-os.makedirs("outputs", exist_ok=True)
-os.makedirs("temp", exist_ok=True)
+# Create output directories with proper permissions
+import tempfile
+import stat
+
+# Use system temp directory for Hugging Face Spaces compatibility
+TEMP_DIR = tempfile.mkdtemp(prefix="smolvlm2_")
+OUTPUTS_DIR = os.path.join(os.getcwd(), "outputs")
+
+os.makedirs(OUTPUTS_DIR, mode=0o755, exist_ok=True)
+os.makedirs(TEMP_DIR, mode=0o755, exist_ok=True)
+
+# Ensure directories are writable
+os.chmod(TEMP_DIR, stat.S_IRWXU | stat.S_IRWXG | stat.S_IRWXO)
+os.chmod(OUTPUTS_DIR, stat.S_IRWXU | stat.S_IRWXG | stat.S_IRWXO)
 
 @app.get("/")
 async def root():
@@ -109,12 +120,17 @@ async def upload_video(
     job_id = str(uuid.uuid4())
     
     try:
-        # Save uploaded video
-        temp_video_path = f"temp/{job_id}_{video.filename}"
+        # Save uploaded video to system temp directory
+        safe_filename = "".join(c for c in video.filename if c.isalnum() or c in '._-')
+        temp_video_path = os.path.join(TEMP_DIR, f"{job_id}_{safe_filename}")
+        
         with open(temp_video_path, "wb") as f:
             content = await video.read()
             f.write(content)
         
+        # Ensure the file is readable
+        os.chmod(temp_video_path, stat.S_IRUSR | stat.S_IWUSR | stat.S_IRGRP | stat.S_IROTH)
+        
         # Store job info
         active_jobs[job_id] = {
             "status": "processing",
@@ -186,7 +202,7 @@ async def download_file(filename: str):
     """
     Download generated files
     """
-    file_path = f"outputs/{filename}"
+    file_path = os.path.join(OUTPUTS_DIR, filename)
     if not os.path.exists(file_path):
         raise HTTPException(status_code=404, detail="File not found")
     
@@ -231,7 +247,7 @@ async def process_video_highlights(
         
         # Analyze segments
         analyzed_segments = []
-        temp_frame_path = f"temp/{job_id}_frame.jpg"
+        temp_frame_path = os.path.join(TEMP_DIR, f"{job_id}_frame.jpg")
         
         for i, segment in enumerate(segments):
             # Update progress
@@ -265,11 +281,11 @@ async def process_video_highlights(
         active_jobs[job_id]["progress"] = 90
         active_jobs[job_id]["message"] = f"Creating highlights video with {len(selected_segments)} segments..."
         
-        # Create output filenames
+        # Create output filenames using absolute paths
         highlights_filename = f"{job_id}_highlights.mp4"
         analysis_filename = f"{job_id}_analysis.json"
-        highlights_path = f"outputs/{highlights_filename}"
-        analysis_path = f"outputs/{analysis_filename}"
+        highlights_path = os.path.join(OUTPUTS_DIR, highlights_filename)
+        analysis_path = os.path.join(OUTPUTS_DIR, analysis_filename)
         
         # Create highlights video
         success = create_highlights_video(video_path, selected_segments, highlights_path)