Skip to content

fix: use protojson for CLI output serialization#37

Merged
odsod merged 1 commit intomainfrom
fix-protojson-cli
Apr 23, 2026
Merged

fix: use protojson for CLI output serialization#37
odsod merged 1 commit intomainfrom
fix-protojson-cli

Conversation

@odsod
Copy link
Copy Markdown
Member

@odsod odsod commented Apr 23, 2026

Summary

  • Replace printJSON (which used encoding/json) with protojson.Format at each call site
  • encoding/json serializes internal proto struct fields (XXX_raceDetectHookData, XXX_presence) instead of actual message data on edition 2023 opaque API protos
  • Fixes way connect mercedes vehicle-services showing empty output instead of service status data

Test plan

  • go build ./... passes
  • go test ./... passes
  • Verified vehicle-compatibility and vehicle-specification already used protojson.Format correctly

@odsod odsod merged commit 2c23d6a into main Apr 23, 2026
1 check passed
@odsod odsod deleted the fix-protojson-cli branch April 23, 2026 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant