fix(integrations): No screenshots when the SDK is disabled (#3333)

Author: krystofwoldrich

CHANGELOG.md

@@ -57,6 +57,7 @@
 - Change log output to show what paths are considered when collecting modules ([#3316](https://github.com/getsentry/sentry-react-native/pull/3316))
 - `Sentry.wrap` doesn't enforce any keys on the wrapped component props ([#3332](https://github.com/getsentry/sentry-react-native/pull/3332))
 - Ignore defaults when warning about duplicate definition of trace propagation targets ([#3327](https://github.com/getsentry/sentry-react-native/pull/3327))
+- Screenshots are not taken when the SDK is disabled ([#3333](https://github.com/getsentry/sentry-react-native/pull/3333))
 - Use deprecated `ReactNativeTracingOptions.tracingOrigins` if set in the options ([#3331](https://github.com/getsentry/sentry-react-native/pull/3331))
 
 ### Dependencies

src/js/client.ts

@@ -16,7 +16,6 @@ import { dateTimestampInSeconds, logger, SentryError } from '@sentry/utils';
 import { Alert } from 'react-native';
 
 import { createIntegration } from './integrations/factory';
-import { Screenshot } from './integrations/screenshot';
 import { defaultSdkInfo } from './integrations/sdkinfo';
 import type { ReactNativeClientOptions } from './options';
 import { ReactNativeTracing } from './tracing';
@@ -52,9 +51,7 @@ export class ReactNativeClient extends BaseClient<ReactNativeClientOptions> {
    * @inheritDoc
    */
   public eventFromException(exception: unknown, hint: EventHint = {}): PromiseLike<Event> {
-    return Screenshot.attachScreenshotToEventHint(hint, this._options).then(hintWithScreenshot =>
-      eventFromException(this._options.stackParser, exception, hintWithScreenshot, this._options.attachStacktrace),
-    );
+    return eventFromException(this._options.stackParser, exception, hint, this._options.attachStacktrace);
   }
 
   /**

src/js/integrations/screenshot.ts

@@ -1,6 +1,7 @@
-import type { EventHint, Integration } from '@sentry/types';
+import type { Event, EventHint, EventProcessor, Integration } from '@sentry/types';
 import { resolvedSyncPromise } from '@sentry/utils';
 
+import type { Screenshot as ScreenshotAttachment } from '../wrapper';
 import { NATIVE } from '../wrapper';
 
 /** Adds screenshots to error events */
@@ -17,6 +18,8 @@ export class Screenshot implements Integration {
 
   /**
    * If enabled attaches a screenshot to the event hint.
+   *
+   * @deprecated Screenshots are now added in global event processor.
    */
   public static attachScreenshotToEventHint(
     hint: EventHint,
@@ -37,6 +40,19 @@ export class Screenshot implements Integration {
   /**
    * @inheritDoc
    */
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
-  public setupOnce(): void {}
+  public setupOnce(addGlobalEventProcessor: (e: EventProcessor) => void): void {
+    addGlobalEventProcessor(async (event: Event, hint: EventHint) => {
+      const hasException = event.exception && event.exception.values && event.exception.values.length > 0;
+      if (!hasException) {
+        return event;
+      }
+
+      const screenshots: ScreenshotAttachment[] | null = await NATIVE.captureScreenshot();
+      if (screenshots && screenshots.length > 0) {
+        hint.attachments = [...screenshots, ...(hint?.attachments || [])];
+      }
+
+      return event;
+    });
+  }
 }