fix(overmind-vue): fix last componen tracking

Author: christianalfoni

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

@@ -1,5 +1,8 @@
 import { EventType } from 'overmind'
 
+const OVERMIND = Symbol('OVERMIND')
+const IS_PRODUCTION = process.env.NODE_ENV === 'production'
+
 let nextComponentId = 0
 
 function createMixin(overmind, propsCallback) {
@@ -8,59 +11,69 @@ function createMixin(overmind, propsCallback) {
 
   return {
     beforeMount(this: any) {
-      this.__tree = (overmind as any).proxyStateTree.getTrackStateTree()
-      this.__componentInstanceId = componentInstanceId++
-      this.__onUpdate = (mutations, paths, flushId) => {
-        this.__currentFlushId = flushId
-        this.$forceUpdate()
+      this[OVERMIND] = {
+        tree: (overmind as any).proxyStateTree.getTrackStateTree(),
+        componentInstanceId: componentInstanceId++,
+        onUpdate: (mutations, paths, flushId) => {
+          this[OVERMIND].currentFlushId = flushId
+          this.$forceUpdate()
+        },
       }
       this.overmind = {
-        state: this.__tree.state,
+        state: this[OVERMIND].tree.state,
         actions: overmind.actions,
         effects: overmind.effects,
         addMutationListener: overmind.addMutationListener,
       }
+
       if (propsCallback) {
         Object.assign(
           this,
           propsCallback({
-            state: this.__tree.state,
+            state: this[OVERMIND].tree.state,
             actions: overmind.actions,
             effects: overmind.effects,
           })
         )
       }
-      this.__tree.track(this.__onUpdate)
+      this[OVERMIND].tree.track(this[OVERMIND].onUpdate)
     },
     beforeUpdate(this: any) {
-      this.__tree.track(this.__onUpdate)
-    },
-    mounted(this: any) {
-      overmind.eventHub.emitAsync(EventType.COMPONENT_ADD, {
-        componentId,
-        componentInstanceId: this.__componentInstanceId,
-        name: this.$options.name || '',
-        paths: Array.from(this.__tree.pathDependencies) as any,
-      })
-    },
-    updated(this: any) {
-      overmind.eventHub.emitAsync(EventType.COMPONENT_UPDATE, {
-        componentId,
-        componentInstanceId: this.__componentInstanceId,
-        name: this.$options.name || '',
-        flushId: this.__currentFlushId,
-        paths: Array.from(this.__tree.pathDependencies) as any,
-      })
+      this[OVERMIND].tree.track(this[OVERMIND].onUpdate)
     },
+    ...(IS_PRODUCTION
+      ? null
+      : {
+          mounted(this: any) {
+            overmind.eventHub.emitAsync(EventType.COMPONENT_ADD, {
+              componentId,
+              componentInstanceId: this[OVERMIND].componentInstanceId,
+              name: this.$options.name || '',
+              paths: Array.from(this[OVERMIND].tree.pathDependencies) as any,
+            })
+          },
+          updated(this: any) {
+            overmind.eventHub.emitAsync(EventType.COMPONENT_UPDATE, {
+              componentId,
+              componentInstanceId: this[OVERMIND].componentInstanceId,
+              name: this.$options.name || '',
+              flushId: this[OVERMIND].currentFlushId,
+              paths: Array.from(this[OVERMIND].tree.pathDependencies) as any,
+            })
+          },
+        }),
     beforeDestroy(this: any) {
+      // @ts-ignore
+      overmind.proxyStateTree.disposeTree(this[OVERMIND].tree)
+      if (IS_PRODUCTION) {
+        return
+      }
+
       overmind.eventHub.emitAsync(EventType.COMPONENT_REMOVE, {
         componentId,
-        componentInstanceId: this.__componentInstanceId,
+        componentInstanceId: this[OVERMIND].componentInstanceId,
         name: this.$options.name || '',
       })
-
-      // @ts-ignore
-      overmind.proxyStateTree.disposeTree(this.__tree)
     },
   }
 }