fix(dts): Fix build dts error (fix #520)

[0x-jerry]

Before this PR, build dts for CameraControls/MapControls/OrbitControls components will failed because of error TS7056, this should fix issue #520.

src/core/controls/OrbitControls.vue:797:1 - error TS7056: The inferred type of this node exceeds the maximum length the compiler will serialize. An explicit type annotation is needed.

797 export default (await import('vue')).defineComponent({
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
798 setup() {
    ~~~~~~~~~
... 
805 props: {} as __VLS_WithDefaults<__VLS_TypePropsToOption<__VLS_PublicProps>, typeof __VLS_withDefaultsArg>,
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
806 });

After this PR, the error TS7056 will disappear.

[netlify]

✅ Deploy Preview for cientos-tresjs ready!

Name	Link
🔨 Latest commit	225edac
🔍 Latest deploy log	https://app.netlify.com/sites/cientos-tresjs/deploys/677652e572096200082ea0e6
😎 Deploy Preview	https://deploy-preview-521--cientos-tresjs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[alvarosabu]

Hi @0x-jerry thanks for solving this, I was losing my mind with this issue never imagined that was actually a problem of using a ref instead of a shallowRef for returning the instance, it make sense, the controls are probably really complex objects with tons of nested properties that was making the the ts compiler cry a little.

OrbitControls is a stateful class that manages its own updates
We don't need reactivity for its internal properties
We're already handling updates through the change, start, and end events
It's more performant since we're not creating unnecessary reactive wrappers

Thanks