Fix overly broad INVALID_FIELD_VALUE mapping, redundant token refresh, and missing cache

- Narrow INVALID_FIELD_VALUE error mapping to require "Unable to determine
  action group setup" message substring, preventing unrelated validation
  errors from raising ActionGroupSetupNotFoundError
- Remove redundant _refresh_token_if_expired() calls in fetch_events and
  unregister_event_listener since _post() already handles this
- Add lru_cache to camelize_key for parity with _decamelize_key

Author: iMicknl

pyoverkiz/_case.py

@@ -31,6 +31,7 @@ def decamelize(data: Any) -> Any:
     return recursive_key_map(data, _decamelize_key)
 
 
+@functools.lru_cache(maxsize=1024)
 def camelize_key(key: str) -> str:
     """Convert a single snake_case key to camelCase."""
     parts = key.split("_")

pyoverkiz/client.py

@@ -458,7 +458,6 @@ async def fetch_events(self) -> list[Event]:
         Per-session rate-limit : 1 calls per 1 SECONDS period for this particular
         operation (polling).
         """
-        await self._refresh_token_if_expired()
         response = await self._post(f"events/{self.event_listener_id}/fetch")
         return converter.structure(decamelize(response), list[Event])
 
@@ -467,7 +466,6 @@ async def unregister_event_listener(self) -> None:
 
         API response status is always 200, even on unknown listener ids.
         """
-        await self._refresh_token_if_expired()
         await self._post(f"events/{self.event_listener_id}/unregister")
         self.event_listener_id = None
 

pyoverkiz/response_handler.py

@@ -46,7 +46,11 @@
 _ERROR_CODE_MESSAGE_MAP: list[tuple[str, str | None, type[BaseOverkizError]]] = [
     # --- errorCode is the sole discriminator ---
     ("DUPLICATE_FIELD_OR_VALUE", None, DuplicateActionOnDeviceError),
-    ("INVALID_FIELD_VALUE", None, ActionGroupSetupNotFoundError),
+    (
+        "INVALID_FIELD_VALUE",
+        "Unable to determine action group setup",
+        ActionGroupSetupNotFoundError,
+    ),
     ("INVALID_API_CALL", None, NoSuchResourceError),
     ("EXEC_QUEUE_FULL", None, ExecutionQueueFullError),
     ("NO_SUCH_DEVICE", None, NoSuchDeviceError),