frames and abs_path

Author: mydea

dev-packages/node-integration-tests/suites/contextLines/filename-with-spaces/test.ts

@@ -18,7 +18,8 @@ describe('ContextLines integration in ESM', () => {
                 stacktrace: {
                   frames: expect.arrayContaining([
                     {
-                      filename: 'app:///suites/contextLines/filename-with-spaces/scenario with space.mjs',
+                      filename: '/suites/contextLines/filename-with-spaces/scenario with space.mjs',
+                      abs_path: expect.stringContaining('/dev-packages/node-integration-tests/suites/contextLines/filename-with-spaces/scenario with space.mjs'),
                       context_line: "Sentry.captureException(new Error('Test Error'));",
                       pre_context: ["import * as Sentry from '@sentry/node';", ''],
                       post_context: ['', '// some more post context'],
@@ -54,7 +55,8 @@ describe('ContextLines integration in CJS', () => {
                 stacktrace: {
                   frames: expect.arrayContaining([
                     {
-                      filename: 'app:///suites/contextLines/filename-with-spaces/scenario with space.cjs',
+                      filename: '/suites/contextLines/filename-with-spaces/scenario with space.cjs',
+                      abs_path: expect.stringContaining('/dev-packages/node-integration-tests/suites/contextLines/filename-with-spaces/scenario with space.cjs'),
                       context_line: "Sentry.captureException(new Error('Test Error'));",
                       pre_context: [
                         '',

dev-packages/node-integration-tests/suites/public-api/captureException/catched-error/test.ts

@@ -21,7 +21,8 @@ test('should work inside catch block', async () => {
               stacktrace: {
                 frames: expect.arrayContaining([
                   expect.objectContaining({
-                    filename: 'app:///suites/public-api/captureException/catched-error/scenario.ts',
+                    filename: '/suites/public-api/captureException/catched-error/scenario.ts',
+                    abs_path: expect.stringContaining('/dev-packages/node-integration-tests/suites/public-api/captureException/catched-error/scenario.ts'),
                     context_line: "  throw new Error('catched_error');",
                     pre_context: [
                       'Sentry.init({',

dev-packages/node-integration-tests/suites/public-api/captureException/simple-error/test.ts

@@ -21,7 +21,8 @@ test('should capture a simple error with message', async () => {
               stacktrace: {
                 frames: expect.arrayContaining([
                   expect.objectContaining({
-                    filename: 'app:///suites/public-api/captureException/simple-error/scenario.ts',
+                    filename: '/suites/public-api/captureException/simple-error/scenario.ts',
+                    abs_path: expect.stringContaining('/dev-packages/node-integration-tests/suites/public-api/captureException/simple-error/scenario.ts'),
                   }),
                 ]),
               },

packages/core/src/integrations/rewriteframes.ts

@@ -137,13 +137,15 @@ export function generateIteratee({
       }
     } else {
       if (isWindowsFrame || startsWithSlash) {
+        const abs_path = frame.filename;
         const filename = isWindowsFrame
-          ? frame.filename
+          ? abs_path
               .replace(/^[a-zA-Z]:/, '') // remove Windows-style prefix
               .replace(/\\/g, '/') // replace all `\\` instances with `/`
-          : frame.filename;
+          : abs_path;
         const base = root ? relative(root, filename) : basename(filename);
         frame.filename = `${prefix}${base}`;
+        frame.abs_path = abs_path;
       }
     }
 

packages/node/src/sdk/index.ts

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