Bump version

.flake8

@@ -1,3 +0,0 @@
-[flake8]
-max-line-length = 88
-max-complexity = 7

.github/workflows/python-check.yml

@@ -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
+    # - name: Analysing the code with pylint
-      run: |
+    #   run: |
-        pylint $(git ls-files '*.py')
+    #     pylint --max-line-length 88 $(git ls-files '*.py')
     - name: Check black style
       run: |
         black . --check

.pylintrc

@@ -1,9 +0,0 @@
-[MESSAGES CONTROL]
-
-disable=missing-docstring
-
-[FORMAT]
-
-max-args=6
-max-attributes=8
-max-line-length=88

MANIFEST.in

@@ -1 +0,0 @@
-include pyhon/py.typed

README.md

@@ -107,6 +107,3 @@ This library is used for the custom [HomeAssistant Integration "Haier hOn"](http
 ## Contribution
 Any kind of contribution is welcome!
 
-| Please add your appliances data to our [hon-test-data collection](https://github.com/Andre0512/hon-test-data). <br/>This helps us to develop new features and not to break compatibility in newer versions. |
-|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
-

mypy.ini

@@ -1,24 +0,0 @@
-[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

pyhon/__main__.py

@@ -6,18 +6,16 @@ import logging
 import sys
 from getpass import getpass
 from pathlib import Path
-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, helper
-from pyhon import Hon, HonAPI, diagnose, printer
 
 _LOGGER = logging.getLogger(__name__)
 
 
-def get_arguments() -> Dict[str, Any]:
+def get_arguments():
     """Get parsed arguments."""
     parser = argparse.ArgumentParser(description="pyhOn: Command Line Utility")
     parser.add_argument("-u", "--user", help="user for haier hOn account")
@@ -26,25 +24,17 @@ def get_arguments() -> Dict[str, Any]:
     keys = subparser.add_parser("keys", help="print as key format")
     keys.add_argument("keys", help="print as key format", action="store_true")
     keys.add_argument("--all", help="print also full keys", action="store_true")
-    export = subparser.add_parser("export")
+    translate = subparser.add_parser(
-    export.add_argument("export", help="export pyhon data", action="store_true")
-    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", 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()
-    )
     return vars(parser.parse_args())
 
 
-async def translate(language: str, json_output: bool = False) -> None:
+async def translate(language, json_output=False):
     async with HonAPI(anonymous=True) as hon:
         keys = await hon.translation_keys(language)
     if json_output:
@@ -57,55 +47,41 @@ async def translate(language: str, json_output: bool = False) -> None:
             .replace("\\r", "")
         )
         keys = json.loads(clean_keys)
-        print(printer.pretty_print(keys))
+        print(helper.pretty_print(keys))
 
 
-def get_login_data(args: Dict[str, str]) -> Tuple[str, str]:
+async def main():
+    args = get_arguments()
+    if language := args.get("translate"):
+        await translate(language, json_output=args.get("json"))
+        return
     if not (user := args["user"]):
         user = input("User for hOn account: ")
     if not (password := args["password"]):
         password = getpass("Password for hOn account: ")
-    return user, password
+    async with Hon(user, password) as hon:
-
-
-async def main() -> None:
-    args = get_arguments()
-    if language := args.get("translate"):
-        await translate(language, json_output=args.get("json", ""))
-        return
-    async with Hon(
-        *get_login_data(args), test_data_path=Path(args.get("import", ""))
-    ) as hon:
         for device in hon.appliances:
-            if args.get("export"):
-                anonymous = args.get("anonymous", False)
-                path = Path(args.get("directory", "."))
-                if not args.get("zip"):
-                    for file in await diagnose.appliance_data(device, path, anonymous):
-                        print(f"Created {file}")
-                else:
-                    archive = await diagnose.zip_archive(device, path, anonymous)
-                    print(f"Created {archive}")
-                continue
             print("=" * 10, device.appliance_type, "-", device.nick_name, "=" * 10)
             if args.get("keys"):
                 data = device.data.copy()
                 attr = "get" if args.get("all") else "pop"
                 print(
-                    printer.key_print(getattr(data["attributes"], attr)("parameters"))
+                    helper.key_print(
+                        data["attributes"].__getattribute__(attr)("parameters")
                     )
-                print(printer.key_print(getattr(data, attr)("appliance")))
+                )
-                print(printer.key_print(data))
+                print(helper.key_print(data.__getattribute__(attr)("appliance")))
+                print(helper.key_print(data))
                 print(
-                    printer.pretty_print(
+                    helper.pretty_print(
-                        printer.create_commands(device.commands, concat=True)
+                        helper.create_command(device.commands, concat=True)
                     )
                 )
             else:
-                print(diagnose.yaml_export(device))
+                print(device.diagnose("  "))
 
 
-def start() -> None:
+def start():
     try:
         asyncio.run(main())
     except KeyboardInterrupt:

pyhon/appliance.py

@@ -1,29 +1,25 @@
 import importlib
+import json
 import logging
-import re
+from contextlib import suppress
+from copy import copy
 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
+from typing import TYPE_CHECKING
 
-from pyhon import diagnose, exceptions
+from pyhon import helper
-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.fixed import HonParameterFixed
 from pyhon.parameter.range import HonParameterRange
-from pyhon.typedefs import Parameter
 
 if TYPE_CHECKING:
     from pyhon import HonAPI
 
 _LOGGER = logging.getLogger(__name__)
 
-T = TypeVar("T")
 
-
-# pylint: disable=too-many-public-methods,too-many-instance-attributes
 class HonAppliance:
     _MINIMAL_UPDATE_INTERVAL = 5  # seconds
 
@@ -32,64 +28,52 @@ class HonAppliance:
     ) -> None:
         if attributes := info.get("attributes"):
             info["attributes"] = {v["parName"]: v["parValue"] for v in attributes}
-        self._info: Dict[str, Any] = info
+        self._info: Dict = info
         self._api: Optional[HonAPI] = api
-        self._appliance_model: Dict[str, Any] = {}
+        self._appliance_model: Dict = {}
 
-        self._commands: Dict[str, HonCommand] = {}
+        self._commands: Dict = {}
-        self._statistics: Dict[str, Any] = {}
+        self._statistics: Dict = {}
-        self._attributes: Dict[str, Any] = {}
+        self._attributes: Dict = {}
         self._zone: int = zone
         self._additional_data: Dict[str, Any] = {}
-        self._last_update: Optional[datetime] = None
+        self._last_update = None
         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:
+    def __getitem__(self, item):
-        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.attributes["parameters"].get(item)
         return self.info[item]
 
-    @overload
+    def get(self, item, default=None):
-    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:
         try:
             return self[item]
         except (KeyError, IndexError):
             return default
 
     def _check_name_zone(self, name: str, frontend: bool = True) -> str:
-        zone = " Z" if frontend else "_z"
+        middle = " Z" if frontend else "_z"
-        attribute: str = self._info.get(name, "")
+        if (attribute := self._info.get(name, "")) and self._zone:
-        if attribute and self._zone:
+            return f"{attribute}{middle}{self._zone}"
-            return f"{attribute}{zone}{self._zone}"
         return attribute
 
     @property
@@ -106,62 +90,45 @@ class HonAppliance:
 
     @property
     def unique_id(self) -> str:
-        default_mac = "xx-xx-xx-xx-xx-xx"
+        return self._check_name_zone("macAddress", frontend=False)
-        import_name = f"{self.appliance_type.lower()}_{self.appliance_model_id}"
-        result = self._check_name_zone("macAddress", frontend=False)
-        result = result.replace(default_mac, import_name)
-        return result
 
     @property
     def model_name(self) -> str:
         return self._check_name_zone("modelName")
 
-    @property
-    def brand(self) -> str:
-        brand = self._check_name_zone("brand")
-        return brand[0].upper() + brand[1:]
-
     @property
     def nick_name(self) -> str:
-        result = self._check_name_zone("nickName")
+        return self._check_name_zone("nickName")
-        if not result or re.findall("^[xX1\\s-]+$", result):
-            return self.model_name
-        return result
 
     @property
     def code(self) -> str:
-        code: str = self.info.get("code", "")
+        if code := self.info.get("code"):
-        if 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:
+    def options(self):
-        return self._info.get("applianceModelId", 0)
-
-    @property
-    def options(self) -> Dict[str, Any]:
         return self._appliance_model.get("options", {})
 
     @property
-    def commands(self) -> Dict[str, HonCommand]:
+    def commands(self):
         return self._commands
 
     @property
-    def attributes(self) -> Dict[str, Any]:
+    def attributes(self):
         return self._attributes
 
     @property
-    def statistics(self) -> Dict[str, Any]:
+    def statistics(self):
         return self._statistics
 
     @property
-    def info(self) -> Dict[str, Any]:
+    def info(self):
         return self._info
 
     @property
-    def additional_data(self) -> Dict[str, Any]:
+    def additional_data(self):
         return self._additional_data
 
     @property
@@ -169,52 +136,137 @@ class HonAppliance:
         return self._zone
 
     @property
-    def api(self) -> "HonAPI":
+    def api(self) -> Optional["HonAPI"]:
-        """api connection object"""
-        if self._api is None:
-            raise exceptions.NoAuthenticationException("Missing hOn login")
         return self._api
 
-    async def load_commands(self) -> None:
+    async def _recover_last_command_states(self):
-        command_loader = HonCommandLoader(self.api, self)
+        command_history = await self.api.command_history(self)
-        await command_loader.load_commands()
+        for name, command in self._commands.items():
-        self._commands = command_loader.commands
+            last = next(
-        self._additional_data = command_loader.additional_data
+                (
-        self._appliance_model = command_loader.appliance_data
+                    index
-        self.sync_params_to_command("settings")
+                    for (index, d) in enumerate(command_history)
-
+                    if d.get("command", {}).get("commandName") == name
-    async def load_attributes(self) -> None:
+                ),
-        attributes = await self.api.load_attributes(self)
+                None,
-        for name, values in 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 last is None:
+                continue
+            parameters = command_history[last].get("command", {}).get("parameters", {})
+            if command.categories and (
+                parameters.get("program") or parameters.get("category")
+            ):
+                if parameters.get("program"):
+                    command.category = parameters.pop("program").split(".")[-1].lower()
+                else:
+                    command.category = parameters.pop("category")
+                command = self.commands[name]
+            for key, data in command.settings.items():
+                if (
+                    not isinstance(data, HonParameterFixed)
+                    and parameters.get(key) is not None
+                ):
+                    with suppress(ValueError):
+                        data.value = parameters.get(key)
+
+    def _get_categories(self, command, data):
+        categories = {}
+        for category, value in data.items():
+            result = self._get_command(value, command, category, categories)
+            if result:
+                if "PROGRAM" in category:
+                    category = category.split(".")[-1].lower()
+                categories[category] = result[0]
+        if categories:
+            if "setParameters" in categories:
+                return [categories["setParameters"]]
+            return [list(categories.values())[0]]
+        return []
+
+    def _get_commands(self, data):
+        commands = []
+        for command, value in data.items():
+            commands += self._get_command(value, command, "")
+        return {c.name: c for c in commands}
+
+    def _get_command(self, data, command="", category="", categories=None):
+        commands = []
+        if isinstance(data, dict):
+            if data.get("description") and data.get("protocolType", None):
+                commands += [
+                    HonCommand(
+                        command,
+                        data,
+                        self,
+                        category_name=category,
+                        categories=categories,
+                    )
+                ]
+            else:
+                commands += self._get_categories(command, data)
+        elif category:
+            self._additional_data.setdefault(command, {})[category] = data
+        else:
+            self._additional_data[command] = data
+        return commands
+
+    async def load_commands(self):
+        raw = await self.api.load_commands(self)
+        self._appliance_model = raw.pop("applianceModel")
+        raw.pop("dictionaryId", None)
+        self._commands = self._get_commands(raw)
+        await self._add_favourites()
+        await self._recover_last_command_states()
+
+    async def _add_favourites(self):
+        favourites = await self._api.command_favourites(self)
+        for favourite in favourites:
+            name = favourite.get("favouriteName")
+            command = favourite.get("command")
+            command_name = command.get("commandName")
+            program_name = command.get("programName", "").split(".")[-1].lower()
+            base = 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 parameter := base.parameters.get(key):
+                        with suppress(ValueError):
+                            parameter.value = value
+            extra_param = HonParameterFixed("favourite", {"fixedValue": "1"}, "custom")
+            base.parameters.update(favourite=extra_param)
+            base.parameters["program"].set_value(name)
+            self._commands[command_name].categories[name] = base
+
+    async def load_attributes(self):
+        self._attributes = await self.api.load_attributes(self)
+        for name, values in self._attributes.pop("shadow").get("parameters").items():
+            self._attributes.setdefault("parameters", {})[name] = values["parNewVal"]
         if self._extra:
             self._attributes = self._extra.attributes(self._attributes)
 
-    async def load_statistics(self) -> None:
+    async def load_statistics(self):
         self._statistics = await self.api.load_statistics(self)
         self._statistics |= await self.api.load_maintenance(self)
 
-    async def update(self, force: bool = False) -> None:
+    async def update(self, force=False):
         now = datetime.now()
-        min_age = now - timedelta(seconds=self._MINIMAL_UPDATE_INTERVAL)
+        if (
-        if force or not self._last_update or self._last_update < min_age:
+            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]]:
+    def command_parameters(self):
         return {n: c.parameter_value for n, c in self._commands.items()}
 
     @property
-    def settings(self) -> Dict[str, Parameter]:
+    def settings(self):
-        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)
@@ -224,7 +276,7 @@ class HonAppliance:
         return result
 
     @property
-    def available_settings(self) -> List[str]:
+    def available_settings(self):
         result = []
         for name, command in self._commands.items():
             for key in command.setting_keys:
@@ -232,7 +284,7 @@ class HonAppliance:
         return result
 
     @property
-    def data(self) -> Dict[str, Any]:
+    def data(self):
         result = {
             "attributes": self.attributes,
             "appliance": self.info,
@@ -243,64 +295,86 @@ class HonAppliance:
         }
         return result
 
-    @property
+    def diagnose(self, whitespace="  ", command_only=False):
-    def diagnose(self) -> str:
+        data = {
-        return diagnose.yaml_export(self, anonymous=True)
+            "attributes": self.attributes.copy(),
-
+            "appliance": self.info,
-    async def data_archive(self, path: Path) -> str:
+            "statistics": self.statistics,
-        return await diagnose.zip_archive(self, path, anonymous=True)
+            "additional_data": self._additional_data,
-
+        }
-    def sync_command_to_params(self, command_name: str) -> None:
+        if command_only:
-        if not (command := self.commands.get(command_name)):
+            data.pop("attributes")
-            return
+            data.pop("appliance")
-        for key in self.attributes.get("parameters", {}):
+            data.pop("statistics")
-            if new := command.parameters.get(key):
+        data |= {n: c.parameter_groups for n, c in self._commands.items()}
-                self.attributes["parameters"][key].update(
+        extra = {n: c.data for n, c in self._commands.items() if c.data}
-                    str(new.intern_value), shield=True
+        if extra:
+            data |= {"extra_command_data": extra}
+        for sensible in ["PK", "SK", "serialNumber", "coords", "device"]:
+            data.get("appliance", {}).pop(sensible, None)
+        result = helper.pretty_print({"data": data}, whitespace=whitespace)
+        result += helper.pretty_print(
+            {
+                "commands": helper.create_command(self.commands),
+                "rules": helper.create_rules(self.commands),
+            },
+            whitespace=whitespace,
         )
+        return result.replace(self.mac_address, "xx-xx-xx-xx-xx-xx")
 
-    def sync_params_to_command(self, command_name: str) -> None:
+    def sync_to_params(self, command_name):
-        if not (command := self.commands.get(command_name)):
+        command: HonCommand = self.commands.get(command_name)
-            return
+        for key, value in self.attributes.get("parameters", {}).items():
-        for key in command.setting_keys:
+            if isinstance(value, str) and (new := command.parameters.get(key)):
-            if (
+                self.attributes["parameters"][key] = str(new.intern_value)
-                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) -> None:
+    def sync_command(self, main, target=None) -> None:
-        base: Optional[HonCommand] = self.commands.get(main)
+        base: 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, parameter in data.parameters.items():
-            for name, target_param in data.parameters.items():
+                if base_value := base.parameters.get(name):
-                if not (base_param := base.parameters.get(name)):
+                    if isinstance(base_value, HonParameterRange) and isinstance(
-                    return
+                        parameter, HonParameterRange
-                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
+                        parameter.max = base_value.max
-            target.min = main.min
+                        parameter.min = base_value.min
-            target.step = main.step
+                        parameter.step = base_value.step
-        elif isinstance(target, HonParameterRange):
+                    elif isinstance(parameter, HonParameterRange):
-            target.max = int(main.value)
+                        parameter.max = int(base_value.value)
-            target.min = int(main.value)
+                        parameter.min = int(base_value.value)
-            target.step = 1
+                        parameter.step = 1
-        elif isinstance(target, HonParameterEnum):
+                    parameter.value = base_value.value
-            target.values = main.values
+
-        target.value = main.value
+
+class HonApplianceTest(HonAppliance):
+    def __init__(self, name):
+        super().__init__(None, {})
+        self._name = name
+        self.load_commands()
+        self.load_attributes()
+        self._info = self._appliance_model
+
+    def load_commands(self):
+        device = Path(__file__).parent / "test_data" / f"{self._name}.json"
+        with open(str(device)) as f:
+            raw = json.loads(f.read())
+        self._appliance_model = raw.pop("applianceModel")
+        raw.pop("dictionaryId", None)
+        self._commands = self._get_commands(raw)
+
+    async def update(self):
+        return
+
+    @property
+    def nick_name(self) -> str:
+        return self._name
+
+    @property
+    def unique_id(self) -> str:
+        return self._name
+
+    @property
+    def mac_address(self) -> str:
+        return "xx-xx-xx-xx-xx-xx"

pyhon/appliances/base.py

@@ -1,25 +1,15 @@
-from typing import Dict, Any, TYPE_CHECKING
-
-from pyhon.parameter.program import HonParameterProgram
-
-if TYPE_CHECKING:
-    from pyhon.appliance import HonAppliance
-
-
 class ApplianceBase:
-    def __init__(self, appliance: "HonAppliance"):
+    def __init__(self, appliance):
         self.parent = appliance
 
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         program_name = "No Program"
-        if program := int(str(data.get("parameters", {}).get("prCode", "0"))):
+        if program := int(data["parameters"].get("prCode", "0")):
             if start_cmd := self.parent.settings.get("startProgram.program"):
-                if isinstance(start_cmd, HonParameterProgram) and (
+                if ids := start_cmd.ids:
-                    ids := start_cmd.ids
-                ):
                     program_name = ids.get(program, program_name)
         data["programName"] = program_name
         return data
 
-    def settings(self, settings: Dict[str, Any]) -> Dict[str, Any]:
+    def settings(self, settings):
         return settings

pyhon/appliances/dw.py

@@ -1,13 +1,10 @@
-# pylint: disable=duplicate-code
-from typing import Any, Dict
-
 from pyhon.appliances.base import ApplianceBase
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
-        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
+        if data["lastConnEvent"]["category"] == "DISCONNECTED":
-            data["parameters"]["machMode"].value = "0"
+            data["parameters"]["machMode"] = "0"
         data["active"] = bool(data.get("activity"))
         return data

pyhon/appliances/ov.py

@@ -1,16 +1,19 @@
-from typing import Any, Dict
-
 from pyhon.appliances.base import ApplianceBase
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
-        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
+        if data["lastConnEvent"]["category"] == "DISCONNECTED":
-            data["parameters"]["temp"].value = "0"
+            data["parameters"]["temp"] = "0"
-            data["parameters"]["onOffStatus"].value = "0"
+            data["parameters"]["onOffStatus"] = "0"
-            data["parameters"]["remoteCtrValid"].value = "0"
+            data["parameters"]["remoteCtrValid"] = "0"
-            data["parameters"]["remainingTimeMM"].value = "0"
+            data["parameters"]["remainingTimeMM"] = "0"
 
         data["active"] = data["parameters"]["onOffStatus"] == "1"
+
+        if program := int(data["parameters"]["prCode"]):
+            ids = self.parent.settings["startProgram.program"].ids
+            data["programName"] = ids.get(program, "")
+
         return data

pyhon/appliances/ref.py

@@ -1,21 +1,19 @@
-from typing import Dict, Any
-
 from pyhon.appliances.base import ApplianceBase
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
         if data["parameters"]["holidayMode"] == "1":
             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"

pyhon/appliances/td.py

@@ -1,20 +1,17 @@
-# pylint: disable=duplicate-code
-from typing import Any, Dict
-
 from pyhon.appliances.base import ApplianceBase
 from pyhon.parameter.fixed import HonParameterFixed
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
-        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
+        if data["lastConnEvent"]["category"] == "DISCONNECTED":
-            data["parameters"]["machMode"].value = "0"
+            data["parameters"]["machMode"] = "0"
         data["active"] = bool(data.get("activity"))
         data["pause"] = data["parameters"]["machMode"] == "3"
         return data
 
-    def settings(self, settings: Dict[str, Any]) -> Dict[str, Any]:
+    def settings(self, settings):
         dry_level = settings.get("startProgram.dryLevel")
         if isinstance(dry_level, HonParameterFixed) and dry_level.value == "11":
             settings.pop("startProgram.dryLevel", None)

pyhon/appliances/wd.py

@@ -1,17 +1,14 @@
-# pylint: disable=duplicate-code
-from typing import Dict, Any
-
 from pyhon.appliances.base import ApplianceBase
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
-        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
+        if data["lastConnEvent"]["category"] == "DISCONNECTED":
-            data["parameters"]["machMode"].value = "0"
+            data["parameters"]["machMode"] = "0"
         data["active"] = bool(data.get("activity"))
         data["pause"] = data["parameters"]["machMode"] == "3"
         return data
 
-    def settings(self, settings: Dict[str, Any]) -> Dict[str, Any]:
+    def settings(self, settings):
         return settings

pyhon/appliances/wh.py

@@ -1,11 +0,0 @@
-from typing import Any, Dict
-
-from pyhon.appliances.base import ApplianceBase
-
-
-class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
-        data = super().attributes(data)
-        data["active"] = data["parameters"]["onOffStatus"] == "1"
-
-        return data

pyhon/appliances/wm.py

@@ -1,17 +1,14 @@
-# pylint: disable=duplicate-code
-from typing import Any, Dict
-
 from pyhon.appliances.base import ApplianceBase
 
 
 class Appliance(ApplianceBase):
-    def attributes(self, data: Dict[str, Any]) -> Dict[str, Any]:
+    def attributes(self, data):
         data = super().attributes(data)
-        if data.get("lastConnEvent", {}).get("category", "") == "DISCONNECTED":
+        if data["lastConnEvent"]["category"] == "DISCONNECTED":
-            data["parameters"]["machMode"].value = "0"
+            data["parameters"]["machMode"] = "0"
         data["active"] = bool(data.get("activity"))
         data["pause"] = data["parameters"]["machMode"] == "3"
         return data
 
-    def settings(self, settings: Dict[str, Any]) -> Dict[str, Any]:
+    def settings(self, settings):
         return settings

pyhon/attributes.py

@@ -1,58 +0,0 @@
-from datetime import datetime, timedelta
-from typing import Optional, Final, Dict
-
-from pyhon.helper import str_to_float
-
-
-class HonAttribute:
-    _LOCK_TIMEOUT: Final = 10
-
-    def __init__(self, data: Dict[str, str] | str):
-        self._value: str = ""
-        self._last_update: Optional[datetime] = None
-        self._lock_timestamp: Optional[datetime] = None
-        self.update(data)
-
-    @property
-    def value(self) -> float | str:
-        """Attribute value"""
-        try:
-            return str_to_float(self._value)
-        except ValueError:
-            return self._value
-
-    @value.setter
-    def value(self, value: str) -> None:
-        self._value = value
-
-    @property
-    def last_update(self) -> Optional[datetime]:
-        """Timestamp of last api update"""
-        return self._last_update
-
-    @property
-    def lock(self) -> bool:
-        """Shows if value changes are forbidden"""
-        if not self._lock_timestamp:
-            return False
-        lock_until = self._lock_timestamp + timedelta(seconds=self._LOCK_TIMEOUT)
-        return lock_until >= datetime.utcnow()
-
-    def update(self, data: Dict[str, str] | str, shield: bool = False) -> bool:
-        if self.lock and not shield:
-            return False
-        if shield:
-            self._lock_timestamp = datetime.utcnow()
-        if isinstance(data, str):
-            self.value = data
-            return True
-        self.value = data.get("parNewVal", "")
-        if last_update := data.get("lastUpdate"):
-            try:
-                self._last_update = datetime.fromisoformat(last_update)
-            except ValueError:
-                self._last_update = None
-        return True
-
-    def __str__(self) -> str:
-        return self._value

pyhon/command_loader.py

@@ -1,227 +0,0 @@
-import asyncio
-from contextlib import suppress
-from copy import copy
-from typing import Dict, Any, Optional, TYPE_CHECKING, List
-
-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.appliance import HonAppliance
-
-
-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._appliance_data: Dict[str, Any] = {}
-        self._additional_data: Dict[str, Any] = {}
-
-    @property
-    def api(self) -> "HonAPI":
-        """api connection object"""
-        if self._api is None:
-            raise NoAuthenticationException("Missing hOn login")
-        return self._api
-
-    @property
-    def appliance(self) -> "HonAppliance":
-        """appliance object"""
-        return self._appliance
-
-    @property
-    def commands(self) -> Dict[str, HonCommand]:
-        """Get list of hon commands"""
-        return self._commands
-
-    @property
-    def appliance_data(self) -> Dict[str, Any]:
-        """Get command appliance data"""
-        return self._appliance_data
-
-    @property
-    def additional_data(self) -> Dict[str, Any]:
-        """Get command additional data"""
-        return self._additional_data
-
-    async def load_commands(self) -> None:
-        """Trigger loading of command data"""
-        await self._load_data()
-        self._appliance_data = self._api_commands.pop("applianceModel", {})
-        self._get_commands()
-        self._add_favourites()
-        self._recover_last_command_states()
-
-    async def _load_commands(self) -> None:
-        self._api_commands = await self._api.load_commands(self._appliance)
-
-    async def _load_favourites(self) -> None:
-        self._favourites = await self._api.load_favourites(self._appliance)
-
-    async def _load_command_history(self) -> None:
-        self._command_history = await self._api.load_command_history(self._appliance)
-
-    async def _load_data(self) -> None:
-        """Callback parallel all relevant data"""
-        await asyncio.gather(
-            *[
-                self._load_commands(),
-                self._load_favourites(),
-                self._load_command_history(),
-            ]
-        )
-
-    @staticmethod
-    def _is_command(data: Dict[str, Any]) -> bool:
-        """Check if dict can be parsed as command"""
-        return (
-            data.get("description") is not None and data.get("protocolType") is not None
-        )
-
-    @staticmethod
-    def _clean_name(category: str) -> str:
-        """Clean up category name"""
-        if "PROGRAM" in category:
-            return category.split(".")[-1].lower()
-        return category
-
-    def _get_commands(self) -> None:
-        """Generates HonCommand dict from api data"""
-        commands = []
-        for name, data in self._api_commands.items():
-            if command := self._parse_command(data, name):
-                commands.append(command)
-        self._commands = {c.name: c for c in commands}
-
-    def _parse_command(
-        self,
-        data: Dict[str, Any] | str,
-        command_name: str,
-        categories: Optional[Dict[str, "HonCommand"]] = None,
-        category_name: str = "",
-    ) -> Optional[HonCommand]:
-        """Try to crate HonCommand object"""
-        if not isinstance(data, dict):
-            self._additional_data[command_name] = data
-            return None
-        if self._is_command(data):
-            return HonCommand(
-                command_name,
-                data,
-                self._appliance,
-                category_name=category_name,
-                categories=categories,
-            )
-        if category := self._parse_categories(data, command_name):
-            return category
-        return None
-
-    def _parse_categories(
-        self, data: Dict[str, Any], command_name: str
-    ) -> Optional[HonCommand]:
-        """Parse categories and create reference to other"""
-        categories: Dict[str, HonCommand] = {}
-        for category, value in data.items():
-            if command := self._parse_command(
-                value, command_name, category_name=category, categories=categories
-            ):
-                categories[self._clean_name(category)] = command
-        if categories:
-            # setParameters should be at first place
-            if "setParameters" in categories:
-                return categories["setParameters"]
-            return list(categories.values())[0]
-        return None
-
-    def _get_last_command_index(self, name: str) -> Optional[int]:
-        """Get index of last command execution"""
-        return next(
-            (
-                index
-                for (index, d) in enumerate(self._command_history)
-                if d.get("command", {}).get("commandName") == name
-            ),
-            None,
-        )
-
-    def _set_last_category(
-        self, command: HonCommand, name: str, parameters: Dict[str, Any]
-    ) -> HonCommand:
-        """Set category to last state"""
-        if command.categories:
-            if program := parameters.pop("program", None):
-                command.category = self._clean_name(program)
-            elif category := parameters.pop("category", None):
-                command.category = category
-            else:
-                return command
-            return self.commands[name]
-        return command
-
-    def _recover_last_command_states(self) -> None:
-        """Set commands to last state"""
-        for name, command in self.commands.items():
-            if (last_index := self._get_last_command_index(name)) is None:
-                continue
-            last_command = self._command_history[last_index]
-            parameters = last_command.get("command", {}).get("parameters", {})
-            command = self._set_last_category(command, name, parameters)
-            for key, data in command.settings.items():
-                if parameters.get(key) is None:
-                    continue
-                with suppress(ValueError):
-                    data.value = parameters.get(key)
-
-    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", {})
-        command = favourite.get("command", {})
-        command_name: str = 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:
-        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
-                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):
-            program.set_value(name)
-        self.commands[command_name].categories[name] = base_command

pyhon/commands.py

@@ -2,14 +2,13 @@ import logging
 from typing import Optional, Dict, Any, List, TYPE_CHECKING, Union
 
 from pyhon import exceptions
-from pyhon.exceptions import ApiError, NoAuthenticationException
+from pyhon.exceptions import ApiError
 from pyhon.parameter.base import HonParameter
 from pyhon.parameter.enum import HonParameterEnum
 from pyhon.parameter.fixed import HonParameterFixed
 from pyhon.parameter.program import HonParameterProgram
 from pyhon.parameter.range import HonParameterRange
 from pyhon.rules import HonRuleSet
-from pyhon.typedefs import Parameter
 
 if TYPE_CHECKING:
     from pyhon import HonAPI
@@ -27,29 +26,28 @@ class HonCommand:
         categories: Optional[Dict[str, "HonCommand"]] = None,
         category_name: str = "",
     ):
-        self._name: str = name
+        self._api: Optional[HonAPI] = appliance.api
-        self._api: Optional[HonAPI] = None
         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:
         return f"{self._name} command"
 
     @property
-    def name(self) -> str:
+    def name(self):
         return self._name
 
     @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
@@ -59,7 +57,7 @@ class HonCommand:
         return self._appliance
 
     @property
-    def data(self) -> Dict[str, Any]:
+    def data(self):
         return self._data
 
     @property
@@ -77,37 +75,25 @@ 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]]) -> None:
+    def _load_parameters(self, attributes):
         for key, items in attributes.items():
             for name, data in items.items():
                 self._create_parameters(data, name, key)
         for rule in self._rules:
             rule.patch()
 
-    def _create_parameters(
+    def _create_parameters(self, data: Dict, name: str, parameter: str) -> None:
-        self, data: Dict[str, Any], name: str, parameter: str
-    ) -> None:
         if name == "zoneMap" and self._appliance.zone:
             data["default"] = self._appliance.zone
         if data.get("category") == "rule":
-            if "fixedValue" in data:
+            if "fixedValue" not in data:
-                self._rules.append(HonRuleSet(self, data["fixedValue"]))
+                _LOGGER.error("Rule not supported: %s", data)
-            elif "enumValues" in data:
-                self._rules.append(HonRuleSet(self, data["enumValues"]))
             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)
@@ -122,34 +108,16 @@ 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:
+    async def send(self) -> bool:
-        grouped_params = (
+        params = self.parameter_groups.get("parameters", {})
-            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:
-                params[key] = parameter.value
-        return await self.send_parameters(params)
-
-    async def send_parameters(self, params: Dict[str, str | float]) -> bool:
         ancillary_params = self.parameter_groups.get("ancillaryParameters", {})
         ancillary_params.pop("programRules", None)
-        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
         )
         if not result:
-                _LOGGER.error(result)
             raise ApiError("Can't send command")
-        except NoAuthenticationException:
-            _LOGGER.error("No Authentication")
-            return False
         return result
 
     @property
@@ -174,7 +142,7 @@ class HonCommand:
         )
 
     @staticmethod
-    def _more_options(first: Parameter, second: Parameter) -> Parameter:
+    def _more_options(first: HonParameter, second: HonParameter):
         if isinstance(first, HonParameterFixed) and not isinstance(
             second, HonParameterFixed
         ):
@@ -184,8 +152,8 @@ class HonCommand:
         return first
 
     @property
-    def available_settings(self) -> Dict[str, Parameter]:
+    def available_settings(self) -> Dict[str, HonParameter]:
-        result: Dict[str, Parameter] = {}
+        result: Dict[str, HonParameter] = {}
         for command in self.categories.values():
             for name, parameter in command.parameters.items():
                 if name in result:
@@ -193,7 +161,3 @@ class HonCommand:
                 else:
                     result[name] = parameter
         return result
-
-    def reset(self) -> None:
-        for parameter in self._parameters.values():
-            parameter.reset()

pyhon/connection/api.py

@@ -1,10 +1,8 @@
 import json
 import logging
 from datetime import datetime
-from pathlib import Path
 from pprint import pformat
-from types import TracebackType
+from typing import Dict, Optional
-from typing import Dict, Optional, Any, List, no_type_check, Type
 
 from aiohttp import ClientSession
 from typing_extensions import Self
@@ -37,12 +35,7 @@ class HonAPI:
     async def __aenter__(self) -> Self:
         return await self.create()
 
-    async def __aexit__(
+    async def __aexit__(self, exc_type, exc_val, exc_tb) -> None:
-        self,
-        exc_type: Optional[Type[BaseException]],
-        exc: Optional[BaseException],
-        traceback: Optional[TracebackType],
-    ) -> None:
         await self.close()
 
     @property
@@ -52,13 +45,13 @@ class HonAPI:
         return self._hon.auth
 
     @property
-    def _hon(self) -> HonConnectionHandler:
+    def _hon(self):
         if self._hon_handler is None:
             raise exceptions.NoAuthenticationException
         return self._hon_handler
 
     @property
-    def _hon_anonymous(self) -> HonAnonymousConnectionHandler:
+    def _hon_anonymous(self):
         if self._hon_anonymous_handler is None:
             raise exceptions.NoAuthenticationException
         return self._hon_anonymous_handler
@@ -73,18 +66,12 @@ class HonAPI:
             ).create()
         return self
 
-    async def load_appliances(self) -> List[Dict[str, Any]]:
+    async def load_appliances(self) -> Dict:
         async with self._hon.get(f"{const.API_URL}/commands/v1/appliance") as resp:
-            result = await resp.json()
+            return await resp.json()
-        if result:
-            appliances: List[Dict[str, Any]] = result.get("payload", {}).get(
-                "appliances", {}
-            )
-            return appliances
-        return []
 
-    async def load_commands(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def load_commands(self, appliance: HonAppliance) -> Dict:
-        params: Dict[str, str | int] = {
+        params: Dict = {
             "applianceType": appliance.appliance_type,
             "applianceModelId": appliance.appliance_model_id,
             "macAddress": appliance.mac_address,
@@ -100,99 +87,87 @@ class HonAPI:
             params["series"] = series
         url: str = f"{const.API_URL}/commands/v1/retrieve"
         async with self._hon.get(url, params=params) as response:
-            result: Dict[str, Any] = (await response.json()).get("payload", {})
+            result: Dict = (await response.json()).get("payload", {})
             if not result or result.pop("resultCode") != "0":
                 _LOGGER.error(await response.json())
                 return {}
             return result
 
-    async def load_command_history(
+    async def command_history(self, appliance: HonAppliance) -> Dict:
-        self, appliance: HonAppliance
-    ) -> List[Dict[str, Any]]:
         url: str = (
             f"{const.API_URL}/commands/v1/appliance/{appliance.mac_address}/history"
         )
         async with self._hon.get(url) as response:
-            result: Dict[str, Any] = await response.json()
+            result: Dict = await response.json()
             if not result or not result.get("payload"):
-            return []
+                return {}
-        command_history: List[Dict[str, Any]] = result["payload"]["history"]
+            return result["payload"]["history"]
-        return command_history
 
-    async def load_favourites(self, appliance: HonAppliance) -> List[Dict[str, Any]]:
+    async def command_favourites(self, appliance: HonAppliance) -> Dict:
         url: str = (
             f"{const.API_URL}/commands/v1/appliance/{appliance.mac_address}/favourite"
         )
         async with self._hon.get(url) as response:
-            result: Dict[str, Any] = await response.json()
+            result: Dict = await response.json()
             if not result or not result.get("payload"):
-            return []
+                return {}
-        favourites: List[Dict[str, Any]] = result["payload"]["favourites"]
+            return result["payload"]["favourites"]
-        return favourites
 
-    async def load_last_activity(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def last_activity(self, appliance: HonAppliance) -> Dict:
         url: str = f"{const.API_URL}/commands/v1/retrieve-last-activity"
-        params: Dict[str, str] = {"macAddress": appliance.mac_address}
+        params: Dict = {"macAddress": appliance.mac_address}
         async with self._hon.get(url, params=params) as response:
-            result: Dict[str, Any] = await response.json()
+            result: Dict = await response.json()
-            if result:
+            if result and (activity := result.get("attributes")):
-                activity: Dict[str, Any] = result.get("attributes", "")
-                if activity:
                 return activity
         return {}
 
-    async def load_appliance_data(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def appliance_model(self, appliance: HonAppliance) -> Dict:
         url: str = f"{const.API_URL}/commands/v1/appliance-model"
-        params: Dict[str, str] = {
+        params: Dict = {
-            "code": appliance.code,
+            "code": appliance.info["code"],
             "macAddress": appliance.mac_address,
         }
         async with self._hon.get(url, params=params) as response:
-            result: Dict[str, Any] = await response.json()
+            result: Dict = await response.json()
-            if result:
+            if result and (activity := result.get("attributes")):
-                appliance_data: Dict[str, Any] = result.get("payload", {}).get(
+                return activity
-                    "applianceModel", {}
-                )
-                return appliance_data
         return {}
 
-    async def load_attributes(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def load_attributes(self, appliance: HonAppliance) -> Dict:
-        params: Dict[str, str] = {
+        params: Dict = {
             "macAddress": appliance.mac_address,
             "applianceType": appliance.appliance_type,
             "category": "CYCLE",
         }
         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 (await response.json()).get("payload", {})
-        return attributes
 
-    async def load_statistics(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def load_statistics(self, appliance: HonAppliance) -> Dict:
-        params: Dict[str, str] = {
+        params: Dict = {
             "macAddress": appliance.mac_address,
             "applianceType": appliance.appliance_type,
         }
         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 (await response.json()).get("payload", {})
-        return statistics
 
-    async def load_maintenance(self, appliance: HonAppliance) -> Dict[str, Any]:
+    async def load_maintenance(self, appliance: HonAppliance):
         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 (await response.json()).get("payload", {})
-        return maintenance
 
     async def send_command(
         self,
         appliance: HonAppliance,
         command: str,
-        parameters: Dict[str, Any],
+        parameters: Dict,
-        ancillary_parameters: Dict[str, Any],
+        ancillary_parameters: Dict,
     ) -> bool:
         now: str = datetime.utcnow().isoformat()
-        data: Dict[str, Any] = {
+        data: Dict = {
             "macAddress": appliance.mac_address,
             "timestamp": f"{now[:-3]}Z",
             "commandName": command,
@@ -210,25 +185,24 @@ class HonAPI:
         }
         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()
+            json_data: Dict = await response.json()
             if json_data.get("payload", {}).get("resultCode") == "0":
                 return True
             _LOGGER.error(await response.text())
             _LOGGER.error("%s - Payload:\n%s", url, pformat(data))
         return False
 
-    async def appliance_configuration(self) -> Dict[str, Any]:
+    async def appliance_configuration(self) -> Dict:
         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()
+            result: Dict = await response.json()
-        data: Dict[str, Any] = result.get("payload", {})
+            if result and (data := result.get("payload")):
                 return data
+        return {}
 
-    async def app_config(
+    async def app_config(self, language: str = "en", beta: bool = True) -> Dict:
-        self, language: str = "en", beta: bool = True
-    ) -> Dict[str, Any]:
         url: str = f"{const.API_URL}/app-config"
-        payload_data: Dict[str, str | int] = {
+        payload_data: Dict = {
             "languageCode": language,
             "beta": beta,
             "appVersion": const.APP_VERSION,
@@ -236,89 +210,20 @@ class HonAPI:
         }
         payload: str = json.dumps(payload_data, separators=(",", ":"))
         async with self._hon_anonymous.post(url, data=payload) as response:
-            result = await response.json()
+            if (result := await response.json()) and (data := result.get("payload")):
-        data: Dict[str, Any] = result.get("payload", {})
                 return data
-
-    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 {}
+
+    async def translation_keys(self, language: str = "en") -> Dict:
+        config = await self.app_config(language=language)
+        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:
             await self._hon_handler.close()
         if self._hon_anonymous_handler is not None:
             await self._hon_anonymous_handler.close()
-
-
-class TestAPI(HonAPI):
-    def __init__(self, path: Path):
-        super().__init__()
-        self._anonymous = True
-        self._path: Path = path
-
-    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 {}
-        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 {}
-
-    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:
-                    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]:
-        return self._load_json(appliance, "commands")
-
-    @no_type_check
-    async def load_command_history(
-        self, appliance: HonAppliance
-    ) -> List[Dict[str, Any]]:
-        return self._load_json(appliance, "command_history")
-
-    async def load_favourites(self, appliance: HonAppliance) -> List[Dict[str, Any]]:
-        return []
-
-    async def load_last_activity(self, appliance: HonAppliance) -> Dict[str, Any]:
-        return {}
-
-    async def load_appliance_data(self, appliance: HonAppliance) -> Dict[str, Any]:
-        return self._load_json(appliance, "appliance_data")
-
-    async def load_attributes(self, appliance: HonAppliance) -> Dict[str, Any]:
-        return self._load_json(appliance, "attributes")
-
-    async def load_statistics(self, appliance: HonAppliance) -> Dict[str, Any]:
-        return self._load_json(appliance, "statistics")
-
-    async def load_maintenance(self, appliance: HonAppliance) -> Dict[str, Any]:
-        return self._load_json(appliance, "maintenance")
-
-    async def send_command(
-        self,
-        appliance: HonAppliance,
-        command: str,
-        parameters: Dict[str, Any],
-        ancillary_parameters: Dict[str, Any],
-    ) -> bool:
-        _LOGGER.info("%s - %s", str(parameters), str(ancillary_parameters))
-        return True

pyhon/connection/auth.py

@@ -6,16 +6,14 @@ 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
 from urllib import parse
 from urllib.parse import quote
 
-import aiohttp
 from aiohttp import ClientResponse
 from yarl import URL
 
 from pyhon import const, exceptions
-from pyhon.connection.device import HonDevice
 from pyhon.connection.handler.auth import HonAuthConnectionHandler
 
 _LOGGER = logging.getLogger(__name__)
@@ -27,52 +25,41 @@ class HonLoginData:
     email: str = ""
     password: str = ""
     fw_uid: str = ""
-    loaded: Optional[Dict[str, Any]] = None
+    loaded: Optional[Dict] = 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
 
-    def __init__(
+    def __init__(self, session, email, password, device) -> None:
-        self,
-        session: aiohttp.ClientSession,
-        email: str,
-        password: str,
-        device: HonDevice,
-    ) -> None:
         self._session = session
         self._request = HonAuthConnectionHandler(session)
         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 +107,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 := re.findall("url = '(.+?)'", text)):
-            if not login_url:
                 if "oauth/done#access_token=" in text:
                     self._parse_token_data(text)
                     raise exceptions.HonNoAuthenticationNeeded()
@@ -134,7 +120,7 @@ class HonAuth:
                 await self._error_logger(response)
         return new_location
 
-    async def _handle_redirects(self, login_url: str) -> str:
+    async def _handle_redirects(self, login_url) -> str:
         redirect1 = await self._manual_redirect(login_url)
         redirect2 = await self._manual_redirect(redirect1)
         return f"{redirect2}&System=IoT_Mobile_App&RegistrationSubChannel=hOn"
@@ -190,19 +176,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 result["events"][0]["attributes"]["values"]["url"]
-                    return 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]
+            self._id_token = id_token[0]
-        return bool(access_token and refresh_token and id_token)
+        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 +219,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 +229,8 @@ class HonAuth:
             except json.JSONDecodeError:
                 await self._error_logger(response)
                 return False
-            self._auth.cognito_token = json_data.get("cognitoUser", {}).get("Token", "")
+            self._cognito_token = json_data.get("cognitoUser", {}).get("Token", "")
-            if not self._auth.cognito_token:
+            if not self._cognito_token:
                 _LOGGER.error(json_data)
                 raise exceptions.HonAuthenticationError()
         return True
@@ -267,7 +252,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 +263,14 @@ class HonAuth:
                 return False
             data = await response.json()
         self._expires = datetime.utcnow()
-        self._auth.id_token = data["id_token"]
+        self._id_token = data["id_token"]
-        self._auth.access_token = data["access_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._cognito_token = ""
-        self._auth.id_token = ""
+        self._id_token = ""
-        self._auth.access_token = ""
+        self._access_token = ""
-        self._auth.refresh_token = ""
+        self._refresh_token = ""

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
@@ -32,14 +32,12 @@ class HonDevice:
     def mobile_id(self) -> str:
         return self._mobile_id
 
-    def get(self, mobile: bool = False) -> Dict[str, str | int]:
+    def get(self, mobile: bool = False) -> Dict:
-        result: Dict[str, str | int] = {
+        result = {
             "appVersion": self.app_version,
             "mobileId": self.mobile_id,
-            "os": self.os_type,
+            "os": self.os,
             "osVersion": self.os_version,
             "deviceModel": self.device_model,
         }
-        if mobile:
+        return (result | {"mobileOs": result.pop("os")}) if mobile else result
-            result |= {"mobileOs": result.pop("os", "")}
-        return result

pyhon/connection/handler/anonym.py

@@ -1,27 +1,21 @@
 import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
-from typing import Dict, Any
+from typing import Callable, Dict
-
-import aiohttp
-from yarl import URL
 
 from pyhon import const
 from pyhon.connection.handler.base import ConnectionHandler
-from pyhon.typedefs import Callback
 
 _LOGGER = logging.getLogger(__name__)
 
 
 class HonAnonymousConnectionHandler(ConnectionHandler):
-    _HEADERS: Dict[str, str] = ConnectionHandler._HEADERS | {"x-api-key": const.API_KEY}
+    _HEADERS: Dict = ConnectionHandler._HEADERS | {"x-api-key": const.API_KEY}
 
     @asynccontextmanager
-    async def _intercept(
+    async def _intercept(self, method: Callable, *args, **kwargs) -> AsyncIterator:
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, 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

pyhon/connection/handler/auth.py

@@ -1,14 +1,12 @@
 import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
-from typing import Optional, List, Tuple, Any, Dict
+from typing import Optional, Callable, List, Tuple
 
 import aiohttp
-from yarl import URL
 
 from pyhon import const
 from pyhon.connection.handler.base import ConnectionHandler
-from pyhon.typedefs import Callback
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -30,9 +28,9 @@ class HonAuthConnectionHandler(ConnectionHandler):
 
     @asynccontextmanager
     async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, Any]
+        self, method: Callable, *args, loop: int = 0, **kwargs
-    ) -> AsyncIterator[aiohttp.ClientResponse]:
+    ) -> AsyncIterator:
         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)))
+            self._called_urls.append((response.status, response.request_info.url))
             yield response

pyhon/connection/handler/base.py

@@ -1,21 +1,18 @@
 import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
-from types import TracebackType
+from typing import Optional, Callable, Dict
-from typing import Optional, Dict, Type, Any
 
 import aiohttp
 from typing_extensions import Self
-from yarl import URL
 
 from pyhon import const, exceptions
-from pyhon.typedefs import Callback
 
 _LOGGER = logging.getLogger(__name__)
 
 
 class ConnectionHandler:
-    _HEADERS: Dict[str, str] = {
+    _HEADERS: Dict = {
         "user-agent": const.USER_AGENT,
         "Content-Type": "application/json",
     }
@@ -27,52 +24,32 @@ class ConnectionHandler:
     async def __aenter__(self) -> Self:
         return await self.create()
 
-    async def __aexit__(
+    async def __aexit__(self, exc_type, exc_val, exc_tb) -> None:
-        self,
-        exc_type: Optional[Type[BaseException]],
-        exc: Optional[BaseException],
-        traceback: Optional[TracebackType],
-    ) -> None:
         await self.close()
 
-    @property
-    def session(self) -> aiohttp.ClientSession:
-        if self._session is None:
-            raise exceptions.NoSessionException
-        return self._session
-
     async def create(self) -> Self:
         if self._create_session:
             self._session = aiohttp.ClientSession()
         return self
 
     @asynccontextmanager
-    async def _intercept(
+    def _intercept(self, method: Callable, *args, loop: int = 0, **kwargs):
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Dict[str, Any]
+        raise NotImplementedError
-    ) -> AsyncIterator[aiohttp.ClientResponse]:
-        async with method(url, *args, **kwargs) as response:
-            yield response
 
     @asynccontextmanager
-    async def get(
+    async def get(self, *args, **kwargs) -> AsyncIterator[aiohttp.ClientResponse]:
-        self, *args: Any, **kwargs: Any
-    ) -> AsyncIterator[aiohttp.ClientResponse]:
         if self._session is None:
             raise exceptions.NoSessionException()
         response: aiohttp.ClientResponse
-        args = self._session.get, *args
+        async with self._intercept(self._session.get, *args, **kwargs) as response:
-        async with self._intercept(*args, **kwargs) as response:
             yield response
 
     @asynccontextmanager
-    async def post(
+    async def post(self, *args, **kwargs) -> AsyncIterator[aiohttp.ClientResponse]:
-        self, *args: Any, **kwargs: Any
-    ) -> AsyncIterator[aiohttp.ClientResponse]:
         if self._session is None:
             raise exceptions.NoSessionException()
         response: aiohttp.ClientResponse
-        args = self._session.post, *args
+        async with self._intercept(self._session.post, *args, **kwargs) as response:
-        async with self._intercept(*args, **kwargs) as response:
             yield response
 
     async def close(self) -> None:

pyhon/connection/handler/hon.py

@@ -2,17 +2,15 @@ import json
 import logging
 from collections.abc import AsyncIterator
 from contextlib import asynccontextmanager
-from typing import Optional, Dict, Any
+from typing import Optional, Callable, Dict
 
 import aiohttp
 from typing_extensions import Self
-from yarl import URL
 
 from pyhon.connection.auth import HonAuth
 from pyhon.connection.device import HonDevice
 from pyhon.connection.handler.base import ConnectionHandler
 from pyhon.exceptions import HonAuthenticationError, NoAuthenticationException
-from pyhon.typedefs import Callback
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -43,10 +41,10 @@ class HonConnectionHandler(ConnectionHandler):
 
     async def create(self) -> Self:
         await super().create()
-        self._auth = HonAuth(self.session, self._email, self._password, self._device)
+        self._auth = HonAuth(self._session, self._email, self._password, self._device)
         return self
 
-    async def _check_headers(self, headers: Dict[str, str]) -> Dict[str, str]:
+    async def _check_headers(self, headers: Dict) -> Dict:
         if not (self.auth.cognito_token and self.auth.id_token):
             await self.auth.authenticate()
         headers["cognito-token"] = self.auth.cognito_token
@@ -55,18 +53,17 @@ class HonConnectionHandler(ConnectionHandler):
 
     @asynccontextmanager
     async def _intercept(
-        self, method: Callback, url: str | URL, *args: Any, **kwargs: Any
+        self, method: Callable, *args, loop: int = 0, **kwargs
-    ) -> AsyncIterator[aiohttp.ClientResponse]:
+    ) -> AsyncIterator:
-        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, **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
+                    method, *args, loop=loop + 1, **kwargs
                 ) as result:
                     yield result
             elif (
@@ -80,7 +77,7 @@ class HonConnectionHandler(ConnectionHandler):
                 )
                 await self.create()
                 async with self._intercept(
-                    method, url, *args, loop=loop + 1, **kwargs
+                    method, *args, loop=loop + 1, **kwargs
                 ) as result:
                     yield result
             elif loop >= 2:
@@ -95,11 +92,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")

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 = (
+# All seen id's (different accounts, different devices) are the same, so I guess this hash is static
-    "3MVG9QDx8IX8nP5T2Ha8ofvlmjLZl5L_gvfbT9."
+CLIENT_ID = "3MVG9QDx8IX8nP5T2Ha8ofvlmjLZl5L_gvfbT9.HJvpHGKoAS_dcMN8LYpTSYeVFCraUnV.2Ag1Ki7m4znVO6"
-    "HJvpHGKoAS_dcMN8LYpTSYeVFCraUnV.2Ag1Ki7m4znVO6"
+APP_VERSION = "2.0.10"
-)
-APP_VERSION = "2.1.2"
 OS_VERSION = 31
 OS = "android"
 DEVICE_MODEL = "exynos9820"

pyhon/diagnose.py

@@ -1,99 +0,0 @@
-import asyncio
-import json
-import re
-import shutil
-from pathlib import Path
-from typing import TYPE_CHECKING, List, Tuple
-
-from pyhon import printer
-
-if TYPE_CHECKING:
-    from pyhon.appliance import HonAppliance
-
-
-def anonymize_data(data: str) -> str:
-    default_date = "1970-01-01T00:00:00.0Z"
-    default_mac = "xx-xx-xx-xx-xx-xx"
-    data = re.sub("[0-9A-Fa-f]{2}(-[0-9A-Fa-f]{2}){5}", default_mac, data)
-    data = re.sub("[\\d-]{10}T[\\d:]{8}(.\\d+)?Z", default_date, data)
-    for sensible in [
-        "serialNumber",
-        "code",
-        "nickName",
-        "mobileId",
-        "PK",
-        "SK",
-        "lat",
-        "lng",
-    ]:
-        for match in re.findall(f'"{sensible}.*?":\\s"?(.+?)"?,?\\n', data):
-            replace = re.sub("[a-z]", "x", match)
-            replace = re.sub("[A-Z]", "X", replace)
-            replace = re.sub("\\d", "1", replace)
-            data = data.replace(match, replace)
-    return data
-
-
-async def load_data(appliance: "HonAppliance", topic: str) -> Tuple[str, str]:
-    return topic, await getattr(appliance.api, f"load_{topic}")(appliance)
-
-
-def write_to_json(data: str, topic: str, path: Path, anonymous: bool = False) -> Path:
-    json_data = json.dumps(data, indent=4)
-    if anonymous:
-        json_data = anonymize_data(json_data)
-    file = path / f"{topic}.json"
-    with open(file, "w", encoding="utf-8") as json_file:
-        json_file.write(json_data)
-    return file
-
-
-async def appliance_data(
-    appliance: "HonAppliance", path: Path, anonymous: bool = False
-) -> List[Path]:
-    requests = [
-        "commands",
-        "attributes",
-        "command_history",
-        "statistics",
-        "maintenance",
-        "appliance_data",
-    ]
-    path /= f"{appliance.appliance_type}_{appliance.model_id}".lower()
-    path.mkdir(parents=True, exist_ok=True)
-    api_data = await asyncio.gather(*[load_data(appliance, name) for name in requests])
-    return [write_to_json(data, topic, path, anonymous) for topic, data in api_data]
-
-
-async def zip_archive(
-    appliance: "HonAppliance", path: Path, anonymous: bool = False
-) -> str:
-    data = await appliance_data(appliance, path, anonymous)
-    archive = data[0].parent
-    shutil.make_archive(str(archive), "zip", archive)
-    shutil.rmtree(archive)
-    return f"{archive.stem}.zip"
-
-
-def yaml_export(appliance: "HonAppliance", anonymous: bool = False) -> str:
-    data = {
-        "attributes": appliance.attributes.copy(),
-        "appliance": appliance.info,
-        "statistics": appliance.statistics,
-        "additional_data": appliance.additional_data,
-    }
-    data |= {n: c.parameter_groups for n, c in appliance.commands.items()}
-    extra = {n: c.data for n, c in appliance.commands.items() if c.data}
-    if extra:
-        data |= {"extra_command_data": extra}
-    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})
-    if anonymous:
-        result = anonymize_data(result)
-    return result

pyhon/helper.py

@@ -1,5 +1,75 @@
-def str_to_float(string: str | float) -> float:
+def key_print(data, key="", start=True):
-    try:
+    result = ""
-        return int(string)
+    if isinstance(data, list):
-    except ValueError:
+        for i, value in enumerate(data):
-        return float(str(string).replace(",", "."))
+            result += key_print(value, key=f"{key}.{i}", start=False)
+    elif isinstance(data, dict):
+        for k, value in sorted(data.items()):
+            result += key_print(value, key=k if start else f"{key}.{k}", start=False)
+    else:
+        result += f"{key}: {data}\n"
+    return result
+
+
+# yaml.dump() would be done the same, but needs an additional dependency...
+def pretty_print(data, key="", intend=0, is_list=False, whitespace="  "):
+    result = ""
+    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):
+        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=key, intend=intend, is_list=True, whitespace=whitespace
+                )
+            elif is_list:
+                result += pretty_print(
+                    value, key=key, intend=intend + 1, whitespace=whitespace
+                )
+            else:
+                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_command(commands, concat=False):
+    result = {}
+    for name, command in commands.items():
+        for parameter, data in command.available_settings.items():
+            if data.typology == "enum":
+                value = data.values
+            elif data.typology == "range":
+                value = {"min": data.min, "max": data.max, "step": data.step}
+            else:
+                continue
+            if not concat:
+                result.setdefault(name, {})[parameter] = value
+            else:
+                result[f"{name}.{parameter}"] = value
+    return result
+
+
+def create_rules(commands, concat=False):
+    result = {}
+    for name, command in commands.items():
+        for parameter, data in command.available_settings.items():
+            value = data.triggers
+            if not value:
+                continue
+            if not concat:
+                result.setdefault(name, {})[parameter] = value
+            else:
+                result[f"{name}.{parameter}"] = value
+    return result

pyhon/hon.py

@@ -1,16 +1,13 @@
 import asyncio
 import logging
-from pathlib import Path
 from types import TracebackType
 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__)
 
@@ -21,14 +18,12 @@ class Hon:
         email: Optional[str] = "",
         password: Optional[str] = "",
         session: Optional[ClientSession] = None,
-        test_data_path: Optional[Path] = None,
     ):
         self._email: Optional[str] = email
         self._password: Optional[str] = password
         self._session: ClientSession | None = session
         self._appliances: List[HonAppliance] = []
         self._api: Optional[HonAPI] = None
-        self._test_data_path: Path = test_data_path or Path().cwd()
 
     async def __aenter__(self) -> Self:
         return await self.create()
@@ -44,7 +39,7 @@ class Hon:
     @property
     def api(self) -> HonAPI:
         if self._api is None:
-            raise NoAuthenticationException
+            raise exceptions.NoAuthenticationException
         return self._api
 
     @property
@@ -71,13 +66,11 @@ class Hon:
         return self._appliances
 
     @appliances.setter
-    def appliances(self, appliances: List[HonAppliance]) -> None:
+    def appliances(self, appliances) -> None:
         self._appliances = appliances
 
-    async def _create_appliance(
+    async def _create_appliance(self, appliance_data: Dict[str, Any], zone=0) -> None:
-        self, appliance_data: Dict[str, Any], api: HonAPI, zone: int = 0
+        appliance = HonAppliance(self._api, appliance_data, zone=zone)
-    ) -> None:
-        appliance = HonAppliance(api, appliance_data, zone=zone)
         if appliance.mac_address == "":
             return
         try:
@@ -94,20 +87,12 @@ class Hon:
         self._appliances.append(appliance)
 
     async def setup(self) -> None:
-        appliances = await self.api.load_appliances()
+        appliance: Dict
-        for appliance in appliances:
+        for appliance in (await self.api.load_appliances())["payload"]["appliances"]:
             if (zones := int(appliance.get("zone", "0"))) > 1:
                 for zone in range(zones):
-                    await self._create_appliance(
+                    await self._create_appliance(appliance.copy(), zone=zone + 1)
-                        appliance.copy(), self.api, zone=zone + 1
+            await self._create_appliance(appliance)
-                    )
-            await self._create_appliance(appliance, self.api)
-        if (
-            test_data := self._test_data_path / "hon-test-data" / "test_data"
-        ).exists() or (test_data := test_data / "test_data").exists():
-            api = TestAPI(test_data)
-            for appliance in await api.load_appliances():
-                await self._create_appliance(appliance, api)
 
     async def close(self) -> None:
         await self.api.close()

pyhon/parameter/base.py

@@ -7,21 +7,12 @@ 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 = attributes.get("category", "")
-        self._category: str = ""
+        self._typology: str = attributes.get("typology", "")
-        self._typology: str = ""
+        self._mandatory: int = attributes.get("mandatory", 0)
-        self._mandatory: int = 0
         self._value: str | float = ""
         self._group: str = group
-        self._triggers: Dict[
+        self._triggers: Dict[str, List[Tuple[Callable, "HonRule"]]] = {}
-            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:
@@ -60,39 +51,20 @@ class HonParameter:
     def group(self) -> str:
         return self._group
 
-    def add_trigger(
+    def add_trigger(self, value, func, data):
-        self, value: str, func: Callable[["HonRule"], None], data: "HonRule"
-    ) -> None:
         if self._value == value:
             func(data)
         self._triggers.setdefault(value, []).append((func, data))
 
-    def check_trigger(self, value: str | float) -> None:
+    def check_trigger(self, value) -> None:
-        triggers = {str(k).lower(): v for k, v in self._triggers.items()}
+        if str(value) in self._triggers:
-        if str(value).lower() in triggers:
+            for trigger in self._triggers[str(value)]:
-            for trigger in triggers[str(value)]:
                 func, args = trigger
                 func(args)
 
     @property
-    def triggers(self) -> Dict[str, Any]:
+    def triggers(self):
-        result: Dict[str, Any] = {}
+        result = {}
         for value, rules in self._triggers.items():
-            for _, rule in rules:
+            result[value] = {rule.param_key: rule.param_value for _, rule in rules}
-                if rule.extras:
-                    param = result.setdefault(value, {})
-                    for extra_key, extra_value in rule.extras.items():
-                        param = param.setdefault(extra_key, {}).setdefault(
-                            extra_value, {}
-                        )
-                else:
-                    param = result.setdefault(value, {})
-                if fixed_value := rule.param_data.get("fixedValue"):
-                    param[rule.param_key] = fixed_value
-                else:
-                    param[rule.param_key] = rule.param_data.get("defaultValue", "")
-
         return result
-
-    def reset(self) -> None:
-        self._set_attributes()

pyhon/parameter/enum.py

@@ -3,26 +3,19 @@ from typing import Dict, Any, List
 from pyhon.parameter.base import HonParameter
 
 
-def clean_value(value: str | float) -> str:
+def clean_value(value):
     return str(value).strip("[]").replace("|", "_").lower()
 
 
 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._default = attributes.get("defaultValue")
-        self._value: str | float = ""
+        self._value = self._default or "0"
-        self._values: List[str] = []
+        self._values: List[str] = attributes.get("enumValues", [])
-        self._set_attributes()
         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})"
 
@@ -31,7 +24,7 @@ class HonParameterEnum(HonParameter):
         return [clean_value(value) for value in self._values]
 
     @values.setter
-    def values(self, values: List[str]) -> None:
+    def values(self, values) -> None:
         self._values = values
 
     @property
@@ -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}")

pyhon/parameter/fixed.py

@@ -6,12 +6,7 @@ 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._value = attributes.get("fixedValue", None)
-        self._set_attributes()
-
-    def _set_attributes(self) -> None:
-        super()._set_attributes()
-        self._value = self._attributes.get("fixedValue", "")
 
     def __repr__(self) -> str:
         return f"{self.__class__} (<{self.key}> fixed)"

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]:
@@ -36,21 +36,19 @@ class HonParameterProgram(HonParameterEnum):
         return sorted(values)
 
     @values.setter
-    def values(self, values: List[str]) -> None:
+    def values(self, values) -> None:
-        raise ValueError("Cant set values {values}")
+        return
 
     @property
     def ids(self) -> Dict[int, str]:
-        values: Dict[int, str] = {}
+        values = {
-        for name, parameter in self._programs.items():
+            int(p.parameters["prCode"].value): n
-            if "iot_" in name:
+            for i, (n, p) in enumerate(self._programs.items())
-                continue
+            if "iot_" not in n
-            if parameter.parameters.get("prCode"):
+            and p.parameters.get("prCode")
-                continue
+            and not ((fav := p.parameters.get("favourite")) and fav.value == "1")
-            if (fav := parameter.parameters.get("favourite")) and fav.value == "1":
+        }
-                continue
-            values[int(parameter.parameters["prCode"].value)] = name
         return dict(sorted(values.items()))
 
-    def set_value(self, value: str) -> None:
+    def set_value(self, value: str):
         self._value = value

pyhon/parameter/range.py

@@ -1,28 +1,25 @@
 from typing import Dict, Any, List
 
-from pyhon.helper import str_to_float
 from pyhon.parameter.base import HonParameter
 
 
+def str_to_float(string: str | float) -> float:
+    try:
+        return int(string)
+    except ValueError:
+        return float(str(string).replace(",", "."))
+
+
 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._min: float = str_to_float(attributes["minimumValue"])
-        self._max: float = 0
+        self._max: float = str_to_float(attributes["maximumValue"])
-        self._step: float = 0
+        self._step: float = str_to_float(attributes["incrementValue"])
-        self._default: float = 0
+        self._default: float = str_to_float(attributes.get("defaultValue", self.min))
-        self._value: float = 0
+        self._value: float = self._default
-        self._set_attributes()
 
-    def _set_attributes(self) -> None:
+    def __repr__(self):
-        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
-
-    def __repr__(self) -> str:
         return f"{self.__class__} (<{self.key}> [{self.min} - {self.max}])"
 
     @property
@@ -58,14 +55,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) % (
+        if self.min <= value <= self.max and not (value - self.min) % self.step:
-            self.step * 100
-        ):
             self._value = value
             self.check_trigger(value)
         else:
-            allowed = f"min {self.min} max {self.max} step {self.step}"
+            raise ValueError(f"Allowed: min {self.min} max {self.max} step {self.step}")
-            raise ValueError(f"Allowed: {allowed} But was: {value}")
 
     @property
     def values(self) -> List[str]:

pyhon/printer.py

@@ -1,87 +0,0 @@
-from typing import Dict, Any, TYPE_CHECKING, List
-
-from pyhon.parameter.enum import HonParameterEnum
-from pyhon.parameter.range import HonParameterRange
-
-if TYPE_CHECKING:
-    from pyhon.commands import HonCommand
-
-
-def key_print(data: Any, key: str = "", start: bool = True) -> str:
-    result = ""
-    if isinstance(data, list):
-        for i, value in enumerate(data):
-            result += key_print(value, key=f"{key}.{i}", start=False)
-    elif isinstance(data, dict):
-        for k, value in sorted(data.items()):
-            result += key_print(value, key=k if start else f"{key}.{k}", start=False)
-    else:
-        result += f"{key}: {data}\n"
-    return result
-
-
-# yaml.dump() would be done the same, but needs an additional dependency...
-def pretty_print(
-    data: Any,
-    key: str = "",
-    intend: int = 0,
-    is_list: bool = False,
-    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):
-        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())):
-            result += pretty_print(
-                value,
-                key=list_key,
-                intend=intend + (is_list if i else 0),
-                is_list=is_list and not i,
-                whitespace=whitespace,
-            )
-    else:
-        result += f"{space}{'- ' if is_list else ''}{key}{': ' if key else ''}{data}\n"
-    return result
-
-
-def create_commands(
-    commands: Dict[str, "HonCommand"], concat: bool = False
-) -> Dict[str, Any]:
-    result: Dict[str, Any] = {}
-    for name, command in commands.items():
-        for parameter, data in command.available_settings.items():
-            if isinstance(data, HonParameterEnum):
-                value: List[str] | Dict[str, str | float] = data.values
-            elif isinstance(data, HonParameterRange):
-                value = {"min": data.min, "max": data.max, "step": data.step}
-            else:
-                continue
-            if not concat:
-                result.setdefault(name, {})[parameter] = value
-            else:
-                result[f"{name}.{parameter}"] = value
-    return result
-
-
-def create_rules(
-    commands: Dict[str, "HonCommand"], concat: bool = False
-) -> Dict[str, Any]:
-    result: Dict[str, Any] = {}
-    for name, command in commands.items():
-        for parameter, data in command.available_settings.items():
-            value = data.triggers
-            if not value:
-                continue
-            if not concat:
-                result.setdefault(name, {})[parameter] = value
-            else:
-                result[f"{name}.{parameter}"] = value
-    return result

pyhon/rules.py

@@ -1,13 +1,11 @@
 from dataclasses import dataclass
-from typing import List, Dict, TYPE_CHECKING, Any, Optional
+from typing import List, Dict, TYPE_CHECKING
 
 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
-    from pyhon.parameter.base import HonParameter
 
 
 @dataclass
@@ -15,131 +13,50 @@ class HonRule:
     trigger_key: str
     trigger_value: str
     param_key: str
-    param_data: Dict[str, Any]
+    param_value: str
-    extras: Optional[Dict[str, str]] = None
 
 
 class HonRuleSet:
-    def __init__(self, command: "HonCommand", rule: Dict[str, Any]):
+    def __init__(self, command: "HonCommand", rule):
         self._command: "HonCommand" = command
         self._rules: Dict[str, List[HonRule]] = {}
         self._parse_rule(rule)
 
-    @property
+    def _parse_rule(self, rule):
-    def rules(self) -> Dict[str, List[HonRule]]:
+        for entity_key, params in rule.items():
-        return self._rules
+            entity_key = self._command.appliance.options.get(entity_key, entity_key)
-
+            for trigger_key, values in params.items():
-    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)
-            for trigger_key, trigger_data in params.items():
-                self._parse_conditions(param_key, trigger_key, trigger_data)
-
-    def _parse_conditions(
-        self,
-        param_key: str,
-        trigger_key: str,
-        trigger_data: Dict[str, Any],
-        extra: Optional[Dict[str, str]] = None,
-    ) -> None:
                 trigger_key = trigger_key.replace("@", "")
-        trigger_key = self._command.appliance.options.get(trigger_key, trigger_key)
+                trigger_key = self._command.appliance.options.get(
-        for multi_trigger_value, param_data in trigger_data.items():
+                    trigger_key, trigger_key
-            for trigger_value in multi_trigger_value.split("|"):
-                if isinstance(param_data, dict) and "typology" in param_data:
-                    self._create_rule(
-                        param_key, trigger_key, trigger_value, param_data, extra
                 )
-                elif isinstance(param_data, dict):
+                for trigger_value, entity_value in values.items():
-                    if extra is None:
+                    if entity_value.get("fixedValue") == f"@{entity_key}":
-                        extra = {}
-                    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,
-        param_key: str,
-        trigger_key: str,
-        trigger_value: str,
-        param_data: Dict[str, Any],
-        extras: Optional[Dict[str, str]] = None,
-    ) -> None:
-        if param_data.get("fixedValue") == f"@{param_key}":
-            return
-        self._rules.setdefault(trigger_key, []).append(
-            HonRule(trigger_key, trigger_value, param_key, param_data, extras)
-        )
-
-    def _duplicate_for_extra_conditions(self) -> None:
-        new: Dict[str, List[HonRule]] = {}
-        for rules in self._rules.values():
-            for rule in rules:
-                if rule.extras is None:
                         continue
-                for key, value in rule.extras.items():
+                    self._rules.setdefault(trigger_key, []).append(
-                    extras = rule.extras.copy()
+                        HonRule(
-                    extras.pop(key)
+                            trigger_key,
-                    extras[rule.trigger_key] = rule.trigger_value
+                            trigger_value,
-                    new.setdefault(key, []).append(
+                            entity_key,
-                        HonRule(key, value, rule.param_key, rule.param_data, extras)
+                            entity_value.get("fixedValue"),
+                        )
                     )
-        for key, rules in new.items():
-            for rule in rules:
-                self._rules.setdefault(key, []).append(rule)
 
-    def _extra_rules_matches(self, rule: HonRule) -> bool:
+    def patch(self):
-        if rule.extras:
-            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)}:
-            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
-        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:
-        self._duplicate_for_extra_conditions()
         for name, parameter in self._command.parameters.items():
             if name not in self._rules:
                 continue
-            for data in self._rules.get(name, []):
+            for data in self._rules.get(name):
-                self._add_trigger(parameter, data)
+
+                def apply(rule):
+                    if param := self._command.parameters.get(rule.param_key):
+                        if isinstance(param, HonParameterEnum) and set(
+                            param.values
+                        ) != {str(rule.param_value)}:
+                            param.values = [str(rule.param_value)]
+                        elif isinstance(param, HonParameterRange):
+                            param.value = float(rule.param_value)
+                            return
+                        param.value = str(rule.param_value)
+
+                parameter.add_trigger(data.trigger_value, apply, data)

pyhon/typedefs.py

@@ -1,27 +0,0 @@
-from typing import Union, Any, TYPE_CHECKING, Protocol
-
-import aiohttp
-from yarl import URL
-
-if TYPE_CHECKING:
-    from pyhon.parameter.base import HonParameter
-    from pyhon.parameter.enum import HonParameterEnum
-    from pyhon.parameter.fixed import HonParameterFixed
-    from pyhon.parameter.program import HonParameterProgram
-    from pyhon.parameter.range import HonParameterRange
-
-
-class Callback(Protocol):  # pylint: disable=too-few-public-methods
-    def __call__(
-        self, url: str | URL, *args: Any, **kwargs: Any
-    ) -> aiohttp.client._RequestContextManager:
-        ...
-
-
-Parameter = Union[
-    "HonParameter",
-    "HonParameterRange",
-    "HonParameterEnum",
-    "HonParameterFixed",
-    "HonParameterProgram",
-]

requirements.txt

@@ -1,3 +1,2 @@
-aiohttp~=3.8.5
+aiohttp==3.8.4
-yarl~=1.9.2
+yarl==1.8.2
-typing-extensions~=4.7.1

requirements_dev.txt

@@ -1,4 +1,4 @@
-black==23.7.0
+black==23.3.0
 flake8==6.0.0
-mypy==1.4.1
+mypy==1.2.0
-pylint==2.17.4
+pylint==2.17.2

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.7",
+    version="0.12.4",
     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.5", "typing-extensions~=4.7.1", "yarl~=1.9.2"],
+    install_requires=["aiohttp"],
     classifiers=[
         "Development Status :: 4 - Beta",
         "Environment :: Console",