Update core/visual_engine.py

core/visual_engine.py

@@ -1,6 +1,5 @@
 # core/visual_engine.py
 from PIL import Image, ImageDraw, ImageFont, ImageOps
-
 # --- MONKEY PATCH FOR Image.ANTIALIAS ---
 try:
     if hasattr(Image, 'Resampling') and hasattr(Image.Resampling, 'LANCZOS'):  # Pillow 9+
@@ -15,14 +14,8 @@ except Exception as e_mp:
     print(f"WARNING: ANTIALIAS monkey-patch error: {e_mp}")
 # --- END MONKEY PATCH ---
 
-from moviepy.editor import (
-    ImageClip,
-    VideoFileClip,
-    concatenate_videoclips,
-    TextClip,
-    CompositeVideoClip,
-    AudioFileClip
-)
+from moviepy.editor import (ImageClip, VideoFileClip, concatenate_videoclips, TextClip,
+                            CompositeVideoClip, AudioFileClip)
 import moviepy.video.fx.all as vfx
 import numpy as np
 import os
@@ -67,15 +60,14 @@ class VisualEngine:
     def __init__(self, output_dir="temp_cinegen_media", default_elevenlabs_voice_id="Rachel"):
         self.output_dir = output_dir
         os.makedirs(self.output_dir, exist_ok=True)
-
         self.font_filename = "DejaVuSans-Bold.ttf"
         font_paths_to_try = [
             self.font_filename,
-            "/usr/share/fonts/truetype/dejavu/DejaVuSans-Bold.ttf",
-            "/usr/share/fonts/truetype/liberation/LiberationSans-Bold.ttf",
-            "/System/Library/Fonts/Supplemental/Arial.ttf",
-            "C:/Windows/Fonts/arial.ttf",
-            "/usr/local/share/fonts/truetype/mycustomfonts/arial.ttf"
+            f"/usr/share/fonts/truetype/dejavu/DejaVuSans-Bold.ttf",
+            f"/usr/share/fonts/truetype/liberation/LiberationSans-Bold.ttf",
+            f"/System/Library/Fonts/Supplemental/Arial.ttf",
+            f"C:/Windows/Fonts/arial.ttf",
+            f"/usr/local/share/fonts/truetype/mycustomfonts/arial.ttf"
         ]
         self.font_path_pil = next((p for p in font_paths_to_try if os.path.exists(p)), None)
         self.font_size_pil = 20
@@ -101,7 +93,6 @@ class VisualEngine:
         self.dalle_model = "dall-e-3"
         self.image_size_dalle3 = "1792x1024"
         self.video_frame_size = (1280, 720)
-
         self.elevenlabs_api_key = None
         self.USE_ELEVENLABS = False
         self.elevenlabs_client = None
@@ -115,14 +106,11 @@ class VisualEngine:
             )
         else:
             self.elevenlabs_voice_settings = None
-
         self.pexels_api_key = None
         self.USE_PEXELS = False
-
         self.runway_api_key = None
         self.USE_RUNWAYML = False
         self.runway_client = None
-
         logger.info("VisualEngine initialized.")
 
     def set_openai_api_key(self, k):
@@ -138,10 +126,7 @@ class VisualEngine:
             try:
                 self.elevenlabs_client = ElevenLabsAPIClient(api_key=api_key)
                 self.USE_ELEVENLABS = bool(self.elevenlabs_client)
-                logger.info(
-                    f"ElevenLabs Client {'Ready' if self.USE_ELEVENLABS else 'Failed Init'} "
-                    f"(Voice ID: {self.elevenlabs_voice_id})."
-                )
+                logger.info(f"ElevenLabs Client {'Ready' if self.USE_ELEVENLABS else 'Failed Init'} (Voice ID: {self.elevenlabs_voice_id}).")
             except Exception as e:
                 logger.error(f"ElevenLabs client init error: {e}. Disabled.", exc_info=True)
                 self.USE_ELEVENLABS = False
@@ -159,14 +144,9 @@ class VisualEngine:
         if k and RUNWAYML_SDK_IMPORTED and RunwayMLClient:
             try:
                 self.USE_RUNWAYML = True
-                logger.info(
-                    f"RunwayML Client (Placeholder SDK) {'Ready.' if self.USE_RUNWAYML else 'Failed Init.'}"
-                )
+                logger.info(f"RunwayML Client (Placeholder SDK) {'Ready.' if self.USE_RUNWAYML else 'Failed Init.'}")
             except Exception as e:
-                logger.error(
-                    f"RunwayML client (Placeholder SDK) init error: {e}. Disabled.",
-                    exc_info=True
-                )
+                logger.error(f"RunwayML client (Placeholder SDK) init error: {e}. Disabled.", exc_info=True)
                 self.USE_RUNWAYML = False
         elif k:
             self.USE_RUNWAYML = True
@@ -197,16 +177,13 @@ class VisualEngine:
     def _create_placeholder_image_content(self, text_description, filename, size=None):
         if size is None:
             size = self.video_frame_size
-
         img = Image.new('RGB', size, color=(20, 20, 40))
         draw = ImageDraw.Draw(img)
         padding = 25
         max_text_width = size[0] - (2 * padding)
         lines = []
-
         if not text_description:
             text_description = "(Placeholder: No text description provided)"
-
         words = text_description.split()
         current_line = ""
         for word in words:
@@ -220,59 +197,45 @@ class VisualEngine:
                 word_width, _ = self._get_text_dimensions(word, self.font)
                 if word_width > max_text_width:
                     avg_char_w = self._get_text_dimensions("A", self.font)[0] or 10
-                    chars_that_fit = int(max_text_width / avg_char_w)
-                    truncated = (
-                        word[:chars_that_fit-3] + "..."
-                        if len(word) > chars_that_fit else word
-                    )
-                    lines.append(truncated)
+                    chars_that_fit = int(max_text_width / avg_char_w) if avg_char_w > 0 else 10
+                    if len(word) > chars_that_fit:
+                        lines.append(word[:chars_that_fit-3] + "...")
+                    else:
+                        lines.append(word)
                     current_line = ""
                 else:
                     current_line = word + " "
         if current_line.strip():
             lines.append(current_line.strip())
-
         if not lines and text_description:
             avg_char_w = self._get_text_dimensions("A", self.font)[0] or 10
-            chars_that_fit = int(max_text_width / avg_char_w)
-            truncated = (
-                text_description[:chars_that_fit-3] + "..."
-                if len(text_description) > chars_that_fit else text_description
-            )
-            lines.append(truncated)
+            chars_that_fit = int(max_text_width / avg_char_w) if avg_char_w > 0 else 10
+            if len(text_description) > chars_that_fit:
+                lines.append(text_description[:chars_that_fit-3] + "...")
+            else:
+                lines.append(text_description)
         elif not lines:
             lines.append("(Placeholder Text Error)")
-
         _, single_line_height = self._get_text_dimensions("Ay", self.font)
         single_line_height = single_line_height if single_line_height > 0 else (self.font_size_pil + 2)
         line_spacing = 2
-        max_lines_to_display = min(
-            len(lines),
-            (size[1] - (2 * padding)) // (single_line_height + line_spacing)
-        ) if single_line_height > 0 else 1
+        max_lines_to_display = min(len(lines), (size[1] - (2 * padding)) // (single_line_height + line_spacing)) if single_line_height > 0 else 1
         if max_lines_to_display <= 0:
             max_lines_to_display = 1
-
-        total_text_block_height = (
-            max_lines_to_display * single_line_height +
-            (max_lines_to_display - 1) * line_spacing
-        )
+        total_text_block_height = max_lines_to_display * single_line_height + (max_lines_to_display - 1) * line_spacing
         y_text_start = padding + (size[1] - (2 * padding) - total_text_block_height) / 2.0
         current_y = y_text_start
-
         for i in range(max_lines_to_display):
             line_content = lines[i]
             line_width_actual, _ = self._get_text_dimensions(line_content, self.font)
             x_text = max(padding, (size[0] - line_width_actual) / 2.0)
             draw.text((x_text, current_y), line_content, font=self.font, fill=(200, 200, 180))
             current_y += single_line_height + line_spacing
-
             if i == 6 and max_lines_to_display > 7 and len(lines) > max_lines_to_display:
                 ellipsis_width, _ = self._get_text_dimensions("...", self.font)
                 x_ellipsis = max(padding, (size[0] - ellipsis_width) / 2.0)
                 draw.text((x_ellipsis, current_y), "...", font=self.font, fill=(200, 200, 180))
                 break
-
         filepath = os.path.join(self.output_dir, filename)
         try:
             img.save(filepath)
@@ -284,23 +247,16 @@ class VisualEngine:
     def _search_pexels_image(self, query, output_filename_base):
         if not self.USE_PEXELS or not self.pexels_api_key:
             return None
-
         headers = {"Authorization": self.pexels_api_key}
         params = {"query": query, "per_page": 1, "orientation": "landscape", "size": "large2x"}
         base_name, _ = os.path.splitext(output_filename_base)
-        pexels_filename = f"{base_name}_pexels_{random.randint(1000, 9999)}.jpg"
+        pexels_filename = base_name + f"_pexels_{random.randint(1000,9999)}.jpg"
         filepath = os.path.join(self.output_dir, pexels_filename)
-
         try:
             logger.info(f"Pexels search: '{query}'")
             effective_query = " ".join(query.split()[:5])
             params["query"] = effective_query
-            response = requests.get(
-                "https://api.pexels.com/v1/search",
-                headers=headers,
-                params=params,
-                timeout=20
-            )
+            response = requests.get("https://api.pexels.com/v1/search", headers=headers, params=params, timeout=20)
             response.raise_for_status()
             data = response.json()
             if data.get("photos") and len(data["photos"]) > 0:
@@ -334,45 +290,28 @@ class VisualEngine:
         if not iip or not os.path.exists(iip):
             logger.error(f"Runway Gen-4 needs input image. Path invalid: {iip}")
             return None
-
         runway_dur = 10 if tds > 7 else 5
         ovfn = sifnb.replace(".png", f"_runway_gen4_d{runway_dur}s.mp4")
         ovfp = os.path.join(self.output_dir, ovfn)
-        logger.info(
-            f"Runway Gen-4 (Placeholder) img: {os.path.basename(iip)}, "
-            f"motion: '{pt[:100]}...', dur: {runway_dur}s"
-        )
+        logger.info(f"Runway Gen-4 (Placeholder) img: {os.path.basename(iip)}, motion: '{pt[:100]}...', dur: {runway_dur}s")
         logger.warning("Using PLACEHOLDER video for Runway Gen-4.")
-
         img_clip = None
         txt_c = None
         final_ph_clip = None
         try:
             img_clip = ImageClip(iip).set_duration(runway_dur)
-            txt = (
-                f"Runway Gen-4 Placeholder\n"
-                f"Input: {os.path.basename(iip)}\n"
-                f"Motion: {pt[:50]}..."
-            )
+            txt = f"Runway Gen-4 Placeholder\nInput: {os.path.basename(iip)}\nMotion: {pt[:50]}..."
             txt_c = TextClip(
                 txt,
                 fontsize=24,
                 color='white',
                 font=self.video_overlay_font,
                 bg_color='rgba(0,0,0,0.5)',
-                size=(int(self.video_frame_size[0] * 0.8), None),
+                size=(self.video_frame_size[0] * 0.8, None),
                 method='caption'
             ).set_duration(runway_dur).set_position('center')
-
             final_ph_clip = CompositeVideoClip([img_clip, txt_c], size=img_clip.size)
-            final_ph_clip.write_videofile(
-                ovfp,
-                fps=24,
-                codec='libx264',
-                preset='ultrafast',
-                logger=None,
-                threads=2
-            )
+            final_ph_clip.write_videofile(ovfp, fps=24, codec='libx264', preset='ultrafast', logger=None, threads=2)
             logger.info(f"Runway Gen-4 placeholder video: {ovfp}")
             return ovfp
         except Exception as e:
@@ -386,47 +325,44 @@ class VisualEngine:
             if final_ph_clip and hasattr(final_ph_clip, 'close'):
                 final_ph_clip.close()
 
-    def _create_placeholder_video_content(self, td, fn, dur=4, sz=None):
-        if sz is None:
-            sz = self.video_frame_size
-        fp = os.path.join(self.output_dir, fn)
-        tc = None
+    def _create_placeholder_video_content(self, text_description, filename, duration=4, size=None):
+        if size is None:
+            size = self.video_frame_size
+        filepath = os.path.join(self.output_dir, filename)
+        txt_clip = None
         try:
-            tc = TextClip(
-                td,
+            txt_clip = TextClip(
+                text_description,
                 fontsize=50,
                 color='white',
                 font=self.video_overlay_font,
                 bg_color='black',
-                size=sz,
+                size=size,
                 method='caption'
-            ).set_duration(dur)
-            tc.write_videofile(
-                fp,
+            ).set_duration(duration)
+            txt_clip.write_videofile(
+                filepath,
                 fps=24,
                 codec='libx264',
                 preset='ultrafast',
                 logger=None,
                 threads=2
             )
-            logger.info(f"Generic placeholder video: {fp}")
-            return fp
+            logger.info(f"Generic placeholder video created successfully: {filepath}")
+            return filepath
         except Exception as e:
-            logger.error(f"Generic placeholder error {fp}: {e}", exc_info=True)
+            logger.error(f"Failed to create generic placeholder video {filepath}: {e}", exc_info=True)
             return None
         finally:
-            if tc and hasattr(tc, 'close'):
-                tc.close()
+            if txt_clip and hasattr(txt_clip, 'close'):
+                try:
+                    txt_clip.close()
+                except Exception as e_close:
+                    logger.warning(f"Error closing TextClip in _create_placeholder_video_content: {e_close}")
 
-    def generate_scene_asset(
-        self,
-        image_generation_prompt_text,
-        motion_prompt_text_for_video,
-        scene_data,
-        scene_identifier_filename_base,
-        generate_as_video_clip=False,
-        runway_target_duration=5
-    ):
+    def generate_scene_asset(self, image_generation_prompt_text, motion_prompt_text_for_video,
+                             scene_data, scene_identifier_filename_base,
+                             generate_as_video_clip=False, runway_target_duration=5):
         base_name = scene_identifier_filename_base
         asset_info = {
             'path': None,
@@ -444,14 +380,11 @@ class VisualEngine:
         }
 
         if self.USE_AI_IMAGE_GENERATION and self.openai_api_key:
-            max_r = 2
+            max_r, att_n = 2, 0
             for att_n in range(max_r):
                 try:
                     img_fp_dalle = os.path.join(self.output_dir, image_filename_for_base)
-                    logger.info(
-                        f"Attempt {att_n + 1} DALL-E (base img): "
-                        f"{image_generation_prompt_text[:100]}..."
-                    )
+                    logger.info(f"Attempt {att_n+1} DALL-E (base img): {image_generation_prompt_text[:100]}...")
                     cl = openai.OpenAI(api_key=self.openai_api_key, timeout=90.0)
                     r = cl.images.generate(
                         model=self.dalle_model,
@@ -483,22 +416,19 @@ class VisualEngine:
                     }
                     break
                 except openai.RateLimitError as e:
-                    logger.warning(f"OpenAI Rate Limit {att_n + 1}: {e}. Retry...")
+                    logger.warning(f"OpenAI Rate Limit {att_n+1}: {e}. Retry...")
                     time.sleep(5 * (att_n + 1))
                     temp_image_asset_info['error_message'] = str(e)
                 except Exception as e:
                     logger.error(f"DALL-E error: {e}", exc_info=True)
                     temp_image_asset_info['error_message'] = str(e)
                     break
-
             if temp_image_asset_info['error']:
-                logger.warning(f"DALL-E failed after {att_n + 1} attempts for base image.")
+                logger.warning(f"DALL-E failed after {att_n+1} attempts for base image.")
 
         if temp_image_asset_info['error'] and self.USE_PEXELS:
-            pqt = scene_data.get(
-                'pexels_search_query_감독',
-                f"{scene_data.get('emotional_beat', '')} {scene_data.get('setting_description', '')}"
-            )
+            pqt = scene_data.get('pexels_search_query_감독',
+                                 f"{scene_data.get('emotional_beat','')} {scene_data.get('setting_description','')}")
             pp = self._search_pexels_image(pqt, image_filename_for_base)
             if pp:
                 input_image_for_runway_path = pp
@@ -515,10 +445,7 @@ class VisualEngine:
         if temp_image_asset_info['error']:
             logger.warning("Base image (DALL-E/Pexels) failed. Placeholder base image.")
             ppt = temp_image_asset_info.get('prompt_used', image_generation_prompt_text)
-            php = self._create_placeholder_image_content(
-                f"[Base Img Placeholder] {ppt[:100]}...",
-                image_filename_for_base
-            )
+            php = self._create_placeholder_image_content(f"[Base Img Placeholder] {ppt[:100]}...", image_filename_for_base)
             if php:
                 input_image_for_runway_path = php
                 temp_image_asset_info = {
@@ -560,10 +487,7 @@ class VisualEngine:
                 return asset_info
             else:
                 asset_info = temp_image_asset_info
-                asset_info['error_message'] = (
-                    asset_info.get('error_message', "") +
-                    " Base image failed, Runway video not attempted."
-                ).strip()
+                asset_info['error_message'] = (asset_info.get('error_message', "") + " Base image failed, Runway video not attempted.").strip()
                 asset_info['type'] = 'image'
                 return asset_info
         else:
@@ -573,15 +497,11 @@ class VisualEngine:
         if not self.USE_ELEVENLABS or not self.elevenlabs_client or not ttn:
             logger.info("11L skip.")
             return None
-
         afp = os.path.join(self.output_dir, ofn)
         try:
             logger.info(f"11L audio (Voice:{self.elevenlabs_voice_id}): {ttn[:70]}...")
             asm = None
-            if (
-                hasattr(self.elevenlabs_client, 'text_to_speech') and
-                hasattr(self.elevenlabs_client.text_to_speech, 'stream')
-            ):
+            if hasattr(self.elevenlabs_client, 'text_to_speech') and hasattr(self.elevenlabs_client.text_to_speech, 'stream'):
                 asm = self.elevenlabs_client.text_to_speech.stream
                 logger.info("Using 11L .text_to_speech.stream()")
             elif hasattr(self.elevenlabs_client, 'generate_stream'):
@@ -589,10 +509,8 @@ class VisualEngine:
                 logger.info("Using 11L .generate_stream()")
             elif hasattr(self.elevenlabs_client, 'generate'):
                 logger.info("Using 11L .generate()")
-                vp = (
-                    Voice(voice_id=str(self.elevenlabs_voice_id), settings=self.elevenlabs_voice_settings)
-                    if Voice and self.elevenlabs_voice_settings else str(self.elevenlabs_voice_id)
-                )
+                vp = Voice(voice_id=str(self.elevenlabs_voice_id),
+                           settings=self.elevenlabs_voice_settings) if Voice and self.elevenlabs_voice_settings else str(self.elevenlabs_voice_id)
                 ab = self.elevenlabs_client.generate(text=ttn, voice=vp, model="eleven_multilingual_v2")
                 with open(afp, "wb") as f:
                     f.write(ab)
@@ -602,37 +520,30 @@ class VisualEngine:
                 logger.error("No 11L audio method.")
                 return None
 
-            vps = {"voice_id": str(self.elevenlabs_voice_id)}
-            if self.elevenlabs_voice_settings:
-                if hasattr(self.elevenlabs_voice_settings, 'model_dump'):
-                    vps["voice_settings"] = self.elevenlabs_voice_settings.model_dump()
-                elif hasattr(self.elevenlabs_voice_settings, 'dict'):
-                    vps["voice_settings"] = self.elevenlabs_voice_settings.dict()
-                else:
-                    vps["voice_settings"] = self.elevenlabs_voice_settings
-
-            adi = asm(text=ttn, model_id="eleven_multilingual_v2", **vps)
-            with open(afp, "wb") as f:
-                for c in adi:
-                    if c:
-                        f.write(c)
-            logger.info(f"11L audio (stream): {afp}")
-            return afp
+            if asm:
+                vps = {"voice_id": str(self.elevenlabs_voice_id)}
+                if self.elevenlabs_voice_settings:
+                    if hasattr(self.elevenlabs_voice_settings, 'model_dump'):
+                        vps["voice_settings"] = self.elevenlabs_voice_settings.model_dump()
+                    elif hasattr(self.elevenlabs_voice_settings, 'dict'):
+                        vps["voice_settings"] = self.elevenlabs_voice_settings.dict()
+                    else:
+                        vps["voice_settings"] = self.elevenlabs_voice_settings
+                adi = asm(text=ttn, model_id="eleven_multilingual_v2", **vps)
+                with open(afp, "wb") as f:
+                    for c in adi:
+                        if c:
+                            f.write(c)
+                logger.info(f"11L audio (stream): {afp}")
+                return afp
         except Exception as e:
             logger.error(f"11L audio error: {e}", exc_info=True)
             return None
 
-    def assemble_animatic_from_assets(
-        self,
-        asset_data_list,
-        overall_narration_path=None,
-        output_filename="final_video.mp4",
-        fps=24
-    ):
+    def assemble_animatic_from_assets(self, asset_data_list, overall_narration_path=None, output_filename="final_video.mp4", fps=24):
         if not asset_data_list:
             logger.warning("No assets for animatic.")
             return None
-
         processed_clips = []
         narration_clip = None
         final_clip = None
@@ -644,9 +555,7 @@ class VisualEngine:
             scene_dur = asset_info.get('duration', 4.5)
             scene_num = asset_info.get('scene_num', i + 1)
             key_action = asset_info.get('key_action', '')
-            logger.info(
-                f"S{scene_num}: Path='{asset_path}', Type='{asset_type}', Dur='{scene_dur}'s"
-            )
+            logger.info(f"S{scene_num}: Path='{asset_path}', Type='{asset_type}', Dur='{scene_dur}'s")
 
             if not (asset_path and os.path.exists(asset_path)):
                 logger.warning(f"S{scene_num}: Not found '{asset_path}'. Skip.")
@@ -670,26 +579,18 @@ class VisualEngine:
                     cv_rgba.paste(thumb, (xo, yo), thumb)
                     final_rgb_pil = Image.new("RGB", self.video_frame_size, (0, 0, 0))
                     final_rgb_pil.paste(cv_rgba, mask=cv_rgba.split()[3])
-                    dbg_path = os.path.join(
-                        self.output_dir, f"debug_PRE_NUMPY_S{scene_num}.png"
-                    )
+                    dbg_path = os.path.join(self.output_dir, f"debug_PRE_NUMPY_S{scene_num}.png")
                     final_rgb_pil.save(dbg_path)
                     logger.info(f"DEBUG: Saved PRE_NUMPY_S{scene_num} to {dbg_path}")
                     frame_np = np.array(final_rgb_pil, dtype=np.uint8)
                     if not frame_np.flags['C_CONTIGUOUS']:
                         frame_np = np.ascontiguousarray(frame_np, dtype=np.uint8)
-                    logger.debug(
-                        f"S{scene_num}: NumPy for MoviePy. "
-                        f"Shape:{frame_np.shape}, DType:{frame_np.dtype}, "
-                        f"C-Contig:{frame_np.flags['C_CONTIGUOUS']}"
-                    )
+                    logger.debug(f"S{scene_num}: NumPy for MoviePy. Shape:{frame_np.shape}, DType:{frame_np.dtype}, C-Contig:{frame_np.flags['C_CONTIGUOUS']}")
                     if frame_np.size == 0 or frame_np.ndim != 3 or frame_np.shape[2] != 3:
                         logger.error(f"S{scene_num}: Invalid NumPy. Skip.")
                         continue
                     clip_base = ImageClip(frame_np, transparent=False).set_duration(scene_dur)
-                    mvpy_dbg_path = os.path.join(
-                        self.output_dir, f"debug_MOVIEPY_FRAME_S{scene_num}.png"
-                    )
+                    mvpy_dbg_path = os.path.join(self.output_dir, f"debug_MOVIEPY_FRAME_S{scene_num}.png")
                     clip_base.save_frame(mvpy_dbg_path, t=0.1)
                     logger.info(f"DEBUG: Saved MOVIEPY_FRAME_S{scene_num} to {mvpy_dbg_path}")
                     clip_fx = clip_base
@@ -702,15 +603,12 @@ class VisualEngine:
                     except Exception as e:
                         logger.error(f"S{scene_num} Ken Burns error: {e}", exc_info=False)
                     current_scene_mvpy_clip = clip_fx
-
                 elif asset_type == 'video':
                     src_clip = None
                     try:
                         src_clip = VideoFileClip(
                             asset_path,
-                            target_resolution=(
-                                self.video_frame_size[1], self.video_frame_size[0]
-                            ) if self.video_frame_size else None,
+                            target_resolution=(self.video_frame_size[1], self.video_frame_size[0]) if self.video_frame_size else None,
                             audio=False
                         )
                         tmp_clip = src_clip
@@ -722,24 +620,15 @@ class VisualEngine:
                                     tmp_clip = src_clip.loop(duration=scene_dur)
                                 else:
                                     tmp_clip = src_clip.set_duration(src_clip.duration)
-                                    logger.info(
-                                        f"S{scene_num} Video clip ({src_clip.duration:.2f}s) "
-                                        f"shorter than target ({scene_dur:.2f}s)."
-                                    )
+                                    logger.info(f"S{scene_num} Video clip ({src_clip.duration:.2f}s) shorter than target ({scene_dur:.2f}s).")
                         current_scene_mvpy_clip = tmp_clip.set_duration(scene_dur)
                         if current_scene_mvpy_clip.size != list(self.video_frame_size):
                             current_scene_mvpy_clip = current_scene_mvpy_clip.resize(self.video_frame_size)
                     except Exception as e:
-                        logger.error(
-                            f"S{scene_num} Video load error '{asset_path}':{e}",
-                            exc_info=True
-                        )
+                        logger.error(f"S{scene_num} Video load error '{asset_path}':{e}", exc_info=True)
                         continue
                     finally:
-                        if (
-                            src_clip and src_clip is not current_scene_mvpy_clip and
-                            hasattr(src_clip, 'close')
-                        ):
+                        if src_clip and src_clip is not current_scene_mvpy_clip and hasattr(src_clip, 'close'):
                             src_clip.close()
                 else:
                     logger.warning(f"S{scene_num} Unknown asset type '{asset_type}'. Skip.")
@@ -747,12 +636,7 @@ class VisualEngine:
 
                 if current_scene_mvpy_clip and key_action:
                     try:
-                        to_dur = (
-                            min(
-                                current_scene_mvpy_clip.duration - 0.5,
-                                current_scene_mvpy_clip.duration * 0.8
-                            ) if current_scene_mvpy_clip.duration > 0.5 else current_scene_mvpy_clip.duration
-                        )
+                        to_dur = min(current_scene_mvpy_clip.duration - 0.5, current_scene_mvpy_clip.duration * 0.8) if current_scene_mvpy_clip.duration > 0.5 else current_scene_mvpy_clip.duration
                         to_start = 0.25
                         txt_c = TextClip(
                             f"Scene {scene_num}\n{key_action}",
@@ -762,13 +646,11 @@ class VisualEngine:
                             bg_color='rgba(10,10,20,0.7)',
                             method='caption',
                             align='West',
-                            size=(int(self.video_frame_size[0] * 0.9), None),
+                            size=(self.video_frame_size[0] * 0.9, None),
                             kerning=-1,
                             stroke_color='black',
                             stroke_width=1.5
-                        ).set_duration(to_dur).set_start(to_start).set_position(
-                            ('center', 0.92), relative=True
-                        )
+                        ).set_duration(to_dur).set_start(to_start).set_position(('center', 0.92), relative=True)
                         current_scene_mvpy_clip = CompositeVideoClip(
                             [current_scene_mvpy_clip, txt_c],
                             size=self.video_frame_size,
@@ -792,37 +674,23 @@ class VisualEngine:
         if not processed_clips:
             logger.warning("No clips processed. Abort.")
             return None
-
         td = 0.75
         try:
             logger.info(f"Concatenating {len(processed_clips)} clips.")
             if len(processed_clips) > 1:
-                final_clip = concatenate_videoclips(
-                    processed_clips,
-                    padding=-td if td > 0 else 0,
-                    method="compose"
-                )
+                final_clip = concatenate_videoclips(processed_clips, padding=-td if td > 0 else 0, method="compose")
             elif processed_clips:
                 final_clip = processed_clips[0]
-
             if not final_clip:
                 logger.error("Concatenation failed.")
                 return None
             logger.info(f"Concatenated dur:{final_clip.duration:.2f}s")
-
             if td > 0 and final_clip.duration > 0:
                 if final_clip.duration > td * 2:
                     final_clip = final_clip.fx(vfx.fadein, td).fx(vfx.fadeout, td)
                 else:
-                    final_clip = final_clip.fx(
-                        vfx.fadein, min(td, final_clip.duration / 2.0)
-                    )
-
-            if (
-                overall_narration_path and
-                os.path.exists(overall_narration_path) and
-                final_clip.duration > 0
-            ):
+                    final_clip = final_clip.fx(vfx.fadein, min(td, final_clip.duration / 2.0))
+            if overall_narration_path and os.path.exists(overall_narration_path) and final_clip.duration > 0:
                 try:
                     narration_clip = AudioFileClip(overall_narration_path)
                     final_clip = final_clip.set_audio(narration_clip)
@@ -831,7 +699,6 @@ class VisualEngine:
                     logger.error(f"Narration add error:{e}", exc_info=True)
             elif final_clip.duration <= 0:
                 logger.warning("Video no duration. No audio.")
-
             if final_clip and final_clip.duration > 0:
                 op = os.path.join(self.output_dir, output_filename)
                 logger.info(f"Writing video:{op} (Dur:{final_clip.duration:.2f}s)")
@@ -841,10 +708,7 @@ class VisualEngine:
                     codec='libx264',
                     preset='medium',
                     audio_codec='aac',
-                    temp_audiofile=os.path.join(
-                        self.output_dir,
-                        f'temp-audio-{os.urandom(4).hex()}.m4a'
-                    ),
+                    temp_audiofile=os.path.join(self.output_dir, f'temp-audio-{os.urandom(4).hex()}.m4a'),
                     remove_temp=True,
                     threads=os.cpu_count() or 2,
                     logger='bar',
@@ -860,14 +724,8 @@ class VisualEngine:
             logger.error(f"Video write error:{e}", exc_info=True)
             return None
         finally:
-            logger.debug(
-                "Closing all MoviePy clips in `assemble_animatic_from_assets` finally block."
-            )
-            clips_to_close = (
-                processed_clips +
-                ([narration_clip] if narration_clip else []) +
-                ([final_clip] if final_clip else [])
-            )
+            logger.debug("Closing all MoviePy clips in `assemble_animatic_from_assets` finally block.")
+            clips_to_close = processed_clips + ([narration_clip] if narration_clip else []) + ([final_clip] if final_clip else [])
             for clip_obj in clips_to_close:
                 if clip_obj and hasattr(clip_obj, 'close'):
                     try: