python-pooldose 0.4.0__tar.gz → 0.4.1__tar.gz

{python_pooldose-0.4.0/src/python_pooldose.egg-info → python_pooldose-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-pooldose
-Version: 0.4.0
+Version: 0.4.1
 Summary: Unoffical async Python client for SEKO PoolDose devices
 Author-email: Lukas Maertin <pypi@lukas-maertin.de>
 License-Expression: MIT
@@ -125,7 +125,7 @@ pip install python-pooldose
 import asyncio
 import json
 from pooldose.client import PooldoseClient
-from pooldose.request_handler import RequestStatus
+from pooldose.client import RequestStatus
 
 HOST = "192.168.1.100"  # Change this to your device's host or IP address
 TIMEOUT = 30
@@ -211,6 +211,8 @@ if __name__ == "__main__":
 
 #### Connection Management
 ```python
+from pooldose.client import PooldoseClient, RequestStatus
+
 # Recommended: Separate initialization and connection
 client = PooldoseClient("192.168.1.100", timeout=30)
 status = await client.connect()
@@ -224,7 +226,7 @@ else:
 
 #### Error Handling
 ```python
-from pooldose.request_handler import RequestStatus
+from pooldose.client import PooldoseClient, RequestStatus
 
 client = PooldoseClient("192.168.1.100")
 status = await client.connect()
@@ -241,9 +243,84 @@ else:
     print(f"Other error: {status}")
 ```
 
+#### Type-specific Access
+```python
+# Get all values by type
+sensors = instant_values.get_sensors()          # All sensor readings
+binary_sensors = instant_values.get_binary_sensors()  # All boolean states
+numbers = instant_values.get_numbers()          # All configurable numbers
+switches = instant_values.get_switches()        # All switch states
+selects = instant_values.get_selects()          # All select options
+
+# Check available types dynamically
+available_types = instant_values.available_types()
+print("Available types:", list(available_types.keys()))
+```
+
+#### Working with Mappings
+```
+Mapping Discovery Process:
+┌─────────────────┐
+│ Device Connect  │
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Get MODEL_ID    │ ──────► PDPR1H1HAW100
+│ Get FW_CODE     │ ──────► 539187
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Load JSON File  │ ──────► model_PDPR1H1HAW100_FW539187.json
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Type Discovery  │
+│ ┌─────────────┐ │
+│ │ Sensors     │ │ ──────► temperature, ph, orp, ...
+│ │ Switches    │ │ ──────► stop_dosing, pump_detection, ...
+│ │ Numbers     │ │ ──────► ph_target, orp_target, ...
+│ │ Selects     │ │ ──────► water_meter_unit, ...
+│ └─────────────┘ │
+└─────────────────┘
+```
+
+```python
+# Query what's available for your specific device
+print("\nAvailable sensors:")
+for name, sensor in client.available_sensors().items():
+    print(f"  {name}: key={sensor.key}")
+    if sensor.conversion:
+        print(f"    conversion: {sensor.conversion}")
+
+print("\nAvailable numbers (settable):")
+for name, number in client.available_numbers().items():
+    print(f"  {name}: key={number.key}")
+
+print("\nAvailable switches:")
+for name, switch in client.available_switches().items():
+    print(f"  {name}: key={switch.key}")
+```
+
 ## API Reference
 
-### PooldoseClient
+### PooldoseClient Class Hierarchy
+```
+PooldoseClient
+├── Device Info
+│   ├── static_values() ──────► StaticValues
+│   └── device_info{} ─────────► dict
+├── Type Discovery
+│   ├── available_types() ────► dict[str, list[str]]
+│   ├── available_sensors() ──► dict[str, SensorMapping]
+│   ├── available_numbers() ──► dict[str, NumberMapping]
+│   ├── available_switches() ─► dict[str, SwitchMapping]
+│   └── available_selects() ──► dict[str, SelectMapping]
+└── Live Data
+    └── instant_values() ─────► InstantValues
+```
 
 #### Constructor
 ```python
@@ -272,7 +349,41 @@ PooldoseClient(host, timeout=10, include_sensitive_data=False)
 - `host` - Device hostname or IP address
 - `timeout` - Request timeout in seconds
 
-### InstantValues
+### RequestStatus
+
+All client methods return `RequestStatus` enum values:
+
+```python
+from pooldose.client import RequestStatus
+
+RequestStatus.SUCCESS                    # Operation successful
+RequestStatus.CONNECTION_ERROR           # Network connection failed
+RequestStatus.HOST_UNREACHABLE           # Device not reachable
+RequestStatus.PARAMS_FETCH_FAILED        # Failed to fetch device parameters
+RequestStatus.API_VERSION_UNSUPPORTED    # API version not supported
+RequestStatus.NO_DATA                    # No data received
+RequestStatus.UNKNOWN_ERROR              # Other error occurred
+```
+
+### InstantValues Interface
+```
+InstantValues
+├── Dictionary Interface
+│   ├── [key] ─────────────────► __getitem__
+│   ├── get(key, default) ────► get method
+│   ├── key in values ────────► __contains__
+│   └── [key] = value ────────► __setitem__ (async)
+├── Type Getters
+│   ├── get_sensors() ────────► dict[str, tuple]
+│   ├── get_binary_sensors() ─► dict[str, bool]
+│   ├── get_numbers() ────────► dict[str, tuple]
+│   ├── get_switches() ───────► dict[str, bool]
+│   └── get_selects() ────────► dict[str, int]
+└── Type Setters (async)
+    ├── set_number(key, value) ──► bool
+    ├── set_switch(key, value) ──► bool
+    └── set_select(key, value) ──► bool
+```
 
 #### Dictionary Interface
 ```python
@@ -321,8 +432,31 @@ client = PooldoseClient(
 status = await client.connect()
 ```
 
+### Security Model
+```
+Data Classification:
+┌─────────────────┐    ┌─────────────────┐    ┌─────────────────┐
+│   Public Data   │    │ Sensitive Data  │    │  Never Exposed  │
+├─────────────────┤    ├─────────────────┤    ├─────────────────┤
+│ • Device Name   │    │ • WiFi Password │    │ • Admin Creds   │
+│ • Model ID      │    │ • AP Password   │    │ • Internal Keys │
+│ • Serial Number │    │                 │    │                 │
+│ • Sensor Values │    │                 │    │                 │
+│ • IP Address    │    │                 │    │                 │
+│ • MAC Address   │    │                 │    │                 │
+└─────────────────┘    └─────────────────┘    └─────────────────┘
+        │                       │                       │
+        ▼                       ▼                       ▼
+  Always Included      include_sensitive_data=True    Never Included
+```
+
 ## Changelog
 
+### [0.4.1] - 2025-07-17
+- **BREAKING**: Moved all RequestStatus into client module - import from `pooldose.client` instead of `pooldose.request_handler`
+- Moved all connect checks into client (incl. API Version check) to avoid public access to requesthandler
+- Clean up code and improved encapsulation
+
 ### [0.4.0] - 2025-07-11
 - **BREAKING**: Removed `create()` factory method
 - **BREAKING**: Changed client initialization pattern to separate `__init__` and async `connect()` methods

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/README.md

@@ -107,7 +107,7 @@ pip install python-pooldose
 import asyncio
 import json
 from pooldose.client import PooldoseClient
-from pooldose.request_handler import RequestStatus
+from pooldose.client import RequestStatus
 
 HOST = "192.168.1.100"  # Change this to your device's host or IP address
 TIMEOUT = 30
@@ -193,6 +193,8 @@ if __name__ == "__main__":
 
 #### Connection Management
 ```python
+from pooldose.client import PooldoseClient, RequestStatus
+
 # Recommended: Separate initialization and connection
 client = PooldoseClient("192.168.1.100", timeout=30)
 status = await client.connect()
@@ -206,7 +208,7 @@ else:
 
 #### Error Handling
 ```python
-from pooldose.request_handler import RequestStatus
+from pooldose.client import PooldoseClient, RequestStatus
 
 client = PooldoseClient("192.168.1.100")
 status = await client.connect()
@@ -223,9 +225,84 @@ else:
     print(f"Other error: {status}")
 ```
 
+#### Type-specific Access
+```python
+# Get all values by type
+sensors = instant_values.get_sensors()          # All sensor readings
+binary_sensors = instant_values.get_binary_sensors()  # All boolean states
+numbers = instant_values.get_numbers()          # All configurable numbers
+switches = instant_values.get_switches()        # All switch states
+selects = instant_values.get_selects()          # All select options
+
+# Check available types dynamically
+available_types = instant_values.available_types()
+print("Available types:", list(available_types.keys()))
+```
+
+#### Working with Mappings
+```
+Mapping Discovery Process:
+┌─────────────────┐
+│ Device Connect  │
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Get MODEL_ID    │ ──────► PDPR1H1HAW100
+│ Get FW_CODE     │ ──────► 539187
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Load JSON File  │ ──────► model_PDPR1H1HAW100_FW539187.json
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Type Discovery  │
+│ ┌─────────────┐ │
+│ │ Sensors     │ │ ──────► temperature, ph, orp, ...
+│ │ Switches    │ │ ──────► stop_dosing, pump_detection, ...
+│ │ Numbers     │ │ ──────► ph_target, orp_target, ...
+│ │ Selects     │ │ ──────► water_meter_unit, ...
+│ └─────────────┘ │
+└─────────────────┘
+```
+
+```python
+# Query what's available for your specific device
+print("\nAvailable sensors:")
+for name, sensor in client.available_sensors().items():
+    print(f"  {name}: key={sensor.key}")
+    if sensor.conversion:
+        print(f"    conversion: {sensor.conversion}")
+
+print("\nAvailable numbers (settable):")
+for name, number in client.available_numbers().items():
+    print(f"  {name}: key={number.key}")
+
+print("\nAvailable switches:")
+for name, switch in client.available_switches().items():
+    print(f"  {name}: key={switch.key}")
+```
+
 ## API Reference
 
-### PooldoseClient
+### PooldoseClient Class Hierarchy
+```
+PooldoseClient
+├── Device Info
+│   ├── static_values() ──────► StaticValues
+│   └── device_info{} ─────────► dict
+├── Type Discovery
+│   ├── available_types() ────► dict[str, list[str]]
+│   ├── available_sensors() ──► dict[str, SensorMapping]
+│   ├── available_numbers() ──► dict[str, NumberMapping]
+│   ├── available_switches() ─► dict[str, SwitchMapping]
+│   └── available_selects() ──► dict[str, SelectMapping]
+└── Live Data
+    └── instant_values() ─────► InstantValues
+```
 
 #### Constructor
 ```python
@@ -254,7 +331,41 @@ PooldoseClient(host, timeout=10, include_sensitive_data=False)
 - `host` - Device hostname or IP address
 - `timeout` - Request timeout in seconds
 
-### InstantValues
+### RequestStatus
+
+All client methods return `RequestStatus` enum values:
+
+```python
+from pooldose.client import RequestStatus
+
+RequestStatus.SUCCESS                    # Operation successful
+RequestStatus.CONNECTION_ERROR           # Network connection failed
+RequestStatus.HOST_UNREACHABLE           # Device not reachable
+RequestStatus.PARAMS_FETCH_FAILED        # Failed to fetch device parameters
+RequestStatus.API_VERSION_UNSUPPORTED    # API version not supported
+RequestStatus.NO_DATA                    # No data received
+RequestStatus.UNKNOWN_ERROR              # Other error occurred
+```
+
+### InstantValues Interface
+```
+InstantValues
+├── Dictionary Interface
+│   ├── [key] ─────────────────► __getitem__
+│   ├── get(key, default) ────► get method
+│   ├── key in values ────────► __contains__
+│   └── [key] = value ────────► __setitem__ (async)
+├── Type Getters
+│   ├── get_sensors() ────────► dict[str, tuple]
+│   ├── get_binary_sensors() ─► dict[str, bool]
+│   ├── get_numbers() ────────► dict[str, tuple]
+│   ├── get_switches() ───────► dict[str, bool]
+│   └── get_selects() ────────► dict[str, int]
+└── Type Setters (async)
+    ├── set_number(key, value) ──► bool
+    ├── set_switch(key, value) ──► bool
+    └── set_select(key, value) ──► bool
+```
 
 #### Dictionary Interface
 ```python
@@ -303,8 +414,31 @@ client = PooldoseClient(
 status = await client.connect()
 ```
 
+### Security Model
+```
+Data Classification:
+┌─────────────────┐    ┌─────────────────┐    ┌─────────────────┐
+│   Public Data   │    │ Sensitive Data  │    │  Never Exposed  │
+├─────────────────┤    ├─────────────────┤    ├─────────────────┤
+│ • Device Name   │    │ • WiFi Password │    │ • Admin Creds   │
+│ • Model ID      │    │ • AP Password   │    │ • Internal Keys │
+│ • Serial Number │    │                 │    │                 │
+│ • Sensor Values │    │                 │    │                 │
+│ • IP Address    │    │                 │    │                 │
+│ • MAC Address   │    │                 │    │                 │
+└─────────────────┘    └─────────────────┘    └─────────────────┘
+        │                       │                       │
+        ▼                       ▼                       ▼
+  Always Included      include_sensitive_data=True    Never Included
+```
+
 ## Changelog
 
+### [0.4.1] - 2025-07-17
+- **BREAKING**: Moved all RequestStatus into client module - import from `pooldose.client` instead of `pooldose.request_handler`
+- Moved all connect checks into client (incl. API Version check) to avoid public access to requesthandler
+- Clean up code and improved encapsulation
+
 ### [0.4.0] - 2025-07-11
 - **BREAKING**: Removed `create()` factory method
 - **BREAKING**: Changed client initialization pattern to separate `__init__` and async `connect()` methods

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-pooldose"
-version = "0.4.0"
+version = "0.4.1"
 description = "Unoffical async Python client for SEKO PoolDose devices"
 authors = [
     { name = "Lukas Maertin", email = "pypi@lukas-maertin.de" }

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/src/pooldose/client.py

@@ -21,6 +21,8 @@ from pooldose.mappings.mapping_info import (
 
 _LOGGER = logging.getLogger(__name__)
 
+API_VERSION_SUPPORTED = "v1/"
+
 class PooldoseClient:
     """
     Async client for SEKO Pooldose API.
@@ -92,6 +94,32 @@ class PooldoseClient:
         _LOGGER.debug("Initialized Pooldose client with device info: %s", self.device_info)
         return RequestStatus.SUCCESS
 
+    def check_apiversion_supported(self) -> tuple[RequestStatus, dict]:
+        """
+        Check if the loaded API version matches the supported version.
+
+        Returns:
+            tuple: (RequestStatus, dict)
+                - dict contains:
+                    "api_version_is": the current API version (or None if not set)
+                    "api_version_should": the supported API version
+                - RequestStatus.SUCCESS if supported,
+                - RequestStatus.API_VERSION_UNSUPPORTED if not supported,
+                - RequestStatus.NO_DATA if not set.
+        """
+        result = {
+            "api_version_is": self._request_handler.api_version,
+            "api_version_should": API_VERSION_SUPPORTED,
+        }
+        if not self._request_handler.api_version:
+            _LOGGER.warning("API version not set, cannot check support")
+            return RequestStatus.NO_DATA, result
+        if self._request_handler.api_version != API_VERSION_SUPPORTED:
+            _LOGGER.warning("Unsupported API version: %s, expected %s", self._request_handler.api_version, API_VERSION_SUPPORTED)
+            return RequestStatus.API_VERSION_UNSUPPORTED, result
+
+        return RequestStatus.SUCCESS, result
+
     async def _load_device_info(self) -> RequestStatus:
         """
         Load device information from the request handler.

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/src/pooldose/request_handler.py

@@ -5,39 +5,15 @@ import json
 import re
 import socket
 from typing import Any
-from enum import Enum
 import asyncio
 import aiohttp
 
+from pooldose.request_status import RequestStatus
+
 # pylint: disable=line-too-long,no-else-return
 
 _LOGGER = logging.getLogger(__name__)
 
-API_VERSION_SUPPORTED = "v1/"
-
-class RequestStatus(Enum):
-    """
-    Enum for standardized return codes of API and client methods.
-
-    Each status represents a specific result or error case:
-    - SUCCESS: Operation was successful.
-    - HOST_UNREACHABLE: The host could not be reached (e.g. network error).
-    - PARAMS_FETCH_FAILED: params.js could not be fetched or parsed.
-    - API_VERSION_UNSUPPORTED: The API version is not supported.
-    - NO_DATA: No data was returned or found.
-    - LAST_DATA: No new data was found, last valid data was returned.
-    - CLIENT_ERROR_SET: Error while setting a value on the client/device.
-    - UNKNOWN_ERROR: An unspecified or unexpected error occurred.
-    """
-    SUCCESS = "success"
-    HOST_UNREACHABLE = "host_unreachable"
-    PARAMS_FETCH_FAILED = "params_fetch_failed"
-    API_VERSION_UNSUPPORTED = "api_version_unsupported"
-    NO_DATA = "no_data"
-    LAST_DATA = "last_data"
-    CLIENT_ERROR_SET = "client_error_set"
-    UNKNOWN_ERROR = "unknown_error"
-
 class RequestHandler:
     """
     Handles all HTTP requests to the Pooldose API.
@@ -126,32 +102,6 @@ class RequestHandler:
             _LOGGER.warning("Error fetching core params: %s", err)
             return None
 
-    def check_apiversion_supported(self) -> tuple[RequestStatus, dict]:
-        """
-        Check if the loaded API version matches the supported version.
-
-        Returns:
-            tuple: (RequestStatus, dict)
-                - dict contains:
-                    "api_version_is": the current API version (or None if not set)
-                    "api_version_should": the supported API version
-                - RequestStatus.SUCCESS if supported,
-                - RequestStatus.API_VERSION_UNSUPPORTED if not supported,
-                - RequestStatus.NO_DATA if not set.
-        """
-        result = {
-            "api_version_is": self.api_version,
-            "api_version_should": API_VERSION_SUPPORTED,
-        }
-        if not self.api_version:
-            _LOGGER.warning("API version not set, cannot check support")
-            return RequestStatus.NO_DATA, result
-        if self.api_version != API_VERSION_SUPPORTED:
-            _LOGGER.warning("Unsupported API version: %s, expected %s", self.api_version, API_VERSION_SUPPORTED)
-            return RequestStatus.API_VERSION_UNSUPPORTED, result
-        else:
-            return RequestStatus.SUCCESS, result
-
     async def get_debug_config(self):
         """
         Asynchronously fetches the debug configuration from the server.

python_pooldose-0.4.1/src/pooldose/request_status.py

@@ -0,0 +1,26 @@
+"""Request Status for async API client for SEKO Pooldose."""
+
+from enum import Enum
+
+class RequestStatus(Enum):
+    """
+    Enum for standardized return codes of API and client methods.
+
+    Each status represents a specific result or error case:
+    - SUCCESS: Operation was successful.
+    - HOST_UNREACHABLE: The host could not be reached (e.g. network error).
+    - PARAMS_FETCH_FAILED: params.js could not be fetched or parsed.
+    - API_VERSION_UNSUPPORTED: The API version is not supported.
+    - NO_DATA: No data was returned or found.
+    - LAST_DATA: No new data was found, last valid data was returned.
+    - CLIENT_ERROR_SET: Error while setting a value on the client/device.
+    - UNKNOWN_ERROR: An unspecified or unexpected error occurred.
+    """
+    SUCCESS = "success"
+    HOST_UNREACHABLE = "host_unreachable"
+    PARAMS_FETCH_FAILED = "params_fetch_failed"
+    API_VERSION_UNSUPPORTED = "api_version_unsupported"
+    NO_DATA = "no_data"
+    LAST_DATA = "last_data"
+    CLIENT_ERROR_SET = "client_error_set"
+    UNKNOWN_ERROR = "unknown_error"

{python_pooldose-0.4.0 → python_pooldose-0.4.1/src/python_pooldose.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-pooldose
-Version: 0.4.0
+Version: 0.4.1
 Summary: Unoffical async Python client for SEKO PoolDose devices
 Author-email: Lukas Maertin <pypi@lukas-maertin.de>
 License-Expression: MIT
@@ -125,7 +125,7 @@ pip install python-pooldose
 import asyncio
 import json
 from pooldose.client import PooldoseClient
-from pooldose.request_handler import RequestStatus
+from pooldose.client import RequestStatus
 
 HOST = "192.168.1.100"  # Change this to your device's host or IP address
 TIMEOUT = 30
@@ -211,6 +211,8 @@ if __name__ == "__main__":
 
 #### Connection Management
 ```python
+from pooldose.client import PooldoseClient, RequestStatus
+
 # Recommended: Separate initialization and connection
 client = PooldoseClient("192.168.1.100", timeout=30)
 status = await client.connect()
@@ -224,7 +226,7 @@ else:
 
 #### Error Handling
 ```python
-from pooldose.request_handler import RequestStatus
+from pooldose.client import PooldoseClient, RequestStatus
 
 client = PooldoseClient("192.168.1.100")
 status = await client.connect()
@@ -241,9 +243,84 @@ else:
     print(f"Other error: {status}")
 ```
 
+#### Type-specific Access
+```python
+# Get all values by type
+sensors = instant_values.get_sensors()          # All sensor readings
+binary_sensors = instant_values.get_binary_sensors()  # All boolean states
+numbers = instant_values.get_numbers()          # All configurable numbers
+switches = instant_values.get_switches()        # All switch states
+selects = instant_values.get_selects()          # All select options
+
+# Check available types dynamically
+available_types = instant_values.available_types()
+print("Available types:", list(available_types.keys()))
+```
+
+#### Working with Mappings
+```
+Mapping Discovery Process:
+┌─────────────────┐
+│ Device Connect  │
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Get MODEL_ID    │ ──────► PDPR1H1HAW100
+│ Get FW_CODE     │ ──────► 539187
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Load JSON File  │ ──────► model_PDPR1H1HAW100_FW539187.json
+└─────────────────┘
+         │
+         ▼
+┌─────────────────┐
+│ Type Discovery  │
+│ ┌─────────────┐ │
+│ │ Sensors     │ │ ──────► temperature, ph, orp, ...
+│ │ Switches    │ │ ──────► stop_dosing, pump_detection, ...
+│ │ Numbers     │ │ ──────► ph_target, orp_target, ...
+│ │ Selects     │ │ ──────► water_meter_unit, ...
+│ └─────────────┘ │
+└─────────────────┘
+```
+
+```python
+# Query what's available for your specific device
+print("\nAvailable sensors:")
+for name, sensor in client.available_sensors().items():
+    print(f"  {name}: key={sensor.key}")
+    if sensor.conversion:
+        print(f"    conversion: {sensor.conversion}")
+
+print("\nAvailable numbers (settable):")
+for name, number in client.available_numbers().items():
+    print(f"  {name}: key={number.key}")
+
+print("\nAvailable switches:")
+for name, switch in client.available_switches().items():
+    print(f"  {name}: key={switch.key}")
+```
+
 ## API Reference
 
-### PooldoseClient
+### PooldoseClient Class Hierarchy
+```
+PooldoseClient
+├── Device Info
+│   ├── static_values() ──────► StaticValues
+│   └── device_info{} ─────────► dict
+├── Type Discovery
+│   ├── available_types() ────► dict[str, list[str]]
+│   ├── available_sensors() ──► dict[str, SensorMapping]
+│   ├── available_numbers() ──► dict[str, NumberMapping]
+│   ├── available_switches() ─► dict[str, SwitchMapping]
+│   └── available_selects() ──► dict[str, SelectMapping]
+└── Live Data
+    └── instant_values() ─────► InstantValues
+```
 
 #### Constructor
 ```python
@@ -272,7 +349,41 @@ PooldoseClient(host, timeout=10, include_sensitive_data=False)
 - `host` - Device hostname or IP address
 - `timeout` - Request timeout in seconds
 
-### InstantValues
+### RequestStatus
+
+All client methods return `RequestStatus` enum values:
+
+```python
+from pooldose.client import RequestStatus
+
+RequestStatus.SUCCESS                    # Operation successful
+RequestStatus.CONNECTION_ERROR           # Network connection failed
+RequestStatus.HOST_UNREACHABLE           # Device not reachable
+RequestStatus.PARAMS_FETCH_FAILED        # Failed to fetch device parameters
+RequestStatus.API_VERSION_UNSUPPORTED    # API version not supported
+RequestStatus.NO_DATA                    # No data received
+RequestStatus.UNKNOWN_ERROR              # Other error occurred
+```
+
+### InstantValues Interface
+```
+InstantValues
+├── Dictionary Interface
+│   ├── [key] ─────────────────► __getitem__
+│   ├── get(key, default) ────► get method
+│   ├── key in values ────────► __contains__
+│   └── [key] = value ────────► __setitem__ (async)
+├── Type Getters
+│   ├── get_sensors() ────────► dict[str, tuple]
+│   ├── get_binary_sensors() ─► dict[str, bool]
+│   ├── get_numbers() ────────► dict[str, tuple]
+│   ├── get_switches() ───────► dict[str, bool]
+│   └── get_selects() ────────► dict[str, int]
+└── Type Setters (async)
+    ├── set_number(key, value) ──► bool
+    ├── set_switch(key, value) ──► bool
+    └── set_select(key, value) ──► bool
+```
 
 #### Dictionary Interface
 ```python
@@ -321,8 +432,31 @@ client = PooldoseClient(
 status = await client.connect()
 ```
 
+### Security Model
+```
+Data Classification:
+┌─────────────────┐    ┌─────────────────┐    ┌─────────────────┐
+│   Public Data   │    │ Sensitive Data  │    │  Never Exposed  │
+├─────────────────┤    ├─────────────────┤    ├─────────────────┤
+│ • Device Name   │    │ • WiFi Password │    │ • Admin Creds   │
+│ • Model ID      │    │ • AP Password   │    │ • Internal Keys │
+│ • Serial Number │    │                 │    │                 │
+│ • Sensor Values │    │                 │    │                 │
+│ • IP Address    │    │                 │    │                 │
+│ • MAC Address   │    │                 │    │                 │
+└─────────────────┘    └─────────────────┘    └─────────────────┘
+        │                       │                       │
+        ▼                       ▼                       ▼
+  Always Included      include_sensitive_data=True    Never Included
+```
+
 ## Changelog
 
+### [0.4.1] - 2025-07-17
+- **BREAKING**: Moved all RequestStatus into client module - import from `pooldose.client` instead of `pooldose.request_handler`
+- Moved all connect checks into client (incl. API Version check) to avoid public access to requesthandler
+- Clean up code and improved encapsulation
+
 ### [0.4.0] - 2025-07-11
 - **BREAKING**: Removed `create()` factory method
 - **BREAKING**: Changed client initialization pattern to separate `__init__` and async `connect()` methods

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/src/python_pooldose.egg-info/SOURCES.txt

@@ -4,6 +4,7 @@ pyproject.toml
 src/pooldose/__init__.py
 src/pooldose/client.py
 src/pooldose/request_handler.py
+src/pooldose/request_status.py
 src/pooldose/mappings/__init__.py
 src/pooldose/mappings/mapping_info.py
 src/pooldose/mappings/model_PDPR1H1HAW100_FW539187.json

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/tests/test_client.py

@@ -1,8 +1,11 @@
 """Tests for Client for Async API client for SEKO Pooldose."""
 
+from unittest.mock import Mock
+
 import pytest
-from pooldose.client import PooldoseClient
-from pooldose.request_handler import RequestStatus
+
+from pooldose.client import PooldoseClient, RequestStatus
+from pooldose.request_handler import RequestHandler
 from pooldose.mappings.mapping_info import MappingInfo
 
 @pytest.mark.asyncio
@@ -39,3 +42,25 @@ async def test_get_model_mapping_file_not_found():
     mapping_info = await MappingInfo.load("DOESNOTEXIST", "000000")
     assert mapping_info.status != RequestStatus.SUCCESS
     assert mapping_info.mapping is None
+
+@pytest.mark.asyncio
+async def test_check_apiversion_supported():
+    """Test API version check logic."""
+    client = PooldoseClient("localhost")
+
+    # Mock the request handler instead of trying to connect
+    mock_handler = Mock(spec=RequestHandler)
+    # pylint: disable=protected-access
+    client._request_handler = mock_handler
+
+    # Test supported API version
+    mock_handler.api_version = "v1/"
+    assert client.check_apiversion_supported()[0] == RequestStatus.SUCCESS
+
+    # Test unsupported API version
+    mock_handler.api_version = "v2/"
+    assert client.check_apiversion_supported()[0] == RequestStatus.API_VERSION_UNSUPPORTED
+
+    # Test missing API version
+    mock_handler.api_version = None
+    assert client.check_apiversion_supported()[0] == RequestStatus.NO_DATA

{python_pooldose-0.4.0 → python_pooldose-0.4.1}/tests/test_request_handler.py

@@ -12,14 +12,3 @@ async def test_host_unreachable(monkeypatch):
     handler = RequestHandler("256.256.256.256", timeout=1)
     status = await handler.connect()
     assert status == RequestStatus.HOST_UNREACHABLE
-
-@pytest.mark.asyncio
-async def test_check_apiversion_supported():
-    """Test API version check logic."""
-    handler = RequestHandler("localhost")
-    handler.api_version = "v1/"
-    assert handler.check_apiversion_supported()[0] == RequestStatus.SUCCESS
-    handler.api_version = "v2/"
-    assert handler.check_apiversion_supported()[0] == RequestStatus.API_VERSION_UNSUPPORTED
-    handler.api_version = None
-    assert handler.check_apiversion_supported()[0] == RequestStatus.NO_DATA