homesec 0.1.1__py3-none-any.whl → 1.0.1__py3-none-any.whl

homesec/app.py

@@ -5,8 +5,9 @@ from __future__ import annotations
 import asyncio
 import logging
 import signal
+from collections.abc import Callable
 from pathlib import Path
-from typing import TYPE_CHECKING, Callable, cast
+from typing import TYPE_CHECKING, cast
 
 from pydantic import BaseModel
 
@@ -17,9 +18,12 @@ from homesec.config import (
     validate_plugin_names,
 )
 from homesec.health import HealthServer
+from homesec.interfaces import EventStore
+from homesec.models.config import NotifierConfig
+from homesec.models.source import FtpSourceConfig, LocalFolderSourceConfig, RTSPSourceConfig
 from homesec.pipeline import ClipPipeline
-from homesec.plugins.analyzers import VLM_REGISTRY, load_vlm_plugin
 from homesec.plugins.alert_policies import ALERT_POLICY_REGISTRY
+from homesec.plugins.analyzers import VLM_REGISTRY, load_vlm_plugin
 from homesec.plugins.filters import FILTER_REGISTRY, load_filter_plugin
 from homesec.plugins.notifiers import (
     NOTIFIER_REGISTRY,
@@ -28,15 +32,11 @@ from homesec.plugins.notifiers import (
     NotifierPlugin,
 )
 from homesec.plugins.storage import STORAGE_REGISTRY, create_storage
-from homesec.sources import FtpSource, LocalFolderSource, RTSPSource
-from homesec.models.config import NotifierConfig
-from homesec.models.source import FtpSourceConfig, LocalFolderSourceConfig, RTSPSourceConfig
-from homesec.interfaces import EventStore
 from homesec.repository import ClipRepository
+from homesec.sources import FtpSource, LocalFolderSource, RTSPSource
 from homesec.state import NoopEventStore, NoopStateStore, PostgresStateStore
 
 if TYPE_CHECKING:
-    from homesec.models.config import Config
     from homesec.interfaces import (
         AlertPolicy,
         ClipSource,
@@ -46,25 +46,26 @@ if TYPE_CHECKING:
         StorageBackend,
         VLMAnalyzer,
     )
+    from homesec.models.config import Config
 
 logger = logging.getLogger(__name__)
 
 
 class Application:
     """Main application that orchestrates all components.
-    
+
     Handles component creation, lifecycle, and graceful shutdown.
     """
 
     def __init__(self, config_path: Path) -> None:
         """Initialize application with config file path.
-        
+
         Args:
             config_path: Path to YAML config file
         """
         self._config_path = config_path
         self._config: Config | None = None
-        
+
         # Components (created in _create_components)
         self._storage: StorageBackend | None = None
         self._state_store: StateStore = NoopStateStore()
@@ -77,41 +78,41 @@ class Application:
         self._sources: list[ClipSource] = []
         self._pipeline: ClipPipeline | None = None
         self._health_server: HealthServer | None = None
-        
+
         # Shutdown state
         self._shutdown_event = asyncio.Event()
         self._shutdown_started = False
 
     async def run(self) -> None:
         """Run the application.
-        
+
         Loads config, creates components, and runs until shutdown signal.
         """
         logger.info("Starting HomeSec application...")
-        
+
         # Load config
         self._config = load_config(self._config_path)
         logger.info("Config loaded from %s", self._config_path)
-        
+
         # Create components
         await self._create_components()
-        
+
         # Set up signal handlers
         self._setup_signal_handlers()
-        
+
         # Start health server
         if self._health_server:
             await self._health_server.start()
-        
+
         # Start sources
         for source in self._sources:
             await source.start()
-        
+
         logger.info("Application started. Waiting for clips...")
-        
+
         # Wait for shutdown signal
         await self._shutdown_event.wait()
-        
+
         # Graceful shutdown
         await self.shutdown()
 
@@ -152,7 +153,7 @@ class Application:
         self._filter = filter_plugin
         self._vlm = vlm_plugin
         alert_policy = self._create_alert_policy(config)
-        
+
         # Create pipeline
         self._pipeline = ClipPipeline(
             config=config,
@@ -166,12 +167,12 @@ class Application:
         )
         # Set event loop for thread-safe callbacks from sources
         self._pipeline.set_event_loop(asyncio.get_running_loop())
-        
+
         # Create sources and register callback
         self._sources = self._create_sources(config)
         for source in self._sources:
             source.register_callback(self._pipeline.on_new_clip)
-        
+
         # Create health server
         health_cfg = config.health
         self._health_server = HealthServer(
@@ -185,7 +186,7 @@ class Application:
             sources=self._sources,
             mqtt_is_critical=health_cfg.mqtt_is_critical,
         )
-        
+
         logger.info("All components created")
 
     def _create_storage(self, config: Config) -> StorageBackend:
@@ -237,10 +238,7 @@ class Application:
         if not self._notifier_entries:
             return
 
-        tasks = [
-            asyncio.create_task(entry.notifier.ping())
-            for entry in self._notifier_entries
-        ]
+        tasks = [asyncio.create_task(entry.notifier.ping()) for entry in self._notifier_entries]
         results = await asyncio.gather(*tasks, return_exceptions=True)
 
         for entry, result in zip(self._notifier_entries, results, strict=True):
@@ -339,7 +337,7 @@ class Application:
     def _setup_signal_handlers(self) -> None:
         """Set up signal handlers for graceful shutdown."""
         loop = asyncio.get_running_loop()
-        
+
         for sig in (signal.SIGINT, signal.SIGTERM):
             loop.add_signal_handler(sig, self._handle_signal, sig)
 
@@ -348,7 +346,7 @@ class Application:
         if self._shutdown_started:
             logger.warning("Shutdown already in progress, ignoring signal")
             return
-        
+
         logger.info("Received signal %s, initiating shutdown...", sig.name)
         self._shutdown_started = True
         self._shutdown_event.set()
@@ -356,18 +354,18 @@ class Application:
     async def shutdown(self) -> None:
         """Graceful shutdown of all components."""
         logger.info("Shutting down application...")
-        
+
         # Stop sources first
         if self._sources:
             await asyncio.gather(
                 *(source.shutdown() for source in self._sources),
                 return_exceptions=True,
             )
-        
+
         # Shutdown pipeline (waits for in-flight clips)
         if self._pipeline:
             await self._pipeline.shutdown()
-        
+
         # Stop health server
         if self._health_server:
             await self._health_server.stop()
@@ -381,13 +379,13 @@ class Application:
         # Close state store
         if self._state_store:
             await self._state_store.shutdown()
-        
+
         # Close storage
         if self._storage:
             await self._storage.shutdown()
-        
+
         # Close notifier
         if self._notifier:
             await self._notifier.shutdown()
-        
+
         logger.info("Application shutdown complete")

homesec/cli.py

@@ -6,15 +6,19 @@ import asyncio
 import sys
 from pathlib import Path
 
+from dotenv import load_dotenv
+
+load_dotenv()
+
 import fire  # type: ignore[import-untyped]
 
-from homesec.logging_setup import configure_logging
 from homesec.app import Application
 from homesec.config import ConfigError, load_config
 from homesec.config.validation import validate_camera_references, validate_plugin_names
+from homesec.logging_setup import configure_logging
 from homesec.maintenance.cleanup_clips import CleanupOptions, run_cleanup
-from homesec.plugins.analyzers import VLM_REGISTRY
 from homesec.plugins.alert_policies import ALERT_POLICY_REGISTRY
+from homesec.plugins.analyzers import VLM_REGISTRY
 from homesec.plugins.filters import FILTER_REGISTRY
 from homesec.plugins.notifiers import NOTIFIER_REGISTRY
 from homesec.plugins.storage import STORAGE_REGISTRY
@@ -30,17 +34,17 @@ class HomeSec:
 
     def run(self, config: str, log_level: str = "INFO") -> None:
         """Run the HomeSec pipeline.
-        
+
         Args:
             config: Path to YAML config file
             log_level: Logging level (DEBUG, INFO, WARNING, ERROR)
         """
         setup_logging(log_level)
-        
+
         config_path = Path(config)
-        
+
         app = Application(config_path)
-        
+
         try:
             asyncio.run(app.run())
         except ConfigError as e:
@@ -51,12 +55,12 @@ class HomeSec:
 
     def validate(self, config: str) -> None:
         """Validate config file without running.
-        
+
         Args:
             config: Path to YAML config file
         """
         config_path = Path(config)
-        
+
         try:
             cfg = load_config(config_path)
 
@@ -75,13 +79,12 @@ class HomeSec:
                 valid_notifiers=sorted(NOTIFIER_REGISTRY.keys()),
                 valid_alert_policies=sorted(ALERT_POLICY_REGISTRY.keys()),
             )
-            
+
             print(f"✓ Config valid: {config_path}")
             camera_names = [camera.name for camera in cfg.cameras]
             print(f"  Cameras: {camera_names}")
             notifier_backends = [
-                f"{notifier.backend} (enabled={notifier.enabled})"
-                for notifier in cfg.notifiers
+                f"{notifier.backend} (enabled={notifier.enabled})" for notifier in cfg.notifiers
             ]
             print(f"  Storage backend: {cfg.storage.backend}")
             print(f"  Notifiers: {notifier_backends}")

homesec/config/loader.py

@@ -22,13 +22,13 @@ class ConfigError(Exception):
 
 def load_config(path: Path) -> Config:
     """Load and validate configuration from YAML file.
-    
+
     Args:
         path: Path to YAML config file
-        
+
     Returns:
         Validated Config instance
-        
+
     Raises:
         ConfigError: If file not found, YAML invalid, or validation fails
     """
@@ -55,13 +55,13 @@ def load_config(path: Path) -> Config:
 
 def load_config_from_dict(data: dict[str, Any]) -> Config:
     """Load and validate configuration from a dict (useful for testing).
-    
+
     Args:
         data: Configuration dictionary
-        
+
     Returns:
         Validated Config instance
-        
+
     Raises:
         ConfigError: If validation fails
     """
@@ -73,14 +73,14 @@ def load_config_from_dict(data: dict[str, Any]) -> Config:
 
 def resolve_env_var(env_var_name: str, required: bool = True) -> str | None:
     """Resolve environment variable by name.
-    
+
     Args:
         env_var_name: Name of the environment variable
         required: If True, raise if not found
-        
+
     Returns:
         Environment variable value, or None if not required and not found
-        
+
     Raises:
         ConfigError: If required and not found
     """
@@ -92,11 +92,11 @@ def resolve_env_var(env_var_name: str, required: bool = True) -> str | None:
 
 def format_validation_error(e: ValidationError, path: Path | None = None) -> str:
     """Format Pydantic validation error for human readability.
-    
+
     Args:
         e: Pydantic ValidationError
         path: Optional config file path for context
-        
+
     Returns:
         Human-readable error message
     """

homesec/config/validation.py

@@ -59,16 +59,13 @@ def validate_plugin_names(
         errors.append(f"Unknown VLM plugin: {config.vlm.backend} (valid: {valid_vlms})")
 
     if valid_storage is not None and config.storage.backend not in valid_storage:
-        errors.append(
-            f"Unknown storage backend: {config.storage.backend} (valid: {valid_storage})"
-        )
+        errors.append(f"Unknown storage backend: {config.storage.backend} (valid: {valid_storage})")
 
     if valid_notifiers is not None:
         for notifier in config.notifiers:
             if notifier.backend not in valid_notifiers:
                 errors.append(
-                    "Unknown notifier backend: "
-                    f"{notifier.backend} (valid: {valid_notifiers})"
+                    f"Unknown notifier backend: {notifier.backend} (valid: {valid_notifiers})"
                 )
 
     if valid_alert_policies is not None:

homesec/errors.py

@@ -5,7 +5,7 @@ from __future__ import annotations
 
 class PipelineError(Exception):
     """Base exception for all pipeline errors.
-    
+
     Compatible with error-as-value pattern: instances can be returned as values
     instead of raised. Preserves stack traces via exception chaining.
     """
@@ -23,9 +23,7 @@ class PipelineError(Exception):
 class UploadError(PipelineError):
     """Storage upload failed."""
 
-    def __init__(
-        self, clip_id: str, storage_uri: str | None, cause: Exception
-    ) -> None:
+    def __init__(self, clip_id: str, storage_uri: str | None, cause: Exception) -> None:
         super().__init__(
             f"Upload failed for {clip_id}", stage="upload", clip_id=clip_id, cause=cause
         )

homesec/health/server.py

@@ -16,7 +16,7 @@ logger = logging.getLogger(__name__)
 
 class HealthServer:
     """HTTP server for health checks.
-    
+
     Provides /health endpoint returning component status.
     """
 
@@ -26,26 +26,26 @@ class HealthServer:
         port: int = 8080,
     ) -> None:
         """Initialize health server.
-        
+
         Args:
             host: Host to bind to
             port: Port to bind to
         """
         self.host = host
         self.port = port
-        
+
         # Components to check (set via set_components)
         self._state_store: StateStore | None = None
         self._storage: StorageBackend | None = None
         self._notifier: Notifier | None = None
         self._sources: list[ClipSource] = []
         self._mqtt_is_critical = False
-        
+
         # Server state
         self._app: web.Application | None = None
         self._runner: web.AppRunner | None = None
         self._site: web.TCPSite | None = None
-        
+
         logger.info("HealthServer initialized: %s:%d", host, port)
 
     def set_components(
@@ -58,7 +58,7 @@ class HealthServer:
         mqtt_is_critical: bool = False,
     ) -> None:
         """Set components to check.
-        
+
         Args:
             state_store: State store to ping
             storage: Storage backend to ping
@@ -76,24 +76,24 @@ class HealthServer:
         """Start HTTP server."""
         self._app = web.Application()
         self._app.router.add_get("/health", self._health_handler)
-        
+
         self._runner = web.AppRunner(self._app)
         await self._runner.setup()
-        
+
         self._site = web.TCPSite(self._runner, self.host, self.port)
         await self._site.start()
-        
+
         logger.info("HealthServer started: http://%s:%d/health", self.host, self.port)
 
     async def stop(self) -> None:
         """Stop HTTP server."""
         if self._runner:
             await self._runner.cleanup()
-        
+
         self._app = None
         self._runner = None
         self._site = None
-        
+
         logger.info("HealthServer stopped")
 
     async def _health_handler(self, request: web.Request) -> web.Response:
@@ -145,7 +145,7 @@ class HealthServer:
         """Check if all clip sources are healthy."""
         if not self._sources:
             return True  # No sources configured
-        
+
         # All sources must be healthy
         return all(source.is_healthy() for source in self._sources)
 
@@ -159,12 +159,14 @@ class HealthServer:
         for source in self._sources:
             camera_name = getattr(source, "camera_name", "unknown")
             last_heartbeat = source.last_heartbeat()
-            details.append({
-                "name": camera_name,
-                "healthy": source.is_healthy(),
-                "last_heartbeat": last_heartbeat,
-                "last_heartbeat_age_s": round(current_time - last_heartbeat, 3),
-            })
+            details.append(
+                {
+                    "name": camera_name,
+                    "healthy": source.is_healthy(),
+                    "last_heartbeat": last_heartbeat,
+                    "last_heartbeat_age_s": round(current_time - last_heartbeat, 3),
+                }
+            )
         return details
 
     async def _check_component(
@@ -183,38 +185,38 @@ class HealthServer:
 
     def _compute_status(self, checks: dict[str, bool]) -> str:
         """Compute overall health status.
-        
+
         Args:
             checks: Dict of component check results
-            
+
         Returns:
             "healthy", "degraded", or "unhealthy"
         """
         # Critical checks (unhealthy if fail)
         if not checks["sources"]:
             return "unhealthy"
-        
+
         if not checks["storage"]:
             return "unhealthy"
-        
+
         # MQTT can be critical (configurable)
         if not checks["mqtt"] and self._mqtt_is_critical:
             return "unhealthy"
-        
+
         # Non-critical checks (degraded if fail)
         if not checks["db"] or not checks["mqtt"]:
             return "degraded"
-        
+
         return "healthy"
 
     def _compute_warnings(self) -> list[str]:
         """Generate warnings for stale heartbeats.
-        
+
         Returns:
             List of warning messages
         """
         warnings: list[str] = []
-        
+
         # Check source heartbeats (warn if > 2 minutes)
         current_time = time.monotonic()
         for source in self._sources:
@@ -222,5 +224,5 @@ class HealthServer:
             if heartbeat_age > 120:  # 2 minutes
                 camera_name = getattr(source, "camera_name", "unknown")
                 warnings.append(f"source_{camera_name}_heartbeat_stale")
-        
+
         return warnings

homesec/interfaces.py

@@ -3,9 +3,10 @@
 from __future__ import annotations
 
 from abc import ABC, abstractmethod
+from collections.abc import Callable
 from datetime import datetime
 from pathlib import Path
-from typing import TYPE_CHECKING, Callable
+from typing import TYPE_CHECKING
 
 if TYPE_CHECKING:
     from homesec.models.alert import Alert, AlertDecision
@@ -73,7 +74,7 @@ class StorageBackend(Shutdownable, ABC):
     """Stores raw clips and derived artifacts."""
 
     @abstractmethod
-    async def put_file(self, local_path: Path, dest_path: str) -> "StorageUploadResult":
+    async def put_file(self, local_path: Path, dest_path: str) -> StorageUploadResult:
         """Upload file to storage. Returns storage result."""
         raise NotImplementedError
 
@@ -143,7 +144,7 @@ class EventStore(ABC):
     @abstractmethod
     async def append(self, event: ClipLifecycleEvent) -> None:
         """Append a lifecycle event.
-        
+
         Events are appended with database-assigned ids.
         Raises on database errors (should be retried by caller).
         """
@@ -156,7 +157,7 @@ class EventStore(ABC):
         after_id: int | None = None,
     ) -> list[ClipLifecycleEvent]:
         """Get all events for a clip, optionally after an event id.
-        
+
         Returns events ordered by id. Returns empty list on error.
         """
         raise NotImplementedError
@@ -175,6 +176,7 @@ class Notifier(Shutdownable, ABC):
         """Health check. Returns True if notifier is reachable."""
         raise NotImplementedError
 
+
 class AlertPolicy(ABC):
     """Decides whether to notify based on analysis results."""
 
@@ -197,7 +199,7 @@ class AlertPolicy(ABC):
         camera_name: str,
         filter_result: FilterResult | None,
         analysis: AnalysisResult | None,
-    ) -> "AlertDecision":
+    ) -> AlertDecision:
         """Build an AlertDecision from should_notify output."""
         from homesec.models.alert import AlertDecision
 
@@ -209,7 +211,9 @@ class ObjectFilter(Shutdownable, ABC):
     """Plugin interface for object detection in video clips."""
 
     @abstractmethod
-    async def detect(self, video_path: Path, overrides: FilterOverrides | None = None) -> FilterResult:
+    async def detect(
+        self, video_path: Path, overrides: FilterOverrides | None = None
+    ) -> FilterResult:
         """Detect objects in video clip.
 
         Implementation notes:
@@ -225,6 +229,7 @@ class ObjectFilter(Shutdownable, ABC):
         """
         raise NotImplementedError
 
+
 class VLMAnalyzer(Shutdownable, ABC):
     """Plugin interface for VLM-based clip analysis."""
 

homesec/logging_setup.py

@@ -35,16 +35,17 @@ _STANDARD_LOGRECORD_ATTRS = {
     "taskName",
 }
 
+
 class _CameraNameFilter(logging.Filter):
     def filter(self, record: logging.LogRecord) -> bool:
-        if not hasattr(record, "camera_name") or getattr(record, "camera_name") in (None, ""):
+        if not hasattr(record, "camera_name") or record.camera_name in (None, ""):
             record.camera_name = _CURRENT_CAMERA_NAME
         return True
 
 
 class _RecordingIdFilter(logging.Filter):
     def filter(self, record: logging.LogRecord) -> bool:
-        if not hasattr(record, "recording_id") or getattr(record, "recording_id") in (None, ""):
+        if not hasattr(record, "recording_id") or record.recording_id in (None, ""):
             record.recording_id = _CURRENT_RECORDING_ID
         return True
 
@@ -86,6 +87,7 @@ def set_camera_name(name: str | None) -> None:
     global _CURRENT_CAMERA_NAME
     _CURRENT_CAMERA_NAME = name or "-"
 
+
 def set_recording_id(recording_id: str | None) -> None:
     """Set the `recording_id` value injected into log records."""
     global _CURRENT_RECORDING_ID
@@ -99,6 +101,7 @@ def _install_camera_filter() -> None:
             continue
         handler.addFilter(_CameraNameFilter())
 
+
 def _install_recording_filter() -> None:
     root = logging.getLogger()
     for handler in root.handlers:
@@ -115,8 +118,7 @@ def configure_logging(*, log_level: str = "INFO", camera_name: str | None = None
     """
     console_level_name = str(log_level).upper()
     default_console_fmt = (
-        "%(asctime)s %(levelname)s [%(camera_name)s] "
-        "%(module)s %(pathname)s:%(lineno)d %(message)s"
+        "%(asctime)s %(levelname)s [%(camera_name)s] %(module)s %(pathname)s:%(lineno)d %(message)s"
     )
     console_fmt = os.getenv("CONSOLE_LOG_FORMAT", default_console_fmt)
 
@@ -160,7 +162,9 @@ def configure_logging(*, log_level: str = "INFO", camera_name: str | None = None
         from homesec.telemetry.db_log_handler import AsyncPostgresJsonLogHandler
     except Exception as exc:
         # Keep the app running even if DB logging deps aren't installed.
-        logging.getLogger(__name__).warning("DB_DSN is set but DB log handler failed to import: %s", exc)
+        logging.getLogger(__name__).warning(
+            "DB_DSN is set but DB log handler failed to import: %s", exc
+        )
         return
 
     for handler in list(root.handlers):