Skip to content

feat(scaling): unified run index - #201

Merged
0x00101010 merged 4 commits into
feat/scaling-guardrails-pagefrom
feat/scaling-runs-index
Oct 16, 2026
Merged

0x00101010 merged 4 commits into
feat/scaling-guardrails-pagefrom
feat/scaling-runs-index

Conversation

@0x00101010

@0x00101010 0x00101010 commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Why

Base-bench runs and load tests live on separate /benchmark pages with different columns, and the load-test list shows only dates, so comparing nights or setting a load test beside a bench run means opening each one. /scaling/runs puts both in one table with the same columns, and says why no run counts as sustained-capacity evidence yet.

What

  • Move the report-api data layer (dataService, types, metricDefinitions, formatters, formatTransactions) from app/benchmark to app/scaling/runs/lib unchanged; app/benchmark imports it from there.
  • toUnifiedRun maps a bench run or a load test to one row: env, machine, duration, profile (sustained at ≥ 1 h, else burst), workload, gas/s, latency, status, areas, and qualifies: { ok: false, reason }. State is unknown until producers record it; bench duration is null (shown as —) because metadata has no block count.
  • /scaling/runs: newest first, grouped by UTC week, with when / kind / env / profile / workload / Mgas/s / latency / status / duration / machine, and filter chips for kind, env and qualifying-only. Rows link to /scaling/runs/bench/<id> and /scaling/runs/load/<network>/<timestamp>, which do not exist yet and 404 for now.
  • A "Runs →" link in the first-screen header; /scaling/runs added to the CI check that internal-only routes 404 on the public build.

Notes

Screenshots

Internal build against a local server serving the committed synthetic fixture (invented numbers, not a real network).

/scaling/runs at 1280 px:

run index at 1280 px

/scaling header with the Runs link:

first screen header

Test plan

  • npm test, npm run typecheck, npm run lint, npm run build, npm run build:internal, npm run llms -- --check, npm run agents -- --check; the public build serves 404 for /scaling/runs.
  • unified.test.ts: bench and load mapping from the synthetic fixture, including an incomplete bench run, a load test without config, a fatal load test, a missing result, and week boundaries.
  • runs/page.test.tsx: both kinds ordered newest first under week headings with detail links; the data-source error is shown.
  • Rendered /scaling/runs in npm run dev:internal against the live report-api and against the fixture; kind, env and qualifying-only chips filter as expected.

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>
@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:00pm UTC

Request Review

Base automatically changed from feat/scaling-area-pages to feat/scaling-guardrails-page October 16, 2026 13:31
@0x00101010
0x00101010 merged commit 26ba767 into feat/scaling-guardrails-page Oct 16, 2026
10 checks passed
@0x00101010
0x00101010 deleted the feat/scaling-runs-index branch October 16, 2026 13:31
@0x00101010

Copy link
Copy Markdown
Contributor Author

GitHub marked this merged because its commits were fast-forwarded into the branch of #199 (consolidating 8 PRs into 3). Nothing reached main; review continues on #199.

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.

1 participant