fixes

Author: mydea

packages/core/src/integrations/rewriteframes.ts

@@ -54,7 +54,7 @@ interface RewriteFramesOptions {
  */
 export const rewriteFramesIntegration = defineIntegration((options: RewriteFramesOptions = {}) => {
   const root = options.root;
-  const prefix = options.prefix || 'app:///';
+  const prefix = options.prefix ?? 'app:///';
 
   const isBrowser = 'window' in GLOBAL_OBJ && !!GLOBAL_OBJ.window;
 

packages/core/test/lib/integrations/rewriteframes.test.ts

@@ -139,6 +139,16 @@ describe('RewriteFrames', () => {
       expect(event.exception!.values![0]?.stacktrace!.frames![0]?.filename).toEqual('foobar/file1.js');
       expect(event.exception!.values![0]?.stacktrace!.frames![1]?.filename).toEqual('foobar/file2.js');
     });
+
+    it('handles empty string as prefix', () => {
+      rewriteFrames = rewriteFramesIntegration({
+        prefix: '',
+      });
+
+      const event = rewriteFrames.processEvent?.(exceptionEvent, {}, {} as any) as Event;
+      expect(event.exception!.values![0]?.stacktrace!.frames![0]?.filename).toEqual('file1.js');
+      expect(event.exception!.values![0]?.stacktrace!.frames![1]?.filename).toEqual('file2.js');
+    });
   });
 
   describe('default iteratee appends basename to `app:///` if frame starts with Windows path prefix', () => {

packages/node/src/sdk/index.ts

@@ -73,7 +73,7 @@ export function getDefaultIntegrationsWithoutPerformance(): Integration[] {
     // This has to run after the other integrations, because it rewrites the frames
     rewriteFramesIntegration({
       root: process.cwd(),
-      prefix: '',
+      prefix: '/',
     }),
     ...getCjsOnlyIntegrations(),
   ];