fix: frame_extractor - add max(1,...) guard for frame_interval

video_intelligence/frame_extractor.py

@@ -25,12 +25,12 @@ def extract_frames(
 ) -> List[ExtractedFrame]:
     """
     Extract frames from video at given FPS.
-    
+
     Args:
         video_path: Path to video file
         fps: Frames per second to extract (default 1.0)
         max_frames: Maximum number of frames to extract
-        
+
     Returns:
         List of ExtractedFrame objects with PIL images and timestamps
     """
@@ -43,8 +43,8 @@ def extract_frames(
     duration_sec = total_frames / video_fps if video_fps > 0 else 0
 
     # Calculate which frames to extract
-    frame_interval = int(video_fps / fps) if fps < video_fps else 1
-    
+    frame_interval = max(1, int(video_fps / fps)) if fps < video_fps else 1
+
     print(f"📹 Video: {video_path}")
     print(f"   Duration: {duration_sec:.1f}s | FPS: {video_fps:.1f} | Total frames: {total_frames}")
     print(f"   Extracting every {frame_interval}th frame ({fps} fps) ...")
@@ -61,7 +61,7 @@ def extract_frames(
             # Convert BGR (OpenCV) to RGB (PIL)
             rgb_frame = cv2.cvtColor(frame, cv2.COLOR_BGR2RGB)
             pil_image = Image.fromarray(rgb_frame)
-            timestamp = frame_count / video_fps
+            timestamp = frame_count / video_fps if video_fps > 0 else 0
 
             frames.append(ExtractedFrame(
                 image=pil_image,