Skip to content

feat: add E2E benchmark design and statistics skills - #162

Closed
MajorLift wants to merge 2 commits into
MetaMask:mainfrom
MajorLift:add/e2e-benchmarks
Closed

MajorLift wants to merge 2 commits into
MetaMask:mainfrom
MajorLift:add/e2e-benchmarks

Conversation

@MajorLift

Copy link
Copy Markdown
Contributor

Motivation

Nothing in the repo says what a benchmark difference means. performance-testing runs the benchmark. How many rounds, how much variance, and whether a 4% gap is a result are all unanswered. Without that a benchmark reports noise as a win, which is worse than not running one, because it produces a number people cite.

Web Vitals numbers from a benchmark and from production routinely disagree, and the disagreement is a property of how each is collected rather than a regression.

Overview

Adds one skill and four knowledge files for E2E performance benchmarks. The profiling, render-delta and data-analysis skills are in #144.

  • benchmark-design covers session hygiene, reporting across rounds, and artifact grouping.

Knowledge: benchmark-statistical-hygiene (one primary analysis fixed before the runs, isolating the fix vector, the artifact sort-order trap), metrics-pipeline-design (the collector, runner, statistics and reporter layers), and two on Web Vitals: runtime metrics, and why production numbers disagree with benchmark numbers.

Showcase

No trial run yet. These are measurement instruments, and the honest evidence for one is a run that moved a decision. There is not one to cite.

How to design and read an extension E2E benchmark run, how to report a
result from several rounds, how per-iteration metrics flow from collector
to reporter, and how Web Vitals are collected in benchmarks versus in
production.
Where the browser delivers no `longtask` entries, the observer's counts
stay at their initial 0, so a zero reads the same whether no task ran long
or none could be seen.
@rvelaz
rvelaz marked this pull request as ready for review September 15, 2026 06:56
@MajorLift
MajorLift marked this pull request as draft September 15, 2026 20:02
@MajorLift MajorLift closed this by deleting the head repository Sep 15, 2026
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