Skip to content

diagnostics_channel: fix default callback context - #66481

Open
mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:fix-trace-callback-default-context
Open

mcollina wants to merge 1 commit into
nodejs:mainfrom
mcollina:fix-trace-callback-default-context

Conversation

@mcollina

@mcollina mcollina commented Oct 3, 2026

Copy link
Copy Markdown
Member

traceCallback() defaults to a frozen context but writes the callback's result or error to it. Allocate a fresh context when undefined is passed and subscribers are present, matching traceSync() and tracePromise().

Set the prototype after allocation to avoid dictionary-mode properties.

traceCallback() defaults to a frozen context but writes the callback's
result or error to it. Allocate a fresh context when undefined is passed
and subscribers are present, matching traceSync() and tracePromise().
Set the prototype after allocation to avoid dictionary-mode properties.

Assisted-by: pi
Signed-off-by: Matteo Collina <hello@matteocollina.com>
@nodejs-github-bot nodejs-github-bot added diagnostics_channel Issues and PRs related to the diagnostics_channel module. needs-ci PRs that need a full CI run. labels Oct 3, 2026
@codecov

codecov Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.42%. Comparing base (cede7e6) to head (36c6c9e).
⚠️ Report is 116 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #66481      +/-   ##
==========================================
+ Coverage   90.39%   90.42%   +0.03%     
==========================================
  Files         792      790       -2     
  Lines      275697   275439     -258     
  Branches    52868    52828      -40     
==========================================
- Hits       249208   249072     -136     
+ Misses      16892    16765     -127     
- Partials     9597     9602       +5     
Files with missing lines Coverage Δ
lib/diagnostics_channel.js 97.63% <100.00%> (+0.01%) ⬆️

... and 60 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bengl bengl 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.

LGTM. Consistent with #65513

@bengl bengl added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Oct 6, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Oct 6, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

diagnostics_channel Issues and PRs related to the diagnostics_channel module. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants