Skip to content

Conversation

@kmiller68
Copy link
Contributor

This is somewhat handy when checking if network requests happen during the benchmark run itself. I clear the screen then click start and if any resources are fetched logging will be printed to the terminal session. It doesn't catch network requests but those are more obvious during development.

For posterity, after a bit of debugging I realized that you have to pass options as npm run server -- --verbose rather than npm run server --verbose

This is somewhat handy when checking if network requests happen during
the benchmark run itself. I clear the screen then click start and if any
resources are fetched logging will be printed to the terminal session.
It doesn't catch network requests but those are more obvious during
development.

For posterity, after a bit of debugging I realized that you have to pass
options as `npm run server -- --verbose` rather than
`npm run server --verbose`
@netlify
Copy link

netlify bot commented Dec 19, 2025

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 8c52642
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/695d26ea1028aa00094c1c0b
😎 Deploy Preview https://deploy-preview-249--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
Copy link
Contributor

I would not be opposed to enable this by default and have a --quiet option instead.

@camillobruni
Copy link
Contributor

Looks like a default value for port is missing?

Error: Error: Port is required
    at serve (file:///home/runner/work/JetStream/JetStream/tests/server.mjs:39:15)
    at file:///home/runner/work/JetStream/JetStream/tests/run-browser.mjs:145:22
    at ModuleJob.run (node:internal/modules/esm/module_job:413:25)
    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:660:26)
    at async asyncRunEntryPointWithESMLoader (node:internal/modules/run_main:101:5)

@kmiller68
Copy link
Contributor Author

Looks like a default value for port is missing?

Error: Error: Port is required
    at serve (file:///home/runner/work/JetStream/JetStream/tests/server.mjs:39:15)
    at file:///home/runner/work/JetStream/JetStream/tests/run-browser.mjs:145:22
    at ModuleJob.run (node:internal/modules/esm/module_job:413:25)
    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:660:26)
    at async asyncRunEntryPointWithESMLoader (node:internal/modules/run_main:101:5)

Weird, I don't see that, nor did I change it AFAIK... How are you running the server? npm run server?

@camillobruni
Copy link
Contributor

@kmiller68
Copy link
Contributor Author

This was from the CI: https://github.com/WebKit/JetStream/actions/runs/20381233883/job/58571960463?pr=249

Oh I see, serve is used by tests/run-browser.mjs too. I'll fix

@kmiller68 kmiller68 merged commit 181f3cc into WebKit:main Jan 6, 2026
23 checks passed
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