react-components: Remove redundant configuration (#1093)

Since for `NODE_ENV !== 'development'`:
* The `html` option is already set to `false`, meaning no html
  plugins need to be deleted.
* Production hints are already enabled by default in webpack 4, for
  `target: 'web'`
* The source map can now be configured via options (#1062)

Author: edmorley

packages/react-components/index.js

@@ -13,6 +13,9 @@ module.exports = (neutrino, opts = {}) => {
     manifest: process.env.NODE_ENV === 'development',
     externals: opts.externals !== false && {},
     style: { extract: { plugin: { filename: '[name].css' } } },
+    devtool: {
+      production: 'source-map'
+    },
     targets: { browsers: 'ie 9' }
   }, opts);
 
@@ -43,20 +46,9 @@ module.exports = (neutrino, opts = {}) => {
 
       neutrino.use(react, options);
 
-      // Remove the html generation plugins added by the react preset
-      Object
-        .keys(neutrino.options.mains)
-        .forEach(key => {
-          neutrino.config.plugins.delete(`html-${key}`);
-        });
-
       neutrino.config
         .when(options.externals, config => config.externals([nodeExternals(options.externals)]))
         .when(hasSourceMap, () => neutrino.use(banner))
-        .devtool('source-map')
-        .performance
-          .hints('error')
-          .end()
         // Disable the chunking behaviour inherited from the react preset
         .optimization
           .splitChunks(false)