Don't pull in all of core-js with core (#1601)

Author: wegry

packages/core/babel.config.js

@@ -1,12 +1,5 @@
 const BABEL_ENV = process.env.BABEL_ENV;
-
-const defaultPlugins = [
-  '@babel/plugin-proposal-class-properties',
-  '@babel/plugin-proposal-object-rest-spread',
-  '@babel/plugin-proposal-optional-chaining',
-  // IE 11 support
-  '@babel/plugin-transform-object-assign'
-];
+const IS_TEST = BABEL_ENV === "test"
 
 module.exports = {
   presets: [
@@ -15,30 +8,14 @@ module.exports = {
       {
         modules: ["cjs", "test"].includes(BABEL_ENV) ? "commonjs" : false,
         targets:
-          BABEL_ENV === "test" ? { node: "current" } : { browsers: "defaults" },
+          IS_TEST ? { node: "current" } : { browsers: "defaults" },
       },
     ],
-    "@babel/preset-react",
+    "@babel/preset-react"
+  ],
+  plugins: [
+    "@babel/plugin-proposal-class-properties",
+    "@babel/plugin-proposal-optional-chaining"
   ],
-  env: {
-    cjs: {
-      plugins: defaultPlugins,
-      ignore: ['test/**/*.js']
-    },
-    umd: {
-      plugins: defaultPlugins,
-      ignore: ['test/**/*.js']
-    },
-    es: {
-      plugins: [
-        ...defaultPlugins,
-        ['@babel/plugin-transform-runtime', { useESModules: true, corejs: 2 }]
-      ],
-      ignore: ['test/**/*.js']
-    },
-    test: {
-      plugins: defaultPlugins,
-      ignore: []
-    }
-  },
+  ignore: IS_TEST ? [] : ['test/**/*.js'],
 };

packages/core/package-lock.json

@@ -42,10 +42,9 @@
     "react": ">=16"
   },
   "dependencies": {
-    "@babel/runtime-corejs2": "^7.8.7",
     "@types/json-schema": "^7.0.4",
     "ajv": "^6.7.0",
-    "core-js": "^2.5.7",
+    "core-js-pure": "^3.6.5",
     "json-schema-merge-allof": "^0.6.0",
     "jsonpointer": "^4.0.1",
     "lodash": "^4.17.15",
@@ -58,11 +57,9 @@
     "@babel/cli": "^7.4.4",
     "@babel/core": "^7.4.5",
     "@babel/plugin-proposal-class-properties": "^7.4.4",
-    "@babel/plugin-proposal-object-rest-spread": "^7.4.4",
     "@babel/plugin-proposal-optional-chaining": "^7.8.3",
     "@babel/plugin-transform-object-assign": "^7.8.3",
     "@babel/plugin-transform-react-jsx": "^7.3.0",
-    "@babel/plugin-transform-runtime": "^7.4.4",
     "@babel/preset-env": "^7.4.5",
     "@babel/preset-react": "^7.0.0",
     "@babel/register": "^7.4.4",

packages/core/package.json

@@ -1,7 +1,7 @@
 import AddButton from "../AddButton";
 import IconButton from "../IconButton";
 import React, { Component } from "react";
-import includes from "core-js/library/fn/array/includes";
+import includes from "core-js-pure/es/array/includes";
 import * as types from "../../types";
 
 import {

packages/core/src/components/fields/ArrayField.js

@@ -1,7 +1,7 @@
 import React from "react";
 import * as ReactIs from "react-is";
 import mergeAllOf from "json-schema-merge-allof";
-import fill from "core-js/library/fn/array/fill";
+import fill from "core-js-pure/es/array/fill";
 import validateFormData, { isValid } from "./validate";
 import union from "lodash/union";
 import jsonpointer from "jsonpointer";

packages/core/src/utils.js

@@ -25,32 +25,17 @@ module.exports = {
     })
   ],
   devtool: "source-map",
-  externals: {
-    react: {
-      root: "React",
-      commonjs: "react",
-      commonjs2: "react",
-      amd: "react"
-    },
-    'react-dom': {
-      root: "ReactDOM",
-      commonjs2: 'react-dom',
-      commonjs: 'react-dom',
-      amd: 'react-dom',
-      umd: 'react-dom',
-    }
-  },
+  externals: [
+    'react',
+    'react-dom'
+  ],
   module: {
     rules: [
       {
         test: /\.js$/,
         use: [
           "babel-loader",
         ],
-        exclude: [
-          path.join(__dirname, "node_modules", "core-js"),
-          path.join(__dirname, "node_modules", "babel-runtime"),
-        ],
       },
     ]
   }