Using reactive and shallowReactive on the same target returns the same proxy for both calls.

[lijingfaatcumt]

Version

3.0.4

Reproduction link

https://jsfiddle.net/vqkho2xt/2/

Steps to reproduce

const origin = {};
const shallowProxy = shallowReactive(origin);
const reactiveProxy = reactive(origin);
console.log(shallowProxy === reactiveProxy);

What is expected?

shallowProxy !== reactiveProxy

What is actually happening?

shallowProxy === reactiveProxy

---

why reactive and shallowReactive should return diffrent object ?
may there is the situation: we maybe want to get two different Proxy with the same origin object, the proxy named reactiveProxy returned by reactive , the proxy named shallowProxy returned by shallowReactive, we change the origin objects deep property by reactiveProxy can trigger denpendencies of the deep propery, but we change the origin objects deep property by shallowProxy should not trigger denpendencies of the deep propery, in this situation, the implements of the vue3 can not satisfy because of that shallowReactive and reactive return the same proxy with the same origin object.

[edison1105]

IMO, this will make shallowProxy and reactiveProxy confused. Developers do not know how to choose which API to use.

[lijingfaatcumt]

IMO, this will make shallowProxy and reactiveProxy confused. Developers do not know how to choose which API to use.

I do not think so, because i think 'shallow' can show the difference, i think we should use 4 weakMap (reactiveMap, shallowReactiveMap, readolyMap, shallowReadonlyMap) to store the map relationship of the origin object to the proxy instead of the 2 maps(reactiveMap and readonlyMap)

[posva]

I think that changing the order of creation of shallowReactive and reactive shouldn't change the outcome of this: https://jsfiddle.net/u8ga1pcb/

[LinusBorg]

We collect all reactive instances in one WeakMap, that ignores what type of reactive they are:

https://github.com/vuejs/vue-next/blob/b81d279e40c3d32a3740ea20d7e182690294f117/packages/reactivity/src/reactive.ts#L22-L30

This likely affects readonly vs. shallowReadonly as well. Meaning we would in fact need 4 Weakmaps to track each type individually

[lijingfaatcumt]

We collect all reactive instances in one WeakMap, that ignores what type of reactive they are:

https://github.com/vuejs/vue-next/blob/b81d279e40c3d32a3740ea20d7e182690294f117/packages/reactivity/src/reactive.ts#L22-L30

This likely affects readonly vs. shallowReadonly as well. Meaning we would in fact need 4 Weakmaps to track each type individually

the actions of reactive and shallowReactive are different? why we ignore the difference?

[lijingfaatcumt]

We collect all reactive instances in one WeakMap, that ignores what type of reactive they are:
https://github.com/vuejs/vue-next/blob/b81d279e40c3d32a3740ea20d7e182690294f117/packages/reactivity/src/reactive.ts#L22-L30

This likely affects readonly vs. shallowReadonly as well. Meaning we would in fact need 4 Weakmaps to track each type individually

the actions of reactive and shallowReactive are different? why we ignore the difference?

i have the origin object, first i reactive origin and get reactive, second i shallowReactive the origin, but i get the reactive, can not get the shallowReactive, and i do not know the result is the reactive, this is confused,because of this, cause the bug #2846

[LinusBorg]

i have the origin object, first i reactive origin and get reactive, second i shallowReactive the origin, but i get the reactive, can not get the shallowReactive, and i do not know the result is the reactive, this is confused,because of this, cause the bug

yes. This is why we marked it as a bug.

I will close your othr issues as a duplicate

[LinusBorg]

PR submitted, happy to get reviews.