microsoft / microsoft/react-native-windows

Implement JSRuntime enableSamplingProfiler, disableSamplingProfiler, and collectSamplingProfile

Open
#14,700 0 comments 0 reactions 1 assignee View on GitHub

@vmoroz is already working on this.

Since Jun 11, 2025.

Area: Debug Infrastructure enhancement Integration Follow-up
Dominant language
C++
Stars
17.3k
Forks
1.2k
Avg merge
1d 13h
Merged PRs (30d)
33

Description

Problem Description

Upstream added the following methods to their RuntimeAgent (see linked commits) and our HermesRuntimeHolder would need to implement these methods.

  • enableSamplingProfiler
  • disableSamplingProfiler
  • collectSamplingProfile

Currently stubbed these methods out but unsure if we need a proper implementation

Link to the integration where the error originated

https://github.com/microsoft/react-native-windows/pull/14693

Link to commit

https://github.com/facebook/react-native/pull/48836/files
https://github.com/facebook/react-native/pull/48315/files

Forked files

Stub Methods in

  • vnext/Shared/HermesRuntimeHolder.cpp
  • vnext/Shared/HermesRuntimeHolder.h
  • TEMP_UntilReactCommonUpdate/cxxreact/JSExecutor.cpp
  • ReactCommon/TEMP_UntilReactCommonUpdate/cxxreact/JSExecutor.h
Upstream PR

N/A - we will need to implement in our code

Build Configuration

None

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.