feat(overmind): allow async mutate operator

Author: christianalfoni

packages/node_modules/overmind-devtools/src/components/ActionOperator/index.tsx

@@ -25,6 +25,7 @@ const ActionOperator: SFC<Props> = ({
   const { state, actions } = useOvermind()
   const isExpanded = state.expandAllActionDetails || !operator.isCollapsed
 
+  console.log(operator.mutations)
   return (
     <div>
       {value === undefined ? null : (

packages/node_modules/overmind/src/index.ts

@@ -176,6 +176,7 @@ export class Overmind<Config extends Configuration> implements Configuration {
       operatorId: -1,
       path: [],
       emit: this.eventHub.emit.bind(this.eventHub),
+      send: this.devtools ? this.devtools.send.bind(this.devtools) : () => {},
     }
   }
   private createContext(value, execution, tree) {
@@ -742,13 +743,27 @@ export function mutate<Input, Config extends Configuration = TheConfig>(
     if (err) next(err)
     else {
       startDebugOperator('mutate', operation, context)
-      const mutationsTree = context.proxyStateTree.getMutationTree()
-      operation({ ...context, state: mutationsTree.state })
-      const newContext = createContext(context, context.value)
-      context.execution.emit(EventType.MUTATIONS, {
-        ...newContext.execution,
-        mutations: mutationsTree.mutations,
+      const mutationTree = context.proxyStateTree.getMutationTree()
+      if (!IS_PRODUCTION) {
+        mutationTree.onMutation((mutation) => {
+          context.execution.emit(EventType.MUTATIONS, {
+            ...context.execution,
+            operatorId: context.execution.operatorId + 1,
+            mutations: [mutation],
+          })
+        })
+      }
+      const maybePromise: any = operation({
+        ...context,
+        state: mutationTree.state,
       })
+      const newContext = createContext(
+        context,
+        maybePromise instanceof Promise
+          ? maybePromise.then(() => context.value)
+          : context.value
+      )
+
       stopDebugOperator(newContext)
       next(null, newContext)
     }

packages/node_modules/overmind/src/internalTypes.ts

@@ -1,5 +1,5 @@
 import { IMutation, ProxyStateTree } from 'proxy-state-tree'
-import { Configuration, TAction, TOperator, StateObject } from './types'
+import { Configuration, TAction, TOperator, TStateObject } from './types'
 
 export type SubType<Base, Condition> = Pick<
   Base,
@@ -115,14 +115,14 @@ export type TBaseContext<Config extends Configuration> = Config['effects'] & {
 
 type Derived = (parent: any, config: any) => any
 
-export type ResolveState<State extends StateObject> = State extends undefined
+export type ResolveState<State extends TStateObject> = State extends undefined
   ? {}
   : {
       [P in keyof State]: State[P] extends Derived
         ? ReturnType<State[P]>
         : State[P] extends Array<any>
         ? State[P]
-        : State[P] extends StateObject
+        : State[P] extends TStateObject
         ? ResolveState<State[P]>
         : State[P]
     }

packages/overmind-website/src/app/actions.ts

@@ -1,5 +1,5 @@
 import { Action, Operator, debounce, pipe } from 'overmind'
-import { Page, RouteContext, GuideParams, VideoParams, Guide } from './types'
+import { Page, RouteContext, GuideParams, VideoParams } from './types'
 import * as o from './operators'
 
 export const openHome: Action<RouteContext> = async ({ state, request }) => {

packages/overmind-website/src/app/index.ts

@@ -3,7 +3,7 @@ import state from './state'
 import onInitialize from './onInitialize'
 import * as actions from './actions'
 import * as effects from './effects'
-import { createHook, createConnect, TConnect } from 'overmind-react'
+import { createHook } from 'overmind-react'
 
 const config = {
   onInitialize,

packages/overmind-website/src/app/state.ts

@@ -25,6 +25,7 @@ type State = {
   isLoadingSearchResult: boolean
   searchResult: SearchResult[]
   showSearchResult: boolean
+  test: string
 }
 
 const state: State = {
@@ -43,6 +44,7 @@ const state: State = {
   isLoadingSearchResult: false,
   searchResult: [],
   showSearchResult: false,
+  test: '',
 }
 
 export default state