Repository navigation
feat(scaling): first screen with ladder, ceilings, guardrails and progress - #196
Merged
Merged
Conversation
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
…g the numbers Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
This was referenced Oct 16, 2026
0x00101010
changed the base branch from
feat/scaling-ceilings-guardrails
to
feat/scaling-schema-v2
October 16, 2026 13:31
0x00101010
added this pull request to stack #202
October 16, 2026 13:36
Collaborator
✅ Heimdall Review Status
|
This was referenced Oct 16, 2026
mw2000
approved these changes
Oct 16, 2026
This branch was successfully deployed
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
Why
The scorecard's headline was four unrelated cards, so a reader could not see how far observed load sits from the operating point, the binding ceiling, and the configured limit, nor which of those numbers is measured rather than modeled. One axis puts them in proportion and tags each one.
What
Ladder: one 0–configured-limit axis in Mgas/s with rungs for observed p50/p99, operating point, qualified sustained, the binding ceiling fromderiveBinding(ceilings)(named by area, "not modeled" listing every area when none binds), and a grey "configured limit" tick that is never styled as capacity. Rungs without a value are listed beside the axis with their tag, never dropped.layoutRungs(pure, unit-tested): sorts rungs by value, alternates sides, drops a label to the next free tier on its side when it would overlap, and anchors labels near the ends so they stay inside the axis. Label width is measured, so 360 px and 1280 px both lay out without overlaps.Info: an inline-SVG "i" that opens a popover on hover, keyboard focus, and tap;aria-describedbyon the trigger; Escape and outside click close it.1 Mgas/s ≈ N payments/s ≈ M swaps/sfrom the snapshot constants, or "constants pending".Notes
Infouses@base-ui/react/popoverrather than the existingInfoTooltipbecause tooltips do not open on tap or stay open on an outside click.Screenshots
/scalingon the internal build with invented fixtures (none of these numbers describe a real network).Crowded fixture, 1280 px:
Crowded fixture, 360 px:
Info popover opened from the keyboard:
All-null fixture (every rung listed beside the axis):
Committed synthetic fixture, now past the 10-day staleness threshold:
Test plan
npm test,npm run typecheck,npm run lint,npm run build,npm run build:internal,npm run llms -- --checkInforeachable by Tab, popover opens on focus,aria-describedbyresolves, Escape closeslayoutRungsunit tests cover sparse, crowded, tier reuse, end anchoring, and clamping above the limitConsolidated
This PR now also carries #197 (ceiling cards, guardrails strip, shared
Tag) and #198 (progress from Linear, collapsed detail sections). Base is nowfeat/scaling-schema-v2(#194). Commits are unchanged; review them in order.