Do not transpile JS packages twice

Automattic · Aug 13, 2020 · 9680f61 · 9680f61
1 parent 0f3a101
commit 9680f61
Show file tree

Hide file tree

Showing 13 changed files with 41 additions and 40 deletions.
diff --git a/packages/composite-checkout/package.json b/packages/composite-checkout/package.json
@@ -3,12 +3,12 @@
 	"version": "1.0.0",
 	"description": "A checkout component for Automattic brands",
 	"main": "dist/cjs/public-api.js",
-	"module": "dist/esm/public-api.js",
+	"module": "src/public-api.js",
 	"sideEffects": false,
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"files": [
 		"dist",

diff --git a/packages/format-currency/package.json b/packages/format-currency/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0-alpha.0",
 	"description": "JavaScript library for formatting currency",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"repository": {
 		"type": "git",
@@ -29,8 +29,8 @@
 	},
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"types": "types"
 }
diff --git a/packages/i18n-calypso/package.json b/packages/i18n-calypso/package.json
@@ -3,7 +3,7 @@
 	"version": "5.0.0",
 	"description": "i18n JavaScript library on top of Tannin originally used in Calypso",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"types": "types/index.d.ts",
 	"sideEffects": false,
 	"repository": {
@@ -38,7 +38,7 @@
 	},
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	}
 }
diff --git a/packages/load-script/package.json b/packages/load-script/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0",
 	"description": "Async Script Loader.",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"keywords": [
 		"wordpress"
 	],
@@ -28,7 +28,7 @@
 	},
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	}
 }
diff --git a/packages/photon/package.json b/packages/photon/package.json
@@ -3,7 +3,7 @@
 	"version": "3.0.0",
 	"description": "JavaScript library for the WordPress.com Photon image manipulation service",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"repository": {
 		"type": "git",
@@ -38,7 +38,7 @@
 	},
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	}
 }
diff --git a/packages/popup-monitor/package.json b/packages/popup-monitor/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0",
 	"description": "Utility to facilitate the monitoring of a popup window close action.",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"keywords": [
 		"wordpress",
@@ -29,8 +29,8 @@
 	],
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"dependencies": {
 		"lodash": "^4.17.15"

diff --git a/packages/request-external-access/package.json b/packages/request-external-access/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0",
 	"description": "Utility for requesting authorization of sharing services.",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"keywords": [
 		"wordpress",
@@ -30,8 +30,8 @@
 	],
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"dependencies": {
 		"@automattic/popup-monitor": "^1.0.0"

diff --git a/packages/social-previews/package.json b/packages/social-previews/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.2",
 	"description": "A suite of components to generate previews for a post for both social and search engines",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": [
 		"*.css",
 		"*.scss"
@@ -34,8 +34,8 @@
 	],
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile && copy-assets"
+		"build": "transpile && copy-assets",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"dependencies": {
 		"lodash": "^4.17.15",

diff --git a/packages/tree-select/package.json b/packages/tree-select/package.json
@@ -10,7 +10,7 @@
 	"homepage": "https://github.com/Automattic/wp-calypso",
 	"license": "GPL-2.0-or-later",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"repository": {
 		"type": "git",
@@ -29,7 +29,7 @@
 	],
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	}
 }
diff --git a/packages/viewport-react/package.json b/packages/viewport-react/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0",
 	"description": "React helpers for tracking viewport changes",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"repository": {
 		"type": "git",
@@ -30,8 +30,8 @@
 	},
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"publishConfig": {
 		"access": "public"

diff --git a/packages/viewport/package.json b/packages/viewport/package.json
@@ -3,7 +3,7 @@
 	"version": "1.0.0",
 	"description": "Vanilla helpers for tracking viewport changes",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"repository": {
 		"type": "git",
@@ -18,8 +18,8 @@
 	"homepage": "https://github.com/Automattic/wp-calypso/tree/HEAD/packages/viewport#readme",
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"publishConfig": {
 		"access": "public"

diff --git a/packages/wpcom-proxy-request/package.json b/packages/wpcom-proxy-request/package.json
@@ -3,7 +3,7 @@
 	"version": "6.0.0",
 	"description": "Proxied cookie-authenticated REST API requests to WordPress.com",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"sideEffects": false,
 	"keywords": [
 		"browser",
@@ -37,8 +37,8 @@
 	"types": "types",
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "npm run clean",
-		"prepare": "transpile"
+		"build": "transpile",
+		"prepack": "yarn run clean && yarn run build"
 	},
 	"dependencies": {
 		"@babel/runtime": "^7.11.1",

diff --git a/packages/wpcom.js/package.json b/packages/wpcom.js/package.json
@@ -3,13 +3,13 @@
 	"description": "Official JavaScript library for the WordPress.com REST API",
 	"version": "6.0.0",
 	"main": "dist/cjs/index.js",
-	"module": "dist/esm/index.js",
+	"module": "src/index.js",
 	"author": "Automattic, Inc.",
 	"private": true,
 	"scripts": {
 		"clean": "npx rimraf dist",
-		"prepublish": "yarn run clean",
-		"prepare": "run-s build:modules build:bundle",
+		"build": "run-s build:modules build:bundle",
+		"prepack": "yarn run clean && yarn run build",
 		"build:modules": "transpile",
 		"build:bundle": "webpack --display errors-only"
 	},
@@ -34,7 +34,8 @@
 	],
 	"browser": {
 		"./dist/cjs/lib/util/fs.js": "./dist/cjs/lib/util/fs-browser.js",
-		"./dist/esm/lib/util/fs.js": "./dist/esm/lib/util/fs-browser.js"
+		"./dist/esm/lib/util/fs.js": "./dist/esm/lib/util/fs-browser.js",
+		"./src/lib/util/fs.js": "./src/lib/util/fs-browser.js"
 	},
 	"dependencies": {
 		"@babel/runtime": "^7.11.1",