fix(action-chain): extend void instead of undefined due to weirdness

Author: christianalfoni

packages/node_modules/action-chain/src/ActionBase.ts

@@ -26,7 +26,7 @@ export function actionBaseFactory<
   Context,
   InitialValue,
   Value = InitialValue,
-  ReturnAction = InitialValue extends undefined
+  ReturnAction = InitialValue extends void
     ? NoValueActionBase<Context, InitialValue, Value>
     : ActionBase<Context, InitialValue, Value>
 >(

packages/node_modules/overmind-devtools/src/client/app/actions.ts

@@ -24,7 +24,7 @@ export default (action: Action) => {
       .mutation(mutations.addNewApp)
       .mutation(mutations.resetNewPortValue)
       .do(helpers.storeApps),
-    changeTab: action().mutation((value: string) => {}),
+    changeTab: action<Tab>().mutation(mutations.changeTab),
     openApp: action<string>(),
   }
 }

packages/node_modules/overmind/src/createActionFactory.ts

@@ -15,17 +15,17 @@ type OperatorCallback<Context, Value, NewValue = Value> = (
 interface Operators<State, Context, InitialValue, Value> {
   mutation(
     cb: (value: Value, state: State) => any
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, Value>
     : Action<State, Context, InitialValue, Value>
   do(
     cb: (value: Value, context: Context) => void
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, Value>
     : Action<State, Context, InitialValue, Value>
   map<NewValue>(
     cb: (value: Value, context: Context) => NewValue | Promise<NewValue>
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, NewValue>
     : Action<State, Context, InitialValue, NewValue>
   try<ResolveValue, RejectValue, NewValue>(
@@ -44,7 +44,7 @@ interface Operators<State, Context, InitialValue, Value> {
         RejectValue
       >
     }
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, ResolveValue | RejectValue>
     : Action<State, Context, InitialValue, ResolveValue | RejectValue>
   when<TrueValue, FalseValue>(
@@ -53,17 +53,17 @@ interface Operators<State, Context, InitialValue, Value> {
       true: Action<State, Context, Value, TrueValue>
       false: Action<State, Context, Value, FalseValue>
     }
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, TrueValue | FalseValue>
     : Action<State, Context, InitialValue, TrueValue | FalseValue>
   filter(
     cb: (value: Value, context: Context) => boolean
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, Value>
     : Action<State, Context, InitialValue, Value>
   debounce(
     timer: number
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, Value>
     : Action<State, Context, InitialValue, Value>
 }
@@ -89,7 +89,7 @@ export default function createActionFactory<State, Context>(proxyStateTree) {
       value: any,
       executionContext: ExecutionContext
     ) => any | Promise<any>
-  ): InitialValue extends undefined
+  ): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue, Value>
     : Action<State, Context, InitialValue, Value> {
     return Object.assign(

packages/node_modules/overmind/src/index.ts

@@ -26,7 +26,7 @@ export type TContext<State, Providers = {}> = Providers & {
 }
 
 export interface IAction<State, Context> {
-  <InitialValue = undefined>(): InitialValue extends undefined
+  <InitialValue = void>(): InitialValue extends void
     ? NoValueAction<State, Context, InitialValue>
     : Action<State, Context, InitialValue>
 }
@@ -70,7 +70,7 @@ export default class App<
       State,
       Providers & { state: State }
     >(proxyStateTree)
-    const action = function<InitialValue>(): InitialValue extends undefined
+    const action = function<InitialValue>(): InitialValue extends void
       ? NoValueAction<State, Providers & { state: State }, InitialValue>
       : Action<State, Providers & { state: State }, InitialValue> {
       return actionFactory<InitialValue>(actionChain)