Add TheBoldFont, update caption styling, and fix atomic write

video_creator/libraries/video_composer.py

@@ -185,7 +185,7 @@ class VideoComposer:
         elif position == "center":
             y_pos = height * 0.5
         else:  # bottom
-            y_pos = height * 0.85
+            y_pos = height * 0.70  # Changed from 0.85 to 0.70 as requested
         
         for caption in captions:
             start_time = caption["startMs"] / 1000
@@ -203,13 +203,18 @@ class VideoComposer:
             try:
                 # Try caption method with fixed height to avoid NoneType error
                 # Allocating 20% of height for caption box
+                
+                # Use TheBoldFont.ttf
+                font_path = Path(__file__).parent.parent.parent / "static" / "fonts" / "TheBoldFont.ttf"
+                font_name = str(font_path) if font_path.exists() else "Liberation-Sans-Bold"
+                
                 txt_clip = TextClip(
                     caption["text"],
                     fontsize=70,
                     color="white",
-                    font="Liberation-Sans-Bold",
+                    font=font_name,
                     stroke_color="black",
-                    stroke_width=3,
+                    stroke_width=1.5,
                     bg_color=final_bg_color,
                     method="caption",
                     size=(int(width * 0.9), int(height * 0.2)),
@@ -222,7 +227,7 @@ class VideoComposer:
                     caption["text"],
                     fontsize=60,
                     color="white",
-                    font="Liberation-Sans-Bold",
+                    font=font_name,
                     stroke_color="black",
                     stroke_width=2,
                     bg_color=final_bg_color,

video_creator/short_creator.py

@@ -182,24 +182,38 @@ class ShortCreator:
         
         # Render final video
         output_path = self.config.videos_dir_path / f"{video_id}.mp4"
+        # Use a temp path for atomic write to prevent premature "ready" status
+        temp_output_path = self.config.videos_dir_path / f"{video_id}.tmp.mp4"
         
-        VideoComposer.render(
-            scenes=scenes,
-            music_path=selected_music["path"],
-            output_path=output_path,
-            orientation=orientation,
-            caption_position=config.captionPosition.value,
-            caption_bg_color=config.captionBackgroundColor,
-            music_volume=config.musicVolume.value,
-            padding_back=config.paddingBack
-        )
+        try:
+            VideoComposer.render(
+                scenes=scenes,
+                music_path=selected_music["path"],
+                output_path=temp_output_path,
+                orientation=orientation,
+                caption_position=config.captionPosition.value,
+                caption_bg_color=config.captionBackgroundColor,
+                music_volume=config.musicVolume.value,
+                padding_back=config.paddingBack
+            )
+            
+            # Atomic rename to final path
+            if temp_output_path.exists():
+                temp_output_path.rename(output_path)
+                logger.info(f"Video {video_id} created successfully at {output_path}")
+            else:
+                raise Exception("Rendered file not found at temp path")
+                
+        except Exception as e:
+            # Cleanup temp file on failure
+            if temp_output_path.exists():
+                temp_output_path.unlink()
+            raise e
         
         # Cleanup temp files
         for temp_file in temp_files:
             if temp_file.exists():
                 temp_file.unlink()
-        
-        logger.info(f"Video {video_id} created successfully at {output_path}")
     
     def get_status(self, video_id: str) -> VideoStatus:
         """Get video processing status"""