Ref(Logs): Move it outside of Experimental (#5122)

* move to upper level

* Update samples/react-native/src/App.tsx

* no references to experiments

* changelog update:

* whitespace fix

Author: lucas-zimerman

CHANGELOG.md

@@ -6,6 +6,32 @@
 > make sure you follow our [migration guide](https://docs.sentry.io/platforms/react-native/migration/) first.
 <!-- prettier-ignore-end -->
 
+## Unreleased
+
+### Important Changes
+
+- The `_experiments.enableLogs` and `_experiments.beforeSendLog` options were removed, use the top-level `enableLogs` and `beforeSendLog` options instead. ([#5122](https://github.com/getsentry/sentry-react-native/pull/5122))
+
+```js
+// before
+Sentry.init({
+  _experiments: {
+    enableLogs: true,
+    beforeSendLog: log => {
+      return log;
+    },
+  },
+});
+
+// after
+Sentry.init({
+  enableLogs: true,
+  beforeSendLog: log => {
+    return log;
+  },
+});
+```
+
 ## 7.0.0-rc.2
 
 ### Important Changes

packages/core/android/src/main/java/io/sentry/react/RNSentryModuleImpl.java

@@ -283,11 +283,8 @@ protected void getSentryAndroidOptions(
     if (rnOptions.hasKey("enableNdk")) {
       options.setEnableNdk(rnOptions.getBoolean("enableNdk"));
     }
-    if (rnOptions.hasKey("_experiments")) {
-      ReadableMap experiments = rnOptions.getMap("_experiments");
-      if (experiments.hasKey("enableLogs")) {
-        options.getLogs().setEnabled(experiments.getBoolean("enableLogs"));
-      }
+    if (rnOptions.hasKey("enableLogs")) {
+      options.getLogs().setEnabled(rnOptions.getBoolean("enableLogs"));
     }
     if (rnOptions.hasKey("spotlight")) {
       if (rnOptions.getType("spotlight") == ReadableType.Boolean) {

packages/core/src/js/client.ts

@@ -64,7 +64,7 @@ export class ReactNativeClient extends Client<ReactNativeClientOptions> {
       this.on('beforeSendSession', addAutoIpAddressToSession);
     }
 
-    if (options._experiments?.enableLogs) {
+    if (options.enableLogs) {
       this.on('flush', () => {
         _INTERNAL_flushLogsBuffer(this);
       });

packages/core/src/js/integrations/default.ts

@@ -85,7 +85,7 @@ export function getDefaultIntegrations(options: ReactNativeClientOptions): Integ
   if (options.enableNative) {
     integrations.push(deviceContextIntegration());
     integrations.push(modulesLoaderIntegration());
-    if (options._experiments?.enableLogs) {
+    if (options.enableLogs) {
       integrations.push(logEnricherIntegration());
     }
     if (options.attachScreenshot) {

packages/core/src/js/options.ts

@@ -10,7 +10,7 @@ type ProfilerProps = React.ComponentProps<typeof Profiler>;
 type BrowserTransportOptions = Parameters<typeof makeFetchTransport>[0];
 
 type BrowserExperiments = NonNullable<BrowserOptions['_experiments']>;
-type SharedExperimentsSubset = Pick<BrowserExperiments, 'enableLogs' | 'beforeSendLog'>;
+type SharedExperimentsSubset = BrowserExperiments;
 
 export interface BaseReactNativeOptions {
   /**

samples/react-native/src/App.tsx

@@ -82,12 +82,12 @@ Sentry.init({
     );
   },
   _experiments: {
-    enableLogs: true,
-    beforeSendLog: (log) => {
-      return log;
-    },
     enableUnhandledCPPExceptionsV2: true,
   },
+  enableLogs: true,
+  beforeSendLog: (log) => {
+    return log;
+  },
   enableUserInteractionTracing: true,
   integrations(integrations) {
     integrations.push(