[@vue/reactivity] Proxied values are unwrapped during insertion into reactive sets, but not deletion

[nmsmith]

Version

3.0.0-alpha.0

Reproduction link

https://jsfiddle.net/bn08c9rL/2/

Steps to reproduce

1. Create a reactive() object.
2. Create a reactive() set.
3. add() the reactive object to the set.
4. Attempt to delete() the reactive object from the set. The item is not deleted.

What is expected?

The item should be deleted from the set.

What is actually happening?

The item is not deleted from the set.

[nmsmith]

In collectionHandlers.ts, the add() function seems to unwrap the "value" before insertion, but the deleteEntry() does not unwrap the "key" (which for sets, is also the value). The issue may be caused by this asymmetrical handling: it seems these functions are supposed to be generic over both Sets and Maps?