Cloud sync: health monitoring, probe interval, UI polish#35
Merged
Conversation
… provider name resolution Core: - HttpStrategy health write semantics (http_status vs top-level status) - BlockSync worker WS-aware polling to reduce monitoring RU burn - Minimum 10s probe interval floor in ProbeCoordinator - Provider name stored on catalog refs for efficient lookup - Profile not found returns 404 (was incorrectly 503) Dashboard: - Distribution card shows provider names instead of raw IDs - Optimistic lag smooths block height display (prevents visual sawtooth) - Effective lag uses optimistic lag when available - Flash component redesigned as compact bottom-right toast - Region selector uses formatted region names - Network legend layout polish Fixes: - Delete dead round_robin.ex (referenced deleted ProviderPool) - Fix round-robin strategy mapping to :load_balanced in RPC socket Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove account_id from RequestOptions, RequestPipeline, Observability, and RoutingDecision - Remove round_robin alias from strategy registry and validation - Fixes all 34 integration test failures (KeyError: account_id not in RequestContext) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
Synced from lasso-cloud commits
b72a0c7..bec376d.Core improvements:
http_statusseparately from top-levelstatusto prevent oscillationDashboard/UI:
Fixes:
round_robin.ex(referenced deleted ProviderPool, caused compile failure):load_balancedin RPC socketSkipped (diverged)
dashboard.ex— 55 proprietary markers, fundamentally different event handling architectureVerification
--warnings-as-errorsmix formatclean🤖 Generated with Claude Code