Skip to content

Add startup benchmark helper #158

Merged
camillobruni merged 8 commits into
WebKit:mainfrom
camillobruni:2025-08-20_startup_helpers
Aug 26, 2025
Merged

Add startup benchmark helper #158
camillobruni merged 8 commits into
WebKit:mainfrom
camillobruni:2025-08-20_startup_helpers

Conversation

@camillobruni
Copy link
Copy Markdown
Contributor

  • Add startup-helper folder:
    • StartupBenchmark handles source code creation
    • BabelCacheBuster.js is a babel plugin that injects a known comment into every function body
  • Add unit tests for StartupBenchmark

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 20, 2025

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 38011f2
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/68ad73c7a90f480008125e2a
😎 Deploy Preview https://deploy-preview-158--webkit-jetstream-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@camillobruni camillobruni requested a review from danleh August 20, 2025 13:14
Copy link
Copy Markdown
Contributor

@danleh danleh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with nits. Also is there a usage example besides the test or link to the cache busting / how it's currently done without the helper?

Comment thread startup-helper/StartupBenchmark.js Outdated
Comment thread startup-helper/StartupBenchmark.js
Comment thread startup-helper/StartupBenchmark.js Outdated
@camillobruni
Copy link
Copy Markdown
Contributor Author

Currently the only cache busting we do is in code-load where we append a random number as a string to the eval'd sources. Something that can be easily circumvented.

@camillobruni camillobruni merged commit e05443c into WebKit:main Aug 26, 2025
10 checks passed
@camillobruni camillobruni deleted the 2025-08-20_startup_helpers branch August 26, 2025 11:05
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.

2 participants