socx-cli 0.13.3__tar.gz → 0.13.5__tar.gz

{socx_cli-0.13.3 → socx_cli-0.13.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: socx-cli
-Version: 0.13.3
+Version: 0.13.5
 Summary: System on chip verification and tooling infrastructure.
 Project-URL: Issues, https://github.com/sagikimhi/socx-cli/issues
 Project-URL: Homepage, https://sagikimhi.dev/socx-cli
@@ -31,6 +31,7 @@ Requires-Python: >=3.12
 Requires-Dist: anyio>=4.12.1
 Requires-Dist: click
 Requires-Dist: copier~=9.11
+Requires-Dist: declare
 Requires-Dist: dynaconf<3.2.12
 Requires-Dist: gitpython
 Requires-Dist: hoptex

{socx_cli-0.13.3 → socx_cli-0.13.5}/pyproject.toml

@@ -29,7 +29,7 @@ socx = 'socx.__main__:main'
 [project]
 name = "socx-cli"
 readme = "README.md"
-version = "0.13.3"
+version = "0.13.5"
 license = "Apache-2.0"
 authors = [{ name = "Sagi Kimhi", email = "sagi.kim5@gmail.com" }]
 maintainers = [{ name = "Sagi Kimhi", email = "sagi.kim5@gmail.com" }]
@@ -59,6 +59,7 @@ dependencies = [
     "pip",
     "rich",
     "click",
+    "declare",
     "typer~=0.19",
     "jinja2",
     "psutil",

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/__init__.py

@@ -47,7 +47,6 @@ __all__ = (
     "DEFAULT_HANDLERS",
     "DEFAULT_TIME_FORMAT",
     "Level",
-    "log",
     "log_it",
     "logger",
     "console",
@@ -135,7 +134,6 @@ __all__ = (
     "Regression",
     "TestStatus",
     "TestResult",
-    "RegressionTree",
     "RegressionProgress",
 )
 
@@ -180,11 +178,6 @@ from socx.core import deduplicate as deduplicate
 
 from socx.utils import join_decorators as join_decorators
 
-from socx.io import log as log
-from socx.io import DEFAULT_LEVEL as DEFAULT_LEVEL
-from socx.io import DEFAULT_FORMAT as DEFAULT_FORMAT
-from socx.io import DEFAULT_HANDLERS as DEFAULT_HANDLERS
-from socx.io import DEFAULT_TIME_FORMAT as DEFAULT_TIME_FORMAT
 from socx.io import Level as Level
 from socx.io import log_it as log_it
 from socx.io import logger as logger
@@ -264,7 +257,6 @@ from socx.regression import TestBase as TestBase
 from socx.regression import Regression as Regression
 from socx.regression import TestStatus as TestStatus
 from socx.regression import TestResult as TestResult
-from socx.regression import RegressionTree as RegressionTree
 from socx.regression import RegressionProgress as RegressionProgress
 
 from socx.cli import FuncType as FuncType

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/cli/callbacks.py

@@ -12,7 +12,7 @@ from rich_click import Context
 from rich_click import Parameter
 from rich_click import RichContext
 
-import socx.io.log as log
+from socx.io.log import Level, set_level, get_level, _get_logger
 from socx.io.console import console
 from socx.io.decorators import log_it
 from socx.config._config import settings
@@ -45,10 +45,11 @@ def cwd_cb(ctx: Context, param: Parameter, value: Path) -> Path:
 @log_it(logger=logger)
 def debug_cb(_: Context, param: Parameter, value: bool) -> bool:
     """Enable debug logging and persist the CLI switch to settings."""
+    socx_logger = _get_logger()
     settings.cli.params[param.name] = value
     if value:
-        log.set_level(log.Level.DEBUG)
-        settings.cli.params["verbosity"] = log.get_level().name
+        set_level(Level.DEBUG, socx_logger)
+        settings.cli.params["verbosity"] = get_level().name
     return value
 
 
@@ -88,10 +89,11 @@ def configure_cb(ctx: Context, param: Parameter, value: str) -> str:
 @log_it(logger=logger)
 def verbosity_cb(_: Context, param: Parameter, value: str) -> str:
     """Update the global log level while respecting existing overrides."""
-    level = log.Level[value.upper()]
+    socx_logger = _get_logger()
+    level = Level[value.upper()]
     if not settings.cli.params.debug:
-        log.set_level(level)
-    settings.cli.params[param.name] = log.get_level().name
+        set_level(level, socx_logger)
+    settings.cli.params[param.name] = get_level().name
     return settings.cli.params[param.name]
 
 

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/core/_paths.py

@@ -109,8 +109,11 @@ USER_LOG_DIR: Path = user_log_path(
 ).resolve()
 """Absolute path to platform's native application logs directory."""
 
-USER_LOG_FILENAME: str = f"{__appname__}.log"
-"""File name of application's native log file used for debug and tracing."""
+USER_LOG_FILENAME: str = f"{__appname__}_log.log"
+"""File name of application's log file."""
+
+USER_ROTATING_LOG_FILENAME: str = f"rotating_{__appname__}_log.log"
+"""File name of application's rotating log file."""
 
 USER_CONFIG_FILENAME: str = f"{__appname__}.yaml"
 """File name searched in user's config directory to load user configs."""
@@ -118,6 +121,9 @@ USER_CONFIG_FILENAME: str = f"{__appname__}.yaml"
 USER_LOG_FILE: Path = USER_LOG_DIR / USER_LOG_FILENAME
 """Absolute path to application's main log for the current local user."""
 
+USER_ROTATING_LOG_FILE: Path = USER_LOG_DIR / USER_ROTATING_LOG_FILENAME
+"""Absolute path to application's rotating log for the current local user."""
+
 USER_CONFIG_FILE: Path = USER_CONFIG_DIR / USER_CONFIG_FILENAME
 """Absolute path to application's user config file."""
 

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/core/paths.py

@@ -19,8 +19,10 @@ __all__ = (
     "USER_RUNTIME_DIR",
     "USER_LOG_FILENAME",
     "USER_CONFIG_FILENAME",
+    "USER_ROTATING_LOG_FILENAME",
     "USER_LOG_FILE",
     "USER_CONFIG_FILE",
+    "USER_ROTATING_LOG_FILE",
     # Local Paths
     "LOCAL_CONFIG_FILENAME",
     "LOCAL_CONFIG_FILE",
@@ -46,8 +48,12 @@ from socx.core._paths import USER_CONFIG_DIR as USER_CONFIG_DIR
 from socx.core._paths import USER_RUNTIME_DIR as USER_RUNTIME_DIR
 from socx.core._paths import USER_LOG_FILENAME as USER_LOG_FILENAME
 from socx.core._paths import USER_CONFIG_FILENAME as USER_CONFIG_FILENAME
+from socx.core._paths import (
+    USER_ROTATING_LOG_FILENAME as USER_ROTATING_LOG_FILENAME,
+)
 from socx.core._paths import USER_LOG_FILE as USER_LOG_FILE
 from socx.core._paths import USER_CONFIG_FILE as USER_CONFIG_FILE
+from socx.core._paths import USER_ROTATING_LOG_FILE as USER_ROTATING_LOG_FILE
 
 # Local Paths
 from socx.core._paths import LOCAL_CONFIG_FILE as LOCAL_CONFIG_FILE

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/io/__init__.py

@@ -1,8 +1,6 @@
 """Expose logging, console, and decorator utilities for I/O facilities."""
 
 __all__ = (
-    # Modules
-    "log",
     # Constants
     "DEFAULT_LEVEL",
     "DEFAULT_FORMAT",
@@ -32,12 +30,6 @@ __all__ = (
     "print_command_outputs",
 )
 
-from socx.io import log as log
-
-from socx.io.log import DEFAULT_LEVEL as DEFAULT_LEVEL
-from socx.io.log import DEFAULT_FORMAT as DEFAULT_FORMAT
-from socx.io.log import DEFAULT_HANDLERS as DEFAULT_HANDLERS
-from socx.io.log import DEFAULT_TIME_FORMAT as DEFAULT_TIME_FORMAT
 from socx.io.log import Level as Level
 from socx.io.log import logger as logger
 from socx.io.log import get_level as get_level

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/io/log.py

@@ -1,19 +1,21 @@
 """Logging helpers that standardise Rich-powered output across SoCX."""
 
 from __future__ import annotations
+from types import ModuleType
 
-import os
 import enum
 import logging
 import logging.handlers
-from typing import Any
+from typing import Any, IO
 from pathlib import Path
 from collections import ChainMap
 from collections.abc import Iterable
 
 from rich.console import Console
 from rich.logging import RichHandler
-from platformdirs import user_log_path
+
+from socx.config._config import settings
+from socx.core.metadata import __appname__
 
 __all__ = (
     # Logging
@@ -39,10 +41,7 @@ __all__ = (
     # Types
     "Level",
     # Defaults
-    "DEFAULT_LEVEL",
-    "DEFAULT_FORMAT",
     "DEFAULT_HANDLERS",
-    "DEFAULT_TIME_FORMAT",
 )
 
 
@@ -59,97 +58,194 @@ class Level(enum.IntEnum):
     CRITICAL = logging.CRITICAL
 
 
-def _get_console_handler(level: Level = Level.INFO) -> logging.Handler:
+def _get_console(
+    file: IO | None = None,
+    stderr: bool = False,
+    markup: bool = True,
+    tab_size: int = 4,
+    force_terminal: bool = True,
+    **kwargs: Any,
+) -> Console:
+    defaults = dict(
+        file=file,
+        markup=markup,
+        stderr=stderr,
+        tab_size=tab_size,
+        force_terminal=force_terminal,
+    )
+    kwargs = dict(ChainMap(kwargs, defaults))
+    return Console(**kwargs)
+
+
+def _get_level(level: str | int | Level) -> Level:
+    if isinstance(level, str):
+        level = Level[level]
+    elif isinstance(level, int):
+        level = Level(level)
+    return level
+
+
+def _get_console_handler(
+    file: IO | None = None,
+    level: int | str | Level = Level.INFO,
+    stderr: bool = False,
+    tab_size: int = 4,
+    tracebacks: bool = True,
+    force_terminal: bool = True,
+    tracebacks_theme: str | None = None,
+    tracebacks_suppress: Iterable[ModuleType] | None = None,
+    tracebacks_show_locals: bool = True,
+) -> logging.Handler:
     """Create a Rich console handler configured for interactive output."""
-    console = Console(tab_size=4, markup=True, force_terminal=True)
-    return RichHandler(
-        level=level,
+    import click
+    import rich_click
+    from socx.io.console import console as _console
+
+    level = _get_level(level)
+    tracebacks_theme = tracebacks_theme or "ansi_dark"
+    tracebacks_suppress = tracebacks_suppress or [click, rich_click]
+    if file is None and stderr is None:
+        console = _console
+    else:
+        console = _get_console(
+            file=file,
+            stderr=stderr,
+            tab_size=tab_size,
+            force_terminal=force_terminal,
+        )
+    handler = RichHandler(
         console=console,
-        tracebacks_suppress=("rich-click", "click"),
-        rich_tracebacks=True,
-        tracebacks_show_locals=True,
+        rich_tracebacks=tracebacks,
+        tracebacks_theme=tracebacks_theme,
+        tracebacks_suppress=tracebacks_suppress,
+        tracebacks_show_locals=tracebacks_show_locals,
     )
+    formatter = logging.Formatter(**settings.logging.formatters.default)
+    handler.name = "console"
+    handler.setLevel(level)
+    handler.setFormatter(formatter)
+    return handler
 
 
 def _get_file_handler(
-    path: str | Path, level: Level = Level.NOTSET
+    path: str | Path,
+    mode: str | None = None,
+    level: Level = Level.INFO,
+    stderr: bool = False,
+    tab_size: int = 4,
+    tracebacks: bool = True,
+    force_terminal: bool = False,
+    tracebacks_theme: str | None = None,
+    tracebacks_suppress: Iterable[ModuleType] | None = None,
+    tracebacks_show_locals: bool = True,
+) -> logging.Handler:
+    import atexit
+
+    def close_if_open(file: IO) -> None:
+        if not file.closed:
+            file.close()
+
+    mode = mode or "a"
+    file = open(path, mode=mode)  # noqa: SIM115
+    atexit.register(close_if_open, file)
+    handler = _get_console_handler(
+        file=file,
+        level=level,
+        stderr=stderr,
+        tab_size=tab_size,
+        tracebacks=tracebacks,
+        force_terminal=force_terminal,
+        tracebacks_theme=tracebacks_theme,
+        tracebacks_suppress=tracebacks_suppress,
+        tracebacks_show_locals=tracebacks_show_locals,
+    )
+    handler.name = "file"
+    handler.setFormatter(DEFAULT_CHILD_FORMATTER)
+    return handler
+
+
+def _get_rotating_file_handler(
+    path: str | Path,
+    level: Level = Level.DEBUG,
+    stderr: bool = False,
+    mode: str | None = None,
 ) -> logging.Handler:
     """Create a Rich handler that writes log output to ``path``."""
+
+    def MBs(n: int) -> int:  # noqa: N802
+        return 1024 * 1024 * n
+
+    mode = mode or "w"
     handler = logging.handlers.RotatingFileHandler(
-        path, maxBytes=1024 * 1024 * 5, backupCount=5
+        path,
+        mode=mode,
+        maxBytes=MBs(10),
+        backupCount=5,
     )
+    handler.name = "rotating_file"
     handler.setLevel(level)
-    handler.setFormatter(
-        logging.Formatter(DEFAULT_CHILD_FORMAT, DEFAULT_TIME_FORMAT)
-    )
+    handler.setFormatter(DEFAULT_CHILD_FORMATTER)
     return handler
 
 
-APP_LIB_NAME = __name__.partition(".")[0]
-"""Application library namespace used for loggers."""
+def _get_handler(handler: str) -> logging.Handler | None:
+    match handler:
+        case "file":
+            return _get_file_handler(**settings.logging.handlers.file)
+        case "console":
+            return _get_console_handler(**settings.logging.handlers.console)
+        case "rotating_file":
+            return _get_rotating_file_handler(
+                **settings.logging.handlers.rotating_file
+            )
+        case _:
+            return None
+
+
+def _get_logger() -> logging.Logger:
+    """Initialise and return the module-level root logger."""
+    socx_logger = logging.getLogger(__appname__)
+
+    if not socx_logger.hasHandlers():
+        unknown_handlers = []
 
-DEFAULT_ENCODING: str = "utf-8"
-"""Default text encoding for emitted log files."""
+        for handler_name in settings.logging.handlers:
+            handler = _get_handler(handler_name)
+            if handler is not None:
+                socx_logger.addHandler(handler)
+            else:
+                unknown_handlers.append(handler_name)
 
-DEFAULT_LEVEL: Level = Level[os.environ.get("SOCX_VERBOSITY", "FATAL")]
-"""Default logger level, a.k.a verbosity."""
+        for handler in unknown_handlers:
+            msg = f"Ignored unknown handler configuration: '{handler}'"
+            socx_logger.warning(msg)
 
-DEFAULT_FORMAT: str = os.environ.get("SOCX_LOG_FORMAT", "%(message)s")
-"""Default log message format used by the root handler."""
+    return socx_logger
 
-DEFAULT_TIME_FORMAT: str = os.environ.get("SOCX_TIME_FORMAT", "[%x %X]")
-"""Default timestamp format injected into log records."""
 
-DEFAULT_CHILD_FORMAT: str = os.environ.get(
-    "SOCX_LOG_FORMAT",
-    "%(asctime)s %(levelname)5s - %(filename)5s:%(lineno)-4d - %(message)s",
+DEFAULT_FORMATTER: logging.Formatter = logging.Formatter(
+    **settings.logging.formatters.default
 )
-"""Message format for child loggers that also emit timestamps."""
+"""Default application logging formatter."""
 
 DEFAULT_CHILD_FORMATTER: logging.Formatter = logging.Formatter(
-    DEFAULT_CHILD_FORMAT, DEFAULT_TIME_FORMAT
+    **settings.logging.formatters.child
 )
 """Formatter applied to file handlers registered on child loggers."""
 
-DEFAULT_LOG_DIRECTORY: Path = Path(
-    os.environ.get(
-        "SOCX_LOG_DIR",
-        user_log_path(appname=APP_LIB_NAME, ensure_exists=True),
-    )
-)
-"""Default application log directory."""
-
-DEFAULT_LOG_FILE: str = os.environ.get("SOCX_LOG_FILE", f"{APP_LIB_NAME}.log")
-"""Default application log file."""
-
 DEFAULT_HANDLERS: list[logging.Handler] = [
-    _get_console_handler(DEFAULT_LEVEL),
-    _get_file_handler(DEFAULT_LOG_DIRECTORY / DEFAULT_LOG_FILE),
+    _get_file_handler(**settings.logging.handlers.file),
+    _get_console_handler(**settings.logging.handlers.console),
+    _get_rotating_file_handler(**settings.logging.handlers.rotating_file),
 ]
 """Handlers attached to the module-level logger by default."""
 
-DEFAULT_LOGGING_CONFIG: dict[str, Any] = dict(
-    level=DEFAULT_LEVEL,
-    format=DEFAULT_FORMAT,
-    handlers=DEFAULT_HANDLERS,
-    encoding=DEFAULT_ENCODING,
-    datefmt=DEFAULT_TIME_FORMAT,
-)
-
-
-def _get_logger(**kwargs: Any) -> logging.Logger:
-    """Initialise and return the module-level root logger."""
-    logging.basicConfig(**dict(ChainMap(kwargs, DEFAULT_LOGGING_CONFIG)))
-    return logging.getLogger(APP_LIB_NAME)
-
 
 def get_logger(name: str, filename: str | None = None) -> logging.Logger:
     """Return a child logger configured with optional file output."""
     rv = logger.getChild(name)
     if filename is not None:
-        handler = _get_file_handler(filename)
-        handler.setFormatter(DEFAULT_CHILD_FORMATTER)
-        rv.addHandler(handler)
+        rv.addHandler(_get_file_handler(filename))
     return rv
 
 
@@ -243,8 +339,11 @@ def get_level(logger_: logging.Logger | None = None) -> Level:
 
 
 def set_level(level: Level, logger_: logging.Logger | None = None) -> None:
-    """Set the log level on the provided logger (defaults to module logger)."""
+    """Set the log level on ``logger_`` and all currently attached handlers."""
     logger_ = logger_ or logger
+    level = level if isinstance(level, str | int) else level.value
+    for handler in logger_.handlers:
+        handler.setLevel(level)
     logger_.setLevel(level)
 
 

{socx_cli-0.13.3 → socx_cli-0.13.5}/socx/regression/__init__.py

@@ -6,7 +6,6 @@ __all__ = (
     "TestStatus",
     "TestResult",
     "Regression",
-    "RegressionTree",
     "RegressionProgress",
 )
 
@@ -15,5 +14,4 @@ from socx.regression.test import TestBase as TestBase
 from socx.regression.test import TestStatus as TestStatus
 from socx.regression.test import TestResult as TestResult
 from socx.regression.regression import Regression as Regression
-from socx.regression.regression import RegressionTree as RegressionTree
 from socx.regression.progress import RegressionProgress as RegressionProgress