Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance benchmark-web-vitals command with additional metrics #41

Draft
wants to merge 3 commits into
base: add/benchmark-web-vitals
Choose a base branch
from

Conversation

felixarntz
Copy link
Collaborator

@felixarntz felixarntz commented Feb 17, 2023

Follow up to #40: The initial version of the benchmark-web-vitals command only comes with load time metrics, since other metrics are a bit more complicated to capture in a headless browser request, e.g. CLS and INP require interactions to happen and capture events throughout the page lifecycle, not just load time related.

This is an early draft intended for iteration in the future.

You can already test this branch, but you will likely notice that you don't get data for CLS and INP for the reason outlined above (and each request will take ~30s as that it when Puppeteer will time out by default). Before this PR can be completed and merged, we will have to figure out how to trigger those metrics to be measured and exposed using the https://github.com/GoogleChrome/web-vitals library that is being used.

@felixarntz
Copy link
Collaborator Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant