Merge branch 'master' into abhi-invoke-client

Author: AbhiPrasad

.eslintrc.js

@@ -31,5 +31,14 @@ module.exports = {
         'jsdoc/require-jsdoc': 'off',
       },
     },
+    {
+      files: ["scenarios/**"],
+      parserOptions: {
+        sourceType: "module",
+      },
+      rules: {
+        "no-console": "off",
+      },
+    },
   ],
 };

.github/workflows/build.yml

@@ -231,17 +231,9 @@ jobs:
         with:
           path: ${{ env.CACHED_BUILD_PATHS }}
           key: ${{ env.BUILD_CACHE_KEY }}
-      - name: Check changed files
-        id: changed-files-specific
-        uses: tj-actions/[email protected]
-        with:
-          files: .*packages\/ember($|/.*)
-      # Only run ember tests if the files above have changed
       - name: Run Ember tests
-        if: steps.changed-files-specific.outputs.any_changed == 'true' || github.event_name == 'push'
         run: yarn test --scope=@sentry/ember
       - name: Compute test coverage
-        if: steps.changed-files-specific.outputs.any_changed == 'true' || github.event_name == 'push'
         uses: codecov/codecov-action@v1
 
   job_artifacts:

.gitignore

@@ -11,6 +11,7 @@ coverage/
 scratch/
 *.pyc
 *.tsbuildinfo
+scenarios/*/dist/
 
 # logs
 yarn-error.log

.size-limit.js

@@ -17,6 +17,12 @@ module.exports = [
     import: '{ init }',
     limit: '100 KB',
   },
+  {
+    name: '@sentry/nextjs Client - Webpack',
+    path: 'packages/nextjs/esm/index.client.js',
+    import: '{ init }',
+    limit: '100 KB',
+  },
   {
     name: '@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)',
     path: 'packages/tracing/build/bundle.tracing.min.js',

packages/ember/config/ember-try.js

@@ -3,26 +3,47 @@
 const getChannelURL = require('ember-source-channel-url');
 const { embroiderSafe } = require('@embroider/test-setup');
 
+/**
+ * Pick which versions of ember against which to test based on whether the tests are running locally, as part of a PR,
+ * or when merging to `master` or creating a release.
+ *
+ * @returns The versions which should be tested, along with their respective config
+ */
 module.exports = async function() {
-  return {
-    useYarn: true,
-    scenarios: [
-      {
-        name: 'ember-lts-3.20',
-        npm: {
-          devDependencies: {
-            'ember-source': '~3.20.0',
-          },
+  // whenever and wherever we test, we want to at least test against the latest version of ember
+  let scenarios = [
+    {
+      name: 'ember-release',
+      npm: {
+        devDependencies: {
+          'ember-source': await getChannelURL('release'),
         },
       },
+    },
+  ];
+
+  // in CI we add a few more tests - LTS and embroider (which is an ember compiler)
+  if (process.env.GITHUB_ACTIONS) {
+    scenarios = scenarios.concat([
       {
-        name: 'ember-release',
+        name: 'ember-lts-3.20',
         npm: {
           devDependencies: {
-            'ember-source': await getChannelURL('release'),
+            'ember-source': '~3.24.0',
           },
         },
       },
+      embroiderSafe(),
+    ]);
+  }
+
+  // finally, just to be extra thorough when merging to master and releasing, we add the beta channel and ember
+  // "classic" (a legacy version which was last current in late 2019)
+  if (
+    process.env.GITHUB_EVENT_NAME === 'push' &&
+    (process.env.GITHUB_HEAD_REF === 'master' || process.env.GITHUB_HEAD_REF.startsWith('release'))
+  ) {
+    scenarios = scenarios.concat([
       {
         name: 'ember-beta',
         npm: {
@@ -32,7 +53,6 @@ module.exports = async function() {
         },
         allowedToFail: true,
       },
-      embroiderSafe(),
       {
         name: 'ember-classic',
         env: {
@@ -48,6 +68,11 @@ module.exports = async function() {
           },
         },
       },
-    ],
+    ]);
+  }
+
+  return {
+    useYarn: true,
+    scenarios,
   };
 };

packages/ember/package.json

@@ -26,9 +26,8 @@
     "prepublishOnly": "ember ts:precompile",
     "postpublish": "ember ts:clean",
     "start": "ember serve",
-    "test": "node ./scripts/run_tests.js",
-    "test:ember": "ember test",
-    "test:ember-compatibility": "ember try:each"
+    "test": "ember try:each",
+    "test:all": "node ./scripts/run-CI-tests.js"
   },
   "dependencies": {
     "@embroider/macros": "~0.47.2",

packages/ember/scripts/run-CI-tests.js

@@ -0,0 +1,17 @@
+/*eslint-env node*/
+const { spawnSync } = require('child_process');
+
+console.log('Mimicking the CI environment in order to run tests against multiple versions of Ember');
+
+const result = spawnSync('yarn test', {
+  shell: true,
+  stdio: 'inherit',
+  env: {
+    ...process.env,
+    GITHUB_ACTIONS: true,
+    GITHUB_EVENT_NAME: 'push',
+    GITHUB_HEAD_REF: 'master',
+  },
+});
+
+process.exit(result.status);

packages/ember/scripts/run_tests.js

@@ -186,6 +186,13 @@ module.exports = {
         ecmaVersion: 2018,
       },
     },
+    {
+      // Configuration for jsx and tsx files
+      files: ['*.tsx', '*.jsx', '*.test.tsx', '*.test.jsx'],
+      parserOptions: {
+        jsx: true,
+      },
+    },
   ],
 
   rules: {

packages/eslint-config-sdk/src/index.js

@@ -45,7 +45,7 @@
     "build:dev": "run-s build:es5 build:esm",
     "build:es5": "tsc -p tsconfig.build.json",
     "build:esm": "tsc -p tsconfig.esm.json",
-    "build:watch": "run-p build:es5:watch build:esm:watch",
+    "build:watch": "run-p build:es5:watch build:esm:watch build:bundle:watch",
     "build:bundle:watch": "rollup --config --watch",
     "build:dev:watch": "run-p build:es5:watch build:esm:watch",
     "build:es5:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",

packages/tracing/package.json

@@ -0,0 +1,7 @@
+import { init, captureException } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+});
+
+captureException(new Error("error here!"));

scenarios/browser/basic-capture-exception/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-capture-exception",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-capture-exception/package.json

@@ -0,0 +1,7 @@
+import { init, captureMessage } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+});
+
+captureMessage("this is a message");

scenarios/browser/basic-capture-message/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-capture-message",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-capture-message/package.json

@@ -0,0 +1,23 @@
+import { init } from "@sentry/browser";
+
+class CustomIntegration {
+  static id = 'CustomIntegration';
+
+  name = CustomIntegration.id;
+  options = undefined;
+
+  constructor(options) {
+    this.options = options;
+  }
+
+  setupOnce(addGlobalEventProcessor, getCurrentHub) {
+    addGlobalEventProcessor(event => event);
+    const hub = getCurrentHub();
+    hub.captureMessage(options.name);
+  }
+}
+
+init({
+  dsn: "https://[email protected]/0000000",
+  integrations: [new CustomIntegration()],
+});

scenarios/browser/basic-custom-integration/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-custom-integration",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-custom-integration/package.json

@@ -0,0 +1,22 @@
+import { init, Transports } from "@sentry/browser";
+
+class CustomTransport extends Transports.BaseTransport {
+  constructor(options) {
+    super(options);
+  }
+
+  sendEvent(event) {
+    console.log("Sending Event");
+    return super.sendEvent(event);
+  }
+
+  sendSession(session) {
+    console.log("Sending Session");
+    return super.sendSession(session);
+  }
+}
+
+init({
+  dsn: "https://[email protected]/0000000",
+  transport: CustomTransport
+});

scenarios/browser/basic-custom-transport/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-custom-transport",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-custom-transport/package.json

@@ -0,0 +1,6 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  sendClientReports: false,
+});

scenarios/browser/basic-no-client-reports/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-no-client-reports",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-no-client-reports/package.json

@@ -0,0 +1,6 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  defaultIntegrations: false,
+});

scenarios/browser/basic-no-default-integrations/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-no-default-integrations",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-no-default-integrations/package.json

@@ -0,0 +1,3 @@
+import { init } from "@sentry/browser";
+
+init();

scenarios/browser/basic-no-dsn/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-no-dsn",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-no-dsn/package.json

@@ -0,0 +1,7 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  release: "[email protected]",
+  autoSessionTracking: false,
+});

scenarios/browser/basic-no-sessions/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-no-sessions",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-no-sessions/package.json

@@ -0,0 +1,6 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  debug: true,
+});

scenarios/browser/basic-with-debug/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-with-debug",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-with-debug/package.json

@@ -0,0 +1,6 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  release: "[email protected]",
+});

scenarios/browser/basic-with-sessions/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-with-sessions",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-with-sessions/package.json

@@ -0,0 +1,6 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+  tunnel: "/errors",
+});

scenarios/browser/basic-with-tunnel/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic-with-tunnel",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic-with-tunnel/package.json

@@ -0,0 +1,5 @@
+import { init } from "@sentry/browser";
+
+init({
+  dsn: "https://[email protected]/0000000",
+});

scenarios/browser/basic/index.js

@@ -0,0 +1,5 @@
+{
+  "name": "basic",
+  "private": true,
+  "version": "0.0.0"
+}

scenarios/browser/basic/package.json

@@ -0,0 +1,5 @@
+import { init } from "@sentry/nextjs/esm/index.client";
+
+init({
+  dsn: "https://[email protected]/0000000",
+});