chore(remix): Update package to 7.x structure (#5266)

This is a follow-up to #5256 and #5264, which added a skeleton remix SDK to the repo. The template that was used to create the new package had parts of the 6.x package structure and configuration still in it. This fixes that by bringing everything up to date to match our other packages.

Author: lobsterkatie

packages/remix/package.json

@@ -21,12 +21,12 @@
     "@sentry/node": "7.1.1",
     "@sentry/react": "7.1.1",
     "@sentry/tracing": "7.1.1",
+    "@sentry/types": "7.1.1",
     "@sentry/utils": "7.1.1",
     "@sentry/webpack-plugin": "1.18.9",
     "tslib": "^1.9.3"
   },
   "devDependencies": {
-    "@sentry/types": "7.1.1",
     "@types/webpack": "^4.41.31",
     "@remix-run/node": "^1.4.3",
     "@remix-run/react": "^1.4.3"
@@ -43,27 +43,25 @@
     }
   },
   "scripts": {
-    "build": "run-p build:rollup",
+    "build": "run-p build:rollup build:types",
     "build:dev": "run-s build",
-    "build:esm": "tsc -p tsconfig.esm.json",
     "build:rollup": "rollup -c rollup.npm.config.js",
     "build:types": "tsc -p tsconfig.types.json",
-    "build:watch": "run-p build:esm:watch",
+    "build:watch": "run-p build:rollup:watch build:types:watch",
     "build:dev:watch": "run-s build:watch",
-    "build:esm:watch": "tsc -p tsconfig.esm.json --watch",
     "build:rollup:watch": "rollup -c rollup.npm.config.js --watch",
     "build:types:watch": "tsc -p tsconfig.types.json --watch",
     "build:npm": "ts-node ../../scripts/prepack.ts && npm pack ./build",
-    "clean": "rimraf build coverage",
+    "circularDepCheck": "madge --circular src/index.ts",
+    "clean": "rimraf build coverage sentry-remix-*.tgz",
     "fix": "run-s fix:eslint fix:prettier",
     "fix:eslint": "eslint . --format stylish --fix",
     "fix:prettier": "prettier --write \"{src,test,scripts}/**/*.ts\"",
-    "link:yarn": "yarn link",
     "lint": "run-s lint:prettier lint:eslint",
     "lint:eslint": "eslint . --cache --cache-location '../../eslintcache/' --format stylish",
     "lint:prettier": "prettier --check \"{src,test,scripts}/**/*.ts\"",
     "test": "run-s test:unit",
-    "test:unit": "jest --passWithNoTests",
+    "test:unit": "jest",
     "test:watch": "jest --watch"
   },
   "volta": {

packages/remix/src/flags.ts

@@ -2,8 +2,6 @@ import { Transaction, TransactionContext } from '@sentry/types';
 import { getGlobalObject, logger } from '@sentry/utils';
 import * as React from 'react';
 
-import { IS_DEBUG_BUILD } from '../flags';
-
 const DEFAULT_TAGS = {
   'routing.instrumentation': 'remix-router',
 } as const;
@@ -85,7 +83,7 @@ export function remixRouterInstrumentation(useEffect: UseEffect, useLocation: Us
 export function withSentryRouteTracing<P extends Record<string, unknown>, R extends React.FC<P>>(OrigApp: R): R {
   // Early return when any of the required functions is not available.
   if (!_useEffect || !_useLocation || !_useMatches || !_customStartTransaction) {
-    IS_DEBUG_BUILD && logger.warn('Remix SDK was unable to wrap your root because of one or more missing parameters.');
+    __DEBUG_BUILD__ && logger.warn('Remix SDK was unable to wrap your root because of one or more missing parameters.');
 
     // @ts-ignore Setting more specific React Component typing for `R` generic above
     // will break advanced type inference done by react router params

packages/remix/src/performance/client.tsx

@@ -0,0 +1,8 @@
+// TODO Once https://github.com/microsoft/TypeScript/issues/33094 is done (if it ever is), this file can disappear, as
+// it's purely a placeholder to satisfy VSCode.
+
+{
+  "extends": "../tsconfig.test.json",
+
+  "include": ["./**/*"]
+}