39 changed files with 282 additions and 563 deletions
--- a/.flake8
+++ b/.flake8
@ -1,3 +0,0 @@
-[flake8]
-max-line-length = 88
-max-complexity = 7
--- a/.github/workflows/python-check.yml
+++ b/.github/workflows/python-check.yml
@ -13,7 +13,7 @@ jobs:
    strategy:
      fail-fast: false
      matrix:
-        python-version: ["3.10", "3.11", "3.12"]
+        python-version: ["3.10", "3.11"]

    steps:
    - uses: actions/checkout@v3
@ -28,13 +28,15 @@ jobs:
        python -m pip install -r requirements_dev.txt
    - name: Lint with flake8
      run: |
-        flake8 . --count --statistics
+        # stop the build if there are Python syntax errors or undefined names
+        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=88 --statistics
    - name: Type check with mypy
      run: |
        mypy pyhon/
-    - name: Analysing the code with pylint
-      run: |
-        pylint $(git ls-files '*.py')
+    # - name: Analysing the code with pylint
+    #   run: |
+    #     pylint --max-line-length 88 $(git ls-files '*.py')
    - name: Check black style
      run: |
        black . --check
--- a/.pylintrc
+++ b/.pylintrc
@ -1,9 +0,0 @@
-[MESSAGES CONTROL]
-
-disable=missing-docstring
-
-[FORMAT]
-
-max-args=6
-max-attributes=8
-max-line-length=88
--- a/MANIFEST.in
+++ b/MANIFEST.in
@ -1 +0,0 @@
-include pyhon/py.typed
--- a/README.md
+++ b/README.md
@ -1,14 +1,3 @@
-# Announcement: I have to take the project down in the next few days
-> Dear User,
-> 
-> We are writing to inform you that we have discovered two Home Assistant integration plug-ins developed by you ( https://github.com/Andre0512/hon and https://github.com/Andre0512/pyhOn ) that are in violation of our terms of service. Specifically, the plug-ins are using our services in an unauthorized manner which is causing significant economic harm to our Company.
-> We take the protection of our intellectual property very seriously and demand that you immediately cease and desist all illegal activities related to the development and distribution of these plug-ins. We also request that you remove the plug-ins from all stores and code hosting platforms where they are currently available.
-> Please be advised that we will take all necessary legal action to protect our interests if you fail to comply with this notice. We reserve the right to pursue all available remedies, including but not limited to monetary damages, injunctive relief, and attorney's fees.
-> We strongly urge you to take immediate action to rectify this situation and avoid any further legal action. If you have any questions or concerns, please do not hesitate to contact us.
-> 
-> Haier Europe Security and Governance Department
-
-
 **This python package is unofficial and is not related in any way to Haier. It was developed by reversed engineered requests and can stop working at anytime!**

 # pyhOn
--- a/mypy.ini
+++ b/mypy.ini
@ -1,24 +1,4 @@
 [mypy]
-check_untyped_defs = true
-disallow_any_generics = true
-disallow_any_unimported = true
-disallow_incomplete_defs = true
-disallow_subclassing_any = true
-disallow_untyped_calls = true
-disallow_untyped_decorators = true
-disallow_untyped_defs = true
-disable_error_code = annotation-unchecked
-enable_error_code = ignore-without-code, redundant-self, truthy-iterable
-follow_imports = silent
-local_partial_types = true
-no_implicit_optional = true
-no_implicit_reexport = true
-show_error_codes = true
-strict_concatenate = false
-strict_equality = true
-warn_incomplete_stub = true
-warn_redundant_casts = true
-warn_return_any = true
-warn_unreachable = true
-warn_unused_configs = true
-warn_unused_ignores = true
+check_untyped_defs = True
+disallow_any_generics = True
+disallow_untyped_defs = True
--- a/pyhon/main.py
+++ b/pyhon/main.py
@ -11,7 +11,6 @@ from typing import Tuple, Dict, Any
 if __name__ == "__main__":
    sys.path.insert(0, str(Path(__file__).parent.parent))

-# pylint: disable=wrong-import-position
 from pyhon import Hon, HonAPI, diagnose, printer

 _LOGGER = logging.getLogger(__name__)
@ -31,13 +30,13 @@ def get_arguments() -> Dict[str, Any]:
    export.add_argument("--zip", help="create zip archive", action="store_true")
    export.add_argument("--anonymous", help="anonymize data", action="store_true")
    export.add_argument("directory", nargs="?", default=Path().cwd())
-    translation = subparser.add_parser(
+    translate = subparser.add_parser(
        "translate", help="print available translation keys"
    )
-    translation.add_argument(
+    translate.add_argument(
        "translate", help="language (de, en, fr...)", metavar="LANGUAGE"
    )
-    translation.add_argument("--json", help="print as json", action="store_true")
+    translate.add_argument("--json", help="print as json", action="store_true")
    parser.add_argument(
        "-i", "--import", help="import pyhon data", nargs="?", default=Path().cwd()
    )
@ -92,13 +91,15 @@ async def main() -> None:
                data = device.data.copy()
                attr = "get" if args.get("all") else "pop"
                print(
-                    printer.key_print(getattr(data["attributes"], attr)("parameters"))
+                    printer.key_print(
+                        data["attributes"].__getattribute__(attr)("parameters")
                    )
-                print(printer.key_print(getattr(data, attr)("appliance")))
+                )
+                print(printer.key_print(data.__getattribute__(attr)("appliance")))
                print(printer.key_print(data))
                print(
                    printer.pretty_print(
-                        printer.create_commands(device.commands, concat=True)
+                        printer.create_command(device.commands, concat=True)
                    )
                )
            else:
--- a/pyhon/appliance.py
+++ b/pyhon/appliance.py
@ -3,15 +3,13 @@ import logging
 import re
 from datetime import datetime, timedelta
 from pathlib import Path
-from typing import Optional, Dict, Any, TYPE_CHECKING, List, TypeVar, overload
+from typing import Optional, Dict, Any, TYPE_CHECKING, List

 from pyhon import diagnose, exceptions
-from pyhon.appliances.base import ApplianceBase
 from pyhon.attributes import HonAttribute
 from pyhon.command_loader import HonCommandLoader
 from pyhon.commands import HonCommand
 from pyhon.parameter.base import HonParameter
-from pyhon.parameter.enum import HonParameterEnum
 from pyhon.parameter.range import HonParameterRange
 from pyhon.typedefs import Parameter

@ -20,10 +18,7 @@ if TYPE_CHECKING:

 _LOGGER = logging.getLogger(__name__)

-T = TypeVar("T")

-
-# pylint: disable=too-many-public-methods,too-many-instance-attributes
 class HonAppliance:
    _MINIMAL_UPDATE_INTERVAL = 5  # seconds

@ -45,41 +40,30 @@ class HonAppliance:
        self._default_setting = HonParameter("", {}, "")

        try:
-            self._extra: Optional[ApplianceBase] = importlib.import_module(
+            self._extra = importlib.import_module(
                f"pyhon.appliances.{self.appliance_type.lower()}"
            ).Appliance(self)
        except ModuleNotFoundError:
            self._extra = None

-    def _get_nested_item(self, item: str) -> Any:
-        result: List[Any] | Dict[str, Any] = self.data
-        for key in item.split("."):
-            if all(k in "0123456789" for k in key) and isinstance(result, list):
-                result = result[int(key)]
-            elif isinstance(result, dict):
-                result = result[key]
-        return result
-
    def __getitem__(self, item: str) -> Any:
        if self._zone:
            item += f"Z{self._zone}"
        if "." in item:
-            return self._get_nested_item(item)
+            result = self.data
+            for key in item.split("."):
+                if all(k in "0123456789" for k in key) and isinstance(result, list):
+                    result = result[int(key)]
+                else:
+                    result = result[key]
+            return result
        if item in self.data:
            return self.data[item]
        if item in self.attributes["parameters"]:
            return self.attributes["parameters"][item].value
        return self.info[item]

-    @overload
-    def get(self, item: str, default: None = None) -> Any:
-        ...
-
-    @overload
-    def get(self, item: str, default: T) -> T:
-        ...
-
-    def get(self, item: str, default: Optional[T] = None) -> Any:
+    def get(self, item: str, default: Any = None) -> Any:
        try:
            return self[item]
        except (KeyError, IndexError):
@ -87,22 +71,21 @@ class HonAppliance:

    def _check_name_zone(self, name: str, frontend: bool = True) -> str:
        zone = " Z" if frontend else "_z"
-        attribute: str = self._info.get(name, "")
-        if attribute and self._zone:
+        if (attribute := self._info.get(name, "")) and self._zone:
            return f"{attribute}{zone}{self._zone}"
        return attribute

    @property
    def appliance_model_id(self) -> str:
-        return str(self._info.get("applianceModelId", ""))
+        return self._info.get("applianceModelId", "")

    @property
    def appliance_type(self) -> str:
-        return str(self._info.get("applianceTypeName", ""))
+        return self._info.get("applianceTypeName", "")

    @property
    def mac_address(self) -> str:
-        return str(self.info.get("macAddress", ""))
+        return self.info.get("macAddress", "")

    @property
    def unique_id(self) -> str:
@ -118,31 +101,29 @@ class HonAppliance:

    @property
    def brand(self) -> str:
-        brand = self._check_name_zone("brand")
-        return brand[0].upper() + brand[1:]
+        return self._check_name_zone("brand")

    @property
    def nick_name(self) -> str:
        result = self._check_name_zone("nickName")
-        if not result or re.findall("^[xX1\\s-]+$", result):
+        if not result or re.findall("^[xX\s]+$", result):
            return self.model_name
        return result

    @property
    def code(self) -> str:
-        code: str = self.info.get("code", "")
-        if code:
+        if code := self.info.get("code"):
            return code
-        serial_number: str = self.info.get("serialNumber", "")
+        serial_number = self.info.get("serialNumber", "")
        return serial_number[:8] if len(serial_number) < 18 else serial_number[:11]

    @property
    def model_id(self) -> int:
-        return int(self._info.get("applianceModelId", 0))
+        return self._info.get("applianceModelId", 0)

    @property
    def options(self) -> Dict[str, Any]:
-        return dict(self._appliance_model.get("options", {}))
+        return self._appliance_model.get("options", {})

    @property
    def commands(self) -> Dict[str, HonCommand]:
@ -181,18 +162,16 @@ class HonAppliance:
        self._commands = command_loader.commands
        self._additional_data = command_loader.additional_data
        self._appliance_model = command_loader.appliance_data
-        self.sync_params_to_command("settings")

    async def load_attributes(self) -> None:
-        attributes = await self.api.load_attributes(self)
-        for name, values in attributes.pop("shadow", {}).get("parameters", {}).items():
+        self._attributes = await self.api.load_attributes(self)
+        for name, values in self._attributes.pop("shadow").get("parameters").items():
            if name in self._attributes.get("parameters", {}):
                self._attributes["parameters"][name].update(values)
            else:
                self._attributes.setdefault("parameters", {})[name] = HonAttribute(
                    values
                )
-        self._attributes |= attributes
        if self._extra:
            self._attributes = self._extra.attributes(self._attributes)

@ -202,11 +181,14 @@ class HonAppliance:

    async def update(self, force: bool = False) -> None:
        now = datetime.now()
-        min_age = now - timedelta(seconds=self._MINIMAL_UPDATE_INTERVAL)
-        if force or not self._last_update or self._last_update < min_age:
+        if (
+            force
+            or not self._last_update
+            or self._last_update
+            < now - timedelta(seconds=self._MINIMAL_UPDATE_INTERVAL)
+        ):
            self._last_update = now
            await self.load_attributes()
-            self.sync_params_to_command("settings")

    @property
    def command_parameters(self) -> Dict[str, Dict[str, str | float]]:
@ -214,7 +196,7 @@ class HonAppliance:

    @property
    def settings(self) -> Dict[str, Parameter]:
-        result: Dict[str, Parameter] = {}
+        result = {}
        for name, command in self._commands.items():
            for key in command.setting_keys:
                setting = command.settings.get(key, self._default_setting)
@ -250,67 +232,32 @@ class HonAppliance:
    async def data_archive(self, path: Path) -> str:
        return await diagnose.zip_archive(self, path, anonymous=True)

-    def sync_command_to_params(self, command_name: str) -> None:
+    def sync_to_params(self, command_name: str) -> None:
        if not (command := self.commands.get(command_name)):
            return
-        for key in self.attributes.get("parameters", {}):
-            if new := command.parameters.get(key):
+        for key, value in self.attributes.get("parameters", {}).items():
+            if isinstance(value, str) and (new := command.parameters.get(key)):
                self.attributes["parameters"][key].update(
                    str(new.intern_value), shield=True
                )

-    def sync_params_to_command(self, command_name: str) -> None:
-        if not (command := self.commands.get(command_name)):
-            return
-        for key in command.setting_keys:
-            if (
-                new := self.attributes.get("parameters", {}).get(key)
-            ) is None or new.value == "":
-                continue
-            setting = command.settings[key]
-            try:
-                if not isinstance(setting, HonParameterRange):
-                    command.settings[key].value = str(new.value)
-                else:
-                    command.settings[key].value = float(new.value)
-            except ValueError as error:
-                _LOGGER.info("Can't set %s - %s", key, error)
-                continue
-
-    def sync_command(
-        self,
-        main: str,
-        target: Optional[List[str] | str] = None,
-        to_sync: Optional[List[str] | bool] = None,
-    ) -> None:
+    def sync_command(self, main: str, target: Optional[List[str]] = None) -> None:
        base: Optional[HonCommand] = self.commands.get(main)
        if not base:
            return
        for command, data in self.commands.items():
            if command == main or target and command not in target:
                continue
-
-            for name, target_param in data.parameters.items():
-                if not (base_param := base.parameters.get(name)):
-                    continue
-                if to_sync and (
-                    (isinstance(to_sync, list) and name not in to_sync)
-                    or not base_param.mandatory
+            for name, parameter in data.parameters.items():
+                if base_value := base.parameters.get(name):
+                    if isinstance(base_value, HonParameterRange) and isinstance(
+                        parameter, HonParameterRange
                    ):
-                    continue
-                self.sync_parameter(base_param, target_param)
-
-    def sync_parameter(self, main: Parameter, target: Parameter) -> None:
-        if isinstance(main, HonParameterRange) and isinstance(
-            target, HonParameterRange
-        ):
-            target.max = main.max
-            target.min = main.min
-            target.step = main.step
-        elif isinstance(target, HonParameterRange):
-            target.max = int(main.value)
-            target.min = int(main.value)
-            target.step = 1
-        elif isinstance(target, HonParameterEnum):
-            target.values = main.values
-        target.value = main.value
+                        parameter.max = base_value.max
+                        parameter.min = base_value.min
+                        parameter.step = base_value.step
+                    elif isinstance(parameter, HonParameterRange):
+                        parameter.max = int(base_value.value)
+                        parameter.min = int(base_value.value)
+                        parameter.step = 1
+                    parameter.value = base_value.value
--- a/pyhon/appliances/dw.py
+++ b/pyhon/appliances/dw.py
@ -1,4 +1,3 @@
-# pylint: disable=duplicate-code
 from typing import Any, Dict

 from pyhon.appliances.base import ApplianceBase
--- a/pyhon/appliances/ov.py
+++ b/pyhon/appliances/ov.py
@ -1,16 +1,24 @@
 from typing import Any, Dict

 from pyhon.appliances.base import ApplianceBase
+from pyhon.parameter.program import HonParameterProgram


 class Appliance(ApplianceBase):
    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
        data = super().attributes(data)
        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
-            data["parameters"]["temp"].value = 0
-            data["parameters"]["onOffStatus"].value = 0
-            data["parameters"]["remoteCtrValid"].value = 0
-            data["parameters"]["remainingTimeMM"].value = 0
+            data["parameters"]["temp"].value = "0"
+            data["parameters"]["onOffStatus"].value = "0"
+            data["parameters"]["remoteCtrValid"].value = "0"
+            data["parameters"]["remainingTimeMM"].value = "0"
+
+        data["active"] = data["parameters"]["onOffStatus"] == "1"
+
+        if program := int(data["parameters"]["prCode"]):
+            if (setting := self.parent.settings["startProgram.program"]) and isinstance(
+                setting, HonParameterProgram
+            ):
+                data["programName"] = setting.ids.get(program, "")

-        data["active"] = data["parameters"]["onOffStatus"].value == 1
        return data
--- a/pyhon/appliances/ref.py
+++ b/pyhon/appliances/ref.py
@ -10,12 +10,12 @@ class Appliance(ApplianceBase):
            data["modeZ1"] = "holiday"
        elif data["parameters"]["intelligenceMode"] == "1":
            data["modeZ1"] = "auto_set"
-        elif data["parameters"].get("quickModeZ1") == "1":
+        elif data["parameters"]["quickModeZ1"] == "1":
            data["modeZ1"] = "super_cool"
        else:
            data["modeZ1"] = "no_mode"

-        if data["parameters"].get("quickModeZ2") == "1":
+        if data["parameters"]["quickModeZ2"] == "1":
            data["modeZ2"] = "super_freeze"
        elif data["parameters"]["intelligenceMode"] == "1":
            data["modeZ2"] = "auto_set"
--- a/pyhon/appliances/td.py
+++ b/pyhon/appliances/td.py
@ -1,4 +1,3 @@
-# pylint: disable=duplicate-code
 from typing import Any, Dict

 from pyhon.appliances.base import ApplianceBase
--- a/pyhon/appliances/wd.py
+++ b/pyhon/appliances/wd.py
@ -1,4 +1,3 @@
-# pylint: disable=duplicate-code
 from typing import Dict, Any

 from pyhon.appliances.base import ApplianceBase
--- a/pyhon/appliances/wh.py
+++ b/pyhon/appliances/wh.py
@ -1,16 +0,0 @@
-from typing import Any, Dict
-
-from pyhon.appliances.base import ApplianceBase
-from pyhon.parameter.base import HonParameter
-
-
-class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
-        data = super().attributes(data)
-        parameter = data.get("parameters", {}).get("onOffStatus")
-        is_class = isinstance(parameter, HonParameter)
-        data["active"] = parameter.value == 1 if is_class else parameter == 1
-        return data
-
-    def settings(self, settings: Dict[str, Any]) -> Dict[str, Any]:
-        return settings
--- a/pyhon/appliances/wm.py
+++ b/pyhon/appliances/wm.py
@ -1,4 +1,3 @@
-# pylint: disable=duplicate-code
 from typing import Any, Dict

 from pyhon.appliances.base import ApplianceBase
--- a/pyhon/command_loader.py
+++ b/pyhon/command_loader.py
@ -1,15 +1,14 @@
 import asyncio
 from contextlib import suppress
 from copy import copy
-from typing import Dict, Any, Optional, TYPE_CHECKING, List
+from typing import Dict, Any, Optional, TYPE_CHECKING, List, Collection

 from pyhon.commands import HonCommand
-from pyhon.exceptions import NoAuthenticationException
 from pyhon.parameter.fixed import HonParameterFixed
 from pyhon.parameter.program import HonParameterProgram

 if TYPE_CHECKING:
-    from pyhon import HonAPI
+    from pyhon import HonAPI, exceptions
    from pyhon.appliance import HonAppliance


@ -17,12 +16,12 @@ class HonCommandLoader:
    """Loads and parses hOn command data"""

    def __init__(self, api: "HonAPI", appliance: "HonAppliance") -> None:
-        self._api: "HonAPI" = api
-        self._appliance: "HonAppliance" = appliance
        self._api_commands: Dict[str, Any] = {}
        self._favourites: List[Dict[str, Any]] = []
        self._command_history: List[Dict[str, Any]] = []
        self._commands: Dict[str, HonCommand] = {}
+        self._api: "HonAPI" = api
+        self._appliance: "HonAppliance" = appliance
        self._appliance_data: Dict[str, Any] = {}
        self._additional_data: Dict[str, Any] = {}

@ -30,7 +29,7 @@ class HonCommandLoader:
    def api(self) -> "HonAPI":
        """api connection object"""
        if self._api is None:
-            raise NoAuthenticationException("Missing hOn login")
+            raise exceptions.NoAuthenticationException("Missing hOn login")
        return self._api

    @property
@ -56,7 +55,7 @@ class HonCommandLoader:
    async def load_commands(self) -> None:
        """Trigger loading of command data"""
        await self._load_data()
-        self._appliance_data = self._api_commands.pop("applianceModel", {})
+        self._appliance_data = self._api_commands.pop("applianceModel")
        self._get_commands()
        self._add_favourites()
        self._recover_last_command_states()
@ -184,44 +183,22 @@ class HonCommandLoader:
    def _add_favourites(self) -> None:
        """Patch program categories with favourites"""
        for favourite in self._favourites:
-            name, command_name, base = self._get_favourite_info(favourite)
-            if not base:
-                continue
-            base_command: HonCommand = copy(base)
-            self._update_base_command_with_data(base_command, favourite)
-            self._update_base_command_with_favourite(base_command)
-            self._update_program_categories(command_name, name, base_command)
-
-    def _get_favourite_info(
-        self, favourite: Dict[str, Any]
-    ) -> tuple[str, str, HonCommand | None]:
-        name: str = favourite.get("favouriteName", {})
+            name = favourite.get("favouriteName", {})
            command = favourite.get("command", {})
-        command_name: str = command.get("commandName", "")
+            command_name = command.get("commandName", "")
            program_name = self._clean_name(command.get("programName", ""))
-        base_command = self.commands[command_name].categories.get(program_name)
-        return name, command_name, base_command
-
-    def _update_base_command_with_data(
-        self, base_command: HonCommand, command: Dict[str, Any]
-    ) -> None:
+            base: HonCommand = copy(
+                self.commands[command_name].categories[program_name]
+            )
            for data in command.values():
                if isinstance(data, str):
                    continue
                for key, value in data.items():
-                if not (parameter := base_command.parameters.get(key)):
-                    continue
+                    if parameter := base.parameters.get(key):
                        with suppress(ValueError):
                            parameter.value = value
-
-    def _update_base_command_with_favourite(self, base_command: HonCommand) -> None:
            extra_param = HonParameterFixed("favourite", {"fixedValue": "1"}, "custom")
-        base_command.parameters.update(favourite=extra_param)
-
-    def _update_program_categories(
-        self, command_name: str, name: str, base_command: HonCommand
-    ) -> None:
-        program = base_command.parameters["program"]
-        if isinstance(program, HonParameterProgram):
+            base.parameters.update(favourite=extra_param)
+            if isinstance(program := base.parameters["program"], HonParameterProgram):
                program.set_value(name)
-        self.commands[command_name].categories[name] = base_command
+            self.commands[command_name].categories[name] = base
--- a/pyhon/commands.py
+++ b/pyhon/commands.py
@ -27,16 +27,17 @@ class HonCommand:
        categories: Optional[Dict[str, "HonCommand"]] = None,
        category_name: str = "",
    ):
-        self._name: str = name
-        self._api: Optional[HonAPI] = None
+        self._api: Optional[HonAPI] = appliance.api
        self._appliance: "HonAppliance" = appliance
+        self._name: str = name
        self._categories: Optional[Dict[str, "HonCommand"]] = categories
        self._category_name: str = category_name
-        self._parameters: Dict[str, Parameter] = {}
+        self._description: str = attributes.pop("description", "")
+        self._protocol_type: str = attributes.pop("protocolType", "")
+        self._parameters: Dict[str, HonParameter] = {}
        self._data: Dict[str, Any] = {}
+        self._available_settings: Dict[str, HonParameter] = {}
        self._rules: List[HonRuleSet] = []
-        attributes.pop("description", "")
-        attributes.pop("protocolType", "")
        self._load_parameters(attributes)

    def __repr__(self) -> str:
@ -48,8 +49,6 @@ class HonCommand:

    @property
    def api(self) -> "HonAPI":
-        if self._api is None and self._appliance:
-            self._api = self._appliance.api
        if self._api is None:
            raise exceptions.NoAuthenticationException("Missing hOn login")
        return self._api
@ -77,23 +76,12 @@ class HonCommand:
            result.setdefault(parameter.group, {})[name] = parameter.intern_value
        return result

-    @property
-    def mandatory_parameter_groups(self) -> Dict[str, Dict[str, Union[str, float]]]:
-        result: Dict[str, Dict[str, Union[str, float]]] = {}
-        for name, parameter in self._parameters.items():
-            if parameter.mandatory:
-                result.setdefault(parameter.group, {})[name] = parameter.intern_value
-        return result
-
    @property
    def parameter_value(self) -> Dict[str, Union[str, float]]:
        return {n: p.value for n, p in self._parameters.items()}

-    def _load_parameters(self, attributes: Dict[str, Dict[str, Any] | Any]) -> None:
+    def _load_parameters(self, attributes: Dict[str, Dict[str, Any]]) -> None:
        for key, items in attributes.items():
-            if not isinstance(items, dict):
-                _LOGGER.info("Loading Attributes - Skipping %s", str(items))
-                continue
            for name, data in items.items():
                self._create_parameters(data, name, key)
        for rule in self._rules:
@ -105,12 +93,10 @@ class HonCommand:
        if name == "zoneMap" and self._appliance.zone:
            data["default"] = self._appliance.zone
        if data.get("category") == "rule":
-            if "fixedValue" in data:
-                self._rules.append(HonRuleSet(self, data["fixedValue"]))
-            elif "enumValues" in data:
-                self._rules.append(HonRuleSet(self, data["enumValues"]))
+            if "fixedValue" not in data:
+                _LOGGER.error("Rule not supported: %s", data)
            else:
-                _LOGGER.warning("Rule not supported: %s", data)
+                self._rules.append(HonRuleSet(self, data["fixedValue"]))
        match data.get("typology"):
            case "range":
                self._parameters[name] = HonParameterRange(name, data, parameter)
@ -125,33 +111,14 @@ class HonCommand:
            name = "program" if "PROGRAM" in self._category_name else "category"
            self._parameters[name] = HonParameterProgram(name, self, "custom")

-    async def send(self, only_mandatory: bool = False) -> bool:
-        grouped_params = (
-            self.mandatory_parameter_groups if only_mandatory else self.parameter_groups
-        )
-        params = grouped_params.get("parameters", {})
-        return await self.send_parameters(params)
-
-    async def send_specific(self, param_names: List[str]) -> bool:
-        params: Dict[str, str | float] = {}
-        for key, parameter in self._parameters.items():
-            if key in param_names or parameter.mandatory:
-                params[key] = parameter.value
-        return await self.send_parameters(params)
-
-    async def send_parameters(self, params: Dict[str, str | float]) -> bool:
+    async def send(self) -> bool:
+        params = self.parameter_groups.get("parameters", {})
        ancillary_params = self.parameter_groups.get("ancillaryParameters", {})
        ancillary_params.pop("programRules", None)
-        if "prStr" in params:
-            params["prStr"] = self._category_name.upper()
-        self.appliance.sync_command_to_params(self.name)
+        self.appliance.sync_to_params(self.name)
        try:
            result = await self.api.send_command(
-                self._appliance,
-                self._name,
-                params,
-                ancillary_params,
-                self._category_name,
+                self._appliance, self._name, params, ancillary_params
            )
            if not result:
                _LOGGER.error(result)
@ -202,7 +169,3 @@ class HonCommand:
                else:
                    result[name] = parameter
        return result
-
-    def reset(self) -> None:
-        for parameter in self._parameters.values():
-            parameter.reset()
--- a/pyhon/connection/api.py
+++ b/pyhon/connection/api.py
@ -75,12 +75,8 @@ class HonAPI:

    async def load_appliances(self) -> List[Dict[str, Any]]:
        async with self._hon.get(f"{const.API_URL}/commands/v1/appliance") as resp:
-            result = await resp.json()
-        if result:
-            appliances: List[Dict[str, Any]] = result.get("payload", {}).get(
-                "appliances", {}
-            )
-            return appliances
+            if result := await resp.json():
+                return result.get("payload", {}).get("appliances", {})
        return []

    async def load_commands(self, appliance: HonAppliance) -> Dict[str, Any]:
@ -116,8 +112,7 @@ class HonAPI:
            result: Dict[str, Any] = await response.json()
            if not result or not result.get("payload"):
                return []
-        command_history: List[Dict[str, Any]] = result["payload"]["history"]
-        return command_history
+            return result["payload"]["history"]

    async def load_favourites(self, appliance: HonAppliance) -> List[Dict[str, Any]]:
        url: str = (
@ -127,17 +122,14 @@ class HonAPI:
            result: Dict[str, Any] = await response.json()
            if not result or not result.get("payload"):
                return []
-        favourites: List[Dict[str, Any]] = result["payload"]["favourites"]
-        return favourites
+            return result["payload"]["favourites"]

    async def load_last_activity(self, appliance: HonAppliance) -> Dict[str, Any]:
        url: str = f"{const.API_URL}/commands/v1/retrieve-last-activity"
        params: Dict[str, str] = {"macAddress": appliance.mac_address}
        async with self._hon.get(url, params=params) as response:
            result: Dict[str, Any] = await response.json()
-            if result:
-                activity: Dict[str, Any] = result.get("attributes", "")
-                if activity:
+            if result and (activity := result.get("attributes")):
                return activity
        return {}

@ -150,10 +142,7 @@ class HonAPI:
        async with self._hon.get(url, params=params) as response:
            result: Dict[str, Any] = await response.json()
            if result:
-                appliance_data: Dict[str, Any] = result.get("payload", {}).get(
-                    "applianceModel", {}
-                )
-                return appliance_data
+                return result.get("payload", {}).get("applianceModel", {})
        return {}

    async def load_attributes(self, appliance: HonAppliance) -> Dict[str, Any]:
@ -164,8 +153,7 @@ class HonAPI:
        }
        url: str = f"{const.API_URL}/commands/v1/context"
        async with self._hon.get(url, params=params) as response:
-            attributes: Dict[str, Any] = (await response.json()).get("payload", {})
-        return attributes
+            return (await response.json()).get("payload", {})

    async def load_statistics(self, appliance: HonAppliance) -> Dict[str, Any]:
        params: Dict[str, str] = {
@ -174,15 +162,13 @@ class HonAPI:
        }
        url: str = f"{const.API_URL}/commands/v1/statistics"
        async with self._hon.get(url, params=params) as response:
-            statistics: Dict[str, Any] = (await response.json()).get("payload", {})
-        return statistics
+            return (await response.json()).get("payload", {})

    async def load_maintenance(self, appliance: HonAppliance) -> Dict[str, Any]:
        url = f"{const.API_URL}/commands/v1/maintenance-cycle"
        params = {"macAddress": appliance.mac_address}
        async with self._hon.get(url, params=params) as response:
-            maintenance: Dict[str, Any] = (await response.json()).get("payload", {})
-        return maintenance
+            return (await response.json()).get("payload", {})

    async def send_command(
        self,
@ -190,7 +176,6 @@ class HonAPI:
        command: str,
        parameters: Dict[str, Any],
        ancillary_parameters: Dict[str, Any],
-        program_name: str = "",
    ) -> bool:
        now: str = datetime.utcnow().isoformat()
        data: Dict[str, Any] = {
@ -209,8 +194,6 @@ class HonAPI:
            "parameters": parameters,
            "applianceType": appliance.appliance_type,
        }
-        if command == "startProgram" and program_name:
-            data.update({"programName": program_name.upper()})
        url: str = f"{const.API_URL}/commands/v1/send"
        async with self._hon.post(url, json=data) as response:
            json_data: Dict[str, Any] = await response.json()
@ -224,8 +207,9 @@ class HonAPI:
        url: str = f"{const.API_URL}/config/v1/program-list-rules"
        async with self._hon_anonymous.get(url) as response:
            result: Dict[str, Any] = await response.json()
-        data: Dict[str, Any] = result.get("payload", {})
+            if result and (data := result.get("payload")):
                return data
+        return {}

    async def app_config(
        self, language: str = "en", beta: bool = True
@ -239,17 +223,17 @@ class HonAPI:
        }
        payload: str = json.dumps(payload_data, separators=(",", ":"))
        async with self._hon_anonymous.post(url, data=payload) as response:
-            result = await response.json()
-        data: Dict[str, Any] = result.get("payload", {})
+            if (result := await response.json()) and (data := result.get("payload")):
                return data
+        return {}

    async def translation_keys(self, language: str = "en") -> Dict[str, Any]:
        config = await self.app_config(language=language)
-        if not (url := config.get("language", {}).get("jsonPath")):
-            return {}
+        if url := config.get("language", {}).get("jsonPath"):
            async with self._hon_anonymous.get(url) as response:
-            result: Dict[str, Any] = await response.json()
+                if result := await response.json():
                    return result
+        return {}

    async def close(self) -> None:
        if self._hon_handler is not None:
@ -266,27 +250,17 @@ class TestAPI(HonAPI):

    def _load_json(self, appliance: HonAppliance, file: str) -> Dict[str, Any]:
        directory = f"{appliance.appliance_type}_{appliance.appliance_model_id}".lower()
-        if not (path := self._path / directory / f"{file}.json").exists():
-            _LOGGER.warning("Can't open %s", str(path))
-            return {}
+        path = f"{self._path}/{directory}/{file}.json"
        with open(path, "r", encoding="utf-8") as json_file:
-            text = json_file.read()
-        try:
-            data: Dict[str, Any] = json.loads(text)
-            return data
-        except json.decoder.JSONDecodeError as error:
-            _LOGGER.error("%s - %s", str(path), error)
-            return {}
+            return json.loads(json_file.read())

    async def load_appliances(self) -> List[Dict[str, Any]]:
        result = []
        for appliance in self._path.glob("*/"):
-            file = appliance / "appliance_data.json"
-            with open(file, "r", encoding="utf-8") as json_file:
-                try:
+            with open(
+                appliance / "appliance_data.json", "r", encoding="utf-8"
+            ) as json_file:
                result.append(json.loads(json_file.read()))
-                except json.decoder.JSONDecodeError as error:
-                    _LOGGER.error("%s - %s", str(file), error)
        return result

    async def load_commands(self, appliance: HonAppliance) -> Dict[str, Any]:
@ -322,12 +296,5 @@ class TestAPI(HonAPI):
        command: str,
        parameters: Dict[str, Any],
        ancillary_parameters: Dict[str, Any],
-        program_name: str = "",
    ) -> bool:
-        _LOGGER.info(
-            "%s - %s - %s",
-            str(parameters),
-            str(ancillary_parameters),
-            str(program_name),
-        )
        return True
--- a/pyhon/connection/auth.py
+++ b/pyhon/connection/auth.py
@ -6,7 +6,7 @@ import urllib
 from contextlib import suppress
 from dataclasses import dataclass
 from datetime import datetime, timedelta
-from typing import Dict, Optional, Any, List
+from typing import Dict, Optional, Any
 from urllib import parse
 from urllib.parse import quote

@ -30,14 +30,6 @@ class HonLoginData:
    loaded: Optional[Dict[str, Any]] = None


-@dataclass
-class HonAuthData:
-    access_token: str = ""
-    refresh_token: str = ""
-    cognito_token: str = ""
-    id_token: str = ""
-
-
 class HonAuth:
    _TOKEN_EXPIRES_AFTER_HOURS = 8
    _TOKEN_EXPIRE_WARNING_HOURS = 7
@ -54,25 +46,28 @@ class HonAuth:
        self._login_data = HonLoginData()
        self._login_data.email = email
        self._login_data.password = password
+        self._access_token = ""
+        self._refresh_token = ""
+        self._cognito_token = ""
+        self._id_token = ""
        self._device = device
        self._expires: datetime = datetime.utcnow()
-        self._auth = HonAuthData()

    @property
    def cognito_token(self) -> str:
-        return self._auth.cognito_token
+        return self._cognito_token

    @property
    def id_token(self) -> str:
-        return self._auth.id_token
+        return self._id_token

    @property
    def access_token(self) -> str:
-        return self._auth.access_token
+        return self._access_token

    @property
    def refresh_token(self) -> str:
-        return self._auth.refresh_token
+        return self._refresh_token

    def _check_token_expiration(self, hours: int) -> bool:
        return datetime.utcnow() >= self._expires + timedelta(hours=hours)
@ -120,8 +115,7 @@ class HonAuth:
        async with self._request.get(url) as response:
            text = await response.text()
            self._expires = datetime.utcnow()
-            login_url: List[str] = re.findall("url = '(.+?)'", text)
-            if not login_url:
+            if not (login_url := re.findall("url = '(.+?)'", text)):
                if "oauth/done#access_token=" in text:
                    self._parse_token_data(text)
                    raise exceptions.HonNoAuthenticationNeeded()
@ -190,19 +184,18 @@ class HonAuth:
            if response.status == 200:
                with suppress(json.JSONDecodeError, KeyError):
                    result = await response.json()
-                    url: str = result["events"][0]["attributes"]["values"]["url"]
-                    return url
+                    return result["events"][0]["attributes"]["values"]["url"]
            await self._error_logger(response)
            return ""

    def _parse_token_data(self, text: str) -> bool:
        if access_token := re.findall("access_token=(.*?)&", text):
-            self._auth.access_token = access_token[0]
+            self._access_token = access_token[0]
        if refresh_token := re.findall("refresh_token=(.*?)&", text):
-            self._auth.refresh_token = refresh_token[0]
+            self._refresh_token = refresh_token[0]
        if id_token := re.findall("id_token=(.*?)&", text):
-            self._auth.id_token = id_token[0]
-        return bool(access_token and refresh_token and id_token)
+            self._id_token = id_token[0]
+        return True if access_token and refresh_token and id_token else False

    async def _get_token(self, url: str) -> bool:
        async with self._request.get(url) as response:
@ -234,7 +227,7 @@ class HonAuth:
        return True

    async def _api_auth(self) -> bool:
-        post_headers = {"id-token": self._auth.id_token}
+        post_headers = {"id-token": self._id_token}
        data = self._device.get()
        async with self._request.post(
            f"{const.API_URL}/auth/v1/login", headers=post_headers, json=data
@ -244,8 +237,8 @@ class HonAuth:
            except json.JSONDecodeError:
                await self._error_logger(response)
                return False
-            self._auth.cognito_token = json_data.get("cognitoUser", {}).get("Token", "")
-            if not self._auth.cognito_token:
+            self._cognito_token = json_data.get("cognitoUser", {}).get("Token", "")
+            if not self._cognito_token:
                _LOGGER.error(json_data)
                raise exceptions.HonAuthenticationError()
        return True
@ -267,7 +260,7 @@ class HonAuth:
    async def refresh(self) -> bool:
        params = {
            "client_id": const.CLIENT_ID,
-            "refresh_token": self._auth.refresh_token,
+            "refresh_token": self._refresh_token,
            "grant_type": "refresh_token",
        }
        async with self._request.post(
@ -278,14 +271,14 @@ class HonAuth:
                return False
            data = await response.json()
        self._expires = datetime.utcnow()
-        self._auth.id_token = data["id_token"]
-        self._auth.access_token = data["access_token"]
+        self._id_token = data["id_token"]
+        self._access_token = data["access_token"]
        return await self._api_auth()

    def clear(self) -> None:
        self._session.cookie_jar.clear_domain(const.AUTH_API.split("/")[-2])
        self._request.called_urls = []
-        self._auth.cognito_token = ""
-        self._auth.id_token = ""
-        self._auth.access_token = ""
-        self._auth.refresh_token = ""
+        self._cognito_token = ""
+        self._id_token = ""
+        self._access_token = ""
+        self._refresh_token = ""
--- a/pyhon/connection/device.py
+++ b/pyhon/connection/device.py
@ -21,7 +21,7 @@ class HonDevice:
        return self._os_version

    @property
-    def os_type(self) -> str:
+    def os(self) -> str:
        return self._os

    @property
@ -36,7 +36,7 @@ class HonDevice:
        result: Dict[str, str | int] = {
            "appVersion": self.app_version,
            "mobileId": self.mobile_id,
-            "os": self.os_type,
+            "os": self.os,
            "osVersion": self.os_version,
            "deviceModel": self.device_model,
        }
--- a/pyhon/connection/handler/anonym.py
+++ b/pyhon/connection/handler/anonym.py
@ -4,7 +4,6 @@ from contextlib import asynccontextmanager
 from typing import Dict, Any

 import aiohttp
-from yarl import URL

 from pyhon import const
 from pyhon.connection.handler.base import ConnectionHandler
@ -18,10 +17,10 @@ class HonAnonymousConnectionHandler(ConnectionHandler):

    @asynccontextmanager
    async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, Any]
+        self, method: Callback, *args: Any, **kwargs: Any
    ) -> AsyncIterator[aiohttp.ClientResponse]:
        kwargs["headers"] = kwargs.pop("headers", {}) | self._HEADERS
-        async with method(url, *args, **kwargs) as response:
+        async with method(*args, **kwargs) as response:
            if response.status == 403:
                _LOGGER.error("Can't authenticate anymore")
            yield response
--- a/pyhon/connection/handler/auth.py
+++ b/pyhon/connection/handler/auth.py
@ -1,10 +1,9 @@
 import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
-from typing import Optional, List, Tuple, Any, Dict
+from typing import Optional, List, Tuple, Any

 import aiohttp
-from yarl import URL

 from pyhon import const
 from pyhon.connection.handler.base import ConnectionHandler
@ -30,9 +29,9 @@ class HonAuthConnectionHandler(ConnectionHandler):

    @asynccontextmanager
    async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, Any]
+        self, method: Callback, *args: Any, **kwargs: Any
    ) -> AsyncIterator[aiohttp.ClientResponse]:
        kwargs["headers"] = kwargs.pop("headers", {}) | self._HEADERS
-        async with method(url, *args, **kwargs) as response:
+        async with method(*args, **kwargs) as response:
            self._called_urls.append((response.status, str(response.request_info.url)))
            yield response
--- a/pyhon/connection/handler/base.py
+++ b/pyhon/connection/handler/base.py
@ -2,11 +2,10 @@ import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
 from types import TracebackType
-from typing import Optional, Dict, Type, Any
+from typing import Optional, Dict, Type, Any, Protocol

 import aiohttp
 from typing_extensions import Self
-from yarl import URL

 from pyhon import const, exceptions
 from pyhon.typedefs import Callback
@ -47,11 +46,10 @@ class ConnectionHandler:
        return self

    @asynccontextmanager
-    async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, Any]
+    def _intercept(
+        self, method: Callback, *args: Any, loop: int = 0, **kwargs: Any
    ) -> AsyncIterator[aiohttp.ClientResponse]:
-        async with method(url, *args, **kwargs) as response:
-            yield response
+        raise NotImplementedError

    @asynccontextmanager
    async def get(
@ -60,8 +58,7 @@ class ConnectionHandler:
        if self._session is None:
            raise exceptions.NoSessionException()
        response: aiohttp.ClientResponse
-        args = self._session.get, *args
-        async with self._intercept(*args, **kwargs) as response:
+        async with self._intercept(self._session.get, *args, **kwargs) as response:  # type: ignore[arg-type]
            yield response

    @asynccontextmanager
@ -71,8 +68,7 @@ class ConnectionHandler:
        if self._session is None:
            raise exceptions.NoSessionException()
        response: aiohttp.ClientResponse
-        args = self._session.post, *args
-        async with self._intercept(*args, **kwargs) as response:
+        async with self._intercept(self._session.post, *args, **kwargs) as response:  # type: ignore[arg-type]
            yield response

    async def close(self) -> None:
--- a/pyhon/connection/handler/hon.py
+++ b/pyhon/connection/handler/hon.py
@ -6,7 +6,6 @@ from typing import Optional, Dict, Any

 import aiohttp
 from typing_extensions import Self
-from yarl import URL

 from pyhon.connection.auth import HonAuth
 from pyhon.connection.device import HonDevice
@ -55,19 +54,16 @@ class HonConnectionHandler(ConnectionHandler):

    @asynccontextmanager
    async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Any
+        self, method: Callback, *args: Any, loop: int = 0, **kwargs: Dict[str, str]
    ) -> AsyncIterator[aiohttp.ClientResponse]:
-        loop: int = kwargs.pop("loop", 0)
        kwargs["headers"] = await self._check_headers(kwargs.get("headers", {}))
-        async with method(url, *args, **kwargs) as response:
+        async with method(args[0], *args[1:], **kwargs) as response:
            if (
                self.auth.token_expires_soon or response.status in [401, 403]
            ) and loop == 0:
                _LOGGER.info("Try refreshing token...")
                await self.auth.refresh()
-                async with self._intercept(
-                    method, url, *args, loop=loop + 1, **kwargs
-                ) as result:
+                async with self._intercept(method, loop=loop + 1, **kwargs) as result:
                    yield result
            elif (
                self.auth.token_is_expired or response.status in [401, 403]
@ -79,9 +75,7 @@ class HonConnectionHandler(ConnectionHandler):
                    await response.text(),
                )
                await self.create()
-                async with self._intercept(
-                    method, url, *args, loop=loop + 1, **kwargs
-                ) as result:
+                async with self._intercept(method, loop=loop + 1, **kwargs) as result:
                    yield result
            elif loop >= 2:
                _LOGGER.error(
@ -95,11 +89,11 @@ class HonConnectionHandler(ConnectionHandler):
                try:
                    await response.json()
                    yield response
-                except json.JSONDecodeError as exc:
+                except json.JSONDecodeError:
                    _LOGGER.warning(
                        "%s - JsonDecodeError %s - %s",
                        response.request_info.url,
                        response.status,
                        await response.text(),
                    )
-                    raise HonAuthenticationError("Decode Error") from exc
+                    raise HonAuthenticationError("Decode Error")
--- a/pyhon/const.py
+++ b/pyhon/const.py
@ -2,11 +2,9 @@ AUTH_API = "https://account2.hon-smarthome.com"
 API_URL = "https://api-iot.he.services"
 API_KEY = "GRCqFhC6Gk@ikWXm1RmnSmX1cm,MxY-configuration"
 APP = "hon"
-CLIENT_ID = (
-    "3MVG9QDx8IX8nP5T2Ha8ofvlmjLZl5L_gvfbT9."
-    "HJvpHGKoAS_dcMN8LYpTSYeVFCraUnV.2Ag1Ki7m4znVO6"
-)
-APP_VERSION = "2.4.7"
+# All seen id's (different accounts, different devices) are the same, so I guess this hash is static
+CLIENT_ID = "3MVG9QDx8IX8nP5T2Ha8ofvlmjLZl5L_gvfbT9.HJvpHGKoAS_dcMN8LYpTSYeVFCraUnV.2Ag1Ki7m4znVO6"
+APP_VERSION = "2.0.10"
 OS_VERSION = 31
 OS = "android"
 DEVICE_MODEL = "exynos9820"
--- a/pyhon/diagnose.py
+++ b/pyhon/diagnose.py
@ -70,7 +70,7 @@ async def zip_archive(
 ) -> str:
    data = await appliance_data(appliance, path, anonymous)
    archive = data[0].parent
-    shutil.make_archive(str(archive), "zip", archive)
+    shutil.make_archive(str(archive.parent), "zip", archive)
    shutil.rmtree(archive)
    return f"{archive.stem}.zip"

@ -89,11 +89,12 @@ def yaml_export(appliance: "HonAppliance", anonymous: bool = False) -> str:
    if anonymous:
        for sensible in ["serialNumber", "coords"]:
            data.get("appliance", {}).pop(sensible, None)
-    result = printer.pretty_print({"data": data})
-    if commands := printer.create_commands(appliance.commands):
-        result += printer.pretty_print({"commands": commands})
-    if rules := printer.create_rules(appliance.commands):
-        result += printer.pretty_print({"rules": rules})
+    data = {
+        "data": data,
+        "commands": printer.create_command(appliance.commands),
+        "rules": printer.create_rules(appliance.commands),
+    }
+    result = printer.pretty_print(data)
    if anonymous:
        result = anonymize_data(result)
    return result
--- a/pyhon/hon.py
+++ b/pyhon/hon.py
@ -7,10 +7,9 @@ from typing import List, Optional, Dict, Any, Type
 from aiohttp import ClientSession
 from typing_extensions import Self

+from pyhon import HonAPI, exceptions
 from pyhon.appliance import HonAppliance
-from pyhon.connection.api import HonAPI
 from pyhon.connection.api import TestAPI
-from pyhon.exceptions import NoAuthenticationException

 _LOGGER = logging.getLogger(__name__)

@ -44,7 +43,7 @@ class Hon:
    @property
    def api(self) -> HonAPI:
        if self._api is None:
-            raise NoAuthenticationException
+            raise exceptions.NoAuthenticationException
        return self._api

    @property
--- a/pyhon/parameter/base.py
+++ b/pyhon/parameter/base.py
@ -7,21 +7,14 @@ if TYPE_CHECKING:
 class HonParameter:
    def __init__(self, key: str, attributes: Dict[str, Any], group: str) -> None:
        self._key = key
-        self._attributes = attributes
-        self._category: str = ""
-        self._typology: str = ""
-        self._mandatory: int = 0
+        self._category: str = attributes.get("category", "")
+        self._typology: str = attributes.get("typology", "")
+        self._mandatory: int = attributes.get("mandatory", 0)
        self._value: str | float = ""
        self._group: str = group
        self._triggers: Dict[
            str, List[Tuple[Callable[["HonRule"], None], "HonRule"]]
        ] = {}
-        self._set_attributes()
-
-    def _set_attributes(self) -> None:
-        self._category = self._attributes.get("category", "")
-        self._typology = self._attributes.get("typology", "")
-        self._mandatory = self._attributes.get("mandatory", 0)

    @property
    def key(self) -> str:
@ -68,9 +61,8 @@ class HonParameter:
        self._triggers.setdefault(value, []).append((func, data))

    def check_trigger(self, value: str | float) -> None:
-        triggers = {str(k).lower(): v for k, v in self._triggers.items()}
-        if str(value).lower() in triggers:
-            for trigger in triggers[str(value)]:
+        if str(value) in self._triggers:
+            for trigger in self._triggers[str(value)]:
                func, args = trigger
                func(args)

@ -93,6 +85,3 @@ class HonParameter:
                    param[rule.param_key] = rule.param_data.get("defaultValue", "")

        return result
-
-    def reset(self) -> None:
-        self._set_attributes()
--- a/pyhon/parameter/enum.py
+++ b/pyhon/parameter/enum.py
@ -10,19 +10,12 @@ def clean_value(value: str | float) -> str:
 class HonParameterEnum(HonParameter):
    def __init__(self, key: str, attributes: Dict[str, Any], group: str) -> None:
        super().__init__(key, attributes, group)
-        self._default: str | float = ""
-        self._value: str | float = ""
-        self._values: List[str] = []
-        self._set_attributes()
+        self._default = attributes.get("defaultValue")
+        self._value = self._default or "0"
+        self._values: List[str] = attributes.get("enumValues", [])
        if self._default and clean_value(self._default.strip("[]")) not in self.values:
            self._values.append(self._default)

-    def _set_attributes(self) -> None:
-        super()._set_attributes()
-        self._default = self._attributes.get("defaultValue", "")
-        self._value = self._default or "0"
-        self._values = self._attributes.get("enumValues", [])
-
    def __repr__(self) -> str:
        return f"{self.__class__} (<{self.key}> {self.values})"

@ -48,4 +41,4 @@ class HonParameterEnum(HonParameter):
            self._value = value
            self.check_trigger(value)
        else:
-            raise ValueError(f"Allowed values: {self._values} But was: {value}")
+            raise ValueError(f"Allowed values {self._values}")
--- a/pyhon/parameter/fixed.py
+++ b/pyhon/parameter/fixed.py
@ -6,19 +6,14 @@ from pyhon.parameter.base import HonParameter
 class HonParameterFixed(HonParameter):
    def __init__(self, key: str, attributes: Dict[str, Any], group: str) -> None:
        super().__init__(key, attributes, group)
-        self._value: str | float = ""
-        self._set_attributes()
-
-    def _set_attributes(self) -> None:
-        super()._set_attributes()
-        self._value = self._attributes.get("fixedValue", "")
+        self._value = attributes.get("fixedValue", None)

    def __repr__(self) -> str:
        return f"{self.__class__} (<{self.key}> fixed)"

    @property
    def value(self) -> str | float:
-        return self._value if self._value != "" else "0"
+        return self._value if self._value is not None else "0"

    @value.setter
    def value(self, value: str | float) -> None:
--- a/pyhon/parameter/program.py
+++ b/pyhon/parameter/program.py
@ -28,7 +28,7 @@ class HonParameterProgram(HonParameterEnum):
        if value in self.values:
            self._command.category = value
        else:
-            raise ValueError(f"Allowed values: {self.values} But was: {value}")
+            raise ValueError(f"Allowed values {self.values}")

    @property
    def values(self) -> List[str]:
@ -37,19 +37,17 @@ class HonParameterProgram(HonParameterEnum):

    @values.setter
    def values(self, values: List[str]) -> None:
-        raise ValueError("Cant set values {values}")
+        return

    @property
    def ids(self) -> Dict[int, str]:
-        values: Dict[int, str] = {}
-        for name, parameter in self._programs.items():
-            if "iot_" in name:
-                continue
-            if parameter.parameters.get("prCode"):
-                continue
-            if (fav := parameter.parameters.get("favourite")) and fav.value == "1":
-                continue
-            values[int(parameter.parameters["prCode"].value)] = name
+        values = {
+            int(p.parameters["prCode"].value): n
+            for i, (n, p) in enumerate(self._programs.items())
+            if "iot_" not in n
+            and p.parameters.get("prCode")
+            and not ((fav := p.parameters.get("favourite")) and fav.value == "1")
+        }
        return dict(sorted(values.items()))

    def set_value(self, value: str) -> None:
--- a/pyhon/parameter/range.py
+++ b/pyhon/parameter/range.py
@ -7,20 +7,11 @@ from pyhon.parameter.base import HonParameter
 class HonParameterRange(HonParameter):
    def __init__(self, key: str, attributes: Dict[str, Any], group: str) -> None:
        super().__init__(key, attributes, group)
-        self._min: float = 0
-        self._max: float = 0
-        self._step: float = 0
-        self._default: float = 0
-        self._value: float = 0
-        self._set_attributes()
-
-    def _set_attributes(self) -> None:
-        super()._set_attributes()
-        self._min = str_to_float(self._attributes.get("minimumValue", 0))
-        self._max = str_to_float(self._attributes.get("maximumValue", 0))
-        self._step = str_to_float(self._attributes.get("incrementValue", 0))
-        self._default = str_to_float(self._attributes.get("defaultValue", self.min))
-        self._value = self._default
+        self._min: float = str_to_float(attributes["minimumValue"])
+        self._max: float = str_to_float(attributes["maximumValue"])
+        self._step: float = str_to_float(attributes["incrementValue"])
+        self._default: float = str_to_float(attributes.get("defaultValue", self.min))
+        self._value: float = self._default

    def __repr__(self) -> str:
        return f"{self.__class__} (<{self.key}> [{self.min} - {self.max}])"
@ -58,14 +49,11 @@ class HonParameterRange(HonParameter):
    @value.setter
    def value(self, value: str | float) -> None:
        value = str_to_float(value)
-        if self.min <= value <= self.max and not ((value - self.min) * 100) % (
-            self.step * 100
-        ):
+        if self.min <= value <= self.max and not (value - self.min) % self.step:
            self._value = value
            self.check_trigger(value)
        else:
-            allowed = f"min {self.min} max {self.max} step {self.step}"
-            raise ValueError(f"Allowed: {allowed} But was: {value}")
+            raise ValueError(f"Allowed: min {self.min} max {self.max} step {self.step}")

    @property
    def values(self) -> List[str]:
--- a/pyhon/printer.py
+++ b/pyhon/printer.py
@ -29,30 +29,37 @@ def pretty_print(
    whitespace: str = "  ",
 ) -> str:
    result = ""
-    space = whitespace * intend
-    if isinstance(data, (dict, list)) and key:
-        result += f"{space}{'- ' if is_list else ''}{key}:\n"
-        intend += 1
    if isinstance(data, list):
+        if key:
+            result += f"{whitespace * intend}{'- ' if is_list else ''}{key}:\n"
+            intend += 1
        for i, value in enumerate(data):
            result += pretty_print(
                value, intend=intend, is_list=True, whitespace=whitespace
            )
    elif isinstance(data, dict):
-        for i, (list_key, value) in enumerate(sorted(data.items())):
+        if key:
+            result += f"{whitespace * intend}{'- ' if is_list else ''}{key}:\n"
+            intend += 1
+        for i, (key, value) in enumerate(sorted(data.items())):
+            if is_list and not i:
                result += pretty_print(
-                value,
-                key=list_key,
-                intend=intend + (is_list if i else 0),
-                is_list=is_list and not i,
-                whitespace=whitespace,
+                    value, key=key, intend=intend, is_list=True, whitespace=whitespace
+                )
+            elif is_list:
+                result += pretty_print(
+                    value, key=key, intend=intend + 1, whitespace=whitespace
                )
            else:
-        result += f"{space}{'- ' if is_list else ''}{key}{': ' if key else ''}{data}\n"
+                result += pretty_print(
+                    value, key=key, intend=intend, whitespace=whitespace
+                )
+    else:
+        result += f"{whitespace * intend}{'- ' if is_list else ''}{key}{': ' if key else ''}{data}\n"
    return result


-def create_commands(
+def create_command(
    commands: Dict[str, "HonCommand"], concat: bool = False
 ) -> Dict[str, Any]:
    result: Dict[str, Any] = {}
--- a/pyhon/py.typed
+++ b/pyhon/py.typed
--- a/pyhon/rules.py
+++ b/pyhon/rules.py
@ -3,7 +3,6 @@ from typing import List, Dict, TYPE_CHECKING, Any, Optional

 from pyhon.parameter.enum import HonParameterEnum
 from pyhon.parameter.range import HonParameterRange
-from pyhon.typedefs import Parameter

 if TYPE_CHECKING:
    from pyhon.commands import HonCommand
@ -25,10 +24,6 @@ class HonRuleSet:
        self._rules: Dict[str, List[HonRule]] = {}
        self._parse_rule(rule)

-    @property
-    def rules(self) -> Dict[str, List[HonRule]]:
-        return self._rules
-
    def _parse_rule(self, rule: Dict[str, Any]) -> None:
        for param_key, params in rule.items():
            param_key = self._command.appliance.options.get(param_key, param_key)
@ -56,11 +51,6 @@ class HonRuleSet:
                    extra[trigger_key] = trigger_value
                    for extra_key, extra_data in param_data.items():
                        self._parse_conditions(param_key, extra_key, extra_data, extra)
-                else:
-                    param_data = {"typology": "fixed", "fixedValue": param_data}
-                    self._create_rule(
-                        param_key, trigger_key, trigger_value, param_data, extra
-                    )

    def _create_rule(
        self,
@ -93,47 +83,29 @@ class HonRuleSet:
            for rule in rules:
                self._rules.setdefault(key, []).append(rule)

-    def _extra_rules_matches(self, rule: HonRule) -> bool:
-        if rule.extras:
+    def _add_trigger(self, parameter: "HonParameter", data: HonRule) -> None:
+        def apply(rule: HonRule) -> None:
+            if rule.extras is not None:
                for key, value in rule.extras.items():
-                if not self._command.parameters.get(key):
-                    return False
                    if str(self._command.parameters.get(key)) != str(value):
-                    return False
-        return True
-
-    def _apply_fixed(self, param: Parameter, value: str | float) -> None:
-        if isinstance(param, HonParameterEnum) and set(param.values) != {str(value)}:
+                        return
+            if param := self._command.parameters.get(rule.param_key):
+                if value := rule.param_data.get("fixedValue", ""):
+                    if isinstance(param, HonParameterEnum) and set(param.values) != {
+                        str(value)
+                    }:
                        param.values = [str(value)]
-            param.value = str(value)
                    elif isinstance(param, HonParameterRange):
-            if float(value) < param.min:
-                param.min = float(value)
-            elif float(value) > param.max:
-                param.max = float(value)
                        param.value = float(value)
                        return
                    param.value = str(value)
-
-    def _apply_enum(self, param: Parameter, rule: HonRule) -> None:
-        if not isinstance(param, HonParameterEnum):
-            return
+                elif rule.param_data.get("typology") == "enum":
+                    if isinstance(param, HonParameterEnum):
                        if enum_values := rule.param_data.get("enumValues"):
                            param.values = enum_values.split("|")
                        if default_value := rule.param_data.get("defaultValue"):
                            param.value = default_value

-    def _add_trigger(self, parameter: "HonParameter", data: HonRule) -> None:
-        def apply(rule: HonRule) -> None:
-            if not self._extra_rules_matches(rule):
-                return
-            if not (param := self._command.parameters.get(rule.param_key)):
-                return
-            if fixed_value := rule.param_data.get("fixedValue", ""):
-                self._apply_fixed(param, fixed_value)
-            elif rule.param_data.get("typology") == "enum":
-                self._apply_enum(param, rule)
-
        parameter.add_trigger(data.trigger_value, apply, data)

    def patch(self) -> None:
--- a/pyhon/typedefs.py
+++ b/pyhon/typedefs.py
@ -11,7 +11,7 @@ if TYPE_CHECKING:
    from pyhon.parameter.range import HonParameterRange


-class Callback(Protocol):  # pylint: disable=too-few-public-methods
+class Callback(Protocol):
    def __call__(
        self, url: str | URL, *args: Any, **kwargs: Any
    ) -> aiohttp.client._RequestContextManager:
--- a/requirements.txt
+++ b/requirements.txt
@ -1,3 +1,2 @@
-aiohttp>=3.8
-yarl>=1.8
-typing-extensions>=4.8
+aiohttp==3.8.4
+yarl==1.8.2
--- a/requirements_dev.txt
+++ b/requirements_dev.txt
@ -1,5 +1,4 @@
-black>=22.12
-flake8>=6.0
-mypy>=0.991
-pylint>=2.15
-setuptools>=62.3
+black==23.3.0
+flake8==6.0.0
+mypy==1.2.0
+pylint==2.17.2
--- a/setup.py
+++ b/setup.py
@ -2,12 +2,12 @@

 from setuptools import setup, find_packages

-with open("README.md", "r", encoding="utf-8") as f:
+with open("README.md", "r") as f:
    long_description = f.read()

 setup(
    name="pyhOn",
-    version="0.15.15",
+    version="0.14.3",
    author="Andre Basche",
    description="Control hOn devices with python",
    long_description=long_description,
@ -21,7 +21,7 @@ setup(
    packages=find_packages(),
    include_package_data=True,
    python_requires=">=3.10",
-    install_requires=["aiohttp>=3.8", "typing-extensions>=4.8", "yarl>=1.8"],
+    install_requires=["aiohttp"],
    classifiers=[
        "Development Status :: 4 - Beta",
        "Environment :: Console",
@ -30,7 +30,6 @@ setup(
        "Operating System :: OS Independent",
        "Programming Language :: Python :: 3.10",
        "Programming Language :: Python :: 3.11",
-        "Programming Language :: Python :: 3.12",
        "Topic :: Software Development :: Libraries :: Python Modules",
    ],
    entry_points={