ref(build): Align package.json build scripts with JS SDK scripts (getsentry/sentry-replay#320)

Align the scripts in `package.json` with the scripts we use in the JS SDK repo. Doing this now doesn't hurt us and it's one less thing to worry about when migrating. Admittedly, the `build:*` scripts look a little scary now but this is just because I wanted to preserve the separate build scripts for worker and core. Therefore, we'll end up with a few more scripts than we have in most other SDKs (s.g. [Svelte](https://github.com/getsentry/sentry-javascript/blob/d22380097b70bc503613cebbf2b914175174221c/packages/svelte/package.jsongetsentry/sentry-replay#L34-L42)).

In the monorepo, we don't set env variables for prod vs. dev builds. I still need to think about what we're going to do about them but we can make these changes iteratively. 

Here's a quick overview of the most important changes. All these scripts are used in the monorepo, not just by us but also by CI, so we need to align them:
* `build` now serves as a "default" build command. I renamed what used to be the `build` script to `build:rollup` (which is also consistent with the monorepo). I left `build:prod` from devEx reasons for now but we can remove it after migrating.
* `build:dev` basically remains unchanged as it's already mostly consistent with the monorepo 
* Added `build:watch` and `build:dev:watch` to re-build on file changes. I think previously, the `--watch` param was only applied to `build-core` but I figured we could also watch for worker changes. Which is why I added the `build:all:watch` sub-script to do this. I left the `dev` script for now to not disturb devEx but after migrating we can remove it, as it isn't used in the monorepo at all.
* Added `build:npm` script to pack a tarball when publishing. In the monorepo, we use a script that makes some changes to `package.json` before packing. We'll need to add this here as well but not now, as it requires more changes. Also, I changed the "Build" GH workflow to use this command instead of `npm pack`.
* Added `lint` and `fix` scripts that use eslint and prettier. I left the original commands eslint untouched, just changed the script names around it (and added the prettier stuff)

I added the `npm-run-all` dev dependency for the time being, to get the `run-p/s` commands we use in the monorepo. After migrating, we can remove the dependency again from this package because it's registered as a top-level dependency in the monorepo.

Author: Lms24

.github/workflows/build.yml

@@ -23,7 +23,7 @@ jobs:
           yarn test
 
       - run: |
-          yarn pack
+          yarn build:npm
 
       - uses: actions/[email protected]
         with:

package.json

@@ -7,19 +7,31 @@
   "types": "dist/types/src/index.d.ts",
   "sideEffects": false,
   "scripts": {
-    "bootstrap": "yarn && cd demo && yarn",
-    "build": "yarn rollup --configPlugin @rollup/plugin-typescript",
-    "build:all": "yarn clean && yarn build:worker && yarn build:core",
+    "bootstrap": "yarn && cd demo && yarn #TODO: change after migration",
+    "build": "NODE_ENV=production yarn build:all",
     "build:dev": "NODE_ENV=development yarn build:all",
-    "build:prod": "NODE_ENV=production yarn build:all",
-    "build:core": "yarn build --config config/rollup.config.core.ts",
-    "build:worker": "yarn build --config config/rollup.config.worker.ts",
-    "clean": "yarn rimraf dist",
-    "dev": "yarn build:dev --watch",
-    "lint": "yarn eslint . --ext .js,.jsx,.ts,.tsx",
-    "prepack": "yarn build:prod",
+    "build:all": "run-s clean build:worker build:core",
+    "build:core": "yarn build:rollup --config config/rollup.config.core.ts",
+    "build:worker": "yarn build:rollup --config config/rollup.config.worker.ts",
+    "build:rollup": "rollup --configPlugin @rollup/plugin-typescript",
+    "build:watch": "NODE_ENV=production yarn build:all:watch",
+    "build:dev:watch": "NODE_ENV=development yarn build:all:watch",
+    "build:all:watch": "yarn clean && run-p \"build:worker --watch\" \"build:core --watch\"",
+    "build:npm": "yarn pack #TODO: use JS sdk prepack script after migration",
+    "circularDepCheck": "#TODO comment in after migration: madge --circular src/index.ts",
+    "clean": "rimraf dist sentry-replay-*.tgz",
+    "fix": "run-s fix:eslint fix:prettier",
+    "fix:eslint": "eslint . --format stylish --fix",
+    "fix:prettier": "prettier --write \"{src,test,scripts,worker}/**/*.ts\"",
+    "lint": "run-s lint:prettier lint:eslint",
+    "lint:eslint": "eslint . --ext .js,.jsx,.ts,.tsx #TODO: we might want to use eslintcache after migration",
+    "lint:prettier": "prettier --check \"{src,test,scripts,worker}/**/*.ts\"",
+    "test": "jest",
+    "test:watch": "jest --watch",
     "start:demo": "yarn build:dev && cd demo && yarn start",
-    "test": "yarn jest"
+    "prepack": "yarn build #TODO: remove after migration",
+    "build:prod": "yarn build #TODO remove, we don't need this anymore after migration",
+    "dev": "yarn build:dev:watch #TODO remove, we don't need this anymore after migration"
   },
   "files": [
     "dist"
@@ -56,6 +68,7 @@
     "jest": "27.5.1",
     "jest-environment-jsdom": "27.5.1",
     "jsdom-worker": "^0.2.1",
+    "npm-run-all": "^4.1.5",
     "pako": "^2.0.4",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.2",