Revert "feat: Expose iOS options to ignore views from subtree traversal (#5545)"

This reverts commit 4997892.

Author: antonis

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryReplayOptionsTests.swift

@@ -48,7 +48,7 @@ final class RNSentryReplayOptions: XCTestCase {
     }
 
     func assertAllDefaultReplayOptionsAreNotNil(replayOptions: [String: Any]) {
-        XCTAssertEqual(replayOptions.count, 11)
+        XCTAssertEqual(replayOptions.count, 9)
         XCTAssertNotNil(replayOptions["sessionSampleRate"])
         XCTAssertNotNil(replayOptions["errorSampleRate"])
         XCTAssertNotNil(replayOptions["maskAllImages"])
@@ -58,8 +58,6 @@ final class RNSentryReplayOptions: XCTestCase {
         XCTAssertNotNil(replayOptions["enableViewRendererV2"])
         XCTAssertNotNil(replayOptions["enableFastViewRendering"])
         XCTAssertNotNil(replayOptions["quality"])
-        XCTAssertNotNil(replayOptions["includedViewClasses"])
-        XCTAssertNotNil(replayOptions["excludedViewClasses"])
     }
 
     func testSessionSampleRate() {
@@ -320,64 +318,4 @@ final class RNSentryReplayOptions: XCTestCase {
 
         XCTAssertEqual(actualOptions.sessionReplay.quality, SentryReplayOptions.SentryReplayQuality.medium)
     }
-
-    func testIncludedViewClasses() {
-        let optionsDict = ([
-            "dsn": "https://abc@def.ingest.sentry.io/1234567",
-            "replaysOnErrorSampleRate": 0.75,
-            "mobileReplayOptions": [ "includedViewClasses": ["UILabel", "UIView", "UITextView"] ]
-        ] as NSDictionary).mutableCopy() as! NSMutableDictionary
-
-        RNSentryReplay.updateOptions(optionsDict)
-
-        let actualOptions = try! SentryOptionsInternal.initWithDict(optionsDict as! [String: Any])
-
-        let includedViewClasses = actualOptions.sessionReplay.includedViewClasses
-        XCTAssertEqual(includedViewClasses.count, 3)
-        XCTAssertTrue(includedViewClasses.contains("UILabel"))
-        XCTAssertTrue(includedViewClasses.contains("UIView"))
-        XCTAssertTrue(includedViewClasses.contains("UITextView"))
-    }
-
-    func testExcludedViewClasses() {
-        let optionsDict = ([
-            "dsn": "https://abc@def.ingest.sentry.io/1234567",
-            "replaysOnErrorSampleRate": 0.75,
-            "mobileReplayOptions": [ "excludedViewClasses": ["UICollectionView", "UITableView", "UIScrollView"] ]
-        ] as NSDictionary).mutableCopy() as! NSMutableDictionary
-
-        RNSentryReplay.updateOptions(optionsDict)
-
-        let actualOptions = try! SentryOptionsInternal.initWithDict(optionsDict as! [String: Any])
-
-        let excludedViewClasses = actualOptions.sessionReplay.excludedViewClasses
-        XCTAssertEqual(excludedViewClasses.count, 3)
-        XCTAssertTrue(excludedViewClasses.contains("UICollectionView"))
-        XCTAssertTrue(excludedViewClasses.contains("UITableView"))
-        XCTAssertTrue(excludedViewClasses.contains("UIScrollView"))
-    }
-
-    func testIncludedAndExcludedViewClasses() {
-        let optionsDict = ([
-            "dsn": "https://abc@def.ingest.sentry.io/1234567",
-            "replaysOnErrorSampleRate": 0.75,
-            "mobileReplayOptions": [
-                "includedViewClasses": ["UILabel", "UIView"],
-                "excludedViewClasses": ["UICollectionView"]
-            ]
-        ] as NSDictionary).mutableCopy() as! NSMutableDictionary
-
-        RNSentryReplay.updateOptions(optionsDict)
-
-        let actualOptions = try! SentryOptionsInternal.initWithDict(optionsDict as! [String: Any])
-
-        let includedViewClasses = actualOptions.sessionReplay.includedViewClasses
-        XCTAssertEqual(includedViewClasses.count, 2)
-        XCTAssertTrue(includedViewClasses.contains("UILabel"))
-        XCTAssertTrue(includedViewClasses.contains("UIView"))
-
-        let excludedViewClasses = actualOptions.sessionReplay.excludedViewClasses
-        XCTAssertEqual(excludedViewClasses.count, 1)
-        XCTAssertTrue(excludedViewClasses.contains("UICollectionView"))
-    }
 }

packages/core/ios/RNSentryReplay.mm

@@ -27,9 +27,6 @@ + (BOOL)updateOptions:(NSMutableDictionary *)options
 
     NSString *qualityString = options[@"replaysSessionQuality"];
 
-    NSArray *includedViewClasses = replayOptions[@"includedViewClasses"];
-    NSArray *excludedViewClasses = replayOptions[@"excludedViewClasses"];
-
     [options setValue:@{
         @"sessionSampleRate" : sessionSampleRate ?: [NSNull null],
         @"errorSampleRate" : errorSampleRate ?: [NSNull null],
@@ -39,8 +36,6 @@ + (BOOL)updateOptions:(NSMutableDictionary *)options
         @"enableViewRendererV2" : replayOptions[@"enableViewRendererV2"] ?: [NSNull null],
         @"enableFastViewRendering" : replayOptions[@"enableFastViewRendering"] ?: [NSNull null],
         @"maskedViewClasses" : [RNSentryReplay getReplayRNRedactClasses:replayOptions],
-        @"includedViewClasses" : includedViewClasses ?: [NSNull null],
-        @"excludedViewClasses" : excludedViewClasses ?: [NSNull null],
         @"sdkInfo" :
             @ { @"name" : REACT_NATIVE_SDK_NAME, @"version" : REACT_NATIVE_SDK_PACKAGE_VERSION }
     }

packages/core/src/js/replay/mobilereplay.ts

@@ -81,34 +81,6 @@ export interface MobileReplayOptions {
    */
   enableFastViewRendering?: boolean;
 
-  /**
-   * Array of view class names to include in subtree traversal during session replay and screenshot capture on iOS.
-   *
-   * Only views that are instances of these classes (or subclasses) will be traversed.
-   * This helps prevent crashes when traversing problematic view hierarchies by allowing you to explicitly include only safe view classes.
-   *
-   * If both `includedViewClasses` and `excludedViewClasses` are set, `excludedViewClasses` takes precedence:
-   * views matching excluded classes won't be traversed even if they match an included class.
-   *
-   * @default undefined
-   * @platform ios
-   */
-  includedViewClasses?: string[];
-
-  /**
-   * Array of view class names to exclude from subtree traversal during session replay and screenshot capture on iOS.
-   *
-   * Views of these classes (or subclasses) will be skipped entirely, including all their children.
-   * This helps prevent crashes when traversing problematic view hierarchies by allowing you to explicitly exclude problematic view classes.
-   *
-   * If both `includedViewClasses` and `excludedViewClasses` are set, `excludedViewClasses` takes precedence:
-   * views matching excluded classes won't be traversed even if they match an included class.
-   *
-   * @default undefined
-   * @platform ios
-   */
-  excludedViewClasses?: string[];
-
   /**
    * Sets the screenshot strategy used by the Session Replay integration on Android.
    *