cleanup realtime state machine and support live references#202
Open
noahgsolomon wants to merge 3 commits intomainfrom
Open
cleanup realtime state machine and support live references#202noahgsolomon wants to merge 3 commits intomainfrom
noahgsolomon wants to merge 3 commits intomainfrom
Conversation
The active state had no handler for token expiry, so tokens were never refreshed. Replace the broken expireToken approach with a recurring background fetch that updates the token in-place via a new authenticated transition, keeping the WebSocket open. Use a generation counter to prevent stale callbacks from spawning parallel refresh loops. Also expire the token on connectionClosed and close so reconnects always re-authenticate instead of reusing a stale token.
interpreter reference Remove unreachable failed state and unused expireToken transition from idle. Store the real robot3 service in the interpreter cache instead of a spread copy so context and machine state are always live.
noahgsolomon
changed the title
noahgsolomon
changed the title
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
1 participant
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.
Remove unreachable
failedstate and unusedexpireTokentransition. Fix
reuseInterpreterto store the actual robot3service instead of a spread copy, so
contextandmachinereferences stay live across state transitions.