digitalkin 0.2.25rc0__py3-none-any.whl → 0.3.2.dev14__py3-none-any.whl

digitalkin/modules/_base_module.py

@@ -1,42 +1,29 @@
 """BaseModule is the abstract base for all modules in the DigitalKin SDK."""
 
 import asyncio
-import contextlib
 import json
+import os
 from abc import ABC, abstractmethod
 from collections.abc import Callable, Coroutine
 from typing import Any, ClassVar, Generic
 
-from pydantic import BaseModel
-
+from digitalkin.grpc_servers.utils.utility_schema_extender import UtilitySchemaExtender
 from digitalkin.logger import logger
-from digitalkin.models.module import (
+from digitalkin.models.module.module import ModuleCodeModel, ModuleStatus
+from digitalkin.models.module.module_context import ModuleContext
+from digitalkin.models.module.module_types import (
+    DataModel,
     InputModelT,
-    ModuleStatus,
     OutputModelT,
     SecretModelT,
     SetupModelT,
 )
-from digitalkin.models.module.module_context import ModuleContext
+from digitalkin.models.module.utility import EndOfStreamOutput, ModuleStartInfoOutput, UtilityProtocol
+from digitalkin.models.services.storage import BaseRole
 from digitalkin.modules.trigger_handler import TriggerHandler
-from digitalkin.services.agent.agent_strategy import AgentStrategy
-from digitalkin.services.cost.cost_strategy import CostStrategy
-from digitalkin.services.filesystem.filesystem_strategy import FilesystemStrategy
-from digitalkin.services.identity.identity_strategy import IdentityStrategy
-from digitalkin.services.registry.registry_strategy import RegistryStrategy
 from digitalkin.services.services_config import ServicesConfig, ServicesStrategy
-from digitalkin.services.snapshot.snapshot_strategy import SnapshotStrategy
-from digitalkin.services.storage.storage_strategy import StorageStrategy
-from digitalkin.utils.llm_ready_schema import llm_ready_schema
 from digitalkin.utils.package_discover import ModuleDiscoverer
-
-
-class ModuleCodeModel(BaseModel):
-    """typed error/code model."""
-
-    code: str
-    message: str
-    short_description: str
+from digitalkin.utils.schema_splitter import SchemaSplitter
 
 
 class BaseModule(  # noqa: PLR0904
@@ -63,37 +50,44 @@ class BaseModule(  # noqa: PLR0904
     triggers_discoverer: ClassVar[ModuleDiscoverer]
 
     # service config params
-    services_config_strategies: ClassVar[dict[str, ServicesStrategy | None]]
-    services_config_params: ClassVar[dict[str, dict[str, Any | None] | None]]
+    services_config_strategies: ClassVar[dict[str, ServicesStrategy | None]] = {}
+    services_config_params: ClassVar[dict[str, dict[str, Any | None] | None]] = {}
     services_config: ServicesConfig
 
-    # services list
-    agent: AgentStrategy
-    cost: CostStrategy
-    filesystem: FilesystemStrategy
-    identity: IdentityStrategy
-    registry: RegistryStrategy
-    snapshot: SnapshotStrategy
-    storage: StorageStrategy
-
-    # runtime params
-    job_id: str
-    mission_id: str
-    setup_id: str
-    setup_version_id: str
-    _status: ModuleStatus
-    _task: asyncio.Task | None
-
-    def _init_strategies(self) -> None:
-        """Initialize the services configuration."""
-        for service_name in self.services_config.valid_strategy_names():
-            service = self.services_config.init_strategy(
+    @classmethod
+    def get_module_id(cls) -> str:
+        """Get the module ID from environment variable or metadata.
+
+        Returns:
+            The module_id from DIGITALKIN_MODULE_ID env var, or metadata module_id,
+            or "unknown" if neither exists.
+        """
+        return os.environ.get("DIGITALKIN_MODULE_ID") or cls.metadata.get("module_id", "unknown")
+
+    def _init_strategies(self, mission_id: str, setup_id: str, setup_version_id: str) -> dict[str, Any]:
+        """Initialize the services configuration.
+
+        Returns:
+            dict of services with name: Strategy
+                agent: AgentStrategy
+                cost: CostStrategy
+                filesystem: FilesystemStrategy
+                identity: IdentityStrategy
+                registry: RegistryStrategy
+                snapshot: SnapshotStrategy
+                storage: StorageStrategy
+                user_profile: UserProfileStrategy
+        """
+        logger.debug("Service initialisation: %s", self.services_config_strategies.keys())
+        return {
+            service_name: self.services_config.init_strategy(
                 service_name,
-                self.mission_id,
-                self.setup_id,
-                self.setup_version_id,
+                mission_id,
+                setup_id,
+                setup_version_id,
             )
-            setattr(self, service_name, service)
+            for service_name in self.services_config.valid_strategy_names()
+        }
 
     def __init__(
         self,
@@ -103,22 +97,19 @@ class BaseModule(  # noqa: PLR0904
         setup_version_id: str,
     ) -> None:
         """Initialize the module."""
-        self.job_id: str = job_id
-        self.mission_id: str = mission_id
-        # Setup reference needed for the overall Kin scope as the filesystem context
-        self.setup_id: str = setup_id
-        # SetupVersion reference needed for the precise Kin scope as the cost
-        self.setup_version_id: str = setup_version_id
         self._status = ModuleStatus.CREATED
-        self._task: asyncio.Task | None = None
-        # Initialize services configuration
-        self._init_strategies()
 
         # Initialize minimum context
         self.context = ModuleContext(
-            storage=self.storage,
-            cost=self.cost,
-            filesystem=self.filesystem,
+            # Initialize services configuration
+            **self._init_strategies(mission_id, setup_id, setup_version_id),
+            session={
+                "setup_id": setup_id,
+                "mission_id": mission_id,
+                "setup_version_id": setup_version_id,
+                "job_id": job_id,
+            },
+            callbacks={"logger": logger},
         )
 
     @property
@@ -131,91 +122,134 @@ class BaseModule(  # noqa: PLR0904
         return self._status
 
     @classmethod
-    def get_secret_format(cls, *, llm_format: bool) -> str:
+    async def get_secret_format(cls, *, llm_format: bool) -> str:
         """Get the JSON schema of the secret format model.
 
-        Raises:
-            NotImplementedError: If the `secret_format` is not defined.
+        Args:
+            llm_format: If True, return LLM-optimized schema format with inlined
+                references and simplified structure.
 
         Returns:
-            The JSON schema of the secret format as a string.
+            The JSON schema of the secret format as a JSON string.
+
+        Raises:
+            NotImplementedError: If the `secret_format` class attribute is not defined.
         """
         if cls.secret_format is not None:
             if llm_format:
-                return json.dumps(llm_ready_schema(cls.secret_format), indent=2)
+                result_json, result_ui = SchemaSplitter.split(cls.secret_format.model_json_schema())
+                return json.dumps({"json_schema": result_json, "ui_schema": result_ui}, indent=2)
             return json.dumps(cls.secret_format.model_json_schema(), indent=2)
         msg = f"{cls.__name__}' class does not define a 'secret_format'."
         raise NotImplementedError(msg)
 
     @classmethod
-    def get_input_format(cls, *, llm_format: bool) -> str:
+    async def get_input_format(cls, *, llm_format: bool) -> str:
         """Get the JSON schema of the input format model.
 
-        Raises:
-            NotImplementedError: If the `input_format` is not defined.
+        Args:
+            llm_format: If True, return LLM-optimized schema format with inlined
+                references and simplified structure.
 
         Returns:
-            The JSON schema of the input format as a string.
+            The JSON schema of the input format as a JSON string.
+
+        Raises:
+            NotImplementedError: If the `input_format` class attribute is not defined.
         """
-        if cls.input_format is not None:
-            if llm_format:
-                return json.dumps(llm_ready_schema(cls.input_format), indent=2)
-            return json.dumps(cls.input_format.model_json_schema(), indent=2)
-        msg = f"{cls.__name__}' class does not define an 'input_format'."
-        raise NotImplementedError(msg)
+        if cls.input_format is None:
+            msg = f"{cls.__name__}' class does not define an 'input_format'."
+            raise NotImplementedError(msg)
+
+        extended_model = UtilitySchemaExtender.create_extended_input_model(cls.input_format)
+
+        if llm_format:
+            result_json, result_ui = SchemaSplitter.split(extended_model.model_json_schema())
+            return json.dumps({"json_schema": result_json, "ui_schema": result_ui}, indent=2)
+        return json.dumps(extended_model.model_json_schema(), indent=2)
 
     @classmethod
-    def get_output_format(cls, *, llm_format: bool) -> str:
+    async def get_output_format(cls, *, llm_format: bool) -> str:
         """Get the JSON schema of the output format model.
 
-        Raises:
-            NotImplementedError: If the `output_format` is not defined.
+        Args:
+            llm_format: If True, return LLM-optimized schema format with inlined
+                references and simplified structure.
 
         Returns:
-            The JSON schema of the output format as a string.
+            The JSON schema of the output format as a JSON string.
+
+        Raises:
+            NotImplementedError: If the `output_format` class attribute is not defined.
         """
-        if cls.output_format is not None:
-            if llm_format:
-                return json.dumps(llm_ready_schema(cls.output_format), indent=2)
-            return json.dumps(cls.output_format.model_json_schema(), indent=2)
-        msg = "'%s' class does not define an 'output_format'."
-        raise NotImplementedError(msg)
+        if cls.output_format is None:
+            msg = f"'{cls.__name__}' class does not define an 'output_format'."
+            raise NotImplementedError(msg)
+
+        extended_model = UtilitySchemaExtender.create_extended_output_model(cls.output_format)
+
+        if llm_format:
+            result_json, result_ui = SchemaSplitter.split(extended_model.model_json_schema())
+            return json.dumps({"json_schema": result_json, "ui_schema": result_ui}, indent=2)
+        return json.dumps(extended_model.model_json_schema(), indent=2)
 
     @classmethod
-    def get_config_setup_format(cls, *, llm_format: bool) -> str:
+    async def get_config_setup_format(cls, *, llm_format: bool) -> str:
         """Gets the JSON schema of the config setup format model.
 
-        The config setup format is used only to initialize the module with configuration data.
-        The setup format is used to initialize an run the module with setup data.
+        The config setup format is used only to initialize the module with configuration
+        data. It includes fields marked with `json_schema_extra={"config": True}` and
+        excludes hidden runtime fields.
 
-        Raises:
-            NotImplementedError: If the `setup_format` is not defined.
+        Dynamic schema fields are always resolved when generating the schema, as this
+        method is typically called during module discovery or schema generation where
+        fresh values are needed.
+
+        Args:
+            llm_format: If True, return LLM-optimized schema format with inlined
+                references and simplified structure.
 
         Returns:
-            The JSON schema of the config setup format as a string.
+            The JSON schema of the config setup format as a JSON string.
+
+        Raises:
+            NotImplementedError: If the `setup_format` class attribute is not defined.
         """
         if cls.setup_format is not None:
-            setup_format = cls.setup_format.get_clean_model(config_fields=True, hidden_fields=False)
+            setup_format = await cls.setup_format.get_clean_model(config_fields=True, hidden_fields=False, force=True)
             if llm_format:
-                return json.dumps(llm_ready_schema(setup_format), indent=2)
+                result_json, result_ui = SchemaSplitter.split(setup_format.model_json_schema())
+                return json.dumps({"json_schema": result_json, "ui_schema": result_ui}, indent=2)
             return json.dumps(setup_format.model_json_schema(), indent=2)
         msg = "'%s' class does not define an 'config_setup_format'."
         raise NotImplementedError(msg)
 
     @classmethod
-    def get_setup_format(cls, *, llm_format: bool) -> str:
+    async def get_setup_format(cls, *, llm_format: bool) -> str:
         """Gets the JSON schema of the setup format model.
 
-        Raises:
-            NotImplementedError: If the `setup_format` is not defined.
+        The setup format is used at runtime and includes hidden fields but excludes
+        config-only fields. This is the schema used when running the module.
+
+        Dynamic schema fields are always resolved when generating the schema, as this
+        method is typically called during module discovery or schema generation where
+        fresh values are needed.
+
+        Args:
+            llm_format: If True, return LLM-optimized schema format with inlined
+                references and simplified structure.
 
         Returns:
-            The JSON schema of the setup format as a string.
+            The JSON schema of the setup format as a JSON string.
+
+        Raises:
+            NotImplementedError: If the `setup_format` class attribute is not defined.
         """
         if cls.setup_format is not None:
-            setup_format = cls.setup_format.get_clean_model(config_fields=False, hidden_fields=True)
+            setup_format = await cls.setup_format.get_clean_model(config_fields=False, hidden_fields=True, force=True)
             if llm_format:
-                return json.dumps(llm_ready_schema(setup_format), indent=2)
+                result_json, result_ui = SchemaSplitter.split(setup_format.model_json_schema())
+                return json.dumps({"json_schema": result_json, "ui_schema": result_ui}, indent=2)
             return json.dumps(setup_format.model_json_schema(), indent=2)
         msg = "'%s' class does not define an 'setup_format'."
         raise NotImplementedError(msg)
@@ -245,17 +279,22 @@ class BaseModule(  # noqa: PLR0904
         return cls.input_format(**input_data)
 
     @classmethod
-    def create_setup_model(cls, setup_data: dict[str, Any], *, config_fields: bool = False) -> SetupModelT:
+    async def create_setup_model(cls, setup_data: dict[str, Any], *, config_fields: bool = False) -> SetupModelT:
         """Create the setup model from the setup data.
 
+        Creates a filtered setup model instance based on the provided data.
+        Uses `get_clean_model()` internally to get the appropriate model class
+        with field filtering applied.
+
         Args:
             setup_data: The setup data to create the model from.
             config_fields: If True, include only fields with json_schema_extra["config"] == True.
 
         Returns:
-            The setup model.
+            An instance of the setup model with the provided data.
         """
-        return cls.setup_format.get_clean_model(config_fields=config_fields, hidden_fields=True)(**setup_data)
+        model_cls = await cls.setup_format.get_clean_model(config_fields=config_fields, hidden_fields=True)
+        return model_cls(**setup_data)
 
     @classmethod
     def create_secret_model(cls, secret_data: dict[str, Any]) -> SecretModelT:
@@ -291,8 +330,18 @@ class BaseModule(  # noqa: PLR0904
         If a package is provided, all .py files within its path are imported; otherwise, the current
         working directory is searched. For each imported module, any class matching the criteria is
         registered via cls.register(). Errors during import are logged at debug level.
+
+        Built-in healthcheck handlers (ping, services, status) are automatically registered
+        to provide standard healthcheck functionality for all modules.
         """
+        from digitalkin.models.module.utility import UtilityRegistry  # noqa: PLC0415
+
         cls.triggers_discoverer.discover_modules()
+
+        # Auto-register built-in SDK triggers (healthcheck, etc.)
+        for trigger_cls in UtilityRegistry.get_builtin_triggers():
+            cls.triggers_discoverer.register_trigger(trigger_cls)
+
         logger.debug("discovered: %s", cls.triggers_discoverer)
 
     @classmethod
@@ -307,23 +356,8 @@ class BaseModule(  # noqa: PLR0904
         """
         return cls.triggers_discoverer.register_trigger(handler_cls)
 
-    async def run_config_setup(self, config_setup_data: SetupModelT) -> SetupModelT:  # noqa: PLR6301
-        """Run config setup the module.
-
-        The config setup is used to initialize the setup with configuration data.
-        This method is typically used to set up the module with necessary configuration before running it,
-        especially for processing data like files.
-        The function needs to save the setup in the storage.
-        The module will be initialize with the setup and not the config setup.
-        This method is optional, the config setup and setup can be the same.
-
-        Returns:
-            The updated setup model after running the config setup.
-        """
-        return config_setup_data
-
     @abstractmethod
-    async def initialize(self, setup_data: SetupModelT) -> None:
+    async def initialize(self, context: ModuleContext, setup_data: SetupModelT) -> None:
         """Initialize the module."""
         raise NotImplementedError
 
@@ -331,22 +365,12 @@ class BaseModule(  # noqa: PLR0904
         self,
         input_data: InputModelT,
         setup_data: SetupModelT,
-        callback: Callable[[OutputModelT], Coroutine[Any, Any, None]],
     ) -> None:
-        """Run the module with the given input and setup data.
-
-        This method validates the input data, determines the protocol from the input,
-        and dispatches the request to the corresponding trigger handler. The trigger handler
-        is responsible for processing the input and invoking the callback with the result.
-
-        Triggers:
-            - The method is triggered when a module run is requested with specific input and setup data.
-            - The protocol specified in the input determines which trigger handler is invoked.
+        """Run the module by dispatching to the appropriate trigger handler.
 
         Args:
-            input_data (InputModelT): The input data to be processed by the module.
-            setup_data (SetupModelT): The setup or configuration data required for the module.
-            callback (Callable[[OutputModelT], Coroutine[Any, Any, None]]): callback to be invoked to stream any result.
+            input_data: Input data to process.
+            setup_data: Configuration data for the module.
 
         Raises:
             ValueError: If no handler for the protocol is found.
@@ -360,7 +384,6 @@ class BaseModule(  # noqa: PLR0904
         await handler_instance.handle(
             input_instance.root,
             setup_data,
-            callback,
             self.context,
         )
 
@@ -369,11 +392,29 @@ class BaseModule(  # noqa: PLR0904
         """Run the module."""
         raise NotImplementedError
 
+    async def run_config_setup(  # noqa: PLR6301
+        self,
+        context: ModuleContext,  # noqa: ARG002
+        config_setup_data: SetupModelT,
+    ) -> SetupModelT:
+        """Run config setup the module.
+
+        The config setup is used to initialize the setup with configuration data.
+        This method is typically used to set up the module with necessary configuration before running it,
+        especially for processing data like files.
+        The function needs to save the setup in the storage.
+        The module will be initialize with the setup and not the config setup.
+        This method is optional, the config setup and setup can be the same.
+
+        Returns:
+            The updated setup model after running the config setup.
+        """
+        return config_setup_data
+
     async def _run_lifecycle(
         self,
         input_data: InputModelT,
         setup_data: SetupModelT,
-        callback: Callable[[OutputModelT], Coroutine[Any, Any, None]],
     ) -> None:
         """Run the module lifecycle.
 
@@ -381,38 +422,56 @@ class BaseModule(  # noqa: PLR0904
             asyncio.CancelledError: If the module is cancelled
         """
         try:
-            logger.info("Starting module %s", self.name)
-            await self.run(input_data, setup_data, callback)
-            logger.info("Module %s finished", self.name)
+            logger.info("Starting module %s", self.name, extra=self.context.session.current_ids())
+            await self.run(input_data, setup_data)
+            logger.info("Module %s finished", self.name, extra=self.context.session.current_ids())
         except asyncio.CancelledError:
             self._status = ModuleStatus.CANCELLED
-            logger.error(f"Module {self.name} cancelled")
+            logger.error("Module %s cancelled", self.name, extra=self.context.session.current_ids())
         except Exception:
             self._status = ModuleStatus.FAILED
-            logger.exception("Error inside module %s", self.name)
+            logger.exception("Error inside module %s", self.name, extra=self.context.session.current_ids())
         else:
             self._status = ModuleStatus.STOPPING
-        finally:
-            await self.stop()
 
     async def start(
         self,
         input_data: InputModelT,
         setup_data: SetupModelT,
-        callback: Callable[[OutputModelT | ModuleCodeModel], Coroutine[Any, Any, None]],
+        callback: Callable[[OutputModelT | ModuleCodeModel | DataModel[UtilityProtocol]], Coroutine[Any, Any, None]],
         done_callback: Callable | None = None,
     ) -> None:
         """Start the module."""
         try:
-            logger.debug("Inititalize module")
-            await self.initialize(setup_data=setup_data)
+            self.context.callbacks.send_message = callback
+
+            tool_cache = setup_data.build_tool_cache()
+            if tool_cache.entries:
+                self.context.tool_cache = tool_cache
+
+            await callback(
+                DataModel(
+                    root=ModuleStartInfoOutput(
+                        job_id=self.context.session.job_id,
+                        mission_id=self.context.session.mission_id,
+                        setup_id=self.context.session.setup_id,
+                        setup_version_id=self.context.session.setup_version_id,
+                        module_id=self.get_module_id(),
+                        module_name=self.name,
+                    ),
+                    annotations={"role": BaseRole.SYSTEM},
+                )
+            )
+
+            logger.info("Initialize module %s", self.context.session.job_id)
+            await self.initialize(self.context, setup_data)
         except Exception as e:
             self._status = ModuleStatus.FAILED
             short_description = "Error initializing module"
             logger.exception("%s: %s", short_description, e)
             await callback(
                 ModuleCodeModel(
-                    code=str(self._status),
+                    code="Error",
                     short_description=short_description,
                     message=str(e),
                 )
@@ -425,53 +484,92 @@ class BaseModule(  # noqa: PLR0904
         try:
             logger.debug("Init the discovered input handlers.")
             self.triggers_discoverer.init_handlers(self.context)
-            logger.debug("Run lifecycle")
-            self._status = ModuleStatus.RUNNING
-            self._task = asyncio.create_task(
-                self._run_lifecycle(input_data, setup_data, callback),
-                name="module_lifecycle",
-            )
-            if done_callback is not None:
-                self._task.add_done_callback(done_callback)
+            logger.debug("Run lifecycle %s", self.context.session.job_id)
+            await self._run_lifecycle(input_data, setup_data)
         except Exception:
             self._status = ModuleStatus.FAILED
             logger.exception("Error during module lifecyle")
+        finally:
+            await self.stop()
 
     async def stop(self) -> None:
         """Stop the module."""
-        logger.info("Stopping module %s with status %s", self.name, self._status)
-        if self._status not in {ModuleStatus.RUNNING, ModuleStatus.STOPPING}:
-            return
-
+        logger.info("Stopping module %s | job_id=%s", self.name, self.context.session.job_id)
         try:
             self._status = ModuleStatus.STOPPING
-            if self._task and not self._task.done():
-                self._task.cancel()
-                with contextlib.suppress(asyncio.CancelledError):
-                    await self._task
             logger.debug("Module %s stopped", self.name)
             await self.cleanup()
+            await self.context.callbacks.send_message(
+                DataModel(
+                    root=EndOfStreamOutput(),
+                    annotations={"role": BaseRole.SYSTEM},
+                )
+            )
             self._status = ModuleStatus.STOPPED
             logger.debug("Module %s cleaned", self.name)
         except Exception:
             self._status = ModuleStatus.FAILED
             logger.exception("Error stopping module")
 
+    async def _resolve_tools(self, config_setup_data: SetupModelT) -> None:
+        """Resolve tool references and build cache.
+
+        Args:
+            config_setup_data: Setup data containing tool references.
+        """
+        logger.info("Starting tool resolution", extra=self.context.session.current_ids())
+        if self.context.registry is not None:
+            config_setup_data.resolve_tool_references(self.context.registry)
+            logger.info("Tool references resolved", extra=self.context.session.current_ids())
+        else:
+            logger.warning("No registry available, skipping tool resolution", extra=self.context.session.current_ids())
+
+        tool_cache = config_setup_data.build_tool_cache()
+        self.context.tool_cache = tool_cache
+        logger.info(
+            "Tool cache built with %d entries: %s",
+            len(tool_cache.entries),
+            list(tool_cache.entries.keys()),
+            extra=self.context.session.current_ids(),
+        )
+
     async def start_config_setup(
         self,
         config_setup_data: SetupModelT,
         callback: Callable[[SetupModelT | ModuleCodeModel], Coroutine[Any, Any, None]],
     ) -> None:
-        """Start the module."""
+        """Run config setup lifecycle with tool resolution in parallel.
+
+        Args:
+            config_setup_data: Initial setup data to configure.
+            callback: Callback to send the configured setup model.
+        """
         try:
-            logger.info("Run Config Setup lifecycle")
+            logger.info("Run Config Setup lifecycle", extra=self.context.session.current_ids())
             self._status = ModuleStatus.RUNNING
-            content = await self.run_config_setup(config_setup_data)
+            self.context.callbacks.set_config_setup = callback
+
+            # Resolve tools first to populate companion fields, then run config setup
+            await self._resolve_tools(config_setup_data)
+            updated_config = await self.run_config_setup(self.context, config_setup_data)
 
+            # Build wrapper: original structure with updated content
             wrapper = config_setup_data.model_dump()
-            wrapper["content"] = content.model_dump()
-            await callback(self.create_setup_model(wrapper))
+            wrapper["content"] = updated_config.model_dump()
+
+            # Debug logging
+            content = wrapper.get("content", {})
+            logger.info(
+                "Config setup wrapper: keys=%s, content_keys=%s, tools_cache=%s",
+                list(wrapper.keys()),
+                list(content.keys()) if isinstance(content, dict) else "N/A",
+                content.get("tools_cache") if isinstance(content, dict) else "N/A",
+                extra=self.context.session.current_ids(),
+            )
+
+            setup_model = await self.create_setup_model(wrapper)
+            await callback(setup_model)
             self._status = ModuleStatus.STOPPING
         except Exception:
             self._status = ModuleStatus.FAILED
-            logger.exception("Error during module lifecyle")
+            logger.exception("Error during config setup lifecycle", extra=self.context.session.current_ids())

digitalkin/modules/archetype_module.py

@@ -2,7 +2,12 @@
 
 from abc import ABC
 
-from digitalkin.models.module import InputModelT, OutputModelT, SecretModelT, SetupModelT
+from digitalkin.models.module.module_types import (
+    InputModelT,
+    OutputModelT,
+    SecretModelT,
+    SetupModelT,
+)
 from digitalkin.modules._base_module import BaseModule