command: model VehicleData field 18 as an opaque bytes payload#16
Open
Bre77 wants to merge 1 commit into
Open
Conversation
Live BLE vehicleDataSubscription pushes carry a VehicleData field 18 (~137 bytes) that the currently-pinned schema doesn't define. The raw bytes weren't preserved for a field-level decode, so it's added as an undecoded bytes field rather than guessing at names/types.
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.
Intent
vehicleDataSubscriptionpushes carry aVehicleDatafield 18 (~137 bytes) that this package'sVehicleDatamessage doesn't define at all - a real proto-version gap, confirmed from our own observations and contributions from the community.bytes unknown = 18;, matching the existingunknownnaming convention already used elsewhere in this file for undocumented payloads. Wire-compatible: any consumer that already skips unknown fields is unaffected, and this just gives it a name and round-trips the raw bytes for anyone who wants to inspect them.buf lint,buf breaking, and the upstream coverage gate are all green.