Merge branch 'main' into dependabot/npm_and_yarn/uuid-14.0.0

Author: antonis

.craft.yml

@@ -9,4 +9,5 @@ targets:
       npm:@sentry/react-native:
         includeNames: /^sentry-react-native-\d.*\.tgz$/
       npm:@sentry/expo-upload-sourcemaps:
+        name: '@sentry/expo-upload-sourcemaps'
         includeNames: /^sentry-expo-upload-sourcemaps-\d.*\.tgz$/

.github/workflows/buildandtest.yml

@@ -166,6 +166,7 @@ jobs:
           name: ${{ github.sha }}
           path: |
             ${{ github.workspace }}/packages/core/*.tgz
+            ${{ github.workspace }}/packages/expo-upload-sourcemaps/*.tgz
 
   job_type_check:
     name: Type Check Typescript 3.8

CHANGELOG.md

@@ -8,6 +8,12 @@
 
 ## Unreleased
 
+### Fixes
+
+- Stop the Hermes sampling profiler on React instance teardown to prevent `pthread_kill` SIGABRT when the JS thread is torn down with profiling active ([#6035](https://github.com/getsentry/sentry-react-native/pull/6035))
+
+## 8.9.1
+
 ### Features
 
 - Add new `@sentry/expo-upload-sourcemaps` package for uploading JavaScript bundles and source maps from Expo builds to Sentry ([#6027](https://github.com/getsentry/sentry-react-native/pull/6027))
@@ -20,7 +26,9 @@
 
 ### Fixes
 
+- Fix `@sentry/expo-upload-sourcemaps` publishing issue ([#6037](https://github.com/getsentry/sentry-react-native/pull/6037))
 - Fix iOS UI profiling options being silently ignored ([#6012](https://github.com/getsentry/sentry-react-native/pull/6012))
+- Fix `_experiments.enableUnhandledCPPExceptionsV2` being silently ignored on iOS ([#6014](https://github.com/getsentry/sentry-react-native/pull/6014))
 - Check `captureReplay` return value in iOS bridge to avoid linking error events to uncaptured replays ([#6008](https://github.com/getsentry/sentry-react-native/pull/6008))
 - Report the expected properties file path and any missing keys when using `flavorAware` on Android, instead of failing with an opaque `Illegal null value provided in this collection` error ([#6031](https://github.com/getsentry/sentry-react-native/pull/6031))
 

SDK-VERSIONS.md

@@ -12,6 +12,7 @@ To manually update the table with the current version, run `./scripts/update-sdk
 
 | React Native SDK | Android SDK | Cocoa SDK | JavaScript SDK |
 | ---------------- | ----------- | --------- | -------------- |
+| [8.9.1](https://github.com/getsentry/sentry-react-native/releases/tag/8.9.1) | [8.40.0](https://github.com/getsentry/sentry-java/releases/tag/8.40.0) | [9.11.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.11.0) | [10.49.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.49.0) |
 | [8.8.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.8.0) | [8.38.0](https://github.com/getsentry/sentry-java/releases/tag/8.38.0) | [9.10.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.10.0) | [10.48.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.48.0) |
 | [8.7.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.7.0) | [8.37.1](https://github.com/getsentry/sentry-java/releases/tag/8.37.1) | [9.8.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.8.0) | [10.47.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.47.0) |
 | [8.6.0](https://github.com/getsentry/sentry-react-native/releases/tag/8.6.0) | [8.36.0](https://github.com/getsentry/sentry-java/releases/tag/8.36.0) | [9.8.0](https://github.com/getsentry/sentry-cocoa/releases/tag/9.8.0) | [10.46.0](https://github.com/getsentry/sentry-javascript/releases/tag/10.46.0) |

dev-packages/e2e-tests/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sentry-react-native-e2e-tests",
-  "version": "8.8.0",
+  "version": "8.9.1",
   "private": true,
   "description": "Sentry React Native End to End Tests Library",
   "main": "dist/index.js",
@@ -14,7 +14,7 @@
     "@babel/preset-env": "^7.25.3",
     "@babel/preset-typescript": "^7.18.6",
     "@sentry/core": "10.49.0",
-    "@sentry/react-native": "8.8.0",
+    "@sentry/react-native": "8.9.1",
     "@types/node": "^20.9.3",
     "@types/react": "^18.2.64",
     "appium": "3.2.2",

dev-packages/type-check/package.json

@@ -1,7 +1,7 @@
 {
   "name": "sentry-react-native-type-check",
   "private": true,
-  "version": "8.8.0",
+  "version": "8.9.1",
   "scripts": {
     "type-check": "./run-type-check.sh"
   }

dev-packages/utils/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sentry-react-native-samples-utils",
-  "version": "8.8.0",
+  "version": "8.9.1",
   "description": "Internal Samples Utils",
   "main": "index.js",
   "license": "MIT",

lerna.json

@@ -1,6 +1,6 @@
 {
   "$schema": "node_modules/lerna/schemas/lerna-schema.json",
-  "version": "8.8.0",
+  "version": "8.9.1",
   "packages": [
     "packages/*",
     "dev-packages/*",

packages/core/RNSentryAndroidTester/app/src/test/java/io/sentry/react/RNSentryProfilingTest.kt

@@ -0,0 +1,33 @@
+package io.sentry.react
+
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.robolectric.RobolectricTestRunner
+
+/**
+ * Tests for the profiling lifecycle on the RNSentryModuleImpl.
+ *
+ * Note: the Hermes sampling profiler is a JNI-backed static API and cannot be loaded in a pure
+ * JVM unit-test environment. These tests exercise only the paths that do not cross the JNI
+ * boundary — specifically, the invalidate() guard that returns early when profiling is not
+ * active. If the guard were to regress and call into Hermes, the tests would fail with
+ * UnsatisfiedLinkError.
+ */
+@RunWith(RobolectricTestRunner::class)
+class RNSentryProfilingTest {
+    @Test
+    fun `invalidate is a no-op on a fresh module`() {
+        val module = Utils.createRNSentryModuleWithMockedContext()
+        // isProfiling starts false; invalidate() should return early without touching
+        // HermesSamplingProfiler. If it didn't, the JNI call would throw UnsatisfiedLinkError.
+        module.invalidate()
+    }
+
+    @Test
+    fun `invalidate is idempotent`() {
+        val module = Utils.createRNSentryModuleWithMockedContext()
+        module.invalidate()
+        module.invalidate()
+        module.invalidate()
+    }
+}

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryTests.m

@@ -1471,6 +1471,60 @@ - (void)testStartCreateOptionsWithDictionaryEmptyExperimentsDoesNotInstallConfig
 }
 #endif
 
+- (void)testStartCreateOptionsWithDictionaryEnableUnhandledCPPExceptionsV2Enabled
+{
+    NSError *error = nil;
+
+    NSDictionary *_Nonnull mockedReactNativeDictionary = @{
+        @"dsn" : @"https://abcd@efgh.ingest.sentry.io/123456",
+        @"_experiments" : @ {
+            @"enableUnhandledCPPExceptionsV2" : @YES,
+        },
+    };
+    SentryOptions *actualOptions =
+        [RNSentryStart createOptionsWithDictionary:mockedReactNativeDictionary error:&error];
+
+    XCTAssertNotNil(actualOptions, @"Did not create sentry options");
+    XCTAssertNil(error, @"Should not pass no error");
+    XCTAssertTrue(actualOptions.experimental.enableUnhandledCPPExceptionsV2,
+        @"enableUnhandledCPPExceptionsV2 should be enabled");
+}
+
+- (void)testStartCreateOptionsWithDictionaryEnableUnhandledCPPExceptionsV2Disabled
+{
+    NSError *error = nil;
+
+    NSDictionary *_Nonnull mockedReactNativeDictionary = @{
+        @"dsn" : @"https://abcd@efgh.ingest.sentry.io/123456",
+        @"_experiments" : @ {
+            @"enableUnhandledCPPExceptionsV2" : @NO,
+        },
+    };
+    SentryOptions *actualOptions =
+        [RNSentryStart createOptionsWithDictionary:mockedReactNativeDictionary error:&error];
+
+    XCTAssertNotNil(actualOptions, @"Did not create sentry options");
+    XCTAssertNil(error, @"Should not pass no error");
+    XCTAssertFalse(actualOptions.experimental.enableUnhandledCPPExceptionsV2,
+        @"enableUnhandledCPPExceptionsV2 should be disabled");
+}
+
+- (void)testStartCreateOptionsWithDictionaryEnableUnhandledCPPExceptionsV2Default
+{
+    NSError *error = nil;
+
+    NSDictionary *_Nonnull mockedReactNativeDictionary = @{
+        @"dsn" : @"https://abcd@efgh.ingest.sentry.io/123456",
+    };
+    SentryOptions *actualOptions =
+        [RNSentryStart createOptionsWithDictionary:mockedReactNativeDictionary error:&error];
+
+    XCTAssertNotNil(actualOptions, @"Did not create sentry options");
+    XCTAssertNil(error, @"Should not pass no error");
+    XCTAssertFalse(actualOptions.experimental.enableUnhandledCPPExceptionsV2,
+        @"enableUnhandledCPPExceptionsV2 should default to disabled");
+}
+
 - (void)testStartEventFromSentryCocoaReactNativeHasOriginAndEnvironmentTags
 {
     SentryEvent *testEvent = [[SentryEvent alloc] init];