Repository navigation
Conversation
wgordon17
force-pushed
the
feat/site-modernization-pr3
branch
from
October 30, 2026 21:37
88661c7 to
e70016b
Compare
- Treat any error from a poll or hot-poll cycle started under a previous identity as stale: never touch loading, expire the token, or surface errors - Discard the stored snapshot when a write exceeds the item cap and clear pending snapshot writes on cross-tab identity switches - Remove test-mock-shaped production code; require epoch callback params; add SNAPSHOT_VERSION, Privacy page snapshot disclosure, and tests
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
DashboardPageto keyedreconcile(..., { key: "id", merge: true })and key repository and workflow group loops acrossIssuesTab,PullRequestsTab, andActionsTabso background refreshes preserve existing DOM nodes without scroll-lock workarounds.localStorageto a dedicatedgithub-tracker-snapshotIndexedDB store with immediate/debounced writes (hot-poll cycles persist too, ~2–4 ms of structured clone per cycle at typical sizes, so reloads show current check statuses), a 5,000-item cap that discards the stored record, eager clearing inauth.ts,identityEpoch()/StaleIdentityErrorguards across all polling and mutation paths, and a Privacy page entry. The snapshot is cleared when a token expires, so signing back in as the same user after a 401 starts without it — unchanged from before, sinceexpireTokenalready removed thelocalStoragecopy — and theexpireTokendocstring now says so./_app/nonexistent.jsand metadata paths and sync the verbatim Cloudflare WAF rules inDEPLOY.md.