Don't use babel-loader's cacheDirectory for production

Author: Lyrkan

lib/loaders/babel.js

@@ -20,9 +20,12 @@ module.exports = {
     getLoaders(webpackConfig) {
         let babelConfig = {
             // improves performance by caching babel compiles
-            // we add this option ALWAYS
+            // this option is always added but is set to FALSE in
+            // production to avoid cache invalidation issues caused
+            // by some Babel presets/plugins (for instance the ones
+            // that use browserslist)
             // https://github.com/babel/babel-loader#options
-            cacheDirectory: true
+            cacheDirectory: !webpackConfig.isProduction()
         };
 
         // configure babel (unless the user is specifying .babelrc)

test/loaders/babel.js

@@ -49,6 +49,18 @@ describe('loaders/babel', () => {
         });
     });
 
+    it('getLoaders() for production', () => {
+        const config = createConfig();
+        config.runtimeConfig.babelRcFileExists = true;
+        config.runtimeConfig.environment = 'production';
+
+        const actualLoaders = babelLoader.getLoaders(config);
+        // cacheDirectory is disabled in production mode
+        expect(actualLoaders[0].options).to.deep.equal({
+            cacheDirectory: false
+        });
+    });
+
     it('getLoaders() with react', () => {
         const config = createConfig();
         config.enableReactPreset();