Skip to content

Support console.debug on stderr#10560

Closed
guybedford wants to merge 1 commit intomainfrom
console-debug-stderr
Closed

Support console.debug on stderr#10560
guybedford wants to merge 1 commit intomainfrom
console-debug-stderr

Conversation

@guybedford
Copy link
Copy Markdown
Contributor

@guybedford guybedford commented Sep 5, 2025

In cloudflare/workerd#4965 we are considering having console.debug output to stderr in Node.js as opposed to stdout, to allow the new process.stderr.write() implementation to log through console.debug instead of console.error and avoid possible error logging noise if we use console.error log types.

This is a test-suite only change which would allow us to land that after the next Miniflare release.

  • Tests
    • Tests included
    • Tests not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: undocumented behaviour
  • Wrangler V3 Backport
    • Wrangler PR:
    • Not necessary because: test suite only change

@guybedford guybedford requested a review from a team as a code owner September 5, 2025 17:13
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 5, 2025 •
edited
Loading

⚠️ No Changeset found

Latest commit: d5e0a24

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Sep 5, 2025
@guybedford guybedford mentioned this pull request Sep 5, 2025
Closed
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Sep 5, 2025 •
edited
Loading

create-cloudflare

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

@cloudflare/kv-asset-handler

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

miniflare

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

@cloudflare/pages-shared

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

@cloudflare/unenv-preset

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

@cloudflare/vite-plugin

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

@cloudflare/vitest-pool-workers

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

@cloudflare/workers-editor-shared

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

wrangler

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

commit: d5e0a24

@vicb
Copy link
Copy Markdown
Contributor

vicb commented Sep 8, 2025

@guybedford I have added questions/comments in the original workerd PR that I miss to understand the rationale of this change.

@petebacondarwin petebacondarwin added the blocked Blocked on other work label Sep 8, 2025
@petebacondarwin
Copy link
Copy Markdown
Contributor

petebacondarwin commented Sep 8, 2025

Marking as blocked on cloudflare/workerd#4965

@petebacondarwin
Copy link
Copy Markdown
Contributor

petebacondarwin commented Sep 10, 2025

Let's close this one and put any changes needed into the PR that turns on native process: #10577

@github-project-automation github-project-automation bot moved this from Untriaged to Done in workers-sdk Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

blocked Blocked on other work no-changeset-required

Projects

workers-sdk
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@guybedford @vicb @petebacondarwin @workers-devprod