Skip to content

Start supporting node in exec-harness#267

Open
GuillaumeLagrange wants to merge 2 commits intomainfrom
cod-2036-node-in-profiling-cli
Open

Start supporting node in exec-harness#267
GuillaumeLagrange wants to merge 2 commits intomainfrom
cod-2036-node-in-profiling-cli

Conversation

@GuillaumeLagrange
Copy link
Contributor

Current limitations:

  1. Walltime benchmarking works, but the profiling is unreliable until we tackle things properly for the whole of walltime nodejs profiling.
  2. Simulation works with the caveat that we do not support subprocesses (yet) -> tools like tsx etc to run typescript are limited.
  3. For both modes, we do not transform symbols from node because post processing is quite destructive, and will keep symbols like JS:^busywait file:///home/guillaume/codspeed/codspeed-integrations-e2e-tests/nodejs/main.js:6:18 in the profile data until we can explicitly set the language for post processing as a user config

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 17, 2026

Merging this PR will not alter performance

✅ 4 untouched benchmarks


Comparing cod-2036-node-in-profiling-cli (4ec8ef6) with main (106e731)

Open in CodSpeed

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