CHANGELOG.md

@@ -5,12 +5,6 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-# [0.24.6] - 2026-01-26
-
-### Changed
-
--   environment variables should now be prefixed with 'OBSWS_CLI_', see [Environment Variables](https://github.com/onyx-and-iris/obsws-cli?tab=readme-ov-file#environment-variables)
-
 # [0.24.0] - 2026-01-09
 
 ### Added

README.md

@@ -62,9 +62,9 @@ Store and load environment variables from:
 -   `user home directory / .config / obsws-cli / obsws.env`
 
 ```env
-OBSWS_CLI_HOST=localhost
+OBS_HOST=localhost
-OBSWS_CLI_PORT=4455
+OBS_PORT=4455
-OBSWS_CLI_PASSWORD=<websocket password>
+OBS_PASSWORD=<websocket password>
 ```
 
 Flags can be used to override environment variables.
@@ -96,8 +96,8 @@ obsws-cli --style="cyan" --no-border sceneitem list
 Or with environment variables:
 
 ```env
-OBSWS_CLI_STYLE=cyan
+OBS_STYLE=cyan
-OBSWS_CLI_STYLE_NO_BORDER=true
+OBS_STYLE_NO_BORDER=true
 ```
 
 ## Root Typer

obsws_cli/__about__.py

@@ -1,4 +1,4 @@
 # SPDX-FileCopyrightText: 2025-present onyx-and-iris <code@onyxandiris.online>
 #
 # SPDX-License-Identifier: MIT
-__version__ = '0.24.6'
+__version__ = '0.24.0'

obsws_cli/alias.py

@@ -24,8 +24,6 @@ class RootTyperAliasGroup(typer.core.TyperGroup):
                 cmd_name = 'hotkey'
             case 'i':
                 cmd_name = 'input'
-            case 'm':
-                cmd_name = 'media'
             case 'prf':
                 cmd_name = 'profile'
             case 'prj':
@@ -42,8 +40,6 @@ class RootTyperAliasGroup(typer.core.TyperGroup):
                 cmd_name = 'sceneitem'
             case 'ss':
                 cmd_name = 'screenshot'
-            case 'set':
-                cmd_name = 'settings'
             case 'st':
                 cmd_name = 'stream'
             case 'sm':

obsws_cli/app.py

@@ -2,7 +2,6 @@
 
 import importlib
 import logging
-import pkgutil
 from typing import Annotated
 
 import obsws_python as obsws
@@ -10,15 +9,32 @@ import typer
 
 from obsws_cli.__about__ import __version__ as version
 
-from . import commands, console, envconfig, styles
+from . import config, console, styles
 from .alias import RootTyperAliasGroup
 
 app = typer.Typer(cls=RootTyperAliasGroup)
-for importer, modname, ispkg in pkgutil.iter_modules(
+for sub_typer in (
-    commands.__path__, commands.__name__ + '.'
+    'filter',
+    'group',
+    'hotkey',
+    'input',
+    'media',
+    'profile',
+    'projector',
+    'record',
+    'replaybuffer',
+    'settings',
+    'scene',
+    'scenecollection',
+    'sceneitem',
+    'screenshot',
+    'stream',
+    'studiomode',
+    'text',
+    'virtualcam',
 ):
-    subtyper = importlib.import_module(modname)
+    module = importlib.import_module(f'.{sub_typer}', package=__package__)
-    app.add_typer(subtyper.app, name=modname.split('.')[-1])
+    app.add_typer(module.app, name=sub_typer)
 
 
 def version_callback(value: bool):
@@ -54,62 +70,62 @@ def main(
         typer.Option(
             '--host',
             '-H',
-            envvar='OBSWS_CLI_HOST',
+            envvar='OBS_HOST',
             help='WebSocket host',
             show_default='localhost',
         ),
-    ] = envconfig.get('host'),
+    ] = config.get('host'),
     port: Annotated[
         int,
         typer.Option(
             '--port',
             '-P',
-            envvar='OBSWS_CLI_PORT',
+            envvar='OBS_PORT',
             help='WebSocket port',
             show_default=4455,
         ),
-    ] = envconfig.get('port'),
+    ] = config.get('port'),
     password: Annotated[
         str,
         typer.Option(
             '--password',
             '-p',
-            envvar='OBSWS_CLI_PASSWORD',
+            envvar='OBS_PASSWORD',
             help='WebSocket password',
             show_default=False,
         ),
-    ] = envconfig.get('password'),
+    ] = config.get('password'),
     timeout: Annotated[
         int,
         typer.Option(
             '--timeout',
             '-T',
-            envvar='OBSWS_CLI_TIMEOUT',
+            envvar='OBS_TIMEOUT',
             help='WebSocket timeout',
             show_default=5,
         ),
-    ] = envconfig.get('timeout'),
+    ] = config.get('timeout'),
     style: Annotated[
         str,
         typer.Option(
             '--style',
             '-s',
-            envvar='OBSWS_CLI_STYLE',
+            envvar='OBS_STYLE',
             help='Set the style for the CLI output',
             show_default='disabled',
             callback=validate_style,
         ),
-    ] = envconfig.get('style'),
+    ] = config.get('style'),
     no_border: Annotated[
         bool,
         typer.Option(
             '--no-border',
             '-b',
-            envvar='OBSWS_CLI_STYLE_NO_BORDER',
+            envvar='OBS_STYLE_NO_BORDER',
             help='Disable table border styling in the CLI output',
             show_default=False,
         ),
-    ] = envconfig.get('style_no_border'),
+    ] = config.get('style_no_border'),
     version: Annotated[
         bool,
         typer.Option(
@@ -126,14 +142,14 @@ def main(
         typer.Option(
             '--debug',
             '-d',
-            envvar='OBSWS_CLI_DEBUG',
+            envvar='OBS_DEBUG',
             is_eager=True,
             help='Enable debug logging',
             show_default=False,
             callback=setup_logging,
             hidden=True,
         ),
-    ] = envconfig.get('debug'),
+    ] = config.get('debug'),
 ):
     """obsws_cli is a command line interface for the OBS WebSocket API."""
     ctx.ensure_object(dict)

obsws_cli/config.py

@@ -0,0 +1,80 @@
+"""module for settings management for obsws-cli."""
+
+from collections import UserDict
+from pathlib import Path
+
+from dotenv import dotenv_values
+
+ConfigValue = str | int
+
+
+class Config(UserDict):
+    """A class to manage config for obsws-cli.
+
+    This class extends UserDict to provide a dictionary-like interface for config.
+    It loads config from environment variables and .env files.
+    The config values are expected to be in uppercase and should start with 'OBS_'.
+
+    Example:
+    -------
+        config = Config()
+        host = config['OBS_HOST']
+        config['OBS_PORT'] = 4455
+
+    """
+
+    PREFIX = 'OBS_'
+
+    def __init__(self, *args, **kwargs):
+        """Initialize the Settings object."""
+        kwargs.update(
+            {
+                **dotenv_values('.env'),
+                **dotenv_values(Path.home() / '.config' / 'obsws-cli' / 'obsws.env'),
+            }
+        )
+        super().__init__(*args, **kwargs)
+
+    def __getitem__(self, key: str) -> ConfigValue:
+        """Get a setting value by key."""
+        key = key.upper()
+        if not key.startswith(Config.PREFIX):
+            key = f'{Config.PREFIX}{key}'
+        return self.data[key]
+
+    def __setitem__(self, key: str, value: ConfigValue):
+        """Set a setting value by key."""
+        key = key.upper()
+        if not key.startswith(Config.PREFIX):
+            key = f'{Config.PREFIX}{key}'
+        self.data[key] = value
+
+
+_config = Config(
+    OBS_HOST='localhost',
+    OBS_PORT=4455,
+    OBS_PASSWORD='',
+    OBS_TIMEOUT=5,
+    OBS_DEBUG=False,
+    OBS_STYLE='disabled',
+    OBS_STYLE_NO_BORDER=False,
+)
+
+
+def get(key: str) -> ConfigValue:
+    """Get a setting value by key.
+
+    Args:
+    ----
+        key (str): The key of the config to retrieve.
+
+    Returns:
+    -------
+        The value of the config.
+
+    Raises:
+    ------
+        KeyError: If the key does not exist in the config.
+
+    """
+    return _config[key]

obsws_cli/envconfig.py

@@ -1,146 +0,0 @@
-"""module for settings management for obsws-cli."""
-
-from collections import UserDict
-from pathlib import Path
-from typing import Any, Union
-
-from dotenv import dotenv_values
-
-ConfigValue = Union[str, int, bool]
-
-
-class EnvConfig(UserDict):
-    """A class to manage .env config for obsws-cli.
-
-    This class extends UserDict to provide a dictionary-like interface for config.
-    It loads config from .env files in the following priority:
-    1. Local .env file
-    2. User config file (~/.config/obsws-cli/obsws.env)
-    3. Default values
-
-    Note, typer handles reading from environment variables automatically. They take precedence
-    over values set in this class.
-
-    The config values are expected to be in uppercase and should start with 'OBSWS_CLI_'.
-
-    Example:
-    -------
-        config = EnvConfig()
-        host = config['OBSWS_CLI_HOST']
-        config['OBSWS_CLI_PORT'] = 4455
-        # Or with defaults
-        timeout = config.get('OBSWS_CLI_TIMEOUT', 30)
-        # Keys will be normalised to uppercase with prefix
-        debug = config.get('debug', False)  # Equivalent to 'OBSWS_CLI_DEBUG'
-
-    """
-
-    PREFIX = 'OBSWS_CLI_'
-
-    def __init__(self, *args, **kwargs):
-        """Initialize the Config object with hierarchical loading."""
-        kwargs.update(
-            {
-                **dotenv_values(Path.home() / '.config' / 'obsws-cli' / 'obsws.env'),
-                **dotenv_values('.env'),
-            }
-        )
-
-        super().__init__(*args, **self._convert_types(kwargs))
-
-    def _convert_types(self, config_data: dict[str, Any]) -> dict[str, ConfigValue]:
-        """Convert string values to appropriate types."""
-        converted = {}
-        for key, value in config_data.items():
-            if isinstance(value, str):
-                if value.lower() in ('true', 'false'):
-                    converted[key] = value.lower() == 'true'
-                elif value.isdigit():
-                    converted[key] = int(value)
-                else:
-                    converted[key] = value
-            else:
-                converted[key] = value
-        return converted
-
-    def __getitem__(self, key: str) -> ConfigValue:
-        """Get a setting value by key."""
-        normalised_key = self._normalise_key(key)
-        try:
-            return self.data[normalised_key]
-        except KeyError as e:
-            raise KeyError(
-                f"Config key '{key}' not found. Available keys: {list(self.data.keys())}"
-            ) from e
-
-    def __setitem__(self, key: str, value: ConfigValue):
-        """Set a setting value by key."""
-        normalised_key = self._normalise_key(key)
-        self.data[normalised_key] = value
-
-    def _normalise_key(self, key: str) -> str:
-        """Normalise a key to uppercase with OBS_ prefix."""
-        key = key.upper()
-        if not key.startswith(self.PREFIX):
-            key = f'{self.PREFIX}{key}'
-        return key
-
-    def get(self, key: str, default=None) -> ConfigValue:
-        """Get a config value with optional default.
-
-        Args:
-        ----
-            key (str): The key to retrieve
-            default: Default value if key is not found
-
-        Returns:
-        -------
-            The config value or default
-
-        """
-        normalised_key = self._normalise_key(key)
-        if not self.has_key(normalised_key):
-            return default
-        return self[normalised_key]
-
-    def has_key(self, key: str) -> bool:
-        """Check if a config key exists.
-
-        Args:
-        ----
-            key (str): The key to check
-
-        Returns:
-        -------
-            bool: True if key exists, False otherwise
-
-        """
-        normalised_key = self._normalise_key(key)
-        return normalised_key in self.data
-
-
-_envconfig = EnvConfig(
-    OBSWS_CLI_HOST='localhost',
-    OBSWS_CLI_PORT=4455,
-    OBSWS_CLI_PASSWORD='',
-    OBSWS_CLI_TIMEOUT=5,
-    OBSWS_CLI_DEBUG=False,
-    OBSWS_CLI_STYLE='disabled',
-    OBSWS_CLI_STYLE_NO_BORDER=False,
-)
-
-
-def get(key: str) -> ConfigValue:
-    """Get a setting value by key from the global config instance.
-
-    Args:
-    ----
-        key (str): The key of the config to retrieve.
-        default: Default value if key is not found.
-
-    Returns:
-    -------
-        The value of the config or default value.
-
-    """
-    return _envconfig.get(key)

obsws_cli/filter.py

@@ -7,8 +7,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util
+from . import console, util
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/group.py

@@ -6,9 +6,9 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util, validate
+from . import console, util, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
-from obsws_cli.protocols import DataclassProtocol
+from .protocols import DataclassProtocol
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -26,14 +26,17 @@ def list_(
         typer.Argument(
             show_default='The current scene',
             help='Scene name to list groups for',
-            callback=validate.scene_in_scenes,
         ),
     ] = None,
 ):
     """List groups in a scene."""
-    if scene_name is None:
+    if not scene_name:
         scene_name = ctx.obj['obsws'].get_current_program_scene().scene_name
 
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f"Scene '{scene_name}' not found.")
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     groups = [
         (item.get('sceneItemId'), item.get('sourceName'), item.get('sceneItemEnabled'))
@@ -89,18 +92,17 @@ def show(
     ctx: typer.Context,
     scene_name: Annotated[
         str,
-        typer.Argument(
+        typer.Argument(..., show_default=False, help='Scene name the group is in'),
-            ...,
-            show_default=False,
-            help='Scene name the group is in',
-            callback=validate.scene_in_scenes,
-        ),
     ],
     group_name: Annotated[
         str, typer.Argument(..., show_default=False, help='Group name to show')
     ],
 ):
     """Show a group in a scene."""
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f"Scene '{scene_name}' not found.")
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     if (group := _get_group(group_name, resp)) is None:
         console.err.print(
@@ -121,19 +123,17 @@ def show(
 def hide(
     ctx: typer.Context,
     scene_name: Annotated[
-        str,
+        str, typer.Argument(..., show_default=False, help='Scene name the group is in')
-        typer.Argument(
-            ...,
-            show_default=False,
-            help='Scene name the group is in',
-            callback=validate.scene_in_scenes,
-        ),
     ],
     group_name: Annotated[
         str, typer.Argument(..., show_default=False, help='Group name to hide')
     ],
 ):
     """Hide a group in a scene."""
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     if (group := _get_group(group_name, resp)) is None:
         console.err.print(
@@ -154,19 +154,17 @@ def hide(
 def toggle(
     ctx: typer.Context,
     scene_name: Annotated[
-        str,
+        str, typer.Argument(..., show_default=False, help='Scene name the group is in')
-        typer.Argument(
-            ...,
-            show_default=False,
-            help='Scene name the group is in',
-            callback=validate.scene_in_scenes,
-        ),
     ],
     group_name: Annotated[
         str, typer.Argument(..., show_default=False, help='Group name to toggle')
     ],
 ):
     """Toggle a group in a scene."""
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     if (group := _get_group(group_name, resp)) is None:
         console.err.print(
@@ -191,19 +189,17 @@ def toggle(
 def status(
     ctx: typer.Context,
     scene_name: Annotated[
-        str,
+        str, typer.Argument(..., show_default=False, help='Scene name the group is in')
-        typer.Argument(
-            ...,
-            show_default=False,
-            help='Scene name the group is in',
-            callback=validate.scene_in_scenes,
-        ),
     ],
     group_name: Annotated[
         str, typer.Argument(..., show_default=False, help='Group name to check status')
     ],
 ):
     """Get the status of a group in a scene."""
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     if (group := _get_group(group_name, resp)) is None:
         console.err.print(

obsws_cli/hotkey.py

@@ -6,8 +6,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/input.py

@@ -7,8 +7,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util, validate
+from . import console, util, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/media.py

@@ -4,8 +4,8 @@ from typing import Annotated, Optional
 
 import typer
 
-from obsws_cli import console, util, validate
+from . import console, util
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -24,9 +24,7 @@ def cursor(
     timecode: Annotated[
         Optional[str],
         typer.Argument(
-            ...,
+            ..., help='The timecode to set the cursor to (format: HH:MM:SS).'
-            help='The timecode to set the cursor to (format: HH:MM:SS).',
-            callback=validate.timecode_format,
         ),
     ] = None,
 ):

obsws_cli/profile.py

@@ -6,8 +6,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util, validate
+from . import console, util, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -62,14 +62,15 @@ def switch(
     profile_name: Annotated[
         str,
         typer.Argument(
-            ...,
+            ..., show_default=False, help='Name of the profile to switch to'
-            show_default=False,
-            help='Name of the profile to switch to',
-            callback=validate.profile_exists,
         ),
     ],
 ):
     """Switch to a profile."""
+    if not validate.profile_exists(ctx, profile_name):
+        console.err.print(f'Profile [yellow]{profile_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_profile_list()
     if resp.current_profile_name == profile_name:
         console.err.print(
@@ -86,15 +87,14 @@ def create(
     ctx: typer.Context,
     profile_name: Annotated[
         str,
-        typer.Argument(
+        typer.Argument(..., show_default=False, help='Name of the profile to create.'),
-            ...,
-            show_default=False,
-            help='Name of the profile to create.',
-            callback=validate.profile_not_exists,
-        ),
     ],
 ):
     """Create a new profile."""
+    if validate.profile_exists(ctx, profile_name):
+        console.err.print(f'Profile [yellow]{profile_name}[/yellow] already exists.')
+        raise typer.Exit(1)
+
     ctx.obj['obsws'].create_profile(profile_name)
     console.out.print(f'Created profile {console.highlight(ctx, profile_name)}.')
 
@@ -104,14 +104,13 @@ def remove(
     ctx: typer.Context,
     profile_name: Annotated[
         str,
-        typer.Argument(
+        typer.Argument(..., show_default=False, help='Name of the profile to remove.'),
-            ...,
-            show_default=False,
-            help='Name of the profile to remove.',
-            callback=validate.profile_exists,
-        ),
     ],
 ):
     """Remove a profile."""
+    if not validate.profile_exists(ctx, profile_name):
+        console.err.print(f'Profile [yellow]{profile_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     ctx.obj['obsws'].remove_profile(profile_name)
     console.out.print(f'Removed profile {console.highlight(ctx, profile_name)}.')

obsws_cli/projector.py

@@ -6,8 +6,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/record.py

@@ -5,8 +5,8 @@ from typing import Annotated, Optional
 
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/replaybuffer.py

@@ -2,8 +2,8 @@
 
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/scene.py

@@ -6,8 +6,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util, validate
+from . import console, util, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -70,14 +70,14 @@ def list_(
 def current(
     ctx: typer.Context,
     preview: Annotated[
-        bool,
+        bool, typer.Option(help='Get the preview scene instead of the program scene')
-        typer.Option(
-            help='Get the preview scene instead of the program scene',
-            callback=validate.studio_mode_enabled,
-        ),
     ] = False,
 ):
     """Get the current program scene or preview scene."""
+    if preview and not validate.studio_mode_enabled(ctx):
+        console.err.print('Studio mode is not enabled, cannot get preview scene.')
+        raise typer.Exit(1)
+
     if preview:
         resp = ctx.obj['obsws'].get_current_preview_scene()
         console.out.print(
@@ -94,22 +94,22 @@ def current(
 def switch(
     ctx: typer.Context,
     scene_name: Annotated[
-        str,
+        str, typer.Argument(..., help='Name of the scene to switch to')
-        typer.Argument(
-            ...,
-            help='Name of the scene to switch to',
-            callback=validate.scene_in_scenes,
-        ),
     ],
     preview: Annotated[
         bool,
-        typer.Option(
+        typer.Option(help='Switch to the preview scene instead of the program scene'),
-            help='Switch to the preview scene instead of the program scene',
-            callback=validate.studio_mode_enabled,
-        ),
     ] = False,
 ):
     """Switch to a scene."""
+    if preview and not validate.studio_mode_enabled(ctx):
+        console.err.print('Studio mode is not enabled, cannot set the preview scene.')
+        raise typer.Exit(1)
+
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     if preview:
         ctx.obj['obsws'].set_current_preview_scene(scene_name)
         console.out.print(

obsws_cli/scenecollection.py

@@ -5,8 +5,8 @@ from typing import Annotated
 import typer
 from rich.table import Table
 
-from obsws_cli import console, validate
+from . import console, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -53,15 +53,16 @@ def current(ctx: typer.Context):
 def switch(
     ctx: typer.Context,
     scene_collection_name: Annotated[
-        str,
+        str, typer.Argument(..., help='Name of the scene collection to switch to')
-        typer.Argument(
-            ...,
-            help='Name of the scene collection to switch to',
-            callback=validate.scene_collection_in_scene_collections,
-        ),
     ],
 ):
     """Switch to a scene collection."""
+    if not validate.scene_collection_in_scene_collections(ctx, scene_collection_name):
+        console.err.print(
+            f'Scene collection [yellow]{scene_collection_name}[/yellow] not found.'
+        )
+        raise typer.Exit(1)
+
     current_scene_collection = (
         ctx.obj['obsws'].get_scene_collection_list().current_scene_collection_name
     )
@@ -81,15 +82,16 @@ def switch(
 def create(
     ctx: typer.Context,
     scene_collection_name: Annotated[
-        str,
+        str, typer.Argument(..., help='Name of the scene collection to create')
-        typer.Argument(
-            ...,
-            help='Name of the scene collection to create',
-            callback=validate.scene_collection_not_in_scene_collections,
-        ),
     ],
 ):
     """Create a new scene collection."""
+    if validate.scene_collection_in_scene_collections(ctx, scene_collection_name):
+        console.err.print(
+            f'Scene collection [yellow]{scene_collection_name}[/yellow] already exists.'
+        )
+        raise typer.Exit(1)
+
     ctx.obj['obsws'].create_scene_collection(scene_collection_name)
     console.out.print(
         f'Created scene collection {console.highlight(ctx, scene_collection_name)}.'

obsws_cli/sceneitem.py

@@ -5,8 +5,8 @@ from typing import Annotated, Optional
 import typer
 from rich.table import Table
 
-from obsws_cli import console, util, validate
+from . import console, util, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 
@@ -24,15 +24,18 @@ def list_(
         typer.Argument(
             show_default='The current scene',
             help='Scene name to list items for',
-            callback=validate.scene_in_scenes,
         ),
     ] = None,
     uuid: Annotated[bool, typer.Option(help='Show UUIDs of scene items')] = False,
 ):
     """List all items in a scene."""
-    if scene_name is None:
+    if not scene_name:
         scene_name = ctx.obj['obsws'].get_current_program_scene().scene_name
 
+    if not validate.scene_in_scenes(ctx, scene_name):
+        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
+        raise typer.Exit(1)
+
     resp = ctx.obj['obsws'].get_scene_item_list(scene_name)
     items = sorted(
         (

obsws_cli/screenshot.py

@@ -6,8 +6,8 @@ from typing import Annotated
 import obsws_python as obsws
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/settings.py

@@ -6,8 +6,8 @@ import typer
 from rich.table import Table
 from rich.text import Text
 
-from obsws_cli import console, util
+from . import console, util
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/stream.py

@@ -2,8 +2,8 @@
 
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/studiomode.py

@@ -2,8 +2,8 @@
 
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/text.py

@@ -4,8 +4,8 @@ from typing import Annotated, Optional
 
 import typer
 
-from obsws_cli import console, validate
+from . import console, validate
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

obsws_cli/validate.py

@@ -1,7 +1,5 @@
 """module containing validation functions."""
 
-from typing import Optional
-
 import typer
 
 from . import console
@@ -10,7 +8,7 @@ from . import console
 skipped_option = typer.Option(parser=lambda _: _, hidden=True, expose_value=False)
 
 
-def input_in_inputs(ctx: typer.Context, input_name: str) -> str:
+def input_in_inputs(ctx: typer.Context, input_name: str) -> bool:
     """Ensure the given input exists in the list of inputs."""
     resp = ctx.obj['obsws'].get_input_list()
     if not any(input.get('inputName') == input_name for input in resp.inputs):
@@ -19,7 +17,7 @@ def input_in_inputs(ctx: typer.Context, input_name: str) -> str:
     return input_name
 
 
-def input_not_in_inputs(ctx: typer.Context, input_name: str) -> str:
+def input_not_in_inputs(ctx: typer.Context, input_name: str) -> bool:
     """Ensure an input does not already exist in the list of inputs."""
     resp = ctx.obj['obsws'].get_input_list()
     if any(input.get('inputName') == input_name for input in resp.inputs):
@@ -28,57 +26,26 @@ def input_not_in_inputs(ctx: typer.Context, input_name: str) -> str:
     return input_name
 
 
-def scene_in_scenes(ctx: typer.Context, scene_name: Optional[str]) -> str | None:
+def scene_in_scenes(ctx: typer.Context, scene_name: str) -> bool:
     """Check if a scene exists in the list of scenes."""
-    if scene_name is None:
-        return
-
     resp = ctx.obj['obsws'].get_scene_list()
-    if not any(scene.get('sceneName') == scene_name for scene in resp.scenes):
+    return any(scene.get('sceneName') == scene_name for scene in resp.scenes)
-        console.err.print(f'Scene [yellow]{scene_name}[/yellow] not found.')
-        raise typer.Exit(1)
-    return scene_name
 
 
-def studio_mode_enabled(ctx: typer.Context, preview: bool) -> bool:
+def studio_mode_enabled(ctx: typer.Context) -> bool:
-    """Ensure studio mode is enabled if preview option is used."""
+    """Check if studio mode is enabled."""
     resp = ctx.obj['obsws'].get_studio_mode_enabled()
-    if preview and not resp.studio_mode_enabled:
+    return resp.studio_mode_enabled
-        console.err.print(
-            'Studio mode is disabled. This action requires it to be enabled.'
-        )
-        raise typer.Exit(1)
-    return preview
 
 
 def scene_collection_in_scene_collections(
     ctx: typer.Context, scene_collection_name: str
-) -> str:
+) -> bool:
-    """Ensure a scene collection exists in the list of scene collections."""
+    """Check if a scene collection exists."""
     resp = ctx.obj['obsws'].get_scene_collection_list()
-    if not any(
+    return any(
         collection == scene_collection_name for collection in resp.scene_collections
-    ):
+    )
-        console.err.print(
-            f'Scene collection [yellow]{scene_collection_name}[/yellow] not found.'
-        )
-        raise typer.Exit(1)
-    return scene_collection_name
-
-
-def scene_collection_not_in_scene_collections(
-    ctx: typer.Context, scene_collection_name: str
-) -> str:
-    """Ensure a scene collection does not already exist in the list of scene collections."""
-    resp = ctx.obj['obsws'].get_scene_collection_list()
-    if any(
-        collection == scene_collection_name for collection in resp.scene_collections
-    ):
-        console.err.print(
-            f'Scene collection [yellow]{scene_collection_name}[/yellow] already exists.'
-        )
-        raise typer.Exit(1)
-    return scene_collection_name
 
 
 def item_in_scene_item_list(
@@ -89,22 +56,16 @@ def item_in_scene_item_list(
     return any(item.get('sourceName') == item_name for item in resp.scene_items)
 
 
-def profile_exists(ctx: typer.Context, profile_name: str) -> str:
+def profile_exists(ctx: typer.Context, profile_name: str) -> bool:
-    """Ensure a profile exists."""
+    """Check if a profile exists."""
     resp = ctx.obj['obsws'].get_profile_list()
-    if not any(profile == profile_name for profile in resp.profiles):
+    return any(profile == profile_name for profile in resp.profiles)
-        console.err.print(f'Profile [yellow]{profile_name}[/yellow] not found.')
-        raise typer.Exit(1)
-    return profile_name
 
 
-def profile_not_exists(ctx: typer.Context, profile_name: str) -> str:
+def monitor_exists(ctx: typer.Context, monitor_index: int) -> bool:
-    """Ensure a profile does not exist."""
+    """Check if a monitor exists."""
-    resp = ctx.obj['obsws'].get_profile_list()
+    resp = ctx.obj['obsws'].get_monitor_list()
-    if any(profile == profile_name for profile in resp.profiles):
+    return any(monitor['monitorIndex'] == monitor_index for monitor in resp.monitors)
-        console.err.print(f'Profile [yellow]{profile_name}[/yellow] already exists.')
-        raise typer.Exit(1)
-    return profile_name
 
 
 def kind_in_input_kinds(ctx: typer.Context, input_kind: str) -> str:
@@ -114,29 +75,3 @@ def kind_in_input_kinds(ctx: typer.Context, input_kind: str) -> str:
         console.err.print(f'Input kind [yellow]{input_kind}[/yellow] not found.')
         raise typer.Exit(1)
     return input_kind
-
-
-def timecode_format(ctx: typer.Context, timecode: Optional[str]) -> str | None:
-    """Validate that a timecode is in HH:MM:SS or MM:SS format."""
-    if timecode is None:
-        return
-
-    match timecode.split(':'):
-        case [mm, ss]:
-            if not (mm.isdigit() and ss.isdigit()):
-                console.err.print(
-                    f'Timecode [yellow]{timecode}[/yellow] is not valid. Use MM:SS or HH:MM:SS format.'
-                )
-                raise typer.Exit(1)
-        case [hh, mm, ss]:
-            if not (hh.isdigit() and mm.isdigit() and ss.isdigit()):
-                console.err.print(
-                    f'Timecode [yellow]{timecode}[/yellow] is not valid. Use MM:SS or HH:MM:SS format.'
-                )
-                raise typer.Exit(1)
-        case _:
-            console.err.print(
-                f'Timecode [yellow]{timecode}[/yellow] is not valid. Use MM:SS or HH:MM:SS format.'
-            )
-            raise typer.Exit(1)
-    return timecode

obsws_cli/virtualcam.py

@@ -2,8 +2,8 @@
 
 import typer
 
-from obsws_cli import console
+from . import console
-from obsws_cli.alias import SubTyperAliasGroup
+from .alias import SubTyperAliasGroup
 
 app = typer.Typer(cls=SubTyperAliasGroup)
 

tests/conftest.py

@@ -21,9 +21,9 @@ def pytest_sessionstart(session):
     """
     # Initialize the OBS WebSocket client
     session.obsws = obsws.ReqClient(
-        host=os.environ['OBSWS_CLI_HOST'],
+        host=os.environ['OBS_HOST'],
-        port=os.environ['OBSWS_CLI_PORT'],
+        port=os.environ['OBS_PORT'],
-        password=os.environ['OBSWS_CLI_PASSWORD'],
+        password=os.environ['OBS_PASSWORD'],
         timeout=5,
     )
     resp = session.obsws.get_version()