Repository navigation
Conversation
ChargerCurrentUserUuid and CompletedSession are never pushed over the live stream, so they only change when the charger is polled -- 10 minutes apart while idle. A short authorized session can pass between two polls and never show up at all. ChargerOperationMode (710) and SessionIdentifier (721) do stream, and change on the same boundaries where those two are written. Trigger the existing coordinator poll sequence when either arrives. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Collaborator
|
Please resolve the conflicts in this PR. Thanks. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
sveinse
approved these changes
Oct 6, 2026
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.
ChargerCurrentUserUuid(722) andCompletedSession(723) are never pushedover the live stream, so they only change on a poll — 10 minutes apart while
the charger is idle. A short authorized session can fall between two polls and
never show up at all. The sensors from #431 read 722 directly.
ChargerOperationMode(710) andSessionIdentifier(721) do stream, andchange on the same session boundaries.
stream_callbacknow runs the existingcoordinator poll sequence (
[2, 7, 15]s) when either arrives.trigger_pollcancels an in-flight sequence, so a burst of frames collapses into one.
Any 710 value triggers, not only Disconnected: a plug-in is where authorization
becomes relevant, and the extra polls on charging transitions are a handful per
session against the 60 s charging interval.
Verified on a Go 2 (fw 3.3.0.0), unplug then plug back in:
An app authorization later in the same session was visible on 722 about 2 s
after its 721 frame. Without this, both wait for the next 10-minute poll; a 25 s
session on 2026-09-09 was missed entirely.
Clearing 722 at session end still relies on #425 — this only makes it appear
sooner.
#418 also adds
tests/test_manager.py; the two files are disjoint, whoevermerges second concatenates.
Fixes #432
🤖 Generated with Claude Code