fix(overmind): ensure flushes are made from async actions in production

Author: christianalfoni

packages/node_modules/overmind/src/index.ts

@@ -156,6 +156,15 @@ export class Overmind<Config extends Configuration> implements BaseApp {
     } else {
       this.initialized = Promise.resolve(null)
     }
+
+    if (IS_PRODUCTION) {
+      let nextTick
+      const flushTree = () => proxyStateTree.flush()
+      this.proxyStateTree.addMutationListener(() => {
+        clearTimeout(nextTick)
+        nextTick = setTimeout(flushTree, 0)
+      })
+    }
   }
   private createExecution(name, action) {
     return {
@@ -204,9 +213,7 @@ export class Overmind<Config extends Configuration> implements BaseApp {
   private createAction(name, action) {
     this.actionReferences.push(action)
     return (value?) => {
-      if (IS_PRODUCTION) {
-        return action(value)
-      } else if (action[IS_PIPE]) {
+      if (IS_PRODUCTION || action[IS_PIPE]) {
         return new Promise((resolve, reject) => {
           const execution = this.createExecution(name, action)
           this.eventHub.emit(EventType.ACTION_START, execution)

packages/node_modules/proxy-state-tree/src/proxify.ts

@@ -1,5 +1,4 @@
 const isPlainObject = require('is-plain-object')
-const IS_PRODUCTION = process.env.NODE_ENV === 'production'
 
 export const IS_PROXY = Symbol('IS_PROXY')
 export const PATH = Symbol('PATH')
@@ -30,7 +29,7 @@ function shouldTrackMutations(tree, path) {
 }
 
 function ensureMutationTrackingIsEnabled(tree, path) {
-  if (!tree.status.has(STATUS.TRACKING_MUTATIONS)) {
+  if (tree.options.devmode && !tree.status.has(STATUS.TRACKING_MUTATIONS)) {
     throw new Error(
       `proxy-state-tree - You are mutating the path "${path}", but it is not allowed`
     )