add available themes to --theme help string

patch bump

README.md

@@ -30,7 +30,7 @@ pipx install simple-recorder
 
 *with pyz*
 
-An executable pyz has been included in [Release](https://github.com/onyx-and-iris/simple-recorder/releases) which you can run in Windows. Follow the steps in this [Setting up Windows for Zipapps](https://jhermann.github.io/blog/python/deployment/2020/02/29/python_zippapps_on_windows.html#Setting-Up-Windows-10-for-Zipapps) guide.
+An executable pyz has been included in [Releases](https://github.com/onyx-and-iris/simple-recorder/releases) which you can run in Windows. Follow the steps in this [Setting up Windows for Zipapps](https://jhermann.github.io/blog/python/deployment/2020/02/29/python_zippapps_on_windows.html#Setting-Up-Windows-10-for-Zipapps) guide.
 
 ## Configuration
 
@@ -65,33 +65,32 @@ Just enter the filename and click *Start Recording*.
 
 #### Themes
 
-Passing flags is fine, however, for example to set the theme:
+However, passing flags is fine, for example to set the theme:
 
 ```console
 simple-recorder --theme="Light Purple"
 ```
 
-Available themes: Light Purple, Neutral Blue, Reds, Sandy Beach, Kayak, Light Blue 2, Dark Teal1
-
 ### CLI
 
 ```shell
 Usage: simple-recorder [OPTIONS] COMMAND
 
-┏━ Subcommands ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
+┏━ Subcommands ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
-┃ start  Start recording                                              ┃
+┃ start  Start recording                                                              ┃
-┃ stop   Stop recording                                               ┃
+┃ stop   Stop recording                                                               ┃
-┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
+┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 
-┏━ Options ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
+┏━ Options ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
-┃ --host <HOST>          OBS WebSocket host                           ┃
+┃ --host <HOST>          OBS WebSocket host                                           ┃
-┃ --port <PORT>          OBS WebSocket port                           ┃
+┃ --port <PORT>          OBS WebSocket port                                           ┃
-┃ --password <PASSWORD>  OBS WebSocket password                       ┃
+┃ --password <PASSWORD>  OBS WebSocket password                                       ┃
-┃ --theme <THEME>        OBS WebSocket theme                          ┃
+┃ --theme <THEME>        GUI theme (Light Purple, Neutral Blue, Reds, Sandy Beach,    ┃
-┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
+┃                        Kayak, Light Blue 2)                                         ┃
+┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 ```
 
-For example:
+To launch the CLI pass any subcommand (start/stop etc...), for example:
 
 ```console
 simple-recorder start "File Name"

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "simple-recorder"
-version = "0.1.3"
+version = "0.1.5"
 description = "A simple OBS recorder"
 authors = [{ name = "onyx-and-iris", email = "code@onyxandiris.online" }]
 dependencies = [

src/simple_recorder/cli.py

@@ -5,7 +5,6 @@ from typing_extensions import override
 
 from .errors import SimpleRecorderError
 from .gui import SimpleRecorderWindow
-from .split import Split
 from .start import Start
 from .stop import Stop
 
@@ -16,18 +15,18 @@ config = ClypiConfig(
 )
 configure(config)
 
+themes = [
+    "Light Purple",
+    "Neutral Blue",
+    "Reds",
+    "Sandy Beach",
+    "Kayak",
+    "Light Blue 2",
+]
+
 
 def theme_parser(value: str) -> str:
     """Parse the theme argument."""
-    themes = [
-        "Light Purple",
-        "Neutral Blue",
-        "Reds",
-        "Sandy Beach",
-        "Kayak",
-        "Light Blue 2",
-        "Dark Teal1",
-    ]
     if value not in themes:
         raise ClypiException(
             f"Invalid theme: {value}. Available themes: {', '.join(themes)}"
@@ -36,14 +35,17 @@ def theme_parser(value: str) -> str:
 
 
 class SimpleRecorder(Command):
-    subcommand: Start | Stop | Split | None = None
+    subcommand: Start | Stop | None = None
     host: str = arg(default="localhost", env="OBS_HOST", help="OBS WebSocket host")
     port: int = arg(default=4455, env="OBS_PORT", help="OBS WebSocket port")
     password: str | None = arg(
         default=None, env="OBS_PASSWORD", help="OBS WebSocket password"
     )
     theme: str = arg(
-        default="Reds", parser=theme_parser, env="OBS_THEME", help="OBS WebSocket theme"
+        default="Reds",
+        parser=theme_parser,
+        env="OBS_THEME",
+        help=f"GUI theme ({', '.join(themes)})",
     )
 
     @override

src/simple_recorder/gui.py

@@ -3,7 +3,6 @@ import logging
 import FreeSimpleGUI as fsg
 from clypi import ClypiException
 
-from .split import Split
 from .start import Start
 from .stop import Stop
 
@@ -21,18 +20,13 @@ class SimpleRecorderWindow(fsg.Window):
         layout = [
             [fsg.Text("Enter recording filename:")],
             [fsg.InputText("", key="-FILENAME-")],
-            [
+            [fsg.Button("Start Recording"), fsg.Button("Stop Recording")],
-                fsg.Button("Start Recording"),
-                fsg.Button("Stop Recording"),
-                fsg.Button("Split Recording"),
-            ],
             [fsg.Text("Status: Not started", key="-OUTPUT-")],
         ]
         super().__init__("Simple Recorder", layout, finalize=True)
         self["-FILENAME-"].bind("<Return>", " || RETURN")
         self["Start Recording"].bind("<Return>", " || RETURN")
         self["Stop Recording"].bind("<Return>", " || RETURN")
-        self["Split Recording"].bind("<Return>", " || RETURN")
 
     async def run(self):
         while True:
@@ -70,19 +64,6 @@ class SimpleRecorderWindow(fsg.Window):
                             f"Error: {e.raw_message}", text_color="red"
                         )
 
-                case ["Split Recording", "RETURN" | None]:
-                    try:
-                        await Split(
-                            host=self.host, port=self.port, password=self.password
-                        ).run()
-                        self["-OUTPUT-"].update(
-                            "Recording split successfully", text_color="green"
-                        )
-                    except ClypiException as e:
-                        self["-OUTPUT-"].update(
-                            f"Error: {e.raw_message}", text_color="red"
-                        )
-
                 case _:
                     self.logger.warning(f"Unhandled event: {event}")
 

src/simple_recorder/split.py

@@ -1,27 +0,0 @@
-import obsws_python as obsws
-from clypi import Command, arg
-
-from .errors import SimpleRecorderError
-
-
-class Split(Command):
-    """Split recording."""
-
-    host: str = arg(inherited=True)
-    port: int = arg(inherited=True)
-    password: str = arg(inherited=True)
-
-    async def run(self):
-        with obsws.ReqClient(
-            host=self.host, port=self.port, password=self.password
-        ) as client:
-            resp = client.get_record_status()
-            if not resp.output_active:
-                raise SimpleRecorderError("Recording is not active.")
-            if resp.output_paused:
-                raise SimpleRecorderError(
-                    "Recording is paused. Please resume before splitting."
-                )
-
-            client.split_record_file()
-            print("Recording split successfully.")