Merge branch 'master' of https://github.com/getsentry/sentry-javascript into onur/test-bundles

Author: onurtemizkan

.size-limit.js

@@ -44,13 +44,6 @@ module.exports = [
     gzip: true,
     limit: '100 KB',
   },
-  {
-    name: '@sentry/react - Webpack (gzipped + minified)',
-    path: 'packages/react/esm/index.js',
-    import: '{ init }',
-    gzip: true,
-    limit: '100 KB',
-  },
   {
     name: '@sentry/nextjs Client - Webpack (gzipped + minified)',
     path: 'packages/nextjs/esm/index.client.js',

lerna.json

@@ -1,6 +1,6 @@
 {
   "lerna": "3.4.0",
-  "version": "6.17.6",
+  "version": "6.17.7",
   "packages": "packages/*",
   "npmClient": "yarn",
   "useWorkspaces": true

package.json

@@ -51,6 +51,7 @@
   ],
   "devDependencies": {
     "@google-cloud/storage": "^5.7.0",
+    "@rollup/plugin-replace": "^3.0.1",
     "@size-limit/preset-small-lib": "^4.5.5",
     "@strictsoftware/typedoc-plugin-monorepo": "^0.3.1",
     "@types/chai": "^4.1.3",

packages/angular/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/angular",
-  "version": "6.17.6",
+  "version": "6.17.7",
   "description": "Official Sentry SDK for Angular",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular",
@@ -21,9 +21,9 @@
     "@angular/router": "10.x || 11.x || 12.x || 13.x"
   },
   "dependencies": {
-    "@sentry/browser": "6.17.6",
-    "@sentry/types": "6.17.6",
-    "@sentry/utils": "6.17.6",
+    "@sentry/browser": "6.17.7",
+    "@sentry/types": "6.17.7",
+    "@sentry/utils": "6.17.7",
     "rxjs": "^6.6.0",
     "tslib": "^1.9.3"
   },

packages/browser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/browser",
-  "version": "6.17.6",
+  "version": "6.17.7",
   "description": "Official Sentry SDK for browsers",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/browser",
@@ -16,9 +16,9 @@
     "access": "public"
   },
   "dependencies": {
-    "@sentry/core": "6.17.6",
-    "@sentry/types": "6.17.6",
-    "@sentry/utils": "6.17.6",
+    "@sentry/core": "6.17.7",
+    "@sentry/types": "6.17.7",
+    "@sentry/utils": "6.17.7",
     "tslib": "^1.9.3"
   },
   "devDependencies": {

packages/browser/rollup.config.js

@@ -3,6 +3,7 @@ import typescript from 'rollup-plugin-typescript2';
 import license from 'rollup-plugin-license';
 import resolve from 'rollup-plugin-node-resolve';
 import commonjs from 'rollup-plugin-commonjs';
+import replace from '@rollup/plugin-replace';
 
 const commitHash = require('child_process')
   .execSync('git rev-parse --short HEAD', { encoding: 'utf-8' })
@@ -13,7 +14,6 @@ const terserInstance = terser({
     // Tell env.ts that we're building a browser bundle and that we do not
     // want to have unnecessary debug functionality.
     global_defs: {
-      __SENTRY_BROWSER_BUNDLE__: true,
       __SENTRY_NO_DEBUG__: false,
     },
   },
@@ -59,6 +59,16 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
+  replace({
+    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
+    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
+    // syntax error)
+    preventAssignment: true,
+    // the replacements to make
+    values: {
+      __SENTRY_BROWSER_BUNDLE__: true,
+    },
+  }),
   resolve({
     mainFields: ['module'],
   }),
@@ -81,6 +91,7 @@ const bundleConfig = {
       banner: `/*! @sentry/browser <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/sentry-javascript */`,
     }),
   ],
+  treeshake: 'smallest',
 };
 
 export default [

packages/browser/src/parsers.ts

@@ -1,7 +1,7 @@
 import { Event, Exception, StackFrame } from '@sentry/types';
 import { extractExceptionKeysForMessage, isEvent, normalizeToSize } from '@sentry/utils';
 
-import { computeStackTrace, StackFrame as TraceKitStackFrame, StackTrace as TraceKitStackTrace } from './tracekit';
+import { computeStackTrace, StackTrace as TraceKitStackTrace } from './tracekit';
 
 const STACKTRACE_LIMIT = 50;
 
@@ -80,15 +80,15 @@ export function eventFromStacktrace(stacktrace: TraceKitStackTrace): Event {
 /**
  * @hidden
  */
-export function prepareFramesForEvent(stack: TraceKitStackFrame[]): StackFrame[] {
-  if (!stack || !stack.length) {
+export function prepareFramesForEvent(stack: StackFrame[]): StackFrame[] {
+  if (!stack.length) {
     return [];
   }
 
   let localStack = stack;
 
-  const firstFrameFunction = localStack[0].func || '';
-  const lastFrameFunction = localStack[localStack.length - 1].func || '';
+  const firstFrameFunction = localStack[0].function || '';
+  const lastFrameFunction = localStack[localStack.length - 1].function || '';
 
   // If stack starts with one of our API calls, remove it (starts, meaning it's the top of the stack - aka last call)
   if (firstFrameFunction.indexOf('captureMessage') !== -1 || firstFrameFunction.indexOf('captureException') !== -1) {
@@ -103,14 +103,12 @@ export function prepareFramesForEvent(stack: TraceKitStackFrame[]): StackFrame[]
   // The frame where the crash happened, should be the last entry in the array
   return localStack
     .slice(0, STACKTRACE_LIMIT)
-    .map(
-      (frame: TraceKitStackFrame): StackFrame => ({
-        colno: frame.column === null ? undefined : frame.column,
-        filename: frame.url || localStack[0].url,
-        function: frame.func || '?',
-        in_app: true,
-        lineno: frame.line === null ? undefined : frame.line,
-      }),
-    )
+    .map(frame => ({
+      filename: frame.filename || localStack[0].filename,
+      function: frame.function || '?',
+      lineno: frame.lineno,
+      colno: frame.colno,
+      in_app: true,
+    }))
     .reverse();
 }