Merge 551d3b0812 into bc816926a5

frigate/genai/__init__.py

@@ -5,7 +5,6 @@ import json
 import logging
 import os
 import re
-import time
 from typing import Any, AsyncGenerator, Callable, Optional
 
 import numpy as np
@@ -51,10 +50,6 @@ def register_genai_provider(key: GenAIProviderEnum) -> Callable:
 class GenAIClient:
     """Generative AI client for Frigate."""
 
-    # Minimum seconds between re-initialization attempts when the provider was
-    # offline at startup
-    REINIT_INTERVAL = 60.0
-
     def __init__(
         self,
         genai_config: GenAIConfig,
@@ -65,34 +60,6 @@ class GenAIClient:
         self.timeout = timeout
         self.validate_model = validate_model
         self.provider = self._init_provider()
-        self._last_init_attempt = time.monotonic()
-
-    def ensure_provider(self) -> bool:
-        """Ensure a provider is available, retrying initialization if needed.
-
-        Providers can fail to initialize at startup when their backing service
-        isn't online yet (common when both are started together). This retries
-        ``_init_provider`` lazily — throttled to ``REINIT_INTERVAL`` — so the
-        client recovers on its own once the service is reachable, without a
-        config reload.
-
-        Returns True if a provider is available.
-        """
-        if self.provider is not None:
-            return True
-
-        now = time.monotonic()
-        if now - self._last_init_attempt < self.REINIT_INTERVAL:
-            return False
-
-        self._last_init_attempt = now
-        self.provider = self._init_provider()
-        if self.provider is not None:
-            logger.info(
-                "GenAI provider %s is now available",
-                self.genai_config.provider,
-            )
-        return self.provider is not None
 
     def generate_review_description(
         self,

frigate/genai/manager.py

@@ -62,9 +62,7 @@ class GenAIClientManager:
     def _get_client(self, name: str) -> "Optional[GenAIClient]":
         """Return the client for *name*, creating it on first access."""
         if name in self._clients:
-            client = self._clients[name]
-            client.ensure_provider()
-            return client
+            return self._clients[name]
 
         from frigate.genai import PROVIDERS
 
@@ -80,7 +78,7 @@ class GenAIClientManager:
             return None
 
         try:
-            client = provider_cls(genai_cfg)
+            client: "GenAIClient" = provider_cls(genai_cfg)
         except Exception as e:
             logger.exception(
                 "Failed to create GenAI client for provider %s: %s",