fix(build): babel cache caused DRAGGABLE_DEBUG to ship

That was frustrating to find - babel's cacheIdentifier does not include
the contents of webpack config, so changing it may cause you to ship env
variables you don't mean to ship

Author: STRML

Makefile

@@ -16,7 +16,7 @@ lint:
 	@$(BIN)/eslint lib/* lib/utils/* specs/*
 	@$(BIN)/tsc -p typings
 
-build: build-cjs build-esm build-web
+build: clean build-cjs build-esm build-web
 
 build-cjs: $(BIN)
 	$(BIN)/babel --out-dir ./build/cjs ./lib
@@ -38,7 +38,7 @@ test-phantom: $(BIN)
 	@NODE_ENV=test $(BIN)/karma start karma-phantomjs.conf.js
 
 dev: $(BIN) clean
-	$(BIN)/webpack-dev-server
+	env DRAGGABLE_DEBUG=1 $(BIN)/webpack-dev-server
 
 node_modules/.bin: install
 

webpack.config.js

@@ -49,19 +49,17 @@ module.exports = (env, argv) => ({
         test: /\.(?:js|es).?$/,
         loader: 'babel-loader',
         options: {
-          cacheDirectory: true,
+          cacheDirectory: false, // intended, have had bugs with env like DRAGGABLE_DEBUG in the past
         },
         exclude: /(node_modules)/
       }
 		]
 	},
   plugins: [
     new webpack.EnvironmentPlugin({
-      // Default values
-      DRAGGABLE_DEBUG: (process.env.DRAGGABLE_DEBUG || ''),
-      NODE_ENV: ['development', 'production'].includes(argv.mode) ? 
-        argv.mode : 
-        (process.env.NODE_ENV || 'production'),
+      // these are default values
+      DRAGGABLE_DEBUG: false,
+      NODE_ENV: ['development', 'production'].includes(argv.mode) ? argv.mode : 'production'
     }),
     // Scope hoisting
     new webpack.optimize.ModuleConcatenationPlugin(),