fix(overmind): fix types to avoid the 'import' function in exported types

Author: gaspard

packages/node_modules/overmind/src/index.ts

@@ -5,13 +5,18 @@ import { ProxyStateTree } from 'proxy-state-tree'
 import { createOperators } from './Action'
 import { Derived } from './derived'
 import { Devtools, Message, safeValue } from './Devtools'
-import { EventType, Events, Options } from './internalTypes'
+import {
+  EventType,
+  Events,
+  Options,
+  ResolveActions,
+  ResolveState,
+} from './internalTypes'
 import { Reaction } from './reaction'
 import {
   BaseApp,
   Configuration,
   TAction,
-  TApp,
   TDerive,
   TOperation,
   TReaction,
@@ -65,15 +70,18 @@ export const log = (...objects: any[]) =>
 
 const hotReloadingCache = {}
 
+// We do not use TApp<Config> directly to type the class in order to avoid
+// the 'import(...)' function to be used in exported types.
+
 export class Overmind<Config extends Configuration> implements BaseApp {
   private proxyStateTree: ProxyStateTree
   initialized: Promise<any>
   eventHub: EventEmitter<Events>
   devtools: Devtools
-  actions: TApp<Config>['actions']
-  state: TApp<Config>['state']
+  actions: ResolveActions<Config['actions'] & {}>
+  state: ResolveState<Config['state'] & {}>
   // We add the `effects` here so that App implements BaseApp (makes types simpler).
-  effects: TApp<Config>['effects'] & { state: TApp<Config>['state'] }
+  effects: Config['effects'] & { state: ResolveState<Config['state'] & {}> }
   constructor(configuration: Config, options: Options = {}) {
     const name = options.name || 'MyApp'
 
@@ -358,7 +366,7 @@ export class Overmind<Config extends Configuration> implements BaseApp {
     return {
       add(
         name: string,
-        stateCb: (state: TApp<Config>['state']) => any,
+        stateCb: (state: ResolveState<Config['state'] & {}>) => any,
         cb: Function
       ) {
         const reaction = new Reaction(