Skip to content

fix(ext/node): export report as named export from node:process#35400

Merged
bartlomieju merged 4 commits into
denoland:mainfrom
Vallabh-1504:process-report
Jun 29, 2026
Merged

fix(ext/node): export report as named export from node:process#35400
bartlomieju merged 4 commits into
denoland:mainfrom
Vallabh-1504:process-report

Conversation

@Vallabh-1504

Copy link
Copy Markdown
Contributor

Description

Fixes #35072

process.report was full implemented and attached to the default export but was never added to the named exports. This caused import { report } from 'node:process' to throw:
Uncaught SyntaxError: The requested module 'node:process' does not provide an export named 'report'

Root cause:

it was missing from the export { ... } named exports line (line 586), so destructured imports failed.

Changes:

  • ext/node/polyfills/process.ts: Added report to the named exports line alongside the other process exports.
  • tests/unit_node/process_test.ts: Added a regression test verifying the named export exists, is the same object reference as process.report, and exposes key methods like getReport and writeReport.
  • export block in process.ts was reformatted by ./x fmt

Test output:

./target/debug/deno test --config tests/config/deno.json --allow-all tests/unit_node/process_test.ts --filter "named export { report } from node:process"
Check tests/unit_node/process_test.ts
running 1 test from ./tests/unit_node/process_test.ts
named export { report } from node:process ... ok (11ms)

ok | 1 passed | 0 failed | 97 filtered out (92ms)

AI Disclosure:

AI assistance was used during development- primarily for build related problem and commands. The fix itself is a one-word change to the exports line.

@deno-cla-assistant

deno-cla-assistant Bot commented Jun 20, 2026

Copy link
Copy Markdown

Deno Individual Contributor License Agreement

All contributors have signed the CLA. Thank you!

Re-run CLA check


This is an automated message from CLA Assistant

@bartlomieju bartlomieju left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@bartlomieju
bartlomieju enabled auto-merge (squash) June 22, 2026 10:58
@bartlomieju
bartlomieju merged commit 6ddc9d7 into denoland:main Jun 29, 2026
136 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.

node:process does not export report

2 participants