fix(setup): should not trigger getter w/ object computed nested (#799)

Co-authored-by: webfansplz <>

Author: webfansplz

src/mixin.ts

@@ -190,7 +190,7 @@ export function mixin(Vue: VueConstructor) {
       return true
     }
 
-    if (!isPlainObject(target) || isRaw(target)) {
+    if (!isPlainObject(target) || isRaw(target) || isRef(target)) {
       return false
     }
     return Object.keys(target).some((x) =>

test/setup.spec.js

@@ -1219,4 +1219,27 @@ describe('setup', () => {
       },
     }).$mount()
   })
+
+  // #794
+  it('should not trigger getter w/ object computed nested', () => {
+    const spy = jest.fn()
+    new Vue({
+      setup() {
+        new Vue({
+          setup() {
+            const person = {
+              name: computed(() => {
+                spy()
+                return 1
+              }),
+            }
+            return {
+              person,
+            }
+          },
+        })
+      },
+    })
+    expect(spy).toHaveBeenCalledTimes(0)
+  })
 })

test/setupContext.spec.ts

@@ -3,6 +3,7 @@ import {
   defineComponent,
   createApp,
   ref,
+  computed,
   nextTick,
   SetupContext,
   getCurrentInstance,
@@ -225,4 +226,26 @@ describe('setupContext', () => {
       `"RangeError: Maximum call stack size exceeded"`
     ).not.toHaveBeenWarned()
   })
+
+  // #794
+  it('should not trigger getter w/ object computed nested', async () => {
+    const spy = jest.fn()
+    createApp(
+      defineComponent({
+        template: `<div/>`,
+        setup() {
+          const person = {
+            name: computed<number>(() => {
+              spy()
+              return 1
+            }),
+          }
+          return {
+            person,
+          }
+        },
+      })
+    ).mount()
+    expect(spy).toHaveBeenCalledTimes(0)
+  })
 })