refactor: remove redux-invariant & redux-logging from production (microsoft#680)

* these middlewares are intend to be use only during development.
* also move a few packages from devDependencies to dependencies

Author: mydiemho

.codecov.yml

@@ -18,3 +18,4 @@ coverage:
   ignore:
     - "src/electron/start.js"
     - "src/providers/export/tensorFlowRecords/tensorFlowRecordsProtoBuf_pb.js"
+    - "src/redux/store/store.ts"

package-lock.json

@@ -28,7 +28,9 @@
         "rc-menu": "^7.4.21",
         "react": "^16.7.0",
         "react-dom": "^16.7.0",
+        "react-jsonschema-form": "^1.3.0",
         "react-localization": "^1.0.13",
+        "react-modal": "^3.8.1",
         "react-redux": "^5.1.1",
         "react-router-dom": "^4.3.1",
         "react-scripts": "2.1.1",
@@ -38,8 +40,6 @@
         "react-vis": "^1.11.6",
         "reactstrap": "^6.5.0",
         "redux": "^4.0.1",
-        "redux-immutable-state-invariant": "^2.1.0",
-        "redux-logger": "^3.0.6",
         "redux-thunk": "^2.3.0",
         "rimraf": "^2.6.2",
         "shortid": "^2.2.14",
@@ -106,8 +106,8 @@
         "jquery": "^3.3.1",
         "node-sass": "^4.10.0",
         "popper.js": "^1.14.6",
-        "react-jsonschema-form": "^1.0.6",
-        "react-modal": "^3.6.1",
+        "redux-immutable-state-invariant": "^2.1.0",
+        "redux-logger": "^3.0.6",
         "redux-mock-store": "^1.5.3",
         "ts-loader": "^5.3.0",
         "tslint": "^5.11.0",

package.json

@@ -1,27 +1,44 @@
 import { applyMiddleware, createStore, Store } from "redux";
-import reduxImmutableStateInvarient from "redux-immutable-state-invariant";
-import { createLogger } from "redux-logger";
 import thunk from "redux-thunk";
 import rootReducer from "../reducers";
 import { IApplicationState } from "../../models/applicationState";
-import { createLocalStorage, mergeInitialState } from "../middleware/localStorage";
+import { mergeInitialState } from "../middleware/localStorage";
+import { Env } from "../../common/environment";
 
 /**
- * Creates initial redux store from initial aplication state
+ * Creates initial redux store from initial application state
  * @param initialState - Initial state of application
  * @param useLocalStorage - Whether or not to use localStorage middleware
  */
 export default function createReduxStore(
     initialState?: IApplicationState,
     useLocalStorage: boolean = false): Store {
     const paths: string[] = ["appSettings", "connections", "recentProjects"];
-    const middlewares = useLocalStorage
-        ? applyMiddleware(thunk, reduxImmutableStateInvarient(), createLogger(), createLocalStorage({ paths }))
-        : applyMiddleware(thunk, reduxImmutableStateInvarient(), createLogger());
+
+    let middlewares = [thunk];
+
+    if (useLocalStorage) {
+        const localStorage = require("../middleware/localStorage");
+        const storage = localStorage.createLocalStorage({paths});
+        middlewares = [
+            ...middlewares,
+            storage,
+        ];
+    }
+
+    if (Env.get() === "development") {
+        const logger = require("redux-logger");
+        const reduxImmutableStateInvariant = require("redux-immutable-state-invariant");
+        middlewares = [
+            ...middlewares,
+            reduxImmutableStateInvariant.default(),
+            logger.createLogger(),
+        ];
+    }
 
     return createStore(
         rootReducer,
         useLocalStorage ? mergeInitialState(initialState, paths) : initialState,
-        middlewares,
+        applyMiddleware(...middlewares),
     );
 }