talks-reducer 0.3.1__tar.gz → 0.3.2__tar.gz

{talks_reducer-0.3.1/talks_reducer.egg-info → talks_reducer-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: talks-reducer
-Version: 0.3.1
+Version: 0.3.2
 Summary: CLI for speeding up long-form talks by removing silence
 Author: Talks Reducer Maintainers
 License-Expression: MIT
@@ -32,6 +32,10 @@ project was renamed from **jumpcutter** to emphasize its focus on conference tal
 - 1h 19m, 751 MB — Talks Reducer
 - 1h 19m, 171 MB — Talks Reducer `--small`
 
+## Changelog
+
+See [CHANGELOG.md](CHANGELOG.md).
+
 ## Install GUI (Windows, macOS)
 Go to the [releases page](https://github.com/popstas/talks-reducer/releases) and download the appropriate artifact:
 
@@ -50,9 +54,9 @@ pip install talks-reducer
 The `--small` preset applies a 720p video scale and 128 kbps audio bitrate, making it useful for sharing talks over constrained
 connections. Without `--small`, the script aims to preserve original quality while removing silence.
 
-> **Tip:** Running `talks-reducer-gui` without arguments opens the Tkinter interface. Passing regular CLI options (for example,
-> `talks-reducer-gui --small input.mp4`) now executes the command-line pipeline, so you can keep a single shortcut for both
-> workflows.
+> **Tip:** The `talks-reducer` and `talks-reducer-gui` commands now behave the same way: launching them without arguments opens
+> the Tkinter interface, while passing regular CLI options (for example, `talks-reducer --small input.mp4`) executes the
+> command-line pipeline. You can keep a single shortcut for both workflows.
 
 When CUDA-capable hardware is available the pipeline leans on GPU encoders to keep export times low, but it still runs great on
 CPUs.

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/README.md

@@ -7,6 +7,10 @@ project was renamed from **jumpcutter** to emphasize its focus on conference tal
 - 1h 19m, 751 MB — Talks Reducer
 - 1h 19m, 171 MB — Talks Reducer `--small`
 
+## Changelog
+
+See [CHANGELOG.md](CHANGELOG.md).
+
 ## Install GUI (Windows, macOS)
 Go to the [releases page](https://github.com/popstas/talks-reducer/releases) and download the appropriate artifact:
 
@@ -25,9 +29,9 @@ pip install talks-reducer
 The `--small` preset applies a 720p video scale and 128 kbps audio bitrate, making it useful for sharing talks over constrained
 connections. Without `--small`, the script aims to preserve original quality while removing silence.
 
-> **Tip:** Running `talks-reducer-gui` without arguments opens the Tkinter interface. Passing regular CLI options (for example,
-> `talks-reducer-gui --small input.mp4`) now executes the command-line pipeline, so you can keep a single shortcut for both
-> workflows.
+> **Tip:** The `talks-reducer` and `talks-reducer-gui` commands now behave the same way: launching them without arguments opens
+> the Tkinter interface, while passing regular CLI options (for example, `talks-reducer --small input.mp4`) executes the
+> command-line pipeline. You can keep a single shortcut for both workflows.
 
 When CUDA-capable hardware is available the pipeline leans on GPU encoders to keep export times low, but it still runs great on
 CPUs.

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "talks-reducer"
-version = "0.3.1"
+version = "0.3.2"
 description = "CLI for speeding up long-form talks by removing silence"
 readme = "README.md"
 requires-python = ">=3.9"

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/talks_reducer/cli.py

@@ -6,6 +6,7 @@ import argparse
 import os
 import sys
 import time
+from importlib import import_module
 from importlib.metadata import version
 from pathlib import Path
 from typing import Dict, List, Optional, Sequence
@@ -23,19 +24,19 @@ def _build_parser() -> argparse.ArgumentParser:
     parser = argparse.ArgumentParser(
         description="Modifies a video file to play at different speeds when there is sound vs. silence.",
     )
-    
+
     # Add version argument
     try:
         pkg_version = version("talks-reducer")
     except Exception:
         pkg_version = "unknown"
-    
+
     parser.add_argument(
         "--version",
         action="version",
         version=f"talks-reducer {pkg_version}",
     )
-    
+
     parser.add_argument(
         "input_file",
         type=str,
@@ -113,31 +114,34 @@ def gather_input_files(paths: List[str]) -> List[str]:
     return files
 
 
+def _launch_gui(argv: Sequence[str]) -> bool:
+    """Attempt to launch the GUI with the provided arguments."""
+
+    try:
+        gui_module = import_module(".gui", __package__)
+    except ImportError:
+        return False
+
+    gui_main = getattr(gui_module, "main", None)
+    if gui_main is None:
+        return False
+
+    return bool(gui_main(list(argv)))
+
+
 def main(argv: Optional[Sequence[str]] = None) -> None:
     """Entry point for the command line interface.
 
     Launch the GUI when run without arguments, otherwise defer to the CLI.
     """
 
-    # Check if running without arguments
     if argv is None:
         argv_list = sys.argv[1:]
     else:
         argv_list = list(argv)
 
-    # Launch GUI if no arguments provided
     if not argv_list:
-        gui_launched = False
-
-        try:
-            from .gui import main as gui_main
-
-            gui_launched = gui_main([])
-        except ImportError:
-            # GUI dependencies not available, show help instead
-            gui_launched = False
-
-        if gui_launched:
+        if _launch_gui(argv_list):
             return
 
         parser = _build_parser()
@@ -145,7 +149,7 @@ def main(argv: Optional[Sequence[str]] = None) -> None:
         return
 
     parser = _build_parser()
-    parsed_args = parser.parse_args(argv)
+    parsed_args = parser.parse_args(argv_list)
     start_time = time.time()
 
     files = gather_input_files(parsed_args.input_file)

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/talks_reducer/ffmpeg.py

@@ -38,6 +38,7 @@ def find_ffmpeg() -> Optional[str]:
     # Try bundled ffmpeg from imageio-ffmpeg first
     try:
         import imageio_ffmpeg
+
         bundled_path = imageio_ffmpeg.get_ffmpeg_exe()
         if bundled_path and os.path.isfile(bundled_path):
             return bundled_path
@@ -161,11 +162,11 @@ def check_cuda_available(ffmpeg_path: Optional[str] = None) -> bool:
     try:
         ffmpeg_path = ffmpeg_path or get_ffmpeg_path()
         result = subprocess.run(
-            [ffmpeg_path, "-encoders"], 
-            capture_output=True, 
-            text=True, 
+            [ffmpeg_path, "-encoders"],
+            capture_output=True,
+            text=True,
             timeout=5,
-            creationflags=creationflags
+            creationflags=creationflags,
         )
     except (
         subprocess.TimeoutExpired,
@@ -193,7 +194,7 @@ def run_timed_ffmpeg_command(
     process_callback: Optional[callable] = None,
 ) -> None:
     """Execute an FFmpeg command while streaming progress information.
-    
+
     Args:
         process_callback: Optional callback that receives the subprocess.Popen object
     """
@@ -243,7 +244,7 @@ def run_timed_ffmpeg_command(
 
             sys.stderr.write(line)
             sys.stderr.flush()
-            
+
             # Send FFmpeg output to reporter for GUI display
             progress_reporter.log(line.strip())
 

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/talks_reducer/gui.py

@@ -1215,10 +1215,10 @@ class TalksReducerGUI:
                 self.drop_hint_button.grid_remove()
                 self.open_button.grid()
                 self.open_button.lift()  # Ensure open_button is above drop_hint_button
-                print("success status")
+                # print("success status")
             else:
                 self.open_button.grid_remove()
-                print("not success status")
+                # print("not success status")
                 if (
                     self.simple_mode_var.get()
                     and not is_processing

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/talks_reducer/pipeline.py

@@ -158,7 +158,7 @@ def speed_up_video(
     )
 
     reporter.log("Extracting audio...")
-    process_callback = getattr(reporter, 'process_callback', None)
+    process_callback = getattr(reporter, "process_callback", None)
     run_timed_ffmpeg_command(
         extract_command,
         reporter=reporter,

{talks_reducer-0.3.1 → talks_reducer-0.3.2/talks_reducer.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: talks-reducer
-Version: 0.3.1
+Version: 0.3.2
 Summary: CLI for speeding up long-form talks by removing silence
 Author: Talks Reducer Maintainers
 License-Expression: MIT
@@ -32,6 +32,10 @@ project was renamed from **jumpcutter** to emphasize its focus on conference tal
 - 1h 19m, 751 MB — Talks Reducer
 - 1h 19m, 171 MB — Talks Reducer `--small`
 
+## Changelog
+
+See [CHANGELOG.md](CHANGELOG.md).
+
 ## Install GUI (Windows, macOS)
 Go to the [releases page](https://github.com/popstas/talks-reducer/releases) and download the appropriate artifact:
 
@@ -50,9 +54,9 @@ pip install talks-reducer
 The `--small` preset applies a 720p video scale and 128 kbps audio bitrate, making it useful for sharing talks over constrained
 connections. Without `--small`, the script aims to preserve original quality while removing silence.
 
-> **Tip:** Running `talks-reducer-gui` without arguments opens the Tkinter interface. Passing regular CLI options (for example,
-> `talks-reducer-gui --small input.mp4`) now executes the command-line pipeline, so you can keep a single shortcut for both
-> workflows.
+> **Tip:** The `talks-reducer` and `talks-reducer-gui` commands now behave the same way: launching them without arguments opens
+> the Tkinter interface, while passing regular CLI options (for example, `talks-reducer --small input.mp4`) executes the
+> command-line pipeline. You can keep a single shortcut for both workflows.
 
 When CUDA-capable hardware is available the pipeline leans on GPU encoders to keep export times low, but it still runs great on
 CPUs.

{talks_reducer-0.3.1 → talks_reducer-0.3.2}/talks_reducer.egg-info/SOURCES.txt

@@ -18,4 +18,5 @@ talks_reducer.egg-info/entry_points.txt
 talks_reducer.egg-info/requires.txt
 talks_reducer.egg-info/top_level.txt
 tests/test_audio.py
+tests/test_cli.py
 tests/test_pipeline_service.py

talks_reducer-0.3.2/tests/test_cli.py

@@ -0,0 +1,78 @@
+"""Tests for the CLI entry point behaviour."""
+
+from __future__ import annotations
+
+from pathlib import Path
+from types import SimpleNamespace
+from unittest import mock
+
+import pytest
+
+from talks_reducer import cli
+
+
+def test_main_launches_gui_when_no_args(monkeypatch: pytest.MonkeyPatch) -> None:
+    """The GUI should be launched when no CLI arguments are provided."""
+
+    launch_calls: list[list[str]] = []
+
+    def fake_launch(argv: list[str]) -> bool:
+        launch_calls.append(list(argv))
+        return True
+
+    def fail_build_parser() -> None:
+        raise AssertionError("Parser should not be built when GUI launches")
+
+    monkeypatch.setattr(cli, "_launch_gui", fake_launch)
+    monkeypatch.setattr(cli, "_build_parser", fail_build_parser)
+
+    cli.main([])
+
+    assert launch_calls == [[]]
+
+
+def test_main_runs_cli_with_arguments(monkeypatch: pytest.MonkeyPatch) -> None:
+    """Providing CLI arguments should bypass the GUI and run the pipeline."""
+
+    parsed_args = SimpleNamespace(
+        input_file=["input.mp4"],
+        output_file=None,
+        temp_folder=None,
+        silent_threshold=None,
+        silent_speed=None,
+        sounded_speed=None,
+        frame_spreadage=None,
+        sample_rate=None,
+        small=False,
+    )
+
+    parser_mock = mock.Mock()
+    parser_mock.parse_args.return_value = parsed_args
+
+    outputs: list[cli.ProcessingOptions] = []
+
+    class DummyReporter:
+        def log(self, _message: str) -> None:  # pragma: no cover - simple stub
+            pass
+
+    def fake_speed_up_video(options: cli.ProcessingOptions, reporter: object):
+        outputs.append(options)
+        return SimpleNamespace(output_file=Path("/tmp/output.mp4"))
+
+    def fake_gather_input_files(_paths: list[str]) -> list[str]:
+        return ["/tmp/input.mp4"]
+
+    def fail_launch(_argv: list[str]) -> bool:
+        raise AssertionError("GUI should not be launched when arguments exist")
+
+    monkeypatch.setattr(cli, "_build_parser", lambda: parser_mock)
+    monkeypatch.setattr(cli, "gather_input_files", fake_gather_input_files)
+    monkeypatch.setattr(cli, "speed_up_video", fake_speed_up_video)
+    monkeypatch.setattr(cli, "TqdmProgressReporter", lambda: DummyReporter())
+    monkeypatch.setattr(cli, "_launch_gui", fail_launch)
+
+    cli.main(["input.mp4"])
+
+    parser_mock.parse_args.assert_called_once_with(["input.mp4"])
+    assert len(outputs) == 1
+    assert outputs[0].input_file == Path("/tmp/input.mp4")