Apply suggestions from code review

Co-authored-by: Yun Feng <[email protected]>

package.json

@@ -40,7 +40,7 @@
     "test": "yarn run concurrently --success=all -r -m=1 'yarn workspaces-to-typescript-project-references --check' 'yarn turbo run test'",
     "test:watch": "yarn turbo run test:watch",
     "test:update": "yarn turbo run test:update",
-    "reformat:all": "yarn turbo run reformat",
+    "format": "yarn prettier --write '**/*.{ts,md}'",
     "dev": "yarn turbo run dev",
     "repl": "cd packages/rrweb && npm run repl",
     "live-stream": "cd packages/rrweb && yarn live-stream",

packages/rrdom/package.json

@@ -19,7 +19,6 @@
     "url": "git+https://github.com/rrweb-io/rrweb.git"
   },
   "scripts": {
-    "reformat": "yarn prettier --write '**/*.{ts,md}'",
     "dev": "rollup -c -w",
     "bundle": "rollup --config",
     "bundle:es-only": "cross-env ES_ONLY=true rollup --config",

packages/rrvideo/package.json

@@ -12,7 +12,7 @@
   "types": "build/index.d.ts",
   "scripts": {
     "build": "tsc",
-    "prepublish": "npm run build"
+    "prepublish": "yarn build"
   },
   "author": "[email protected]",
   "license": "MIT",

packages/rrweb-player/package.json

@@ -28,12 +28,11 @@
   },
   "scripts": {
     "build": "rollup -c",
-    "reformat": "yarn prettier --write '**/*.{ts,md}'",
     "dev": "rollup -c -w",
-    "prepublishOnly": "npm run build",
+    "prepublishOnly": "yarn build",
     "start": "sirv public",
     "validate": "svelte-check",
-    "prepublish": "npm run build",
+    "prepublish": "yarn build",
     "lint": "yarn eslint src/**/*.ts"
   },
   "description": "rrweb's replayer UI",

packages/rrweb-snapshot/package.json

@@ -10,10 +10,9 @@
     "test:update": "jest --updateSnapshot",
     "bundle": "rollup --config",
     "bundle:es-only": "cross-env ES_ONLY=true rollup --config",
-    "reformat": "yarn prettier --write '**/*.{ts,md}'",
-    "dev": "npm run bundle:es-only --watch",
+    "dev": "yarn bundle:es-only --watch",
     "typings": "tsc -d --declarationDir typings",
-    "prepublish": "npm run typings && npm run bundle",
+    "prepublish": "yarn typings && yarn bundle",
     "lint": "yarn eslint src"
   },
   "type": "module",

packages/rrweb/package.json

@@ -6,16 +6,15 @@
     "prepare": "npm run prepack",
     "prepack": "npm run bundle",
     "retest": "jest --testPathIgnorePatterns test/benchmark",
-    "build-and-test": "npm run bundle:browser && npm run retest",
-    "test": "npm run test:headless",
-    "test:headless": "PUPPETEER_HEADLESS=true npm run build-and-test",
-    "test:headful": "PUPPETEER_HEADLESS=false npm run build-and-test",
-    "test:watch": "npm run test:headless -- --watch",
-    "test:update": "npm run test:headless -- --updateSnapshot",
-    "repl": "npm run bundle:browser && node scripts/repl.js",
-    "live-stream": "npm run bundle:browser && node scripts/stream.js",
-    "dev": "npm run bundle:browser --watch",
-    "reformat": "yarn prettier --write '**/*.{ts,md}'",
+    "build-and-test": "yarn bundle:browser && yarn retest",
+    "test:headless": "PUPPETEER_HEADLESS=true yarn build-and-test",
+    "test:headful": "PUPPETEER_HEADLESS=false yarn build-and-test",
+    "test": "yarn test:headless",
+    "test:watch": "yarn test:headless -- --watch",
+    "test:update": "yarn test:headless -- --updateSnapshot",
+    "repl": "yarn bundle:browser && node scripts/repl.js",
+    "live-stream": "yarn bundle:browser && node scripts/stream.js",
+    "dev": "yarn bundle:browser --watch",
     "bundle:browser": "cross-env BROWSER_ONLY=true rollup --config",
     "bundle": "rollup --config",
     "typings": "tsc -d --declarationDir typings",

packages/types/package.json

@@ -9,7 +9,6 @@
     "@rrweb/types"
   ],
   "scripts": {
-    "reformat": "yarn prettier --write '**/*.{ts,md}'",
     "dev": "vite",
     "build": "tsc -noEmit && vite build",
     "check-types": "tsc -noEmit",

turbo.json

@@ -9,7 +9,6 @@
     "test": {},
     "test:watch": {},
     "test:update": {},
-    "reformat": {},
     "dev": {},
     "lint": {},
     "check-types": {}