fix(teslemetry): drop raw parameter from find_authorized_clients

list_authorized_clients() already returns the raw response, so
find_authorized_clients() stays typed-only with no raw option.

Author: Bre77

docs/teslemetry.md

@@ -559,12 +559,11 @@ an unrecognized response shape raises
 never mistaken for "no authorized clients". The raw response is still
 available on `raw` for anything not modeled.
 
-`find_authorized_clients(raw=True)` skips the typed parse and returns the
-exact response `list_authorized_clients()` would, matching the call shape of
-a local gateway path that also wants to opt out of typing. The parsing itself
-lives in the module-level `tesla_fleet_api.teslemetry.energysite.parse_authorized_clients`
-function, which other callers (e.g. a local/LAN client using the same
-envelope shape) can reuse directly to get the same `AuthorizedClients` result.
+The parsing itself lives in the module-level
+`tesla_fleet_api.teslemetry.energysite.parse_authorized_clients` function,
+which other callers (e.g. a local/LAN client using the same envelope shape)
+can reuse directly to get the same `AuthorizedClients` result. For the
+unparsed response, use `list_authorized_clients()`.
 
 `remove_authorized_client(public_key)` accepts raw DER bytes or an already
 base64-encoded key string. Removal requires no physical presence proof, so any

tesla_fleet_api/teslemetry/energysite.py

@@ -8,7 +8,7 @@
 import warnings
 from collections.abc import Awaitable, Callable
 from dataclasses import dataclass
-from typing import Any, Literal, overload, cast
+from typing import Any, cast
 
 from tesla_fleet_api.const import (
     AuthorizationRole,
@@ -406,15 +406,7 @@ async def list_authorized_clients(self) -> dict[str, Any]:
             f"api/1/energy_sites/{self.energy_site_id}/command/authorized_clients",
         )
 
-    @overload
-    async def find_authorized_clients(
-        self, raw: Literal[False] = False
-    ) -> AuthorizedClients: ...
-    @overload
-    async def find_authorized_clients(self, raw: Literal[True]) -> dict[str, Any]: ...
-    async def find_authorized_clients(
-        self, raw: bool = False
-    ) -> AuthorizedClients | dict[str, Any]:
+    async def find_authorized_clients(self) -> AuthorizedClients:
         """List authorized clients on the energy gateway, parsed into a typed result.
 
         Prefer this over :meth:`list_authorized_clients` for consumers that
@@ -426,14 +418,9 @@ async def find_authorized_clients(
         treating either as "no clients". See :class:`AuthorizedClients` for
         the exact parsing semantics.
 
-        ``raw=True`` returns the unparsed response exactly as
-        :meth:`list_authorized_clients` does, skipping the typed parse - for
-        callers that want the same call shape aligned with the local
-        gateway path while still opting out of typing.
+        For the unparsed response, use :meth:`list_authorized_clients`.
         """
         response = await self.list_authorized_clients()
-        if raw:
-            return response
         return parse_authorized_clients(response)
 
     async def remove_authorized_client(self, public_key: bytes | str) -> dict[str, Any]:

tests/test_teslemetry_authorized_clients.py

@@ -96,20 +96,6 @@ async def test_private_alias_still_works_for_one_release(self) -> None:
 
 
 class GetAuthorizedClientsTests(IsolatedAsyncioTestCase):
-    async def test_raw_true_returns_unparsed_response(self) -> None:
-        payload = {
-            "response": {
-                "authorized_clients": [
-                    {"public_key": PUBLIC_KEY_B64, "state": 3},
-                ]
-            }
-        }
-        site = _make_site(payload)
-
-        result = await site.find_authorized_clients(raw=True)
-
-        self.assertEqual(result, payload)
-
     async def test_default_raw_false_behaviour_unchanged(self) -> None:
         payload = {
             "response": {