fix(other): CHECKOUT-6524 Relax the error filtering rule to report errors if some of the frames in the stack trace are thrown from the app code

Author: davidchin

packages/core/src/app/common/error/SentryErrorLogger.spec.ts

@@ -107,7 +107,7 @@ describe('SentryErrorLogger', () => {
         expect(clientOptions.beforeSend!(event, hint)).toBeNull();
     });
 
-    it('logs exception event if all frames in stacktrace reference app file', () => {
+    it('logs exception event if some frames in stacktrace reference app file', () => {
         new SentryErrorLogger(config, options);
 
         const clientOptions: BrowserOptions = createClient.mock.calls[0][0];
@@ -117,8 +117,9 @@ describe('SentryErrorLogger', () => {
                     {
                         stacktrace: {
                             frames: [
-                                { filename: 'app:///js/app-123.js' },
-                                { filename: 'app:///js/app-456.js' },
+                                { filename: 'app:///js/app-123.js', in_app: true },
+                                { filename: '<anonymous>', in_app: true },
+                                { filename: 'app:///js/app-456.js', in_app: true },
                             ],
                         },
                         type: 'Error',

packages/core/src/app/common/error/SentryErrorLogger.ts

@@ -157,8 +157,8 @@ export default class SentryErrorLogger implements ErrorLogger {
                 return false;
             }
 
-            return exception.stacktrace.frames.every((frame) =>
-                frame.filename?.startsWith(FILENAME_PREFIX),
+            return exception.stacktrace.frames.some((frame) =>
+                frame.filename?.startsWith(FILENAME_PREFIX) && frame.in_app,
             );
         });
     }