fix(ble): send gateway DIN in add_managed_charging_site (#154)

* feat(ble): send gateway DIN in add_managed_charging_site

Bump tesla-protocol to 2.2.0 for SiteController.din, the field the
vehicle uses to match a registered managed-charging site to its
gateway. add_managed_charging_site() previously sent an empty
SiteController, so any site it registered never matched.

* docs(agents): stop hardcoding HA's protobuf pin

The pinned version drifts (HA moved 6.32.0 -> 7.36.0); point at the
authoritative file instead of a stale number.

* no-mistakes(document): docs(agents): fix stale add_managed_charging_site API description

Author: Bre77

AGENTS.md

@@ -204,22 +204,24 @@ shims, not generated code. To pick up new message definitions, bump the floor in
 `pyproject.toml` — there is no local regeneration step.
 
 **Runtime-version pin (Home Assistant compatibility).** protobuf refuses to load
-gencode stamped *newer* than the installed runtime. Home Assistant core pins
-`protobuf==6.32.0`, so any `tesla-protocol` version depended on must stamp
-gencode **≤ 6.32.0** and declare a `protobuf` requirement compatible with
-`==6.32.0` — check both before bumping. Keep `pyproject.toml`'s `protobuf` floor
-in sync with what `tesla-protocol` requires, and keep the `tesla-protocol` floor
-at `>=1.4.0` (earlier `.pyi` imports fail strict pyright; 1.4.0 is also the first
-release allowing protobuf 7).
+gencode stamped *newer* than the installed runtime. Check Home Assistant core's
+current `protobuf` pin (`homeassistant/package_constraints.txt`) before bumping
+`tesla-protocol` — any version depended on must stamp gencode at or below that
+pin and declare a `protobuf` requirement compatible with it. Keep
+`pyproject.toml`'s `protobuf` floor in sync with what `tesla-protocol` requires.
 
 Command coverage is locked by `tests/test_proto_coverage_lock.py`, which fails if
 any `VehicleAction`/`GetVehicleData` field has no wrapper (`commands.py`) or
 reader (`bluetooth.py`) and is not allowlisted with a reason — keep that test in
 sync with a `tesla-protocol` bump rather than special-casing new fields. Naming:
 `legacy_vehicle_state()` (`bluetooth.py`) reads CarServer's `GetVehicleState`
 sub-state; `vehicle_state()` is the VCSEC `VehicleStatus`, a different
-message/domain. `set_rate_tariff`/`add_managed_charging_site` take
-`tesla_protocol` message types directly rather than a parallel flattened API.
+message/domain. `set_rate_tariff` takes `tesla_protocol` message types directly
+rather than a parallel flattened API. `add_managed_charging_site` takes a
+flattened `(public_key, din, lat, lon)` API; `public_key` must already be
+converted from DER to a raw EC point (the Fleet API route does that
+conversion server-side, this one does not), and `din` is the gateway's DIN
+the vehicle uses to match the registration to the site controller.
 
 ## Code Style
 

CLAUDE.md

@@ -0,0 +1,2 @@
+<!-- Points Claude at AGENTS.md via import; edit AGENTS.md, not this file. -->
+@AGENTS.md

CLAUDE.md

@@ -4,7 +4,7 @@ requires = ["setuptools>=77.0"]
 
 [project]
 name = "tesla_fleet_api"
-version = "1.14.1"
+version = "1.15.0"
 license = "Apache-2.0"
 description = "Tesla Fleet API library for Python"
 readme = "README.md"
@@ -21,8 +21,8 @@ dependencies = [
     "aiofiles>=24",
     "aiolimiter>=1",
     "cryptography>=43",
-    "protobuf>=6.32.0",
-    "tesla-protocol>=1.4.0",
+    "protobuf>=6.33.5",
+    "tesla-protocol>=2.2.0",
     "bleak>=0.22",
     "bleak-retry-connector>=3.9",
 ]

pyproject.toml

@@ -1,7 +1,7 @@
 """Tesla Fleet API"""
 
 __author__ = "hello@teslemetry.com"
-__version__ = "1.14.1"
+__version__ = "1.15.0"
 
 from tesla_fleet_api.const import Region, is_valid_region
 from tesla_fleet_api.funnel import (

tesla_fleet_api/__init__.py

@@ -2992,16 +2992,25 @@ async def get_rate_tariff(self) -> dict[str, Any]:
         )
 
     async def add_managed_charging_site(
-        self, public_key: str, lat: float, lon: float
+        self, public_key: str, din: str, lat: float, lon: float
     ) -> dict[str, Any]:
-        """Registers a managed charging site (utility managed-charging program) for this vehicle."""
+        """Registers a managed charging site (utility managed-charging program) for this vehicle.
+
+        `public_key` is the raw EC point string the vehicle expects, not the DER
+        the gateway's `get_signed_commands_public_key` returns; convert DER to a
+        raw EC point before calling this (the Fleet API's equivalent route does
+        this conversion server-side). `din` is the gateway's DIN, which the
+        vehicle uses to match this registration to the site controller.
+        """
         return await self._sendInfotainment(
             Action(
                 vehicleAction=VehicleAction(
                     addManagedChargingSiteRequest=AddManagedChargingSiteRequest(
                         site=ManagedChargingSite(
                             public_key=public_key,
-                            manager_type=ManagerType(site_controller=SiteController()),
+                            manager_type=ManagerType(
+                                site_controller=SiteController(din=din)
+                            ),
                             lat_lon=LatLong(latitude=lat, longitude=lon),
                         )
                     )

tesla_fleet_api/tesla/vehicle/commands.py

@@ -63,16 +63,21 @@ async def test_sends_get_rate_tariff_request(self) -> None:
 
 
 class AddManagedChargingSiteTests(MockedBleTransportTestCase):
-    async def test_sends_public_key_and_coordinates(self) -> None:
+    async def test_sends_public_key_din_and_coordinates(self) -> None:
         vehicle, send = self.make_vehicle()
         send.return_value = infotainment_action_ok_reply()
 
-        await vehicle.add_managed_charging_site("pubkey-bytes", 37.3230, -122.0322)
+        await vehicle.add_managed_charging_site(
+            "pubkey-bytes", "1232100-00-E-1234567890AB", 37.3230, -122.0322
+        )
 
         vehicle_action = _decode_vehicle_action(vehicle, send.await_args.args[0])
         site = vehicle_action.addManagedChargingSiteRequest.site
         self.assertEqual(site.public_key, "pubkey-bytes")
         self.assertTrue(site.manager_type.HasField("site_controller"))
+        self.assertEqual(
+            site.manager_type.site_controller.din, "1232100-00-E-1234567890AB"
+        )
         # LatLong lat/lon are 32-bit floats, so compare at reduced precision.
         self.assertAlmostEqual(site.lat_lon.latitude, 37.3230, places=4)
         self.assertAlmostEqual(site.lat_lon.longitude, -122.0322, places=4)