Self-healing git status subscriptions - #100
Merged
Merged
Conversation
This reverts commit c36a7cd.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
github-actions
Bot
added
vouch:trusted
The transport announces every reconnect attempt via onResubscribe before any data arrives. Treating those announcements as recovery reset the first-snapshot watchdog and refilled the rebuild budget, so attempts arriving faster than the watchdog postponed it forever and the bounded rebuild cycle never converged. Only a delivered status ends the waiting phase now; attempt starts during one are inert. Also hardens the palette browser test to await the committed Sources view and an attached action before clicking, which was the actual CI failure signature all along.
Collaborator
Author
|
Landed on main as 2ec2b50 (linearized; the branch-tip SHA changed so GitHub can't auto-mark this merged). Includes the bounded-recovery guard and the palette test hardening. |
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.
Re-lands the self-healing status subscription stack reverted from main in c36a7cd.
What it does: watchdog rebuilds a silent stream (2x, 20s apart) before showing the stale notice; panel Retry rebuilds the stream as well as refreshing; the 5s local poll feeds a broken atom via the shared applyGitStatusStreamEvent reducer; rate-limited client.stream.* diagnostic spans land in server.trace.ndjson.
Why it is a draft: the ChatView full-app palette browser test ("selects an environment before browsing...") fails on CI machines with this stack while passing locally (repeated consecutive full-suite runs). Two churn sources were already eliminated (pending<->stale rebuild oscillation, poll-heal vs retry-stale alternation - both pinned by reference-stability regression tests) and the CI test still timed out three times. Root cause of the remaining CI-only instability is unidentified; this PR exists so CI can run the suite against iterations of this branch without holding main red.
Last green main baseline for comparison: abe76b8.