Skip to content

feat(scaling): detail pages and unified run index - #199

Merged
mw2000 merged 9 commits into
feat/scaling-ladderfrom
feat/scaling-guardrails-page
Oct 16, 2026
Merged

mw2000 merged 9 commits into
feat/scaling-ladderfrom
feat/scaling-guardrails-page

Conversation

@0x00101010

@0x00101010 0x00101010 commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Why

The guardrail strip on the first screen hides each guardrail's help behind a popover, and outcomes metrics had no permanent home. A guardrails page gives both a home. The first screen's snapshot loading and as-of header move into a shared helper so detail pages show the same freshness warning.

What

  • loadSnapshot.tsx: snapshot loading, the unconfigured/invalid states and the as-of/stale header (optional breadcrumb back to /scaling), used by /scaling and the new page.
  • /scaling/guardrails: every guardrail with value, threshold, tag, owner, status and inline help, then the outcomes section as a table (metric and note, value, status, window with range, query, source) plus its gaps.
  • First-screen guardrail heading becomes a Guardrails → link.
  • The validator rejects a sections[].id outside da, execution, state, networking, outcomes, so a section cannot exist without a page.
  • /scaling/guardrails is added to the CI check that internal-only routes 404 on the public build.

Notes

Screenshots

Internal build with the committed synthetic fixture (invented numbers, not a real network).

/scaling/guardrails at 1280 px:

guardrails page at 1280 px

/scaling/guardrails at 360 px (tables scroll horizontally inside their card):

guardrails page at 360 px

/scaling at 1280 px with the Guardrails → link:

first screen at 1280 px

Test plan

  • npm test, npm run typecheck, npm run lint, npm run build, npm run build:internal, npm run llms -- --check
  • page.test.tsx: guardrails page shows the breadcrumb, as-of header, each guardrail's help and the outcomes row with query and window; first screen links to it.
  • snapshot.test.ts: an unknown section id fails validation.
  • Rendered /scaling and /scaling/guardrails at 360 px and 1280 px with the synthetic fixture; external dev server returns 404 for both.

Consolidated

This PR now also carries #200 (per-area detail pages) and #201 (unified /scaling/runs index; run data layer moved from app/benchmark). Base is now feat/scaling-ladder (#196). Commits are unchanged; review them in order. Stack: #194 → #196 → #199.

0x00101010 and others added 3 commits October 16, 2026 05:17
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@vercel

vercel Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Oct 16, 2026 1:32pm UTC

Request Review

0x00101010 and others added 2 commits October 16, 2026 05:40
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
0x00101010 and others added 4 commits October 16, 2026 05:49
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
Co-authored-by: Toshi <toshi-noreply@coinbase.com>
@0x00101010
0x00101010 changed the base branch from feat/scaling-progress to feat/scaling-ladder October 16, 2026 13:31
@0x00101010 0x00101010 changed the title feat(scaling): shared snapshot header and guardrails page feat(scaling): detail pages and unified run index Oct 16, 2026
@0x00101010
0x00101010 added this pull request to stack #202 October 16, 2026 13:36
@cb-heimdall

cb-heimdall commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@mw2000
mw2000 added this pull request to the merge queue Oct 16, 2026
Merged via the queue into main with commit b349a0b Oct 16, 2026
19 checks passed
@mw2000
mw2000 deleted the feat/scaling-guardrails-page branch October 16, 2026 16:55

This branch was successfully deployed

1 active deployment
Preview — 26ba767b Deployed Oct 16, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants