isWrappable(new Class()) return false

[Reinhard2019]

Describe the bug

In this issue: #19
@ryansolid Decided to only wrap plain Objects and Arrays like MobX.

But i think a class object also should be wrap.

What do you think?

Your Example Website or App

https://stackblitz.com/edit/typescript-mkiunp?file=index.ts

Steps to Reproduce the Bug or Issue

class Obj {}
console.log('new Obj():', isWrappable(new Obj()))

Expected behavior

return true

Screenshots or Videos

Platform

• OS: [e.g. macOS, Windows, Linux]
• Browser: [e.g. Chrome, Safari, Firefox]
• Version: [e.g. 91.1]

Additional context

No response

[ryansolid]

No this is by design. Is wrappable false means that it can only wrap it atomically instead of each property. We did this to avoid wrapping built in classes like Date, Regexp, HTMLElement. And things like Set and Map which could be wrapped differently depending on intent. The original issue explains how this is a much easier identification model than trying to exempt specific classes.

[Reinhard2019]

No this is by design. Is wrappable false means that it can only wrap it atomically instead of each property. We did this to avoid wrapping built in classes like Date, Regexp, HTMLElement. And things like Set and Map which could be wrapped differently depending on intent. The original issue explains how this is a much easier identification model than trying to exempt specific classes.

But when i use createStore({ obj: new Obj() })，store.obj will not be monitored.

Can you give me an escape window?

[Reinhard2019]

@ryansolid Do you think it is possible to add a symbol to solve this problem? Like this:

If you think it's ok, i will commit a pr. thx!