feat: Remove esm rewrite script, expose module in package.json (#1930)

* feat: Remove esm rewrite script, expose module

* feat: Add changelog

Author: HazAT

.gitignore

@@ -5,7 +5,7 @@ packages/*/package-lock.json
 # build and test
 build/
 packages/*/dist/
-/packages/*/esm
+packages/*/esm/
 coverage/
 scratch/
 *.pyc

CHANGELOG.md

@@ -29,10 +29,12 @@ since we removed some methods from the public API and removed some classes from
 - **breaking** [utils] ref: Rename `limitObjectDepthToSize` to `normalizeToSize` and rewrite its internals
 - **breaking** [utils] ref: Rename `safeNormalize` to `normalize` and rewrite its internals
 - **breaking** [utils] ref: Remove `serialize`, `deserialize`, `clone` and `serializeObject` functions
-- **breaking** [utils] ref: Rewrite normalization functions by removing most of them and leaving just `normalize` and `normalizeToSize`
+- **breaking** [utils] ref: Rewrite normalization functions by removing most of them and leaving just `normalize` and
+  `normalizeToSize`
 - **breaking** [core] ref: Extract all pluggable integrations into a separate `@sentry/integrations` package
 - **breaking** [core] ref: Move `extraErrorData` integration to `@sentry/integrations` package
 - [core] feat: Add `maxValueLength` option to adjust max string length for values, default is 250.
+- **breaking** [all] ref: Expose `module` in `package.json` as entry point for esm builds.
 
 ## 4.6.4
 

packages/browser/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"
@@ -55,9 +56,7 @@
   "scripts": {
     "build": "run-p build:esm build:es5",
     "build:es5": "rollup --config",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
+    "build:esm": "tsc -p tsconfig.esm.json",
     "build:watch": "rollup --config --watch",
     "clean": "rimraf dist coverage .rpt2_cache build",
     "lint": "run-s lint:prettier lint:tslint",

packages/core/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"
@@ -33,9 +34,7 @@
   "scripts": {
     "build": "run-p build:es5 build:esm",
     "build:es5": "tsc -p tsconfig.build.json",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
+    "build:esm": "tsc -p tsconfig.esm.json",
     "build:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",
     "clean": "rimraf dist coverage",
     "lint": "run-s lint:prettier lint:tslint",

packages/hub/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"
@@ -31,9 +32,7 @@
   "scripts": {
     "build": "run-p build:es5 build:esm",
     "build:es5": "tsc -p tsconfig.build.json",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
+    "build:esm": "tsc -p tsconfig.esm.json",
     "build:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",
     "clean": "rimraf dist coverage",
     "lint": "run-s lint:prettier lint:tslint",

packages/minimal/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"
@@ -31,9 +32,7 @@
   "scripts": {
     "build": "run-p build:es5 build:esm",
     "build:es5": "tsc -p tsconfig.build.json",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
+    "build:esm": "tsc -p tsconfig.esm.json",
     "build:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",
     "clean": "rimraf dist coverage",
     "lint": "run-s lint:prettier lint:tslint",

packages/node/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"

packages/types/package.json

@@ -10,6 +10,7 @@
     "node": ">=6"
   },
   "main": "dist/index.js",
+  "module": "esm/index.js",
   "types": "dist/index.d.ts",
   "publishConfig": {
     "access": "public"
@@ -24,9 +25,7 @@
   "scripts": {
     "build": "run-p build:es5 build:esm",
     "build:es5": "tsc -p tsconfig.build.json",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
+    "build:esm": "tsc -p tsconfig.esm.json",
     "build:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",
     "lint": "run-s lint:prettier lint:tslint",
     "lint:prettier": "prettier-check \"{src,test}/**/*.ts\"",

packages/utils/package.json

@@ -27,13 +27,10 @@
     "typescript": "^3.3.3"
   },
   "scripts": {
-    "build": "run-s clean && run-p build:es5 build:esm",
+    "build": "run-s clean && run-p build:es5",
     "build:es5": "tsc -p tsconfig.build.json",
-    "build:esm": "run-s build:esm:transpile build:esm:rewrite",
-    "build:esm:transpile": "tsc -p tsconfig.esm.json",
-    "build:esm:rewrite": "node ../../scripts/esm-rewrite.js",
     "build:watch": "tsc -p tsconfig.build.json -w --preserveWatchOutput",
-    "clean": "rimraf dist coverage *.js *.js.map *.d.ts",
+    "clean": "rimraf dist esm coverage *.js *.js.map *.d.ts",
     "lint": "run-s lint:prettier lint:tslint",
     "lint:prettier": "prettier-check \"{src,test}/**/*.ts\"",
     "lint:tslint": "tslint -t stylish -p .",