fix: remove platform checks from production bundle, fixes #1162

index.js

@@ -1,21 +1,23 @@
 'use strict'
 
-var evalAllowed = false;
-try {
-  eval('evalAllowed = true');
-} catch (e) {
-  // eval not allowed due to CSP
-}
+if (!module.hot || process.env.NODE_ENV === 'production') {
+  module.exports = require('./dist/react-hot-loader.production.min.js');
+} else {
+  var evalAllowed = false;
+  try {
+    eval('evalAllowed = true');
+  } catch (e) {
+    // eval not allowed due to CSP
+  }
 
-// RHL needs setPrototypeOf to operate Component inheritance, and eval to patch methods
-var platformSupported = !!Object.setPrototypeOf && evalAllowed;
+  // RHL needs setPrototypeOf to operate Component inheritance, and eval to patch methods
+  var jsFeaturesPresent = typeof window !== 'undefined' && !!Object.setPrototypeOf;
 
-if (!module.hot || process.env.NODE_ENV === 'production' || !platformSupported) {
-  if (module.hot) {
+  if (!jsFeaturesPresent && evalAllowed) {
     // we are not in prod mode, but RHL could not be activated
-    console.warn('React-Hot-Loaded is not supported in this environment');
+    console.warn('React-Hot-Loader is not supported in this environment');
+    module.exports = require('./dist/react-hot-loader.production.min.js');
+  } else {
+    module.exports = window.reactHotLoaderGlobal = require('./dist/react-hot-loader.development.js');
   }
-  module.exports = require('./dist/react-hot-loader.production.min.js');
-} else {
-  module.exports = require('./dist/react-hot-loader.development.js');
 }

src/index.dev.js

@@ -10,4 +10,6 @@ export { enter as enterModule, leave as leaveModule } from './global/modules'
 export * from './utils.dev'
 export default ReactHotLoader
 
-ReactHotLoader.patch(React, ReactDOM)
+if (typeof React !== 'undefined' && typeof ReactDOM !== 'undefined') {
+  ReactHotLoader.patch(React, ReactDOM)
+}