boneIO 1.3.0.dev18__py3-none-any.whl → 1.3.0.dev20__py3-none-any.whl

boneio/core/config/config_helper.py

@@ -40,7 +40,7 @@ class ConfigHelper:
         device_type: str = "boneIO Black",
         ha_discovery: bool = True,
         ha_discovery_prefix: str = HOMEASSISTANT,
-        network_info: dict = {},
+        network_info: dict | None = None,
         is_web_active: bool = False,
         web_port: int = 8090,
         proxy_port: int | None = None,
@@ -56,7 +56,7 @@ class ConfigHelper:
         self._name = name
         
         # Generate serial number from MAC - always required for topic prefix
-        self._serial_no = get_serial_from_mac(network_info)
+        self._serial_no = get_serial_from_mac(network_info or {})
         
         # Build fixed topic prefix: boneio/blk_{serial}
         # This is no longer configurable - always uses this format
@@ -309,10 +309,7 @@ class ConfigHelper:
     def is_topic_in_autodiscovery(self, topic: str) -> bool:
         topic_parts_raw = topic[len(f"{self._ha_discovery_prefix}/") :].split("/")
         ha_type = topic_parts_raw[0]
-        if ha_type in self._autodiscovery_messages:
-            if topic in self._autodiscovery_messages[ha_type]:
-                return True
-        return False
+        return ha_type in self._autodiscovery_messages and topic in self._autodiscovery_messages[ha_type]
     
     def clear_autodiscovery_type(self, ha_type: str):
         self._autodiscovery_messages[ha_type] = {}

boneio/core/config/yaml_util.py

@@ -1,3 +1,4 @@
+import contextlib
 import fnmatch
 import logging
 import os
@@ -309,7 +310,7 @@ def merge_board_config(config: dict) -> dict:
         if not board_config:
             raise ConfigurationException(f"Bottom board configuration file {board_file} is empty")
     except FileNotFoundError:
-        raise ConfigurationException(f"Board configuration for {board_name} version {version} not found")
+        raise ConfigurationException(f"Board configuration for {board_name} version {version} not found") from None
     _LOGGER.debug(f"Loaded board configuration: {board_name}")
 
     # Copy MCP configuration if not already defined
@@ -427,9 +428,8 @@ class CustomValidator(Validator):
             doc_value = self.document[key]  # type: ignore[attr-defined]
             if isinstance(doc_value, str):
                 doc_value = doc_value.lower()
-            if doc_value in [v.lower() if isinstance(v, str) else v for v in values]:
-                if field not in self.document:  # type: ignore[attr-defined]
-                    self._error(field, f"required when {key} is {doc_value}")  # type: ignore[attr-defined]
+            if doc_value in [v.lower() if isinstance(v, str) else v for v in values] and field not in self.document:  # type: ignore[attr-defined]
+                self._error(field, f"required when {key} is {doc_value}")  # type: ignore[attr-defined]
 
     def _validate_forbidden_if(self, forbidden_if, field, value):
         """Validate that a field is forbidden if a condition is met.
@@ -448,9 +448,12 @@ class CustomValidator(Validator):
             if isinstance(doc_value, str):
                 doc_value = doc_value.lower()
 
-            if doc_value in [v.lower() if isinstance(v, str) else v for v in values]:
-                if field in self.document and value != default_value:  # type: ignore[attr-defined]
-                    self._error(field, f"forbidden when {key} is {doc_value}")  # type: ignore[attr-defined]
+            if (
+                doc_value in [v.lower() if isinstance(v, str) else v for v in values]
+                and field in self.document
+                and value != default_value
+            ):  # type: ignore[attr-defined]
+                self._error(field, f"forbidden when {key} is {doc_value}")  # type: ignore[attr-defined]
 
     def _normalize_coerce_action_field(self, value):
         """Handle conditional defaults for action fields."""
@@ -809,7 +812,6 @@ def _compute_config_dir_hash(config_file: str) -> str:
     hashed — other files in the directory (e.g. ``state.json``) are ignored.
     """
     import hashlib
-    import re
     from pathlib import Path
 
     config_path = Path(config_file)
@@ -1062,7 +1064,7 @@ def load_config_from_file(
     try:
         config_yaml = load_yaml_file(config_file)
     except FileNotFoundError as err:
-        raise ConfigurationException(err)
+        raise ConfigurationException(err) from err
     _LOGGER.debug("[STARTUP TIMING] load_yaml_file: %.2fs", _time.monotonic() - _t0)
     if not config_yaml:
         _LOGGER.warning("Missing yaml file. %s", config_file)
@@ -1205,7 +1207,6 @@ def update_config_section(config_file: str, section: str, data: dict | list) ->
     Returns:
         dict: Status response with success/error message
     """
-    import os
     from pathlib import Path
 
     config_dir = Path(config_file).parent
@@ -1222,10 +1223,8 @@ def update_config_section(config_file: str, section: str, data: dict | list) ->
                     if addr.startswith("0x") or addr.startswith("0X"):
                         entry["address"] = int(addr, 16)
                     else:
-                        try:
+                        with contextlib.suppress(ValueError):
                             entry["address"] = int(addr, 10)
-                        except ValueError:
-                            pass  # Keep as string if conversion fails
 
     # Strip default values to keep YAML clean
     cleaned_data = strip_default_values(data, {}, section)
@@ -1366,8 +1365,6 @@ def update_yaml_field(config_file: str, section: str, field: str, value: str) ->
     Returns:
         dict: Status response with success/error message.
     """
-    import re
-
     try:
         with open(config_file, encoding="utf-8") as f:
             lines = f.readlines()
@@ -1382,7 +1379,7 @@ def update_yaml_field(config_file: str, section: str, field: str, value: str) ->
         insert_index = -1
         updated_lines = []
 
-        for i, line in enumerate(lines):
+        for line in lines:
             stripped = line.strip()
             line_indent = len(line) - len(line.lstrip())
 

boneio/core/manager/inputs.py

@@ -177,7 +177,7 @@ class InputManager:
         gpio: dict,
         pin: str,
         existing_input: GpioEventButton | None = None,
-        actions: dict = {},
+        actions: dict | None = None,
     ) -> GpioEventButton | None:
         """Configure event input sensor with multiclick detection.
 
@@ -190,6 +190,7 @@ class InputManager:
         Returns:
             Configured GpioEventButton instance or None on error
         """
+        actions = actions or {}
         try:
             # Determine display name (ensure it's always a string)
             if "name" in gpio:
@@ -313,7 +314,7 @@ class InputManager:
         gpio: dict,
         pin: str,
         existing_input: GpioInputBinarySensor | None = None,
-        actions: dict = {},
+        actions: dict | None = None,
     ) -> GpioInputBinarySensor | None:
         """Configure binary sensor input with state detection.
 
@@ -326,6 +327,7 @@ class InputManager:
         Returns:
             Configured GpioInputBinarySensor instance or None on error
         """
+        actions = actions or {}
         try:
             # Determine display name (ensure it's always a string)
             if "name" in gpio:
@@ -564,7 +566,6 @@ class InputManager:
         import time
 
         from boneio.models import InputState
-        from boneio.models.events import InputEvent
 
         timestamp = time.time()