Conversation
Move the device-drivers pin from 92adaf0 to eeb3e06 and bundle teslamate_vehicle, tesla_cloud and vag_vehicle. Nothing else under drivers/lua changed between the two commits, so no bundled driver moves. fleetping's list of shipped driver IDs is regenerated from the pin. Refs #1442 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the device-drivers pin from
92adaf0toeeb3e06and adds three read-only vehicle drivers to the bundle:teslamate_vehicle,tesla_cloudandvag_vehicle. Refs #1442: they read vehicle SoC through vendor drivers that emitDerVehicle, with no Home Assistant ingest in Core.Between the two pins nothing else under
drivers/luachanged, so no bundled driver moves.Owner decision
The changeset is
minor, which the owner approves in the PR.Bundling is not needed to test. All three are in the signed
drivers-betachannel, and one site can install one without a new Core:Bundled, every box carries three drivers that no one has yet run against a car. They do nothing until configured.
The drivers
teslamate_vehicletesla_cloudvehicle_dataonly for an awake car, every 4 minutes while it charges and every 20 otherwise. The signed artifact may POST only to/oauth2/v3/token.vag_vehicleNone wakes a car or starts a charge. Settings has no form for them; they are set up in YAML, as each driver's header comment shows.
Evidence
scripts/sync-bundled-drivers.shwrote 43 drivers fromeeb3e06. The three new files are byte-identical to device-driversmain.go test ./internal/drivers: passes, includingTestBundledDriversMeetStartupContractfor all 43.make verify: vet, tests and build clean (71 packages). The first run failed only becausego/internal/fleetping/shipped_gen.go, the list of shipped driver IDs, is generated from the pin; it is regenerated here withgo generate ./internal/fleetping/....NewLuaDriverin FTW's own host (gopher-lua,json_decode,emit, telemetry store), with only HTTP and MQTT stubbed:tesla_cloudset make Tesla, the VIN, model "Model 3" from the VIN, and a 60 s poll interval. Its calls were one token POST, one state GET and onevehicle_dataGET.vag_vehicletook 55 ms per poll for 100 kB of JSON and 0.6 s for 1 MB on an Apple M-series core, and refused a 5 MB file. That run is what found the gopher-luaregistry overflowthat fix(vag_vehicle): unzip without overflowing gopher-lua's value stack device-drivers#142 fixes.Paired with srcfl/device-drivers#139, #140, #141 and #142, all merged.
🤖 Generated with Claude Code