Fix: get_video_path always returns Path (matching video_creator pattern)

modules/story_reels/router.py

@@ -3,7 +3,7 @@ Story Reels Router - API Endpoints
 Consistent with Video Creator API pattern
 """
 from fastapi import APIRouter, HTTPException
-from fastapi.responses import FileResponse, RedirectResponse
+from fastapi.responses import FileResponse
 import logging
 
 from .schemas import (
@@ -86,14 +86,17 @@ async def get_story_status(video_id: str):
     }
 )
 async def download_story_reel(video_id: str):
-    """Download/stream a story reel - redirects to /videos static"""
+    """Download/stream a story reel"""
     video_path = story_creator.get_video_path(video_id)
     
-    if not video_path or not video_path.exists():
+    if not video_path.exists():
         raise HTTPException(status_code=404, detail="Video not found")
     
-    # Redirect to /videos static mount for better streaming support
-    return RedirectResponse(url=f"/videos/{video_id}.mp4", status_code=302)
+    return FileResponse(
+        video_path,
+        media_type="video/mp4",
+        filename=f"story_{video_id}.mp4"
+    )
 
 
 @router.get("/story-reels",

modules/story_reels/services/story_creator.py

@@ -116,12 +116,8 @@ class StoryCreator:
         }
     
     def get_video_path(self, job_id: str) -> Path:
-        """Get video file path for download"""
-        video_path = self.config.videos_dir_path / f"{job_id}.mp4"
-        logger.info(f"Looking for video at: {video_path}, exists: {video_path.exists()}")
-        if video_path.exists():
-            return video_path
-        return None
+        """Get video file path (always returns Path, caller checks exists)"""
+        return self.config.videos_dir_path / f"{job_id}.mp4"
     
     def get_preview(self, job_id: str, scene_id: int) -> Dict:
         """Get scene preview"""