boneIO 1.2.0.dev23__py3-none-any.whl → 1.2.0.dev25__py3-none-any.whl

boneio/components/cover/cover.py

@@ -265,6 +265,10 @@ class BaseCover(BaseCoverABC, BasicMqtt):
         _LOGGER.debug("Toggle cover %s from input.", self._id)
         if self._current_operation != IDLE:
             await self.stop()
+        elif self._position >= 100:
+            await self.close()
+        elif self._position <= 0:
+            await self.open()
         elif self._last_operation == CLOSING:
             await self.open()
         else:
@@ -303,6 +307,10 @@ class BaseCover(BaseCoverABC, BasicMqtt):
             await self.stop()
         elif self._position <= always_open_till:
             await self.open()
+        elif self._position >= 100:
+            await self.close()
+        elif self._position <= 0:
+            await self.open()
         elif self._last_operation == CLOSING:
             await self.open()
         else:

boneio/const.py

@@ -187,6 +187,33 @@ cover_actions = {
     'TILT_CLOSE': 'tilt_close',
 }
 
+# Map of cover action method name -> accepted extra_data keys.
+# Actions not listed here (open, close, toggle, stop, etc.) accept no extra params.
+_COVER_ACTION_PARAMS: dict[str, set[str]] = {
+    "smart_toggle": {"always_open_till"},
+    "set_cover_position": {"position"},
+    "set_tilt": {"tilt_position"},
+    "set_cover_tilt_position": {"tilt_position"},
+}
+
+
+def filter_cover_extra_data(action_name: str, extra_data: dict) -> dict:
+    """Filter extra_data to only include params accepted by the cover action.
+
+    Args:
+        action_name: The cover method name (e.g. 'open', 'toggle', 'smart_toggle')
+        extra_data: Raw extra_data dict from action config
+
+    Returns:
+        Filtered dict with only accepted params for the action.
+    """
+    if not extra_data:
+        return {}
+    allowed = _COVER_ACTION_PARAMS.get(action_name)
+    if allowed is None:
+        return {}
+    return {k: v for k, v in extra_data.items() if k in allowed}
+
 INA219 = "ina219"
 VIRTUAL_ENERGY_SENSOR = "virtual_energy_sensor"
 PINS = {

boneio/core/manager/covers.py

@@ -13,7 +13,7 @@ import json
 import logging
 from typing import TYPE_CHECKING, Any
 
-from boneio.const import COVER, DEVICE_CLASS, ID, NAME, RESTORE_STATE, SHOW_HA, cover_actions
+from boneio.const import COVER, DEVICE_CLASS, ID, NAME, RESTORE_STATE, SHOW_HA, cover_actions, filter_cover_extra_data
 from boneio.core.utils import TimePeriod, strip_accents
 from boneio.exceptions import CoverConfigurationException
 from boneio.integration import ha_cover_availabilty_message
@@ -387,8 +387,10 @@ class CoverManager:
         
         try:
             _f = getattr(cover, action_to_execute)
-            if extra_data:
-                await _f(**extra_data)
+            # Filter extra_data to only pass params accepted by the action
+            filtered = filter_cover_extra_data(action_to_execute, extra_data or {})
+            if filtered:
+                await _f(**filtered)
             else:
                 await _f()
         except Exception as err:

boneio/core/manager/manager.py

@@ -28,6 +28,7 @@ from boneio.const import (
     STATE,
     TOGGLE,
     cover_actions,
+    filter_cover_extra_data,
     output_actions,
 )
 from boneio.core.config import ConfigHelper
@@ -669,6 +670,8 @@ class Manager:
                     continue
                 action_to_execute = action_definition.get("action_to_execute")
                 extra_data = action_definition.get("extra_data", {})
+                # Filter extra_data to only pass params accepted by each action
+                filtered_data = filter_cover_extra_data(action_to_execute, extra_data)
                 _LOGGER.debug(
                     "Executing action %s for cover %s. Duration: %s",
                     action_to_execute,
@@ -676,7 +679,7 @@ class Manager:
                     time.time() - start_time,
                 )
                 _f = getattr(cover, action_to_execute)
-                await _f(**extra_data)
+                await _f(**filtered_data)
                 
             elif action == OUTPUT_OVER_MQTT:
                 boneio_id = action_definition.get("boneio_id")

boneio/core/manager/remote.py

@@ -587,6 +587,7 @@ class RemoteDeviceManager:
             return await device.control_cover(
                 cover_id=cover_id,
                 action=action,
+                **kwargs,
             )
         
         # For MQTT devices, use standard control_cover

boneio/core/remote/esphome.py

@@ -364,7 +364,10 @@ class ESPHomeRemoteDevice(RemoteDevice):
                             "current_operation": current_op,
                             "last_known_operation": last_op,
                         }
-                        _LOGGER.debug("Cover '%s' position: %.2f", cover_id, state.position)
+                        _LOGGER.debug(
+                            "Cover '%s' state update: position=%.2f, current_op=%s, last_op=%s",
+                            cover_id, state.position, current_op, last_op,
+                        )
                         break
         except Exception as e:
             _LOGGER.debug("Error processing state change: %s", e)
@@ -854,27 +857,52 @@ class ESPHomeRemoteDevice(RemoteDevice):
                 self._client.cover_command(cover_key, position=0.0)
             elif action_upper == "STOP":
                 self._client.cover_command(cover_key, stop=True)
-            elif action_upper == "TOGGLE":
-                # Toggle based on current operation or last known operation
+            elif action_upper in ("TOGGLE", "SMART_TOGGLE"):
+                # Toggle based on current operation, position, or last known operation
                 state = self._cover_states.get(cover_id, {})
                 current_op = state.get("current_operation", 0) # 0=IDLE, 1=OPENING, 2=CLOSING
                 last_op = state.get("last_known_operation", 2)
                 cover_pos = state.get("position")
+                always_open_till = kwargs.get("always_open_till", 50) if action_upper == "SMART_TOGGLE" else None
                 _LOGGER.debug(
-                    "TOGGLE cover '%s': key=%s, state=%s, current_op=%s, last_op=%s, position=%s, all_states=%s",
-                    cover_id, cover_key, state, current_op, last_op, cover_pos,
-                    {k: v for k, v in self._cover_states.items()},
+                    "ESPHome %s cover '%s': current_op=%s, last_op=%s, position=%s, always_open_till=%s",
+                    action_upper, cover_id, current_op, last_op, cover_pos, always_open_till,
                 )
                 
                 if current_op != 0: # If moving, stop it
-                    _LOGGER.debug("TOGGLE -> STOP (cover is moving, current_op=%s)", current_op)
+                    _LOGGER.debug("%s -> STOP (cover is moving, current_op=%s)", action_upper, current_op)
                     self._client.cover_command(cover_key, stop=True)
-                elif last_op == 2: # If last operation was CLOSING, open it
-                    _LOGGER.debug("TOGGLE -> OPEN (last_op=CLOSING)")
-                    self._client.cover_command(cover_key, position=1.0)
-                else: # Otherwise close it
-                    _LOGGER.debug("TOGGLE -> CLOSE (last_op=%s)", last_op)
-                    self._client.cover_command(cover_key, position=0.0)
+                elif cover_pos is not None:
+                    # Position is known — use it for smarter decisions
+                    pos_pct = int(cover_pos * 100)
+                    if always_open_till is not None and pos_pct <= always_open_till:
+                        # SMART_TOGGLE: below threshold → always open
+                        _LOGGER.debug("SMART_TOGGLE -> OPEN (pos=%d%% <= threshold=%d%%)", pos_pct, always_open_till)
+                        self._client.cover_command(cover_key, position=1.0)
+                    elif cover_pos >= 0.99:
+                        # Fully open → close
+                        _LOGGER.debug("%s -> CLOSE (fully open, pos=%.2f)", action_upper, cover_pos)
+                        self._client.cover_command(cover_key, position=0.0)
+                    elif cover_pos <= 0.01:
+                        # Fully closed → open
+                        _LOGGER.debug("%s -> OPEN (fully closed, pos=%.2f)", action_upper, cover_pos)
+                        self._client.cover_command(cover_key, position=1.0)
+                    elif last_op == 2:
+                        # Last was closing → open
+                        _LOGGER.debug("%s -> OPEN (last_op=CLOSING)", action_upper)
+                        self._client.cover_command(cover_key, position=1.0)
+                    else:
+                        # Last was opening → close
+                        _LOGGER.debug("%s -> CLOSE (last_op=%s)", action_upper, last_op)
+                        self._client.cover_command(cover_key, position=0.0)
+                else:
+                    # No position known — use last_op only
+                    if last_op == 2:
+                        _LOGGER.debug("%s -> OPEN (no position, last_op=CLOSING)", action_upper)
+                        self._client.cover_command(cover_key, position=1.0)
+                    else:
+                        _LOGGER.debug("%s -> CLOSE (no position, last_op=%s)", action_upper, last_op)
+                        self._client.cover_command(cover_key, position=0.0)
             elif action_upper == "TOGGLE_OPEN":
                 state = self._cover_states.get(cover_id, {})
                 current_op = state.get("current_operation", 0)

boneio/version.py

@@ -1,2 +1,2 @@
 # flake8: noqa
-__version__ = "1.2.0dev23"
+__version__ = "1.2.0dev25"

{boneio-1.2.0.dev23.dist-info → boneio-1.2.0.dev25.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: boneio
-Version: 1.2.0.dev23
+Version: 1.2.0.dev25
 Summary: Python App for BoneIO
 Author-email: Paweł Szafer <pszafer@gmail.com>
 License: GNU General Public License v3.0

{boneio-1.2.0.dev23.dist-info → boneio-1.2.0.dev25.dist-info}/RECORD

@@ -1,9 +1,9 @@
 boneio/__init__.py,sha256=Lru1q30NeF7kcGS9XQNMseoVX3Rr2MGmAFHBE5E-bEI,174
 boneio/bonecli.py,sha256=7T5UAeiH65BUiPQXWLT5LrKcYL2apV3asvAFieHx2fg,10054
-boneio/const.py,sha256=5PuhJyhA8ufWj_U2hvlQCmJlFfTn1kap1yMVvqJwZ_M,6134
+boneio/const.py,sha256=703TSqvrACoftJdukAHAQFSENFS6F3i9d2T-bVasHPQ,7089
 boneio/exceptions.py,sha256=cBph-TsLF7FSz6p4wYeMs3pjVmVRaNQRJP82JgYYV-c,810
 boneio/runner.py,sha256=0SmyRqss0B7qJqglzt8lauuuB_Cv29rnB-pRQZ37dBk,12476
-boneio/version.py,sha256=3Pz8zAfxlbuE6_VQOvYDMiNWHKKn_-23XoJXdKfXeZs,42
+boneio/version.py,sha256=pbxLfl4OE6rc2VupmHc3fxPz3xsSnQRqYIup-ybNrT0,42
 boneio/boards/0.2/input.yaml,sha256=3HA3rBG23frhZWDcNu76U-Hz5r_uG6WvSsGnXkNE4nE,1581
 boneio/boards/0.2/output_24_16.yaml,sha256=GQGM3gUK7UobdGL-Ejt9UKdCs1HLPgZ7kziVZ1tjQlA,1462
 boneio/boards/0.2/output_32_10.yaml,sha256=169ubuzRqFLGiOr7zgB1xL1tKdQyOx2FJZvTCkxCA4U,1920
@@ -44,7 +44,7 @@ boneio/boards/0.8/output_cover.yaml,sha256=UW430upxrzznsowQ2naHxpmfE-rpz_Dxiyq7T
 boneio/boards/0.8/output_cover_mix.yaml,sha256=3XnGY2hzD7cGhcg6k3G4RCMXrvL09CgeJ1ImdqOIdRM,2656
 boneio/components/__init__.py,sha256=tdQjs22qE6-hlNE1vT4V2xEEHJjJIzlbfQwK2cuJbMo,33
 boneio/components/cover/__init__.py,sha256=g0sgP_x8pMW6YiuDY0h7jqmrO7-AP1UiVCRSHu0Ih0M,143
-boneio/components/cover/cover.py,sha256=k8tnlJAsEMZ61LepmseCtixPF6OIqxS9bJ-ZZbqfZ84,11150
+boneio/components/cover/cover.py,sha256=jB_EmzbFUJWZQrDquJ5DpCvrGiTL_fR7GqJPCrWStHU,11412
 boneio/components/cover/time_based.py,sha256=uwCYGFZh_00GfFfUv5eNQBtbTX-hGaVZJfwLDEpsQUY,5864
 boneio/components/cover/venetian.py,sha256=-anHgL51Lar11OQhhCCaIQfk6C2Fm-MG6LSli4Xou1o,13053
 boneio/components/group/__init__.py,sha256=X4HnybJVvguhKt81qeQSzSeSiUTsTDieSfxIOGCaERc,87
@@ -85,13 +85,13 @@ boneio/core/config/migrations/v2_transition.py,sha256=X-3kPn9y7eJtUlcK1eMTDwQVJA
 boneio/core/events/__init__.py,sha256=8jUQZitD9wyVr877acMuMNwOC4Wg7qIdtmL4RSzhyTQ,249
 boneio/core/events/bus.py,sha256=6AwQyFvwjyD1PGxvQ5eEiOGlaCw3mlmKzxMdJ4XLkPU,17391
 boneio/core/manager/__init__.py,sha256=Vp6LqoIoQnHrGlZRRTNb1VVuzp7Y6VWsor1FmoVjL6k,1071
-boneio/core/manager/covers.py,sha256=-WM8miMLF2Qgfm77u5gSo61G-0aQaDcFrCreoX6j750,18621
+boneio/core/manager/covers.py,sha256=UZ8M_tr3gqsC2N35O2flI09P7aYRjS6PC7BlFdSMiFk,18801
 boneio/core/manager/display.py,sha256=qXOHMbSer62Br3HdQGHC8G1VJBa5mwSpI8RmKSCF4dM,9237
 boneio/core/manager/inputs.py,sha256=vOOY5G_vl5MM9ZVspeIzAt4mSROHwi3OfB8HFJU_944,37306
-boneio/core/manager/manager.py,sha256=YE5RiXjdWEdaibsHGgxt1IDBULWrATEpzR9nWNXhCtA,49531
+boneio/core/manager/manager.py,sha256=eV7jejrVlVGgnmzg9grf1yEY3lHIu6DrAux12B9XCuQ,49730
 boneio/core/manager/modbus.py,sha256=Pn96F4uXBL0UMAcqM21C7NK6hR9J8soW0quJFdPSfZQ,15788
 boneio/core/manager/outputs.py,sha256=Ao7qX6lR67IidxeVwjibNflaET_pnK2OGGBm_VMRTXI,33839
-boneio/core/manager/remote.py,sha256=1AihcJUuCWLuYc2ZStzd4QgARXdBxyQOWreHKfkqPHk,38376
+boneio/core/manager/remote.py,sha256=HS6WZXepQ6Iwh3Sar_n_6ABXmtzvUwa2Dg3rcJnciT0,38402
 boneio/core/manager/sensors.py,sha256=MRVJU4rRtsBqDslmQvOgMQleTMvOkD3dYKQRscZNgME,47494
 boneio/core/manager/templates.py,sha256=QUzi3irFXFXssycyaysDBzqMHfbx9cuD0PtIzybhHC8,30748
 boneio/core/manager/update.py,sha256=vsOZNHY8gULmEwG14IgjXKXZsYG1DMzpDM7YML5PFq4,25004
@@ -107,7 +107,7 @@ boneio/core/messaging/mqtt.py,sha256=zSQGGGIj7h5bmJ9zD5Svq5LDusRdtvOY6PJzULeXnbA
 boneio/core/messaging/queue.py,sha256=bZvGAQhMLAInaU5G67cpmbdIEjgn8MGmU5B1MlaPY2A,2395
 boneio/core/remote/__init__.py,sha256=BcVoQXS4iAjkSHI4l3NTEBweIpIh1rvcu43APW-Y1Oo,1048
 boneio/core/remote/base.py,sha256=aX1WWEiQSBUd8qV9G5W21Uzs_2EUdfNP-jPOn77M2dY,5544
-boneio/core/remote/esphome.py,sha256=J40_zEuvfanR38ZBASH9t8Zn-ISpuevv34Z8iZqod1s,43662
+boneio/core/remote/esphome.py,sha256=e6u28vNnj6wAuseqrUQws8SrJ_r3Kmo4_2dxlv3L4Y4,45539
 boneio/core/remote/mqtt.py,sha256=B5H_3bpBZtAYIXcIbyN846CmCzJX1Q2MNQBEsZZJJXU,7472
 boneio/core/remote/wled.py,sha256=rEDkHDejbJwFKwpgQIwWSXK9ceotx2mhNWxuKjgWUjk,19190
 boneio/core/sensor/__init__.py,sha256=pAcSvGsufNAT5Jy79bC4Ncqo420EjtdZSYrUwSB5yCY,274
@@ -430,9 +430,9 @@ boneio/webui/schema/virtual_energy_sensor.schema.json,sha256=BN6Egh-CAnXSa80bQ6Y
 boneio/webui/schema/web.schema.json,sha256=cactLq1MVwDL5hu8bLeuq5yiz49BWVS4OtjWLmhbw2M,2592
 boneio/webui/services/__init__.py,sha256=do_t59tE9zi1qNKkbkq6Et-gc5et4iKyYMntnQBXVG8,282
 boneio/webui/services/logs.py,sha256=ampdsOx_xwOr0gRC-fW65c1y5Po0nACgUw0tG2bdqUM,11426
-boneio-1.2.0.dev23.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
-boneio-1.2.0.dev23.dist-info/METADATA,sha256=f1Lucr9Iqqo49W7p6v4StTaGzp5xMRGXJXhvrEx0eeU,1924
-boneio-1.2.0.dev23.dist-info/WHEEL,sha256=YCfwYGOYMi5Jhw2fU4yNgwErybb2IX5PEwBKV4ZbdBo,91
-boneio-1.2.0.dev23.dist-info/entry_points.txt,sha256=JX7gluDIkQ6MBlzzxGHMItnBjciiJ73tsXwYcazMW_U,47
-boneio-1.2.0.dev23.dist-info/top_level.txt,sha256=ag9S5n1_KXZfyk9tN8R0YUDYbdA--e5a2w2FOD_yMdg,7
-boneio-1.2.0.dev23.dist-info/RECORD,,
+boneio-1.2.0.dev25.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
+boneio-1.2.0.dev25.dist-info/METADATA,sha256=WgIzYieCGRXlsW5xrNW2LUrZ6jaivJWlzipgw7QELhk,1924
+boneio-1.2.0.dev25.dist-info/WHEEL,sha256=YCfwYGOYMi5Jhw2fU4yNgwErybb2IX5PEwBKV4ZbdBo,91
+boneio-1.2.0.dev25.dist-info/entry_points.txt,sha256=JX7gluDIkQ6MBlzzxGHMItnBjciiJ73tsXwYcazMW_U,47
+boneio-1.2.0.dev25.dist-info/top_level.txt,sha256=ag9S5n1_KXZfyk9tN8R0YUDYbdA--e5a2w2FOD_yMdg,7
+boneio-1.2.0.dev25.dist-info/RECORD,,