refactor(reactivity): simplified isProxy judgment

Author: OnlyWick

packages/reactivity/__tests__/reactive.spec.ts

@@ -1,5 +1,14 @@
 import { isRef, ref } from '../src/ref'
-import { isReactive, markRaw, reactive, toRaw } from '../src/reactive'
+import {
+  isProxy,
+  isReactive,
+  markRaw,
+  reactive,
+  readonly,
+  shallowReactive,
+  shallowReadonly,
+  toRaw,
+} from '../src/reactive'
 import { computed } from '../src/computed'
 import { effect } from '../src/effect'
 
@@ -302,4 +311,21 @@ describe('reactivity/reactive', () => {
     const observed = reactive(original)
     expect(isReactive(observed)).toBe(false)
   })
+
+  test('isProxy', () => {
+    const foo = {}
+    expect(isProxy(foo)).toBe(false)
+
+    const fooRe = reactive(foo)
+    expect(isProxy(fooRe)).toBe(true)
+
+    const fooSRe = shallowReactive(foo)
+    expect(isProxy(fooSRe)).toBe(true)
+
+    const fooRl = readonly(foo)
+    expect(isProxy(fooRl)).toBe(true)
+
+    const fooSRl = shallowReadonly(foo)
+    expect(isProxy(fooSRl)).toBe(true)
+  })
 })

packages/reactivity/src/reactive.ts

@@ -329,8 +329,9 @@ export function isShallow(value: any): boolean {
  * @param value - The value to check.
  * @see {@link https://vuejs.org/api/reactivity-utilities.html#isproxy}
  */
-export function isProxy(value: unknown): boolean {
-  return isReactive(value) || isReadonly(value)
+export function isProxy(value: any): boolean {
+  // see BaseReactiveHandler, createInstrumentationGetter
+  return value ? !!value[ReactiveFlags.RAW] : false
 }
 
 /**