Merge branch 'main' into expo

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## Unreleased
+
+### Fixes
+
+- Errors from InternalBytecode.js are no longer marked as in_app ([#3518](https://github.com/getsentry/sentry-react-native/pull/3518))
+
 ## 5.16.0-alpha.4
 
 ### Fixes

src/js/integrations/rewriteframes.ts

@@ -57,6 +57,10 @@ export function createReactNativeRewriteFrames(): RewriteFrames {
       }
 
       const appPrefix = 'app://';
+      // https://github.com/getsentry/sentry-react-native/issues/3348
+      if (frame.filename === '/InternalBytecode.js') {
+        frame.in_app = false;
+      }
       // We always want to have a triple slash
       frame.filename =
         frame.filename.indexOf('/') === 0 ? `${appPrefix}${frame.filename}` : `${appPrefix}/${frame.filename}`;

test/integrations/rewriteframes.test.ts

@@ -859,4 +859,37 @@ describe('RewriteFrames', () => {
       },
     });
   });
+
+  it('InternalBytecode should be flaged as not InApp', async () => {
+    mockFunction(isHermesEnabled).mockReturnValue(true);
+
+    const IOS_REACT_NATIVE_HERMES = {
+      message: 'Error: lets throw!',
+      name: 'Error',
+      stack:
+        'at anonymous (/Users/username/react-native/sdks/hermes/build_iphonesimulator/lib/InternalBytecode/InternalBytecode.js:139:27)',
+    };
+
+    const exception = await exceptionFromError(IOS_REACT_NATIVE_HERMES);
+
+    expect(exception).toEqual({
+      value: 'Error: lets throw!',
+      type: 'Error',
+      mechanism: {
+        handled: true,
+        type: 'generic',
+      },
+      stacktrace: {
+        frames: [
+          {
+            filename: 'app:///InternalBytecode.js',
+            function: 'anonymous',
+            lineno: 139,
+            colno: 27,
+            in_app: false,
+          },
+        ],
+      },
+    });
+  });
 });