Reduce `@cloudflare/vitest-pool-workers` log noise by sheplu · Pull Request #13007 · cloudflare/workers-sdk

sheplu · 2026-03-22T10:07:25Z

Fixes #12014

Add a new parameter to allow logLevel threshold on the vitest-pool-workers allowing to hide debug messages like vpw:debug

I don't think the failling CI is due to the change I did

Tests
- Tests included/updated
- Automated tests not possible - manual testing has been completed as follows:
- Additional testing not necessary because:
Public documentation
- Cloudflare docs PR(s): doc: add logLevel for vitest-pool-worker cloudflare-docs#29210
- Documentation not necessary because:

A picture of a cute animal (not mandatory, but encouraged)

changeset-bot · 2026-03-22T10:07:32Z

🦋 Changeset detected

Latest commit: 27cf551

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

workers-devprod · 2026-03-22T10:07:46Z

Codeowners approval required for this PR:

@cloudflare/wrangler

Show detailed file reviewers

packages/vitest-pool-workers/src/pool/index.ts: [@cloudflare/wrangler]

devin-ai-integration

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

pkg-pr-new · 2026-03-22T13:44:59Z

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@13007

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@13007

miniflare

npm i https://pkg.pr.new/miniflare@13007

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@13007

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@13007

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@13007

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@13007

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@13007

wrangler

npm i https://pkg.pr.new/wrangler@13007

commit: 27cf551

stale

penalosa · 2026-03-26T23:04:05Z

Instead of adding a configurable logLevel option, I've updated this PR to take a simpler approach:

Default log level changed from VERBOSE to WARN — only actionable warnings and errors are printed. This removes the [vpw:debug] and [vpw:info] noise from every test run.
Debug output moved behind NODE_DEBUG — set NODE_DEBUG=vitest-pool-workers to get the detailed messages back when troubleshooting.
"Removing compat flag" promoted to warn — this message fires when we override a user's explicit flag, so it should be visible.

sheplu · 2026-03-26T23:06:16Z

That's great! I wasn't sure if it was better to provide a configuration on userland or force a loglevel to a higher level!

Both looks great to me but maybe some people would prefer to have something configurable?
In my side, hissing everything bellow a warn lever is ok!

mrmckeb · 2026-03-27T05:32:23Z

I'm also seeing info logs, which I'm not sure add a lot of value... I feel like, by default, I should only see warning/error logs in output. What do you think @sheplu?

[vpw:debug] Adding `enable_nodejs_process_v2` compatibility flag during tests as this feature is needed to support the Vitest runner.
[vpw:info] Starting runtime for ...
[vpw:info] Starting runtime for ...
[vpw:info] Starting runtime for ...
[vpw:info] Starting runtime for ...

sheplu · 2026-03-27T08:56:05Z

@mrmckeb When I did the PR I was not sure to which level should be the default, I did not want to move from debug to warn - but I would agree that the info logs are not that useful. The only information would be to now that it started properly, which trigger a lot of noise if you do have a lot of tests. So I think warn by default may be a good call!

sheplu · 2026-04-03T12:19:37Z

@penalosa any news regarding this PR?

…t-pool-workers

…example

…kers Instead of adding a new config option, reduce the default log level from VERBOSE to WARN and move debug/info messages behind util.debuglog(). - Default log level is now WARN (only actionable messages print) - Debug output available via NODE_DEBUG=vitest-pool-workers - Promote 'Removing compat flag' to warn (overrides user's explicit config) - Remove logLevel schema field, setPoolLogLevel, and associated test

…rop vpw: prefix

workers-devprod · 2026-04-19T09:12:04Z

Codeowners approval required for this PR:

✅ @cloudflare/wrangler

Show detailed file reviewers

workers-devprod

Codeowners reviews satisfied

sheplu requested a review from a team as a code owner March 22, 2026 10:07

sheplu mentioned this pull request Mar 22, 2026

doc: add logLevel for vitest-pool-worker cloudflare/cloudflare-docs#29210

Closed

4 tasks

devin-ai-integration Bot reviewed Mar 22, 2026

View reviewed changes

workers-devprod added this to workers-sdk Mar 22, 2026

github-project-automation Bot moved this to Untriaged in workers-sdk Mar 22, 2026

penalosa force-pushed the allow-loglevel branch from d55253d to c8de3be Compare March 26, 2026 18:47

penalosa previously approved these changes Mar 26, 2026

View reviewed changes

github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Mar 26, 2026

penalosa changed the title ~~feat: allow user to configure logLevel on vitest-pool-workers~~ Reduce @cloudflare/vitest-pool-workers log noise Mar 26, 2026

sheplu and others added 4 commits April 19, 2026 10:09

[vitest-pool-workers] feat: allow user to configure logLevel on vites…

d61b34c

…t-pool-workers

fixup: remove dead guard, simplify logLevelFromString, fix changeset …

19ce637

…example

Keep inspector port as log.info (user-actionable); default to INFO; d…

97bbf5a

…rop vpw: prefix

petebacondarwin force-pushed the allow-loglevel branch from a311593 to 97bbf5a Compare April 19, 2026 09:11

petebacondarwin requested a review from workers-devprod as a code owner April 19, 2026 09:11

petebacondarwin approved these changes Apr 19, 2026

View reviewed changes

workers-devprod approved these changes Apr 19, 2026

View reviewed changes

petebacondarwin requested changes Apr 19, 2026

View reviewed changes

Comment thread packages/vitest-pool-workers/src/pool/index.ts Outdated

github-project-automation Bot moved this from Approved to In Review in workers-sdk Apr 19, 2026

Update packages/vitest-pool-workers/src/pool/index.ts

27cf551

petebacondarwin approved these changes Apr 19, 2026

View reviewed changes

github-project-automation Bot moved this from In Review to Approved in workers-sdk Apr 19, 2026

petebacondarwin merged commit 2c3258d into cloudflare:main Apr 19, 2026
59 of 60 checks passed

github-project-automation Bot moved this from Approved to Done in workers-sdk Apr 19, 2026

workers-devprod mentioned this pull request Apr 19, 2026

Version Packages #13511

Merged

MattIPv4 mentioned this pull request May 1, 2026

Bump the development group across 1 directory with 7 updates cdnjs/api-server#204

Merged

Navigation Menu

Uh oh!

Reduce `@cloudflare/vitest-pool-workers` log noise - #13007

Reduce `@cloudflare/vitest-pool-workers` log noise#13007
petebacondarwin merged 5 commits into
cloudflare:mainfrom
sheplu:allow-loglevel

sheplu commented Mar 22, 2026 •

edited

Loading

Uh oh!

changeset-bot Bot commented Mar 22, 2026 •

edited

Loading

Uh oh!

workers-devprod commented Mar 22, 2026 •

edited

Loading

Uh oh!

devin-ai-integration Bot left a comment

Uh oh!

pkg-pr-new Bot commented Mar 22, 2026 •

edited

Loading

Uh oh!

penalosa commented Mar 26, 2026

Uh oh!

sheplu commented Mar 26, 2026

Uh oh!

mrmckeb commented Mar 27, 2026

Uh oh!

sheplu commented Mar 27, 2026

Uh oh!

sheplu commented Apr 3, 2026

Uh oh!

workers-devprod commented Apr 19, 2026 •

edited

Loading

Uh oh!

workers-devprod left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

sheplu commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

changeset-bot Bot commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

workers-devprod commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

✅ Devin Review: No Issues Found

Uh oh!

pkg-pr-new Bot commented Mar 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

penalosa commented Mar 26, 2026

Uh oh!

sheplu commented Mar 26, 2026

Uh oh!

mrmckeb commented Mar 27, 2026

Uh oh!

sheplu commented Mar 27, 2026

Uh oh!

sheplu commented Apr 3, 2026

Uh oh!

workers-devprod commented Apr 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

workers-devprod left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

sheplu commented Mar 22, 2026 •

edited

Loading

changeset-bot Bot commented Mar 22, 2026 •

edited

Loading

workers-devprod commented Mar 22, 2026 •

edited

Loading

pkg-pr-new Bot commented Mar 22, 2026 •

edited

Loading

workers-devprod commented Apr 19, 2026 •

edited

Loading