slidemovie 0.2.2__tar.gz → 0.4.0__tar.gz

{slidemovie-0.2.2/slidemovie.egg-info → slidemovie-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: slidemovie
-Version: 0.2.2
+Version: 0.4.0
 Summary: Markdown and PowerPoint to narration video generator
 Author: Katsutoshi Seki
 License-Expression: MIT
@@ -51,7 +51,7 @@ Dynamic: license-file
 ## ✨ Features
 
 *   **Markdown-Based**: Write your slide content and narration script in a single text file.
-*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini** or **OpenAI** (via `multiai-tts`).
+*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini**, **OpenAI** or **Azure** (via `multiai-tts`).
 *   **PowerPoint Integration**: Use PowerPoint's AI "Designer" to create professional visuals instantly.
 *   **No Video Editing**: Audio and visuals are automatically synchronized.
 *   **Incremental Builds**: Only regenerates changed slides to save time and API costs.
@@ -69,7 +69,7 @@ Dynamic: license-file
 pip install slidemovie
 ```
 
-*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation.html) for details.*
+*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation/) for details.*
 
 ### 2. Create a Project
 

{slidemovie-0.2.2 → slidemovie-0.4.0}/README.md

@@ -21,7 +21,7 @@
 ## ✨ Features
 
 *   **Markdown-Based**: Write your slide content and narration script in a single text file.
-*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini** or **OpenAI** (via `multiai-tts`).
+*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini**, **OpenAI** or **Azure** (via `multiai-tts`).
 *   **PowerPoint Integration**: Use PowerPoint's AI "Designer" to create professional visuals instantly.
 *   **No Video Editing**: Audio and visuals are automatically synchronized.
 *   **Incremental Builds**: Only regenerates changed slides to save time and API costs.
@@ -39,7 +39,7 @@
 pip install slidemovie
 ```
 
-*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation.html) for details.*
+*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation/) for details.*
 
 ### 2. Create a Project
 

{slidemovie-0.2.2 → slidemovie-0.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "slidemovie"
-version = "0.2.2"
+version = "0.4.0"
 description = "Markdown and PowerPoint to narration video generator"
 readme = "README.md"
 requires-python = ">=3.8"

{slidemovie-0.2.2 → slidemovie-0.4.0}/slidemovie/cli.py

@@ -14,6 +14,7 @@ logging.basicConfig(
 )
 logger = logging.getLogger(__name__)
 
+
 def main():
     """
     Entry point for the slidemovie command-line tool.
@@ -76,11 +77,18 @@ def main():
     )
 
     # --- TTS Settings Options (CLI Overrides) ---
-    parser.add_argument("--tts-provider", help="TTS Provider (e.g., google, openai)")
+    parser.add_argument(
+        "--tts-provider",
+        help="TTS Provider (e.g., google, openai)")
     parser.add_argument("--tts-model", help="TTS Model name")
     parser.add_argument("--tts-voice", help="TTS Voice/Speaker setting")
-    parser.add_argument("--prompt", help="Override TTS system prompt (automatically enables prompt usage)")
-    parser.add_argument("--no-prompt", action="store_true", help="Disable TTS system prompt")
+    parser.add_argument(
+        "--prompt",
+        help="Override TTS system prompt (automatically enables prompt usage)")
+    parser.add_argument(
+        "--no-prompt",
+        action="store_true",
+        help="Disable TTS system prompt")
 
     # --- Other Options ---
     parser.add_argument(
@@ -89,23 +97,24 @@ def main():
         help="Enable debug mode (Verbose logging, etc)."
     )
 
-    args = parser.parse_args()
-
-    # Exit if no action is specified
-    if not args.pptx and not args.video:
-        parser.print_help()
-        sys.exit(1)
-
     # 1. Initialize Movie instance (Load configuration files)
     try:
         movie = slidemovie.Movie()
     except NameError:
-        logger.error("Movie class is not defined. Make sure to import it correctly.")
+        logger.error(
+            "Movie class is not defined. Make sure to import it correctly.")
         sys.exit(1)
     except Exception as e:
         logger.error(f"Failed to initialize Movie class: {e}")
         sys.exit(1)
 
+    args = parser.parse_args()
+
+    # Exit if no action is specified
+    if not args.pptx and not args.video:
+        parser.print_help()
+        sys.exit(1)
+
     # 2. Override settings with CLI options
     if args.tts_provider:
         movie.tts_provider = args.tts_provider
@@ -118,7 +127,9 @@ def main():
         movie.tts_use_prompt = True
     if args.no_prompt:
         movie.tts_use_prompt = False
-    
+    if args.filename:
+        movie.output_filename = args.filename
+
     if args.debug:
         movie.ffmpeg_loglevel = 'info'
         movie.show_skip = True
@@ -131,13 +142,13 @@ def main():
     try:
         if args.sub:
             # Hierarchical Mode (Parent/Child)
-            logger.info(f"Configuring subproject paths: {args.project_name}/{args.sub}")
+            logger.info(
+                f"Configuring subproject paths: {args.project_name}/{args.sub}")
             movie.configure_subproject_paths(
                 parent_project_name=args.project_name,
                 subproject_name=args.sub,
                 source_parent_dir=args.source_dir,
-                output_root_dir=args.output_root,
-                output_filename=args.filename
+                output_root_dir=args.output_root
             )
         else:
             # Standard Mode (Flat)
@@ -145,8 +156,7 @@ def main():
             movie.configure_project_paths(
                 project_name=args.project_name,
                 source_dir=args.source_dir,
-                output_root_dir=args.output_root,
-                output_filename=args.filename
+                output_root_dir=args.output_root
             )
     except Exception as e:
         logger.error(f"Failed to configure paths: {e}")
@@ -162,7 +172,8 @@ def main():
         movie.build_slide_pptx()
         logger.info("PPTX generation process finished.")
         if not args.video:
-            logger.info("Please edit the generated PPTX file and run with --video to create the movie.")
+            logger.info(
+                "Please edit the generated PPTX file and run with --video to create the movie.")
 
     # Generate Video (--video)
     if args.video:
@@ -170,8 +181,9 @@ def main():
         logger.info("MODE: Build All Video Assets")
         logger.info("=" * 60)
         movie.build_all()
-        
+
         logger.info("All video processes finished.")
 
+
 if __name__ == "__main__":
     main()

{slidemovie-0.2.2 → slidemovie-0.4.0}/slidemovie/core.py

@@ -14,6 +14,7 @@ from datetime import datetime
 # Configure module logger
 logger = logging.getLogger(__name__)
 
+
 class Movie():
     """
     A class to automatically generate narration videos based on PowerPoint slides and Markdown notes.
@@ -55,10 +56,12 @@ class Movie():
             - pandoc
         """
         required_tools = ['ffmpeg', 'ffprobe', 'pandoc']
-        missing_tools = [tool for tool in required_tools if not shutil.which(tool)]
-        
+        missing_tools = [
+            tool for tool in required_tools if not shutil.which(tool)]
+
         if missing_tools:
-            logger.error(f"Required external commands not found: {', '.join(missing_tools)}")
+            logger.error(
+                f"Required external commands not found: {', '.join(missing_tools)}")
             logger.error("Please install them before running this tool.")
             sys.exit(1)
 
@@ -86,6 +89,7 @@ class Movie():
             show_skip (bool): Whether to log skipped tasks. Default: False.
             max_retry (int): Max retries for TTS API errors. Default: 2.
             output_root (str): Root directory for video output. Default: None.
+            output_filename (str): Output video filename (without extension). Default: None (Uses project ID).
         """
         return {
             # TTS settings
@@ -116,18 +120,19 @@ class Movie():
             "show_skip": False,
             "max_retry": 2,
 
-            # Output path setting (Used if not provided via CLI)
-            "output_root": None
+            # Output path settings (Used if not provided via CLI)
+            "output_root": None,
+            "output_filename": None
         }
 
     def _load_settings(self):
         """
         Loads settings from JSON files and merges them with defaults.
-        
+
         It looks for configuration in:
         1. ~/.config/slidemovie/config.json
         2. ./config.json
-        
+
         Finally, it sets the configuration values as instance attributes.
         """
         # 1. Get default settings
@@ -139,10 +144,11 @@ class Movie():
             try:
                 os.makedirs(config_dir, exist_ok=True)
             except OSError as e:
-                logger.warning(f"Failed to create config directory {config_dir}: {e}")
+                logger.warning(
+                    f"Failed to create config directory {config_dir}: {e}")
 
         home_config_path = os.path.join(config_dir, "config.json")
-        
+
         if not os.path.exists(home_config_path):
             # Create default config file if it doesn't exist
             try:
@@ -162,7 +168,7 @@ class Movie():
 
         # 3. Process ./config.json (Current directory)
         local_config_path = "./config.json"
-        
+
         if os.path.exists(local_config_path):
             try:
                 with open(local_config_path, 'r', encoding='utf-8') as f:
@@ -173,7 +179,7 @@ class Movie():
                 logger.warning(f"Failed to load {local_config_path}: {e}")
 
         # 4. Set attributes
-        
+
         # Special handling: Convert screen_size from list to tuple
         if "screen_size" in config and isinstance(config["screen_size"], list):
             config["screen_size"] = tuple(config["screen_size"])
@@ -182,18 +188,16 @@ class Movie():
         for key, value in config.items():
             setattr(self, key, value)
 
-
-    def configure_project_paths(self, project_name, source_dir, output_root_dir=None, output_filename=None):
+    def configure_project_paths(
+            self, project_name, source_dir, output_root_dir=None):
         """
         Configures paths for a standard (flat) project structure.
-        
+
         Args:
             project_name (str): The name/ID of the project.
             source_dir (str): The directory containing source files (.md, .pptx).
-            output_root_dir (str, optional): Root directory for video output. 
+            output_root_dir (str, optional): Root directory for video output.
                                              Defaults to `self.output_root` or `{source_dir}/movie`.
-            output_filename (str, optional): Filename for the output video (without extension).
-                                             Defaults to `project_name`.
         """
         # Determine output root directory
         target_root = None
@@ -206,22 +210,25 @@ class Movie():
         else:
             target_root = f'{source_dir}/movie'
             is_automatic_path = True
-        
+
         # Expand path
         target_root = os.path.expanduser(target_root)
 
         # Handle directory existence
         if is_automatic_path:
-            # If the path is automatically determined, create it if it doesn't exist
+            # If the path is automatically determined, create it if it doesn't
+            # exist
             if not os.path.isdir(target_root):
                 try:
                     os.makedirs(target_root, exist_ok=True)
                     logger.info(f'Created output directory: {target_root}')
                 except OSError as e:
-                    logger.error(f'Failed to create directory {target_root}: {e}')
+                    logger.error(
+                        f'Failed to create directory {target_root}: {e}')
                     sys.exit(1)
         else:
-            # If the path is explicitly specified (CLI or Config), strict check is applied
+            # If the path is explicitly specified (CLI or Config), strict check
+            # is applied
             if not os.path.isdir(target_root):
                 logger.error(f'Directory {target_root} does not exist.')
                 sys.exit(1)
@@ -229,33 +236,34 @@ class Movie():
         # Set member variables
         self.source_dir = source_dir
         self.project_id = project_name
-        
-        if not output_filename:
-            output_filename = project_name
+
+        # Determine output filename
+        # Priority: self.output_filename (Config/CLI) > project_name (Default)
+        final_filename = self.output_filename if self.output_filename else project_name
 
         # Construct file paths
         self.md_file = f'{self.source_dir}/{project_name}.md'
         self.status_file = f'{self.source_dir}/status.json'
         self.video_length_file = f'{self.source_dir}/video_length.csv'
-        
+
         # Create intermediate/output directories
         self.movie_dir = f'{target_root}/{project_name}'
         if not os.path.isdir(self.movie_dir):
             os.mkdir(self.movie_dir)
 
         self.slide_file = f'{self.source_dir}/{project_name}.pptx'
-        self.video_file = f'{self.movie_dir}/{output_filename}.mp4'
+        self.video_file = f'{self.movie_dir}/{final_filename}.mp4'
 
-    def configure_subproject_paths(self, parent_project_name, subproject_name, source_parent_dir, output_root_dir=None, output_filename=None):
+    def configure_subproject_paths(
+            self, parent_project_name, subproject_name, source_parent_dir, output_root_dir=None):
         """
         Configures paths for a nested project structure (Parent Folder -> Child Folder).
-        
+
         Args:
             parent_project_name (str): The name of the parent project.
             subproject_name (str): The name of the subproject (child folder name).
             source_parent_dir (str): The directory containing the parent project folder.
             output_root_dir (str, optional): Root directory for video output.
-            output_filename (str, optional): Filename for the output video (without extension).
         """
         # Determine output root directory
         target_root = None
@@ -268,64 +276,68 @@ class Movie():
         else:
             target_root = f'{source_parent_dir}/movie'
             is_automatic_path = True
-        
+
         # Expand path
         target_root = os.path.expanduser(target_root)
 
         # Handle directory existence
         if is_automatic_path:
-            # If the path is automatically determined, create it if it doesn't exist
+            # If the path is automatically determined, create it if it doesn't
+            # exist
             if not os.path.isdir(target_root):
                 try:
                     os.makedirs(target_root, exist_ok=True)
                     logger.info(f'Created output directory: {target_root}')
                 except OSError as e:
-                    logger.error(f'Failed to create directory {target_root}: {e}')
+                    logger.error(
+                        f'Failed to create directory {target_root}: {e}')
                     sys.exit(1)
         else:
-            # If the path is explicitly specified (CLI or Config), strict check is applied
+            # If the path is explicitly specified (CLI or Config), strict check
+            # is applied
             if not os.path.isdir(target_root):
                 logger.error(f'Directory {target_root} does not exist.')
                 sys.exit(1)
 
         # Source directory is "Parent/Child"
         self.source_dir = f'{source_parent_dir}/{subproject_name}'
-        
+
         # Project ID format: "Parent-Child"
         self.project_id = f'{parent_project_name}-{subproject_name}'
-        
-        if not output_filename:
-            output_filename = self.project_id
+
+        # Determine output filename
+        # Priority: self.output_filename (Config/CLI) > self.project_id (Default)
+        final_filename = self.output_filename if self.output_filename else self.project_id
 
         # Construct file paths
         self.md_file = f'{self.source_dir}/{subproject_name}.md'
         self.status_file = f'{self.source_dir}/status.json'
         self.video_length_file = f'{self.source_dir}/video_length.csv'
-        
+
         # Create output directory hierarchy (movie/parent/child)
         parent_movie_dir = f'{target_root}/{parent_project_name}'
         self.movie_dir = f'{parent_movie_dir}/{subproject_name}'
-        
+
         if not os.path.isdir(parent_movie_dir):
             os.mkdir(parent_movie_dir)
         if not os.path.isdir(self.movie_dir):
             os.mkdir(self.movie_dir)
 
         self.slide_file = f'{self.source_dir}/{subproject_name}.pptx'
-        self.video_file = f'{self.movie_dir}/{output_filename}.mp4'
+        self.video_file = f'{self.movie_dir}/{final_filename}.mp4'
 
     def build_all(self):
         """
         Orchestrates the creation of the complete video from Markdown and PPTX files.
-        
-        Note: This does not update the PPTX file from Markdown. 
+
+        Note: This does not update the PPTX file from Markdown.
         Run `build_slide_pptx()` beforehand if necessary.
         """
         self._check_external_tools()
         if not os.path.isfile(self.md_file):
             logger.error(f'{self.md_file} does not exist.')
             sys.exit(1)
-            
+
         # 1. Generate narration audio from Markdown notes
         self.build_slide_audio()
         # 2. Generate slide images from PPTX
@@ -375,7 +387,8 @@ class Movie():
                 audio_state["wav_file"]
             )
 
-            # Regeneration check (Status mismatch OR Hash mismatch OR File missing)
+            # Regeneration check (Status mismatch OR Hash mismatch OR File
+            # missing)
             if (audio_status != "generated" or
                 saved_notes_hash != current_notes_hash or
                     not os.path.isfile(wav_path)):
@@ -384,7 +397,8 @@ class Movie():
 
                 add_prompt = audio_state.get("additional_prompt", "")
                 if norm == "":
-                    logger.error(f'Error: "::: notes" not found in {slide_id}.')
+                    logger.error(
+                        f'Error: "::: notes" not found in {slide_id}.')
                     sys.exit()
                 self._speak_to_wav(
                     norm, wav_path, additional_prompt=add_prompt)
@@ -533,7 +547,8 @@ class Movie():
                             f"[SKIP] {slide_id} (Video: unchanged/Source:{video_file_src})")
                     continue
 
-                logger.info(f"Converting video: {video_file_src} -> {slide_id}.mp4")
+                logger.info(
+                    f"Converting video: {video_file_src} -> {slide_id}.mp4")
 
                 # FFmpeg command: Resize + Audio re-encode
                 cmd = [
@@ -580,7 +595,8 @@ class Movie():
                 png_file = os.path.join(self.movie_dir, f"{slide_id}.png")
                 wav_file = os.path.join(self.movie_dir, f"{slide_id}.wav")
 
-                if not os.path.isfile(png_file) or not os.path.isfile(wav_file):
+                if not os.path.isfile(
+                        png_file) or not os.path.isfile(wav_file):
                     # Skip if assets are missing
                     logger.warning(f"Material missing, skipping: {slide_id}")
                     continue
@@ -1040,7 +1056,8 @@ class Movie():
         current_config = self._get_build_config()
 
         if stored_config is None:
-            logger.info("No build_config in state file. Applying current settings.")
+            logger.info(
+                "No build_config in state file. Applying current settings.")
             state["build_config"] = current_config
             self._save_audio_state(state)
             stored_config = current_config
@@ -1048,7 +1065,8 @@ class Movie():
         if stored_config != current_config:
             import pprint
             logger.error("build_config inconsistency detected.")
-            logger.error("Changing resolution/FPS mid-process is not supported. Aborting.")
+            logger.error(
+                "Changing resolution/FPS mid-process is not supported. Aborting.")
             logger.error("-" * 40)
             logger.error("[Stored Config]")
             logger.error(pprint.pformat(stored_config))
@@ -1064,7 +1082,8 @@ class Movie():
 
         # Auto-fill if missing (Migration)
         if stored_tts is None:
-            logger.info("No TTS config in state file. Applying current settings.")
+            logger.info(
+                "No TTS config in state file. Applying current settings.")
             state["tts_config"] = current_tts
             self._save_audio_state(state)
 
@@ -1080,13 +1099,15 @@ class Movie():
             logger.warning("[Current Config (Now)]")
             logger.warning(pprint.pformat(current_tts))
             logger.warning("=" * 60)
-            logger.warning("Generating audio with different settings may result in inconsistent audio in the video.")
+            logger.warning(
+                "Generating audio with different settings may result in inconsistent audio in the video.")
 
             while True:
                 choice = input(
                     "Select action: 1) Ignore and Continue (Overwrite config)  2) Abort [1/2]: ").strip()
                 if choice == '1':
-                    logger.info("Applying new settings and continuing. Updating state file.")
+                    logger.info(
+                        "Applying new settings and continuing. Updating state file.")
                     state["tts_config"] = current_tts
                     self._save_audio_state(state)
                     break
@@ -1286,17 +1307,21 @@ class Movie():
             additional_prompt (str): Additional prompt for specific slides.
         """
         client = multiai_tts.Prompt()
-        client.set_tts_model(self.tts_provider, self.tts_model)
         if self.tts_provider == 'openai':
+            client.set_tts_model(self.tts_provider, self.tts_model)
             client.tts_voice_openai = self.tts_voice
         if self.tts_provider == 'google':
+            client.set_tts_model(self.tts_provider, self.tts_model)
             client.tts_voice_google = self.tts_voice
+        if self.tts_provider == 'azure':
+            client.set_tts_provider(self.tts_provider)
+            client.tts_voice_azure = self.tts_voice
 
         if self.tts_use_prompt:
             full_prompt_text = f'{self.prompt}{additional_prompt}\n{text}'
         else:
             full_prompt_text = text
-
+        
         for attempt in range(self.max_retry):
             client.save_tts(full_prompt_text, wav_path)
 

{slidemovie-0.2.2 → slidemovie-0.4.0/slidemovie.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: slidemovie
-Version: 0.2.2
+Version: 0.4.0
 Summary: Markdown and PowerPoint to narration video generator
 Author: Katsutoshi Seki
 License-Expression: MIT
@@ -51,7 +51,7 @@ Dynamic: license-file
 ## ✨ Features
 
 *   **Markdown-Based**: Write your slide content and narration script in a single text file.
-*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini** or **OpenAI** (via `multiai-tts`).
+*   **AI Narration**: Automatically generates natural voiceovers using **Google Gemini**, **OpenAI** or **Azure** (via `multiai-tts`).
 *   **PowerPoint Integration**: Use PowerPoint's AI "Designer" to create professional visuals instantly.
 *   **No Video Editing**: Audio and visuals are automatically synchronized.
 *   **Incremental Builds**: Only regenerates changed slides to save time and API costs.
@@ -69,7 +69,7 @@ Dynamic: license-file
 pip install slidemovie
 ```
 
-*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation.html) for details.*
+*Note: You also need to install **FFmpeg**, **Pandoc**, **LibreOffice**, and **Poppler**, and set up your **AI API Key** (Google or OpenAI). See the [documentation](https://sekika.github.io/slidemovie/installation/) for details.*
 
 ### 2. Create a Project
 

{slidemovie-0.2.2 → slidemovie-0.4.0}/tests/test_cli.py

@@ -36,11 +36,11 @@ def test_cli_pptx_mode(mock_movie_class):
     mock_movie_class.assert_called_once()
     
     # Check if default path configuration was called (Flat mode)
+    # [Fix] output_filename argument removed from call signature
     mock_instance.configure_project_paths.assert_called_with(
         project_name='MyProject',
         source_dir='.',
-        output_root_dir=None,
-        output_filename=None
+        output_root_dir=None
     )
     
     # Check if correct build method was called
@@ -62,17 +62,29 @@ def test_cli_video_mode_subproject(mock_movie_class):
     assert mock_instance.show_skip is True
     
     # Verify subproject path configuration
+    # [Fix] output_filename argument removed from call signature
     mock_instance.configure_subproject_paths.assert_called_with(
         parent_project_name='ParentProj',
         subproject_name='ChildProj',
         source_parent_dir='.',
-        output_root_dir=None,
-        output_filename=None
+        output_root_dir=None
     )
     
     # Verify build_all was called
     mock_instance.build_all.assert_called_once()
 
+def test_cli_filename_argument(mock_movie_class):
+    """Test if -f/--filename argument sets the movie attribute correctly."""
+    # Test short flag -f
+    test_args = ['slidemovie', 'Proj', '--video', '-f', 'custom_name']
+    
+    mock_instance = mock_movie_class.return_value
+    
+    with patch.object(sys, 'argv', test_args):
+        cli.main()
+        
+    assert mock_instance.output_filename == 'custom_name'
+
 def test_cli_override_tts_options(mock_movie_class):
     """Test if CLI arguments override TTS configuration."""
     test_args = [

{slidemovie-0.2.2 → slidemovie-0.4.0}/tests/test_core.py

@@ -27,28 +27,26 @@ def movie(mock_tools):
     """Fixture to create a Movie instance."""
     m = Movie()
     # Reset output_root to None to ensure tests rely on the source_dir structure
-    # and ignore any user-defined 'output_root' in local config.json.
     m.output_root = None
+    # Ensure output_filename is None by default (mimic init state)
+    m.output_filename = None
     return m
 
 class TestMovieConfig:
     def test_default_settings(self, movie):
         """Test if settings are loaded (checking key existence)."""
-        # We check existence because actual values might be overridden by local config
         assert hasattr(movie, 'tts_provider')
         assert hasattr(movie, 'screen_size')
+        # [Fix] Check for output_filename
+        assert hasattr(movie, 'output_filename')
 
     def test_load_settings_override(self, mock_tools, tmp_path):
         """Test overriding settings via config.json logic."""
-        # Note: Since _load_settings is called in __init__, testing exact loading
-        # behavior without mocking open() globally is complex. 
-        # Here we just verify the instance attributes can be set.
         pass 
 
 class TestPathConfiguration:
     def test_configure_project_paths_flat(self, movie, tmp_path):
         """Test path configuration for standard (flat) mode."""
-        # Ensure output_root is None so it falls back to source_dir/movie
         movie.output_root = None
         
         source_dir = tmp_path / "src"
@@ -66,6 +64,26 @@ class TestPathConfiguration:
         expected_movie_dir = source_dir / "movie" / "test_proj"
         assert movie.movie_dir == str(expected_movie_dir)
         assert os.path.exists(movie.movie_dir)
+        
+        # [Fix] Verify default video filename (same as project name)
+        assert movie.video_file.endswith("test_proj.mp4")
+
+    def test_configure_project_paths_with_custom_filename(self, movie, tmp_path):
+        """Test path configuration with custom output_filename attribute set."""
+        source_dir = tmp_path / "src"
+        source_dir.mkdir()
+        
+        # Manually set the attribute (as CLI or Config would do)
+        movie.output_filename = "custom_output_name"
+        
+        movie.configure_project_paths(
+            project_name="test_proj",
+            source_dir=str(source_dir)
+        )
+        
+        # Verify the filename uses the custom name, not project name
+        assert movie.video_file.endswith("custom_output_name.mp4")
+        assert "test_proj.mp4" not in movie.video_file
 
     def test_configure_subproject_paths(self, movie, tmp_path):
         """Test path configuration for subproject (Parent/Child) mode."""
@@ -86,6 +104,9 @@ class TestPathConfiguration:
         # Output: parent/movie/parent_proj/child_sub
         expected_movie_dir = parent_dir / "movie" / "parent_proj" / "child_sub"
         assert movie.movie_dir == str(expected_movie_dir)
+        
+        # [Fix] Verify default filename (Parent-Child)
+        assert movie.video_file.endswith("parent_proj-child_sub.mp4")
 
 class TestMarkdownProcessing:
     def test_ensure_slide_ids(self, movie, tmp_path):
@@ -147,7 +168,7 @@ class TestBuildLogic:
         movie.slide_file = str(tmp_path / "test.pptx")
         movie.source_dir = str(tmp_path)
         
-        # [Fix] Manually set project_id as it is required by _init_audio_state
+        # Manually set project_id as it is required by _init_audio_state
         movie.project_id = "test_project"
         
         # Mock status file