feat(core): Wire TurboModulePerfLogger on iOS and Android

CHANGELOG.md

@@ -8,6 +8,10 @@
 
 ## Unreleased
 
+### Features
+
+- Add `enableTurboModuleTracking` opt-in experimental option to enable Turbo Module performance tracking in the New Architecture ([#6307](https://github.com/getsentry/sentry-react-native/pull/6307))
+
 ### Fixes
 
 - Apply `SENTRY_ENVIRONMENT`, `SENTRY_RELEASE` and `SENTRY_DIST` build-time overrides to the JS bundled options to match the native SDKs ([#6330](https://github.com/getsentry/sentry-react-native/pull/6330))

packages/core/.npmignore

@@ -14,6 +14,7 @@
 !react-native.config.js
 !/ios/**/*
 !/android/**/*
+!/cpp/**/*
 
 # New Architecture Codegen
 !src/js/NativeRNSentry.ts

packages/core/RNSentry.podspec

@@ -41,7 +41,11 @@ Pod::Spec.new do |s|
 
   s.preserve_paths = '*.js'
 
-  s.source_files = 'ios/**/*.{h,m,mm}'
+  # `cpp/` holds platform-agnostic C++ used by both iOS and Android. On iOS it
+  # is pulled in here; on Android it is compiled by the dedicated CMake target
+  # in `android/CMakeLists.txt`. The files are guarded with
+  # `RCT_NEW_ARCH_ENABLED` so they compile to empty TUs on Old Arch.
+  s.source_files = 'ios/**/*.{h,m,mm}', 'cpp/**/*.{h,cpp}'
   s.public_header_files = 'ios/RNSentry.h', 'ios/RNSentrySDK.h', 'ios/RNSentryStart.h', 'ios/RNSentryVersion.h', 'ios/RNSentryBreadcrumb.h', 'ios/RNSentryReplay.h', 'ios/RNSentryReplayBreadcrumbConverter.h', 'ios/Replay/RNSentryReplayMask.h', 'ios/Replay/RNSentryReplayUnmask.h', 'ios/RNSentryTimeToDisplay.h'
 
   s.compiler_flags = other_cflags

packages/core/RNSentryAndroidTester/app/src/test/java/io/sentry/rnsentryandroidtester/RNSentryTurboModulePerfTrackerTest.kt

@@ -0,0 +1,96 @@
+package io.sentry.rnsentryandroidtester
+
+import io.sentry.react.RNSentryTurboModulePerfTracker
+import org.junit.After
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
+import org.junit.Before
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.robolectric.RobolectricTestRunner
+
+/**
+ * Unit coverage for the JVM-side wrapper around the native perf-logger toggle.
+ *
+ * In a host JVM (where this test runs) there is no Android system loader for
+ * `libsentry-tm-perf-logger.so`, so any call into the native method must throw
+ * `UnsatisfiedLinkError`. The tracker is expected to swallow that error and
+ * flip an internal latch so subsequent calls short-circuit without retrying.
+ *
+ * Uses Robolectric so the `android.util.Log` call inside the tracker's `catch`
+ * branch resolves to a real implementation instead of the default-not-mocked
+ * stub the bare JUnit4 runner exposes.
+ */
+@RunWith(RobolectricTestRunner::class)
+class RNSentryTurboModulePerfTrackerTest {
+    @Before
+    fun resetLatch() {
+        // Each test exercises the latch transition from scratch; without this
+        // reset the second test in execution order would see the latch already
+        // tripped from the previous one.
+        RNSentryTurboModulePerfTracker.resetNativeUnavailableForTests()
+    }
+
+    @After
+    fun cleanUp() {
+        RNSentryTurboModulePerfTracker.resetNativeUnavailableForTests()
+    }
+
+    @Test
+    fun setEnabledSwallowsUnsatisfiedLinkErrorOnFirstCall() {
+        // No `.so` loaded in the test JVM → the JNI symbol is missing. The
+        // tracker must absorb the resulting `UnsatisfiedLinkError` so the
+        // caller does not see a crash on a misconfigured host.
+        RNSentryTurboModulePerfTracker.setEnabled(true)
+        // Reaching this point means the error was caught, which is the contract.
+        assertTrue(
+            "after a failed link, the tracker must latch the failure",
+            RNSentryTurboModulePerfTracker.isNativeUnavailableForTests(),
+        )
+    }
+
+    @Test
+    fun subsequentCallsShortCircuitAfterLatchTrips() {
+        // Trip the latch via the first call.
+        RNSentryTurboModulePerfTracker.setEnabled(true)
+        assertTrue(RNSentryTurboModulePerfTracker.isNativeUnavailableForTests())
+
+        // The second call must not throw or attempt to relink. The contract is
+        // "exactly one UnsatisfiedLinkError per process lifetime" — anything
+        // else means the tracker is hammering the runtime on every setEnabled.
+        RNSentryTurboModulePerfTracker.setEnabled(false)
+        RNSentryTurboModulePerfTracker.setEnabled(true)
+        assertTrue(
+            "latch must stay tripped across repeated calls",
+            RNSentryTurboModulePerfTracker.isNativeUnavailableForTests(),
+        )
+    }
+
+    @Test
+    fun setEnabledFalseDoesNotLoadNativeLibrary() {
+        // The lazy-load contract: hosts that never opt in to
+        // `enableTurboModuleTracking` pay no shared-library cost. A bare
+        // `initNativeSdk` with the option absent or `false` calls
+        // `setEnabled(false)` from Java, and we expect this NOT to attempt
+        // `System.loadLibrary`. The proxy for "did we attempt the load?" is
+        // the `nativeUnavailable` latch — in the test JVM the load would
+        // fail, so if it ran we would see the latch tripped.
+        RNSentryTurboModulePerfTracker.setEnabled(false)
+        assertFalse(
+            "setEnabled(false) on a fresh tracker must not attempt to load the native library",
+            RNSentryTurboModulePerfTracker.isNativeUnavailableForTests(),
+        )
+    }
+
+    @Test
+    fun resetClearsTheLatch() {
+        RNSentryTurboModulePerfTracker.setEnabled(true)
+        assertTrue(RNSentryTurboModulePerfTracker.isNativeUnavailableForTests())
+
+        RNSentryTurboModulePerfTracker.resetNativeUnavailableForTests()
+        assertFalse(
+            "the @TestOnly reset must clear the latch so tests can re-exercise it",
+            RNSentryTurboModulePerfTracker.isNativeUnavailableForTests(),
+        )
+    }
+}

packages/core/RNSentryCocoaTester/RNSentryCocoaTester.xcodeproj/project.pbxproj

@@ -17,6 +17,7 @@
 		33DEDFED2D8DC825006066E4 /* RNSentryOnDrawReporter+Test.mm in Sources */ = {isa = PBXBuildFile; fileRef = 33DEDFEC2D8DC820006066E4 /* RNSentryOnDrawReporter+Test.mm */; };
 		33DEDFF02D9185EB006066E4 /* RNSentryTimeToDisplayTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 33DEDFEF2D9185E3006066E4 /* RNSentryTimeToDisplayTests.swift */; };
 		33F58AD02977037D008F60EA /* RNSentryTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 33F58ACF2977037D008F60EA /* RNSentryTests.m */; };
+		2639D71D3BD04F17B0BAC987 /* RNSentryTurboModulePerfControllerTests.mm in Sources */ = {isa = PBXBuildFile; fileRef = E795057A6D534A80A9D06356 /* RNSentryTurboModulePerfControllerTests.mm */; };
 		AEFB00422CC90C4B00EC8A9A /* RNSentryBreadcrumbTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 3360843C2C340C76008CC412 /* RNSentryBreadcrumbTests.swift */; };
 		B4DEB41739F14AA38202D4D4 /* RNSentryUriValidationTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 3E3742693F7643C2ADE1BDF2 /* RNSentryUriValidationTests.m */; };
 		B5859A50A3E865EF5E61465A /* libPods-RNSentryCocoaTesterTests.a in Frameworks */ = {isa = PBXBuildFile; fileRef = 650CB718ACFBD05609BF2126 /* libPods-RNSentryCocoaTesterTests.a */; };
@@ -51,6 +52,7 @@
 		33F58ACF2977037D008F60EA /* RNSentryTests.m */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.objc; path = RNSentryTests.m; sourceTree = "<group>"; };
 		3E3742693F7643C2ADE1BDF2 /* RNSentryUriValidationTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = RNSentryUriValidationTests.m; sourceTree = "<group>"; };
 		650CB718ACFBD05609BF2126 /* libPods-RNSentryCocoaTesterTests.a */ = {isa = PBXFileReference; explicitFileType = archive.ar; includeInIndex = 0; path = "libPods-RNSentryCocoaTesterTests.a"; sourceTree = BUILT_PRODUCTS_DIR; };
+		E795057A6D534A80A9D06356 /* RNSentryTurboModulePerfControllerTests.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = RNSentryTurboModulePerfControllerTests.mm; sourceTree = "<group>"; };
 		E2321E7CFA55AB617247098E /* Pods-RNSentryCocoaTesterTests.debug.xcconfig */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xcconfig; name = "Pods-RNSentryCocoaTesterTests.debug.xcconfig"; path = "Target Support Files/Pods-RNSentryCocoaTesterTests/Pods-RNSentryCocoaTesterTests.debug.xcconfig"; sourceTree = "<group>"; };
 		F48F26542EA2A481008A185E /* RNSentryEmitNewFrameEvent.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = RNSentryEmitNewFrameEvent.h; path = ../ios/RNSentryEmitNewFrameEvent.h; sourceTree = SOURCE_ROOT; };
 		F48F26552EA2A4D4008A185E /* RNSentryFramesTrackerListener.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = RNSentryFramesTrackerListener.h; path = ../ios/RNSentryFramesTrackerListener.h; sourceTree = SOURCE_ROOT; };
@@ -111,6 +113,7 @@
 				33F58ACF2977037D008F60EA /* RNSentryTests.m */,
 				3339C4802D6625570088EB3A /* RNSentryUserTests.m */,
 				3E3742693F7643C2ADE1BDF2 /* RNSentryUriValidationTests.m */,
+				E795057A6D534A80A9D06356 /* RNSentryTurboModulePerfControllerTests.mm */,
 				33AFDFEC2B8D14B300AAB120 /* RNSentryFramesTrackerListenerTests.m */,
 				33AFDFF02B8D15E500AAB120 /* RNSentryDependencyContainerTests.m */,
 				3360843C2C340C76008CC412 /* RNSentryBreadcrumbTests.swift */,
@@ -266,6 +269,7 @@
 				33F58AD02977037D008F60EA /* RNSentryTests.m in Sources */,
 				3339C4812D6625570088EB3A /* RNSentryUserTests.m in Sources */,
 				B4DEB41739F14AA38202D4D4 /* RNSentryUriValidationTests.m in Sources */,
+				2639D71D3BD04F17B0BAC987 /* RNSentryTurboModulePerfControllerTests.mm in Sources */,
 				33DEDFF02D9185EB006066E4 /* RNSentryTimeToDisplayTests.swift in Sources */,
 				3380C6C42CE25ECA0018B9B6 /* RNSentryReplayPostInitTests.swift in Sources */,
 				33AFDFED2B8D14B300AAB120 /* RNSentryFramesTrackerListenerTests.m in Sources */,

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentry+Test.h

@@ -7,6 +7,8 @@
 
 + (BOOL)captureReplayWithReturnValue;
 
++ (BOOL)turboModuleTrackingEnabledFromOptions:(NSDictionary *)options;
+
 #if TARGET_OS_IPHONE || TARGET_OS_MACCATALYST
 + (BOOL)isPathUnderAllowedRootsForTesting:(NSString *)path;
 #endif