fix(overmind-react): detect unmounted component and force SSR when running in Node

Author: christianalfoni

packages/node_modules/overmind-devtools-vscode/package.json

@@ -3,7 +3,7 @@
   "displayName": "Overmind Devtools VSCode",
   "description": "Devtools for Overmind",
   "publisher": "christianalfoni",
-  "version": "0.0.16",
+  "version": "0.0.17",
   "repository": {
     "type": "git",
     "url": "https://github.com/cerebral/overmind.git"

packages/node_modules/overmind-react/src/index.ts

@@ -24,6 +24,9 @@ import {
 } from 'react'
 
 const IS_PRODUCTION = process.env.NODE_ENV === 'production'
+const IS_TEST = process.env.NODE_ENV === 'test'
+const isNode =
+  !IS_TEST && process && process.title && process.title.includes('node')
 
 export type IReactComponent<P = any> =
   | StatelessComponent<P>
@@ -81,8 +84,19 @@ export const createHook = <Config extends IConfiguration>(
   }
   const useForceRerender = () => {
     const [, setState] = useState(() => true)
+    const mountedRef = useRef(true)
+
+    useEffect(
+      () => () => {
+        mountedRef.current = false
+      },
+      []
+    )
+
     const forceRerender = (_, __, flushId): void => {
-      setState(flushId)
+      if (mountedRef.current) {
+        setState(flushId)
+      }
     }
     return forceRerender
   }
@@ -98,22 +112,14 @@ export const createHook = <Config extends IConfiguration>(
       )
     }
 
-    if ((overmind as any).mode.mode === MODE_SSR) {
+    if (isNode || (overmind as any).mode.mode === MODE_SSR) {
       return {
         state: overmind.state,
         actions: overmind.actions,
         effects: overmind.effects,
         addMutationListener: overmind.addMutationListener,
       }
     }
-
-    const component = useCurrentComponent()
-    const name = component.name
-    component.__componentId =
-      typeof component.__componentId === 'undefined'
-        ? nextComponentId++
-        : component.__componentId
-
     const { current: tree } = useRef<any>(
       (overmind as any).proxyStateTree.getTrackStateTree()
     )
@@ -132,6 +138,13 @@ export const createHook = <Config extends IConfiguration>(
 
       useLayoutEffect(() => tree.stopTracking())
     } else {
+      const component = useCurrentComponent()
+      const name = component.name
+      component.__componentId =
+        typeof component.__componentId === 'undefined'
+          ? nextComponentId++
+          : component.__componentId
+
       const { current: componentInstanceId } = useRef<any>(
         currentComponentInstanceId++
       )