conson-xp 2.0.2__py3-none-any.whl → 2.0.4__py3-none-any.whl

{conson_xp-2.0.2.dist-info → conson_xp-2.0.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: conson-xp
-Version: 2.0.2
+Version: 2.0.4
 Summary: XP Protocol Communication Tools
 Author-Email: ldvchosal <ldvchosal@github.com>
 License: MIT License

{conson_xp-2.0.2.dist-info → conson_xp-2.0.4.dist-info}/RECORD

@@ -1,8 +1,8 @@
-conson_xp-2.0.2.dist-info/METADATA,sha256=9UzUpaHd8pegRLDFIPEzaMHr2L3ziBK7QhA2EYqSJ28,11319
-conson_xp-2.0.2.dist-info/WHEEL,sha256=tsUv_t7BDeJeRHaSrczbGeuK-TtDpGsWi_JfpzD255I,90
-conson_xp-2.0.2.dist-info/entry_points.txt,sha256=1OcdIcDM1hz3ljCXgybaPUh1IOFEwkaTgLIW9u9zGeg,50
-conson_xp-2.0.2.dist-info/licenses/LICENSE,sha256=rxj6woMM-r3YCyGq_UHFtbh7kHTAJgrccH6O-33IDE4,1419
-xp/__init__.py,sha256=qQYrRExAxcFFDY2HISDUiyYNIaz1ywu0tBMS6dAtJ9I,181
+conson_xp-2.0.4.dist-info/METADATA,sha256=CJMAU_3xThp1pBfy62H2HtM-hDsN8T9UxxqFRTflo0o,11319
+conson_xp-2.0.4.dist-info/WHEEL,sha256=tsUv_t7BDeJeRHaSrczbGeuK-TtDpGsWi_JfpzD255I,90
+conson_xp-2.0.4.dist-info/entry_points.txt,sha256=1OcdIcDM1hz3ljCXgybaPUh1IOFEwkaTgLIW9u9zGeg,50
+conson_xp-2.0.4.dist-info/licenses/LICENSE,sha256=rxj6woMM-r3YCyGq_UHFtbh7kHTAJgrccH6O-33IDE4,1419
+xp/__init__.py,sha256=8uU6iQSfgIQU3EC4KbiWh-hGKhEpZZZ7YaahwbEQvaE,181
 xp/cli/__init__.py,sha256=QjnKB1KaI2aIyKlzrnvCwfbBuUj8HNgwNMvNJVQofbI,81
 xp/cli/__main__.py,sha256=l2iKwMdat5rTGd3JWs-uGksnYYDDffp_Npz05QdKEeU,117
 xp/cli/commands/__init__.py,sha256=9TGP3uTxAU-s-kVChvQ-Fn3-HVYj-QpeQ05Is-20HRo,4788
@@ -22,7 +22,7 @@ xp/cli/commands/conbus/conbus_linknumber_commands.py,sha256=3bkik8nHXY89XfzeUnKW
 xp/cli/commands/conbus/conbus_modulenumber_commands.py,sha256=fOIO0f85iDJBfGnRCEN8zK6j4i43uuF_9YKQc_nQ39A,3628
 xp/cli/commands/conbus/conbus_msactiontable_commands.py,sha256=oMHfrxDR9yj0Pvx92ZHnwYU3CZqwNNvfAfEoLuo_jhQ,11585
 xp/cli/commands/conbus/conbus_output_commands.py,sha256=XMEMb5tM0ul7lwhwoo4QRgWprXZ31qswXBno8KCeFMo,5342
-xp/cli/commands/conbus/conbus_raw_commands.py,sha256=o1dVZqiRw7J3_8Bge7DJucFyAqt7GNP-Azf-f3ir3SU,2019
+xp/cli/commands/conbus/conbus_raw_commands.py,sha256=bCFF1q61VSfdwS_Ch7jTbv-Q-t6n_EwEHRw_wlvNrr0,1984
 xp/cli/commands/conbus/conbus_receive_commands.py,sha256=LdxoOUdM5atkC8fFlp-GK7HQ7oxTnjrSFDx-lQ3nme0,1925
 xp/cli/commands/conbus/conbus_scan_commands.py,sha256=QuRT4Vx5TqervCOUsZ4wcxvy7Jwra4hl2-DIt8cDcoE,1828
 xp/cli/commands/file_commands.py,sha256=dx6a4xNxuReXq7dOyL8ebn6lyifzDm0d0OsuDjyr_90,5547
@@ -136,7 +136,7 @@ xp/services/conbus/conbus_event_raw_service.py,sha256=viXuEXw165-RytdqC76wQShJLD
 xp/services/conbus/conbus_export_actiontable_service.py,sha256=d6Y2RDiVOH-jTMWOIE_gY_WBJnHFrbJGguk-WZWbNPs,11818
 xp/services/conbus/conbus_export_service.py,sha256=RP8nADTIs4FGUf_BFLRZMtEJZdXV94zg3QrlWaDnhKA,17536
 xp/services/conbus/conbus_output_service.py,sha256=e57bRkLgPnJuB8hkllNh0kgGkjPt9IK75tuBxd_bOkE,9361
-xp/services/conbus/conbus_raw_service.py,sha256=OQuV521VOQraf2PGF2B9868vh7sDgmfc19YebrkZnyw,5844
+xp/services/conbus/conbus_raw_service.py,sha256=xmFotLJqPU8CSDumB2gnxqLW7OO-Jlc7WyS0f2cgayA,5967
 xp/services/conbus/conbus_receive_service.py,sha256=TFf3W65brGsy6QZICpIs0Xy9bgqyL1vgQuhS_eHuIZs,5416
 xp/services/conbus/conbus_scan_service.py,sha256=_Ka0OUDNYhDgZIR49Q0P5GTxJq6RcAAX2DVqEDdtb5U,6888
 xp/services/conbus/write_config_service.py,sha256=BCfmLNPRDpwSwRMRYJvx2FXA8IZsdgmyeTXIYvmb4ys,9004
@@ -166,8 +166,8 @@ xp/services/telegram/telegram_output_service.py,sha256=9deqtcPndRqJ-3XQUWlJhXaVc
 xp/services/telegram/telegram_service.py,sha256=jPu0Xrh3IpvqPLyuQT5Vf8HHw00vBingONHdxf_9TkI,13315
 xp/services/telegram/telegram_version_service.py,sha256=oXnZ_K7OQ7xD-GEj3zDYp52KlkqVuHpO4bf7gMlC_w4,10574
 xp/services/term/__init__.py,sha256=BIeOK042bMR-0l6MA80wdW5VuHlpWOXtRER9IG5ilQA,245
-xp/services/term/homekit_accessory_driver.py,sha256=jHhHHOOvzdDjuYys1cUZHcH2uwZy_Y6o5aGAQqXGAVg,5935
-xp/services/term/homekit_service.py,sha256=8aEsmC-EwKTxf0Hxp0aFBdo4doUvhVHeVwfuuc1ttWw,28925
+xp/services/term/homekit_accessory_driver.py,sha256=asIZz-1aJQv7m0oP28vwMWPh8x6WsMHN9pxJ0jIdUPk,7833
+xp/services/term/homekit_service.py,sha256=FNm01Sp5F4bjJqRLMpl_FGbPgeuUkWxcu7sCBOihR0c,30892
 xp/services/term/protocol_monitor_service.py,sha256=5YBI0Nu7B7gMhaTbUhL6k9LSRfnCIj6CwrCYHiMHavA,10067
 xp/services/term/state_monitor_service.py,sha256=EK9tNBfamAIV0z0EMsXDYWC-rXv6l6k_bHsC8xyEFSo,17116
 xp/term/__init__.py,sha256=Xg2DhBeI3xQJLfc7_BPWI1por-rUXemyer5OtOt9Cus,51
@@ -191,4 +191,4 @@ xp/utils/logging.py,sha256=wJ1d-yg97NiZUrt2F8iDMcmnHVwC-PErcI-7dpyiRDc,3777
 xp/utils/serialization.py,sha256=TS1OwpTOemSvXsCGw3js4JkYYFEqkzrPe8V9QYQefdw,4684
 xp/utils/state_machine.py,sha256=W9AY4ntRZnFeHAa5d43hm37j53uJPlqkRvWTPiBhJ_0,2464
 xp/utils/time_utils.py,sha256=K17godWpL18VEypbTlvNOEDG6R3huYnf29yjkcnwRpU,3796
-conson_xp-2.0.2.dist-info/RECORD,,
+conson_xp-2.0.4.dist-info/RECORD,,

xp/__init__.py

@@ -4,7 +4,7 @@ XP CLI tool for remote console bus operations.
 conson-xp package.
 """
 
-__version__ = "2.0.2"
+__version__ = "2.0.4"
 __manufacturer__ = "salchichon"
 __model__ = "xp.cli"
 __serial__ = "2025.09.23.000"

xp/cli/commands/conbus/conbus_raw_commands.py

@@ -14,25 +14,24 @@ from xp.services.conbus.conbus_raw_service import ConbusRawService
 
 
 @conbus.command("raw")
-@click.argument("raw_telegrams")
+@click.argument("telegrams", nargs=-1, required=True)
 @click.pass_context
 @connection_command()
-def send_raw_telegrams(ctx: Context, raw_telegrams: str) -> None:
+def send_raw_telegrams(ctx: Context, telegrams: tuple[str, ...]) -> None:
     r"""
     Send raw telegram sequence to Conbus server.
 
-    Accepts a string containing one or more telegrams in format <...>.
-    Multiple telegrams should be concatenated without separators.
+    Accepts one or more telegrams in format S2113010000F02D (without <> nor CHECKSUM) as separate arguments.
 
     Args:
         ctx: Click context object.
-        raw_telegrams: Raw telegram string(s).
+        telegrams: Raw telegram string(s).
 
     Examples:
         \b
-        xp conbus raw '<S2113010000F02D12>'
-        xp conbus raw '<S2113010000F02D12><S2113010001F02D12><S2113010002F02D12>'
-        xp conbus raw '<S0012345003F02D12FM>...<S0012345009F02D12FF>'
+        xp conbus raw 'S2113010000F02D'
+        xp conbus raw 'S2113010000F02D' 'S2113010001F02D'
+        xp conbus raw 'S0012345003F02D12F' 'S0012345009F02D12'
     """
     service: ConbusRawService = (
         ctx.obj.get("container").get_container().resolve(ConbusRawService)
@@ -62,8 +61,8 @@ def send_raw_telegrams(ctx: Context, raw_telegrams: str) -> None:
         service.on_progress.connect(on_progress)
         service.on_finish.connect(on_finish)
         # Setup
-        service.send_raw_telegram(
-            raw_input=raw_telegrams,
+        service.send_raw_telegrams(
+            telegrams=list(telegrams),
             timeout_seconds=5.0,
         )
         # Start (blocks until completion)

xp/services/conbus/conbus_raw_service.py

@@ -51,7 +51,7 @@ class ConbusRawService:
         self.conbus_protocol.on_timeout.connect(self.timeout)
         self.conbus_protocol.on_failed.connect(self.failed)
 
-        self.raw_input: str = ""
+        self.telegrams: list[str] = []
         self.service_response: ConbusRawResponse = ConbusRawResponse(
             success=False,
         )
@@ -60,8 +60,11 @@ class ConbusRawService:
 
     def connection_made(self) -> None:
         """Handle connection established event."""
-        self.logger.debug(f"Connection established, sending {self.raw_input}")
-        self.conbus_protocol.send_raw_telegram(self.raw_input)
+        self.logger.debug(
+            f"Connection established, sending {len(self.telegrams)} telegrams"
+        )
+        for telegram in self.telegrams:
+            self.conbus_protocol.send_raw_telegram(telegram)
 
     def telegram_sent(self, telegram_sent: str) -> None:
         """
@@ -108,22 +111,22 @@ class ConbusRawService:
         self.service_response.error = message
         self.on_finish.emit(self.service_response)
 
-    def send_raw_telegram(
+    def send_raw_telegrams(
         self,
-        raw_input: str,
+        telegrams: list[str],
         timeout_seconds: Optional[float] = None,
     ) -> None:
         """
-        Send a raw telegram string to the Conbus server.
+        Send raw telegrams to the Conbus server.
 
         Args:
-            raw_input: Raw telegram string to send.
+            telegrams: List of raw telegram strings to send.
             timeout_seconds: Timeout in seconds.
         """
-        self.logger.info("Starting send_raw_telegram")
+        self.logger.info(f"Starting send_raw_telegrams with {len(telegrams)} telegrams")
         if timeout_seconds:
             self.conbus_protocol.timeout_seconds = timeout_seconds
-        self.raw_input = raw_input
+        self.telegrams = telegrams
 
     def set_timeout(self, timeout_seconds: float) -> None:
         """
@@ -151,7 +154,7 @@ class ConbusRawService:
         """
         # Reset state for singleton reuse
         self.service_response = ConbusRawResponse(success=False)
-        self.raw_input = ""
+        self.telegrams = []
         return self
 
     def __exit__(

xp/services/term/homekit_accessory_driver.py

@@ -10,9 +10,18 @@ from pyhap.const import CATEGORY_LIGHTBULB, CATEGORY_OUTLET
 
 from xp.models.homekit.homekit_config import HomekitConfig
 
+# Callback type: (accessory_name, is_on, brightness_or_none)
+OnSetCallback = Callable[[str, bool, Optional[int]], None]
+
 
 class XPAccessory(Accessory):
-    """Single accessory wrapping a Conbus output."""
+    """
+    Single accessory wrapping a Conbus output.
+
+    Attributes:
+        logger: Logger instance for this accessory.
+        current_brightness: Current brightness value 0-100.
+    """
 
     def __init__(
         self,
@@ -35,12 +44,19 @@ class XPAccessory(Accessory):
         super().__init__(driver._driver, display_name, aid=aid)
         self._hk_driver = driver
         self._accessory_id = name
+        self._is_dimmable = service_type == "dimminglight"
+        self._char_brightness: Optional[object] = None
+        self._current_brightness: int = 100
         self.logger = logging.getLogger(__name__)
 
-        if service_type == "dimminglight":
+        if self._is_dimmable:
             self.category = CATEGORY_LIGHTBULB
             serv = self.add_preload_service("Lightbulb", chars=["On", "Brightness"])
-            # Note: Brightness setter_callback deferred to future update
+            self._char_brightness = serv.configure_char(
+                "Brightness",
+                setter_callback=self._set_brightness,
+                value=self._current_brightness,
+            )
         elif service_type == "outlet":
             self.category = CATEGORY_OUTLET
             serv = self.add_preload_service("Outlet")
@@ -58,16 +74,36 @@ class XPAccessory(Accessory):
             value: True for on, False for off.
         """
         if self._hk_driver._on_set:
-            self._hk_driver._on_set(self._accessory_id, value)
+            self._hk_driver._on_set(self._accessory_id, value, None)
+
+    def _set_brightness(self, value: int) -> None:
+        """
+        Handle HomeKit set brightness request.
+
+        Args:
+            value: Brightness value 0-100.
+        """
+        if self._hk_driver._on_set:
+            self._hk_driver._on_set(self._accessory_id, True, value)
+        self._current_brightness = value
 
-    def update_state(self, is_on: bool) -> None:
+    def update_state(self, is_on: bool, brightness: Optional[int] = None) -> None:
         """
         Update accessory state from Conbus event.
 
         Args:
             is_on: True if accessory is on, False otherwise.
+            brightness: Optional brightness value 0-100.
         """
         self._char_on.set_value(is_on)
+        if brightness is not None and self._char_brightness:
+            self._char_brightness.set_value(brightness)  # type: ignore[attr-defined]
+            self._current_brightness = brightness
+
+    @property
+    def current_brightness(self) -> int:
+        """Get current brightness value."""
+        return self._current_brightness
 
 
 class HomekitAccessoryDriver:
@@ -84,14 +120,15 @@ class HomekitAccessoryDriver:
         self._homekit_config = homekit_config
         self._driver: Optional[AccessoryDriver] = None
         self._accessories: Dict[str, XPAccessory] = {}
-        self._on_set: Optional[Callable[[str, bool], None]] = None
+        self._on_set: Optional[OnSetCallback] = None
 
-    def set_callback(self, on_set: Callable[[str, bool], None]) -> None:
+    def set_callback(self, on_set: OnSetCallback) -> None:
         """
         Set callback for HomeKit set events.
 
         Args:
-            on_set: Callback(accessory_name, is_on) called when HomeKit app toggles.
+            on_set: Callback(accessory_name, is_on, brightness) called when HomeKit app changes state.
+                    brightness is None for on/off only, or 0-100 for dimming.
         """
         self._on_set = on_set
 
@@ -157,15 +194,34 @@ class HomekitAccessoryDriver:
         except Exception as e:
             self.logger.error(f"Error stopping AccessoryDriver: {e}", exc_info=True)
 
-    def update_state(self, accessory_name: str, is_on: bool) -> None:
+    def update_state(
+        self, accessory_name: str, is_on: bool, brightness: Optional[int] = None
+    ) -> None:
         """
         Update accessory state from Conbus event.
 
         Args:
             accessory_name: Accessory name to update.
             is_on: True if accessory is on, False otherwise.
+            brightness: Optional brightness value 0-100.
         """
-        if acc := self._accessories.get(accessory_name):
-            acc.update_state(is_on)
+        acc = self._accessories.get(accessory_name)
+        if acc:
+            acc.update_state(is_on, brightness)
         else:
             self.logger.warning(f"Unknown accessory name: {accessory_name}")
+
+    def get_brightness(self, accessory_name: str) -> int:
+        """
+        Get current brightness for an accessory.
+
+        Args:
+            accessory_name: Accessory name.
+
+        Returns:
+            Current brightness 0-100, defaults to 100 if not found.
+        """
+        acc = self._accessories.get(accessory_name)
+        if acc:
+            return acc.current_brightness
+        return 100

xp/services/term/homekit_service.py

@@ -300,23 +300,82 @@ class HomekitService:
         await self._accessory_driver.stop()
         self.cleanup()
 
-    def _on_homekit_set(self, accessory_name: str, is_on: bool) -> None:
+    def _on_homekit_set(
+        self, accessory_name: str, is_on: bool, brightness: Optional[int]
+    ) -> None:
         """
-        Handle HomeKit app toggle request.
+        Handle HomeKit app set request (on/off or brightness).
 
         Args:
             accessory_name: Accessory name from HomeKit.
             is_on: True for on, False for off.
+            brightness: Brightness value 0-100, or None for on/off only.
         """
         config = self._find_accessory_config(accessory_name)
-        if config:
+        if not config:
+            self.logger.warning(f"No config found for accessory: {accessory_name}")
+            return
+
+        if brightness is not None:
+            # Handle brightness change
+            self._handle_brightness_change(accessory_name, config, brightness)
+        else:
+            # Handle on/off toggle
             action = config.on_action if is_on else config.off_action
             self.send_action(action)
             self.on_status_message.emit(
                 f"HomeKit: {accessory_name} {'ON' if is_on else 'OFF'}"
             )
+
+    def _handle_brightness_change(
+        self,
+        accessory_name: str,
+        config: "HomekitAccessoryConfig",
+        target_brightness: int,
+    ) -> None:
+        """
+        Handle brightness change by sending dimup/dimdown actions.
+
+        Calculates delta from current brightness and sends appropriate
+        number of LEVELINC or LEVELDEC commands (step = 10%).
+
+        Args:
+            accessory_name: Accessory name.
+            config: Accessory configuration.
+            target_brightness: Target brightness 0-100.
+        """
+        current = self._accessory_driver.get_brightness(accessory_name)
+        delta = target_brightness - current
+
+        if delta == 0:
+            return
+
+        # Determine action and steps (10% per step)
+        step_size = 10
+        steps = abs(delta) // step_size
+
+        if delta > 0:
+            # Increase brightness
+            if not config.dimup_action:
+                self.logger.warning(f"No dimup_action for {accessory_name}")
+                return
+            action = config.dimup_action
+            direction = "+"
         else:
-            self.logger.warning(f"No config found for accessory: {accessory_name}")
+            # Decrease brightness
+            if not config.dimdown_action:
+                self.logger.warning(f"No dimdown_action for {accessory_name}")
+                return
+            action = config.dimdown_action
+            direction = "-"
+
+        # Send action for each step
+        for _ in range(steps):
+            self.send_action(action)
+
+        self.on_status_message.emit(
+            f"HomeKit: {accessory_name} {current}% → {target_brightness}% ({direction}{steps * step_size}%)"
+        )
 
     def send_action(self, action: str) -> None:
         """