Апгрейд версии

Reviewed-on: #2

README.md

@@ -3,6 +3,14 @@
 This package was created to run my applications.
 The ConfigManager class implements the entry point for the program and provides the actual application configuration. It also simplifies logging setup.
 
+## Logging (v2)
+Logging is configured from the config file only if it contains a **`log`** section in [dictConfig](https://docs.python.org/3/library/logging.config.html#logging.config.dictConfig) format. If there is no `log` section, the manager logs a warning and the default Python level (WARNING) remains, so INFO/DEBUG messages may not appear.
+
+**How to verify that logging config is applied:**
+- Ensure your config file path is correct and the file is loaded on startup (no error in logs about reading config).
+- Ensure the config has a `log` key with `version: 1`, `handlers`, and `loggers` (see `tests/config.yaml` for an example).
+- After startup you should see an INFO message: `"Logging configuration applied"` (from `config_manager.v1.log_manager`). If you do not see it, either the `log` section is missing (you will see a warning) or the root/package log level is above INFO.
+
 ## Installation
 ``pip install git+https://git.lesha.spb.ru/alex/config_manager.git``
 

pyproject.toml

@@ -4,8 +4,8 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "config_manager"
-version = "2.0.0"
-description = "Config manager for building applications"
+version = "2.1.7"
+description = "Фикс вечного цикла при ошибке"
 authors = [
     { name = "Aleksei Zosimov", email = "lesha.spb@gmail.com" }
 ]
@@ -13,6 +13,8 @@ readme = "README.md"
 requires-python = ">=3.8"
 dependencies = [
     "PyYAML>=6.0",
+    "fastapi>=0.100.0",
+    "uvicorn[standard]>=0.22.0",
 ]
 
 [project.urls]

src/config_manager/__init__.py

@@ -1,3 +1,3 @@
-from .v2.manager import ConfigManagerV2 as ConfigManager
+from .v2 import ConfigManagerV2 as ConfigManager
 from .v1.cfg_manager import ConfigManager as LegacyConfigManager
 from .v1.log_manager import LogManager

src/config_manager/v1/cfg_manager.py

@@ -9,8 +9,8 @@ from typing import Any, Optional
 from .log_manager import LogManager
 
 class ConfigManager:
-    DEFAULT_UPDATE_INTERVAL = 5.0
-    DEFAULT_WORK_INTERVAL = 2.0
+    DEFAULT_UPDATE_INTERVAL = 5
+    DEFAULT_WORK_INTERVAL = 2
 
     def __init__(self, path: str, log_manager: Optional[LogManager] = None):
         self.path = path
@@ -81,9 +81,13 @@ class ConfigManager:
         pass
 
     async def _worker_loop(self) -> None:
-        while not self._halt.is_set():
-            await asyncio.to_thread(self.execute)
-            await asyncio.sleep(self.work_interval)
+        self.logger.warning("Worker loop started")
+        try:
+            while not self._halt.is_set():
+                await asyncio.to_thread(self.execute)
+                await asyncio.sleep(self.work_interval)
+        finally:
+            self.logger.warning("Worker loop stopped")
 
     async def _periodic_update_loop(self) -> None:
         while not self._halt.is_set():

src/config_manager/v1/log_manager.py

@@ -22,6 +22,9 @@ class LogManager:
         """
         logging_config = config.get("log")
         if not logging_config:
+            self.logger.warning(
+                "Config has no 'log' section; logging parameters from config are not applied (default level may be WARNING)."
+            )
             return
         
         try:

src/config_manager/v2/__init__.py

@@ -1,4 +1,7 @@
-from .manager import ConfigManagerV2
-from .types import HealthServerSettings
+"""Публичный API V2: точка входа в менеджер конфигурации и настройки management-сервера.
 
-__all__ = ["ConfigManagerV2", "HealthServerSettings"]
+Экспортирует ConfigManagerV2 и типы настроек для использования приложениями."""
+from .core import ConfigManagerV2
+from .types import HealthServerSettings, ManagementServerSettings
+
+__all__ = ["ConfigManagerV2", "ManagementServerSettings", "HealthServerSettings"]

src/config_manager/v2/config_loader.py

@@ -1,52 +0,0 @@
-from __future__ import annotations
-
-import asyncio
-import hashlib
-import json
-import os
-from typing import Any, Optional
-
-import yaml
-
-
-class ConfigLoader:
-    def __init__(self, path: str):
-        """Initialize loader state for a specific config file path."""
-        self.path = path
-        self.config: Any = None
-        self.last_valid_config: Any = None
-        self._last_seen_hash: Optional[str] = None
-
-    def _read_file_sync(self) -> str:
-        """Read raw config text from disk synchronously."""
-        with open(self.path, "r", encoding="utf-8") as fh:
-            return fh.read()
-
-    async def read_file_async(self) -> str:
-        """Read raw config text from disk in a worker thread."""
-        return await asyncio.to_thread(self._read_file_sync)
-
-    def parse_config(self, data: str) -> Any:
-        """Parse config text as YAML or JSON based on file extension."""
-        extension = os.path.splitext(self.path)[1].lower()
-        if extension in (".yaml", ".yml"):
-            return yaml.safe_load(data)
-        return json.loads(data)
-
-    @staticmethod
-    def _calculate_hash(data: str) -> str:
-        """Calculate a stable content hash for change detection."""
-        return hashlib.sha256(data.encode("utf-8")).hexdigest()
-
-    async def load_if_changed(self) -> tuple[bool, Any]:
-        """Load and parse config only when file content changed."""
-        raw_data = await self.read_file_async()
-        current_hash = self._calculate_hash(raw_data)
-        if current_hash == self._last_seen_hash:
-            return False, self.config
-
-        self._last_seen_hash = current_hash
-        parsed = self.parse_config(raw_data)
-        self.config = parsed
-        self.last_valid_config = parsed
-        return True, parsed

src/config_manager/v2/control/__init__.py

@@ -1,3 +1,6 @@
+"""Каналы внешнего управления: абстракция и реализация (например, Telegram).
+
+Позволяет запускать, останавливать и запрашивать статус менеджера через ботов и другие интерфейсы."""
 from .base import ControlChannel
 from .telegram import TelegramControlChannel
 

src/config_manager/v2/control/base.py

@@ -1,3 +1,6 @@
+"""Базовый абстрактный канал управления и типы обработчиков команд.
+
+Определяет контракт: старт/стоп канала и привязка обработчиков start/stop/status."""
 from __future__ import annotations
 
 from abc import ABC, abstractmethod
@@ -12,10 +15,10 @@ StatusHandler = Callable[[], Awaitable[str]]
 class ControlChannel(ABC):
     @abstractmethod
     async def start(self, on_start: StartHandler, on_stop: StopHandler, on_status: StatusHandler) -> None:
-        """Start channel and bind command handlers."""
+        """Запустить канал и привязать обработчики команд."""
         raise NotImplementedError
 
     @abstractmethod
     async def stop(self) -> None:
-        """Stop channel and release its resources."""
+        """Остановить канал и освободить его ресурсы."""
         raise NotImplementedError

src/config_manager/v2/control/telegram.py

@@ -1,3 +1,6 @@
+"""Реализация канала управления через Telegram Bot API (long polling).
+
+Принимает команды /start, /stop, /status в указанном чате и вызывает привязанные обработчики."""
 from __future__ import annotations
 
 import asyncio
@@ -15,10 +18,10 @@ class TelegramControlChannel(ControlChannel):
         self,
         token: str,
         chat_id: int,
-        poll_interval: float = 2.0,
+        poll_interval: int = 2,
         logger: Optional[logging.Logger] = None,
     ):
-        """Initialize Telegram polling channel with bot and chat settings."""
+        """Инициализация канала опроса Telegram с настройками бота и чата."""
         self._token = token
         self._chat_id = chat_id
         self._poll_interval = poll_interval
@@ -31,7 +34,7 @@ class TelegramControlChannel(ControlChannel):
         self._logger = logger or logging.getLogger(__name__)
 
     async def start(self, on_start: StartHandler, on_stop: StopHandler, on_status: StatusHandler) -> None:
-        """Start polling Telegram updates and register command callbacks."""
+        """Запустить опрос обновлений Telegram и зарегистрировать колбэки команд."""
         if self._task is not None and not self._task.done():
             return
         self._on_start = on_start
@@ -41,7 +44,7 @@ class TelegramControlChannel(ControlChannel):
         self._task = asyncio.create_task(self._poll_loop())
 
     async def stop(self) -> None:
-        """Stop polling loop and wait until task termination."""
+        """Остановить цикл опроса и дождаться завершения задачи."""
         self._stop_event.set()
         if self._task is not None:
             self._task.cancel()
@@ -52,7 +55,7 @@ class TelegramControlChannel(ControlChannel):
             self._task = None
 
     async def _poll_loop(self) -> None:
-        """Continuously fetch updates and dispatch supported commands."""
+        """Непрерывно получать обновления и вызывать поддерживаемые команды."""
         while not self._stop_event.is_set():
             try:
                 updates = await asyncio.to_thread(self._fetch_updates)
@@ -67,7 +70,7 @@ class TelegramControlChannel(ControlChannel):
                 continue
 
     def _fetch_updates(self) -> list[dict]:
-        """Pull new Telegram updates using the latest offset."""
+        """Запросить новые обновления Telegram с учётом последнего offset."""
         params = {"timeout": 0}
         if self._offset is not None:
             params["offset"] = self._offset
@@ -82,7 +85,7 @@ class TelegramControlChannel(ControlChannel):
         return result
 
     async def _process_update(self, update: dict) -> None:
-        """Handle one Telegram update and execute mapped command."""
+        """Обработать одно обновление Telegram и выполнить соответствующую команду."""
         message = update.get("message") or {}
         text = (message.get("text") or "").strip().lower()
         chat = message.get("chat") or {}
@@ -103,7 +106,7 @@ class TelegramControlChannel(ControlChannel):
         await asyncio.to_thread(self._send_message, reply)
 
     def _send_message(self, text: str) -> None:
-        """Send plain-text reply to the configured Telegram chat."""
+        """Отправить текстовый ответ в настроенный чат Telegram."""
         encoded = urllib.parse.urlencode({"chat_id": self._chat_id, "text": text})
         url = f"https://api.telegram.org/bot{self._token}/sendMessage"
         req = urllib.request.Request(url, data=encoded.encode("utf-8"), method="POST")

src/config_manager/v2/core/__init__.py

@@ -0,0 +1,8 @@
+"""Ядро V2: жизненный цикл менеджера, загрузка конфига и цикл воркера.
+
+Содержит ConfigManagerV2, загрузчик конфигурации и планировщик повторяющегося выполнения execute()."""
+from .config_loader import ConfigLoader
+from .manager import ConfigManagerV2
+from .scheduler import WorkerLoop
+
+__all__ = ["ConfigLoader", "ConfigManagerV2", "WorkerLoop"]

src/config_manager/v2/core/config_loader.py

@@ -0,0 +1,93 @@
+"""Загрузчик конфигурации из файла (YAML/JSON) с обнаружением изменений по хешу.
+
+Читает файл синхронно и асинхронно, парсит по расширению и возвращает последний валидный конфиг при ошибках."""
+from __future__ import annotations
+
+import asyncio
+import hashlib
+import json
+import logging
+import os
+from typing import Any, Optional
+
+import yaml
+
+logger = logging.getLogger(__name__)
+
+
+class ConfigLoader:
+    def __init__(self, path: str):
+        """Инициализация состояния загрузчика для указанного пути к файлу конфига."""
+        self.path = path
+        self.config: Any = None
+        self.last_valid_config: Any = None
+        self._last_seen_hash: Optional[str] = None
+        logger.warning("ConfigLoader.__init__ result: path=%s", self.path)
+
+    def _read_file_sync(self) -> str:
+        """Синхронно прочитать сырой текст конфига с диска."""
+        with open(self.path, "r", encoding="utf-8") as fh:
+            data = fh.read()
+        logger.warning("ConfigLoader._read_file_sync result: bytes=%s", len(data))
+        return data
+
+    async def read_file_async(self) -> str:
+        """Прочитать сырой текст конфига с диска в рабочем потоке."""
+        result = await asyncio.to_thread(self._read_file_sync)
+        logger.warning("ConfigLoader.read_file_async result: bytes=%s", len(result))
+        return result
+
+    def parse_config(self, data: str) -> Any:
+        """Распарсить текст конфига как YAML или JSON по расширению файла."""
+        extension = os.path.splitext(self.path)[1].lower()
+        try:
+            if extension in (".yaml", ".yml"):
+                result = yaml.safe_load(data)
+            else:
+                result = json.loads(data)
+        except Exception:  # noqa: BLE001
+            logger.exception("ConfigLoader.parse_config error: extension=%s", extension)
+            raise
+        logger.warning(
+            "ConfigLoader.parse_config result: extension=%s type=%s",
+            extension,
+            type(result).__name__,
+        )
+        return result
+
+    @staticmethod
+    def _calculate_hash(data: str) -> str:
+        """Вычислить устойчивый хеш содержимого для обнаружения изменений."""
+        result = hashlib.sha256(data.encode("utf-8")).hexdigest()
+        logger.warning("ConfigLoader._calculate_hash result: hash=%s", result)
+        return result
+
+    async def load_if_changed(self) -> tuple[bool, Any]:
+        """Загрузить и распарсить конфиг только при изменении содержимого файла."""
+        raw_data = await self.read_file_async()
+        current_hash = self._calculate_hash(raw_data)
+        if current_hash == self._last_seen_hash:
+            logger.warning("ConfigLoader.load_if_changed result: changed=False")
+            return False, self.config
+
+        self._last_seen_hash = current_hash
+        parsed = self.parse_config(raw_data)
+        self.config = parsed
+        self.last_valid_config = parsed
+        logger.warning("ConfigLoader.load_if_changed result: changed=True")
+        return True, parsed
+
+
+def extract_scheduler_intervals(
+    config: Any,
+    default_update: float,
+    default_work: float,
+) -> tuple[float, float]:
+    """Извлекает update_interval и work_interval из конфига (dict). Возвращает значения по умолчанию при ошибке."""
+    if not isinstance(config, dict):
+        return default_update, default_work
+    upd = config.get("update_interval")
+    wrk = config.get("work_interval")
+    u = float(upd) if isinstance(upd, (int, float)) and upd > 0 else default_update
+    w = float(wrk) if isinstance(wrk, (int, float)) and wrk > 0 else default_work
+    return u, w

src/config_manager/v2/core/manager.py

@@ -0,0 +1,348 @@
+"""Config manager v2: runtime orchestration and configuration updates."""
+from __future__ import annotations
+
+import asyncio
+import logging
+import os
+import time
+from typing import Any, Optional
+
+from ...v1.log_manager import LogManager
+from ..control.base import ControlChannel
+from ..management import (
+    ControlChannelBridge,
+    HealthAggregator,
+    ManagementApiBridge,
+    ManagementServer,
+)
+from ..types import HealthPayload, LifecycleState, ManagementServerSettings
+from .config_loader import ConfigLoader
+from .scheduler import WorkerLoop
+
+logger = logging.getLogger(__name__)
+
+
+def _read_env_interval(name: str, default_value: float) -> float:
+    """Read positive float interval from env."""
+    raw_value = os.environ.get(name)
+    if raw_value is None:
+        return float(default_value)
+    try:
+        parsed = float(raw_value)
+        if parsed <= 0:
+            raise ValueError(f"{name} must be greater than zero")
+        return parsed
+    except Exception:  # noqa: BLE001
+        logger.exception(
+            "ConfigManagerV2 interval parse error: env=%s raw_value=%s fallback=%s",
+            name,
+            raw_value,
+            default_value,
+        )
+        return float(default_value)
+
+
+class _RuntimeController:
+    """Runtime loops and lifecycle supervision."""
+
+    def _on_execute_success(self) -> None:
+        self._last_success_timestamp = time.monotonic()
+        self._last_execute_error = None
+        self.logger.warning(
+            "ConfigManagerV2._on_execute_success result: last_success_timestamp=%s",
+            self._last_success_timestamp,
+        )
+
+    def _on_execute_error(self, exc: Exception) -> None:
+        self._last_execute_error = str(exc)
+        self.logger.exception("ConfigManagerV2._on_execute_error")
+        self.logger.warning(
+            "ConfigManagerV2._on_execute_error result: last_execute_error=%s",
+            self._last_execute_error,
+        )
+
+    async def _worker_loop(self) -> None:
+        self.logger.warning(
+            "ConfigManagerV2._worker_loop result: started work_interval=%s",
+            self.work_interval,
+        )
+        worker = WorkerLoop(
+            execute=self.execute,
+            get_interval=lambda: self.work_interval,
+            halt_event=self._halt,
+            on_error=self._on_execute_error,
+            on_success=self._on_execute_success,
+        )
+        try:
+            await worker.run()
+            self.logger.warning("ConfigManagerV2._worker_loop result: completed")
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._worker_loop error")
+            raise
+        finally:
+            self.logger.warning("ConfigManagerV2._worker_loop result: stopped")
+
+    async def _periodic_update_loop(self) -> None:
+        self.logger.warning(
+            "ConfigManagerV2._periodic_update_loop result: started update_interval=%s",
+            self.update_interval,
+        )
+        try:
+            while not self._halt.is_set():
+                await self._update_config()
+                try:
+                    await asyncio.wait_for(self._halt.wait(), timeout=max(self.update_interval, 0.05))
+                except asyncio.TimeoutError:
+                    continue
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._periodic_update_loop error")
+            raise
+        finally:
+            self.logger.warning("ConfigManagerV2._periodic_update_loop result: stopped")
+
+    async def _status_text(self) -> str:
+        health = await self._health_aggregator.collect()
+        detail = health.get("detail")
+        if detail:
+            status_text = f"state={self._state.value}; health={health['status']}; detail={detail}"
+            self.logger.warning("ConfigManagerV2._status_text result: %s", status_text)
+            return status_text
+        status_text = f"state={self._state.value}; health={health['status']}"
+        self.logger.warning("ConfigManagerV2._status_text result: %s", status_text)
+        return status_text
+
+    async def _start_control_channel(self) -> None:
+        if self._control_channel is None:
+            self.logger.warning("ConfigManagerV2._start_control_channel result: no control channel")
+            return
+        try:
+            await self._control_channel.start(
+                self._control_bridge.on_start,
+                self._control_bridge.on_stop,
+                self._control_bridge.on_status,
+            )
+            self.logger.warning("ConfigManagerV2._start_control_channel result: started")
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._start_control_channel error")
+
+    async def _stop_control_channel(self) -> None:
+        if self._control_channel is None:
+            self.logger.warning("ConfigManagerV2._stop_control_channel result: no control channel")
+            return
+        try:
+            await self._control_channel.stop()
+            self.logger.warning("ConfigManagerV2._stop_control_channel result: stopped")
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._stop_control_channel error")
+
+    async def _start_management_server(self) -> None:
+        if self._management_server is None:
+            self.logger.warning("ConfigManagerV2._start_management_server result: disabled")
+            return
+        try:
+            await self._management_server.start()
+            self.logger.warning(
+                "ConfigManagerV2._start_management_server result: started port=%s",
+                self._management_server.port,
+            )
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._start_management_server error")
+            self.logger.warning(
+                "ConfigManagerV2._start_management_server result: failed worker will continue",
+            )
+
+    def _on_runtime_task_done(self, task: asyncio.Task) -> None:
+        if task.cancelled():
+            self.logger.warning("ConfigManagerV2._on_runtime_task_done result: cancelled")
+            return
+        try:
+            exc = task.exception()
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._on_runtime_task_done error while reading task exception")
+            return
+        if exc is None:
+            self.logger.warning("ConfigManagerV2._on_runtime_task_done result: completed")
+            return
+        self.logger.error(
+            "ConfigManagerV2 background task failed",
+            exc_info=(type(exc), exc, exc.__traceback__),
+        )
+        self.logger.warning("ConfigManagerV2._on_runtime_task_done result: failed")
+
+    async def _run(self) -> None:
+        self._state = LifecycleState.STARTING
+        self.logger.warning("ConfigManagerV2._run result: state=%s", self._state.value)
+        self._halt.clear()
+        await self._update_config()
+
+        await self._start_management_server()
+        await self._start_control_channel()
+
+        self._state = LifecycleState.RUNNING
+        self.logger.warning("ConfigManagerV2._run result: state=%s", self._state.value)
+        tasks = [
+            asyncio.create_task(self._worker_loop(), name="v2-worker-loop"),
+            asyncio.create_task(self._periodic_update_loop(), name="v2-config-loop"),
+        ]
+        try:
+            await asyncio.gather(*tasks)
+            self.logger.warning("ConfigManagerV2._run result: background loops completed")
+        except asyncio.CancelledError:
+            self.logger.warning("ConfigManagerV2._run result: cancelled")
+            raise
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._run error")
+            raise
+        finally:
+            self._state = LifecycleState.STOPPING
+            self.logger.warning("ConfigManagerV2._run result: state=%s", self._state.value)
+            self._halt.set()
+            for task in tasks:
+                task.cancel()
+            await asyncio.gather(*tasks, return_exceptions=True)
+            self._state = LifecycleState.STOPPED
+            self._task = None
+            self.logger.warning(
+                "ConfigManagerV2._run result: state=%s api_and_control_available=%s",
+                self._state.value,
+                True,
+            )
+
+
+class ConfigManagerV2(_RuntimeController):
+    """Public manager API."""
+
+    DEFAULT_UPDATE_INTERVAL = 5
+    DEFAULT_WORK_INTERVAL = 2
+
+    def __init__(
+        self,
+        path: str,
+        log_manager: Optional[LogManager] = None,
+        management_settings: Optional[ManagementServerSettings] = None,
+        control_channel: Optional[ControlChannel] = None,
+    ):
+        self.logger = logging.getLogger(__name__)
+        self.path = path
+        self.config: Any = None
+        self.update_interval = _read_env_interval("UPDATE_INTERVAL", float(self.DEFAULT_UPDATE_INTERVAL))
+        self.work_interval = _read_env_interval("WORK_INTERVAL", float(self.DEFAULT_WORK_INTERVAL))
+
+        self._loader = ConfigLoader(path)
+        self._log_manager = log_manager or LogManager()
+        self._control_channel = control_channel
+        self._halt = asyncio.Event()
+        self._task: Optional[asyncio.Task] = None
+        self._loop: Optional[asyncio.AbstractEventLoop] = None
+        self._state = LifecycleState.IDLE
+        self._last_execute_error: Optional[str] = None
+        self._last_success_timestamp: Optional[float] = None
+
+        settings = management_settings or ManagementServerSettings(enabled=True)
+        self._management_settings = settings
+        self._health_timeout = settings.health_timeout
+        self._health_aggregator = HealthAggregator(
+            get_state=lambda: self._state,
+            get_last_error=lambda: self._last_execute_error,
+            get_last_success_timestamp=lambda: self._last_success_timestamp,
+            health_timeout=self._health_timeout,
+            get_app_health=self.get_health_status,
+        )
+        self._api_bridge = ManagementApiBridge(start_fn=self.start, stop_fn=self.stop)
+        self._control_bridge = ControlChannelBridge(
+            halt=self._halt,
+            get_state=lambda: self._state,
+            get_status=self._status_text,
+        )
+        self._management_server: Optional[ManagementServer] = None
+        if settings.enabled:
+            self._management_server = ManagementServer(
+                host=settings.host,
+                port=settings.port,
+                timeout=settings.timeout,
+                health_provider=self._health_aggregator.collect,
+                on_start=self._api_bridge.on_start,
+                on_stop=self._api_bridge.on_stop,
+            )
+        self.logger.warning(
+            "ConfigManagerV2.__init__ result: path=%s update_interval=%s work_interval=%s management_enabled=%s",
+            self.path,
+            self.update_interval,
+            self.work_interval,
+            self._management_server is not None,
+        )
+
+    def _apply_config(self, new_config: Any) -> None:
+        self.config = new_config
+        if isinstance(new_config, dict):
+            try:
+                self._log_manager.apply_config(new_config)
+            except Exception:  # noqa: BLE001
+                self.logger.exception("ConfigManagerV2._apply_config error while applying logging config")
+                raise
+        self.logger.warning(
+            "ConfigManagerV2._apply_config result: config_type=%s is_dict=%s",
+            type(new_config).__name__,
+            isinstance(new_config, dict),
+        )
+
+    async def _update_config(self) -> None:
+        try:
+            changed, new_config = await self._loader.load_if_changed()
+            if not changed:
+                self.logger.warning("ConfigManagerV2._update_config result: no changes")
+                return
+            self._apply_config(new_config)
+            self.logger.warning("ConfigManagerV2._update_config result: config updated")
+        except Exception as exc:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2._update_config error")
+            if self._loader.last_valid_config is None:
+                self.logger.warning(
+                    "ConfigManagerV2._update_config result: no fallback config available detail=%s",
+                    str(exc),
+                )
+                return
+            try:
+                self._apply_config(self._loader.last_valid_config)
+                self.logger.warning(
+                    "ConfigManagerV2._update_config result: fallback to last valid config applied",
+                )
+            except Exception:  # noqa: BLE001
+                self.logger.exception("ConfigManagerV2._update_config fallback error")
+
+    def execute(self) -> None:
+        """Override in subclasses."""
+
+    def get_health_status(self) -> HealthPayload:
+        return {"status": "ok"}
+
+    async def start(self) -> None:
+        if self._task is not None and not self._task.done():
+            self.logger.warning("ConfigManagerV2.start result: already running")
+            return
+        try:
+            self._loop = asyncio.get_running_loop()
+        except RuntimeError:
+            self.logger.exception("ConfigManagerV2.start error: must be called from within async context")
+            raise
+        self._task = asyncio.create_task(self._run(), name="config-manager-v2")
+        self._task.add_done_callback(self._on_runtime_task_done)
+        self.logger.warning("ConfigManagerV2.start result: background task started")
+
+    async def stop(self) -> None:
+        if self._task is None:
+            self.logger.warning("ConfigManagerV2.stop result: not running")
+            return
+        self._halt.set()
+        if asyncio.current_task() is self._task:
+            self.logger.warning("ConfigManagerV2.stop result: stop requested from runtime task")
+            return
+        try:
+            await self._task
+        except asyncio.CancelledError:
+            self.logger.warning("ConfigManagerV2.stop result: runtime task cancelled")
+        except Exception:  # noqa: BLE001
+            self.logger.exception("ConfigManagerV2.stop error while awaiting runtime task")
+            raise
+        finally:
+            self.logger.warning("ConfigManagerV2.stop result: completed")

src/config_manager/v2/core/scheduler.py

@@ -0,0 +1,54 @@
+"""Цикл воркера: повторяющийся вызов блокирующего execute() в потоке с паузой между итерациями.
+
+Поддерживает halt-событие для остановки, колбэки on_success/on_error для учёта ошибок и здоровья."""
+from __future__ import annotations
+
+import asyncio
+import logging
+from collections.abc import Callable
+from typing import Optional
+
+logger = logging.getLogger(__name__)
+
+
+class WorkerLoop:
+    def __init__(
+        self,
+        execute: Callable[[], None],
+        get_interval: Callable[[], int | float],
+        halt_event: asyncio.Event,
+        on_error: Optional[Callable[[Exception], None]] = None,
+        on_success: Optional[Callable[[], None]] = None,
+    ):
+        """Сохранить колбэки и примитивы синхронизации для выполнения воркера."""
+        self._execute = execute
+        self._get_interval = get_interval
+        self._halt_event = halt_event
+        self._on_error = on_error
+        self._on_success = on_success
+        logger.warning(
+            "WorkerLoop.__init__ result: execute=%s",
+            getattr(execute, "__name__", execute.__class__.__name__),
+        )
+
+    async def run(self) -> None:
+        """Вызывать execute циклически до запроса остановки."""
+        logger.warning("WorkerLoop.run result: started")
+        while not self._halt_event.is_set():
+            try:
+                await asyncio.to_thread(self._execute)
+                if self._on_success is not None:
+                    self._on_success()
+                logger.warning("WorkerLoop.run result: execute completed")
+            except Exception as exc:  # noqa: BLE001
+                logger.exception("WorkerLoop.run error during execute")
+                if self._on_error is not None:
+                    self._on_error(exc)
+                logger.warning("WorkerLoop.run result: execute failed")
+
+            timeout = max(self._get_interval(), 0.01)
+            try:
+                await asyncio.wait_for(self._halt_event.wait(), timeout=timeout)
+            except asyncio.TimeoutError:
+                continue
+        logger.warning("WorkerLoop.run result: stopped")

src/config_manager/v2/health.py

@@ -1,80 +0,0 @@
-from __future__ import annotations
-
-import asyncio
-import json
-from collections.abc import Awaitable, Callable
-from typing import Optional
-
-from .types import HealthPayload
-
-
-class HealthServer:
-    def __init__(
-        self,
-        host: str,
-        port: int,
-        path: str,
-        timeout: float,
-        health_provider: Callable[[], Awaitable[HealthPayload]],
-    ):
-        """Configure lightweight HTTP health server parameters and callback."""
-        self._host = host
-        self._port = port
-        self._path = path
-        self._timeout = timeout
-        self._health_provider = health_provider
-        self._server: Optional[asyncio.base_events.Server] = None
-
-    async def start(self) -> None:
-        """Start listening for healthcheck requests if not running."""
-        if self._server is not None:
-            return
-        self._server = await asyncio.start_server(self._handle_connection, self._host, self._port)
-
-    async def stop(self) -> None:
-        """Stop the health server and release the listening socket."""
-        if self._server is None:
-            return
-        self._server.close()
-        await self._server.wait_closed()
-        self._server = None
-
-    async def _handle_connection(self, reader: asyncio.StreamReader, writer: asyncio.StreamWriter) -> None:
-        """Process one HTTP connection and return JSON health payload."""
-        status_code = 404
-        payload: HealthPayload = {"status": "unhealthy", "detail": "not found"}
-
-        try:
-            request_line = await reader.readline()
-            parts = request_line.decode("utf-8", errors="ignore").strip().split(" ")
-            if len(parts) >= 2:
-                method, path = parts[0], parts[1]
-                if method == "GET" and path == self._path:
-                    status_code, payload = await self._build_health_response()
-        except Exception:  # noqa: BLE001
-            status_code = 500
-            payload = {"status": "unhealthy", "detail": "internal error"}
-
-        body = json.dumps(payload).encode("utf-8")
-        phrase = "OK" if status_code == 200 else "SERVICE UNAVAILABLE" if status_code == 503 else "NOT FOUND"
-        response = (
-            f"HTTP/1.1 {status_code} {phrase}\r\n"
-            "Content-Type: application/json\r\n"
-            f"Content-Length: {len(body)}\r\n"
-            "Connection: close\r\n"
-            "\r\n"
-        ).encode("utf-8") + body
-
-        writer.write(response)
-        await writer.drain()
-        writer.close()
-        await writer.wait_closed()
-
-    async def _build_health_response(self) -> tuple[int, HealthPayload]:
-        """Build HTTP status code and body from application health callback."""
-        try:
-            payload = await asyncio.wait_for(self._health_provider(), timeout=self._timeout)
-            status = payload.get("status", "unhealthy")
-            return (200, payload) if status == "ok" else (503, payload)
-        except Exception as exc:  # noqa: BLE001
-            return 503, {"status": "unhealthy", "detail": str(exc)}

src/config_manager/v2/management/__init__.py

@@ -0,0 +1,14 @@
+"""Management API: HTTP-сервер для /health и /actions/start|stop, адаптеры и сбор здоровья.
+
+Объединяет сервер, мосты к жизненному циклу и агрегатор здоровья для единого контракта API."""
+from .bridges import ControlChannelBridge, ManagementApiBridge
+from .health_aggregator import HealthAggregator
+from .management_server import HealthServer, ManagementServer
+
+__all__ = [
+    "ControlChannelBridge",
+    "HealthAggregator",
+    "HealthServer",
+    "ManagementApiBridge",
+    "ManagementServer",
+]

src/config_manager/v2/management/bridges.py

@@ -0,0 +1,90 @@
+"""Адаптеры, связывающие жизненный цикл менеджера с HTTP API и каналами управления.
+
+ManagementApiBridge отдаёт start/stop в HTTP; ControlChannelBridge — start/stop/status в Telegram и др."""
+from __future__ import annotations
+
+import asyncio
+import logging
+from collections.abc import Awaitable, Callable
+
+from ..types import LifecycleState
+
+logger = logging.getLogger(__name__)
+
+
+class ManagementApiBridge:
+    """Предоставляет start/stop жизненного цикла как async-колбэки для ManagementServer (/actions/start, /actions/stop)."""
+
+    def __init__(
+        self,
+        start_fn: Callable[[], Awaitable[None]],
+        stop_fn: Callable[[], Awaitable[None]],
+    ):
+        self._start_fn = start_fn
+        self._stop_fn = stop_fn
+        logger.warning("ManagementApiBridge.__init__ result: callbacks configured")
+
+    async def on_start(self) -> str:
+        """Выполнить start и вернуть сообщение для HTTP-ответа."""
+        try:
+            await self._start_fn()
+        except Exception:  # noqa: BLE001
+            logger.exception("ManagementApiBridge.on_start error")
+            raise
+        result = "start completed"
+        logger.warning("ManagementApiBridge.on_start result: %s", result)
+        return result
+
+    async def on_stop(self) -> str:
+        """Выполнить stop и вернуть сообщение для HTTP-ответа."""
+        try:
+            await self._stop_fn()
+        except Exception:  # noqa: BLE001
+            logger.exception("ManagementApiBridge.on_stop error")
+            raise
+        result = "stop completed"
+        logger.warning("ManagementApiBridge.on_stop result: %s", result)
+        return result
+
+
+class ControlChannelBridge:
+    """Предоставляет halt и status как обработчики start/stop/status для ControlChannel (например Telegram)."""
+
+    def __init__(
+        self,
+        halt: asyncio.Event,
+        get_state: Callable[[], LifecycleState],
+        get_status: Callable[[], Awaitable[str]],
+    ):
+        self._halt = halt
+        self._get_state = get_state
+        self._get_status = get_status
+        logger.warning("ControlChannelBridge.__init__ result: callbacks configured")
+
+    async def on_start(self) -> str:
+        """Обработать внешний start: сбросить halt; идемпотентно при уже running."""
+        if self._get_state() == LifecycleState.RUNNING:
+            result = "already running"
+            logger.warning("ControlChannelBridge.on_start result: %s", result)
+            return result
+        self._halt.clear()
+        result = "start signal accepted"
+        logger.warning("ControlChannelBridge.on_start result: %s", result)
+        return result
+
+    async def on_stop(self) -> str:
+        """Обработать внешний stop: установить halt."""
+        self._halt.set()
+        result = "stop signal accepted"
+        logger.warning("ControlChannelBridge.on_stop result: %s", result)
+        return result
+
+    async def on_status(self) -> str:
+        """Вернуть текущий текст статуса."""
+        try:
+            result = await self._get_status()
+        except Exception:  # noqa: BLE001
+            logger.exception("ControlChannelBridge.on_status error")
+            raise
+        logger.warning("ControlChannelBridge.on_status result: %s", result)
+        return result

src/config_manager/v2/management/health_aggregator.py

@@ -0,0 +1,67 @@
+"""Собирает состояние жизненного цикла и здоровья в один ответ для /health.
+
+Здоровье = был успешный execute() за последние health_timeout секунд; иначе unhealthy с деталью (ошибка или таймаут)."""
+from __future__ import annotations
+
+import logging
+import time
+from collections.abc import Callable
+
+from ..types import HealthPayload, LifecycleState
+
+logger = logging.getLogger(__name__)
+
+
+class HealthAggregator:
+    """Формирует ответ здоровья по времени последнего успешного execute() и таймауту."""
+
+    def __init__(
+        self,
+        get_state: Callable[[], LifecycleState],
+        get_last_error: Callable[[], str | None],
+        get_last_success_timestamp: Callable[[], float | None],
+        health_timeout: int,
+        get_app_health: Callable[[], HealthPayload],
+    ):
+        self._get_state = get_state
+        self._get_last_error = get_last_error
+        self._get_last_success_timestamp = get_last_success_timestamp
+        self._health_timeout = health_timeout
+        self._get_app_health = get_app_health
+        logger.warning("HealthAggregator.__init__ result: health_timeout=%s", self._health_timeout)
+
+    async def collect(self) -> HealthPayload:
+        """Вернуть ok, если был успешный execute() за последние health_timeout сек; иначе unhealthy. Всегда добавляем state."""
+        state = self._get_state()
+        state_value = state.value
+
+        # Только при state=RUNNING возможен status=ok; при остановке (STOPPING/STOPPED) сразу unhealthy.
+        if state != LifecycleState.RUNNING:
+            result = {"status": "unhealthy", "detail": f"state={state_value}", "state": state_value}
+            logger.warning("HealthAggregator.collect result: %s", result)
+            return result
+
+        last_success = self._get_last_success_timestamp()
+        now = time.monotonic()
+
+        if last_success is None:
+            detail = self._get_last_error() or "no successful run yet"
+            result = {"status": "unhealthy", "detail": detail, "state": state_value}
+            logger.warning("HealthAggregator.collect result: %s", result)
+            return result
+
+        if (now - last_success) > self._health_timeout:
+            detail = self._get_last_error() or f"no successful run within {self._health_timeout}s"
+            result = {"status": "unhealthy", "detail": detail, "state": state_value}
+            logger.warning("HealthAggregator.collect result: %s", result)
+            return result
+
+        result = self._get_app_health()
+        status = result.get("status", "unhealthy")
+        if status != "ok":
+            unhealthy = {"status": "unhealthy", "detail": result.get("detail", "app reported non-ok"), "state": state_value}
+            logger.warning("HealthAggregator.collect result: %s", unhealthy)
+            return unhealthy
+        healthy = {**result, "state": state_value}
+        logger.warning("HealthAggregator.collect result: %s", healthy)
+        return healthy

src/config_manager/v2/management/management_server.py

@@ -0,0 +1,264 @@
+"""Management HTTP API with /health, /actions/start and /actions/stop."""
+from __future__ import annotations
+
+import asyncio
+import json
+import logging
+from collections.abc import Awaitable, Callable
+from typing import Any, Optional
+
+from fastapi import FastAPI
+from fastapi.responses import JSONResponse
+from uvicorn import Config, Server
+
+from ..types import HealthPayload
+
+PATH_HEALTH = "/health"
+PATH_ACTION_START = "/actions/start"
+PATH_ACTION_STOP = "/actions/stop"
+logger = logging.getLogger(__name__)
+
+
+class UvicornServerRunner:
+    """Lifecycle wrapper around uvicorn Server."""
+
+    def __init__(self, host: str, port: int, timeout: int):
+        self._host = host
+        self._port = port
+        self._timeout = timeout
+        self._server: Optional[Server] = None
+        self._serve_task: Optional[asyncio.Task[None]] = None
+        self._bound_port: Optional[int] = None
+        logger.warning(
+            "UvicornServerRunner.__init__ result: host=%s port=%s timeout=%s",
+            self._host,
+            self._port,
+            self._timeout,
+        )
+
+    async def _raise_if_start_task_failed(self) -> None:
+        if self._serve_task is None or not self._serve_task.done():
+            return
+        try:
+            await self._serve_task
+        except SystemExit as exc:
+            raise RuntimeError(f"Management server exited during startup with code {exc.code}") from exc
+        except Exception as exc:  # noqa: BLE001
+            raise RuntimeError("Management server failed during startup") from exc
+        raise RuntimeError("Management server stopped unexpectedly during startup")
+
+    async def _wait_until_started(self) -> None:
+        if self._server is None:
+            raise RuntimeError("Management server is not initialized")
+        loop = asyncio.get_running_loop()
+        deadline = loop.time() + max(float(self._timeout), 1.0)
+        while not self._server.started:
+            await self._raise_if_start_task_failed()
+            if loop.time() >= deadline:
+                raise TimeoutError("Management server startup timed out")
+            await asyncio.sleep(0.05)
+
+    def _resolve_bound_port(self) -> int:
+        if self._server is None:
+            return self._port
+        servers = getattr(self._server, "servers", None)
+        if not servers:
+            return self._port
+        sockets = getattr(servers[0], "sockets", None)
+        if not sockets:
+            return self._port
+        sockname = sockets[0].getsockname()
+        if isinstance(sockname, tuple) and len(sockname) > 1:
+            return int(sockname[1])
+        return self._port
+
+    async def _cleanup_start_failure(self) -> None:
+        if self._server is not None:
+            self._server.should_exit = True
+        if self._serve_task is not None:
+            try:
+                await self._serve_task
+            except BaseException:  # noqa: BLE001
+                logger.exception("UvicornServerRunner._cleanup_start_failure error")
+        self._server = None
+        self._serve_task = None
+        self._bound_port = None
+        logger.warning("UvicornServerRunner._cleanup_start_failure result: state reset")
+
+    async def start(self, app: FastAPI) -> None:
+        if self._serve_task is not None and not self._serve_task.done():
+            logger.warning("UvicornServerRunner.start result: already running")
+            return
+        if self._serve_task is not None and self._serve_task.done():
+            self._serve_task = None
+        try:
+            config = Config(app=app, host=self._host, port=self._port, log_level="warning")
+            self._server = Server(config)
+            self._serve_task = asyncio.create_task(self._server.serve(), name="management-server-serve")
+            await self._wait_until_started()
+            self._bound_port = self._resolve_bound_port()
+            logger.warning(
+                "UvicornServerRunner.start result: running host=%s requested_port=%s bound_port=%s",
+                self._host,
+                self._port,
+                self._bound_port,
+            )
+        except Exception:
+            logger.exception("UvicornServerRunner.start error")
+            await self._cleanup_start_failure()
+            raise
+
+    async def stop(self) -> None:
+        if self._server is None or self._serve_task is None:
+            logger.warning("UvicornServerRunner.stop result: already stopped")
+            return
+        self._server.should_exit = True
+        try:
+            await self._serve_task
+        except BaseException:  # noqa: BLE001
+            logger.exception("UvicornServerRunner.stop error")
+            raise
+        finally:
+            self._server = None
+            self._serve_task = None
+            self._bound_port = None
+        logger.warning("UvicornServerRunner.stop result: stopped")
+
+    @property
+    def port(self) -> int:
+        result = self._bound_port if self._bound_port is not None else self._port
+        logger.warning("UvicornServerRunner.port result: %s", result)
+        return result
+
+
+class ManagementServer:
+    """Management API endpoints and callback adapters."""
+
+    def __init__(
+        self,
+        host: str,
+        port: int,
+        timeout: int,
+        health_provider: Callable[[], Awaitable[HealthPayload]],
+        on_start: Optional[Callable[[], Awaitable[str]]] = None,
+        on_stop: Optional[Callable[[], Awaitable[str]]] = None,
+    ):
+        self._timeout = timeout
+        self._health_provider = health_provider
+        self._on_start = on_start
+        self._on_stop = on_stop
+        self._runner = UvicornServerRunner(host=host, port=port, timeout=timeout)
+        self._app = self._create_app()
+        logger.warning(
+            "ManagementServer.__init__ result: host=%s port=%s timeout=%s",
+            host,
+            port,
+            timeout,
+        )
+
+    def _create_app(self) -> FastAPI:
+        app = FastAPI(title="Config Manager Management API")
+
+        @app.get(PATH_HEALTH)
+        async def health() -> JSONResponse:
+            return await self._health_response()
+
+        @app.get(PATH_ACTION_START)
+        @app.post(PATH_ACTION_START)
+        async def action_start() -> JSONResponse:
+            return await self._action_response("start", self._on_start)
+
+        @app.get(PATH_ACTION_STOP)
+        @app.post(PATH_ACTION_STOP)
+        async def action_stop() -> JSONResponse:
+            return await self._action_response("stop", self._on_stop)
+
+        logger.warning(
+            "ManagementServer._create_app result: routes=%s,%s,%s",
+            PATH_HEALTH,
+            PATH_ACTION_START,
+            PATH_ACTION_STOP,
+        )
+        return app
+
+    async def _health_response(self) -> JSONResponse:
+        try:
+            payload = await asyncio.wait_for(self._health_provider(), timeout=self._timeout)
+            status_code = 200 if payload.get("status", "unhealthy") == "ok" else 503
+            logger.warning(
+                "ManagementServer._health_response result: status_code=%s payload=%s",
+                status_code,
+                payload,
+            )
+            return JSONResponse(content=payload, status_code=status_code)
+        except Exception as exc:  # noqa: BLE001
+            logger.exception("ManagementServer._health_response error")
+            return JSONResponse(content={"status": "unhealthy", "detail": str(exc)}, status_code=503)
+
+    async def _action_response(
+        self,
+        action: str,
+        callback: Optional[Callable[[], Awaitable[str]]],
+    ) -> JSONResponse:
+        if callback is None:
+            logger.warning(
+                "ManagementServer._action_response result: action=%s status_code=404 detail=handler not configured",
+                action,
+            )
+            return JSONResponse(
+                content={"status": "error", "detail": f"{action} handler is not configured"},
+                status_code=404,
+            )
+        try:
+            detail = await callback()
+            if not detail:
+                detail = f"{action} action accepted"
+            logger.warning(
+                "ManagementServer._action_response result: action=%s status_code=200 detail=%s",
+                action,
+                detail,
+            )
+            return JSONResponse(content={"status": "ok", "detail": detail}, status_code=200)
+        except Exception as exc:  # noqa: BLE001
+            logger.exception("ManagementServer._action_response error: action=%s", action)
+            return JSONResponse(content={"status": "error", "detail": str(exc)}, status_code=500)
+
+    def _build_health_response(self) -> Awaitable[tuple[int, HealthPayload]]:
+        async def _run() -> tuple[int, HealthPayload]:
+            response = await self._health_response()
+            body: Any = response.body
+            if isinstance(body, bytes):
+                body = json.loads(body.decode("utf-8"))
+            logger.warning(
+                "ManagementServer._build_health_response result: status_code=%s payload=%s",
+                response.status_code,
+                body,
+            )
+            return response.status_code, body
+
+        return _run()
+
+    async def start(self) -> None:
+        try:
+            await self._runner.start(self._app)
+            logger.warning("ManagementServer.start result: started")
+        except Exception:  # noqa: BLE001
+            logger.exception("ManagementServer.start error")
+            raise
+
+    async def stop(self) -> None:
+        try:
+            await self._runner.stop()
+            logger.warning("ManagementServer.stop result: stopped")
+        except BaseException:  # noqa: BLE001
+            logger.exception("ManagementServer.stop error")
+            raise
+
+    @property
+    def port(self) -> int:
+        result = self._runner.port
+        logger.warning("ManagementServer.port result: %s", result)
+        return result
+
+
+HealthServer = ManagementServer

src/config_manager/v2/manager.py

@@ -1,256 +0,0 @@
-from __future__ import annotations
-
-import asyncio
-import logging
-from collections.abc import Awaitable
-from typing import Any, Optional
-
-from ..v1.log_manager import LogManager
-from .config_loader import ConfigLoader
-from .control.base import ControlChannel
-from .health import HealthServer
-from .scheduler import WorkerLoop
-from .types import HealthPayload, HealthServerSettings, LifecycleState
-
-
-class ConfigManagerV2:
-    DEFAULT_UPDATE_INTERVAL = 5.0
-    DEFAULT_WORK_INTERVAL = 2.0
-
-    def __init__(
-        self,
-        path: str,
-        log_manager: Optional[LogManager] = None,
-        health_settings: Optional[HealthServerSettings] = None,
-        control_channel: Optional[ControlChannel] = None,
-    ):
-        """Initialize manager subsystems and runtime state."""
-        self.path = path
-        self.config: Any = None
-        self.update_interval = self.DEFAULT_UPDATE_INTERVAL
-        self.work_interval = self.DEFAULT_WORK_INTERVAL
-
-        self._loader = ConfigLoader(path)
-        self._log_manager = log_manager or LogManager()
-        self._control_channel = control_channel
-
-        self._halt = asyncio.Event()
-        self._task: Optional[asyncio.Task] = None
-        self._loop: Optional[asyncio.AbstractEventLoop] = None
-
-        self._state = LifecycleState.IDLE
-        self._last_execute_error: Optional[str] = None
-
-        self._health_settings = health_settings or HealthServerSettings(enabled=False)
-        self._health_server: Optional[HealthServer] = None
-        if self._health_settings.enabled:
-            self._health_server = HealthServer(
-                host=self._health_settings.host,
-                port=self._health_settings.port,
-                path=self._health_settings.path,
-                timeout=self._health_settings.timeout,
-                health_provider=self._collect_health,
-            )
-
-        self.logger = logging.getLogger(__name__)
-
-    def _read_file_sync(self) -> str:
-        """Read config file synchronously via the shared loader."""
-        return self._loader._read_file_sync()
-
-    async def _read_file_async(self) -> str:
-        """Read config file asynchronously via a worker thread."""
-        return await self._loader.read_file_async()
-
-    def _parse_config(self, data: str) -> Any:
-        """Parse raw config text to a Python object."""
-        return self._loader.parse_config(data)
-
-    def _update_intervals_from_config(self) -> None:
-        """Refresh work and update intervals from current config."""
-        if not isinstance(self.config, dict):
-            return
-
-        upd = self.config.get("update_interval")
-        wrk = self.config.get("work_interval")
-
-        if isinstance(upd, (int, float)) and upd > 0:
-            self.update_interval = float(upd)
-        else:
-            self.update_interval = self.DEFAULT_UPDATE_INTERVAL
-
-        if isinstance(wrk, (int, float)) and wrk > 0:
-            self.work_interval = float(wrk)
-        else:
-            self.work_interval = self.DEFAULT_WORK_INTERVAL
-
-    async def _update_config(self) -> None:
-        """Reload config and apply new settings when content changes."""
-        try:
-            changed, new_config = await self._loader.load_if_changed()
-            if not changed:
-                return
-
-            self.config = new_config
-            self._update_intervals_from_config()
-            if isinstance(new_config, dict):
-                self._log_manager.apply_config(new_config)
-        except Exception as exc:  # noqa: BLE001
-            # Keep current config untouched and continue with last valid settings.
-            self.logger.error("Error reading/parsing config file: %s", exc)
-            if self._loader.last_valid_config is not None:
-                self.config = self._loader.last_valid_config
-                self._update_intervals_from_config()
-
-    def execute(self) -> None:
-        """Override in subclass to implement one unit of blocking work."""
-
-    def get_health_status(self) -> HealthPayload:
-        """Return application-specific health payload for /health."""
-        return {"status": "ok"}
-
-    async def _collect_health(self) -> HealthPayload:
-        """Aggregate lifecycle and app status into one health result."""
-        if self._state not in {LifecycleState.RUNNING, LifecycleState.STOPPING}:
-            return {"status": "unhealthy", "detail": f"state={self._state.value}"}
-
-        if self._last_execute_error is not None:
-            return {"status": "unhealthy", "detail": self._last_execute_error}
-
-        result = self.get_health_status()
-        status = result.get("status", "unhealthy")
-        if status not in {"ok", "degraded", "unhealthy"}:
-            return {"status": "unhealthy", "detail": "invalid health status"}
-        return result
-
-    def _on_execute_success(self) -> None:
-        """Clear the last execution error marker after successful run."""
-        self._last_execute_error = None
-
-    def _on_execute_error(self, exc: Exception) -> None:
-        """Store and log execution failure details for health reporting."""
-        self._last_execute_error = str(exc)
-        self.logger.error("Execution error: %s", exc)
-
-    async def _worker_loop(self) -> None:
-        """Run execute() repeatedly until shutdown is requested."""
-        worker = WorkerLoop(
-            execute=self.execute,
-            get_interval=lambda: self.work_interval,
-            halt_event=self._halt,
-            on_error=self._on_execute_error,
-            on_success=self._on_execute_success,
-        )
-        await worker.run()
-
-    async def _periodic_update_loop(self) -> None:
-        """Periodically check config file for updates until shutdown."""
-        while not self._halt.is_set():
-            await self._update_config()
-            try:
-                await asyncio.wait_for(self._halt.wait(), timeout=max(self.update_interval, 0.05))
-            except asyncio.TimeoutError:
-                continue
-
-    async def _status_text(self) -> str:
-        """Build human-readable runtime status for control channels."""
-        health = await self._collect_health()
-        detail = health.get("detail")
-        if detail:
-            return f"state={self._state.value}; health={health['status']}; detail={detail}"
-        return f"state={self._state.value}; health={health['status']}"
-
-    async def _control_start(self) -> str:
-        """Handle external start command for control channels."""
-        if self._state == LifecycleState.RUNNING:
-            return "already running"
-        self._halt.clear()
-        return "start signal accepted"
-
-    async def _control_stop(self) -> str:
-        """Handle external stop command for control channels."""
-        self._halt.set()
-        return "stop signal accepted"
-
-    async def _control_status(self) -> str:
-        """Handle external status command for control channels."""
-        return await self._status_text()
-
-    async def _start_control_channel(self) -> None:
-        """Start configured control channel with bound command handlers."""
-        if self._control_channel is None:
-            return
-        await self._control_channel.start(self._control_start, self._control_stop, self._control_status)
-
-    async def _stop_control_channel(self) -> None:
-        """Stop configured control channel if it is active."""
-        if self._control_channel is None:
-            return
-        await self._control_channel.stop()
-
-    async def _run(self) -> None:
-        """Run manager lifecycle and coordinate all background tasks."""
-        self._state = LifecycleState.STARTING
-        self._halt.clear()
-        await self._update_config()
-
-        if self._health_server is not None:
-            await self._health_server.start()
-        await self._start_control_channel()
-
-        self._state = LifecycleState.RUNNING
-        self.logger.info("ConfigManagerV2 started")
-
-        tasks = [
-            asyncio.create_task(self._worker_loop(), name="v2-worker-loop"),
-            asyncio.create_task(self._periodic_update_loop(), name="v2-config-loop"),
-        ]
-
-        try:
-            await asyncio.gather(*tasks)
-        except asyncio.CancelledError:
-            raise
-        finally:
-            self._state = LifecycleState.STOPPING
-            self._halt.set()
-            for task in tasks:
-                task.cancel()
-            await asyncio.gather(*tasks, return_exceptions=True)
-            await self._stop_control_channel()
-            if self._health_server is not None:
-                await self._health_server.stop()
-            self._state = LifecycleState.STOPPED
-            self.logger.info("ConfigManagerV2 stopped")
-
-    async def start(self) -> None:
-        """Start manager lifecycle from an active asyncio context."""
-        if self._task is not None and not self._task.done():
-            self.logger.warning("ConfigManagerV2 is already running")
-            return
-
-        try:
-            self._loop = asyncio.get_running_loop()
-        except RuntimeError:
-            self.logger.error("start() must be called from within an async context")
-            raise
-
-        self._task = asyncio.create_task(self._run(), name="config-manager-v2")
-        try:
-            await self._task
-        finally:
-            self._task = None
-
-    async def stop(self) -> None:
-        """Request graceful shutdown and wait for manager completion."""
-        if self._task is None:
-            self.logger.warning("ConfigManagerV2 is not running")
-            return
-
-        self._halt.set()
-        if asyncio.current_task() is self._task:
-            return
-
-        try:
-            await self._task
-        except asyncio.CancelledError:
-            pass

src/config_manager/v2/scheduler.py

@@ -1,39 +0,0 @@
-from __future__ import annotations
-
-import asyncio
-from collections.abc import Callable
-from typing import Optional
-
-
-class WorkerLoop:
-    def __init__(
-        self,
-        execute: Callable[[], None],
-        get_interval: Callable[[], float],
-        halt_event: asyncio.Event,
-        on_error: Optional[Callable[[Exception], None]] = None,
-        on_success: Optional[Callable[[], None]] = None,
-    ):
-        """Store callbacks and synchronization primitives for worker execution."""
-        self._execute = execute
-        self._get_interval = get_interval
-        self._halt_event = halt_event
-        self._on_error = on_error
-        self._on_success = on_success
-
-    async def run(self) -> None:
-        """Run execute repeatedly until halt is requested."""
-        while not self._halt_event.is_set():
-            try:
-                await asyncio.to_thread(self._execute)
-                if self._on_success is not None:
-                    self._on_success()
-            except Exception as exc:  # noqa: BLE001
-                if self._on_error is not None:
-                    self._on_error(exc)
-
-            timeout = max(self._get_interval(), 0.01)
-            try:
-                await asyncio.wait_for(self._halt_event.wait(), timeout=timeout)
-            except asyncio.TimeoutError:
-                continue

src/config_manager/v2/types.py

@@ -1,3 +1,6 @@
+"""Общие типы V2: состояние здоровья, жизненного цикла и настройки management-сервера.
+
+Используются в core, management и control для единообразных контрактов."""
 from __future__ import annotations
 
 from dataclasses import dataclass
@@ -11,6 +14,8 @@ HealthState = Literal["ok", "degraded", "unhealthy"]
 class HealthPayload(TypedDict, total=False):
     status: HealthState
     detail: str
+    state: str
+    """Текущее состояние жизненного цикла (idle/starting/running/stopping/stopped)."""
 
 
 class LifecycleState(str, Enum):
@@ -22,9 +27,16 @@ class LifecycleState(str, Enum):
 
 
 @dataclass
-class HealthServerSettings:
+class ManagementServerSettings:
+    """Настройки management HTTP-сервера и healthcheck (один объект на оба)."""
     enabled: bool = False
     host: str = "0.0.0.0"
     port: int = 8000
-    path: str = "/health"
-    timeout: float = 3.0
+    timeout: int = 3
+    """Таймаут запроса health (секунды)."""
+    health_timeout: int = 30
+    """Секунды без успешного execute(), после которых health = unhealthy."""
+
+
+# Backward-compatible alias.
+HealthServerSettings = ManagementServerSettings

tests/config.yaml

@@ -9,8 +9,7 @@ log:
   formatters:
     standard:
       format: '%(asctime)s %(module)15s [%(levelname)8s]: %(message)s'
-    telegram:
-      format: '%(message)s'
+
 
   handlers:
     console:
@@ -26,28 +25,20 @@ log:
       filename: logs/log.log
       mode: a
       maxBytes: 500000
-      backupCount: 15
-
-    #telegram:
-    #  level: CRITICAL
-    #  formatter: telegram
-    #  class: logging_telegram_handler.TelegramHandler
-    #  chat_id: 211945135
-    #  alias: "PDC"
-
+      backupCount: 3
 
 # -- Логгеры --
-  loggers:
-    '':
-      handlers: [console, file]
-      level: INFO
-      propagate: False
+  root:
+    handlers: [console, file]
+    level: INFO
 
+  loggers:
     __main__:
       handlers: [console, file]
       level: DEBUG
       propagate: False
 
-    config_manager:
+    config_manager.src.config_manager.v2.manager:
       handlers: [console, file]
       level: DEBUG
+

tests/test_app.py

@@ -1,34 +1,59 @@
-#import os
-#os.chdir(os.path.dirname(__file__))
+# import os
+# os.chdir(os.path.dirname(__file__))
+
+import asyncio
+import logging
+from pathlib import Path
 
 from config_manager import ConfigManager
-import logging
-import asyncio
-from typing import Optional
-
-
+from config_manager.v1.log_manager import LogManager
+from config_manager.v2 import ManagementServerSettings
 
 logger = logging.getLogger()
 
+# Таймаут health: без успешного execute() дольше этого времени — unhealthy.
+HEALTH_TIMEOUT = 3.0
+
+
 class MyApp(ConfigManager):
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.iter = 0
 
     def execute(self) -> None:
-        logger.info(f"current iteration {self.iter}")
+        """Успешный прогон сбрасывает таймер health (обновляет время последнего успеха)."""
+        #logger.critical("current iteration %s", self.iter)
+        #logger.error("current iteration %s", self.iter)
+        logger.warning("current iteration %s", self.iter)
+        #logger.info("current iteration %s", self.iter)
+        #logger.debug("current iteration %s", self.iter)
         self.iter += 1
 
-async def main():
-    app = MyApp("config.yaml")
-    logger.info("App started")
-    await app.start()
 
-    logger.info("App finished")
+async def main() -> None:
+    log_manager = LogManager()
+    # Один объект: и HTTP management-сервер (enabled, port), и health (health_timeout).
+    management_settings = ManagementServerSettings(
+        enabled=True,
+        port=8000,
+        health_timeout=HEALTH_TIMEOUT,
+    )
+    config_path = Path(__file__).parent / "config.yaml"
+    print(config_path)
+    app = MyApp(
+        str(config_path),
+        log_manager=log_manager,
+        management_settings=management_settings,
+    )
+    logger.info("App starting (health_timeout=%s)", HEALTH_TIMEOUT)
+    # Менеджер запускаем в фоне (start() не возвращает управление до stop).
+    asyncio.create_task(app.start())
 
+    logger.info("App running; Ctrl+C to stop")
     while True:
         await asyncio.sleep(1)
 
-if __name__ == "__main__":
-    asyncio.run(main())
 
+if __name__ == "__main__":
+    logging.basicConfig(level=logging.INFO)
+    asyncio.run(main())

tests/v2/test_config_reload_fallback.py

@@ -1,9 +1,12 @@
 import asyncio
 
-from config_manager.v2 import ConfigManagerV2
+from config_manager.v2 import ConfigManagerV2, ManagementServerSettings
 
 
 class ReloadApp(ConfigManagerV2):
+    DEFAULT_UPDATE_INTERVAL = 0.05
+    DEFAULT_WORK_INTERVAL = 0.2
+
     def execute(self) -> None:
         return
 
@@ -11,9 +14,9 @@ class ReloadApp(ConfigManagerV2):
 def test_invalid_config_keeps_last_valid(tmp_path):
     async def scenario() -> None:
         cfg = tmp_path / "config.yaml"
-        cfg.write_text("work_interval: 0.2\nupdate_interval: 0.05\n", encoding="utf-8")
+        cfg.write_text("log: {}\n", encoding="utf-8")
 
-        app = ReloadApp(str(cfg))
+        app = ReloadApp(str(cfg), management_settings=ManagementServerSettings(enabled=False))
         runner = asyncio.create_task(app.start())
 
         await asyncio.sleep(0.12)

tests/v2/test_contract_v2.py

@@ -1,9 +1,12 @@
 import asyncio
 
-from config_manager.v2 import ConfigManagerV2
+from config_manager.v2 import ConfigManagerV2, ManagementServerSettings
 
 
 class DemoApp(ConfigManagerV2):
+    DEFAULT_UPDATE_INTERVAL = 0.05
+    DEFAULT_WORK_INTERVAL = 0.05
+
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.calls = 0
@@ -15,9 +18,9 @@ class DemoApp(ConfigManagerV2):
 def test_execute_loop_runs(tmp_path):
     async def scenario() -> None:
         cfg = tmp_path / "config.yaml"
-        cfg.write_text("work_interval: 0.05\nupdate_interval: 0.05\n", encoding="utf-8")
+        cfg.write_text("log: {}\n", encoding="utf-8")
 
-        app = DemoApp(str(cfg))
+        app = DemoApp(str(cfg), management_settings=ManagementServerSettings(enabled=False))
         runner = asyncio.create_task(app.start())
 
         await asyncio.sleep(0.18)

tests/v2/test_control_channel.py

@@ -1,6 +1,6 @@
 import asyncio
 
-from config_manager.v2 import ConfigManagerV2
+from config_manager.v2 import ConfigManagerV2, ManagementServerSettings
 from config_manager.v2.control.base import ControlChannel, StartHandler, StatusHandler, StopHandler
 
 
@@ -23,6 +23,9 @@ class DummyControlChannel(ControlChannel):
 
 
 class ControlledApp(ConfigManagerV2):
+    DEFAULT_UPDATE_INTERVAL = 0.05
+    DEFAULT_WORK_INTERVAL = 0.05
+
     def execute(self) -> None:
         return
 
@@ -30,10 +33,14 @@ class ControlledApp(ConfigManagerV2):
 def test_control_channel_can_stop_manager(tmp_path):
     async def scenario() -> None:
         cfg = tmp_path / "config.yaml"
-        cfg.write_text("work_interval: 0.05\nupdate_interval: 0.05\n", encoding="utf-8")
+        cfg.write_text("log: {}\n", encoding="utf-8")
 
         channel = DummyControlChannel()
-        app = ControlledApp(str(cfg), control_channel=channel)
+        app = ControlledApp(
+            str(cfg),
+            control_channel=channel,
+            management_settings=ManagementServerSettings(enabled=False),
+        )
 
         runner = asyncio.create_task(app.start())
         await asyncio.sleep(0.12)
@@ -49,6 +56,6 @@ def test_control_channel_can_stop_manager(tmp_path):
         assert "stop signal accepted" in stop_text
 
         await runner
-        assert channel.stopped is True
+        # Менеджер при остановке не вызывает control_channel.stop() (канал остаётся доступным)
 
     asyncio.run(scenario())

tests/v2/test_health_endpoint.py

@@ -1,6 +1,7 @@
 import asyncio
+import json
 
-from config_manager.v2.health import HealthServer
+from config_manager.v2.management import ManagementServer
 
 
 def test_health_mapping_ok_to_200():
@@ -8,10 +9,9 @@ def test_health_mapping_ok_to_200():
         return {"status": "ok"}
 
     async def scenario() -> None:
-        server = HealthServer(
+        server = ManagementServer(
             host="127.0.0.1",
             port=8000,
-            path="/health",
             timeout=0.2,
             health_provider=provider,
         )
@@ -27,10 +27,9 @@ def test_health_mapping_unhealthy_to_503():
         return {"status": "unhealthy", "detail": "worker failed"}
 
     async def scenario() -> None:
-        server = HealthServer(
+        server = ManagementServer(
             host="127.0.0.1",
             port=8000,
-            path="/health",
             timeout=0.2,
             health_provider=provider,
         )
@@ -39,3 +38,63 @@ def test_health_mapping_unhealthy_to_503():
         assert payload["status"] == "unhealthy"
 
     asyncio.run(scenario())
+
+
+def test_action_routes_call_callbacks():
+    events: list[str] = []
+
+    async def provider():
+        return {"status": "ok"}
+
+    async def on_start() -> str:
+        events.append("start")
+        return "start accepted"
+
+    async def on_stop() -> str:
+        events.append("stop")
+        return "stop accepted"
+
+    async def request(port: int, path: str) -> tuple[int, dict[str, str]]:
+        reader, writer = await asyncio.open_connection("127.0.0.1", port)
+        writer.write(
+            f"GET {path} HTTP/1.1\r\nHost: 127.0.0.1\r\nConnection: close\r\n\r\n".encode("utf-8")
+        )
+        await writer.drain()
+        raw = await reader.read()
+        writer.close()
+        await writer.wait_closed()
+
+        header, body = raw.split(b"\r\n\r\n", maxsplit=1)
+        status_code = int(header.split(b" ")[1])
+        payload = json.loads(body.decode("utf-8"))
+        return status_code, payload
+
+    async def scenario() -> None:
+        server = ManagementServer(
+            host="127.0.0.1",
+            port=0,
+            timeout=0.2,
+            health_provider=provider,
+            on_start=on_start,
+            on_stop=on_stop,
+        )
+        await server.start()
+        try:
+            port = server.port
+            assert port > 0
+
+            start_code, start_payload = await request(port, "/actions/start")
+            stop_code, stop_payload = await request(port, "/actions/stop")
+        finally:
+            await server.stop()
+
+        assert start_code == 200
+        assert start_payload["status"] == "ok"
+        assert start_payload["detail"] == "start accepted"
+
+        assert stop_code == 200
+        assert stop_payload["status"] == "ok"
+        assert stop_payload["detail"] == "stop accepted"
+        assert events == ["start", "stop"]
+
+    asyncio.run(scenario())

tests/v2/test_stop_graceful.py

@@ -2,10 +2,13 @@ import asyncio
 import threading
 import time
 
-from config_manager.v2 import ConfigManagerV2
+from config_manager.v2 import ConfigManagerV2, ManagementServerSettings
 
 
 class BlockingApp(ConfigManagerV2):
+    DEFAULT_UPDATE_INTERVAL = 0.05
+    DEFAULT_WORK_INTERVAL = 0.05
+
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.started_event = threading.Event()
@@ -23,9 +26,9 @@ class BlockingApp(ConfigManagerV2):
 def test_stop_waits_for_active_execute_and_prevents_next_run(tmp_path):
     async def scenario() -> None:
         cfg = tmp_path / "config.yaml"
-        cfg.write_text("work_interval: 0.05\nupdate_interval: 0.05\n", encoding="utf-8")
+        cfg.write_text("log: {}\n", encoding="utf-8")
 
-        app = BlockingApp(str(cfg))
+        app = BlockingApp(str(cfg), management_settings=ManagementServerSettings(enabled=False))
         runner = asyncio.create_task(app.start())
 
         started = await asyncio.to_thread(app.started_event.wait, 1.0)