From 0397ef99867bd7c2640e136755c5f145df7fe02e Mon Sep 17 00:00:00 2001 From: Jovi De Croock Date: Mon, 24 Feb 2025 19:45:08 +0100 Subject: [PATCH 1/5] Remove unused deps --- babel.config.js | 2 -- package-lock.json | 34 ---------------------------------- package.json | 2 -- 3 files changed, 38 deletions(-) diff --git a/babel.config.js b/babel.config.js index 9601cdf30b..e94d79edaa 100644 --- a/babel.config.js +++ b/babel.config.js @@ -30,9 +30,7 @@ module.exports = function (api) { ] ], plugins: [ - '@babel/plugin-proposal-object-rest-spread', '@babel/plugin-transform-react-jsx', - 'babel-plugin-transform-async-to-promises', ['babel-plugin-transform-rename-properties', { rename }] ], include: ['**/src/**/*.js', '**/test/**/*.js'], diff --git a/package-lock.json b/package-lock.json index b343c1e104..ce4edf8927 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,6 @@ "@actions/github": "^6.0.0", "@actions/glob": "^0.5.0", "@babel/core": "^7.26.0", - "@babel/plugin-proposal-object-rest-spread": "^7.20.7", "@babel/plugin-transform-react-jsx": "^7.25.9", "@babel/plugin-transform-react-jsx-source": "^7.25.9", "@babel/preset-env": "^7.26.0", @@ -23,7 +22,6 @@ "@types/node": "^14.14.10", "@types/sinon": "^9.0.11", "babel-plugin-istanbul": "^7.0.0", - "babel-plugin-transform-async-to-promises": "^0.8.18", "babel-plugin-transform-rename-properties": "0.1.0", "chai": "^4.1.2", "check-export-map": "^1.3.1", @@ -588,26 +586,6 @@ "@babel/core": "^7.0.0" } }, - "node_modules/@babel/plugin-proposal-object-rest-spread": { - "version": "7.20.7", - "resolved": "https://registry.npmjs.org/@babel/plugin-proposal-object-rest-spread/-/plugin-proposal-object-rest-spread-7.20.7.tgz", - "integrity": "sha512-d2S98yCiLxDVmBmE8UjGcfPvNEUbA1U5q5WxaWFUGRzJSVAZqm5W6MbPct0jxnegUZ0niLeNX+IOzEs7wYg9Dg==", - "deprecated": "This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-object-rest-spread instead.", - "dev": true, - "dependencies": { - "@babel/compat-data": "^7.20.5", - "@babel/helper-compilation-targets": "^7.20.7", - "@babel/helper-plugin-utils": "^7.20.2", - "@babel/plugin-syntax-object-rest-spread": "^7.8.3", - "@babel/plugin-transform-parameters": "^7.20.7" - }, - "engines": { - "node": ">=6.9.0" - }, - "peerDependencies": { - "@babel/core": "^7.0.0-0" - } - }, "node_modules/@babel/plugin-proposal-private-property-in-object": { "version": "7.21.0-placeholder-for-preset-env.2", "resolved": "https://registry.npmjs.org/@babel/plugin-proposal-private-property-in-object/-/plugin-proposal-private-property-in-object-7.21.0-placeholder-for-preset-env.2.tgz", @@ -692,18 +670,6 @@ "@babel/core": "^7.0.0-0" } }, - "node_modules/@babel/plugin-syntax-object-rest-spread": { - "version": "7.8.3", - "resolved": "https://registry.npmjs.org/@babel/plugin-syntax-object-rest-spread/-/plugin-syntax-object-rest-spread-7.8.3.tgz", - "integrity": "sha512-XoqMijGZb9y3y2XskN+P1wUGiVwWZ5JmoDRwx5+3GmEplNyVM2s2Dg8ILFQm8rWM48orGy5YpI5Bl8U1y7ydlA==", - "dev": true, - "dependencies": { - "@babel/helper-plugin-utils": "^7.8.0" - }, - "peerDependencies": { - "@babel/core": "^7.0.0-0" - } - }, "node_modules/@babel/plugin-syntax-unicode-sets-regex": { "version": "7.18.6", "resolved": "https://registry.npmjs.org/@babel/plugin-syntax-unicode-sets-regex/-/plugin-syntax-unicode-sets-regex-7.18.6.tgz", diff --git a/package.json b/package.json index 8de2b94167..7ab1693f68 100644 --- a/package.json +++ b/package.json @@ -207,7 +207,6 @@ "@actions/github": "^6.0.0", "@actions/glob": "^0.5.0", "@babel/core": "^7.26.0", - "@babel/plugin-proposal-object-rest-spread": "^7.20.7", "@babel/plugin-transform-react-jsx": "^7.25.9", "@babel/plugin-transform-react-jsx-source": "^7.25.9", "@babel/preset-env": "^7.26.0", @@ -218,7 +217,6 @@ "@types/node": "^14.14.10", "@types/sinon": "^9.0.11", "babel-plugin-istanbul": "^7.0.0", - "babel-plugin-transform-async-to-promises": "^0.8.18", "babel-plugin-transform-rename-properties": "0.1.0", "chai": "^4.1.2", "check-export-map": "^1.3.1", From cea2684f4682aa8dbe8f400a3d782654f13c3a16 Mon Sep 17 00:00:00 2001 From: Jovi De Croock Date: Mon, 24 Feb 2025 19:46:36 +0100 Subject: [PATCH 2/5] Remove polyfills as well --- package.json | 1 - test/polyfills.js | 13 ------------- 2 files changed, 14 deletions(-) diff --git a/package.json b/package.json index 7ab1693f68..5093920a11 100644 --- a/package.json +++ b/package.json @@ -220,7 +220,6 @@ "babel-plugin-transform-rename-properties": "0.1.0", "chai": "^4.1.2", "check-export-map": "^1.3.1", - "core-js": "^3.38.1", "coveralls": "^3.1.1", "cross-env": "^7.0.3", "errorstacks": "^2.4.1", diff --git a/test/polyfills.js b/test/polyfills.js index c24406d74a..e2d6a0f2ad 100644 --- a/test/polyfills.js +++ b/test/polyfills.js @@ -1,16 +1,3 @@ -// ES2015 APIs used by developer tools integration -import 'core-js/es/map'; -import 'core-js/es/promise'; -import 'core-js/es/array/fill'; -import 'core-js/es/array/from'; -import 'core-js/es/array/find'; -import 'core-js/es/array/includes'; -import 'core-js/es/string/includes'; -import 'core-js/es/object/assign'; -import 'core-js/es/string/starts-with'; -import 'core-js/es/string/code-point-at'; -import 'core-js/es/string/from-code-point'; -import 'core-js/es/string/repeat'; import * as kl from 'kolorist'; // Something that's loaded before this file polyfills Symbol object. From 9929e68b32ae48d94f720da89e7deb05696aeb31 Mon Sep 17 00:00:00 2001 From: Jovi De Croock Date: Mon, 24 Feb 2025 19:47:26 +0100 Subject: [PATCH 3/5] Remove kolorist transpile --- karma.conf.js | 27 --------------------------- 1 file changed, 27 deletions(-) diff --git a/karma.conf.js b/karma.conf.js index a27e1a6fc4..258acdb9a4 100644 --- a/karma.conf.js +++ b/karma.conf.js @@ -111,33 +111,6 @@ function createEsbuildPlugin() { }; }); - // Transpile node_modules that are es2015+ to es5 for IE11 - build.onLoad({ filter: /kolorist/ }, async args => { - const contents = await fs.readFile(args.path, 'utf-8'); - - const tmp = await babel.transformAsync(contents, { - filename: args.path, - presets: [ - [ - '@babel/preset-env', - { - loose: true, - modules: false, - targets: { - browsers: ['last 2 versions', 'IE >= 11'] - } - } - ] - ] - }); - - return { - contents: tmp.code, - resolveDir: path.dirname(args.path), - loader: 'js' - }; - }); - // Apply babel pass whenever we load a .js file build.onLoad({ filter: /\.[mc]?js$/ }, async args => { const contents = await fs.readFile(args.path, 'utf-8'); From e367eae953aca08f415e11e9b1a31e90fb1cbf7e Mon Sep 17 00:00:00 2001 From: Jovi De Croock Date: Mon, 24 Feb 2025 19:51:34 +0100 Subject: [PATCH 4/5] Remove check export map --- package-lock.json | 27 --------------------------- package.json | 1 - 2 files changed, 28 deletions(-) diff --git a/package-lock.json b/package-lock.json index ce4edf8927..7f1ae460d5 100644 --- a/package-lock.json +++ b/package-lock.json @@ -24,8 +24,6 @@ "babel-plugin-istanbul": "^7.0.0", "babel-plugin-transform-rename-properties": "0.1.0", "chai": "^4.1.2", - "check-export-map": "^1.3.1", - "core-js": "^3.38.1", "coveralls": "^3.1.1", "cross-env": "^7.0.3", "errorstacks": "^2.4.1", @@ -3641,19 +3639,6 @@ "node": "*" } }, - "node_modules/check-export-map": { - "version": "1.3.1", - "resolved": "https://registry.npmjs.org/check-export-map/-/check-export-map-1.3.1.tgz", - "integrity": "sha512-HX3ZRYk8/CLBJ6kjf7B3U4RGbhcbboswAq9CVsbuEIDQ0OoRI15ms2h3z3RL7sOo6KEVgVb/BiVGw/XMdCLAJg==", - "dev": true, - "dependencies": { - "kolorist": "^1.5.0", - "mri": "^1.2.0" - }, - "bin": { - "check-export-map": "src/cli.js" - } - }, "node_modules/chokidar": { "version": "3.5.3", "resolved": "https://registry.npmjs.org/chokidar/-/chokidar-3.5.3.tgz", @@ -3858,18 +3843,6 @@ "node": ">= 0.6" } }, - "node_modules/core-js": { - "version": "3.38.1", - "resolved": "https://registry.npmjs.org/core-js/-/core-js-3.38.1.tgz", - "integrity": "sha512-OP35aUorbU3Zvlx7pjsFdu1rGNnD4pgw/CWoYzRY3t2EzoVT7shKHY1dlAy3f41cGIO7ZDPQimhGFTlEYkG/Hw==", - "dev": true, - "hasInstallScript": true, - "license": "MIT", - "funding": { - "type": "opencollective", - "url": "https://opencollective.com/core-js" - } - }, "node_modules/core-js-compat": { "version": "3.39.0", "resolved": "https://registry.npmjs.org/core-js-compat/-/core-js-compat-3.39.0.tgz", diff --git a/package.json b/package.json index 5093920a11..6410bffa45 100644 --- a/package.json +++ b/package.json @@ -219,7 +219,6 @@ "babel-plugin-istanbul": "^7.0.0", "babel-plugin-transform-rename-properties": "0.1.0", "chai": "^4.1.2", - "check-export-map": "^1.3.1", "coveralls": "^3.1.1", "cross-env": "^7.0.3", "errorstacks": "^2.4.1", From c38b497ecc2b22bd8e3d4631b7a80594b61a31b3 Mon Sep 17 00:00:00 2001 From: Jovi De Croock Date: Mon, 24 Feb 2025 19:56:00 +0100 Subject: [PATCH 5/5] Remove npm-merge-driver after https://github.com/npm/npm/commit/e27674c221dc17473f23bffa50123e49a021ae34 --- package-lock.json | 13 ------------- package.json | 3 +-- 2 files changed, 1 insertion(+), 15 deletions(-) diff --git a/package-lock.json b/package-lock.json index 7f1ae460d5..1f0244adaf 100644 --- a/package-lock.json +++ b/package-lock.json @@ -41,7 +41,6 @@ "kolorist": "^1.8.0", "microbundle": "^0.15.1", "mocha": "^11.0.0", - "npm-merge-driver-install": "^3.0.0", "npm-run-all2": "^7.0.0", "oxlint": "^0.14.0", "preact-render-to-string": "^6.5.0", @@ -7102,18 +7101,6 @@ "url": "https://github.com/sponsors/sindresorhus" } }, - "node_modules/npm-merge-driver-install": { - "version": "3.0.0", - "resolved": "https://registry.npmjs.org/npm-merge-driver-install/-/npm-merge-driver-install-3.0.0.tgz", - "integrity": "sha512-NiwTYRDEhFK1Pl15jGB6GQNU1fN4kgYM/VMZgP6n19xBY8l+ed6xcK8C0J0+86o3noL5HKgJKZ1MUuMnxm7d5w==", - "dev": true, - "bin": { - "npm-merge-driver-install": "src/install.js", - "npm-merge-driver-is-installed": "src/is-installed.js", - "npm-merge-driver-merge": "src/merge.js", - "npm-merge-driver-uninstall": "src/uninstall.js" - } - }, "node_modules/npm-normalize-package-bin": { "version": "4.0.0", "resolved": "https://registry.npmjs.org/npm-normalize-package-bin/-/npm-normalize-package-bin-4.0.0.tgz", diff --git a/package.json b/package.json index 6410bffa45..f81bb8e960 100644 --- a/package.json +++ b/package.json @@ -113,7 +113,7 @@ }, "types": "src/index.d.ts", "scripts": { - "prepare": "husky && run-s build && npm-merge-driver-install", + "prepare": "husky && run-s build", "build": "npm-run-all --parallel build:*", "build:core": "microbundle build --raw --no-generateTypes -f cjs,esm,umd", "build:core-min": "microbundle build --raw --no-generateTypes -f cjs,esm,umd,iife src/cjs.js -o dist/preact.min.js", @@ -236,7 +236,6 @@ "kolorist": "^1.8.0", "microbundle": "^0.15.1", "mocha": "^11.0.0", - "npm-merge-driver-install": "^3.0.0", "npm-run-all2": "^7.0.0", "oxlint": "^0.14.0", "preact-render-to-string": "^6.5.0",