BREAKING: Build packages with tsup (#3998)

## Explanation

This changes the build system to build all packages with `tsup`, instead
of `tsc`. `tsc` is still used for generating the declaration files and
type checking, and `tsup` is used for building the `.js` and (new)
`.mjs` files. The benefit of this is that we now have a ESM build as
well as a CJS build.

## References

See MetaMask/utils#144.

## Changelog

<!--
If you're making any consumer-facing changes, list those changes here as
if you were updating a changelog, using the template below as a guide.

(CATEGORY is one of BREAKING, ADDED, CHANGED, DEPRECATED, REMOVED, or
FIXED. For security-related issues, follow the Security Advisory
process.)

Please take care to name the exact pieces of the API you've added or
changed (e.g. types, interfaces, functions, or methods).

If there are any breaking changes, make sure to offer a solution for
consumers to follow once they upgrade to the changes.

Finally, if you're only making changes to development scripts or tests,
you may replace the template below with "None".
-->

### `@metamask/accounts-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/address-book-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/announcement-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/approval-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/assets-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/base-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/build-utils`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/composable-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/controller-utils`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/ens-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/eth-json-rpc-provider`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/gas-fee-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/json-rpc-engine`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/json-rpc-middleware-stream`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/keyring-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/logging-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/message-manager`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/name-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/network-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/notification-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/permission-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/permission-log-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/phishing-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/polling-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/preferences-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/queued-request-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/rate-limit-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/selected-network-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/signature-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/transaction-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

### `@metamask/user-operation-controller`

- **BREAKING**: Add ESM build.
  - It's no longer possible to import files from `./dist` directly.

## Checklist

- [ ] I've updated the test suite for new or updated code as appropriate
- [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate

Author: Mrtenz

.yarn/patches/tsup-npm-8.0.2-86e40f68a7.patch

@@ -0,0 +1,13 @@
+diff --git a/dist/index.js b/dist/index.js
+index 4500c4e43c3bbd24aa60b7d4cf95aa3fee8eb185..9c442bc216f99b7cfadb5ac62cb98d3ae9ce2f56 100644
+--- a/dist/index.js
++++ b/dist/index.js
+@@ -1813,6 +1813,8 @@ var cjsSplitting = () => {
+       }
+       const { transform: transform3 } = await Promise.resolve().then(() => require("sucrase"));
+       const result = transform3(code, {
++        // https://github.com/egoist/tsup/issues/1087
++        disableESTransforms: true,
+         filePath: info.path,
+         transforms: ["imports"],
+         sourceMapOptions: this.options.sourcemap ? {

constraints.pro

@@ -244,6 +244,27 @@ gen_enforced_field(WorkspaceCwd, 'types', './dist/index.d.ts') :-
 gen_enforced_field(WorkspaceCwd, 'types', null) :-
   workspace_field(WorkspaceCwd, 'private', true).
 
+% The exports for all published packages must be the same.
+gen_enforced_field(WorkspaceCwd, 'exports["."].import', './dist/index.mjs') :-
+  \+ workspace_field(WorkspaceCwd, 'private', true).
+gen_enforced_field(WorkspaceCwd, 'exports["."].require', './dist/index.js') :-
+  \+ workspace_field(WorkspaceCwd, 'private', true).
+gen_enforced_field(WorkspaceCwd, 'exports["."].types', './dist/types/index.d.ts') :-
+  \+ workspace_field(WorkspaceCwd, 'private', true).
+gen_enforced_field(WorkspaceCwd, 'exports["./package.json"]', './package.json') :-
+  \+ workspace_field(WorkspaceCwd, 'private', true).
+% Non-published packages must not specify exports.
+gen_enforced_field(WorkspaceCwd, 'exports', null) :-
+  workspace_field(WorkspaceCwd, 'private', true).
+
+% Published packages must not have side effects.
+gen_enforced_field(WorkspaceCwd, 'sideEffects', false) :-
+  \+ workspace_field(WorkspaceCwd, 'private', true),
+  WorkspaceCwd \= 'packages/base-controller'.
+% Non-published packages must not specify side effects.
+gen_enforced_field(WorkspaceCwd, 'sideEffects', null) :-
+  workspace_field(WorkspaceCwd, 'private', true).
+
 % The list of files included in published packages must only include files
 % generated during the build step.
 gen_enforced_field(WorkspaceCwd, 'files', ['dist/']) :-
@@ -254,6 +275,10 @@ gen_enforced_field(WorkspaceCwd, 'files', ['dist/']) :-
 gen_enforced_field(WorkspaceCwd, 'files', []) :-
   WorkspaceCwd = '.'.
 
+% All non-root packages must have the same "build" script.
+gen_enforced_field(WorkspaceCwd, 'scripts.build', 'tsup --config ../../tsup.config.ts --tsconfig ./tsconfig.build.json --clean') :-
+  WorkspaceCwd \= '.'.
+
 % All non-root packages must have the same "build:docs" script.
 gen_enforced_field(WorkspaceCwd, 'scripts.build:docs', 'typedoc') :-
   WorkspaceCwd \= '.'.

package.json

@@ -12,9 +12,11 @@
     "packages/*"
   ],
   "scripts": {
-    "build": "tsc --build tsconfig.build.json --verbose",
+    "build": "yarn run build:source && yarn run build:types",
     "build:clean": "rimraf dist '**/*.tsbuildinfo' && yarn build",
     "build:docs": "yarn workspaces foreach --parallel --interlaced --verbose run build:docs",
+    "build:source": "yarn workspaces foreach --exclude @metamask/core-monorepo --parallel --interlaced --verbose run build",
+    "build:types": "tsc --build tsconfig.build.json --verbose",
     "build:watch": "yarn run build --watch",
     "changelog:update": "yarn workspaces foreach --parallel --interlaced --verbose run changelog:update",
     "changelog:validate": "yarn workspaces foreach --parallel --interlaced --verbose run changelog:validate",
@@ -40,6 +42,9 @@
   "simple-git-hooks": {
     "pre-push": "yarn lint"
   },
+  "resolutions": {
+    "tsup@^8.0.2": "patch:tsup@npm%3A8.0.2#./.yarn/patches/tsup-npm-8.0.2-86e40f68a7.patch"
+  },
   "devDependencies": {
     "@babel/core": "^7.23.5",
     "@babel/plugin-transform-modules-commonjs": "^7.23.3",
@@ -77,9 +82,9 @@
     "nock": "^13.3.1",
     "prettier": "^2.7.1",
     "prettier-plugin-packagejson": "^2.4.5",
-    "rimraf": "^3.0.2",
     "simple-git-hooks": "^2.8.0",
     "ts-node": "^10.9.1",
+    "tsup": "^8.0.2",
     "typescript": "~4.8.4",
     "yargs": "^17.7.2"
   },
@@ -92,7 +97,8 @@
       "@lavamoat/preinstall-always-fail": false,
       "@keystonehq/bc-ur-registry-eth>hdkey>secp256k1": true,
       "babel-runtime>core-js": false,
-      "simple-git-hooks": false
+      "simple-git-hooks": false,
+      "tsup>esbuild": true
     }
   }
 }

packages/accounts-controller/package.json

@@ -15,12 +15,22 @@
     "url": "https://github.com/MetaMask/core.git"
   },
   "license": "MIT",
+  "sideEffects": false,
+  "exports": {
+    ".": {
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js",
+      "types": "./dist/types/index.d.ts"
+    },
+    "./package.json": "./package.json"
+  },
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
     "dist/"
   ],
   "scripts": {
+    "build": "tsup --config ../../tsup.config.ts --tsconfig ./tsconfig.build.json --clean",
     "build:docs": "typedoc",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/accounts-controller",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/accounts-controller",

packages/accounts-controller/tsconfig.build.json

@@ -2,7 +2,7 @@
   "extends": "../../tsconfig.packages.build.json",
   "compilerOptions": {
     "baseUrl": "./",
-    "outDir": "./dist",
+    "outDir": "./dist/types",
     "rootDir": "./src",
     "skipLibCheck": true
   },

packages/address-book-controller/package.json

@@ -15,12 +15,22 @@
     "url": "https://github.com/MetaMask/core.git"
   },
   "license": "MIT",
+  "sideEffects": false,
+  "exports": {
+    ".": {
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js",
+      "types": "./dist/types/index.d.ts"
+    },
+    "./package.json": "./package.json"
+  },
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
     "dist/"
   ],
   "scripts": {
+    "build": "tsup --config ../../tsup.config.ts --tsconfig ./tsconfig.build.json --clean",
     "build:docs": "typedoc",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/address-book-controller",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/address-book-controller",

packages/address-book-controller/tsconfig.build.json

@@ -2,7 +2,7 @@
   "extends": "../../tsconfig.packages.build.json",
   "compilerOptions": {
     "baseUrl": "./",
-    "outDir": "./dist",
+    "outDir": "./dist/types",
     "rootDir": "./src"
   },
   "references": [

packages/announcement-controller/package.json

@@ -15,12 +15,22 @@
     "url": "https://github.com/MetaMask/core.git"
   },
   "license": "MIT",
+  "sideEffects": false,
+  "exports": {
+    ".": {
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js",
+      "types": "./dist/types/index.d.ts"
+    },
+    "./package.json": "./package.json"
+  },
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
     "dist/"
   ],
   "scripts": {
+    "build": "tsup --config ../../tsup.config.ts --tsconfig ./tsconfig.build.json --clean",
     "build:docs": "typedoc",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/announcement-controller",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/announcement-controller",

packages/announcement-controller/tsconfig.build.json

@@ -2,7 +2,7 @@
   "extends": "../../tsconfig.packages.build.json",
   "compilerOptions": {
     "baseUrl": "./",
-    "outDir": "./dist",
+    "outDir": "./dist/types",
     "rootDir": "./src"
   },
   "references": [{ "path": "../base-controller/tsconfig.build.json" }],

packages/approval-controller/package.json

@@ -15,12 +15,22 @@
     "url": "https://github.com/MetaMask/core.git"
   },
   "license": "MIT",
+  "sideEffects": false,
+  "exports": {
+    ".": {
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js",
+      "types": "./dist/types/index.d.ts"
+    },
+    "./package.json": "./package.json"
+  },
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
     "dist/"
   ],
   "scripts": {
+    "build": "tsup --config ../../tsup.config.ts --tsconfig ./tsconfig.build.json --clean",
     "build:docs": "typedoc",
     "changelog:update": "../../scripts/update-changelog.sh @metamask/approval-controller",
     "changelog:validate": "../../scripts/validate-changelog.sh @metamask/approval-controller",