Repository navigation
fix: re-read Keychain on limit refresh and toggle the floating pet from the footer - #302
Merged
chattymin merged 1 commit intoSep 18, 2026
Conversation
…om the footer A still-valid cached Claude token hid an in-place account switch from the Refresh button, because only a 401 cleared the cache. User refresh now skips that cache. The popover footer can show the pet again after Hide, using the same floatingPetEnabled flag as Settings.
Owner
|
Thank you for your contribution! |
8 of 10 tasks
6 of 10 tasks
donky-ey
pushed a commit
to donky-ey/PokeDexBar
that referenced
this pull request
Oct 15, 2026
…che (upstream chattymin#302) A still-valid cached Claude token hid an in-place account switch from the Refresh button, because only a 401 cleared the cache. User refresh now passes bypassCache; the automatic poll keeps the cache and stays out of the Keychain (asserted via the KeychainReader counter). The popover footer can also show the floating pet again after Hide, using the same floatingPetEnabled flag as Settings.
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.
Summary
/logout→/loginto another email on the same Team, that token stays valid, so Keychain was never opened and the official 5h/weekly bars stayed on the previous account. The user-action path now passesbypassCache: true. Auto-poll still does not touch Keychain. The same one-line rule is applied to Antigravity's user refresh, which had the same still-valid-cache hole.floatingPetEnabledflag, on every tab, with the existing Show / Hide strings.Type of change
UI changes
Checklist
swift buildandswift testpass locallyCloses #300
Closes #301