Skip to content

node.js webpack experimental support#37458

Open
tlhunter wants to merge 3 commits into
masterfrom
tlhunter/webpack
Open

node.js webpack experimental support#37458
tlhunter wants to merge 3 commits into
masterfrom
tlhunter/webpack

Conversation

@tlhunter

@tlhunter tlhunter commented Jun 11, 2026

Copy link
Copy Markdown
Member

What does this PR do? What is the motivation?

  • Documents the experimental Webpack support for the Node.js tracer.
  • Changes esbuild to ESBuild as per their website.

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:

Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.

AI assistance

  • none

Additional notes

@tlhunter tlhunter requested a review from a team as a code owner June 11, 2026 19:07
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@evazorro evazorro self-assigned this Jun 11, 2026

@evazorro evazorro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I see that you've changed esbuild->ESBuild throughout, but their website styles it as esbuild: https://esbuild.github.io/. I'd recommend changing it back. I also left a question and a wording suggestion. Thank you for contributing!

})
```

**Note**: Every application is different and you may need to experiment with the list of `external` packages.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure how helpful this note is; is there more guidance we can provide here? What does "experiment" mean in this case?

Comment thread content/en/tracing/trace_collection/dd_libraries/nodejs.md Outdated
@evazorro evazorro removed their assignment Jun 11, 2026
@tlhunter

Copy link
Copy Markdown
Member Author

You're right, it is always esbuild. I only glanced at the google search result:

2026-06-11-13-43-17

tlhunter and others added 2 commits June 11, 2026 13:44
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